This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


Re: [Xen-devel] [PATCH] Parallel/Serial/Monitor/pyGRUB ReiserFS/others s

To: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>, Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: Re: [Xen-devel] [PATCH] Parallel/Serial/Monitor/pyGRUB ReiserFS/others support/fixes
From: Daniel Kiper <dkiper@xxxxxxxxxxxx>
Date: Thu, 24 Dec 2009 01:57:53 +0100
Delivery-date: Wed, 23 Dec 2009 16:58:42 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C7577C3B.51DC%keir.fraser@xxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <20091223005430.GA3449@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> <C7577C3B.51DC%keir.fraser@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.3.28i

>>> > xen-unstable-hvmloader-strip_20091222.patch:
>>> >   Strip hvmloader after compile. dh_strip from debhelper
>>> >   does not strip binaries without execute attribute.
>>> I don't see the point of doing this. We don't in general strip stuff
>>> we build under tools/, the extra disc space taken is trivial, and
>>> symbols can be convenient to have around. I just don't see the win.
>>>  - Not applying to xen-unstable or xen-3.4
>> Yes, it is the packager's responsibility to take care of this if they
>> deem it necessary, either manually or by adding +x so that dh_strip
>> takes care of it.
> Actually hvmloader seems to be a+x already, in all my build trees.

xen-3.4.2/tools/firmware/hvmloader/hvmloader: 0755
xen-3.4.2/dist/install/usr/lib/xen/boot/hvmloader: 0644

OK, I agree that this patch break general
rule and should not be applied.

> > xen-unstable-parallel-serial-monitor_20091223.patch:
> Xend parts look good. The hvmloader parts look unnecessary (since we
> currently always declare LPT1 present) and dangerous (Windows guests which
> were installed on old Xen iirc get upset if they later boot and see LTP1
> not-present in ACPI tables). There'd need to be a *good* reason for hiding
> LTP1 in DSDT therefore.
>  - Partially applied to unstable; not applied to 3.4

1) I think that parallel port configuration should be similar to
   serial port configuration. With that idea in mind this patch was
   created (and some code were copied from serial config).
2) I tested this patch with Windows Vista Business SP2 and
   Finnix Ver. 92.1 as a guests and everything worked without
   any problem.
3) I do not like that broken/disabled devices appear in config/device manager
   without any warning. In this situation disabled LPT1 port appear in
   Windows Device Manager without any warning. I think it is confusing
   for users when they see working devices in WDM and they could not use
4) As I mentioned earlier, all patches (including this one) do not
   change default behavior of Xen. It means that default QEMU configuration
   is not changed when parallel option is not specified in config file.
   If somebody would like to disable LPT1, he/she should have possibility
   to do that in proper way. I think it is another question that he/she
   is aware what is he/she doing.

That is why I think xen-unstable-parallel-serial-monitor_20091223.patch
patch should be fully applied.

Daniel Kiper

Xen-devel mailing list