Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1294121

Re: [PATCHv6 5/7] x86/pci: Initial commit for new VMD device driver

Show all headers | View raw

On Thu, Dec 17, 2015 at 12:14:48PM -0600, Bjorn Helgaas wrote:
> On Mon, Dec 07, 2015 at 02:32:27PM -0700, Keith Busch wrote:
> > +/*
> > + * VMD h/w converts posted config writes to non-posted. The read-back in this
> > + * function forces the completion so it returns only after the config space was
> > + * written, as expected.
> 
> This comment sounds backwards:
> 
>   posted writes don't wait for completion
>   non-posted writes do wait for completion
> 
> If the hardware converts to non-posted writes, you shouldn't need a
> read-back.  It seems like you would need the read-back if the hardware
> converted non-posted to posted.

Oops, the comment has it backwards. Non-posted config writes become
posted memory write requests with this h/w.
 
> Seems like it might be nice to have something in dmesg that would connect
> this PCI device to the new PCI domain.  It's a new, unusual topology and a
> hint might help everybody understand what's going on.

Sounds good, will add.

In addition that, I'd like to mention this patch links a new domain's
root bus kobject under the VMD end-point's so the sysfs hierarchy captures
this as well.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCHv6 5/7] x86/pci: Initial commit for new VMD device driver Bjorn Helgaas <helgaas@kernel.org> - 2015-12-17 19:20 +0100
  Re: [PATCHv6 5/7] x86/pci: Initial commit for new VMD device driver Keith Busch <keith.busch@intel.com> - 2015-12-17 19:30 +0100

csiph-web