[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250410162516.6ebfa8ee.alex.williamson@redhat.com>
Date: Thu, 10 Apr 2025 16:25:16 -0600
From: Alex Williamson <alex.williamson@...hat.com>
To: Sean Christopherson <seanjc@...gle.com>
Cc: "Michael S. Tsirkin" <mst@...hat.com>, Jason Wang <jasowang@...hat.com>,
Paolo Bonzini <pbonzini@...hat.com>, kvm@...r.kernel.org,
virtualization@...ts.linux.dev, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org, Oliver Upton <oliver.upton@...ux.dev>, David
Matlack <dmatlack@...gle.com>, Like Xu <like.xu.linux@...il.com>, Yong He
<alexyonghe@...cent.com>
Subject: Re: [PATCH 3/7] irqbypass: Take ownership of producer/consumer
token tracking
On Thu, 10 Apr 2025 15:04:35 -0700
Sean Christopherson <seanjc@...gle.com> wrote:
> On Thu, Apr 10, 2025, Alex Williamson wrote:
> > The "token" terminology seems a little out of place after all is said
> > and done in this series.
>
> Ugh, yeah, good point. I don't know why I left it as "token".
>
> > Should it just be an "index" in anticipation of the usage with xarray and
> > changed to an unsigned long? Or at least s/token/eventfd/ and changed to an
> > eventfd_ctx pointer?
>
> My strong vote is for "struct eventfd_ctx *eventfd;"
WFM, thanks,
Alex
Powered by blists - more mailing lists