From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 7637 invoked by alias); 13 Oct 2011 16:13:16 -0000 Received: (qmail 7623 invoked by uid 22791); 13 Oct 2011 16:13:14 -0000 X-SWARE-Spam-Status: No, hits=0.7 required=5.0 tests=AWL,BAYES_50,RCVD_IN_DNSWL_NONE,RCVD_IN_SORBS_WEB,SPF_SOFTFAIL X-Spam-Check-By: sourceware.org Received: from mtaout23.012.net.il (HELO mtaout23.012.net.il) (80.179.55.175) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 13 Oct 2011 16:12:53 +0000 Received: from conversion-daemon.a-mtaout23.012.net.il by a-mtaout23.012.net.il (HyperSendmail v2007.08) id <0LT000F00I86Y000@a-mtaout23.012.net.il> for gdb-patches@sourceware.org; Thu, 13 Oct 2011 18:12:41 +0200 (IST) Received: from HOME-C4E4A596F7 ([84.229.17.178]) by a-mtaout23.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0LT000FGNID2WB60@a-mtaout23.012.net.il>; Thu, 13 Oct 2011 18:12:39 +0200 (IST) Date: Thu, 13 Oct 2011 16:13:00 -0000 From: Eli Zaretskii Subject: Re: [PATCH] Add the "-info-os" command to MI In-reply-to: <4E970915.8000207@codesourcery.com> To: Kwok Cheung Yeung Cc: gdb-patches@sourceware.org Reply-to: Eli Zaretskii Message-id: <83wrc827lz.fsf@gnu.org> References: <4E970915.8000207@codesourcery.com> X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2011-10/txt/msg00402.txt.bz2 > Date: Thu, 13 Oct 2011 16:51:49 +0100 > From: Kwok Cheung Yeung > > This patch adds the "-info-os" command to MI as the counterpart to the GDB "info > os" command. It behaves in the same way as "info os", only the output is > formatted according to MI conventions. I've also added an entry in the > documentation. Thanks. > On @sc{gnu}/Linux, the following values of INFOTYPE are valid: > > -@anchor{linux info os infotypes} > +@anchor{Linux OS information types} I don't see these lines in today's CVS version of gdb.texinfo. Does your patch depend on some other patch? Anyway, why is "info os processes" suddenly treated as a Linux-specific command? "info os" is supposed to be platform-agnostic; if it is a Linux-only feature, it will be more appropriate to call it "info linux", like "info dos" and "info w32" that we already have. > +The types of information available depend on the target operating > +system. The available types for @sc{gnu}/Linux are listed in > +@ref{Linux OS information types}. As I said above, the referenced section is not Linux-specific, and the CVS version of the manual does not have any details there. So something (perhaps in my brain) is missing. > +@smallexample > +(gdb) > +-info-os > +^done,OSDataTable=@{nr_rows="9",nr_cols="2",hdr=[@{width="10",alignment= > +"-1",col_name="col0",colhdr="Type"@},@{width="10",alignment="-1",col_nam > +e="col1",colhdr="Description"@}],body=[item=@{col0="processes",col1="Lis > +ting of all processes"@},item=@{col0="procgroups",col1="Listing of all p > +rocess groups"@},item=@{col0="threads",col1="Listing of all threads"@},i > +tem=@{col0="files",col1="Listing of all file descriptors"@},item=@{col0= > +"sockets",col1="Listing of all internet-domain sockets"@},item=@{col0="s > +hm",col1="Listing of all shared-memory regions"@},item=@{col0="semaphore > +s",col1="Listing of all semaphores"@},item=@{col0="msg",col1="Listing of > + all message queues"@},item=@{col0="modules",col1="Listing of all loaded > + kernel modules"@}]@} The lines here are broken in arbitrary places, which will look illegible in the manual. Please reformat the example so that the output could be readable by humans. Thanks.