Re: [PATCH v8 0/10] More device removal fixes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 02/09/13 10:28, Bart Van Assche wrote:
On 02/09/13 00:29, Joe Lawrence wrote:
I haven't had time to rerun the test without the two patches that wait in
scsi_remove_host(), however I did rerun the test and verify the same
behavior as in my earlier mail.  I didn't see any __scsi_remove_device()
instances running.

Some more investigation revealed that MD RAID was holding a reference to
the removed device.  (In short, mdadm --remove had failed and left the
device as a faulty member of the array.)  When I did finally manage to
kick that disk from the MD device, scsi host/device removal continued to
completion as expected.

There's a bit more context to the MD situation that I'll post to the raid
list once I get the details together for Neil.  I will CC you if you are
interested in following.

The loop in scsi_remove_host() waits too long. It should stop waiting as
soon as the blk_cleanup_queue() calls for all sdev's have finished
instead of waiting until all sdev users have closed these sdev's. I will
repost patches 07/10 and 08/10.

(replying to my own e-mail)

Hello Joe,

It would be appreciated if you could repeat your test with this kernel tree (tested with iSCSI and SRP): http://github.com/bvanassche/linux/tree/device-removal-fixes. If that test succeed I will repost the patches in that tree.

Thanks,

Bart.
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux