linux.git/block/genhd.c, branch v2.6.27

block: restore original behavior of /proc/partition when there's no partition

2008-09-01T06:55:10+00:00

/proc/partitions didn't use to write out the header if there was no
partition.  However, recent commit 66c64afe changed the behavior.
This is nothing major but there's no reason to change user visible
behavior without a good rationale.  Restore the original behavior.

Note that 2.6.28 has clean up changes scheduled which will replace
this rather hacky implementation.

Signed-off-by: Tejun Heo 
Cc: Greg KH 
Cc: Kay Sievers 
Signed-off-by: Jens Axboe

remove blk_register_filter and blk_unregister_filter in gendisk

2008-08-29T09:47:07+00:00

This patch remove blk_register_filter and blk_unregister_filter in
gendisk, and adds them to sd.c, sr.c. and ide-cd.c

The commit abf5439370491dd6fbb4fe1a7939680d2a9bc9d4 moved cmdfilter
from gendisk to request_queue. It turned out that in some subsystems
multiple gendisks share a single request_queue. So we get:

Using physmap partition information
Creating 3 MTD partitions on "physmap-flash":
0x00000000-0x01c00000 : "User FS"
0x01c00000-0x01c40000 : "booter"
kobject (8511c410): tried to init an initialized object, something is seriously wrong.
Call Trace:
[<8036644c>] dump_stack+0x8/0x34
[<8021f050>] kobject_init+0x50/0xcc
[<8021fa18>] kobject_init_and_add+0x24/0x58
[<8021d20c>] blk_register_filter+0x4c/0x64
[<8021c194>] add_disk+0x78/0xe0
[<8027d14c>] add_mtd_blktrans_dev+0x254/0x278
[<8027c8f0>] blktrans_notify_add+0x40/0x78
[<80279c00>] add_mtd_device+0xd0/0x150
[<8027b090>] add_mtd_partitions+0x568/0x5d8
[<80285458>] physmap_flash_probe+0x2ac/0x334
[<802644f8>] driver_probe_device+0x12c/0x244
[<8026465c>] __driver_attach+0x4c/0x84
[<80263c64>] bus_for_each_dev+0x58/0xac
[<802633ec>] bus_add_driver+0xc4/0x24c
[<802648e0>] driver_register+0xcc/0x184
[<80100460>] _stext+0x60/0x1bc

In the long term, we need to fix such subsystems but we need a quick
fix now. This patch add the command filter support to only sd and sr
though it might be useful for other SG_IO users (such as cciss).

Signed-off-by: FUJITA Tomonori 
Reported-by: Manuel Lauss 
Signed-off-by: Jens Axboe

block: drop references taken by class_find_device()

2008-08-21T17:15:38+00:00

Otherwise we leak references, which is not a good thing to do.


Signed-off-by: Kay Sievers 
Signed-off-by: Greg Kroah-Hartman

block: fix partial read() of /proc/{partitions,diskstats}

2008-08-21T17:15:38+00:00

The proc files get truncated if they do not fit into the buffer with
a single read(). We need to move the seq_file index from the callback
of class_find_device() to the caller of class_find_device(), to keep
its value across multiple invocations of the callback.

Signed-off-by: Kay Sievers 
Signed-off-by: Greg Kroah-Hartman

block: make /proc/partitions and /proc/diskstats use class_find_device()

2008-07-22T04:54:49+00:00

Use the proper class iterator function instead of mucking around in the
internals of the class structures.

Cc: Kay Sievers 
Signed-off-by: Greg Kroah-Hartman

block: move header for /proc/partitions to seq_start

2008-07-22T04:54:49+00:00

The seq_start call is the better place for the header for the file, that
way we don't have to be mucking in the class structure to try to figure
out if this is the first partition or not.

Cc: Kay Sievers 
Signed-off-by: Greg Kroah-Hartman

block: make proc files seq_start use the class_find_device()

2008-07-22T04:54:49+00:00

Use the proper class iterator function instead of mucking around in the
internals of the class structures.

Cc: Kay Sievers 
Signed-off-by: Greg Kroah-Hartman

block: make /proc/diskstats only build if CONFIG_PROC_FS is enabled

2008-07-22T04:54:49+00:00

These functions are only needed if CONFIG_PROC_FS is enabled, so save
the space when it is not.

This also makes it easier for a patch later in this series to work
properly if CONFIG_PROC_FS is not enabled.

Signed-off-by: Randy Dunlap 
Signed-off-by: Greg Kroah-Hartman

block: make blk_lookup_devt use the class iterator function

2008-07-22T04:54:48+00:00

Use the proper class iterator function instead of mucking around in the
internals of the class structures.

Cc: Kay Sievers 
Signed-off-by: Greg Kroah-Hartman

block: make printk_partition use the class iterator function

2008-07-22T04:54:48+00:00

Use the proper class iterator function instead of mucking around in the
internals of the class structures.

Cc: Kay Sievers 
Signed-off-by: Greg Kroah-Hartman