On Thu, Mar 17, 2011 at 04:41:08PM +0200, Michael S. Tsirkin wrote: > > +static int piix3_post_load(void *opaque, int version_id) > > +{ > > + PIIX3State *piix3 = opaque; > > + piix3_rebuild_irq_levels(piix3); > > Don't we need to set_irq_pic here as well? > And in that case, just make the for loop > part of piix3_rebuild_irq_levels.
No, we need not. Because pic irq state is saved/loaded by pic itself. So todo here is to make pic_levels consistent with other part. -- yamahata