Hello Bastian,

On Mon, Jul 12, 2010 at 07:18:28PM +0200, Bastian Blank wrote:
> I would believe that the correct fix is to remove the usage of the
> lowlevel interface.

I think i agree.

I just took a glance over the code, and also fired up a python shell to
see what the code is actually used for.

>From what i can tell xen.lowlevel.xc allows the script to find out the
capabilities of the enviorment, in the script it is *only* called when
checking if the dom0 supports running 64-bit pvguests, i.e. the
supports64bitPVguest() function.

Anyway it doesn't seem like a piece of code that is necessary for most
users. In fact, deleting this import line worked just as well.

The only thing that bothers me is that it is called used in the bit of
code labeled:

"If nothing has been specified, look for a Solaris domU"

I don't have (nor do i want) any solaris domU's to test this code with.
However i can say that both this import line, as well as the
supports64bitPVguest function and the subsequent calls to it are absent
in the Xen installation i have on one of my CentOS machines (xen 3.4.2).
The version listed at the top of the script in that installation is
also 0.6.

Anyway I think it'd be best to prepare a patch to delete this cruft in
the Debian package. It's a shame that it breaks pygrub right now.

Regards,

Dmitry



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to