On Wed, Nov 18, 2020 at 11:29:35AM +0100, Claudio Fontana wrote: > apply this to the registration of the cpus accel interfaces, > > but this will be also in preparation for later use of this > new module init step to also defer the registration of the cpu models, > in order to make them subclasses of a per-accel cpu type. > > Signed-off-by: Claudio Fontana <cfont...@suse.de> > --- [...] > + /* > + * accelerator has been chosen and initialized, now it is time to > + * register the cpu accel interface. > + */ > + module_call_init(MODULE_INIT_ACCEL_CPU);
I don't get why we would use a new module initialization level for this. If the accelerator object was already created, we can just ask the existing accel object to do whatever initialization step is necessary. e.g. we can add a AccelClass.cpu_accel_ops field, and call: cpus_register_accel(current_machine->accelerator->cpu_accel_ops); -- Eduardo
