Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754278AbYH0CN5 (ORCPT ); Tue, 26 Aug 2008 22:13:57 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753182AbYH0CNs (ORCPT ); Tue, 26 Aug 2008 22:13:48 -0400 Received: from smtp1.linux-foundation.org ([140.211.169.13]:34596 "EHLO smtp1.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752192AbYH0CNr (ORCPT ); Tue, 26 Aug 2008 22:13:47 -0400 Date: Tue, 26 Aug 2008 19:08:35 -0700 From: Andrew Morton To: "Tim Anderson" Cc: , , "'David Woodhouse'" , , "'Bruce Leonard'" Subject: Re: [PATCH 2/2][MTD] Add support for > 2GiB MTD devices Message-Id: <20080826190835.802cd676.akpm@linux-foundation.org> In-Reply-To: <008601c907e5$e1bac9f0$9101320a@mvista.com> References: <008601c907e5$e1bac9f0$9101320a@mvista.com> X-Mailer: Sylpheed 2.4.8 (GTK+ 2.12.5; x86_64-redhat-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3033 Lines: 85 On Tue, 26 Aug 2008 18:40:26 -0700 "Tim Anderson" wrote: > > -----Original Message----- > > From: linux-mtd-bounces@lists.infradead.org > > [mailto:linux-mtd-bounces@lists.infradead.org] On Behalf Of > > Bruce_Leonard@selinc.com > > Sent: Tuesday, August 26, 2008 6:21 PM > > To: Andrew Morton > > Cc: linux-mtd-bounces@lists.infradead.org; > > linux-mtd@lists.infradead.org; David Woodhouse; > > linux-kernel@vger.kernel.org; Bruce Leonard > > Subject: Re: [PATCH 2/2][MTD] Add support for > 2GiB MTD devices > > > > linux-mtd-bounces@lists.infradead.org wrote on 08/26/2008 04:55:36 PM: > > > > > On Thu, 21 Aug 2008 19:00:55 -0700 (GMT-07:00) > > > Bruce Leonard wrote: > > > > > > > --- a/include/mtd/mtd-abi.h > > > > +++ b/include/mtd/mtd-abi.h > > > > @@ -6,7 +6,7 @@ > > > > #define __MTD_ABI_H__ > > > > > > > > struct erase_info_user { > > > > - uint32_t start; > > > > + uint64_t start; > > > > uint32_t length; > > > > }; > > > > > > > > @@ -50,7 +50,7 @@ struct mtd_oob_buf { > > > > struct mtd_info_user { > > > > uint8_t type; > > > > uint32_t flags; > > > > - uint32_t size; // Total size of the MTD > > > > + uint64_t size; // Total size of the MTD > > > > uint32_t erasesize; > > > > uint32_t writesize; > > > > uint32_t oobsize; // Amount of OOB data per block (e.g. 16) > > > > > > This changes the kernel<->userspace ABI and is hence a big no-no. I > > > assume that this change will cause old userspace to > > malfunction on new > > > kernels, and vice versa. > > > > > > > Well, in my posting I noted that the mtd-utils were broken > > because of this > > but I didn't really have any idea as to how to fix things. I > > can see why > > it would be a big no-no to change this. Do you have any > > suggestions on > > what I could do differently to prevent making that change? > > > > > Supporting >2Gb MTD devices sounds useful (I'm surprised > > that we don't > > > already do so). > > > > > > > There was a LOT of interest in this over the last few months > > while I was > > working on it, but a very suprising silence has developed > > since I posted > > the patches. I guess I'm more cutting edge than I thought :). > > (top-posting repaired. Please don't.) > Bruce, > > Since you don't want to change the ABI you need to extend it. Andrew, should > he add an new ioctl that passes a new structure definition? That way the > original code works and you have to do a new ioctl to get the new size data. > Yes, I expect that a new ioctl would be best. Sometimes it is possible to extend an existing one (and its associated payload) but that requires versioning info which usually wasn't thought of on day one. But David's the man. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/