On 5/21/20 5:42 AM, David Gibson wrote: > SEVState is contained with SevGuestState. We've now fixed redundancies > and name conflicts, so there's no real point to the nested structure. Just > move all the fields of SEVState into SevGuestState. > > This eliminates the SEVState structure, which as a bonus removes the > confusion with the SevState enum. > > Signed-off-by: David Gibson <david@xxxxxxxxxxxxxxxxxxxxx> > --- > target/i386/sev.c | 79 ++++++++++++++++++++--------------------------- > 1 file changed, 34 insertions(+), 45 deletions(-) [...] > sev_guest = sev; > - s = &sev->state; > - s->state = SEV_STATE_UNINIT; > + sev->state = SEV_STATE_UNINIT; :) Reviewed-by: Philippe Mathieu-Daudé <philmd@xxxxxxxxxx>