mbox series

[SRU,F:linux-bluefield,v2,0/2] UBUNTU: SAUCE: ipmb_dev_int.c: sync up with upstream

Message ID 20220705143715.23618-1-asmaa@nvidia.com
Headers show
Series UBUNTU: SAUCE: ipmb_dev_int.c: sync up with upstream | expand

Message

Asmaa Mnebhi July 5, 2022, 2:37 p.m. UTC
Buglink: https://bugs.launchpad.net/bugs/1980525

SRU Justification:

[Impact]

Sync up the ipmb_dev_int.c with the latest version available in the upstream master branch.

[Fix]

* cherry-pick fc26067c7417e7fafed7bcc97bda155d91988734
  ipmi: remove open coded version of SMBus block write
* backport ipmb_dev_int.c related changes from
  commit 0924c5a0cbed5831a2a0ff2f9a19692265f1ab92
  The above commit makes modifications to 3 files:
  ipmb_dev_int.c, ipmi_ipmb.c and ipmi_ssif.c
  ipmi_ipmb.c doesn't exist on 5.4, so we just pull the change targeting ipmb_dev_int.c

[Test Case]

* check ipmb_dev_int is loaded without failure (check dmesg and lsmod)
* verify that /dev/ipmb-2 is created
* run ipmi command from bmc to send requests to BF DPU and verify that the response is as expected

[Regression Potential]

Any of the test cases above could be impacted due to these changes.

Comments

Tim Gardner July 5, 2022, 2:50 p.m. UTC | #1
On 7/5/22 08:37, Asmaa Mnebhi wrote:
> Buglink: https://bugs.launchpad.net/bugs/1980525
> 
> SRU Justification:
> 
> [Impact]
> 
> Sync up the ipmb_dev_int.c with the latest version available in the upstream master branch.
> 
> [Fix]
> 
> * cherry-pick fc26067c7417e7fafed7bcc97bda155d91988734
>    ipmi: remove open coded version of SMBus block write
> * backport ipmb_dev_int.c related changes from
>    commit 0924c5a0cbed5831a2a0ff2f9a19692265f1ab92
>    The above commit makes modifications to 3 files:
>    ipmb_dev_int.c, ipmi_ipmb.c and ipmi_ssif.c
>    ipmi_ipmb.c doesn't exist on 5.4, so we just pull the change targeting ipmb_dev_int.c
> 
> [Test Case]
> 
> * check ipmb_dev_int is loaded without failure (check dmesg and lsmod)
> * verify that /dev/ipmb-2 is created
> * run ipmi command from bmc to send requests to BF DPU and verify that the response is as expected
> 
> [Regression Potential]
> 
> Any of the test cases above could be impacted due to these changes.
> 
Acked-by: Tim Gardner <tim.gardner@canonical.com>
Zachary Tahenakos July 5, 2022, 7:06 p.m. UTC | #2
Acked-by: Zachary Tahenakos <zachary.tahenakos@canonical.com>

On 7/5/22 10:37 AM, Asmaa Mnebhi wrote:
> Buglink: https://bugs.launchpad.net/bugs/1980525
>
> SRU Justification:
>
> [Impact]
>
> Sync up the ipmb_dev_int.c with the latest version available in the upstream master branch.
>
> [Fix]
>
> * cherry-pick fc26067c7417e7fafed7bcc97bda155d91988734
>    ipmi: remove open coded version of SMBus block write
> * backport ipmb_dev_int.c related changes from
>    commit 0924c5a0cbed5831a2a0ff2f9a19692265f1ab92
>    The above commit makes modifications to 3 files:
>    ipmb_dev_int.c, ipmi_ipmb.c and ipmi_ssif.c
>    ipmi_ipmb.c doesn't exist on 5.4, so we just pull the change targeting ipmb_dev_int.c
>
> [Test Case]
>
> * check ipmb_dev_int is loaded without failure (check dmesg and lsmod)
> * verify that /dev/ipmb-2 is created
> * run ipmi command from bmc to send requests to BF DPU and verify that the response is as expected
>
> [Regression Potential]
>
> Any of the test cases above could be impacted due to these changes.
>
Zachary Tahenakos July 27, 2022, 12:41 p.m. UTC | #3
Applied to f:bluefield. Thanks.

-Zack

On Tue, Jul 5, 2022 at 10:37 AM Asmaa Mnebhi <asmaa@nvidia.com> wrote:

> Buglink: https://bugs.launchpad.net/bugs/1980525
>
> SRU Justification:
>
> [Impact]
>
> Sync up the ipmb_dev_int.c with the latest version available in the
> upstream master branch.
>
> [Fix]
>
> * cherry-pick fc26067c7417e7fafed7bcc97bda155d91988734
>   ipmi: remove open coded version of SMBus block write
> * backport ipmb_dev_int.c related changes from
>   commit 0924c5a0cbed5831a2a0ff2f9a19692265f1ab92
>   The above commit makes modifications to 3 files:
>   ipmb_dev_int.c, ipmi_ipmb.c and ipmi_ssif.c
>   ipmi_ipmb.c doesn't exist on 5.4, so we just pull the change targeting
> ipmb_dev_int.c
>
> [Test Case]
>
> * check ipmb_dev_int is loaded without failure (check dmesg and lsmod)
> * verify that /dev/ipmb-2 is created
> * run ipmi command from bmc to send requests to BF DPU and verify that the
> response is as expected
>
> [Regression Potential]
>
> Any of the test cases above could be impacted due to these changes.
>
> --
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
>