git.droids-corp.org - dpdk.git/log

app: fix unused values

Fix warnings of type "Value stored to 'xxx' is never read".

Signed-off-by: Zijie Pan <zijie.pan@6wind.com>
Acked-by: Ivan Boule <ivan.boule@6wind.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>

commit | commitdiff | tree

Ivan Boule [Thu, 22 Sep 2011 16:01:35 +0000 (18:01 +0200)]

app: fix volatile read for GCC >= 4.6

When a memory address must be read, for instance a [mapped] PCI register,
the read value is assigned to a local variable that is not used after,
as for instance:
x = *((uint8_t *) mem_addr);

Such instructions do not compile with gcc 4.6.
The fix consists in adding the "volatile" attribute to the accessed data type
and to not assign the read value:
*((volatile uint8_t *) mem_addr);

Signed-off-by: Ivan Boule <ivan.boule@6wind.com>
Acked-by: Olivier Matz <olivier.matz@6wind.com>

commit | commitdiff | tree

Ivan Boule [Tue, 26 Apr 2011 14:45:43 +0000 (16:45 +0200)]

app/testpmd: fix compliance with __rte_mbuf_sanity_check()

Build mbuf and lists of mbufs in a way compliant with the checks performed
by the function __rte_mbuf_sanity_check() when CONFIG_RTE_LIBRTE_MBUF_DEBUG=y

Signed-off-by: Ivan Boule <ivan.boule@6wind.com>
Acked-by: Olivier Matz <olivier.matz@6wind.com>

commit | commitdiff | tree

Damien Millescamps [Tue, 15 May 2012 14:27:04 +0000 (16:27 +0200)]

app/testpmd: fix config crash

A chained topology must always have an increment of 1.
Here, it was 2 if ports number is even.

Signed-off-by: Damien Millescamps <damien.millescamps@6wind.com>
Acked-by: Ivan Boule <ivan.boule@6wind.com>
Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>

commit | commitdiff | tree

Intel [Tue, 12 Mar 2013 11:03:00 +0000 (12:03 +0100)]

lib: minor changes

Signed-off-by: Intel

commit | commitdiff | tree

Adrien Mazarguil [Wed, 19 Dec 2012 16:05:54 +0000 (17:05 +0100)]

lib: fix non-C99 macros definitions in exported headers

The original definitions prevent external programs/libraries from compiling
without warnings when using these headers and -std=gnu99 (relaxed C99 mode).

Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
Acked-by: Ivan Boule <ivan.boule@6wind.com>
Acked-by: Damien Millescamps <damien.millescamps@6wind.com>

commit | commitdiff | tree

Zijie Pan [Thu, 6 Dec 2012 07:36:51 +0000 (15:36 +0800)]

lib: fix uninitialized value

Fix warning "The left expression of the compound assignment
is an uninitialized value".

Signed-off-by: Zijie Pan <zijie.pan@6wind.com>
Acked-by: Ivan Boule <ivan.boule@6wind.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>

commit | commitdiff | tree

Zijie Pan [Fri, 1 Feb 2013 11:43:29 +0000 (12:43 +0100)]

lib: fix uninitialized variables

Fix compilation errors caused by uninitialized variables.

Signed-off-by: Krzysztof Witek <krzysztof.witek@6wind.com>
Signed-off-by: Zijie Pan <zijie.pan@6wind.com>
Acked-by: Ivan Boule <ivan.boule@6wind.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>

commit | commitdiff | tree

Zijie Pan [Tue, 4 Dec 2012 09:17:43 +0000 (17:17 +0800)]

lib: fix unused values

Fix warnings of type "Value stored to 'xxx' is never read".

Signed-off-by: Zijie Pan <zijie.pan@6wind.com>
Acked-by: Ivan Boule <ivan.boule@6wind.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>

commit | commitdiff | tree

Intel [Tue, 12 Mar 2013 11:03:00 +0000 (12:03 +0100)]

kni: various changes

Signed-off-by: Intel

commit | commitdiff | tree

Intel [Tue, 12 Mar 2013 11:03:00 +0000 (12:03 +0100)]

kni: disable IGB_DCA

Signed-off-by: Intel

commit | commitdiff | tree

Intel [Tue, 12 Mar 2013 11:03:00 +0000 (12:03 +0100)]

kni: fix disabling

Signed-off-by: Intel

commit | commitdiff | tree

Ivan Boule [Wed, 23 Jan 2013 13:22:14 +0000 (14:22 +0100)]

kni: fix build with kernel < 2.6.34

Signed-off-by: Ivan Boule <ivan.boule@6wind.com>
Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>

commit | commitdiff | tree

