From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from galois.linutronix.de ([2001:470:1f0b:1c35:abcd:42:0:1]) by merlin.infradead.org with esmtps (Exim 4.76 #1 (Red Hat Linux)) id 1RkYFh-000073-E9 for linux-mtd@lists.infradead.org; Tue, 10 Jan 2012 09:44:23 +0000 Message-ID: <4F0C086A.5070608@linutronix.de> Date: Tue, 10 Jan 2012 10:44:10 +0100 From: Sebastian Andrzej Siewior MIME-Version: 1.0 To: Brian Norris Subject: Re: [PATCH v3 0/6] NAND BBM + BBT updates References: <1326140612-26323-1-git-send-email-computersforpeace@gmail.com> In-Reply-To: <1326140612-26323-1-git-send-email-computersforpeace@gmail.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Dan Carpenter , Kulikov Vasiliy , Nicolas Ferre , Dominik Brodowski , Peter Wippich , Gabor Juhos , linux-mtd@lists.infradead.org, Jonas Gorski , Jamie Iles , Ivan Djelic , Robert Jarzmik , David Woodhouse , Maxim Levitsky , Dmitry Eremin-Solenikov , Kevin Cernekee , Barry Song <21cnbao@gmail.com>, Jim Quinlan , Andres Salomon , Axel Lin , Anatolij Gustschin , Mike Frysinger , Arnd Bergmann , Lei Wen , Sascha Hauer , Artem Bityutskiy , Florian Fainelli , Artem Bityutskiy , Adrian Hunter , Matthieu CASTET , Kyungmin Park , Shmulik Ladkani , Wolfram Sang , Chuanxiao Dong , Joe Perches , Guillaume LECERF , Roman Tereshonkov List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 01/09/2012 09:23 PM, Brian Norris wrote: > This patch series is an update to a previous patch (that has split into > a few patches) with a few additional patches at the end. The important > segments of this series involve the default steps for marking new bad > blocks when using a flash-based BBT. The new default behavior will write > to the BBT as well as attempting to write a BBM to the OOB area of the > bad block. See the patch descriptions for details. Why do we update BBT and OOB and have the date in two places? One Argument was that the boot loader may not have support for BBT and uses OOB instead. If so, why not update the boot loader and make sure both users (OS and boot loader) use the same data? Any other other arguments why updating OOB is a good idea? > The first patch, regarding NAND_NO_WRITE_OOB, is a first attempt at So now the old-default behavior requires a flag. If I remember correctly the OLPC used a different BBT layout and OOB was used for some other purpose. I remember that we had a controller which wrote ECC into OOB on its own and the driver could not write into OOB in ECC mode. But then I don't known if this was simply not implemented in the driver. So never mind. > satisfying Sebastian's concerns that some systems utilize the entire OOB > area for ECC, and so we need an option to prevent writing markers to > OOB. My attempt to prevent other OOB writes may be misguided, > incomplete, flawed in some other way, or some combination of the three. > Please provide constructive criticism. and I am still not convinced that it is a good idea to provide one information in two places. It seems to be redundant. If there are other people supporting this, I am not in your way. Sebastian