Messages in this thread Patch in this message | | | Date | Sat, 20 Oct 2007 01:07:09 -0400 | From | emist <> | Subject | [PATCH] Bug fix for the s390 dcssblk driver |
| |
Hello,
The following patch fixes and issue in the s390 dcssblk driver. The issue is caused when an unsuccessful attempt is made in order to change a segment's type through the device attribute file "shared". This causes the driver to remove the device in question, removing with it the device attribute which is currently handling the call. The result is a hang on the driver as it removes memory from under its feet.
Not exactly sure if this explanation makes sense or its entirely accurate. This is what I believe at this point from encountering and fixing the error. Anyway here is the patch, hope it helps.
# This patch fixes a memory corruption bug in the s390 dcssblk driver. # The bug occurs when an attempt to change the type of a segment # returns an error. At this point the driver tries to remove the segment in # question while some of the device's attributes are in use. This causes the # driver to hang. # # questions/comments @ emistz@gmail.com
diff -urN linux-2.6.23.1/drivers/s390/block/dcssblk.c linuxx/drivers/s390/block/dcssblk.c --- linux-2.6.23.1/drivers/s390/block/dcssblk.c 2007-10-12 12:43:44.000000000 -0400 +++ linuxx/drivers/s390/block/dcssblk.c 2007-10-20 00:51:19.000000000 -0400 @@ -253,8 +253,12 @@ SEGMENT_EXCLUSIVE); if (rc < 0) { BUG_ON(rc == -EINVAL); - if (rc != -EAGAIN) - goto removeseg; + if (rc != -EAGAIN){ + PRINT_DEBUG("Could not reload segment %s in the specified format, reloading\n", + dev_info->segment_name); + rc = segment_modify_shared(dev_info->segment_name, SEGMENT_SHARED); + goto out; + } } else { dev_info->is_shared = 0; set_disk_ro(dev_info->gd, 0); | |