diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2008-03-02 09:09:22 -0500 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2008-10-21 07:47:32 -0400 |
commit | d4430d62fa77208824a37fe6f85ab2831d274769 (patch) | |
tree | 5d4d0bca31e63eb208fbebe4f39c912b964c1e4d /drivers/s390 | |
parent | badf8082c33d18b118d3a6f1b32d5ea6b97d3839 (diff) |
[PATCH] beginning of methods conversion
To keep the size of changesets sane we split the switch by drivers;
to keep the damn thing bisectable we do the following:
1) rename the affected methods, add ones with correct
prototypes, make (few) callers handle both. That's this changeset.
2) for each driver convert to new methods. *ALL* drivers
are converted in this series.
3) kill the old (renamed) methods.
Note that it _is_ a flagday; all in-tree drivers are converted and by the
end of this series no trace of old methods remain. The only reason why
we do that this way is to keep the damn thing bisectable and allow per-driver
debugging if anything goes wrong.
New methods:
open(bdev, mode)
release(disk, mode)
ioctl(bdev, mode, cmd, arg) /* Called without BKL */
compat_ioctl(bdev, mode, cmd, arg)
locked_ioctl(bdev, mode, cmd, arg) /* Called with BKL, legacy */
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'drivers/s390')
-rw-r--r-- | drivers/s390/block/dasd.c | 8 | ||||
-rw-r--r-- | drivers/s390/block/dcssblk.c | 4 | ||||
-rw-r--r-- | drivers/s390/char/tape_block.c | 6 |
3 files changed, 9 insertions, 9 deletions
diff --git a/drivers/s390/block/dasd.c b/drivers/s390/block/dasd.c index 0a225ccda026..6bf68e5fe89f 100644 --- a/drivers/s390/block/dasd.c +++ b/drivers/s390/block/dasd.c | |||
@@ -2087,10 +2087,10 @@ static int dasd_getgeo(struct block_device *bdev, struct hd_geometry *geo) | |||
2087 | struct block_device_operations | 2087 | struct block_device_operations |
2088 | dasd_device_operations = { | 2088 | dasd_device_operations = { |
2089 | .owner = THIS_MODULE, | 2089 | .owner = THIS_MODULE, |
2090 | .open = dasd_open, | 2090 | .__open = dasd_open, |
2091 | .release = dasd_release, | 2091 | .__release = dasd_release, |
2092 | .ioctl = dasd_ioctl, | 2092 | .__ioctl = dasd_ioctl, |
2093 | .compat_ioctl = dasd_compat_ioctl, | 2093 | .__compat_ioctl = dasd_compat_ioctl, |
2094 | .getgeo = dasd_getgeo, | 2094 | .getgeo = dasd_getgeo, |
2095 | }; | 2095 | }; |
2096 | 2096 | ||
diff --git a/drivers/s390/block/dcssblk.c b/drivers/s390/block/dcssblk.c index a7ff167d5b81..413460cc3dd8 100644 --- a/drivers/s390/block/dcssblk.c +++ b/drivers/s390/block/dcssblk.c | |||
@@ -42,8 +42,8 @@ static char dcssblk_segments[DCSSBLK_PARM_LEN] = "\0"; | |||
42 | static int dcssblk_major; | 42 | static int dcssblk_major; |
43 | static struct block_device_operations dcssblk_devops = { | 43 | static struct block_device_operations dcssblk_devops = { |
44 | .owner = THIS_MODULE, | 44 | .owner = THIS_MODULE, |
45 | .open = dcssblk_open, | 45 | .__open = dcssblk_open, |
46 | .release = dcssblk_release, | 46 | .__release = dcssblk_release, |
47 | .direct_access = dcssblk_direct_access, | 47 | .direct_access = dcssblk_direct_access, |
48 | }; | 48 | }; |
49 | 49 | ||
diff --git a/drivers/s390/char/tape_block.c b/drivers/s390/char/tape_block.c index a25b8bf54f41..f1a741c9a6f0 100644 --- a/drivers/s390/char/tape_block.c +++ b/drivers/s390/char/tape_block.c | |||
@@ -52,9 +52,9 @@ static int tapeblock_revalidate_disk(struct gendisk *); | |||
52 | 52 | ||
53 | static struct block_device_operations tapeblock_fops = { | 53 | static struct block_device_operations tapeblock_fops = { |
54 | .owner = THIS_MODULE, | 54 | .owner = THIS_MODULE, |
55 | .open = tapeblock_open, | 55 | .__open = tapeblock_open, |
56 | .release = tapeblock_release, | 56 | .__release = tapeblock_release, |
57 | .ioctl = tapeblock_ioctl, | 57 | .__ioctl = tapeblock_ioctl, |
58 | .media_changed = tapeblock_medium_changed, | 58 | .media_changed = tapeblock_medium_changed, |
59 | .revalidate_disk = tapeblock_revalidate_disk, | 59 | .revalidate_disk = tapeblock_revalidate_disk, |
60 | }; | 60 | }; |