Ping
On 5 March 2015 at 17:22, Mike Holmes wrote:
> v2
> Inadvertently send v1 which was != local commits
>
> Mike Holmes (3):
> test: debug: Add unused attribute MACRO
> validation: timer: use unused attribute
> test: debug: replace example with test
>
> test/test_debug.h | 11
Clarify the locations that variables may be declared
Signed-off-by: Mike Holmes
---
CONTRIBUTING | 15 +++
1 file changed, 15 insertions(+)
diff --git a/CONTRIBUTING b/CONTRIBUTING
index 75fb711..ca1ce3f 100644
--- a/CONTRIBUTING
+++ b/CONTRIBUTING
@@ -16,6 +16,21 @@ ODP code shall
Please ignore this
On 11 March 2015 at 19:25, Mike Holmes wrote:
> This documentation was moved from the architecture doc to this API doc.
>
> Signed-off-by: Mike Holmes
> ---
> doc/api_guide_lines.dox | 178
>
> 1 file changed, 178 insertions(+
This documentation was moved from the architecture doc to this API doc.
Signed-off-by: Mike Holmes
---
doc/api_guide_lines.dox | 178
1 file changed, 178 insertions(+)
create mode 100644 doc/api_guide_lines.dox
diff --git a/doc/api_guide_lines.d
I never run it as root usually, but I did just now to check.
Again build.sh had errors and running the for loop directly did not
On 11 March 2015 at 17:43, Maxim Uvarov wrote:
> On 03/11/15 23:47, Mike Holmes wrote:
>
>> for i in {1..10}; do make clean check -e; done
>>
> Mike do you run that u
I simultaneously ran two independent instances of odp with "make check"
and neither had an issue, one had this patch the other did not.
At the same time I used the ./build.sh script from the cleanup branch of
https://git.linaro.org/people/anders.roxell/check-odp.git and it was
pointed at a patche
https://bugs.linaro.org/show_bug.cgi?id=1284
Mike Holmes changed:
What|Removed |Added
Assignee|petri.savolai...@linaro.org |ciprian.ba...@linaro.org
--
You are receiving t
On Wed, Mar 11, 2015 at 1:30 PM, Mike Holmes wrote:
> This docuemtnation has been moved to the API document
>
> Signed-off-by: Mike Holmes
>
Reviewed-by: Bill Fischofer
> ---
> api_guide_lines.dox | 178
>
> 1 file changed, 178 deletions(
On Wed, Mar 11, 2015 at 1:24 PM, Mike Holmes wrote:
> This documentation was moved from the architecture doc to this API doc.
>
> Signed-off-by: Mike Holmes
>
Reviewed-by: Bill Fischofer
> ---
> doc/api_guide_lines.dox | 178
>
> 1 file chang
This docuemtnation has been moved to the API document
Signed-off-by: Mike Holmes
---
api_guide_lines.dox | 178
1 file changed, 178 deletions(-)
delete mode 100644 api_guide_lines.dox
diff --git a/api_guide_lines.dox b/api_guide_lines.dox
de
On Wed, Mar 11, 2015 at 7:59 AM, Petri Savolainen <
petri.savolai...@nokia.com> wrote:
> Cleaned up usage of buf count, thread args and globals.
>
> Signed-off-by: Petri Savolainen
>
Reviewed-and-tested-by: Bill Fischofer
> ---
> test/validation/odp_schedule.c | 102
> ---
I originally was going to point out the 'typo' but my first reaction was
that if should be of rather than in. However, on reflection it seemed that
if was what was intended. However, I agree that this revised wording is
clearer.
On Wed, Mar 11, 2015 at 10:36 AM, Ola Liljedahl
wrote:
> Ensure
On 11 March 2015 at 10:32, Ola Liljedahl wrote:
> These examples won't leak memory continuously so why is this a problem?
> Are we catering to e.g. bare metal environments which may not clean up
> after applications have terminated?
>
I think the tests should be clean so they can be reused in ba
I'm not sure what you're trying to do. You don't need to add files into
the ODP directory structure in order to compile them. Are you just trying
to compile an ODP application?
On Wed, Mar 11, 2015 at 6:39 AM, Ayushman Kashyap
wrote:
> Hi
>
> I want to add some file in *platform/linux-generic*
https://bugs.linaro.org/show_bug.cgi?id=1330
--- Comment #2 from Ciprian Barbu ---
Patch sent:
https://patches.linaro.org/45655/
--
You are receiving this mail because:
You are the assignee for the bug.___
lng-odp mailing list
lng-odp@lists.linaro.org
Ensure that the timeout user_ptr and timer fields are set when the
corresponding timer is immediately cancelled.
https://bugs.linaro.org/show_bug.cgi?id=1313
Signed-off-by: Ola Liljedahl
---
(This document/code contribution attached is provided under the terms of
agreement LES-LTM-21309)
Passes
On 03/11/2015 04:46 PM, Savolainen, Petri (Nokia - FI/Espoo) wrote:
Continuation of cpumask discussion on the call… For creating arbitrary
masks (e.g. fill in a mask with CPU ids of another ODP program in the
system, or testing all possible mask bits), we may need two more calls:
/**
* @return
Continuation of cpumask discussion on the call… For creating arbitrary masks
(e.g. fill in a mask with CPU ids of another ODP program in the system, or
testing all possible mask bits), we may need two more calls:
/**
* @return Maximum number of CPUs a mask can hold
*/
int odp_cpumask_max_cpus(
On 11 March 2015 at 15:26, Ola Liljedahl wrote:
> On 11 March 2015 at 14:49, Savolainen, Petri (Nokia - FI/Espoo) <
> petri.savolai...@nokia.com> wrote:
>
>>
>>
>> > -Original Message-
>> > From: lng-odp-boun...@lists.linaro.org [mailto:lng-odp-
>> > boun...@lists.linaro.org] On Behalf Of
On 11 March 2015 at 14:49, Savolainen, Petri (Nokia - FI/Espoo) <
petri.savolai...@nokia.com> wrote:
>
>
> > -Original Message-
> > From: lng-odp-boun...@lists.linaro.org [mailto:lng-odp-
> > boun...@lists.linaro.org] On Behalf Of ext Ola Liljedahl
> > Sent: Tuesday, March 10, 2015 4:44 PM
On Fri, Mar 06, 2015 at 01:24:52PM +, Stuart Haslam wrote:
> The default logger prints all log levels to stderr. To make things
> easier when debugging failures change it to print only errors to stderr
> and everything else to stdout.
ping - this is ready to be merged.
>
> Signed-off-by: Stu
On Thu, Mar 05, 2015 at 12:12:31PM +, Stuart Haslam wrote:
> There's a potential race condition whereby the test case could start
> running before the virtual ethernet interfaces are fully brought up.
> So replace the the arbitrary delay with a check for the interface's
> operational state befo
https://bugs.linaro.org/show_bug.cgi?id=1337
Mike Holmes changed:
What|Removed |Added
CC||mike.hol...@linaro.org
--- Comment #1 from Mike
https://bugs.linaro.org/show_bug.cgi?id=1117
--- Comment #2 from Ciprian Barbu ---
Patches v1 sent:
https://patches.linaro.org/45656/
https://patches.linaro.org/45657/
https://patches.linaro.org/45658/
--
You are receiving this mail because:
You are on the CC list for the bug.__
This series continues the patch I sent some while ago to replace strtok with
strtok_r in odp_pktio:
https://git.linaro.org/lng/odp.git/commit/273955e6db6bb220f2736d3709e4237c50d04772
The bug that this series fixes only refers to resource leaks, but the initial
strtok_r fix should have been done fo
The odp_l2fwd example leaks some strings allocated during parse_args.
https://bugs.linaro.org/show_bug.cgi?id=1117
CID 56899: Resource leak (RESOURCE_LEAK)
Signed-off-by: Ciprian Barbu
---
example/l2fwd/odp_l2fwd.c | 29 +++--
1 file changed, 15 insertions(+), 14 deleti
The odp_ipsec example leaks some strings allocated during parse_args.
https://bugs.linaro.org/show_bug.cgi?id=1117
CID 56899: Resource leak (RESOURCE_LEAK)
Signed-off-by: Ciprian Barbu
---
example/ipsec/odp_ipsec.c | 30 ++
1 file changed, 14 insertions(+), 16 delet
> -Original Message-
> From: lng-odp-boun...@lists.linaro.org [mailto:lng-odp-
> boun...@lists.linaro.org] On Behalf Of ext Ola Liljedahl
> Sent: Tuesday, March 10, 2015 4:44 PM
> To: lng-odp@lists.linaro.org
> Subject: [lng-odp] [PATCH] linux-generic: odp_timer: set user_ptr for
> cancel
Can we take this off line, it is straying from a technical discussion.
On 11 March 2015 at 09:29, Ola Liljedahl wrote:
> On 11 March 2015 at 13:16, Maxim Uvarov wrote:
>
>> On 03/10/15 18:46, Ola Liljedahl wrote:
>>
>>> On 10 March 2015 at 16:11, Maxim Uvarov wrote:
>>>
On 03/10/15 18:08,
On 11 March 2015 at 13:16, Maxim Uvarov wrote:
> On 03/10/15 18:46, Ola Liljedahl wrote:
>
>> On 10 March 2015 at 16:11, Maxim Uvarov wrote:
>>
>>> On 03/10/15 18:08, Ola Liljedahl wrote:
>>>
On 10 March 2015 at 15:59, Maxim Uvarov
wrote:
> On 03/10/15 17:43, Ola Liljedahl wr
On 11 March 2015 at 18:31, Ciprian Barbu wrote:
> So after a little chat we had, Maxim suggested I remove the ioctl code
> that needs struct ifreq. There is a USE_MAC_ADDR_HACK define in
> odp_ipsec that is now set to 1, removing all that code should fix the
> problem.
>
> The problem was describe
Remove the old hack and switch to using the provided API
This also fixes https://bugs.linaro.org/show_bug.cgi?id=1330
Signed-off-by: Ciprian Barbu
---
example/ipsec/odp_ipsec.c | 58 ++-
1 file changed, 2 insertions(+), 56 deletions(-)
diff --git a/ex
Cleaned up usage of buf count, thread args and globals.
Signed-off-by: Petri Savolainen
---
test/validation/odp_schedule.c | 102 -
1 file changed, 49 insertions(+), 53 deletions(-)
diff --git a/test/validation/odp_schedule.c b/test/validation/odp_schedul
On 03/10/15 18:31, Taras Kondratiuk wrote:
int odp_init_global(odp_init_t *params ODP_UNUSED,
odp_platform_init_t *platform_params ODP_UNUSED)
{
+ odp_proc.log_fn = odp_override_log;
+ odp_proc.abort_fn = odp_override_abort;
+
+ if (params != NULL) {
+
On Wed, Mar 11, 2015 at 2:12 PM, Ola Liljedahl wrote:
> On 11 March 2015 at 10:58, Ciprian Barbu wrote:
>>
>> On Tue, Mar 10, 2015 at 7:19 PM, Ola Liljedahl
>> wrote:
>> > On 10 March 2015 at 17:11, Ciprian Barbu
>> > wrote:
>> >> On Tue, Mar 10, 2015 at 6:06 PM, Ciprian Barbu
>> >> wrote:
>>
On 03/10/15 18:46, Ola Liljedahl wrote:
On 10 March 2015 at 16:11, Maxim Uvarov wrote:
On 03/10/15 18:08, Ola Liljedahl wrote:
On 10 March 2015 at 15:59, Maxim Uvarov wrote:
On 03/10/15 17:43, Ola Liljedahl wrote:
Ensure that the timeout user_ptr and timer fields are set when the
correspond
This patchset is propose to simplify current linux-genric timer code.
There are no functional changes, just removing ifdef from function body.
After that changes function looks much more simple, at least now I can
understand what do they do.
Maxim.
Maxim Uvarov (3):
linux-generic: timer simplif
No functional changes, just make code more readable.
Signed-off-by: Maxim Uvarov
---
platform/linux-generic/odp_timer.c | 76 +++---
1 file changed, 54 insertions(+), 22 deletions(-)
diff --git a/platform/linux-generic/odp_timer.c
b/platform/linux-generic/odp_ti
Current timer_expire() function implementation for linux-generic is
too long and complex. This patch makes it simple to read. No functional
changes.
Signed-off-by: Maxim Uvarov
---
platform/linux-generic/odp_timer.c | 102 ++---
1 file changed, 61 insertions(+), 4
Using ifdefs inside function make it hard to read. No functional changes.
Signed-off-by: Maxim Uvarov
---
platform/linux-generic/odp_timer.c | 19 ---
1 file changed, 16 insertions(+), 3 deletions(-)
diff --git a/platform/linux-generic/odp_timer.c
b/platform/linux-generic/odp_t
On 11 March 2015 at 10:58, Ciprian Barbu wrote:
> On Tue, Mar 10, 2015 at 7:19 PM, Ola Liljedahl
> wrote:
> > On 10 March 2015 at 17:11, Ciprian Barbu
> wrote:
> >> On Tue, Mar 10, 2015 at 6:06 PM, Ciprian Barbu <
> ciprian.ba...@linaro.org> wrote:
> >>> On Tue, Mar 10, 2015 at 4:31 PM, Maxim U
Actually, this is equal to odp_cpu_max():
odp_cpumask_t mask;
odp_cpu_mask(&mask)
odp_cpumask_last(&mask)
So maybe odp_cpu_mask() is only new thing we need.
-Petri
From: lng-odp-boun...@lists.linaro.org
[mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of ext Savolainen, Petri
(Nokia - FI/E
CPU IDs are system dependent. So e.g. odp_cpu_count() == 2 would not
necessarily result a cpumask with cpus 0 and 1 set, but e.g. cpus 7 and 17 set.
Typically, the mask comes from user/system level (user has decided to run the
ODP app on cpus 7 and 17).
We may need two new functions:
/**
* @re
Hi
I want to add some file in *platform/linux-generic* and
*platform/linux-genic/include*. But Not able to compile simple case. Please
see the scenario .
I have a file say:
* demo.c: *
#include
#include
int get_sys_page_size()
{
int page_size;
page_size = getpagesize();
https://bugs.linaro.org/show_bug.cgi?id=1340
Bug ID: 1340
Summary: CID 88056: Error handling issues: odp_pktio.c
Product: OpenDataPlane
Version: 1.0
Hardware: Other
OS: Linux
Status: UNCONFIRMED
Severity:
On Wed, Mar 11, 2015 at 11:36:25AM +0300, Maxim Uvarov wrote:
> Fix CID 88056 to check return code of pool_destroy.
>
> Signed-off-by: Maxim Uvarov
Reviewed-by: Stuart Haslam
> ---
> test/validation/odp_pktio.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test/val
On 03/11/2015 10:37 AM, Radu-Andrei Bulie wrote:
Hi,
Regarding the functions that set the proto flags (e.g
*odp_packet_has_xx_set*) isn’t more
suitable not to pass an int value as second parameter, in fact not to
have any parameter at all because
howsoever inside the function we have some bit
https://bugs.linaro.org/show_bug.cgi?id=1168
--- Comment #1 from Ciprian Barbu ---
The bug was eventually fixed by Petri:
https://git.linaro.org/lng/odp.git/commit/70bd7de7182eaae5c920979e4d25cb512e6fffb8
--
You are receiving this mail because:
You are on the CC list for the bug.___
Hi,
Regarding the functions that set the proto flags (e.g odp_packet_has_xx_set)
isn't more
suitable not to pass an int value as second parameter, in fact not to have any
parameter at all because
howsoever inside the function we have some bit fields, and setting protos would
mean just shifti
Fix CID 88056 to check return code of pool_destroy.
Signed-off-by: Maxim Uvarov
---
test/validation/odp_pktio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test/validation/odp_pktio.c b/test/validation/odp_pktio.c
index 6359452..e022c33 100644
--- a/test/validation/odp_pk
https://bugs.linaro.org/show_bug.cgi?id=1337
Bug ID: 1337
Summary: No means of knowing the size of the CPU mask
(odp_cpumask_t)
Product: OpenDataPlane
Version: 1.0
Hardware: Other
OS: Linux
Statu
51 matches
Mail list logo