All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jiang Liu <liuj97@gmail.com>
To: Bjorn Helgaas <bhelgaas@google.com>
Cc: ksummit-2012-discuss
	<ksummit-2012-discuss@lists.linux-foundation.org>,
	"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>
Subject: Re: PCI mini-summit notes
Date: Wed, 29 Aug 2012 23:48:46 +0800	[thread overview]
Message-ID: <503E39DE.7000109@gmail.com> (raw)
In-Reply-To: <CAErSpo73r3yzzndUek-NJ3d0pDKtg8zTrnjk4ihudBE5gngBAA@mail.gmail.com>

On 08/29/2012 03:28 PM, Bjorn Helgaas wrote:
> We held a PCI mini-summit on Aug 28 in San Diego in conjunction with
> Kernel Summit and the Linux Plumbers Conference.  I want to thank
> everybody who participated.  We had a good discussion and I really
> appreciate all the input and ideas everybody provided.
> 
> My summary of the major discussions we had is below.
> 
> Bjorn
> 
> 
> 
> Host bridge hotplug
> 
>     There's a lot of interest in this functionality, mostly on x86 using
>     ACPI-mediated hotplug.
> 
>     The acpiphp driver handles both host bridge and PCI device hotplug.  We
>     believe these should be separated.
> 
>     Host bridge hotplug requires IOAPIC and DMAR hotplug with proper sequencing
>     (started before PCI enumeration and removed after PCI drivers are removed).
>     On x86, we think this should happen naturally if we add this support to the
>     ACPI pci_root.c driver.  We do need some tweaks to x86 IOAPIC init and
>     IOMMU drivers.
> 
>     We'd like a sysfs interface to this, and it's not clear what form
>     it should take.  One way is to add hooks in the PCI side, e.g.,
>     /sys/devices/.../pci_bus/remove.  This has the advantage of looking the
>     same across all architectures, but it doesn't map well to firmware
>     interfaces and it's not obvious how to deal with hot-adds, when the pci_bus
>     doesn't exist yet.  Another way would be to have them connected to the host
>     bridge and its enclosing scope, e.g., /sys/devices/.../PNP0A08:00/remove
>     and /sys/devices/.../LNXSYBUS:00/rescan.  This is architecture-specific but
>     has the advantage of matching the logical system topology.
> 
> Hot Plug Issues
> 
>     We know we have locking issues and races in the PCI device hotplug area.
>     We have some pending patches to address these.  They may be merged for 3.7
>     or 3.8.
> 
>     We still have some device setup being done by initcalls, and obviously this
>     doesn't work for hot-added devices.  We've fixed some of these areas, but
>     there are a few more to do.
> 
>     What about CONFIG_HOTPLUG?  We didn't discuss this in the mini-summit,
>     but it was raised on the ksummit-discuss list.
> 
> SR-IOV Management
> 
>     Currently drivers implement module parameters like "max_vfs".  This means
>     all devices claimed by the driver get the same number of VFs, and you can't
>     change anything without unloading and reloading the driver.
> 
>     Consensus that we should try to implement a knob for this in sysfs so it
>     can be generic (not in each driver) and set individually for each device.
Hi Bjorn,
	One of my team member reported another corner case for SR-IOV. There's
are two NIC cards in the system driven by the same driver, but one supports
SR-IOV and the other doesn't. It runs into trouble if "max_vfs" parameter is
set for the NIC driver.
	--Gerry



  reply	other threads:[~2012-08-29 15:48 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-29  7:28 PCI mini-summit notes Bjorn Helgaas
2012-08-29 15:48 ` Jiang Liu [this message]
2012-08-29 22:44   ` Jon Mason
2012-08-29 22:54     ` Yinghai Lu
2012-08-30  6:02       ` Bjorn Helgaas
2012-09-05  1:33         ` Don Dutile
2012-09-05  1:41           ` Don Dutile
2012-09-03  8:51       ` Ram Pai

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=503E39DE.7000109@gmail.com \
    --to=liuj97@gmail.com \
    --cc=bhelgaas@google.com \
    --cc=ksummit-2012-discuss@lists.linux-foundation.org \
    --cc=linux-pci@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.