From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:33326)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <abologna@redhat.com>) id 1bw9I4-0002Yv-Dp
	for qemu-devel@nongnu.org; Mon, 17 Oct 2016 10:53:25 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <abologna@redhat.com>) id 1bw9I0-00016c-3G
	for qemu-devel@nongnu.org; Mon, 17 Oct 2016 10:53:24 -0400
Received: from mx1.redhat.com ([209.132.183.28]:37072)
	by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32)
	(Exim 4.71) (envelope-from <abologna@redhat.com>) id 1bw9Hz-00016N-UU
	for qemu-devel@nongnu.org; Mon, 17 Oct 2016 10:53:20 -0400
Message-ID: <1476715995.3742.11.camel@redhat.com>
From: Andrea Bolognani <abologna@redhat.com>
Date: Mon, 17 Oct 2016 16:53:15 +0200
In-Reply-To: <fcd1a6f0-1159-1039-9aad-436312257539@redhat.com>
References: <1476366759-29119-1-git-send-email-marcel@redhat.com>
	<38124233-a3e5-9073-262e-e7ef96676eea@redhat.com>
	<1476713923.3742.9.camel@redhat.com>
	<fcd1a6f0-1159-1039-9aad-436312257539@redhat.com>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Subject: Re: [Qemu-devel] [PATCH V2 RESEND] docs: add PCIe devices placement
 guidelines
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel/>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Laszlo Ersek <lersek@redhat.com>, Marcel Apfelbaum <marcel@redhat.com>, qemu-devel@nongnu.org
Cc: Gerd Hoffmann <kraxel@redhat.com>, Laine Stump <laine@redhat.com>, Peter Maydell <peter.maydell@linaro.org>, Andrew Jones <drjones@redhat.com>, Alex Williamson <alex.williamson@redhat.com>, Daniel Berrange <berrange@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>

On Mon, 2016-10-17 at 16:26 +0200, Laszlo Ersek wrote:
> > > > +2.1 Root Bus (pcie.0)
> > > > +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> > > > +Place only the following kinds of devices directly on the Root C=
omplex:
> > > > +=C2=A0=C2=A0=C2=A0=C2=A0(1) Devices with dedicated, specific fun=
ctionality (network card,
> > > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0graphics card, I=
DE controller, etc); place only legacy PCI devices on
> > > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0the Root Complex=
. These will be considered Integrated Endpoints.
> > > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0Note: Integrated=
 devices are not hot-pluggable.
> >=C2=A0
> > s/Integrated devices/Integrated Endpoints/ (which I assume
> > is a Spec-Originated Term) in the last sentence, to be
> > consistent with the one right before it.
> >=C2=A0
> > I'm also not sure what you mean by devices with "dedicated,
> > specific functionality", and unfortunately the examples don't
> > seem to be helping me.
>=C2=A0
> That language is from me (I suggested it earlier). It means devices tha=
t
> you want to use for some actual task, rather than just to build your PC=
I
> (Express) hierarchy with them. "Leaf nodes" or whatever. "Everything
> non-bridge".

Isn't that basically the definition of Endpoint? It certainly
is what I convinced myself "Endpoint" means :)

Either way, I think we should spend a few more words there,
for clarity's sake. Rewording the phrase to put more emphasis
on the "legacy PCI devices only" part would probably be
warranted as well.

--=C2=A0
Andrea Bolognani / Red Hat / Virtualization