From: Matias Ezequiel Vara Larsen <mvaralar@redhat.com>
To: Parav Pandit <parav@nvidia.com>
Cc: "mst@redhat.com" <mst@redhat.com>,
"cohuck@redhat.com" <cohuck@redhat.com>,
"virtio-comment@lists.linux.dev" <virtio-comment@lists.linux.dev>,
"slp@redhat.com" <slp@redhat.com>,
Shahaf Shuler <shahafs@nvidia.com>
Subject: Re: [PATCH] virtio-gpu: Fix missing driver conformance file
Date: Tue, 8 Jul 2025 10:28:14 +0200 [thread overview]
Message-ID: <aGzWnpWZJhFMcA1S@fedora> (raw)
In-Reply-To: <CY8PR12MB7195AC481F68F1F828646192DC4EA@CY8PR12MB7195.namprd12.prod.outlook.com>
On Tue, Jul 08, 2025 at 08:11:43AM +0000, Parav Pandit wrote:
>
>
> > From: Matias Ezequiel Vara Larsen <mvaralar@redhat.com>
> > Sent: 08 July 2025 01:36 PM
> >
> > On Tue, Jul 08, 2025 at 06:56:26AM +0000, Parav Pandit wrote:
> > > Hi ,
> > >
> > > > From: Matias Ezequiel Vara Larsen <mvaralar@redhat.com>
> > > > Sent: 16 June 2025 04:00 PM
> > > >
> > > > On Sat, Jun 14, 2025 at 09:13:02PM +0300, Parav Pandit wrote:
> > > > > Cited patch in the fixes tag missed to include the
> > > > > driver-conformance file. Add it.
> > > > >
> > > > > Fixes: f52fb20cbe38 ("virtio-gpu: support blob alignment
> > > > > information")
> > > > > Signed-off-by: Parav Pandit <parav@nvidia.com>
> > > > > ---
> > > > > device-types/gpu/driver-conformance.tex | 8 ++++++++
> > > > > 1 file changed, 8 insertions(+)
> > > > > create mode 100644 device-types/gpu/driver-conformance.tex
> > > > >
> > > >
> > > > Reviewed-by: Matias Ezequiel Vara Larsen <mvaralar@redhat.com>
> > >
> > > Can you please raise the voting ballot for this patch which fixes
> > https://github.com/oasis-tcs/virtio-spec/issues/223?
> > > I didn't create a new github issue as 223 is open and its half written patches
> > are not merged.
> > > It is fixed with this patch.
> > >
> >
> > Why is it still open if the ballot has been voted?
> >
> > Thanks, Matias
>
> I dint merge because basic makepdf etc scripts fail without this proposed fix.
> And this proposed fix is non editorial in my view.
> So this proposed patch needs vote. Once voting passes, I will merge the original patch + this fix for the same issue 223.
>
I see, since it looks like a non editorial change, should we open a
ballot for that? Can't we just merge it?
Matias
next prev parent reply other threads:[~2025-07-08 8:28 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-14 18:13 [PATCH] virtio-gpu: Fix missing driver conformance file Parav Pandit
2025-06-16 10:30 ` Matias Ezequiel Vara Larsen
2025-07-08 6:56 ` Parav Pandit
2025-07-08 8:06 ` Matias Ezequiel Vara Larsen
2025-07-08 8:11 ` Parav Pandit
2025-07-08 8:28 ` Matias Ezequiel Vara Larsen [this message]
2025-07-08 8:29 ` Parav Pandit
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=aGzWnpWZJhFMcA1S@fedora \
--to=mvaralar@redhat.com \
--cc=cohuck@redhat.com \
--cc=mst@redhat.com \
--cc=parav@nvidia.com \
--cc=shahafs@nvidia.com \
--cc=slp@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.