Re: [PATCH 2/2] hpsa: destroy sas transport properties before scsi_host

[Martin Wilck]

On Tue, 2017-10-10 at 23:04 +, Don Brace wrote:
> Now that Hannes's patch  9441284fbc39610c0f9ec0ed118ff85d78352906
> has been applied, this patch corrects the stack trace issue.
> 
> Would you like to re-submit this patch or would you like me to send
> it up?
> I'll run some quick tests if you do decide to send it up. 
> If you want me to send it up, you will get the credit anyway.

>From my PoV, just go ahead and re-submit. It'll be faster than me
diving into this again.

Thanks,
Martin

RE: [PATCH 2/2] hpsa: destroy sas transport properties before scsi_host

[Don Brace]

> -Original Message-
> From: Johannes Thumshirn [mailto:jthumsh...@suse.de]
> Sent: Monday, November 21, 2016 8:15 AM
> To: Martin Wilck <mwi...@suse.de>
> Cc: Don Brace <don.br...@microsemi.com>; dl-esc-Team ESD Storage Dev
> Support <esc-teamesdstoragedevsupp...@microsemi.com>;
> iss_storage...@hp.com; linux-scsi@vger.kernel.org; jbottom...@odin.com;
> h...@lst.de; h...@suse.de
> Subject: Re: [PATCH 2/2] hpsa: destroy sas transport properties before
> scsi_host
> 
> EXTERNAL EMAIL
> 
> 
> On Mon, Nov 21, 2016 at 03:04:29PM +0100, Martin Wilck wrote:
> > Unloading the hpsa driver causes warnings
> >
> > [ 1063.793652] WARNING: CPU: 1 PID: 4850 at ../fs/sysfs/group.c:237
> device_del+0x54/0x240()
> > [ 1063.793659] sysfs group 81cf21a0 not found for kobject 'port-2:0'
> >
> > with two different stacks:
> > 1)
> > [ 1063.793774]  [] device_del+0x54/0x240
> > [ 1063.793780]  []
> transport_remove_classdev+0x4a/0x60
> > [ 1063.793784]  []
> attribute_container_device_trigger+0xa6/0xb0
> > [ 1063.793802]  [] sas_port_delete+0x126/0x160
> [scsi_transport_sas]
> > [ 1063.793819]  [] hpsa_free_sas_port+0x3c/0x70 [hpsa]
> >
> > 2)
> > [ 1063.797103]  [] device_del+0x54/0x240
> > [ 1063.797118]  [] sas_port_delete+0x12e/0x160
> [scsi_transport_sas]
> > [ 1063.797134]  [] hpsa_free_sas_port+0x3c/0x70 [hpsa]
> >
> > This is caused by the fact that host device hostX is deleted before the
> > SAS transport devices hostX/port-a:b.
> >
> > This patch fixes this by reverting the order of device deletions.
> >
> > References: bsc#1010946
> > Signed-off-by: Martin Wilck <mwi...@suse.de>
> > ---
> 
> With the References changed to the bug link like in patch 1/2
> Reviewed-by: Johannes Thumshirn <jthumsh...@suse.de>

Now that Hannes's patch  9441284fbc39610c0f9ec0ed118ff85d78352906
has been applied, this patch corrects the stack trace issue.

Would you like to re-submit this patch or would you like me to send it up?
I'll run some quick tests if you do decide to send it up. 
If you want me to send it up, you will get the credit anyway.

Thanks for your help and attention to this issue.
And thanks again to Hannes.

Thanks,
Don Brace
ESC - Smart Storage
Microsemi Corporation


> --
> Johannes Thumshirn  Storage
> jthumsh...@suse.de+49 911 74053 689
> SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
> GF: Felix Imendörffer, Jane Smithard, Graham Norton
> HRB 21284 (AG Nürnberg)
> Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850

Re: [PATCH 2/2] hpsa: destroy sas transport properties before scsi_host

[Johannes Thumshirn]

On Mon, Nov 21, 2016 at 03:04:29PM +0100, Martin Wilck wrote:
> Unloading the hpsa driver causes warnings
> 
> [ 1063.793652] WARNING: CPU: 1 PID: 4850 at ../fs/sysfs/group.c:237 
> device_del+0x54/0x240()
> [ 1063.793659] sysfs group 81cf21a0 not found for kobject 'port-2:0'
> 
> with two different stacks:
> 1)
> [ 1063.793774]  [] device_del+0x54/0x240
> [ 1063.793780]  [] transport_remove_classdev+0x4a/0x60
> [ 1063.793784]  [] 
> attribute_container_device_trigger+0xa6/0xb0
> [ 1063.793802]  [] sas_port_delete+0x126/0x160 
> [scsi_transport_sas]
> [ 1063.793819]  [] hpsa_free_sas_port+0x3c/0x70 [hpsa]
> 
> 2)
> [ 1063.797103]  [] device_del+0x54/0x240
> [ 1063.797118]  [] sas_port_delete+0x12e/0x160 
> [scsi_transport_sas]
> [ 1063.797134]  [] hpsa_free_sas_port+0x3c/0x70 [hpsa]
> 
> This is caused by the fact that host device hostX is deleted before the
> SAS transport devices hostX/port-a:b.
> 
> This patch fixes this by reverting the order of device deletions.
> 
> References: bsc#1010946
> Signed-off-by: Martin Wilck 
> ---

With the References changed to the bug link like in patch 1/2 
Reviewed-by: Johannes Thumshirn 

-- 
Johannes Thumshirn  Storage
jthumsh...@suse.de+49 911 74053 689
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)
Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html