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] compile error of HVM pv drivers

To: "Keir Fraser" <keir@xxxxxxxxxxxxx>, "Kieran Mansley" <kmansley@xxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] compile error of HVM pv drivers
From: "Zhao, Yunfeng" <yunfeng.zhao@xxxxxxxxx>
Date: Mon, 30 Jul 2007 23:38:51 +0800
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Mon, 30 Jul 2007 08:36:34 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <C2D3BCC3.1347E%keir@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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcfSuSOEYcsCnj6sEdyq8gAX8io7RQAASASwAAAyeLYAAOv5AA==
Thread-topic: [Xen-devel] compile error of HVM pv drivers
"netfront/accel.c" needs include platform-compat.h
#ifdef HAVE_XEN_PLATFORM_COMPAT_H
#include <xen/platform-compat.h>
#endif

>-----Original Message-----
>From: Keir Fraser [mailto:keir@xxxxxxxxxxxxx]
>Sent: 2007年7月30日 23:05
>To: Zhao, Yunfeng; Kieran Mansley
>Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
>Subject: Re: [Xen-devel] compile error of HVM pv drivers
>
>I think you'll find that doesn't fix the problem. The () is there for a
>reason.
>
> K.
>
>On 30/7/07 16:02, "Zhao, Yunfeng" <yunfeng.zhao@xxxxxxxxx> wrote:
>
>> The problem may be from the new changes in
>> unmodified_drivers/linux-2.6/compat-include/xen/platform-compat.h
>>
>> the following two lines:
>> #define netif_tx_lock_bh(dev) (spin_lock_bh(&(dev)->xmit_lock))
>> #define netif_tx_unlock_bh(dev) (spin_unlock_bh(&(dev)->xmit_lock))
>> should be
>> #define netif_tx_lock_bh(dev) (spin_lock_bh(&dev->xmit_lock))
>> #define netif_tx_unlock_bh(dev) (spin_unlock_bh(&dev->xmit_lock))
>> "()" is useless for dev.
>>
>> Thanks
>> Yunfeng
>>
>>
>>
>>> -----Original Message-----
>>> From: Keir Fraser [mailto:keir@xxxxxxxxxxxxx]
>>> Sent: 2007年7月30日 22:52
>>> To: Kieran Mansley; Zhao, Yunfeng
>>> Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
>>> Subject: Re: [Xen-devel] compile error of HVM pv drivers
>>>
>>> On 30/7/07 15:47, "Kieran Mansley" <kmansley@xxxxxxxxxxxxxx> wrote:
>>>
>>>> In that case replacing the calls (there are two in that file) to
>>>> netif_tx_lock_bh() with:
>>>>
>>>>  spin_lock_bh(&vif_state->np->netdev->xmit_lock);
>>>>
>>>> And similarly the calls to netif_tx_unlock_bh() with spin_unlock_bh(),
>>>> may solve your problem.
>>>>
>>>> If this works for you I'll make a patch that uses the kernel version to
>>>> decide which is appropriate.
>>>
>>> Compat issues when building PV-on-HVM drivers are patched up out-of-band in
>>> the special
>>> unmodified_drivers/linux-2.6/compat-include/xen/platform-compat.h
>>> header file.
>>>
>>> -- Keir
>>
>> _______________________________________________
>> 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