From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH 3/4] kni: replace strict_strtoul with kstrtoul Date: Wed, 10 Dec 2014 09:39:14 +0100 Message-ID: <1663200.Cog0FQ0fDE@xps13> References: <1418182383-5744-1-git-send-email-jmiao@redhat.com> <1418182383-5744-4-git-send-email-jmiao@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: Jincheng Miao Return-path: In-Reply-To: <1418182383-5744-4-git-send-email-jmiao-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 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 Jincheng, 2014-12-10 11:33, Jincheng Miao: > From upstream kernel commit 3db2e9cd, strict_strto* serial functions > are removed. So that we should directly used kstrtoul instead. > > And add kni/compat.h for be compatible with older kernel. > > Signed-off-by: Jincheng Miao [...] > new file mode 100644 > index 0000000..c8c662c > --- /dev/null > +++ b/lib/librte_eal/linuxapp/kni/compat.h > @@ -0,0 +1,16 @@ > +/* > + * Minimal wrappers to allow compiling kni on older kernels. > + */ > + > +#ifndef RHEL_RELEASE_VERSION > +#define RHEL_RELEASE_VERSION(a, b) (((a) << 8) + (b)) > +#endif > + > +#if LINUX_VERSION_CODE < KERNEL_VERSION(2, 6, 39) && \ > + (!(defined(RHEL_RELEASE_CODE) && \ > + RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(6, 4))) The first indent character must be a tab (highlighted by checkpatch). [...] > --- a/lib/librte_eal/linuxapp/kni/kni_vhost.c > +++ b/lib/librte_eal/linuxapp/kni/kni_vhost.c > @@ -740,7 +740,7 @@ set_sock_en(struct device *dev, struct device_attribute *attr, > unsigned long en; > int err = 0; > > - if (0 != strict_strtoul(buf, 0, &en)) > + if (0 != kstrtoul(buf, 0, &en)) > return -EINVAL; > > if (en) It seems you forgot to include the new compat.h. Did you do some tests with different Fedora/RHEL versions? Thanks -- Thomas