mbox series

[0/3] Three reliability improvements

Message ID 20191112035752.8338-1-bvanassche@acm.org (mailing list archive)
Headers show
Series Three reliability improvements | expand

Message

Bart Van Assche Nov. 12, 2019, 3:57 a.m. UTC
Hi Martin,

The three patches in this series fix the complaints that were reported while
running the libiscsi test tool against the SCSI target code. I'm not sure why
I hadn't noticed these issues before - maybe new tests have been added to the
test tool. Please consider these patches for Linux kernel version v5.5.

Thanks,

Bart.

Bart Van Assche (3):
  target/core: Release SPC-2 reservations when closing a session
  target/core: Fix a use-after-free in the TMF handling code
  target/iscsi: Wait for all commands to finish before freeing a session

 drivers/target/iscsi/iscsi_target.c    | 10 ++++++++--
 drivers/target/target_core_transport.c | 21 +++++++++++++++++++++
 2 files changed, 29 insertions(+), 2 deletions(-)

Comments

Bart Van Assche Nov. 12, 2019, 5:05 p.m. UTC | #1
On 11/11/19 7:57 PM, Bart Van Assche wrote:
> The three patches in this series fix the complaints that were reported while
> running the libiscsi test tool against the SCSI target code. I'm not sure why
> I hadn't noticed these issues before - maybe new tests have been added to the
> test tool. Please consider these patches for Linux kernel version v5.5.

This is the second version of this patch series. The changes compared to 
v1 are:
- The patch that declares target_release_reservation() static has been 
left out because a patch in this series introduces a call to that 
function outside the file in which that function has been defined.
- The patch for releasing SPC-2 reservations has been reworked as 
requested by Roman.
- Two new patches have been added.

Bart.
Martin K. Petersen Nov. 13, 2019, 1:15 a.m. UTC | #2
Bart,

> - The patch that declares target_release_reservation() static has been
> left out because a patch in this series introduces a call to that
> function outside the file in which that function has been defined.

OK, I'll drop it again. Thought this series replaced patch two of the
first.