> -----Original Message----- > From: Wei Liu [mailto:wei.l...@citrix.com] > Sent: 28 September 2017 11:24 > To: Xen-devel <xen-de...@lists.xenproject.org> > Cc: Wei Liu <wei.l...@citrix.com>; Ian Jackson <ian.jack...@citrix.com>; > Paul Durrant <paul.durr...@citrix.com> > Subject: [PATCH 2/2] libxendevicemodel: initialise extent to zero > > The pad field needs to be zero as required by the hypervisor. > > Instead of setting the pad separately, memset extent to zero. > > Signed-off-by: Wei Liu <wei.l...@citrix.com> > --- > Cc: Ian Jackson <ian.jack...@eu.citrix.com> > Cc: Paul Durrant <paul.durr...@citrix.com> > > Backport to 4.9 > --- > tools/libs/devicemodel/core.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/tools/libs/devicemodel/core.c b/tools/libs/devicemodel/core.c > index fcb260d29b..af06936262 100644 > --- a/tools/libs/devicemodel/core.c > +++ b/tools/libs/devicemodel/core.c > @@ -485,6 +485,8 @@ int xendevicemodel_modified_memory( > { > struct xen_dm_op_modified_memory_extent extent; > > + memset(&extent, 0, sizeof(extent)); > +
Probably better to just use a C99 initializer instead so the compiler deals with the pad. Paul > extent.first_pfn = first_pfn; > extent.nr = nr; > > -- > 2.11.0 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
