From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Sanford, Robert" Subject: Re: [PATCH 1/2] igb_uio: fix compability on old kernel Date: Fri, 22 Aug 2014 12:29:31 -0500 Message-ID: References: <20140725103627.4ca989b7@haswell.linuxnetplumber.net> <9317787.qKVt3oyQbN@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset="Windows-1252" Content-Transfer-Encoding: quoted-printable To: Thomas Monjalon , Stephen Hemminger , "dev-VfR2kkLFssw@public.gmane.org" Return-path: In-Reply-To: <9317787.qKVt3oyQbN@xps13> Content-Language: en-US List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" Hi Thomas, Not that I would *like* to fix this, but I *need* to fix it. We are using CentOS 6.5, which I believe is based on RHEL. We have kernel 2.6.32-431.3.1.el6.x86_64. I realize that we need to add/change ifdefs around pci_num_vf, pci_intx_mask_supported, and pci_check_and_mask_intx in igb_uio/compat.h. Any more specific suggestions on how to (elegantly) fix it for us, but not break it for anyone else? -- Regards, Robert >2014-07-25 10:36, Stephen Hemminger: >> Add more compatibility wrappers, and split out all the wrapper >> code to a separate file. Builds on Debian Squeeze (2.6.32) which >> is oldest version of kernel current DPDK supports. >>=20 >> Signed-off-by: Stephen Hemminger > >Acked-by: Thomas Monjalon > >Applied for version 1.7.1. > >There are still some compilation issues with RHEL: >include/linux/pci.h:1572: note: previous declaration of =8Cpci_num_vf=B9 w= as >here >include/linux/pci.h:868: note: previous declaration of >=8Cpci_intx_mask_supported=B9 was here >include/linux/pci.h:869: note: previous declaration of >=8Cpci_check_and_mask_intx=B9 was here > >Some ifdefs are missing but I don't want to dig into RHEL kernel headers >to >find what is the first RHEL release to support these functions. >By the way, if someone knows an easy method to get all RHEL kernel headers >or to know the release where a symbol appeared, it would be very useful. > >--=20 >Thomas