[dpdk-dev] [PATCH] igb/ixgbe: fix index overflow when resetting 4096 queue rings

2013-11-15 Thread Thomas Monjalon
15/11/2013 14:26, Ivan Boule : > On 11/15/2013 02:19 PM, Thomas Monjalon wrote: > > Rings are resetted with a loop because memset cannot be used without > > issuing a warning about volatile casting. > > The index of the loop was a 16-bit variable which is is sufficient for > > ring entries number b

[dpdk-dev] [PATCH] mk: fix EXTRA_LDFLAGS override

2013-11-15 Thread Thomas Monjalon
15/11/2013 15:22, Julien Courtat : > The GCC prefix -Wl was ignored because the command line value has higher > priority. It ended in impossibilty for GCC to pass parameters to LD. > The prefixed value must override the command line one. > > Signed-off-by: Julien Courtat Acked and applied. Than

[dpdk-dev] [PATCH] mk: fix command-line dependency check for quoted strings

2013-11-15 Thread Thomas Monjalon
15/11/2013 15:38, Adrien Mazarguil : > Before recompiling a file, rte.compile-pre.mk checks whether the command > line is different from the previous one. > > This is done by storing for each object file the entire command line in a > kind of dependency file with a .cmd extension (see obj2cmd). If

[dpdk-dev] [PATCH] mk: fix command-line dependency check for quoted strings

2013-11-15 Thread Adrien Mazarguil
Before recompiling a file, rte.compile-pre.mk checks whether the command line is different from the previous one. This is done by storing for each object file the entire command line in a kind of dependency file with a .cmd extension (see obj2cmd). If that file exists, the line is retrieved first

[dpdk-dev] [PATCH] mk: fix EXTRA_LDFLAGS override

2013-11-15 Thread Julien Courtat
The GCC prefix -Wl was ignored because the command line value has higher priority. It ended in impossibilty for GCC to pass parameters to LD. The prefixed value must override the command line one. Signed-off-by: Julien Courtat --- mk/rte.app.mk|2 +- mk/rte.shared.mk |2 +- 2 files

[dpdk-dev] [PATCH] igb/ixgbe: fix index overflow when resetting 4096 queue rings

2013-11-15 Thread Ivan Boule
On 11/15/2013 02:19 PM, Thomas Monjalon wrote: > Rings are resetted with a loop because memset cannot be used without > issuing a warning about volatile casting. > The index of the loop was a 16-bit variable which is is sufficient for > ring entries number but not for the byte size of the whole rin

[dpdk-dev] [PATCH] igb/ixgbe: fix index overflow when resetting 4096 queue rings

2013-11-15 Thread Thomas Monjalon
Rings are resetted with a loop because memset cannot be used without issuing a warning about volatile casting. The index of the loop was a 16-bit variable which is is sufficient for ring entries number but not for the byte size of the whole ring. The overflow happens when rings are configured for 4