Hi, Bruce
Your patch works! I applied the patch, and the problem disappeared.
Thanks ^_^
/ChenXiaodong
> Date: Wed, 22 Apr 2015 14:34:17 +0100
> From: bruce.richardson at intel.com
> To: ch.xd at live.cn
> CC: pablo.de.lara.guarch at intel.com; dev at dpdk.org
> Subject: Re: [dpdk-dev] problem:
update of used->idx and read of avail->flags could be reordered.
memory fence should be used to ensure the order, otherwise guest could see a
stale used->idx value after it toggles the interrupt suppression flag.
Signed-off-by: Huawei Xie
---
lib/librte_vhost/vhost_rxtx.c | 3 +++
1 file
Try going here and you can input your email and password to unsubscribe.
http://dpdk.org/ml/listinfo/dev
Or just here for all email lists:
http://dpdk.org/ml
Regards,
++Keith
On 4/22/15, 5:23 PM, "Vipin Agrawal" wrote:
>What do I need to do to not receive any more emails?
>
>Thanks,
>Vipin
On 2015/04/21 22:15, Iremonger, Bernard wrote:
>> -Original Message-
>> From: Tetsuya Mukawa [mailto:mukawa at igel.co.jp]
>> Sent: Friday, March 27, 2015 8:39 AM
>> To: dev at dpdk.org
>> Cc: Iremonger, Bernard; Richardson, Bruce; david.marchand at 6wind.com;
>> Tetsuya Mukawa
>>
BTW, the BIOS fix has been posted for the NUMA node reporting issue in Linux
kernel when COD is disabled. Now the /sys/devices/*/*/numa_node will provide
you the correct information.
Regards
Kannan Babu
-Original Message-
From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Bruce
On 2015-04-22 17:33, Ravi Kerur wrote:
> +/**
> + * Compare bytes between two locations. The locations must not overlap.
> + *
> + * @note This is implemented as a macro, so it's address should not be taken
> + * and care is needed as parameter expressions may be evaluated multiple
> times.
> + *
On Thu, Apr 23, 2015 at 09:24:52AM +0200, Pawel Wodkowski wrote:
> On 2015-04-22 17:33, Ravi Kerur wrote:
> >+/**
> >+ * Compare bytes between two locations. The locations must not overlap.
> >+ *
> >+ * @note This is implemented as a macro, so it's address should not be taken
> >+ * and care is
On Wed, Apr 22, 2015 at 03:57:44PM -0700, Stephen Hemminger wrote:
> On Wed, 22 Apr 2015 14:05:48 -0700
> Ravi Kerur wrote:
>
> > Add code to set names to threads via pthread APIs.
> > In Linux corresponding _getname_ is available, however, FreeBSD
> > doesn't have corresponding _get_name API
There are no memcpy functions in rte_ip.h so there is no need to include
rte_memcpy.h in that file.
Signed-off-by: Bruce Richardson
---
lib/librte_net/rte_ip.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/lib/librte_net/rte_ip.h b/lib/librte_net/rte_ip.h
index 74c9ced..cdbce20 100644
---
A large number of our header files and libraries are dependent on one another,
which can lead to problems with circular dependencies if trying to tie some of
those libraries together, e.g. when prototyping with pktdev, or other schemes
to get a common API for ethdev/rings/KNI. :-)
One small way
The ethdev header file includes the mbuf header file, but it does not
need to do so as it only uses pointers to the struct rte_mbuf type, and
does not use any of the mbuf internals, nor any of the mbuf functions or
macros. Therefore the inclusion is unnecessary, and can be replaced by a
forward
The virtio_net header file includes the mbuf header file, but it does not
need to do so as it only uses pointers to the struct rte_mbuf type, and
does not use any of the mbuf internals, nor any of the mbuf functions or
macros. Therefore the inclusion is unnecessary, and can be replaced by a
The virtqueue header files include the mbuf header file, but they do not
need to do so as they only use pointers to the struct rte_mbuf type, and
do not use any of the mbuf internals, nor any of the mbuf functions or
macros. Therefore the inclusion is unnecessary, and can be replaced by a
forward
The rte_table header file includes the mbuf header file, but it does not
need to do so as it only uses pointers to the struct rte_mbuf type, and
does not use any of the mbuf internals, nor any of the mbuf functions or
macros. Therefore the inclusion is unnecessary, and can be replaced by a
forward
The distributor header file includes the mbuf header file, but it does not
need to do so as it only uses pointers to the struct rte_mbuf type, and
does not use any of the mbuf internals, nor any of the mbuf functions or
macros. Therefore the inclusion is unnecessary, and can be replaced by a
The pipeline header file includes the mbuf header file, but it does not
need to do so as it only uses pointers to the struct rte_mbuf type, and
does not use any of the mbuf internals, nor any of the mbuf functions or
macros. Therefore the inclusion is unnecessary, and can be replaced by a
forward
2015-04-23 13:49, Bruce Richardson:
> There are no memcpy functions in rte_ip.h so there is no need to include
> rte_memcpy.h in that file.
>
> Signed-off-by: Bruce Richardson
Applied, thanks
On Thu, Apr 23, 2015 at 12:24 AM, Pawel Wodkowski <
pawelx.wodkowski at intel.com> wrote:
> On 2015-04-22 17:33, Ravi Kerur wrote:
>
>> +/**
>> + * Compare bytes between two locations. The locations must not overlap.
>> + *
>> + * @note This is implemented as a macro, so it's address should not
On Thu, Apr 23, 2015 at 2:23 AM, Ananyev, Konstantin <
konstantin.ananyev at intel.com> wrote:
>
>
> > -Original Message-
> > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bruce Richardson
> > Sent: Thursday, April 23, 2015 9:12 AM
> > To: Wodkowski, PawelX
> > Cc: dev at
On Thu, Apr 23, 2015 at 06:53:44AM -0700, Ravi Kerur wrote:
> On Thu, Apr 23, 2015 at 2:23 AM, Ananyev, Konstantin <
> konstantin.ananyev at intel.com> wrote:
>
> >
> >
> > > -Original Message-
> > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bruce Richardson
> > > Sent:
pcap_rx_queue/pcap_tx_queue should store it's own copy of name/type values,
not the pointer to temporary allocated space.
Signed-off-by: Konstantin Ananyev
---
lib/librte_pmd_pcap/rte_eth_pcap.c | 51 +-
1 file changed, 34 insertions(+), 17 deletions(-)
diff
Changes in v7 includes
Fix clang and ICC compilation errors reported by Bruce.
Remove pthread _setname_ APIs introduced in v6 as a separate
patch is sent.
Testing:
Linux - Ubuntu x86_64 14.04
Compilation successful (x86_64-native-linuxapp-gcc
and x86_64-native-linuxapp-clang).
"make test" results
Changes in v7
Remove _setname_ pthread calls.
Use rte_gettid() API in RTE_LOG to print thread_id.
Changes in v6
Remove RTE_EXEC_ENV_BSDAPP from eal_common_thread.c file.
Add pthread_setname_np/pthread_set_name_np for Linux/FreeBSD
respectively. Plan to use _getname_ in RTE_LOG when available.
Use
Changes in v7
Fix clang and ICC compilation errors.
Changes in v6
Split eal_common_system.c and eal_common_runtime.c into
eal_common_sysfs.c
eal_common_mem_cfg.c
eal_common_proc_type.c
eal_comm_app_usage.c
based on functionality.
Changes in v5
Rebase to latest code.
Changes in v4
Remove
Changes in v7
None
Changes in v6
None
Changes in v5
Rebase to latest code.
Changes in v4
Implement cpu_detected() for BSD.
Have common RTE_LOG for Linux and BSD in rte_eal_cpu_init().
Remove RTE_EXEC_ENV_BSDAPP in common file.
Changes in v3
Changed subject to be more explicit on file name
Changes in v7
None
Changes in v6
Added new line between Copyright and header file inclusion
in eal_common_timer.c.
Changes in v5
Rebase to latest code.
Changes in v4
Removed extern declaration of eal_tsc_resolution_hz,
instead provided _set_ API.
Make set_tsc_freq_from_clock as wrapper function
Changes in v7
None
Changes in v6
Removed unnecessary comments in function declaration.
Changes in v5
Rebase to latest code.
Changes in v4
Make rte_eal_hugepage_init and rte_eal_hugepage_attach as
wrapper functions for BSD.
Changes in v3
Changed subject to be more explicit on file name
Changes in v7
Rebase to latest code.
Changes in v6
Split changes due to complexity. v6 includes moving
rte_eal_pci_probe_one_driver function and its associated
utility functions only.
Changes in v5
Rebase to latest code.
Removed RTE_EXEC_ENV_BSDAPP from earlier changes.
Changes in v4
Move
On Thu, Apr 23, 2015 at 7:00 AM, Bruce Richardson <
bruce.richardson at intel.com> wrote:
> On Thu, Apr 23, 2015 at 06:53:44AM -0700, Ravi Kerur wrote:
> > On Thu, Apr 23, 2015 at 2:23 AM, Ananyev, Konstantin <
> > konstantin.ananyev at intel.com> wrote:
> >
> > >
> > >
> > > > -Original
29 matches
Mail list logo