> -----Original Message-----
> From: Jimi Xenidis [mailto:[EMAIL PROTECTED] 
>
> start is in bytes here
> >
> >              DBG("%s: marking 0x%x - 0x%lx\n", __func__,
> > -                pg << PAGE_SHIFT, start);
> > +                pg << PAGE_SHIFT, start << PAGE_SHIFT);
> >
> becomes "in pages" here
> >              start >>= PAGE_SHIFT;
> >              while (pg < MEM_AVAILABLE_PAGES && pg < start) {
> >

I see... Thanks.  There was a similar bug (for real) in code
removed by an earlier patch that led me to jump to the
conclusion the same shift was needed there.

Stuart

_______________________________________________
Xen-ppc-devel mailing list
Xen-ppc-devel@lists.xensource.com
http://lists.xensource.com/xen-ppc-devel

Reply via email to