The existing test uses odp_cpu_count() to attempt to determine the
capacity of odp_cpumask_t. This was a workaround as there was previously
no reliable method to determine this but the odp_cpumask_setall() API was
added to address this, so use it.
Signed-off-by: Stuart Haslam
Split the majority of the test cases out into a separate file in
preparation for them being reused to perform thrmask tests.
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
---
test/validation/Makefile.inc | 3 +-
test/validation/common/Makefile.am | 4 +-
https://bugs.linaro.org/show_bug.cgi?id=1706
Bug ID: 1706
Summary: odp_term_local() not being called from main thread
Product: OpenDataPlane
Version: 1.1
Hardware: Other
OS: Linux
Status: UNCONFIRMED
The odp_cpumask_def_worker() and odp_cpumask_def_control() APIs are used
to populate a mask with the CPUs available for running threads of the
same type. This does not necessarily mean those CPUs are dedicated to
running threads of that type, on a particular platform the same CPU may
appear in
Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org
---
Changelog notes for next release.
thread mask api will also added before release.
Maxim.
debian/changelog | 50 +++
include/odp/api/version.h | 2 +-
2 files changed, 51
odp_cpumask_def_control() was neglecting to zero out the provided mask.
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
Reviewed-by: Christophe Milard christophe.mil...@linaro.org
---
platform/linux-generic/odp_cpumask.c | 1 +
1 file changed, 1 insertion(+)
diff --git
odph_linux_pthread_create() now returns the number of threads created,
so modify odp_cunit_thread_create() to do the same.
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
Reviewed-by: Christophe Milard christophe.mil...@linaro.org
---
test/validation/common/odp_cunit_common.c | 5 ++---
1
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
---
test/validation/Makefile.inc | 1 +
test/validation/common/.gitignore| 1 +
test/validation/common/Makefile.am | 5 -
test/validation/common/mask_common.h | 12
test/validation/thread/Makefile.am | 2 ++
Refactor the cpumask tests a bit so that they may be reused to perform
the thrmask testing.
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
Reviewed-by: Christophe Milard christophe.mil...@linaro.org
---
test/validation/common/mask_common.c | 478 +--
stdint.h is required for uint32_t
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
Reviewed-by: Christophe Milard christophe.mil...@linaro.org
---
test/validation/common/odp_cunit_common.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/test/validation/common/odp_cunit_common.h
Convert libraries used under test/validation to libtool convenience
libraries (.la files) rather than regular static libraries. These are used
elsewhere in the tree and allow for dependencies to be tracked
automatically.
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
---
.gitignore
Add tests for odp_thrmask_worker() and odp_thrmask_control()
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
Reviewed-by: Christophe Milard christophe.mil...@linaro.org
---
test/validation/thread/thread.c | 70 +
1 file changed, 70 insertions(+)
From: Alexandru Badicioiu alexandru.badici...@linaro.org
This patch adds IPSec protocol processing capabilities to crypto
sesssions. Implementations which have these capabilities in hardware
crypto engines can use the extension to offload the application from
IPSec protocol processing.
On 2015-07-21 14:32, Stuart Haslam wrote:
Split the majority of the test cases out into a separate file in
preparation for them being reused to perform thrmask tests.
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
---
test/validation/Makefile.inc | 7 +-
On 07/22/2015 12:00 PM, Maxim Uvarov wrote:
On 07/17/15 12:15, Nicolas Morey-Chaisemartin wrote:
.open = sock_mmsg_open,
-.close = sock_close_pkt,
+.close = sock_close,
.recv = sock_mmsg_recv_pkt,
.send = sock_mmsg_send_pkt,
Nicolas, I'm going to apply first
On Wed, Jul 22, 2015 at 11:06:00AM +0200, Christophe Milard wrote:
On 2015-07-21 14:32, Stuart Haslam wrote:
Split the majority of the test cases out into a separate file in
preparation for them being reused to perform thrmask tests.
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
On 2015-07-21 14:32, Stuart Haslam wrote:
Refactor the cpumask tests a bit so that they may be reused to perform
the thrmask testing.
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
Reviewed-by: Christophe Milard christophe.mil...@linaro.org
---
test/validation/common/mask_common.c
On 2015-07-21 14:32, Stuart Haslam wrote:
The odp_cpumask_def_worker() and odp_cpumask_def_control() APIs are used
to populate a mask with the CPUs available for running threads of the
same type. This does not necessarily mean those CPUs are dedicated to
running threads of that type, on a
On 07/22/15 13:57, Nicolas Morey-Chaisemartin wrote:
On 07/22/2015 12:00 PM, Maxim Uvarov wrote:
On 07/17/15 12:15, Nicolas Morey-Chaisemartin wrote:
.open = sock_mmsg_open,
-.close = sock_close_pkt,
+.close = sock_close,
.recv = sock_mmsg_recv_pkt,
.send =
Lets bring it up in the call, looks like there are issues of grouping,
naming and how many APIs there are to retrieve the data.
On 22 July 2015 at 05:31, Maxim Uvarov maxim.uva...@linaro.org wrote:
I would add only one odp api call to fetch platform config.
Like
int
On 2015-07-21 14:32, Stuart Haslam wrote:
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
---
test/validation/Makefile.inc | 1 +
test/validation/common/.gitignore| 1 +
test/validation/common/Makefile.am | 5 -
test/validation/common/mask_common.h | 12
That patch 1/1 is merged.
Maxim.
On 07/17/15 12:14, Nicolas Morey-Chaisemartin wrote:
Function names are now coherent with pktio_if_opts names
Signed-off-by: Nicolas Morey-Chaisemartin nmo...@kalray.eu
---
platform/linux-generic/pktio/loop.c | 4 ++--
On 2015-07-21 14:32, Stuart Haslam wrote:
stdint.h is required for uint32_t
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
Reviewed-by: Christophe Milard christophe.mil...@linaro.org
---
test/validation/common/odp_cunit_common.h | 1 +
1 file changed, 1 insertion(+)
diff --git
On 2015-07-21 14:32, Stuart Haslam wrote:
Add tests for odp_thrmask_worker() and odp_thrmask_control()
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
Reviewed-by: Christophe Milard christophe.mil...@linaro.org
---
test/validation/thread/thread.c | 70
On 07/17/15 12:15, Nicolas Morey-Chaisemartin wrote:
.open = sock_mmsg_open,
- .close = sock_close_pkt,
+ .close = sock_close,
.recv = sock_mmsg_recv_pkt,
.send = sock_mmsg_send_pkt,
Nicolas, I'm going to apply first patch. But in that patch it's not
clear
On 2015-07-21 14:32, Stuart Haslam wrote:
The existing test uses odp_cpu_count() to attempt to determine the
capacity of odp_cpumask_t. This was a workaround as there was previously
no reliable method to determine this but the odp_cpumask_setall() API was
added to address this, so use it.
On 2015-07-21 14:32, Stuart Haslam wrote:
odph_linux_pthread_create() now returns the number of threads created,
so modify odp_cunit_thread_create() to do the same.
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
Reviewed-by: Christophe Milard christophe.mil...@linaro.org
---
On 2015-07-21 14:32, Stuart Haslam wrote:
odp_cpumask_def_control() was neglecting to zero out the provided mask.
Signed-off-by: Stuart Haslam stuart.has...@linaro.org
Reviewed-by: Christophe Milard christophe.mil...@linaro.org
---
platform/linux-generic/odp_cpumask.c | 1 +
1 file
I would add only one odp api call to fetch platform config.
Like
int odp_get_platform_config(struct platform_config *config);
where:
struct platform_config {
struct {
some queue settings
} queue;
struct {
some pool settings
} pool;
}
One function
On 07/22/15 15:02, Nicolas Morey-Chaisemartin wrote:
On 07/22/2015 01:23 PM, Maxim Uvarov wrote:
On 07/22/15 13:57, Nicolas Morey-Chaisemartin wrote:
On 07/22/2015 12:00 PM, Maxim Uvarov wrote:
On 07/17/15 12:15, Nicolas Morey-Chaisemartin wrote:
.open = sock_mmsg_open,
-.close
Unifies the format for all pktio_if_ops_t functions
Signed-off-by: Nicolas Morey-Chaisemartin nmo...@kalray.eu
---
v2: Drop suffix for recv/send functionx too
platform/linux-generic/pktio/loop.c| 12 +--
platform/linux-generic/pktio/socket.c | 32
Minor fixes
On 22 July 2015 at 04:26, alexandru.badici...@linaro.org wrote:
From: Alexandru Badicioiu alexandru.badici...@linaro.org
This patch adds IPSec protocol processing capabilities to crypto
sesssions. Implementations which have these capabilities in hardware
crypto engines can use
Hi,
Even with this current pool parameters if the application provides
seg_len pool parameter equal to the expected packet length then the
implementation will have to store the entire packet in a single segment but
it will not be performant in platforms which support segmentation. I
believe this
On 21 July 2015 at 13:05, Nicolas Morey-Chaisemartin nmo...@kalray.eu
wrote:
On 07/20/2015 07:24 PM, Bala Manoharan wrote:
Hi,
Few comments inline
On 20 July 2015 at 22:38, Nicolas Morey-Chaisemartin nmo...@kalray.eu
nmo...@kalray.eu wrote:
Replace current segmentation with an
Building generated this warning message:
copying selected object files to avoid basename conflicts...
Reported-by: Mile Holmes mike.hol...@linaro.org
Signed-off-by: Anders Roxell anders.rox...@linaro.org
---
platform/linux-generic/Makefile.am | 2 +-
In the original packet API design it was proposed that one of the pool
options should be UNSEGMENTED, which says the application does not wish to
see segments at all. The implementation would then either comply or else
fail the pool create if it is unable to support unsegmented pools.
However,
On Tue, Jul 21, 2015 at 3:05 AM, Nicolas Morey-Chaisemartin
nmo...@kalray.eu wrote:
It is the pool responsibility to ensure that there is enough space
available for headroom and tailroom in the segment(s)
Also fixes the issue where ODP_CONFIG_PACKET_SEG_MIN_LEN
ODP_CONFIG_PACKET_HEADROOM
Signed-off-by: Mike Holmes mike.hol...@linaro.org
---
platform/linux-generic/include/odp_buffer_internal.h | 9 -
platform/linux-generic/odp_buffer.c | 8
2 files changed, 17 deletions(-)
diff --git a/platform/linux-generic/include/odp_buffer_internal.h
On 2015-07-20 16:32, Mike Holmes wrote:
Signed-off-by: Mike Holmes mike.hol...@linaro.org
Reviewed-by: Anders Roxell anders.rox...@linaro.org
---
Makefile.inc | 1 -
bootstrap | 2 +-
configure.ac | 1 +
This is a duplicate of Patch http://patches.opendataplane.org/patch/2215/
that I submitted on July 10th.
On Wed, Jul 22, 2015 at 9:35 AM, Mike Holmes mike.hol...@linaro.org wrote:
Signed-off-by: Mike Holmes mike.hol...@linaro.org
---
platform/linux-generic/include/odp_buffer_internal.h | 9
I will cherry-pick original patch from api-next to master.
Maxim.
On 07/22/15 17:46, Bill Fischofer wrote:
This is a duplicate of Patch
http://patches.opendataplane.org/patch/2215/ that I submitted on July
10th.
On Wed, Jul 22, 2015 at 9:35 AM, Mike Holmes mike.hol...@linaro.org
On 07/22/2015 01:23 PM, Maxim Uvarov wrote:
On 07/22/15 13:57, Nicolas Morey-Chaisemartin wrote:
On 07/22/2015 12:00 PM, Maxim Uvarov wrote:
On 07/17/15 12:15, Nicolas Morey-Chaisemartin wrote:
.open = sock_mmsg_open,
-.close = sock_close_pkt,
+.close = sock_close,
42 matches
Mail list logo
