Hi Kiyoshi,
Kiyoshi Ueda wrote:
Hi Hannes,
On 2009/03/10 16:17 +0900, Hannes Reinecke wrote:
[ .. ]
Yes, I've found that one later on.
The real problem was in clone_setup_bios(), which might end up calling an
invalid end_io_data pointer. Patch is attached.
Nice catch! Thank you for the patch.
Oh, nae bother. Took me only a month to track it down :-(
-static void free_bio_clone(struct request *clone)
+static void free_bio_clone(struct request *clone, struct mapped_device *md)
I have changed the argument order to match with other free_* functions:
free_bio_clone(struct mapped_device *md, struct request *clone)
Sure. I wasn't sure myself which way round the arguments should be.
Do you have an updated patch of your suspend fixes? We've run into an issue
here which looks suspiciously close to that one (I/O is completed on a deleted
pgpath), so we would be happy to test it these out.
Cheers,
Hannes
--
Dr. Hannes Reinecke zSeries & Storage
hare@xxxxxxx +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Markus Rex, HRB 16746 (AG Nürnberg)
--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel
