Il 19/09/2014 12:43, Pavel Dovgaluk ha scritto: > I've tested this patch with replay. I enabled VM reset (which was previously > disabled for replay) > while loading the VM state and discovered the following problem. > vapic_enable function in kvmapic.c retrieves cpu number with the > get_kpcr_number() function. > When cpu number is -1 vapic_enable exits and does not call apic_enable_vapic, > which should > setup vapic_paddr field. > Without this call vapic_paddr remains initialized with default value and > behavior of the virtual > machine becomes different.
IIUC the fix would be to move part of vapic_enable out to its separate function, and call it from do_vapic_enable? Could you prepare a patch? Paolo