From: Amanda Liem <Amanda.Liem@amd.com>
To: <virtio-comment@lists.linux.dev>
Cc: <parav@nvidia.com>, <Nathaniel.McCallum@amd.com>,
Amanda Liem <Amanda.Liem@amd.com>
Subject: Re: [PATCH] virtio-net: Add VIRTIO_NET_F_LABEL for a per-instance label
Date: Thu, 24 Sep 2026 10:28:56 -0500 [thread overview]
Message-ID: <20260924152856.3326-1-Amanda.Liem@amd.com> (raw)
In-Reply-To: <SJ0PR12MB68063B6C5A125573CE53A717DCA62@SJ0PR12MB6806.namprd12.prod.outlook.com>
On Mon, Aug 18, 2026 at 17:16, Parav Pandit <parav@nvidia.com> wrote:
> Please add the motivation for this addition describing what problem
> exists and how does this addition solve it.
Will do - I'll add it to the commit message in v2. Our use case is a
confidential VM whose image author ships a trusted manifest of expected
interfaces - the guest uses the label as a lookup key to associate each
device with its manifest entry before applying config. The manifest, not
the label, holds the authority. The guest corroborates the label against
the manifest rather than trusting it on its own, so it is not a channel
the host can use to steer configuration.
We'd follow the spec change with a Linux virtio-net driver patch exposing
the label via sysfs, so userspace (our manifest evaluator included) can
consume it without manifest logic in the driver. The manifest is just one
consumer - virtio need not standardize its format.
> And also have you considered querying this field via CVQ.
> If yes, than please explain the motivation to place this in config
> space.
Yes, a control virtqueue query could convey the same value. We prefer
config space because the label is a small, static, read-only property that
can be read at device init without negotiating VIRTIO_NET_F_CTRL_VQ or
standing up a control virtqueue just to retrieve it. The footprint is
bounded - the fields exist only when the feature is negotiated, and the
label is a fixed 64-byte buffer read once. (The v1 mention of PCI/MMIO/CCW
wasn't meant to distinguish config space from CVQ - both are
transport-independent.)
v2 to follow.
Thanks,
Amanda
prev parent reply other threads:[~2026-09-24 15:29 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-18 17:05 [PATCH] virtio-net: Add VIRTIO_NET_F_LABEL for a per-instance label Amanda Liem
2026-08-18 17:15 ` Parav Pandit
2026-09-24 15:28 ` Amanda Liem [this message]
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=20260924152856.3326-1-Amanda.Liem@amd.com \
--to=amanda.liem@amd.com \
--cc=Nathaniel.McCallum@amd.com \
--cc=parav@nvidia.com \
--cc=virtio-comment@lists.linux.dev \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).