On 05/25/2012 08:46 PM, Dan Williams wrote: > On Fri, May 25, 2012 at 8:13 AM, Tomas Henzl <thenzl@xxxxxxxxxx> wrote: >> May 25 04:25:54 localhost kernel: [ 461.525209] BUG: unable to handle kernel NULL pointer dereference at 0000000000000079 >> May 25 04:25:54 localhost kernel: [ 461.525242] IP: [<ffffffff811f0c45>] sysfs_create_dir+0x35/0xc0 > Have a look at: > > http://marc.info/?l=linux-scsi&m=133793175125892&w=2 > > ...it addresses tearing down targets before they are added which > appears to be the signature here. Yes it helps, I can not reproduce this^ BUG any longer. The problem I'm trying to fix can be seen when the lld is removed while the scsi async scan is running. The reason is, that the pointers to the lld stored in scsi_host_template are accesed even after the lld is removed. Several possibilities how to fix this have evolved during the time the two main options are -1) protect the scan process with try_modele_get/put, there is a problem with 'rmmod --wait'. This seems now, with your patch, to be removed (the try_module-get-put fix is still needed). I don't believe this is 100% safe because of the problems in scsi_device_get. 2) the patch posted here http://marc.info/?l=linux-scsi&m=133527313522298&w=2 it cancels the async scan, when the driver calls scsi_remove_host.(it's faster :) I hope it is safe -I haven't seen any problems, tested without your patch. I think your use case differs, so this patch won't fix problems you see. -------------- >From my point of view your patch is needed and one of the two patches referenced above is needed too, I don't care which one will be used. Tomas > > -- > Dan > -- > 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 -- 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
