> > Do you agree to add this comment (before applying this *needed* patch)?
> >
> > +# Workaround lack of DT_NEEDED entry
> >
> Acked-by: Neil Horman
Applied with above comment and Neil's explanations in commit log.
Thanks
> We can solve the larger dependency chain issue later if we need to
On Fri, Nov 28, 2014 at 03:31:00PM +, Bruce Richardson wrote:
> When compiling with clang, errors were being emitted due to truncation
> of values when assigning to the tx_offload_mask bit fields.
>
> dpdk.org/lib/librte_pmd_ixgbe/ixgbe_rxtx.c:404:27: fatal error: implicit
> truncation from '
On 29/11/14 4:47 pm, "Thomas Monjalon" wrote:
>29/11/2014 07:22, Sujith Sankar :
>> Sorry for the typo in the subject. It is enicpmd.
>
>Actually no, it should be enic.
Usually we use enic to denote the kernel mode driver and enicpmd to denote
the PMD.
But it is alright to follow what?s being
This patch fixes the warnings and error reported by clang compiler on Linux.
Reported-by: Bruce Richardson
Signed-off-by: Sujith Sankar
---
lib/librte_pmd_enic/enic.h | 40 +
lib/librte_pmd_enic/enic_compat.h | 1 +
lib/librte_pmd_enic/enic_main.c
29/11/2014 07:22, Sujith Sankar :
> Sorry for the typo in the subject. It is enicpmd.
Actually no, it should be enic.
For virtio, it is not virtiopmd, for ixgbe, it is not ixgbepmd, etc :)
Talking about title, please start with lower case and a verb, e.g. fix build
with clang.
To have a good ide
error: implicit truncation from 'int' to bitfield
changes value from -1 to 127 [-Werror,-Wbitfield-constant-conversion]
tx_offload_mask.l2_len = ~0;
Converted to use correct bit values
tx_offload_mask.l2_len = 0x7f;
Sign
On Tue, Nov 25, 2014 at 04:18:02PM +, Pablo de Lara wrote:
> From: Alan Carew
>
> The manager is responsible for adding communications channels to the Monitor
> thread, tracking and reporting VM state and employs the libvirt API for
> synchronization with the KVM Hypervisor. The manager inter
On Fri, Nov 28, 2014 at 04:35:04PM +0100, Thomas Monjalon wrote:
> Hi Sergio, Neil,
>
> Do you agree to add this comment (before applying this *needed* patch)?
>
> +# Workaround lack of DT_NEEDED entry
>
Acked-by: Neil Horman
We can solve the larger dependency chain issue later if we need to
N
Hi all,
Sorry for the typo in the subject. It is enicpmd.
Bruce,
Please try out this patch on FreeBSD and let me know if any fix is
required.
Thanks,
-Sujith
On 29/11/14 12:47 pm, "Sujith Sankar (ssujith)" wrote:
>This patch fixes the warnings and error reported by clang compiler on
>Linux.
>
9 matches
Mail list logo
