Re: [PATCH] docs: add zpci information to pci-addresses.rst

2020-04-16 Thread Boris Fiuczynski
On 4/16/20 2:21 PM, Andrea Bolognani wrote: On Thu, 2020-04-16 at 11:46 +0200, Boris Fiuczynski wrote: On 4/15/20 6:38 PM, Andrea Bolognani wrote: The idea behind it was to show users that they shouldn't expect the address in the domain XML to match the one in the guest OS, with a few

Re: [PATCH] docs: add zpci information to pci-addresses.rst

2020-04-16 Thread Andrea Bolognani
On Thu, 2020-04-16 at 11:46 +0200, Boris Fiuczynski wrote: > On 4/15/20 6:38 PM, Andrea Bolognani wrote: > > The idea behind it was to show users that they shouldn't expect the > > address in the domain XML to match the one in the guest OS, with a > > few real-life examples to illustrate the

Re: [PATCH] docs: add zpci information to pci-addresses.rst

2020-04-16 Thread Boris Fiuczynski
On 4/15/20 6:38 PM, Andrea Bolognani wrote: On Wed, 2020-04-15 at 16:45 +0200, Cornelia Huck wrote: On Wed, 15 Apr 2020 16:23:46 +0200 Boris Fiuczynski wrote: Once I understand you confusion above I will provide a patch... I'd say just go ahead. While I appreciate the sentiment, I would

Re: [PATCH] docs: add zpci information to pci-addresses.rst

2020-04-15 Thread Andrea Bolognani
On Wed, 2020-04-15 at 12:51 +0200, Cornelia Huck wrote: > Add some information on how pci address work on s390x. > > Signed-off-by: Cornelia Huck > --- > docs/pci-addresses.rst | 63 ++ > 1 file changed, 63 insertions(+) There are a few comments that I

Re: [PATCH] docs: add zpci information to pci-addresses.rst

2020-04-15 Thread Cornelia Huck
On Wed, 15 Apr 2020 18:38:06 +0200 Andrea Bolognani wrote: > On Wed, 2020-04-15 at 16:45 +0200, Cornelia Huck wrote: > > On Wed, 15 Apr 2020 16:23:46 +0200 > > Boris Fiuczynski wrote: > > > Once I understand you confusion above I will provide a patch... > > > > I'd say just go ahead. >

Re: [PATCH] docs: add zpci information to pci-addresses.rst

2020-04-15 Thread Andrea Bolognani
On Wed, 2020-04-15 at 16:45 +0200, Cornelia Huck wrote: > On Wed, 15 Apr 2020 16:23:46 +0200 > Boris Fiuczynski wrote: > > Once I understand you confusion above I will provide a patch... > > I'd say just go ahead. While I appreciate the sentiment, I would rather not have an excessive amount of

Re: [PATCH] docs: add zpci information to pci-addresses.rst

2020-04-15 Thread Cornelia Huck
On Wed, 15 Apr 2020 16:23:46 +0200 Boris Fiuczynski wrote: > On 4/15/20 3:42 PM, Cornelia Huck wrote: > > On Tue, 14 Apr 2020 23:06:47 +0200 > > Boris Fiuczynski wrote: > > > >> On 4/15/20 12:51 PM, Cornelia Huck wrote: > >>> +In the simplest case, the following XML snippet > >>> + > >>>

Re: [PATCH] docs: add zpci information to pci-addresses.rst

2020-04-15 Thread Boris Fiuczynski
On 4/15/20 3:42 PM, Cornelia Huck wrote: On Tue, 14 Apr 2020 23:06:47 +0200 Boris Fiuczynski wrote: On 4/15/20 12:51 PM, Cornelia Huck wrote: +zPCI addresses +== + +For s390x machines, PCI addresses are handled yet differently. No +topology information is relayed in the PCI

Re: [PATCH] docs: add zpci information to pci-addresses.rst

2020-04-15 Thread Cornelia Huck
On Tue, 14 Apr 2020 23:06:47 +0200 Boris Fiuczynski wrote: > On 4/15/20 12:51 PM, Cornelia Huck wrote: > > +zPCI addresses > > +== > > + > > +For s390x machines, PCI addresses are handled yet differently. No > > +topology information is relayed in the PCI addresses; instead, the > >

Re: [PATCH] docs: add zpci information to pci-addresses.rst

2020-04-15 Thread Boris Fiuczynski
Sorry about the time travel. I will try to give up that super power... :( On 4/14/20 11:06 PM, Boris Fiuczynski wrote: On 4/15/20 12:51 PM, Cornelia Huck wrote: Add some information on how pci address work on s390x. Signed-off-by: Cornelia Huck Conny, thanks for catching this "wacky

Re: [PATCH] docs: add zpci information to pci-addresses.rst

2020-04-15 Thread Boris Fiuczynski
On 4/15/20 12:51 PM, Cornelia Huck wrote: Add some information on how pci address work on s390x. Signed-off-by: Cornelia Huck Conny, thanks for catching this "wacky case"... :) I took the liberty to remove "completely" because there needs to be room for pci multifunction support... :D ---

Re: [PATCH] docs: add zpci information to pci-addresses.rst

2020-04-15 Thread Ján Tomko
On a Wednesday in 2020, Cornelia Huck wrote: Add some information on how pci address work on s390x. Signed-off-by: Cornelia Huck --- docs/pci-addresses.rst | 63 ++ 1 file changed, 63 insertions(+) diff --git a/docs/pci-addresses.rst

[PATCH] docs: add zpci information to pci-addresses.rst

2020-04-15 Thread Cornelia Huck
Add some information on how pci address work on s390x. Signed-off-by: Cornelia Huck --- docs/pci-addresses.rst | 63 ++ 1 file changed, 63 insertions(+) diff --git a/docs/pci-addresses.rst b/docs/pci-addresses.rst index 923783a151b0..9e241a24fcfb 100644