[PULL][SRU Bionic] hisi_sas bugfixes

Kleber Souza kleber.souza at canonical.com
Thu May 17 09:37:00 UTC 2018


On 05/16/18 21:13, dann frazier wrote:
> On Wed, May 16, 2018 at 4:13 AM, Kleber Souza
> <kleber.souza at canonical.com> wrote:
>> On 05/04/18 19:54, dann frazier wrote:
>>> BugLink: https://bugs.launchpad.net/bugs/1762824
>>>
>>> All clean cherry-picks from upstream, limited to the hisi_sas driver.
>>>
>>> The descriptions of 2 of the patches underplay the functional changes,
>>> so I'll elaborate here on why I'm including them. "scsi: hisi_sas:
>>> Code cleanup and minor bug fixes" because it corrects the interrupt
>>> handler return values. "scsi: hisi_sas: remove unused variable
>>> hisi_sas_devices.running_req" is an obviously correct change,
>>> and allows the driver to drop several atomic instructions.
>>>
>>> Tested on a HiSilicon D05 and a HiSilicon D06 board.
>>>
>>> The following changes since commit d4174baeefbd0a9634291145600b90e13163f27d:
>>>
>>>   UBUNTU: Ubuntu-4.15.0-20.21 (2018-04-23 23:56:56 -0500)
>>>
>>> are available in the Git repository at:
>>>
>>>   git://git.launchpad.net/~dannf/ubuntu/+source/linux/+git/linux hisi-sas-updates
>>>
>>> for you to fetch changes up to 2e3b262ef293485798233fc4fa53758d548e99d2:
>>>
>>>   scsi: hisi_sas: Code cleanup and minor bug fixes (2018-05-03 19:48:52 -0600)
>>
>> Hi Dann,
>>
>> I'm OK with the changes and I would give my ACK, however the BugLink
>> references to a bug that has already been used for a previous patch-set
>> submission and it has been marked as 'Fix Released'. In that case we
>> recommend opening a new bug so we can track better the progress of these
>> follow-up fixes.
> 
> Oops! The BugLink should be:
> 
>   BugLink: https://bugs.launchpad.net/bugs/1768974
> 
> This is correct in the individual commits - looks like just a cut &
> paste error in the PR e-mail.
> Thanks for reviewing Kleber - let me know if you need a new PR e-mail
> w/ this correction.
> 

Hi Dann,

Sorry, I didn't see the BugLink in the commits were different. That's
fine, no need to send a new PR.

Thanks!

>   -dann
> 
>>
>> Thanks,
>> Kleber
>>
>>>
>>> ----------------------------------------------------------------
>>> Xiang Chen (2):
>>>       scsi: hisi_sas: remove unused variable hisi_sas_devices.running_req
>>>       scsi: hisi_sas: Code cleanup and minor bug fixes
>>>
>>> Xiaofei Tan (6):
>>>       scsi: hisi_sas: dt-bindings: add an property of signal attenuation
>>>       scsi: hisi_sas: support the property of signal attenuation for v2 hw
>>>       scsi: hisi_sas: fix the issue of link rate inconsistency
>>>       scsi: hisi_sas: fix the issue of setting linkrate register
>>>       scsi: hisi_sas: increase timer expire of internal abort task
>>>       scsi: hisi_sas: fix return value of hisi_sas_task_prep()
>>>
>>>  .../devicetree/bindings/scsi/hisilicon-sas.txt     |  7 +++
>>>  drivers/scsi/hisi_sas/hisi_sas.h                   |  1 -
>>>  drivers/scsi/hisi_sas/hisi_sas_main.c              | 34 +++++-------
>>>  drivers/scsi/hisi_sas/hisi_sas_v1_hw.c             | 13 +++--
>>>  drivers/scsi/hisi_sas/hisi_sas_v2_hw.c             | 62 +++++++++++++++++-----
>>>  drivers/scsi/hisi_sas/hisi_sas_v3_hw.c             | 46 ++++++++--------
>>>  6 files changed, 96 insertions(+), 67 deletions(-)
>>>




More information about the kernel-team mailing list