Re: [Xen-devel] [PATCH v2 2/2] libxendevicemodel: initialise extent to zero

[Paul Durrant <Paul.Durrant@xxxxxxxxxx>]

> -----Original Message-----
> From: Wei Liu [mailto:wei.liu2@xxxxxxxxxx]
> Sent: 28 September 2017 11:46
> To: Paul Durrant <Paul.Durrant@xxxxxxxxxx>
> Cc: Wei Liu <wei.liu2@xxxxxxxxxx>; Xen-devel <xen-
> devel@xxxxxxxxxxxxxxxxxxxx>; Anthony Perard <anthony.perard@xxxxxxxxxx>;
> Ian Jackson <Ian.Jackson@xxxxxxxxxx>
> Subject: Re: [PATCH v2 2/2] libxendevicemodel: initialise extent to zero
> 
> On Thu, Sep 28, 2017 at 11:40:45AM +0100, Paul Durrant wrote:
> > > -----Original Message-----
> > > From: Wei Liu [mailto:wei.liu2@xxxxxxxxxx]
> > > Sent: 28 September 2017 11:36
> > > To: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>
> > > Cc: Anthony Perard <anthony.perard@xxxxxxxxxx>; Wei Liu
> > > <wei.liu2@xxxxxxxxxx>; Ian Jackson <Ian.Jackson@xxxxxxxxxx>; Paul
> Durrant
> > > <Paul.Durrant@xxxxxxxxxx>
> > > Subject: [PATCH v2 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, use C99 initialiser to clear
> > > extent.
> > >
> > > Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx>
> > > ---
> > > Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> > > Cc: Paul Durrant <paul.durrant@xxxxxxxxxx>
> > >
> > > Backport to 4.9
> > > ---
> > >  tools/libs/devicemodel/core.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/tools/libs/devicemodel/core.c
> b/tools/libs/devicemodel/core.c
> > > index fcb260d29b..986f147658 100644
> > > --- a/tools/libs/devicemodel/core.c
> > > +++ b/tools/libs/devicemodel/core.c
> > > @@ -483,7 +483,7 @@ int xendevicemodel_modified_memory(
> > >      xendevicemodel_handle *dmod, domid_t domid, uint64_t first_pfn,
> > >      uint32_t nr)
> > >  {
> > > -    struct xen_dm_op_modified_memory_extent extent;
> > > +    struct xen_dm_op_modified_memory_extent extent = { };
> >
> > Not quite. I meant:
> >
> > struct xen_dm_op_modified_memory_extent extent = {
> >     .first_pfn = first_pfn,
> >     .nr = nr,
> > };
> >
> > then you can drop the lines below.
> 
> Here you go.
> 
> ---8<---
> From 75c955c2d2c7077fe59b5e3b14e3deff613025e8 Mon Sep 17 00:00:00
> 2001
> From: Wei Liu <wei.liu2@xxxxxxxxxx>
> Date: Thu, 28 Sep 2017 11:00:00 +0100
> Subject: [PATCH] libxendevicemodel: initialise extent.pad to zero
> 
> The pad field needs to be zero as required by the hypervisor.
> 
> Instead of setting the pad separately, use C99 initialiser.
> 
> Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx>
> ---
> Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> Cc: Paul Durrant <paul.durrant@xxxxxxxxxx>
> 
> Backport to 4.9
> ---
>  tools/libs/devicemodel/core.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/tools/libs/devicemodel/core.c b/tools/libs/devicemodel/core.c
> index fcb260d29b..0f2c1a791f 100644
> --- a/tools/libs/devicemodel/core.c
> +++ b/tools/libs/devicemodel/core.c
> @@ -483,10 +483,10 @@ int xendevicemodel_modified_memory(
>      xendevicemodel_handle *dmod, domid_t domid, uint64_t first_pfn,
>      uint32_t nr)
>  {
> -    struct xen_dm_op_modified_memory_extent extent;
> -
> -    extent.first_pfn = first_pfn;
> -    extent.nr = nr;
> +    struct xen_dm_op_modified_memory_extent extent = {
> +        .first_pfn = first_pfn,
> +        .nr = nr,
> +    };

Yep, LGTM.

Reviewed-by: Paul Durrant <paul.durrant@xxxxxxxxxx>

> 
>      return xendevicemodel_modified_memory_bulk(dmod, domid, &extent,
> 1);
>  }
> --
> 2.11.0


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel