Re: [PATCH] Re: Oops using 2.6.28.n after a lazy umount of a cryptedloop-device

From: Alexander Holler
Date: Fri Mar 06 2009 - 01:16:52 EST


Hello,

thanks for the fast response and patch.

Milan Broz schrieb:
Please Can you try attached patch if helps here?
(Patch is not perfect, but should help, at least identify that
it is the same problem I am fixing:-)

The patch works (I had to add an #include <linux/delay.h> /* msleep */).

I've tested it using 2.6.28.7 and the script below. With your patch the script was running over night looping about 400 times without any error. A crosscheck without the patch needed only 10 iterations to get an oops. So I assume you have fixed the problem I had. ;)

There stills seems to be another problem left, I've got 3 times the kernel-message

device-mapper: ioctl: unable to remove open device temporary-cryptsetup-21571

during cryptsetup luksOpen. I never realized that msg before, but I've found an old one in my logs too.

Anyway, this does not result in an oops, so I'm happy. Thanks a lot.

Kind regards,

Alexander Holler


----------- oopstest.exp -----------------------
!/usr/bin/expect
system modprobe dm-crypt
system modprobe loop
for { set i 0 } { $i < 1000 } {} {
incr i
send_user "Test $i\n"
system losetup /dev/loop1 /Daten/Daten.crypt
spawn cryptsetup luksOpen /dev/loop1 crypted
expect passphrase:
sleep 2
send oopstest\r
sleep 60
send_user \n
system fsck.ext3 /dev/mapper/crypted
sleep 2
system mount -t ext3 -o rw,user,exec,noatime /dev/mapper/crypted /Daten/crypted
system dd if=/dev/urandom of=/Daten/crypted/random bs=1024 count=1024
system umount -l /Daten/crypted
system cryptsetup luksClose crypted
system losetup -d /dev/loop1
}
----------- oopstest.exp -----------------------

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/