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: Tue, 26 Aug 2014 11:08:53 -0500 Message-ID: References: <20140725103627.4ca989b7@haswell.linuxnetplumber.net> <9317787.qKVt3oyQbN@xps13> <20140823081453.6e3ae5d7@uryu.home.lan> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Cc: "dev-VfR2kkLFssw@public.gmane.org" To: Stephen Hemminger Return-path: In-Reply-To: <20140823081453.6e3ae5d7-CA4OZQ/Yy2Lykuyl+CZolw@public.gmane.org> 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" > >> This is what we came up with. It works for us. In our kernel headers' >> linux/pci.h, pci_num_vf is enclosed within "#ifdef >>CONFIG_PCI_IOV/#endif"; >> pci_intx_mask_supported and pci_check_and_mask_intx are enclosed within >> "#ifdef HAVE_PCI_SET_MWI/#endif". >>=20 >> What do you think? > >Maybe we can just get rid of kernel version checks all together and >just use the HAVE_ checks. > >I will test on stock 2.6.32. Sorry, I spoke too soon. Although it builds with the ifdefs, I just found that it does not load. "kernel: igb_uio: Unknown symbol irq_to_desc"