[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2] libxl: memory size in kb requires 64 bit variable



On Thu, 2016-07-28 at 14:48 +0200, Juergen Gross wrote:
> On 28/07/16 14:11, Dario Faggioli wrote:
> >  
> > > +static int libxl__memkb_32to64(libxl_ctx *ctx, int rc,
> > > +                               uint64_t val64, uint32_t *ptr32)
> > > +{
> > > +    GC_INIT(ctx);
> > > +
> > > +    if (rc)
> > > +        goto out;
> > > +
> > > +    *ptr32 = val64;
> > > +    if (*ptr32 == val64)
> > > +        goto out;
> > > +
> > > +    LOGE(ERROR, "memory size %"PRIu64" too large for 32 bit
> > > value\n", val64);
> > > +    rc = ERROR_FAIL;
> > > +
> > > +out:
> > > +    GC_FREE;
> > > +    return rc;
> > > +}
> > > +
> > ...does more 64to32 then 32to64, but maybe that's me, and, anyway,
> > it's
> > certainly not a big deal.
> Hmm, yes, I can see your point.
> 
> Will do V3. Only doing the rename, can I keep your Reviewed-by: ?
> 
Yes, you can.

Dario
-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)

Attachment: signature.asc
Description: This is a digitally signed message part

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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.