X-Git-Url: http://mj.ucw.cz/gitweb/?a=blobdiff_plain;f=pcilmr.c;h=cb8bd77a04fded0a3d8c3c811aa040d538778717;hb=c5cf60d7c979bc9393e72968cc1504754f016c51;hp=3634c9770fb390f8d213dda024d961289bbb45f1;hpb=ce123c881d762646a9762394fee661248bd8fc24;p=pciutils.git diff --git a/pcilmr.c b/pcilmr.c index 3634c97..cb8bd77 100644 --- a/pcilmr.c +++ b/pcilmr.c @@ -8,20 +8,29 @@ * SPDX-License-Identifier: GPL-2.0-or-later */ -#include #include #include #include +#include #include "lmr/lmr.h" const char program_name[] = "pcilmr"; +enum mode { MARGIN, FULL, SCAN }; + static const char usage_msg[] - = "Usage:\n" - "pcilmr [] \n\n" + = "! Utility requires preliminary preparation of the system. Refer to the pcilmr man page !\n\n" + "Usage:\n" + "pcilmr [--margin] [] ...\n" + "pcilmr --full []\n" + "pcilmr --scan\n\n" "Device Specifier:\n" ":\t[:]:.\n\n" + "Modes:\n" + "--margin\t\tMargin selected Links\n" + "--full\t\t\tMargin all ready for testing Links in the system (one by one)\n" + "--scan\t\t\tScan for Links available for margining\n\n" "Margining options:\n\n" "Margining Test settings:\n" "-c\t\t\tPrint Device Lane Margining Capabilities only. Do not run margining.\n" @@ -44,16 +53,19 @@ static const char usage_msg[] "-v \t\tSpecify maximum number of steps for Voltage Margining.\n" "Use only one of -T/-t options at the same time (same for -V/-v).\n" "Without these options utility will use MaxSteps from Device\n" - "capabilities as test limit.\n\n"; + "capabilities as test limit.\n\n" + "Margining Log settings:\n" + "-o \t\tSave margining results in csv form into the\n" + "\t\t\tspecified directory. Utility will generate file with the\n" + "\t\t\tname in form of 'lmr__Rx#_.csv'\n" + "\t\t\tfor each successfully tested receiver.\n"; static struct pci_dev * dev_for_filter(struct pci_access *pacc, char *filter) { struct pci_filter pci_filter; - char dev[17] = { 0 }; - strncpy(dev, filter, sizeof(dev) - 1); pci_filter_init(pacc, &pci_filter); - if (pci_filter_parse_slot(&pci_filter, dev)) + if (pci_filter_parse_slot(&pci_filter, filter)) die("Invalid device ID: %s\n", filter); if (pci_filter.bus == -1 || pci_filter.slot == -1 || pci_filter.func == -1) @@ -100,33 +112,98 @@ parse_csv_arg(char *arg, u8 *vals) return cnt; } +static void +scan_links(struct pci_access *pacc, bool only_ready) +{ + if (only_ready) + printf("Links ready for margining:\n"); + else + printf("Links with Lane Margining at the Receiver capabilities:\n"); + bool flag = true; + for (struct pci_dev *up = pacc->devices; up; up = up->next) + { + if (pci_find_cap(up, PCI_EXT_CAP_ID_LMR, PCI_CAP_EXTENDED)) + { + struct pci_dev *down = find_down_port_for_up(pacc, up); + + if (down && margin_verify_link(down, up)) + { + margin_log_bdfs(down, up); + if (!only_ready && (margin_check_ready_bit(down) || margin_check_ready_bit(up))) + printf(" - Ready"); + printf("\n"); + flag = false; + } + } + } + if (flag) + printf("Links not found or you don't have enough privileges.\n"); + pci_cleanup(pacc); + exit(0); +} + +static u8 +find_ready_links(struct pci_access *pacc, struct pci_dev **down_ports, struct pci_dev **up_ports, + bool cnt_only) +{ + u8 cnt = 0; + for (struct pci_dev *up = pacc->devices; up; up = up->next) + { + if (pci_find_cap(up, PCI_EXT_CAP_ID_LMR, PCI_CAP_EXTENDED)) + { + struct pci_dev *down = find_down_port_for_up(pacc, up); + + if (down && margin_verify_link(down, up) + && (margin_check_ready_bit(down) || margin_check_ready_bit(up))) + { + if (!cnt_only) + { + up_ports[cnt] = up; + down_ports[cnt] = down; + } + cnt++; + } + } + } + return cnt; +} + int main(int argc, char **argv) { struct pci_access *pacc; - struct pci_dev *up_port; - struct pci_dev *down_port; + struct pci_dev **up_ports; + struct pci_dev **down_ports; + u8 ports_n = 0; - struct margin_link link; + struct margin_link *links; + bool *checks_status_ports; bool status = true; + enum mode mode; - struct margin_results *results; - u8 results_n; + /* each link has several receivers -> several results */ + struct margin_results **results; + u8 *results_n; - struct margin_args args; + struct margin_args *args; u8 steps_t_arg = 0; u8 steps_v_arg = 0; u8 parallel_lanes_arg = 1; u8 error_limit = 4; + u8 lanes_arg[32]; + u8 recvs_arg[6]; u8 lanes_n = 0; u8 recvs_n = 0; bool run_margin = true; + char *dir_for_csv = NULL; + bool save_csv = false; + u64 total_steps = 0; pacc = pci_alloc(); @@ -145,9 +222,38 @@ main(int argc, char **argv) margin_global_logging = true; + struct option long_options[] + = { { .name = "margin", .has_arg = no_argument, .flag = NULL, .val = 0 }, + { .name = "scan", .has_arg = no_argument, .flag = NULL, .val = 1 }, + { .name = "full", .has_arg = no_argument, .flag = NULL, .val = 2 }, + { 0, 0, 0, 0 } }; + int c; + c = getopt_long(argc, argv, ":", long_options, NULL); + + switch (c) + { + case -1: /* no options (strings like component are possible) */ + /* FALLTHROUGH */ + case 0: + mode = MARGIN; + break; + case 1: + mode = SCAN; + if (optind == argc) + scan_links(pacc, false); + optind--; + break; + case 2: + mode = FULL; + break; + default: /* unknown option symbol */ + mode = MARGIN; + optind--; + break; + } - while ((c = getopt(argc, argv, ":r:e:l:cp:t:v:VT")) != -1) + while ((c = getopt(argc, argv, ":r:e:l:cp:t:v:VTo:")) != -1) { switch (c) { @@ -170,20 +276,26 @@ main(int argc, char **argv) run_margin = false; break; case 'l': - lanes_n = parse_csv_arg(optarg, args.lanes); + lanes_n = parse_csv_arg(optarg, lanes_arg); break; case 'e': error_limit = atoi(optarg); break; case 'r': - recvs_n = parse_csv_arg(optarg, args.recvs); + recvs_n = parse_csv_arg(optarg, recvs_arg); + break; + case 'o': + dir_for_csv = optarg; + save_csv = true; break; default: die("Invalid arguments\n\n%s", usage_msg); } } - if (optind != argc - 1) + if (mode == FULL && optind != argc) + status = false; + if (mode == MARGIN && optind == argc) status = false; if (!status && argc > 1) die("Invalid arguments\n\n%s", usage_msg); @@ -193,59 +305,91 @@ main(int argc, char **argv) exit(0); } - up_port = dev_for_filter(pacc, argv[argc - 1]); + if (mode == FULL) + { + ports_n = find_ready_links(pacc, NULL, NULL, true); + if (ports_n == 0) + { + die("Links not found or you don't have enough privileges.\n"); + } + else + { + up_ports = xmalloc(ports_n * sizeof(*up_ports)); + down_ports = xmalloc(ports_n * sizeof(*down_ports)); + find_ready_links(pacc, down_ports, up_ports, false); + } + } + else if (mode == MARGIN) + { + ports_n = argc - optind; + up_ports = xmalloc(ports_n * sizeof(*up_ports)); + down_ports = xmalloc(ports_n * sizeof(*down_ports)); - down_port = find_down_port_for_up(pacc, up_port); - if (!down_port) - die("Cannot find Upstream Component for the specified device: %s\n", argv[argc - 1]); + u8 cnt = 0; + while (optind != argc) + { + up_ports[cnt] = dev_for_filter(pacc, argv[optind]); + down_ports[cnt] = find_down_port_for_up(pacc, up_ports[cnt]); + if (!down_ports[cnt]) + die("Cannot find Upstream Component for the specified device: %s\n", argv[optind]); + cnt++; + optind++; + } + } + else + die("Bug in the args parsing!\n"); - if (!pci_find_cap(up_port, PCI_CAP_ID_EXP, PCI_CAP_NORMAL)) + if (!pci_find_cap(up_ports[0], PCI_CAP_ID_EXP, PCI_CAP_NORMAL)) die("Looks like you don't have enough privileges to access " "Device Configuration Space.\nTry to run utility as root.\n"); - if (!margin_fill_link(down_port, up_port, &link)) - { - printf("Link "); - margin_log_bdfs(down_port, up_port); - printf(" is not ready for margining.\n" - "Link data rate must be 16 GT/s or 32 GT/s.\n" - "Downstream Component must be at D0 PM state.\n"); - status = false; - } + results = xmalloc(ports_n * sizeof(*results)); + results_n = xmalloc(ports_n * sizeof(*results_n)); + links = xmalloc(ports_n * sizeof(*links)); + checks_status_ports = xmalloc(ports_n * sizeof(*checks_status_ports)); + args = xmalloc(ports_n * sizeof(*args)); - if (status) + for (int i = 0; i < ports_n; i++) { - args.error_limit = error_limit; - args.lanes_n = lanes_n; - args.recvs_n = recvs_n; - args.steps_t = steps_t_arg; - args.steps_v = steps_v_arg; - args.parallel_lanes = parallel_lanes_arg; - args.run_margin = run_margin; - args.verbosity = 1; - args.steps_utility = &total_steps; + args[i].error_limit = error_limit; + args[i].parallel_lanes = parallel_lanes_arg; + args[i].run_margin = run_margin; + args[i].verbosity = 1; + args[i].steps_t = steps_t_arg; + args[i].steps_v = steps_v_arg; + for (int j = 0; j < recvs_n; j++) + args[i].recvs[j] = recvs_arg[j]; + args[i].recvs_n = recvs_n; + for (int j = 0; j < lanes_n; j++) + args[i].lanes[j] = lanes_arg[j]; + args[i].lanes_n = lanes_n; + args[i].steps_utility = &total_steps; enum margin_test_status args_status; - if ((args_status = margin_process_args(&link.down_port, &args)) != MARGIN_TEST_OK) + if (!margin_fill_link(down_ports[i], up_ports[i], &links[i])) { - status = false; - margin_log_link(&link); - if (args_status == MARGIN_TEST_ARGS_RECVS) - margin_log("\nInvalid RecNums specified.\n"); - else if (args_status == MARGIN_TEST_ARGS_LANES) - margin_log("\nInvalid lanes specified.\n"); + checks_status_ports[i] = false; + results[i] = xmalloc(sizeof(*results[i])); + results[i]->test_status = MARGIN_TEST_PREREQS; + continue; } - } - if (status) - { + if ((args_status = margin_process_args(&links[i].down_port, &args[i])) != MARGIN_TEST_OK) + { + checks_status_ports[i] = false; + results[i] = xmalloc(sizeof(*results[i])); + results[i]->test_status = args_status; + continue; + } + + checks_status_ports[i] = true; struct margin_params params; - for (int i = 0; i < args.recvs_n; i++) + for (int j = 0; j < args[i].recvs_n; j++) { - if (margin_read_params(pacc, args.recvs[i] == 6 ? up_port : down_port, args.recvs[i], - ¶ms)) + if (margin_read_params(pacc, args[i].recvs[j] == 6 ? up_ports[i] : down_ports[i], + args[i].recvs[j], ¶ms)) { u8 steps_t = steps_t_arg ? steps_t_arg : params.timing_steps; u8 steps_v = steps_v_arg ? steps_v_arg : params.volt_steps; @@ -253,7 +397,7 @@ main(int argc, char **argv) parallel_lanes_arg; u8 step_multiplier - = args.lanes_n / parallel_recv + ((args.lanes_n % parallel_recv) > 0); + = args[i].lanes_n / parallel_recv + ((args[i].lanes_n % parallel_recv) > 0); total_steps += steps_t * step_multiplier; if (params.ind_left_right_tim) @@ -266,13 +410,40 @@ main(int argc, char **argv) } } } + } - results = margin_test_link(&link, &args, &results_n); + for (int i = 0; i < ports_n; i++) + { + if (checks_status_ports[i]) + results[i] = margin_test_link(&links[i], &args[i], &results_n[i]); + else + { + results_n[i] = 1; + if (results[i]->test_status == MARGIN_TEST_PREREQS) + { + printf("Link "); + margin_log_bdfs(down_ports[i], up_ports[i]); + printf(" is not ready for margining.\n" + "Link data rate must be 16 GT/s or 32 GT/s.\n" + "Downstream Component must be at D0 PM state.\n"); + } + else if (results[i]->test_status == MARGIN_TEST_ARGS_RECVS) + { + margin_log_link(&links[i]); + printf("\nInvalid RecNums specified.\n"); + } + else if (results[i]->test_status == MARGIN_TEST_ARGS_LANES) + { + margin_log_link(&links[i]); + printf("\nInvalid lanes specified.\n"); + } + } + printf("\n----\n\n"); } - if (status && run_margin) + if (run_margin) { - printf("\nResults:\n"); + printf("Results:\n"); printf("\nPass/fail criteria:\nTiming:\n"); printf("Minimum Offset (spec): %d %% UI\nRecommended Offset: %d %% UI\n", MARGIN_TIM_MIN, MARGIN_TIM_RECOMMEND); @@ -283,11 +454,27 @@ main(int argc, char **argv) printf("THR -\tThe set (using the utility options) \n\tstep threshold has been reached\n\n"); printf("Notations:\nst - steps\n\n"); - margin_results_print_brief(results, results_n); + for (int i = 0; i < ports_n; i++) + { + printf("Link "); + margin_log_bdfs(down_ports[i], up_ports[i]); + printf(":\n\n"); + margin_results_print_brief(results[i], results_n[i]); + if (save_csv) + margin_results_save_csv(results[i], results_n[i], dir_for_csv, up_ports[i]); + printf("\n"); + } } - if (status) - margin_free_results(results, results_n); + for (int i = 0; i < ports_n; i++) + margin_free_results(results[i], results_n[i]); + free(results_n); + free(results); + free(up_ports); + free(down_ports); + free(links); + free(checks_status_ports); + free(args); pci_cleanup(pacc); return 0;