From e55794ab00daa7b3bdc786718b22bff7f35f1ac5 Mon Sep 17 00:00:00 2001 From: Nikita Proshkin Date: Wed, 22 May 2024 19:06:34 +0300 Subject: [PATCH] pcilmr: Update usage and man: new arguments format and grading Signed-off-by: Nikita Proshkin --- lmr/margin_args.c | 39 ++++--------- pcilmr.man | 138 ++++++++++++++++++++++++++++++++++++---------- 2 files changed, 120 insertions(+), 57 deletions(-) diff --git a/lmr/margin_args.c b/lmr/margin_args.c index 8a6345f..57a1d0a 100644 --- a/lmr/margin_args.c +++ b/lmr/margin_args.c @@ -16,44 +16,25 @@ const char *usage = "! Utility requires preliminary preparation of the system. Refer to the pcilmr man page !\n\n" - "Usage:\n" - "pcilmr [--margin] [] ...\n" - "pcilmr --full []\n" + "Brief usage (see man for all options):\n" + "pcilmr [--margin] [] [] [ [] ...]\n" + "pcilmr --full []\n" "pcilmr --scan\n\n" - "Device Specifier:\n" + "You can specify Downstream or Upstream Port of the Link.\nPort 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" - "-l [,...]\tSpecify lanes for margining. Default: all link lanes.\n" - "\t\t\tRemember that Device may use Lane Reversal for Lane numbering.\n" - "\t\t\tHowever, utility uses logical lane numbers in arguments and for logging.\n" - "\t\t\tUtility will automatically determine Lane Reversal and tune its calls.\n" - "-e \t\tSpecify Error Count Limit for margining. Default: 4.\n" + "Margining options (see man for all options):\n\n" + "Common (for all specified links) options:\n" + "-c\t\t\tPrint Device Lane Margining Capabilities only. Do not run margining.\n\n" + "Link specific options:\n" "-r [,...]\tSpecify Receivers to select margining targets.\n" "\t\t\tDefault: all available Receivers (including Retimers).\n" - "-p \tSpecify number of lanes to margin simultaneously.\n" - "\t\t\tDefault: 1.\n" - "\t\t\tAccording to spec it's possible for Receiver to margin up\n" - "\t\t\tto MaxLanes + 1 lanes simultaneously, but usually this works\n" - "\t\t\tbad, so this option is for experiments mostly.\n" - "-T\t\t\tTime Margining will continue until the Error Count is no more\n" - "\t\t\tthan an Error Count Limit. Use this option to find Link limit.\n" - "-V\t\t\tSame as -T option, but for Voltage.\n" "-t \t\tSpecify maximum number of steps for Time Margining.\n" - "-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" - "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"; + "-v \t\tSpecify maximum number of steps for Voltage Margining.\n"; static struct pci_dev * dev_for_filter(struct pci_access *pacc, char *filter) diff --git a/pcilmr.man b/pcilmr.man index 673262f..3f4140c 100644 --- a/pcilmr.man +++ b/pcilmr.man @@ -4,10 +4,10 @@ pcilmr \- margin PCIe Links .SH SYNOPSIS .B pcilmr .RB [ "--margin" ] -.RI [ "" ] " ..." +.RI [ "" ] " " [ "" "] [" " " [ "" ] " ..." ] .br .B pcilmr --full -.RI [ "" ] +.RI [ "" ] .br .B pcilmr --scan .SH CONFIGURATION @@ -64,26 +64,81 @@ Utility allows to get an approximation of the eye margin diagram in the form of links without a hardware debugger and without the need to stop the target system. Utility can be useful to debug link issues due to receiver margins. -However, the utility results may be not particularly accurate and, as it was found out during -testing, specific devices provide rather dubious capability support and the reliability of -the information they provide is questionable. The PCIe specification provides reference values -for the eye diagram, which are also used by the +.B pcilmr +requires root privileges (to access Extended Configuration Space), but during our testing +there were no problems with the devices and they successfully returned to their normal initial +state after the end of testing. + +.SH RESULTS GRADING +The PCIe specification provides reference values for the eye diagram, which are also used by the .B pcilmr to evaluate the results, but it seems that it makes sense to contact the manufacturer of a particular device for references. -The PCIe Base Specification Revision 5.0 sets allowed range for Timing Margin from 20%\~UI to 50%\~UI and -for Voltage Margin from 50\~mV to 500\~mV. Utility uses 30%\~UI as the recommended -value for Timing - taken from NVIDIA presentation ("PCIe 4.0 Mass Electrical Margins Data -Collection"). +The utility uses values set in PCIe Base Spec Rev. 5.0 Section 8.4.2 as the default eye width and height +minimum references. Recommended values were taken from +the PCIe Architecture PHY Test Spec Rev 5.0 (Transmitter Electrical Compliance). + +Reference grading values currently used by the utility are presented in the table below: + +.TS +box tab(:); +C | Cb S | Cb S +C | Cb | Cb | Cb | Cb +Lb | C | C | C | C. +\&:16 GT/s (Gen 4):32 GT/s (Gen 5) +\&:EW:EH:EW:EH +_ +Min:T{ +18.75 ps +.br +30% UI +T}:15 mV:T{ +9.375 ps +.br +30% UI +T}:15 mV +_ +Rec:T{ +23.75 ps +.br +38% UI +T}:21 mV:T{ +10.157 ps +.br +33% UI +T}:19.75 mV +.TE .B pcilmr -requires root privileges (to access Extended Configuration Space), but during our testing -there were no problems with the devices and they successfully returned to their normal initial -state after the end of testing. +uses full eye width and height values to grade lanes. However, it is possible that +device supports only one side margining. In such cases by default utility will +calculate EW or EH as a double one side result. + +If info for specific device is available, you can configure grading criteria +and tweak utility behavior in one-side only cases for your device using +.I -g +link specific option (see below). + +.SH HARDWARE QUIRKS SUPPORT + +Thanks to testing or directly from the manufacturer's documentation, we know that +some devices require special treatment during the margining. +Utility detects such devices based on their Vendor ID - Device ID pair. +Right now the list of special devices is hardcoded in +.I margin_hw +file. For such devices utility can automatically adjust port margining parameters +or grading options. + +For example, for Ice Lake CPUs RC ports +.B pcilmr +will change device MaxVoltageOffset value and will force the use of +.RI ' "one side is the whole" "' grading mode." .SH OPTIONS .SS Device Specifier +.B "You can specify Downstream or Upstream Port of the Link." +.TP .B "" \t .RI [ "" :] : . (see @@ -102,13 +157,29 @@ option) Links in the system (one by one). Scan for Links with negotiated speed 16 GT/s or higher. Mark "Ready" those of them in which at least one of the Link sides have Margining Ready bit set meaning that these Links are ready for testing and you can run utility on them. -.SS Margining Test options -.TP +.SS Margining Common (for all specified links) options .B -c Print Device Lane Margining Capabilities only. Do not run margining. .TP +.BI -e " " +Specify Error Count Limit for margining. +.br +Default: 4. +.TP +.BI -o " " +Save margining results in csv form into the specified directory. Utility +will generate file with the name in form of +.RI "\[dq]lmr_" "" "_Rx" # _ ".csv\[dq]" +for each successfully tested receiver. +.TP +.BI -d "