mbox series

[0/2] scsi: target: iscsi: Fix two protocol issues

Message ID cover.1701540918.git.lduncan@suse.com
Headers show
Series scsi: target: iscsi: Fix two protocol issues | expand

Message

Lee Duncan Dec. 7, 2023, 5:42 p.m. UTC 
From: Lee Duncan <lduncan@suse.com>

Recent boot testing using the fastlinq qedi iSCSI Converged Network
Adapter and an LIO target uncovered a couple of issues with the
kernel iSCSI target driver. The first patch addresses an issue
with the handling of iSCSI "immediate commands", which are allowed
but not common, and the second patch lowers the noise level of the
driver when initiators send PDUs with the read and/or write bits
set but no data, which is also allowed. (See RFC 3720)

Lee Duncan (2):
  scsi: target: iscsi: handle SCSI immediate commands
  scsi: target: iscsi: don't warn of R/W when no data

 drivers/target/iscsi/iscsi_target.c      | 17 ++++++-----------
 drivers/target/iscsi/iscsi_target_util.c | 10 ++++++++--
 2 files changed, 14 insertions(+), 13 deletions(-)

Comments

Lee Duncan Jan. 26, 2024, 5:42 p.m. UTC | #1 
Ping.

I believe I've addressed Chris' issues. Chris?

On Sat, Dec 2, 2023 at 10:45 AM lduncan@suse.com <lduncan@suse.com> wrote:
>
> From: Lee Duncan <lduncan@suse.com>
>
> Recent boot testing using the fastlinq qedi iSCSI Converged Network
> Adapter and an LIO target uncovered a couple of issues with the
> kernel iSCSI target driver. The first patch addresses an issue
> with the handling of iSCSI "immediate commands", which are allowed
> but not common, and the second patch lowers the noise level of the
> driver when initiators send PDUs with the read and/or write bits
> set but no data, which is also allowed. (See RFC 3720)
>
> Lee Duncan (2):
>   scsi: target: iscsi: handle SCSI immediate commands
>   scsi: target: iscsi: don't warn of R/W when no data
>
>  drivers/target/iscsi/iscsi_target.c      | 17 ++++++-----------
>  drivers/target/iscsi/iscsi_target_util.c | 10 ++++++++--
>  2 files changed, 14 insertions(+), 13 deletions(-)
>
> --
> 2.43.0
>