[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 3/7] device-core: use RCU for list of childs of a bus
From: |
Maxim Levitsky |
Subject: |
Re: [PATCH v2 3/7] device-core: use RCU for list of childs of a bus |
Date: |
Thu, 09 Jul 2020 12:40:28 +0300 |
User-agent: |
Evolution 3.34.4 (3.34.4-1.fc31) |
On Wed, 2020-05-27 at 15:45 +0100, Stefan Hajnoczi wrote:
> On Mon, May 11, 2020 at 07:09:47PM +0300, Maxim Levitsky wrote:
> > diff --git a/include/hw/qdev-core.h b/include/hw/qdev-core.h
> > index d87d989e72..ef47cb2d9c 100644
> > --- a/include/hw/qdev-core.h
> > +++ b/include/hw/qdev-core.h
> > @@ -3,6 +3,8 @@
> >
> > #include "qemu/queue.h"
> > #include "qemu/bitmap.h"
> > +#include "qemu/rcu.h"
> > +#include "qemu/rcu_queue.h"
> > #include "qom/object.h"
> > #include "hw/hotplug.h"
> > #include "hw/resettable.h"
> > @@ -230,6 +232,7 @@ struct BusClass {
> > };
> >
> > typedef struct BusChild {
> > + struct rcu_head rcu;
> > DeviceState *child;
> > int index;
> > QTAILQ_ENTRY(BusChild) sibling;
>
> Please add a doc comment to struct BusState saying the children field is
> an RCU QTAILQ and writers must hold the QEMU global mutex.
>
> Stefan
Done.
Best regards,
Maxim Levitsky
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH v2 3/7] device-core: use RCU for list of childs of a bus,
Maxim Levitsky <=