RE: [PATCH] csiostor:Fix locking issues in the function csio_scsim_cleanup_io_lnode
> -Original Message- > From: Praveen Madhavan > Sent: Friday, September 4, 2015 6:12 AM > To: Anish Bhatt; Nicholas Krause; jbottom...@odin.com > Cc: h...@suse.de; micha...@cs.wisc.edu; da...@davemloft.net; > Hariprasad S; linux-s...@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: RE: [PATCH] csiostor:Fix locking issues in the function > csio_scsim_cleanup_io_lnode > > csio_scsim_cleanup_io_lnode always called with spin_lock_irq held. > Please refer: > csio_attr.c:624:csio_scsim_cleanup_io_lnode(csio_hw_to_scsim(hw), > ln); > csio_attr.c:649: > csio_scsim_cleanup_io_lnode(csio_hw_to_scsim(hw), ln); > > -Original Message- > From: linux-scsi-ow...@vger.kernel.org [mailto:linux-scsi- > ow...@vger.kernel.org] On Behalf Of Anish Bhatt > Sent: Thursday, September 03, 2015 7:07 AM > To: Nicholas Krause; jbottom...@odin.com > Cc: h...@suse.de; micha...@cs.wisc.edu; da...@davemloft.net; > Hariprasad S; linux-s...@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: RE: [PATCH] csiostor:Fix locking issues in the function > csio_scsim_cleanup_io_lnode > > > -Original Message- > > From: Nicholas Krause [mailto:xerofo...@gmail.com] > > Sent: Wednesday, September 2, 2015 10:36 AM > > To: jbottom...@odin.com > > Cc: h...@suse.de; micha...@cs.wisc.edu; da...@davemloft.net; Anish > > Bhatt; Hariprasad S; linux-s...@vger.kernel.org; linux- > > ker...@vger.kernel.org > > Subject: [PATCH] csiostor:Fix locking issues in the function > > csio_scsim_cleanup_io_lnode > > > > This fixes locking issues in the function csio_scsim_cleanup_io_lnode > > by locking around the call to the function csio_csci_gather_active_ios > > with the function pair spin_lock_irq/spin_unlock_irq as any function > > calling this particular function must do in order to avoid concurrent > > threads of execution on the passed structure pointer of type csio_hw > > as this structure pointer can be shared across mutliple threads in the > kernel. > > > > Signed-off-by: Nicholas Krause > > --- > > drivers/scsi/csiostor/csio_scsi.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/scsi/csiostor/csio_scsi.c > > b/drivers/scsi/csiostor/csio_scsi.c > > index 2c4562d..c318855 100644 > > --- a/drivers/scsi/csiostor/csio_scsi.c > > +++ b/drivers/scsi/csiostor/csio_scsi.c > > @@ -1327,7 +1327,9 @@ csio_scsim_cleanup_io_lnode(struct csio_scsim > > *scm, struct csio_lnode *ln) > > sld.level = CSIO_LEV_LNODE; > > sld.lnode = ln; > > INIT_LIST_HEAD(>cmpl_q); > > + spin_lock_irq(>lock); > > csio_scsi_gather_active_ios(scm, , >cmpl_q); > > + spin_unlock_irq(>lock); > > > > /* No I/Os pending on this lnode */ > > if (list_empty(>cmpl_q)) > > -- > > 2.1.4 > > Acked-By: Anish Bhatt My bad, NACK. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH] csiostor:Fix locking issues in the function csio_scsim_cleanup_io_lnode
csio_scsim_cleanup_io_lnode always called with spin_lock_irq held. Please refer: csio_attr.c:624:csio_scsim_cleanup_io_lnode(csio_hw_to_scsim(hw), ln); csio_attr.c:649: csio_scsim_cleanup_io_lnode(csio_hw_to_scsim(hw), ln); -Original Message- From: linux-scsi-ow...@vger.kernel.org [mailto:linux-scsi-ow...@vger.kernel.org] On Behalf Of Anish Bhatt Sent: Thursday, September 03, 2015 7:07 AM To: Nicholas Krause; jbottom...@odin.com Cc: h...@suse.de; micha...@cs.wisc.edu; da...@davemloft.net; Hariprasad S; linux-s...@vger.kernel.org; linux-kernel@vger.kernel.org Subject: RE: [PATCH] csiostor:Fix locking issues in the function csio_scsim_cleanup_io_lnode > -Original Message- > From: Nicholas Krause [mailto:xerofo...@gmail.com] > Sent: Wednesday, September 2, 2015 10:36 AM > To: jbottom...@odin.com > Cc: h...@suse.de; micha...@cs.wisc.edu; da...@davemloft.net; Anish > Bhatt; Hariprasad S; linux-s...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject: [PATCH] csiostor:Fix locking issues in the function > csio_scsim_cleanup_io_lnode > > This fixes locking issues in the function csio_scsim_cleanup_io_lnode > by locking around the call to the function csio_csci_gather_active_ios > with the function pair spin_lock_irq/spin_unlock_irq as any function > calling this particular function must do in order to avoid concurrent > threads of execution on the passed structure pointer of type csio_hw > as this structure pointer can be shared across mutliple threads in the kernel. > > Signed-off-by: Nicholas Krause > --- > drivers/scsi/csiostor/csio_scsi.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/scsi/csiostor/csio_scsi.c > b/drivers/scsi/csiostor/csio_scsi.c > index 2c4562d..c318855 100644 > --- a/drivers/scsi/csiostor/csio_scsi.c > +++ b/drivers/scsi/csiostor/csio_scsi.c > @@ -1327,7 +1327,9 @@ csio_scsim_cleanup_io_lnode(struct csio_scsim > *scm, struct csio_lnode *ln) > sld.level = CSIO_LEV_LNODE; > sld.lnode = ln; > INIT_LIST_HEAD(>cmpl_q); > + spin_lock_irq(>lock); > csio_scsi_gather_active_ios(scm, , >cmpl_q); > + spin_unlock_irq(>lock); > > /* No I/Os pending on this lnode */ > if (list_empty(>cmpl_q)) > -- > 2.1.4 Acked-By: Anish Bhatt -- 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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH] csiostor:Fix locking issues in the function csio_scsim_cleanup_io_lnode
csio_scsim_cleanup_io_lnode always called with spin_lock_irq held. Please refer: csio_attr.c:624:csio_scsim_cleanup_io_lnode(csio_hw_to_scsim(hw), ln); csio_attr.c:649: csio_scsim_cleanup_io_lnode(csio_hw_to_scsim(hw), ln); -Original Message- From: linux-scsi-ow...@vger.kernel.org [mailto:linux-scsi-ow...@vger.kernel.org] On Behalf Of Anish Bhatt Sent: Thursday, September 03, 2015 7:07 AM To: Nicholas Krause; jbottom...@odin.com Cc: h...@suse.de; micha...@cs.wisc.edu; da...@davemloft.net; Hariprasad S; linux-s...@vger.kernel.org; linux-kernel@vger.kernel.org Subject: RE: [PATCH] csiostor:Fix locking issues in the function csio_scsim_cleanup_io_lnode > -Original Message- > From: Nicholas Krause [mailto:xerofo...@gmail.com] > Sent: Wednesday, September 2, 2015 10:36 AM > To: jbottom...@odin.com > Cc: h...@suse.de; micha...@cs.wisc.edu; da...@davemloft.net; Anish > Bhatt; Hariprasad S; linux-s...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject: [PATCH] csiostor:Fix locking issues in the function > csio_scsim_cleanup_io_lnode > > This fixes locking issues in the function csio_scsim_cleanup_io_lnode > by locking around the call to the function csio_csci_gather_active_ios > with the function pair spin_lock_irq/spin_unlock_irq as any function > calling this particular function must do in order to avoid concurrent > threads of execution on the passed structure pointer of type csio_hw > as this structure pointer can be shared across mutliple threads in the kernel. > > Signed-off-by: Nicholas Krause <xerofo...@gmail.com> > --- > drivers/scsi/csiostor/csio_scsi.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/scsi/csiostor/csio_scsi.c > b/drivers/scsi/csiostor/csio_scsi.c > index 2c4562d..c318855 100644 > --- a/drivers/scsi/csiostor/csio_scsi.c > +++ b/drivers/scsi/csiostor/csio_scsi.c > @@ -1327,7 +1327,9 @@ csio_scsim_cleanup_io_lnode(struct csio_scsim > *scm, struct csio_lnode *ln) > sld.level = CSIO_LEV_LNODE; > sld.lnode = ln; > INIT_LIST_HEAD(>cmpl_q); > + spin_lock_irq(>lock); > csio_scsi_gather_active_ios(scm, , >cmpl_q); > + spin_unlock_irq(>lock); > > /* No I/Os pending on this lnode */ > if (list_empty(>cmpl_q)) > -- > 2.1.4 Acked-By: Anish Bhatt <an...@chelsio.com> -- 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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH] csiostor:Fix locking issues in the function csio_scsim_cleanup_io_lnode
> -Original Message- > From: Praveen Madhavan > Sent: Friday, September 4, 2015 6:12 AM > To: Anish Bhatt; Nicholas Krause; jbottom...@odin.com > Cc: h...@suse.de; micha...@cs.wisc.edu; da...@davemloft.net; > Hariprasad S; linux-s...@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: RE: [PATCH] csiostor:Fix locking issues in the function > csio_scsim_cleanup_io_lnode > > csio_scsim_cleanup_io_lnode always called with spin_lock_irq held. > Please refer: > csio_attr.c:624:csio_scsim_cleanup_io_lnode(csio_hw_to_scsim(hw), > ln); > csio_attr.c:649: > csio_scsim_cleanup_io_lnode(csio_hw_to_scsim(hw), ln); > > -Original Message- > From: linux-scsi-ow...@vger.kernel.org [mailto:linux-scsi- > ow...@vger.kernel.org] On Behalf Of Anish Bhatt > Sent: Thursday, September 03, 2015 7:07 AM > To: Nicholas Krause; jbottom...@odin.com > Cc: h...@suse.de; micha...@cs.wisc.edu; da...@davemloft.net; > Hariprasad S; linux-s...@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: RE: [PATCH] csiostor:Fix locking issues in the function > csio_scsim_cleanup_io_lnode > > > -Original Message- > > From: Nicholas Krause [mailto:xerofo...@gmail.com] > > Sent: Wednesday, September 2, 2015 10:36 AM > > To: jbottom...@odin.com > > Cc: h...@suse.de; micha...@cs.wisc.edu; da...@davemloft.net; Anish > > Bhatt; Hariprasad S; linux-s...@vger.kernel.org; linux- > > ker...@vger.kernel.org > > Subject: [PATCH] csiostor:Fix locking issues in the function > > csio_scsim_cleanup_io_lnode > > > > This fixes locking issues in the function csio_scsim_cleanup_io_lnode > > by locking around the call to the function csio_csci_gather_active_ios > > with the function pair spin_lock_irq/spin_unlock_irq as any function > > calling this particular function must do in order to avoid concurrent > > threads of execution on the passed structure pointer of type csio_hw > > as this structure pointer can be shared across mutliple threads in the > kernel. > > > > Signed-off-by: Nicholas Krause <xerofo...@gmail.com> > > --- > > drivers/scsi/csiostor/csio_scsi.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/scsi/csiostor/csio_scsi.c > > b/drivers/scsi/csiostor/csio_scsi.c > > index 2c4562d..c318855 100644 > > --- a/drivers/scsi/csiostor/csio_scsi.c > > +++ b/drivers/scsi/csiostor/csio_scsi.c > > @@ -1327,7 +1327,9 @@ csio_scsim_cleanup_io_lnode(struct csio_scsim > > *scm, struct csio_lnode *ln) > > sld.level = CSIO_LEV_LNODE; > > sld.lnode = ln; > > INIT_LIST_HEAD(>cmpl_q); > > + spin_lock_irq(>lock); > > csio_scsi_gather_active_ios(scm, , >cmpl_q); > > + spin_unlock_irq(>lock); > > > > /* No I/Os pending on this lnode */ > > if (list_empty(>cmpl_q)) > > -- > > 2.1.4 > > Acked-By: Anish Bhatt <an...@chelsio.com> My bad, NACK. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH] csiostor:Fix locking issues in the function csio_scsim_cleanup_io_lnode
> -Original Message- > From: Nicholas Krause [mailto:xerofo...@gmail.com] > Sent: Wednesday, September 2, 2015 10:36 AM > To: jbottom...@odin.com > Cc: h...@suse.de; micha...@cs.wisc.edu; da...@davemloft.net; Anish > Bhatt; Hariprasad S; linux-s...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject: [PATCH] csiostor:Fix locking issues in the function > csio_scsim_cleanup_io_lnode > > This fixes locking issues in the function csio_scsim_cleanup_io_lnode by > locking around the call to the function csio_csci_gather_active_ios with the > function pair spin_lock_irq/spin_unlock_irq as any function calling this > particular function must do in order to avoid concurrent threads of execution > on the passed structure pointer of type csio_hw as this structure pointer can > be shared across mutliple threads in the kernel. > > Signed-off-by: Nicholas Krause > --- > drivers/scsi/csiostor/csio_scsi.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/scsi/csiostor/csio_scsi.c > b/drivers/scsi/csiostor/csio_scsi.c > index 2c4562d..c318855 100644 > --- a/drivers/scsi/csiostor/csio_scsi.c > +++ b/drivers/scsi/csiostor/csio_scsi.c > @@ -1327,7 +1327,9 @@ csio_scsim_cleanup_io_lnode(struct csio_scsim > *scm, struct csio_lnode *ln) > sld.level = CSIO_LEV_LNODE; > sld.lnode = ln; > INIT_LIST_HEAD(>cmpl_q); > + spin_lock_irq(>lock); > csio_scsi_gather_active_ios(scm, , >cmpl_q); > + spin_unlock_irq(>lock); > > /* No I/Os pending on this lnode */ > if (list_empty(>cmpl_q)) > -- > 2.1.4 Acked-By: Anish Bhatt -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH] csiostor:Fix locking issues in the function csio_scsim_cleanup_io_lnode
> -Original Message- > From: Nicholas Krause [mailto:xerofo...@gmail.com] > Sent: Wednesday, September 2, 2015 10:36 AM > To: jbottom...@odin.com > Cc: h...@suse.de; micha...@cs.wisc.edu; da...@davemloft.net; Anish > Bhatt; Hariprasad S; linux-s...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject: [PATCH] csiostor:Fix locking issues in the function > csio_scsim_cleanup_io_lnode > > This fixes locking issues in the function csio_scsim_cleanup_io_lnode by > locking around the call to the function csio_csci_gather_active_ios with the > function pair spin_lock_irq/spin_unlock_irq as any function calling this > particular function must do in order to avoid concurrent threads of execution > on the passed structure pointer of type csio_hw as this structure pointer can > be shared across mutliple threads in the kernel. > > Signed-off-by: Nicholas Krause> --- > drivers/scsi/csiostor/csio_scsi.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/scsi/csiostor/csio_scsi.c > b/drivers/scsi/csiostor/csio_scsi.c > index 2c4562d..c318855 100644 > --- a/drivers/scsi/csiostor/csio_scsi.c > +++ b/drivers/scsi/csiostor/csio_scsi.c > @@ -1327,7 +1327,9 @@ csio_scsim_cleanup_io_lnode(struct csio_scsim > *scm, struct csio_lnode *ln) > sld.level = CSIO_LEV_LNODE; > sld.lnode = ln; > INIT_LIST_HEAD(>cmpl_q); > + spin_lock_irq(>lock); > csio_scsi_gather_active_ios(scm, , >cmpl_q); > + spin_unlock_irq(>lock); > > /* No I/Os pending on this lnode */ > if (list_empty(>cmpl_q)) > -- > 2.1.4 Acked-By: Anish Bhatt -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/