WARNING - OLD ARCHIVES

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/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

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

To: Daniel Kiper <dkiper@xxxxxxxxxxxx>, Keir Fraser <keir.fraser@xxxxxxxxxxxxx>, Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH] Parallel/Serial/Monitor/pyGRUB ReiserFS/others support/fixes
From: "Xu, Dongxiao" <dongxiao.xu@xxxxxxxxx>
Date: Thu, 24 Dec 2009 15:43:00 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc:
Delivery-date: Wed, 23 Dec 2009 23:44:08 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20091224005753.GA12270@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
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> <20091224005753.GA12270@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcqENEKONb536lDVSJSGkFN+XntkJwANtxrA
Thread-topic: [Xen-devel] [PATCH] Parallel/Serial/Monitor/pyGRUB ReiserFS/others support/fixes
Hi, Daniel, 
        Could you help to check your patch which is checked in as C/S 20717? 
It seems that VM save/restore/migration and reboot is broken since this 
changeset.

Thanks!
Dongxiao

Daniel Kiper wrote:
> Hello,
> 
>>>>> 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.
> 
> FYI:
> 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 them.
> 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
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel