Re: [PATCH] bfa: fix access to bfad_im_port_s

2017-12-06 Thread James Bottomley
On Wed, 2017-12-06 at 09:07 +0100, Johannes Thumshirn wrote: > James Bottomley writes: > > > > On Tue, 2017-11-28 at 16:26 +0100, Johannes Thumshirn wrote: > > > > > > Commit 'cd21c605b2cf ("scsi: fc: provide fc_bsg_to_shost() > > > helper")' > > > changed access to bfa's 'struct bfad_im_port_s'

Re: [PATCH] bfa: fix access to bfad_im_port_s

2017-12-06 Thread Johannes Thumshirn
James Bottomley writes: > On Tue, 2017-11-28 at 16:26 +0100, Johannes Thumshirn wrote: >> Commit 'cd21c605b2cf ("scsi: fc: provide fc_bsg_to_shost() helper")' >> changed access to bfa's 'struct bfad_im_port_s' by using shost_priv() >> instead of shost->hostdata[0]. >> >> This lead to crashes like

Re: [PATCH] bfa: fix access to bfad_im_port_s

2017-12-05 Thread James Bottomley
On Tue, 2017-11-28 at 16:26 +0100, Johannes Thumshirn wrote: > Commit 'cd21c605b2cf ("scsi: fc: provide fc_bsg_to_shost() helper")' > changed access to bfa's 'struct bfad_im_port_s' by using shost_priv() > instead of shost->hostdata[0]. > > This lead to crashes like in the following back-trace: >

Re: [PATCH] bfa: fix access to bfad_im_port_s

2017-11-28 Thread Martin K. Petersen
Johannes, > Commit 'cd21c605b2cf ("scsi: fc: provide fc_bsg_to_shost() helper")' > changed access to bfa's 'struct bfad_im_port_s' by using shost_priv() > instead of shost->hostdata[0]. Applied to 4.15/scsi-fixes. Thanks! -- Martin K. Petersen Oracle Linux Engineering

Re: [PATCH] bfa: fix access to bfad_im_port_s

2017-11-28 Thread Hannes Reinecke
On 11/28/2017 04:26 PM, Johannes Thumshirn wrote: > Commit 'cd21c605b2cf ("scsi: fc: provide fc_bsg_to_shost() helper")' > changed access to bfa's 'struct bfad_im_port_s' by using shost_priv() > instead of shost->hostdata[0]. > > This lead to crashes like in the following back-trace: > > task: ff

[PATCH] bfa: fix access to bfad_im_port_s

2017-11-28 Thread Johannes Thumshirn
Commit 'cd21c605b2cf ("scsi: fc: provide fc_bsg_to_shost() helper")' changed access to bfa's 'struct bfad_im_port_s' by using shost_priv() instead of shost->hostdata[0]. This lead to crashes like in the following back-trace: task: 880046375300 ti: 8800a2ef8000 task.ti: 8800a2ef8000 RI