Nicolas Dichtel [Wed, 20 Feb 2013 08:56:56 +0000 (09:56 +0100)]

igb_uio: fix build with kernel >= 3.8

__devinit has been removed in linux commit 54b956b903
("Remove __dev* markings from init.h", 3.8-rc4).

Signed-off-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Acked-by: Ivan Boule <ivan.boule@6wind.com>
Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>

commit | commitdiff | tree

Intel [Tue, 12 Mar 2013 11:03:00 +0000 (12:03 +0100)]

igb: fix RX for Springville (i210)

Signed-off-by: Intel

commit | commitdiff | tree

Adrien Mazarguil [Wed, 19 Dec 2012 16:55:37 +0000 (17:55 +0100)]

pci: allow drivers to be bound several times to the same PCI device

Drivers with the flag RTE_PCI_DRV_MULTIPLE enabled will be bound several
times to the same device until they return an error.

These drivers must internally keep track of each device state in order for
this to work reliably.

This flag currently required for Ethernet adapters with several physical
ports but a single address on the PCI bus.

Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
Acked-by: Ivan Boule <ivan.boule@6wind.com>
Acked-by: Damien Millescamps <damien.millescamps@6wind.com>

commit | commitdiff | tree

Intel [Tue, 12 Mar 2013 11:03:00 +0000 (12:03 +0100)]

mem: 64-bit mempool sizes

Signed-off-by: Intel

commit | commitdiff | tree

Intel [Tue, 12 Mar 2013 11:03:00 +0000 (12:03 +0100)]

mem: error if requesting more segments than MAX_MEMSEG

Signed-off-by: Intel

commit | commitdiff | tree

Intel [Tue, 12 Mar 2013 11:03:00 +0000 (12:03 +0100)]

config: increase MAX_MEMSEG from 32 to 256

Signed-off-by: Intel

commit | commitdiff | tree

Thomas Monjalon [Thu, 2 Aug 2012 15:17:37 +0000 (17:17 +0200)]

eal: fix access to huge page with high address

The returned value of lseek is a virtual address
which can be different from the offset.
Indeed, if the return address has a 64-bit canonical form,
the 16 higher bits are all 1 if bit 47 is 1.

So the check was wrong. It is better to test against an error value.

Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
Acked-by: Damien Millescamps <damien.millescamps@6wind.com>

commit | commitdiff | tree

Intel [Tue, 12 Mar 2013 11:03:00 +0000 (12:03 +0100)]

eal: add bsf32

Signed-off-by: Intel

commit | commitdiff | tree

Intel [Tue, 12 Mar 2013 11:03:00 +0000 (12:03 +0100)]

pmac: integration without lib

This library could be used for pattern matching and ACL.
Code of librte_pmac is not released as Open Source.

Signed-off-by: Intel

commit | commitdiff | tree

Intel [Tue, 12 Mar 2013 11:03:00 +0000 (12:03 +0100)]

cmdline: fix for long input

Signed-off-by: Intel

commit | commitdiff | tree

Didier Pallard [Wed, 9 Jan 2013 16:15:12 +0000 (17:15 +0100)]

mk: fix verbose display of install command

Correctly display executed install command in
rte.install-post.mk install rules

Signed-off-by: Didier Pallard <didier.pallard@6wind.com>
Acked-by: Ivan Boule <ivan.boule@6wind.com>

commit | commitdiff | tree

Zijie Pan [Fri, 1 Feb 2013 11:42:18 +0000 (12:42 +0100)]

mk: allow corei7-avx flag with gcc 4.7

In rte.toolchain-compat.mk, ensure MACHINE_CFLAGS is initialized when
using GCC 4.7. Update the GCC 4.6 test to also include GCC 4.7.

Signed-off-by: Krzysztof Witek <krzysztof.witek@6wind.com>
Signed-off-by: Zijie Pan <zijie.pan@6wind.com>
Acked-by: Ivan Boule <ivan.boule@6wind.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>

commit | commitdiff | tree

Thierry Herbelot [Mon, 2 Apr 2012 12:27:43 +0000 (14:27 +0200)]

mk: fix typo in LDFLAGS for 32-bit

Enable compilation for 32-bit architecture.

Signed-off-by: Thierry Herbelot <thierry.herbelot@6wind.com>
Acked-by: Ivan Boule <ivan.boule@6wind.com>

commit | commitdiff | tree

Samuel Gauthier [Thu, 20 Sep 2012 08:07:44 +0000 (10:07 +0200)]

mk: use $CC to detect toolchain version

Using gcc is wrong, we should use the cross gcc to detect the
version of gcc.

Signed-off-by: Samuel Gauthier <samuel.gauthier@6wind.com>
Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>

commit | commitdiff | tree