From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with archive (Exim 4.43) id 1KDiGf-0000Yx-QN for mharc-grub-devel@gnu.org; Tue, 01 Jul 2008 11:59:45 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1KDiGe-0000Ys-Rl for grub-devel@gnu.org; Tue, 01 Jul 2008 11:59:44 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1KDiGd-0000Ye-Ci for grub-devel@gnu.org; Tue, 01 Jul 2008 11:59:44 -0400 Received: from [199.232.76.173] (port=60481 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1KDiGd-0000Yb-Av for grub-devel@gnu.org; Tue, 01 Jul 2008 11:59:43 -0400 Received: from aybabtu.com ([69.60.117.155]:43288) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1KDiGc-0005mj-Nu for grub-devel@gnu.org; Tue, 01 Jul 2008 11:59:43 -0400 Received: from [192.168.10.10] (helo=thorin) by aybabtu.com with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.69) (envelope-from ) id 1KDiCs-0003aF-Ov for grub-devel@gnu.org; Tue, 01 Jul 2008 17:55:51 +0200 Received: from rmh by thorin with local (Exim 4.63) (envelope-from ) id 1KDiG2-0001sq-CO for grub-devel@gnu.org; Tue, 01 Jul 2008 17:59:06 +0200 Date: Tue, 1 Jul 2008 17:59:06 +0200 From: Robert Millan To: The development of GRUB 2 Message-ID: <20080701155906.GC6985@thorin> References: <20080619133147.GA5018@thorin> <485AA89A.5030404@isaac.cedarswampstudios.org> <20080621141907.GC24465@thorin> <1214058907.10018.39.camel@localhost> <20080626140836.GB7511@thorin> <1214493651.24653.10.camel@localhost> <20080629112241.GA2347@thorin> <1214846014.17554.6.camel@dv> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1214846014.17554.6.camel@dv> Organization: free as in freedom X-Message-Flag: Worried about Outlook viruses? Switch to Thunderbird! www.mozilla.com/thunderbird X-Debbugs-No-Ack: true User-Agent: Mutt/1.5.13 (2006-08-11) X-detected-kernel: by monty-python.gnu.org: Genre and OS details not recognized. Subject: Re: [PATCH] add a counter in grub_dprintf X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: The development of GRUB 2 List-Id: The development of GRUB 2 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 01 Jul 2008 15:59:45 -0000 On Mon, Jun 30, 2008 at 01:13:34PM -0400, Pavel Roskin wrote: > On Sun, 2008-06-29 at 13:22 +0200, Robert Millan wrote: > > On Thu, Jun 26, 2008 at 05:20:51PM +0200, Javier Martín wrote: > > > > Whoops, sorry, I meant that grub_mkimage would create two core.img > > > images, one per kernel.img version, and then grub_install would copy > > > them as usual. Then grub_setup would embed/set up the normal core.img > > > or the debug version based on a switch. What I propose is basically > > > modifying grub_mkimage to allow choosing the kernel.img file used and > > > then adding some lines to grub_install so that the two versions of > > > core.img are generated (but only the selected is embedded). > > > > Ah; then it sounds fine, I guess. > > Sorry for entering this discussion so late. I can tell from my > experience that most debug code is only useful to debug once specific > problem. Once the problem is fixed, the debug code is not needed. If > another problem is found, then some new debug code should be written to > debug that problem. > > Generic debug code may be useful to get information from users if only > they can reproduce the problem. But even then, if may be better to add > a few lines to test certain assumptions rather than try to glean useful > pieces of information from a long generic log. Ironicaly, this was also true for my original purpose of measuring performance during boot. The standard dprintf calls ate so much time (because of the serial terminal) that they didn't provide any useful output of the time spent doing something else. In the end, I had to add custom calls to print time in the specific places where I wanted them. > Adding a complicated mechanism for having debug and non-debug images > looks like an overkill to me. It could create more problems than it > would fix. But then, what do you suggest? That we remove debug support altogether? -- Robert Millan I know my rights; I want my phone call! What good is a phone call… if you are unable to speak? (as seen on /.)