On Thu, 2007年03月08日 at 08:59 -0700, Alex Williamson wrote:
> On Thu, 2007年03月08日 at 10:28 -0500, George S. Coker, II wrote:
> > + /*
> > + * Try all modules and see whichever could be the binary policy.
> > + * Adjust the initrdidx if module[1] is the binary policy.
> > + */
> > + for (i = mbi->mods_count-1; i >= 1; i--) {
> > +#if defined(__i386__)
> > + _policy_start = (u32 *)(initial_images_start + 
> > (mod[i].mod_start-mod[0].mod_start));
> > +#elif defined(__x86_64__)
> > + _policy_start = __va(initial_images_start + 
> > (mod[i].mod_start-mod[0].mod_start));
> > +#else
> > +#error Architecture unsupported by XSM
> > +#endif
>
> This is unacceptable, please make it fail gracefully on non-x86.
Indeed, it looks like this logic is outdated. This was based on some
older code from ACM. The attached patch addresses this issue.
> Thanks,
>
> Alex
>
-- 
George S. Coker, II <gscoker@xxxxxxxxxxxxxx> 443-479-6944

Attachment: xsm-policy-init-030807.diff
Description: Text Data

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