adam radford wrote: > Randy, > > Is there any reason why any of these proc_name patches need to > go in after James Smart's explanation: Nope. Thanks for your comments. ~Randy > http://marc.theaimsgroup.com/?l=linux-scsi&m=115040210406375&w=2 > > in particular, the part about: > > "- scsi_proc_hostdir_add() validates that sht->proc_info is non-null before > attempting to use sht->proc_name." > > proc_info is null for the 3w-9xxx driver. > > -Adam > > On 6/20/06, Randy Dunlap <randy.dunlap@xxxxxxxxxx> wrote: >> Add .proc_name member to scsi_host_template for 3w-9xxx. >> >> http://www.kernel.org/git/?p=linux/kernel/git/bcollins/ubuntu-dapper.git;a=commitdiff;h=12fdf64086335ed596848cf17602e7a71e15ae9b >> >> >> >> --- >> drivers/scsi/3w-9xxx.c | 1 + >> 1 file changed, 1 insertion(+) >> >> --- linux-2617-pv.orig/drivers/scsi/3w-9xxx.c >> +++ linux-2617-pv/drivers/scsi/3w-9xxx.c >> @@ -2011,6 +2011,7 @@ static void twa_unmap_scsi_data(TW_Devic >> /* scsi_host_template initializer */ >> static struct scsi_host_template driver_template = { >> .module = THIS_MODULE, >> + .proc_name = "3w-9xxx", >> .name = "3ware 9000 Storage Controller", >> .queuecommand = twa_scsi_queue, >> .eh_host_reset_handler = twa_scsi_eh_reset, >> >> >> >> - - : 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
