mbox series

[SRU,F,0/2] Support VMD in Tiger Lake

Message ID 20200416085659.94602-1-vicamo.yang@canonical.com
Headers show
Series Support VMD in Tiger Lake | expand

Message

You-Sheng Yang April 16, 2020, 8:56 a.m. UTC 
BugLink: https://bugs.launchpad.net/bugs/1855954

[Impact]
On platforms with VMD support, block devices won't be scanned if VMD
controller is not previously probed.

[Fix]
Two commits from v5.5-rc1 to be backported.

[Test Case]
On platforms with VMD support, boot with prebuilt kernel and check if
Intel Volume Management Device appears among PCI devices:

  $ lspci | grep 'Volume Management Device'
  0000:00:0e.0 RAID bus controller: Intel Corporation Volume Management
Device NVMe RAID Controller

[Regression Potential]
Low. This only enables VMD controller on platforms with such device.

Jon Derrick (2):
  PCI: vmd: Add bus 224-255 restriction decode
  PCI: vmd: Add device id for VMD device 8086:9A0B

 drivers/pci/controller/vmd.c | 32 ++++++++++++++++++++++++--------
 include/linux/pci_ids.h      |  1 +
 2 files changed, 25 insertions(+), 8 deletions(-)

Comments

Anthony Wong April 16, 2020, 4:26 p.m. UTC | #1 
On Thu, Apr 16, 2020 at 04:56:57PM +0800, You-Sheng Yang wrote:
> BugLink: https://bugs.launchpad.net/bugs/1855954
> 
> [Impact]
> On platforms with VMD support, block devices won't be scanned if VMD
> controller is not previously probed.
> 
> [Fix]
> Two commits from v5.5-rc1 to be backported.
> 
> [Test Case]
> On platforms with VMD support, boot with prebuilt kernel and check if
> Intel Volume Management Device appears among PCI devices:
> 
>   $ lspci | grep 'Volume Management Device'
>   0000:00:0e.0 RAID bus controller: Intel Corporation Volume Management
> Device NVMe RAID Controller
> 
> [Regression Potential]
> Low. This only enables VMD controller on platforms with such device.
> 
> Jon Derrick (2):
>   PCI: vmd: Add bus 224-255 restriction decode
>   PCI: vmd: Add device id for VMD device 8086:9A0B
> 
>  drivers/pci/controller/vmd.c | 32 ++++++++++++++++++++++++--------
>  include/linux/pci_ids.h      |  1 +
>  2 files changed, 25 insertions(+), 8 deletions(-)

Clean cherry-picks.
Acked-by: Anthony Wong <anthony.wong@canonical.com>

These patches are important for Ubuntu installation on VMD enabled
systems. Can we include them before Focal release?
Po-Hsu Lin April 17, 2020, 3:19 a.m. UTC | #2 
Clean cherry-pick with positive test result.
Acked-by: Po-Hsu Lin <po-hsu.lin@canonical.com>
Seth Forshee April 17, 2020, 3:10 p.m. UTC | #3 
On Thu, Apr 16, 2020 at 04:56:57PM +0800, You-Sheng Yang wrote:
> BugLink: https://bugs.launchpad.net/bugs/1855954
> 
> [Impact]
> On platforms with VMD support, block devices won't be scanned if VMD
> controller is not previously probed.
> 
> [Fix]
> Two commits from v5.5-rc1 to be backported.
> 
> [Test Case]
> On platforms with VMD support, boot with prebuilt kernel and check if
> Intel Volume Management Device appears among PCI devices:
> 
>   $ lspci | grep 'Volume Management Device'
>   0000:00:0e.0 RAID bus controller: Intel Corporation Volume Management
> Device NVMe RAID Controller
> 
> [Regression Potential]
> Low. This only enables VMD controller on platforms with such device.

Applied to focal/master-next, thanks!