[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
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
On 30/7/07 16:02, "Zhao, Yunfeng" <yunfeng.zhao@xxxxxxxxx> wrote:
> The problem may be from the new changes in
> 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.
>> -----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:
>>> 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
>> header file.
>> -- Keir
> Xen-devel mailing list
Xen-devel mailing list