On Fri, 31 Jul 2020 at 13:44, Eduardo Habkost <ehabk...@redhat.com> wrote:
> >
> > +static inline void cpu_class_disable_bql_interrupt(CPUClass *cc)
> > +{
> > + cc->bql_interrupt = false;
> > +}
>
> Class data is not supposed to change outside class_init. Why do
> you need this function? I don't see it being used anywhere in
> this series.
This function was to be called from changes in a later patch series
that depend on these changes. BTW, I added a correction above,
it should be disable, not enable. The idea is that it is initialized to true,
but then the per arch changes would use this call at init time to set
it to false
as needed.
We can remove this function from this series and add it in later when
it gets used,
it might make things more clear.
Thanks,
-Rob
> --
> Eduardo
>
