Message ID | 20240307215717.14585-1-michael.reed@canonical.com |
---|---|
Headers | show |
Series | 22.04.04: megaraid_sas: Critical Bug Fixes | expand |
On 07.03.24 22:57, Michael Reed wrote: > From: Michael Reed <Michael.Reed@canonical.com> > > BugLink: https://bugs.launchpad.net/bugs/2046722 > > SRU Justification: > > [Impact] > > This update has been initiated to include a few critical bug fixes from upstream into the upcoming 22.04.04 point kernel. Below are the upstream commit IDs: > > [Fix] > 0938f9fa4208 scsi: megaraid_sas: Driver version update to 07.727.03.00-rc1 > 2d83fb023c90 scsi: megaraid_sas: Log message when controller reset is requested but not issued > 8e3ed9e78651 scsi: megaraid_sas: Increase register read retry count from 3 to 30 for selected registers > 0b0747d507bf scsi: megaraid_sas: Fix deadlock on firmware crashdump > 2309df27111a scsi: megaraid_sas: Fix crash after a double completion > 0808ed6ebbc2 scsi: megaraid_sas: Fix fw_crash_buffer_show() > > The change log for the above commit IDs is small and doesn't require rigorous validation. So please include these patches. > > [Test Plan] > > 1. Driver load and unload > 2. Create and delete different virtual drives (R0, R1, R5 etc.,) > 3. create JBOD > 4. Run IOs with different profiles (io_depth, io_size etc.,) on VD and JBOD > 5. controller reset while running IOs > > [ Where problems could occur ] > It has a critical patch which fixes the issue related to various register reads. So IO runs and various internal commands where problems could occur > [ Other Info ] > > Jammy > https://code.launchpad.net/~mreed8855/ubuntu/+source/linux/+git/jammy/+ref/lp_2046722_megaraid_sas_update > > Mantic > https://code.launchpad.net/~mreed8855/ubuntu/+source/linux/+git/mantic/+ref/lp_2046722_megaraid_sas_update > > Noble > These patches are already in Noble > > Chandrakanth patil (2): > scsi: megaraid_sas: Log message when controller reset is requested but > not issued > scsi: megaraid_sas: Driver version update to 07.727.03.00-rc1 > > drivers/scsi/megaraid/megaraid_sas.h | 4 ++-- > drivers/scsi/megaraid/megaraid_sas_fusion.c | 3 +++ > 2 files changed, 5 insertions(+), 2 deletions(-) > I am tending to NACK this submission but do not immediately do so. However this looks rather confusing and possible dangerous on a maintenance level. The bug report and some information in the cover indicates that there are updates for Jammy and Mantic in progress. But this very much invisible in the submission. The pull requests are hidden in the cover email and rather hard to spot. The more obvious part are the 2 patches that are wanted for N/M/J. I would bet that you would end up with just those 2 applied everywhere. And then have the odd state of using a version which does not represent function level. IMO this would be better arranged as: [SRU N/M/J] Cover +- [SRU N][PATCH 1/2]... +- [SRU N][PATCH 2/2]... +- [SRU M][PULL]... +- [SRU J][PULL]...
Hi Stefan, Go ahead and NACK this patch set and I will rearrange it based on your suggestion and resubmit it as version 2. - Michael On Wed, Mar 13, 2024 at 4:05 AM Stefan Bader <stefan.bader@canonical.com> wrote: > On 07.03.24 22:57, Michael Reed wrote: > > From: Michael Reed <Michael.Reed@canonical.com> > > > > BugLink: https://bugs.launchpad.net/bugs/2046722 > > > > SRU Justification: > > > > [Impact] > > > > This update has been initiated to include a few critical bug fixes from > upstream into the upcoming 22.04.04 point kernel. Below are the upstream > commit IDs: > > > > [Fix] > > 0938f9fa4208 scsi: megaraid_sas: Driver version update to > 07.727.03.00-rc1 > > 2d83fb023c90 scsi: megaraid_sas: Log message when controller reset is > requested but not issued > > 8e3ed9e78651 scsi: megaraid_sas: Increase register read retry count from > 3 to 30 for selected registers > > 0b0747d507bf scsi: megaraid_sas: Fix deadlock on firmware crashdump > > 2309df27111a scsi: megaraid_sas: Fix crash after a double completion > > 0808ed6ebbc2 scsi: megaraid_sas: Fix fw_crash_buffer_show() > > > > The change log for the above commit IDs is small and doesn't require > rigorous validation. So please include these patches. > > > > [Test Plan] > > > > 1. Driver load and unload > > 2. Create and delete different virtual drives (R0, R1, R5 etc.,) > > 3. create JBOD > > 4. Run IOs with different profiles (io_depth, io_size etc.,) on VD and > JBOD > > 5. controller reset while running IOs > > > > [ Where problems could occur ] > > It has a critical patch which fixes the issue related to various > register reads. So IO runs and various internal commands where problems > could occur > > [ Other Info ] > > > > Jammy > > > https://code.launchpad.net/~mreed8855/ubuntu/+source/linux/+git/jammy/+ref/lp_2046722_megaraid_sas_update > > > > Mantic > > > https://code.launchpad.net/~mreed8855/ubuntu/+source/linux/+git/mantic/+ref/lp_2046722_megaraid_sas_update > > > > Noble > > These patches are already in Noble > > > > Chandrakanth patil (2): > > scsi: megaraid_sas: Log message when controller reset is requested but > > not issued > > scsi: megaraid_sas: Driver version update to 07.727.03.00-rc1 > > > > drivers/scsi/megaraid/megaraid_sas.h | 4 ++-- > > drivers/scsi/megaraid/megaraid_sas_fusion.c | 3 +++ > > 2 files changed, 5 insertions(+), 2 deletions(-) > > > > I am tending to NACK this submission but do not immediately do so. > However this looks rather confusing and possible dangerous on a > maintenance level. The bug report and some information in the cover > indicates that there are updates for Jammy and Mantic in progress. But > this very much invisible in the submission. The pull requests are hidden > in the cover email and rather hard to spot. The more obvious part are > the 2 patches that are wanted for N/M/J. I would bet that you would end > up with just those 2 applied everywhere. And then have the odd state of > using a version which does not represent function level. > IMO this would be better arranged as: > > [SRU N/M/J] Cover > +- [SRU N][PATCH 1/2]... > +- [SRU N][PATCH 2/2]... > +- [SRU M][PULL]... > +- [SRU J][PULL]... > > -- > - Stefan > >
On 07.03.24 22:57, Michael Reed wrote: > From: Michael Reed <Michael.Reed@canonical.com> > > BugLink: https://bugs.launchpad.net/bugs/2046722 > > SRU Justification: > > [Impact] > > This update has been initiated to include a few critical bug fixes from upstream into the upcoming 22.04.04 point kernel. Below are the upstream commit IDs: > > [Fix] > 0938f9fa4208 scsi: megaraid_sas: Driver version update to 07.727.03.00-rc1 > 2d83fb023c90 scsi: megaraid_sas: Log message when controller reset is requested but not issued > 8e3ed9e78651 scsi: megaraid_sas: Increase register read retry count from 3 to 30 for selected registers > 0b0747d507bf scsi: megaraid_sas: Fix deadlock on firmware crashdump > 2309df27111a scsi: megaraid_sas: Fix crash after a double completion > 0808ed6ebbc2 scsi: megaraid_sas: Fix fw_crash_buffer_show() > > The change log for the above commit IDs is small and doesn't require rigorous validation. So please include these patches. > > [Test Plan] > > 1. Driver load and unload > 2. Create and delete different virtual drives (R0, R1, R5 etc.,) > 3. create JBOD > 4. Run IOs with different profiles (io_depth, io_size etc.,) on VD and JBOD > 5. controller reset while running IOs > > [ Where problems could occur ] > It has a critical patch which fixes the issue related to various register reads. So IO runs and various internal commands where problems could occur > [ Other Info ] > > Jammy > https://code.launchpad.net/~mreed8855/ubuntu/+source/linux/+git/jammy/+ref/lp_2046722_megaraid_sas_update > > Mantic > https://code.launchpad.net/~mreed8855/ubuntu/+source/linux/+git/mantic/+ref/lp_2046722_megaraid_sas_update > > Noble > These patches are already in Noble > > Chandrakanth patil (2): > scsi: megaraid_sas: Log message when controller reset is requested but > not issued > scsi: megaraid_sas: Driver version update to 07.727.03.00-rc1 > > drivers/scsi/megaraid/megaraid_sas.h | 4 ++-- > drivers/scsi/megaraid/megaraid_sas_fusion.c | 3 +++ > 2 files changed, 5 insertions(+), 2 deletions(-) > Rejected for the following reasons: - the submission is not clear about requirements for mantic and jammy those require further patches which are contained in the cover email as pull requests - suggest to target noble with individual patches but include mantic and jammy with explicit pull request mails [SRU N/M/J] Cover +- [SRU N][PATCH 1/2]... +- [SRU N][PATCH 2/2]... +- [SRU M][PULL]... +- [SRU J][PULL]... -Stefan
From: Michael Reed <Michael.Reed@canonical.com> BugLink: https://bugs.launchpad.net/bugs/2046722 SRU Justification: [Impact] This update has been initiated to include a few critical bug fixes from upstream into the upcoming 22.04.04 point kernel. Below are the upstream commit IDs: [Fix] 0938f9fa4208 scsi: megaraid_sas: Driver version update to 07.727.03.00-rc1 2d83fb023c90 scsi: megaraid_sas: Log message when controller reset is requested but not issued 8e3ed9e78651 scsi: megaraid_sas: Increase register read retry count from 3 to 30 for selected registers 0b0747d507bf scsi: megaraid_sas: Fix deadlock on firmware crashdump 2309df27111a scsi: megaraid_sas: Fix crash after a double completion 0808ed6ebbc2 scsi: megaraid_sas: Fix fw_crash_buffer_show() The change log for the above commit IDs is small and doesn't require rigorous validation. So please include these patches. [Test Plan] 1. Driver load and unload 2. Create and delete different virtual drives (R0, R1, R5 etc.,) 3. create JBOD 4. Run IOs with different profiles (io_depth, io_size etc.,) on VD and JBOD 5. controller reset while running IOs [ Where problems could occur ] It has a critical patch which fixes the issue related to various register reads. So IO runs and various internal commands where problems could occur [ Other Info ] Jammy https://code.launchpad.net/~mreed8855/ubuntu/+source/linux/+git/jammy/+ref/lp_2046722_megaraid_sas_update Mantic https://code.launchpad.net/~mreed8855/ubuntu/+source/linux/+git/mantic/+ref/lp_2046722_megaraid_sas_update Noble These patches are already in Noble Chandrakanth patil (2): scsi: megaraid_sas: Log message when controller reset is requested but not issued scsi: megaraid_sas: Driver version update to 07.727.03.00-rc1 drivers/scsi/megaraid/megaraid_sas.h | 4 ++-- drivers/scsi/megaraid/megaraid_sas_fusion.c | 3 +++ 2 files changed, 5 insertions(+), 2 deletions(-)