[Ksummit-2012-discuss] PCI breakout session

Yinghai Lu yinghai at kernel.org
Mon Jun 18 18:19:07 UTC 2012

On Mon, Jun 18, 2012 at 10:39 AM, Bjorn Helgaas <bhelgaas at google.com> wrote:
> You mean like "en1k" (currently badly implemented because it's a quirk
> and can't work with any reassignment)?

do you mean


| [PATCH v1 8/9] PCI: reimplement P2P bridge 1K I/O windows (Intel P64H2)
| 9d265124d051 and 15a260d53f7c added quirks for P2P bridges that support
| I/O windows that start/end at 1K boundaries, not just the 4K boundaries
| defined by the PCI spec.  For details, see the IOBL_ADR register and the
| EN1K bit in the CNF register in the Intel 82870P2 (P64H2).
| These quirks complicate the code that reads P2P bridge windows
| (pci_read_bridge_io() and pci_cfg_fake_ranges()) because the bridge
| I/O resource is updated in the HEADER quirk, in pci_read_bridge_io(),
| in pci_setup_bridge(), and again in the FINAL quirk.  This is confusing
| and makes it impossible to reassign the bridge windows after FINAL
| quirks are run.
| This patch adds support for 1K windows in the generic paths, so the
| HEADER quirk only has to enable this support.  The FINAL quirk, which
| used to undo damage done by pci_setup_bridge(), is no longer needed.
| Note that the bridge window assignment code, e.g., pbus_size_io(), should
| pay attention to dev->io_window_1k, too, but I didn't fix that.

and attached one should address io_window_ik checking in pbus_size_io()

-------------- next part --------------
A non-text attachment was scrubbed...
Name: io_window_1k.patch
Type: application/octet-stream
Size: 2141 bytes
Desc: not available
URL: <http://lists.linux-foundation.org/pipermail/ksummit-2012-discuss/attachments/20120618/8b69d6a2/attachment.obj>

More information about the Ksummit-2012-discuss mailing list