From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with archive (Exim 4.43) id 1JSImr-0007XB-Ud for mharc-grub-devel@gnu.org; Thu, 21 Feb 2008 16:17:02 -0500 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1JSImq-0007Vd-82 for grub-devel@gnu.org; Thu, 21 Feb 2008 16:17:00 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1JSImp-0007VD-HK for grub-devel@gnu.org; Thu, 21 Feb 2008 16:16:59 -0500 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1JSImp-0007V3-4r for grub-devel@gnu.org; Thu, 21 Feb 2008 16:16:59 -0500 Received: from ns39764.ovh.net ([91.121.25.85] helo=nexedi.com) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1JSImo-0008N8-U4 for grub-devel@gnu.org; Thu, 21 Feb 2008 16:16:59 -0500 Received: from [10.8.0.46] (unknown [10.8.0.46]) by nexedi.com (Postfix) with ESMTP id 2A0DC3D92A for ; Thu, 21 Feb 2008 22:25:00 +0100 (CET) From: "Yoshinori K. Okuji" Organization: enbug.org To: The development of GRUB 2 Date: Thu, 21 Feb 2008 22:16:56 +0100 User-Agent: KMail/1.9.4 References: <200802202237.23419.okuji@enbug.org> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200802212216.56602.okuji@enbug.org> X-detected-kernel: by monty-python.gnu.org: Linux 2.6 (newer, 3) Subject: Re: [PATCH] Journal support for ext2 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, 21 Feb 2008 21:17:00 -0000 On Thursday 21 February 2008 04:12, Bean wrote: > On Thu, Feb 21, 2008 at 5:37 AM, Yoshinori K. Okuji wrote: > > On Monday 18 February 2008 00:25, Bean wrote: > > > Some adjustment in this patch: as the get_block function is only used > > > when journal type is GRUB_FSHELP_JOURNAL_TYPE_FILE, i move it to > > > grub_fshelp_journal instead of using it as a parameter for > > > grub_fshelp_map_block. also add changelog for reiserfs.c. > > > > Thanks for your effort. However, I don't like one part. It is about > > types. Although I understand why you did this way, it is not a good thing > > that you use "int" for block addressing. This is a fundamental issue in > > fshelp. Because of this, filesystem support is not 64-bit safe. > > > > I don't urge you to fix everything, but please don't increase the mess. > > Ok, perhaps i can change all the block type to grub_uint64_t while i'm > fixing this ? Great. :) Okuji