mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [GIT PULL] target fixes for v3.4-rc7
@ 2012-05-11 21:52 Nicholas A. Bellinger
  2012-05-11 21:58 ` Nicholas A. Bellinger
  0 siblings, 1 reply; 2+ messages in thread
From: Nicholas A. Bellinger @ 2012-05-11 21:52 UTC (permalink / raw)
  To: Linus Torvalds; +Cc: target-devel, linux-scsi, LKML

Hi Linus !

This time around for -rc7, there is only a single outstanding target
pending bugfix ready to be merged..

Please go ahead and pull from:

  git://git.kernel.org/pub/scm/linux/kernel/git/nab/target-pending.git 3.4-urgent

This patch removes some incorrect legacy code to free se_lun_acl memory
in the NodeACL release path that could potentially trigger an OOPs
during shutdown once dynamic -> explicit initiator NodeACL conversion
has occurred.

That said, we've been able to trigger an OOPs in v4.0 code for this
special case when the associated MappedLUNs had not also been made
explicit based on active TPG LUN layout during the conversion, so it
really makes senses to go ahead and drop this extra cruft to avoid any
possible issues here.

This ends up only effecting iscsi-target module code (it's the only
user) and is CC'ed to stable.

Thanks!

--nab

Nicholas Bellinger (1):
  target: Drop incorrect se_lun_acl release for dynamic -> explict ACL
    conversion

 drivers/target/target_core_tpg.c |   21 ---------------------
 1 files changed, 0 insertions(+), 21 deletions(-)



^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [GIT PULL] target fixes for v3.4-rc7
  2012-05-11 21:52 [GIT PULL] target fixes for v3.4-rc7 Nicholas A. Bellinger
@ 2012-05-11 21:58 ` Nicholas A. Bellinger
  0 siblings, 0 replies; 2+ messages in thread
From: Nicholas A. Bellinger @ 2012-05-11 21:58 UTC (permalink / raw)
  To: Linus Torvalds; +Cc: target-devel, linux-scsi, LKML

On Fri, 2012-05-11 at 14:52 -0700, Nicholas A. Bellinger wrote:
> Hi Linus !
> 
> This time around for -rc7, there is only a single outstanding target
> pending bugfix ready to be merged..
> 
> Please go ahead and pull from:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/nab/target-pending.git 3.4-urgent
> 
> This patch removes some incorrect legacy code to free se_lun_acl memory
> in the NodeACL release path that could potentially trigger an OOPs
> during shutdown once dynamic -> explicit initiator NodeACL conversion
> has occurred.
> 
> That said, we've been able to trigger an OOPs in v4.0 code for this
> special case when the associated MappedLUNs had not also been made
> explicit based on active TPG LUN layout during the conversion, so it
> really makes senses to go ahead and drop this extra cruft to avoid any
> possible issues here.
> 
> This ends up only effecting iscsi-target module code (it's the only
> user) and is CC'ed to stable.
> 
> Thanks!
> 
> --nab
> 
> Nicholas Bellinger (1):
>   target: Drop incorrect se_lun_acl release for dynamic -> explict ACL
>     conversion
> 
>  drivers/target/target_core_tpg.c |   21 ---------------------
>  1 files changed, 0 insertions(+), 21 deletions(-)
> 

Whoops sorry, missed the now unused variables in this patch..

Here is the updated diff-stat for target-pending/3.4-urgent

 drivers/target/target_core_tpg.c |   22 ----------------------
 1 files changed, 0 insertions(+), 22 deletions(-)

Thanks,

--nab



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-05-11 21:58 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-05-11 21:52 [GIT PULL] target fixes for v3.4-rc7 Nicholas A. Bellinger
2012-05-11 21:58 ` Nicholas A. Bellinger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®