[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC 1/8] memory: Allow eventfd add/del without starting a transacti
From: |
Stefan Hajnoczi |
Subject: |
Re: [RFC 1/8] memory: Allow eventfd add/del without starting a transaction |
Date: |
Tue, 30 Mar 2021 11:16:50 +0100 |
On Tue, Mar 30, 2021 at 09:47:49AM +0200, Greg Kurz wrote:
> On Mon, 29 Mar 2021 18:03:49 +0100
> Stefan Hajnoczi <stefanha@redhat.com> wrote:
>
> > On Thu, Mar 25, 2021 at 04:07:28PM +0100, Greg Kurz wrote:
> > > diff --git a/include/exec/memory.h b/include/exec/memory.h
> > > index 5728a681b27d..98ed552e001c 100644
> > > --- a/include/exec/memory.h
> > > +++ b/include/exec/memory.h
> > > @@ -1848,13 +1848,25 @@ void
> > > memory_region_clear_flush_coalesced(MemoryRegion *mr);
> > > * @match_data: whether to match against @data, instead of just @addr
> > > * @data: the data to match against the guest write
> > > * @e: event notifier to be triggered when @addr, @size, and @data all
> > > match.
> > > + * @transaction: whether to start a transaction for the change
> >
> > "start" is unclear. Does it begin a transaction and return with the
> > transaction unfinished? I think instead the function performs the
> > eventfd addition within a transaction. It would be nice to clarify this.
> >
>
> What about:
>
> * @transaction: if true, the eventfd is added within a nested transaction,
> * if false, it is up to the caller to ensure this is called
> * within a transaction.
Sounds good, thanks!
Stefan
signature.asc
Description: PGP signature