public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
From: Vitaly Wool <vwool@ru.mvista.com>
To: linux-mtd@lists.infradead.org
Subject: [RFC] the way to proceed with OOB data
Date: Thu, 15 Dec 2005 13:41:16 +0300	[thread overview]
Message-ID: <43A1484C.5030300@ru.mvista.com> (raw)

Hi folks,

you may have noticed the RFC patches for OOB handling rework I've posted 
to this list about two weeks ago.
The main idea of those was a) to provide uniform method for handling OOB 
by such MTD users like JFFS2/YAFFS2 b) to hide the OOB internal 
structure from the users. However, people talking to me on that convined 
me that those changes were too radical for the moment and also pointed 
out the following problems that arise if my changes are accepted:
1. No more legacy support for jffs/yaffs OOB layouts. This is the point 
I don't consider to be major, since if something's deprecated it means 
that it'd go away at some point, so maybe now is the time? But maybe 
most of you guys think differently.
2. No more ability to read raw OOB data from the userspace which might 
turn to be a serious problem while debugging stuff. I must admit this is 
something we should preserve.
3. No binary compatibility for MTD utilities. Here I'm also willing to 
admit being too radical.

On the other hand, it's really necessary to provide means for  kernel 
MTD users not to mess with the oobinfos.
So, the question is: how to do it then? I see several ways of doing that.
1. Change the API for nand_read_ecc and nand_write_ecc, namely add OOB 
read/write length parameter. This is what Charles proposes; it's nice 
for MTD users but it will overcomplicate nand_read_ecc/nand_write_ecc, 
which are complicated enough at the moment.
2. Change the API for nand_read_oob/nand_write_oob so that it read/wrote 
either raw OOB or free OOB. This way has been discouraged by Artem 
during our discussion in #mtd, but this might be an option.
3. Leave everything as is and add a new functionto the MTD interface 
that will read/write only free OOB bytes as a single chunk of data.

Each of the options implies the need to extend ioctl interface to get 
oobavail/oob free data.

Any comments are welcome.

Vitaly

             reply	other threads:[~2005-12-15 10:40 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-12-15 10:41 Vitaly Wool [this message]
2005-12-15 22:44 ` [RFC] the way to proceed with OOB data Charles Manning
2005-12-16  9:31   ` Vitaly Wool
2005-12-16 19:53     ` Charles Manning
2005-12-16 22:24       ` Thomas Gleixner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=43A1484C.5030300@ru.mvista.com \
    --to=vwool@ru.mvista.com \
    --cc=linux-mtd@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox