linux.git/drivers/mtd/mtd_blkdevs.c, branch v2.6.32

block: use normal I/O path for discard requests

2009-10-01T19:19:30+00:00

prepare_discard_fn() was being called in a place where memory allocation
was effectively impossible.  This makes it inappropriate for all but
the most trivial translations of Linux's DISCARD operation to the block
command set.  Additionally adding a payload there makes the ownership
of the bio backing unclear as it's now allocated by the device driver
and not the submitter as usual.

It is replaced with QUEUE_FLAG_DISCARD which is used to indicate whether
the queue supports discard operations or not.  blkdev_issue_discard now
allocates a one-page, sector-length payload which is the right thing
for the common ATA and SCSI implementations.

The mtd implementation of prepare_discard_fn() is replaced with simply
checking for the request being a discard.

Largely based on a previous patch from Matthew Wilcox 
which did the prepare_discard_fn but not the different payload allocation
yet.

Signed-off-by: Christoph Hellwig 
Signed-off-by: Jens Axboe

const: make block_device_operations const

2009-09-22T14:17:25+00:00

Signed-off-by: Alexey Dobriyan 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

mtd: blkdevs: do not forget to get MTD devices

2009-08-03T13:20:26+00:00

Nowadays MTD devices have to be "get" before they can be
used. This has to be done with 'get_mtd_device()'. The
'blktrans_open()' function did not do this and instead
used 'try_module_get()'. Fix this.

Since 'get_mtd_device()' already gets the module, extra
'try_module_get()' is not needed.

This fixes oops when one tries to use mtdblock on top of
gluebi.

Reported-by: Holger Brunck 
Signed-off-by: Artem Bityutskiy 
Signed-off-by: David Woodhouse

Merge git://git.infradead.org/mtd-2.6

2009-06-22T23:56:22+00:00

* git://git.infradead.org/mtd-2.6: (63 commits)
  mtd: OneNAND: Allow setting of boundary information when built as module
  jffs2: leaking jffs2_summary in function jffs2_scan_medium
  mtd: nand: Fix memory leak on txx9ndfmc probe failure.
  mtd: orion_nand: use burst reads with double word accesses
  mtd/nand: s3c6400 support for s3c2410 driver
  [MTD] [NAND] S3C2410: Use DIV_ROUND_UP
  [MTD] [NAND] S3C2410: Deal with unaligned lengths in S3C2440 buffer read/write
  [MTD] [NAND] S3C2410: Allow the machine code to get the BBT table from NAND
  [MTD] [NAND] S3C2410: Added a kerneldoc for s3c2410_nand_set
  mtd: physmap_of: Add multiple regions and concatenation support
  mtd: nand: max_retries off by one in mxc_nand
  mtd: nand: s3c2410_nand_setrate(): use correct macros for 2412/2440
  mtd: onenand: add bbt_wait & unlock_all as replaceable for some platform
  mtd: Flex-OneNAND support
  mtd: nand: add OMAP2/OMAP3 NAND driver
  mtd: maps: Blackfin async: fix memory leaks in probe/remove funcs
  mtd: uclinux: mark local stuff static
  mtd: uclinux: do not allow to be built as a module
  mtd: uclinux: allow systems to override map addr/size
  mtd: blackfin NFC: fix hang when using NAND on BF527-EZKITs
  ...

[MTD] set blkdev parent to the mtd device, not its parent

2009-05-26T15:41:21+00:00

Signed-off-by: David Woodhouse

block: Do away with the notion of hardsect_size

2009-05-22T21:22:54+00:00

Until now we have had a 1:1 mapping between storage device physical
block size and the logical block sized used when addressing the device.
With SATA 4KB drives coming out that will no longer be the case.  The
sector size will be 4KB but the logical block size will remain
512-bytes.  Hence we need to distinguish between the physical block size
and the logical ditto.

This patch renames hardsect_size to logical_block_size.

Signed-off-by: Martin K. Petersen 
Signed-off-by: Jens Axboe

block: implement and enforce request peek/start/fetch

2009-05-11T07:52:18+00:00

Till now block layer allowed two separate modes of request execution.
A request is always acquired from the request queue via
elv_next_request().  After that, drivers are free to either dequeue it
or process it without dequeueing.  Dequeue allows elv_next_request()
to return the next request so that multiple requests can be in flight.

