group.of.nepali.translators team mailing list archive

[Joseph Salisbury <joseph.salisbury@xxxxxxxxxxxxx>]

** Also affects: linux (Ubuntu Artful)
   Importance: High
     Assignee: Joseph Salisbury (jsalisbury)
       Status: In Progress

** Changed in: linux (Ubuntu Artful)
       Status: In Progress => Fix Committed

-- 
You received this bug notification because you are a member of नेपाली
भाषा समायोजकहरुको समूह, which is subscribed to Xenial.
Matching subscriptions: Ubuntu 16.04 Bugs
https://bugs.launchpad.net/bugs/1684971

Title:
  [Hyper-V][SAUCE] pci-hyperv: Use only 16 bit integer for PCI domain

Status in linux package in Ubuntu:
  Fix Committed
Status in linux source package in Xenial:
  In Progress
Status in linux source package in Yakkety:
  In Progress
Status in linux source package in Zesty:
  In Progress
Status in linux source package in Artful:
  Fix Committed

Bug description:
  The following patch fixes a problem with "[PATCH] pci-hyperv: Use
  device serial number as PCI domain" where some drivers were expecting
  a u16 instead of a u32 for PCI device serial numbers, as observed by
  Oops and hangs in Azure on NC and NV GPU instances.

  From: Haiyang Zhang <haiyangz@xxxxxxxxxxxxx>

  This patch uses the lower 16 bits of the serial number as PCI
  domain, otherwise some drivers may not be able to handle it.

  Signed-off-by: Haiyang Zhang <haiyangz@xxxxxxxxxxxxx>
  ---
   drivers/pci/host/pci-hyperv.c |    4 +++-
   1 files changed, 3 insertions(+), 1 deletions(-)

  diff --git a/drivers/pci/host/pci-hyperv.c b/drivers/pci/host/pci-hyperv.c
  index e73880c..b18dff3 100644
  --- a/drivers/pci/host/pci-hyperv.c
  +++ b/drivers/pci/host/pci-hyperv.c
  @@ -1334,9 +1334,11 @@ static void put_pcichild(struct hv_pci_dev *hpdev,
   	 * can have shorter names than based on the bus instance UUID.
   	 * Only the first device serial number is used for domain, so the
   	 * domain number will not change after the first device is added.
  +	 * The lower 16 bits of the serial number is used, otherwise some
  +	 * drivers may not be able to handle it.
   	 */
   	if (list_empty(&hbus->children))
  -		hbus->sysdata.domain = desc->ser;
  +		hbus->sysdata.domain = desc->ser & 0xFFFF;
   	list_add_tail(&hpdev->list_entry, &hbus->children);
   	spin_unlock_irqrestore(&hbus->device_list_lock, flags);
   	return hpdev;
  -- 
  1.7.1

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1684971/+subscriptions