The 12/31/2016 16:28, Martin Mares wrote: > Hi! > > > This adds support for decoding Atomic ops added in ECN > > https://pcisig.com/sites/default/files/specification_documents/ECN_Atomic_Ops_080417.pdf > > This is much better, however, it still prints extra spaces sometimes. > Could you please fix it? > Can you let me know in which situation would extra spaces be printed? Do you want me to get rid of spaces between the fields ? The outputs look like this Root complex: DevCap2: Completion Timeout: Range BCD, TimeoutDis+, LTR-, OBFF Not Supported ARIFwd+ AtomicOpsCap: Routing- 32bit+ 64bit+ 128bitCAS+ DevCtl2: Completion Timeout: 65ms to 210ms, TimeoutDis-, LTR-, OBFF Disabled ARIFwd- AtomicOpsCtl: ReqEn- EgressBlck- LnkCtl2: Target Link Speed: 8GT/s, EnterCompliance- SpeedDis- endpoint with memory bar: DevCap2: Completion Timeout: Not Supported, TimeoutDis+, LTR-, OBFF Not Supported AtomicOpsCap: 32bit- 64bit- 128bitCAS- DevCtl2: Completion Timeout: 50us to 50ms, TimeoutDis-, LTR-, OBFF Disabled AtomicOpsCtl: ReqEn- LnkCtl2: Target Link Speed: 8GT/s, EnterCompliance- SpeedDis- upstream/downstream port: DevCap2: Completion Timeout: Not Supported, TimeoutDis-, LTR-, OBFF Not Supported AtomicOpsCap: Routing- DevCtl2: Completion Timeout: 50us to 50ms, TimeoutDis-, LTR-, OBFF Disabled AtomicOpsCtl: EgressBlck- LnkCtl2: Target Link Speed: 2.5GT/s, EnterCompliance- SpeedDis- > Also, I woule like to have a test case for the new code -- a configuration > space dump similar to those in tests/*. Do you have one? Sure.I can provide this as an updated patch. > > Martin -- Regards Satanand -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
