From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with archive (Exim 4.43) id 1M4fUX-0003yv-0d for mharc-grub-devel@gnu.org; Thu, 14 May 2009 14:17:13 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1M4fUU-0003wa-JZ for grub-devel@gnu.org; Thu, 14 May 2009 14:17:10 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1M4fUP-0003r2-L2 for grub-devel@gnu.org; Thu, 14 May 2009 14:17:09 -0400 Received: from [199.232.76.173] (port=37756 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1M4fUP-0003qg-DU for grub-devel@gnu.org; Thu, 14 May 2009 14:17:05 -0400 Received: from c60.cesmail.net ([216.154.195.49]:40863) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_ARCFOUR_SHA1:16) (Exim 4.60) (envelope-from ) id 1M4fUP-0004Ak-3m for grub-devel@gnu.org; Thu, 14 May 2009 14:17:05 -0400 Received: from unknown (HELO smtprelay2.cesmail.net) ([192.168.1.112]) by c60.cesmail.net with ESMTP; 14 May 2009 14:17:04 -0400 Received: from [192.168.0.22] (static-72-92-88-10.phlapa.fios.verizon.net [72.92.88.10]) by smtprelay2.cesmail.net (Postfix) with ESMTPSA id 9545734C6A for ; Thu, 14 May 2009 14:17:23 -0400 (EDT) From: Pavel Roskin To: The development of GRUB 2 In-Reply-To: References: <49E3E0FB.1070907@verizon.net> <1241409431.26483.33.camel@mj> <1241635288.4535.6.camel@accesodirecto.casa> <1241875621.22837.24.camel@accesodirecto.casa> <1241883761.22837.97.camel@accesodirecto.casa> <1242265918.21129.27.camel@mj> <1242309828.12546.15.camel@mj> Content-Type: text/plain Date: Thu, 14 May 2009 14:17:02 -0400 Message-Id: <1242325022.6056.7.camel@mj> Mime-Version: 1.0 X-Mailer: Evolution 2.26.1 (2.26.1-2.fc11) Content-Transfer-Encoding: 7bit X-detected-operating-system: by monty-python.gnu.org: Genre and OS details not recognized. Subject: Re: status grub2 port of grub-legasy map command 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: Thu, 14 May 2009 18:17:10 -0000 On Thu, 2009-05-14 at 17:01 +0200, Vladimir 'phcoder' Serbinenko wrote: > On Thu, May 14, 2009 at 4:03 PM, Pavel Roskin wrote: > > I'm sorry, I didn't realize you were opposed to the patch. I assumed > > that you just wanted to make some improvements. Nobody was against the > > drivemap command in principle. The discussion was about minor details. > Ok. Sorry for my harsh reaction. Now I'm under a lot of stress. No problem. > > I'm fine with the change from "const void" to "const char", but we need > > to remove a preceding comment about void labels. > It's not that I'm opposed to void in principle. Just using the same > constructions to do the same things in different files makes code > easier to learn and port I wonder if we can go the other way and use void for all labels without storage. Indeed, it's too easy to misuse a char variable by forgetting the ampersand before it. > > As for the parse_biosdisk() change, I'd like to see an explanation. > The explanation is that if user uses ata or usbms code and code calls > biosdisk, BIOS may issue a command which may conflict with ata/usbms. > Unfortunately it's not a scenario we're able to circumvent (BIOS is > headache) so I prefer to err on a safe side I agree that we should avoid touching the hardware. Besides, after loading ata we may not see some drives that BIOS can see. Validation of user input is good, but only if it's implemented correctly. Another approach may be to use biosdisk calls only if biosdisk is active. Otherwise, trust the user and turn off validation. -- Regards, Pavel Roskin