On Mon, Jan 24, 2022 at 11:15:25AM +0100, Paolo Bonzini wrote: > On 1/24/22 08:55, Yang Zhong wrote: > > > >+ if (buflen > sizeof(struct kvm_xsave)) { > >+ e = &x86_ext_save_areas[XSTATE_XTILE_DATA_BIT]; > >+ > >+ if (e->size && e->offset) { > >+ const XSaveXTILEDATA *tiledata = buf + e->offset; > >+ > >+ memcpy(&env->xtiledata, tiledata, sizeof(env->xtiledata)); > >+ } > >+ } > > Slightly nicer: > > e = &x86_ext_save_areas[XSTATE_XTILE_DATA_BIT]; > if (e->size && e->offset && buflen >= e->size + e->offset) { > ... > } > > Same for xsave.
Thanks Paolo, new version will change this. Yang > > Paolo