Executing requests without dequeueing has its merits mostly in
allowing drivers for simpler devices which can't do sg to deal with
segments only without considering request boundary.  However, the
benefit this brings is dubious and declining while the cost of the API
ambiguity is increasing.  Segment based drivers are usually for very
old or limited devices and as converting to dequeueing model isn't
difficult, it doesn't justify the API overhead it puts on block layer
and its more modern users.

Previous patches converted all block low level drivers to dequeueing
model.  This patch completes the API transition by...

* renaming elv_next_request() to blk_peek_request()

* renaming blkdev_dequeue_request() to blk_start_request()

* adding blk_fetch_request() which is combination of peek and start

* disallowing completion of queued (not started) requests

* applying new API to all LLDs

Renamings are for consistency and to break out of tree code so that
it's apparent that out of tree drivers need updating.

[ Impact: block request issue API cleanup, no functional change ]

Signed-off-by: Tejun Heo 
Cc: Rusty Russell 
Cc: James Bottomley 
Cc: Mike Miller 
Cc: unsik Kim 
Cc: Paul Clements 
Cc: Tim Waugh 
Cc: Geert Uytterhoeven 
Cc: David S. Miller 
Cc: Laurent Vivier 
Cc: Jeff Garzik 
Cc: Jeremy Fitzhardinge 
Cc: Grant Likely 
Cc: Adrian McMenamin 
Cc: Stephen Rothwell 
Cc: Bartlomiej Zolnierkiewicz 
Cc: Borislav Petkov 
Cc: Sergei Shtylyov 
Cc: Alex Dubov 
Cc: Pierre Ossman 
Cc: David Woodhouse 
Cc: Markus Lidel 
Cc: Stefan Weinhuber 
Cc: Martin Schwidefsky 
Cc: Pete Zaitcev 
Cc: FUJITA Tomonori 
Signed-off-by: Jens Axboe

mtd_blkdevs: dequeue in-flight request

2009-05-11T07:52:17+00:00

mtd_blkdevs processes requests one-by-one synchronously from a kthread
and can be easily converted to dequeueing model.  Convert it.

[ Impact: dequeue in-flight request ]

Signed-off-by: Tejun Heo 
Cc: David Woodhouse 
Signed-off-by: Jens Axboe

block: blk_rq_[cur_]_{sectors|bytes}() usage cleanup

2009-05-11T07:50:55+00:00

With the previous changes, the followings are now guaranteed for all
requests in any valid state.

* blk_rq_sectors() == blk_rq_bytes() >> 9
* blk_rq_cur_sectors() == blk_rq_cur_bytes() >> 9

Clean up accessor usages.  Notable changes are

* nbd,i2o_block: end_all used instead of explicit byte count
* scsi_lib: unnecessary conditional on request type removed

[ Impact: cleanup ]

Signed-off-by: Tejun Heo 
Cc: Paul Clements 
Cc: Pete Zaitcev 
Cc: Alex Dubov 
Cc: Markus Lidel 
Cc: David Woodhouse 
Cc: James Bottomley 
Cc: Boaz Harrosh 
Signed-off-by: Jens Axboe

block: convert to pos and nr_sectors accessors

2009-05-11T07:50:54+00:00

With recent cleanups, there is no place where low level driver
directly manipulates request fields.  This means that the 'hard'
request fields always equal the !hard fields.  Convert all
rq->sectors, nr_sectors and current_nr_sectors references to
accessors.

While at it, drop superflous blk_rq_pos() < 0 test in swim.c.

[ Impact: use pos and nr_sectors accessors ]

Signed-off-by: Tejun Heo 
Acked-by: Geert Uytterhoeven 
Tested-by: Grant Likely 
Acked-by: Grant Likely 
Tested-by: Adrian McMenamin 
Acked-by: Adrian McMenamin 
Acked-by: Mike Miller 
Cc: James Bottomley 
Cc: Bartlomiej Zolnierkiewicz 
Cc: Borislav Petkov 
Cc: Sergei Shtylyov 
Cc: Eric Moore 
Cc: Alan Stern 
Cc: FUJITA Tomonori 
Cc: Pete Zaitcev 
Cc: Stephen Rothwell 
Cc: Paul Clements 
Cc: Tim Waugh 
Cc: Jeff Garzik 
Cc: Jeremy Fitzhardinge 
Cc: Alex Dubov 
Cc: David Woodhouse 
Cc: Martin Schwidefsky 
Cc: Dario Ballabio 
Cc: David S. Miller 
Cc: Rusty Russell 
Cc: unsik Kim 
Cc: Laurent Vivier 
Signed-off-by: Jens Axboe