Message ID | 20210908164622.19240-1-njavali@marvell.com |
---|---|
Headers | show |
Series | qla2xxx driver bug fixes | expand |
Bart, > -----Original Message----- > From: Bart Van Assche <bvanassche@acm.org> > Sent: Wednesday, September 8, 2021 11:58 PM > To: Nilesh Javali <njavali@marvell.com>; martin.petersen@oracle.com > Cc: linux-scsi@vger.kernel.org; GR-QLogic-Storage-Upstream <GR-QLogic- > Storage-Upstream@marvell.com>; djeffery@redhat.com; > loberman@redhat.com > Subject: [EXT] Re: [PATCH v2 01/10] qla2xxx: Add support for mailbox > passthru > > External Email > > ---------------------------------------------------------------------- > On 9/8/21 9:46 AM, Nilesh Javali wrote: > > +struct qla_mbx_passthru { > > + uint16_t reserved1[2]; > > + uint16_t mbx_in[32]; > > + uint16_t mbx_out[32]; > > + uint32_t reserved2[16]; > > +} __packed; > > Why does this data structure start with 4 reserved bytes? Specifically, more than the driver, the reserved bytes would be used by the application, and driver uses only the mbx_in and mbx_out fields to access 32 register content. Thanks, Nilesh
Nilesh, > Please apply the qla2xxx driver bug fixes to the scsi tree at your > earliest convenience. Applied to 5.16/scsi-staging, thanks! -- Martin K. Petersen Oracle Linux Engineering
On Wed, 8 Sep 2021 09:46:12 -0700, Nilesh Javali wrote: > Martin, > > Please apply the qla2xxx driver bug fixes to the scsi tree at your > earliest convenience. > > v2: > - Added Fixes and Cc tags for few fixes > - Replace hb with heartbeat to make more readable > - Added Reviewed-by tag > > [...] Applied to 5.16/scsi-queue, thanks! [01/10] qla2xxx: Add support for mailbox passthru https://git.kernel.org/mkp/scsi/c/9e1c3206960f [02/10] qla2xxx: Display 16G only as supported speeds for 3830c card https://git.kernel.org/mkp/scsi/c/52cca50d35f8 [03/10] qla2xxx: Check for firmware capability before creating QPair https://git.kernel.org/mkp/scsi/c/8192817efbc3 [04/10] qla2xxx: Fix crash in NVME abort path https://git.kernel.org/mkp/scsi/c/e6e22e6cc296 [05/10] qla2xxx: edif: Use link event to wake up app https://git.kernel.org/mkp/scsi/c/527d46e0b014 [06/10] qla2xxx: Fix kernel crash when accessing port_speed sysfs file https://git.kernel.org/mkp/scsi/c/3ef68d4f0c9e [07/10] qla2xxx: Call process_response_queue() in Tx path https://git.kernel.org/mkp/scsi/c/38c61709e662 [08/10] qla2xxx: Move heart beat handling from dpc thread to workqueue https://git.kernel.org/mkp/scsi/c/3a4e1f3b3a3c [09/10] qla2xxx: Fix use after free in eh_abort path https://git.kernel.org/mkp/scsi/c/3d33b303d4f3 [10/10] qla2xxx: Update version to 10.02.07.100-k https://git.kernel.org/mkp/scsi/c/b0fe235dad77 -- Martin K. Petersen Oracle Linux Engineering