On Tue, Mar 20, 2018 at 11:43:00AM +0000, Roger Pau Monné wrote:
> On Tue, Mar 20, 2018 at 05:03:09AM -0600, Jan Beulich wrote:
> > >>> On 16.03.18 at 14:30, <roger....@citrix.com> wrote:
> > > +            printk(XENLOG_G_WARNING
> > > +                   "Failed to remove MSIX table [%" PRI_gfn ", %" 
> > > PRI_gfn "]: %d\n",
> > > +                   PFN_DOWN(start), PFN_DOWN(end), rc);
> > 
> > In cases like this (where you don't use plain start/end anywhere,
> > but you do use the same calculation on them twice each), it's
> > certainly more efficient for the local variables to be frame numbers
> > right away.
> 
> Right, I've fixed it and also changed the printf formatters to lu, I
> guess at some point in the series I used to print gfn values.

Meant lx not lu...

Sorry.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to