From owner-freebsd-hackers@FreeBSD.ORG Mon Dec 8 17:00:18 2014 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id A5F2766D; Mon, 8 Dec 2014 17:00:18 +0000 (UTC) Received: from mailout05.t-online.de (mailout05.t-online.de [194.25.134.82]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "mailout00.t-online.de", Issuer "TeleSec ServerPass DE-1" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 65E4FD8F; Mon, 8 Dec 2014 17:00:18 +0000 (UTC) Received: from fwd27.aul.t-online.de (fwd27.aul.t-online.de [172.20.26.132]) by mailout05.t-online.de (Postfix) with SMTP id 442CB5C4233; Mon, 8 Dec 2014 18:00:10 +0100 (CET) Received: from [192.168.119.11] (GW0+x-ZC8hL9woRE6nB1RewrIJq6BNXdqElO2R7LXAJaJOU6c7oe4k6j2e9Aq3BwrC@[84.154.99.91]) by fwd27.t-online.de with (TLSv1.2:ECDHE-RSA-AES256-SHA encrypted) esmtp id 1Xy1fM-0upLeK0; Mon, 8 Dec 2014 18:00:08 +0100 Message-ID: <5485D915.5060609@freebsd.org> Date: Mon, 08 Dec 2014 18:00:05 +0100 From: Stefan Esser User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 MIME-Version: 1.0 To: "freebsd-hackers@freebsd.org" Subject: [Patch] updated: Add JSON and XML output to pciconf (libxo support - D1206) References: <133375.35515.bm@smtp125.mail.ir2.yahoo.com> In-Reply-To: <133375.35515.bm@smtp125.mail.ir2.yahoo.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-ID: GW0+x-ZC8hL9woRE6nB1RewrIJq6BNXdqElO2R7LXAJaJOU6c7oe4k6j2e9Aq3BwrC X-TOI-MSGID: 9ba29516-ed72-44aa-8c96-8e31f3a29034 Cc: Phil Shafer X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 08 Dec 2014 17:00:18 -0000 Am 08.12.2014 um 14:39 schrieb se@freebsd.org: >> >From se@FreeBSD.org Mon Dec 8 14:34:01 2014 > Return-Path: > X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on [...] Sorry for the malformed message, I have no idea why the mail header became part of the message body. (The message was sent as a "bounce" by "mutt", and that used to work for decades ...) Content of this message should just have been my update on the patch that adds XO support to pciconf and which is available for review as D1206 on reviews.freebsd.org (https://reviews.freebsd.org/D1206). Regards, STefan And here is just the relevant text from that mal-formatted message: > Update after review by Phil Shafer and extend to cover errors and capabilities > > Summary: > > This version contains many fixes to issues pointed out by Phil and > a number of further enhancements: > - Use of XO lists and instances to structure the information > - Descriptive identifiers instead of abbreviations > - Addition of XO support to PCI/PCIe error reporting > - Addition of XO support to PCI/PCIe capability information printing > > I do not have access to a PCI or PCI-Express Spec. with information on > the capabilities and their specified names, but have only used publicly > available information to select XML/JSON labels. A pointer to a freely > available specification of PCI/PCIe capabilities (just the path that > might help chose good names for data fields) would be highly appreciated. > > Test Plan: > > This version of pciconf should generate 100% identical output, except > when one of the structured XO formats is requested. > > Please test with: > pciconf -lbecv > > I do not know whether the labels used for XML amd JSON output of PCI/PCIe > capabilities are well chosen. Several names are very long, but I wanted to > avoid cryptic abbreviations. Suggestions for better or more correct names > are welcome. > > I could not test output of Intel or AMD (Hyper Transport) specific > capabilities (my Intel system uses SATA AHCI, but I do not see that in > the reported capabilities; I do not have any FreeBSD systems on AMD > hardware, currently). > > I'm adding a few reviewers (jhb because he did the initial commit of cap.c > and kib because of the HT specific information that I cannot test). I hope > you don't mind being selected as potential reviewers.