Am 11.07.2013 03:45, schrieb peter.crosthwa...@xilinx.com: > From: Peter Crosthwaite <peter.crosthwa...@xilinx.com> > > QOM automatically inherits class and instance size from the parent > class. No need to redefine as the same value as the parent.
Quoting my original comment: "It would be fair to mention since which commit because that was originally not the case." http://git.qemu.org/?p=qemu.git;a=commit;h=aca59af612840772f18598363b65a25bf02bb569 E.g., "By now, QOM automatically..." or "Since commit ..." > > CC: qemu-triv...@nongnu.org We have an active ARM maintainer, so no reason to involve qemu-trivial here by our definition. Possibly that would've required Cc: anyway? Not actually CC'ed. > > Signed-off-by: Peter Crosthwaite <peter.crosthwa...@xilinx.com> Still: Reviewed-by: Andreas Färber <afaer...@suse.de> PMM, do you want to pick this one up for target-arm.next? The rest depends on central infrastructure under discussion. Suggest renaming to "target-arm: Delete un-needed instance/class sizes for ARMCPU subclasses" for simplicity and so that it aligns nicely with others. Regards, Andreas > --- > > target-arm/cpu.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/target-arm/cpu.c b/target-arm/cpu.c > index 1bc227e..ed53df8 100644 > --- a/target-arm/cpu.c > +++ b/target-arm/cpu.c > @@ -823,9 +823,7 @@ static void cpu_register(const ARMCPUInfo *info) > { > TypeInfo type_info = { > .parent = TYPE_ARM_CPU, > - .instance_size = sizeof(ARMCPU), > .instance_init = info->initfn, > - .class_size = sizeof(ARMCPUClass), > .class_init = info->class_init, > }; > -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
