[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAEH94Li8gdMZnwSHG_xQj761uRxedRX2nUVmhFV4Kt_EDeuzYQ@mail.gmail.com>
Date: Fri, 2 Dec 2011 10:25:21 +0800
From: Zhi Yong Wu <zwu.kernel@...il.com>
To: Jesse Gross <jesse@...ira.com>
Cc: dev@...nvswitch.org, netdev@...r.kernel.org, aliguori@...ibm.com,
stefanha@...il.com
Subject: Re: [ovs-dev] [PATCH] datapath: Fix build breakage on kernel 2.6.40
On Thu, Dec 1, 2011 at 5:55 AM, Jesse Gross <jesse@...ira.com> wrote:
> On Wed, Nov 23, 2011 at 1:58 AM, Zhi Yong Wu <zwu.kernel@...il.com> wrote:
>> diff --git a/datapath/linux/compat/include/linux/skbuff.h b/datapath/linux/compat/include/linux/skbuff.h
>> index 311bfdb..22ba2e6 100644
>> --- a/datapath/linux/compat/include/linux/skbuff.h
>> +++ b/datapath/linux/compat/include/linux/skbuff.h
>> @@ -239,7 +239,7 @@ static inline struct page *skb_frag_page(const skb_frag_t *frag)
>> }
>> #endif
>>
>> -#if LINUX_VERSION_CODE < KERNEL_VERSION(3,0,0)
>> +#ifndef HAVE_SKB_RESET_MAC_LEN
>
> 2.6.40 is the early name for 3.0. Does it work if you just replace
> the check with KERNEL_VERSION(2,6,40)?
Yeah, it can work now, but i don't know if this issue exist on other
old kernel version < 2.6.40.
Should we send one updated patch for this based on your suggestion?
--
Regards,
Zhi Yong Wu
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists