From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 8873 invoked by alias); 27 Dec 2011 21:25:19 -0000 Received: (qmail 8863 invoked by uid 22791); 27 Dec 2011 21:25:18 -0000 X-SWARE-Spam-Status: No, hits=-2.7 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from sibelius.xs4all.nl (HELO glazunov.sibelius.xs4all.nl) (83.163.83.176) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 27 Dec 2011 21:25:05 +0000 Received: from glazunov.sibelius.xs4all.nl (kettenis@localhost [127.0.0.1]) by glazunov.sibelius.xs4all.nl (8.14.5/8.14.3) with ESMTP id pBRLOg1o001614; Tue, 27 Dec 2011 22:24:42 +0100 (CET) Received: (from kettenis@localhost) by glazunov.sibelius.xs4all.nl (8.14.5/8.14.3/Submit) id pBRLOdtk012930; Tue, 27 Dec 2011 22:24:39 +0100 (CET) Date: Tue, 27 Dec 2011 21:30:00 -0000 Message-Id: <201112272124.pBRLOdtk012930@glazunov.sibelius.xs4all.nl> From: Mark Kettenis To: eliz@gnu.org CC: stanshebs@earthlink.net, gdb-patches@sourceware.org In-reply-to: (message from Eli Zaretskii on Tue, 27 Dec 2011 01:29:19 -0500) Subject: Re: [PATCH] Add extra 'info os' information types for Linux (trunk and 7.4) References: <4E95DC58.7030805@codesourcery.com> <4ECD3496.1070609@codesourcery.com> <4EF9497B.9020501@earthlink.net> 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-12/txt/msg00849.txt.bz2 > Date: Tue, 27 Dec 2011 01:29:19 -0500 > From: Eli Zaretskii > > > Date: Mon, 26 Dec 2011 20:28:43 -0800 > > From: Stan Shebs > > > > Here is a third revision of the 'info os' additions for Linux; it rolls > > up Kwok's original patch plus requested edits, plus a few more comments > > and tweaks. I plan to commit this in a day or so, if there are no > > objections. > > I already voiced an objection the first time: I think Linux-specific > OS information doesn't belong to "info os", which should be for > commands generally available on all supported systems. I would > support an "info linux" command for what you want here. I have largely ignored this discussion so far. The operating system of my choice comes with a fairly complete and consistent set of tools to gather information from the OS, and I feel GDB should not try to reimplement functionality that already exists. But I realize that things are a little bit different on Linux, especially when debugging "embedded" systems remotely. There is a clear need for this sort of additional information about the system the process is running on, given the fact that diffs adding this kind of functionality keep popping up. Eli, I don't think your objection makes a lot of sense. The "info os" is a generic command for displaying "osdata" that's made available by the backend. It is for the backend to decide what information is made available. This diff just adds a bit more "osdata" to the Linux native backend and the Linux gdbserver remote backend. I do think though that the way Stan's diff changes the documentation of the "info os" command isn't quite right. The generic description of the "info os" command should come first, followed by documentation of the specific "osdata" types for the backends that are part of GDB. I am a bit worried though that the "info proc" stuff that was recently discussed seems to overlap with the "info os" stuff implemented here. I also feel that new functionality like this doesn't really belong on the 7.4 branch.