https://bugs.linaro.org/show_bug.cgi?id=1830
Bala Manoharan changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
On Tue, Dec 29, 2015 at 6:25 AM, Petri Savolainen <
petri.savolai...@nokia.com> wrote:
> Improved errno specification. Specified that errno may be
> set (on failure) also when API spec does not demand it. Errno
> values are implementation specific.
>
> Signed-off-by: Petri Savolainen
Signed-off-by: Maxim Uvarov
---
CHANGELOG | 104 ++
1 file changed, 104 insertions(+)
diff --git a/CHANGELOG b/CHANGELOG
index 4e9db28..bd5b998 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,3 +1,107 @@
Signed-off-by: Maxim Uvarov
---
pkg/rpm/odp.spec | 2 ++
1 file changed, 2 insertions(+)
diff --git a/pkg/rpm/odp.spec b/pkg/rpm/odp.spec
index e76fe58..00e1a61 100644
--- a/pkg/rpm/odp.spec
+++ b/pkg/rpm/odp.spec
@@ -68,6 +68,8 @@ and guides in HTMLformats.
%post -p
Maxim Uvarov (4):
changelog: update from 1.5 to 1.6
debian changelog: update from 1.5 to 1.6
update version number from v1.5.0.0 to v1.6.0.0
rpm changelog: update from 1.5 to 1.6
CHANGELOG | 104 ++
include/odp/api/version.h |
v3: fix Anders comments
v2: included and excluded patches specified by Petri
CI also passed.
Maxim Uvarov (5):
changelog: update from 1.5 to 1.6
debian changelog: update from 1.5 to 1.6
debian fix
update version number from v1.5.0.0 to v1.6.0.0
rpm changelog: update from 1.5 to 1.6
---
pkg/debian/changelog | 1 -
1 file changed, 1 deletion(-)
diff --git a/pkg/debian/changelog b/pkg/debian/changelog
index 4a9a7eb..1a0b23c 100644
--- a/pkg/debian/changelog
+++ b/pkg/debian/changelog
@@ -3,7 +3,6 @@ opendataplane (1.6.0.0-1) unstable; urgency=low
-- Maxim Uvarov
On 12/29/2015 18:06, Anders Roxell wrote:
On 2015-12-29 18:01, Maxim Uvarov wrote:
Signed-off-by: Maxim Uvarov
---
pkg/rpm/odp.spec | 2 ++
1 file changed, 2 insertions(+)
diff --git a/pkg/rpm/odp.spec b/pkg/rpm/odp.spec
index e76fe58..00e1a61 100644
---
Will it be merged for 1.6?
-Original Message-
From: Hemant Agrawal [mailto:hem...@freescale.com]
Sent: Tuesday, December 15, 2015 5:32 PM
To: lng-odp@lists.linaro.org
Cc: Agrawal Hemant-B10814
Subject: [API-NEXT PATCHv5] api: init: removed platform_init struct
Signed-off-by: Maxim Uvarov
---
include/odp/api/version.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/odp/api/version.h b/include/odp/api/version.h
index 5498c9f..95fac3c 100644
--- a/include/odp/api/version.h
+++
Signed-off-by: Maxim Uvarov
---
include/odp/api/version.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/odp/api/version.h b/include/odp/api/version.h
index 5498c9f..95fac3c 100644
--- a/include/odp/api/version.h
+++
Signed-off-by: Maxim Uvarov
---
pkg/rpm/odp.spec | 2 ++
1 file changed, 2 insertions(+)
diff --git a/pkg/rpm/odp.spec b/pkg/rpm/odp.spec
index e76fe58..6c675a7 100644
--- a/pkg/rpm/odp.spec
+++ b/pkg/rpm/odp.spec
@@ -68,6 +68,8 @@ and guides in HTMLformats.
%post -p
Please skip that version, there is v4.
Maxim.
On 12/29/2015 18:40, Maxim Uvarov wrote:
v3: fix Anders comments
v2: included and excluded patches specified by Petri
CI also passed.
Maxim Uvarov (5):
changelog: update from 1.5 to 1.6
debian changelog: update from 1.5 to 1.6
debian fix
Signed-off-by: Maxim Uvarov
---
include/odp/api/version.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/odp/api/version.h b/include/odp/api/version.h
index 5498c9f..95fac3c 100644
--- a/include/odp/api/version.h
+++
Signed-off-by: Maxim Uvarov
---
pkg/debian/changelog | 5 +
1 file changed, 5 insertions(+)
diff --git a/pkg/debian/changelog b/pkg/debian/changelog
index 7680cf1..1a0b23c 100644
--- a/pkg/debian/changelog
+++ b/pkg/debian/changelog
@@ -1,3 +1,8 @@
+opendataplane
Signed-off-by: Maxim Uvarov
---
pkg/rpm/odp.spec | 2 ++
1 file changed, 2 insertions(+)
diff --git a/pkg/rpm/odp.spec b/pkg/rpm/odp.spec
index e76fe58..6c675a7 100644
--- a/pkg/rpm/odp.spec
+++ b/pkg/rpm/odp.spec
@@ -68,6 +68,8 @@ and guides in HTMLformats.
%post -p
Signed-off-by: Maxim Uvarov
---
CHANGELOG | 104 ++
1 file changed, 104 insertions(+)
diff --git a/CHANGELOG b/CHANGELOG
index 4e9db28..bd5b998 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,3 +1,107 @@
Signed-off-by: Maxim Uvarov
---
pkg/debian/changelog | 6 ++
1 file changed, 6 insertions(+)
diff --git a/pkg/debian/changelog b/pkg/debian/changelog
index 7680cf1..4a9a7eb 100644
--- a/pkg/debian/changelog
+++ b/pkg/debian/changelog
@@ -1,3 +1,9 @@
+opendataplane
In general I like this restructure. I'll add this to the topics for the
ARCH call on the 4th. Some general comments:
1. Renaming SCHED_SYNC_NONE to SCHED_SYNC_PARALLEL is good as we talk
about Parallel, Atomic, and Ordered queues so this is reducing confusion.
2. If we're introducing a
Merged,
Maxim.
On 12/29/2015 20:09, Bill Fischofer wrote:
On Tue, Dec 29, 2015 at 6:25 AM, Petri Savolainen
> wrote:
Improved errno specification. Specified that errno may be
set (on failure) also when API spec does not
My question relates to pktio when ODP tasks are implemented as unix
processes (as opposed to threads).
I can see that the pktio_entry struct used is allocated as shared mem.
If I take the socket pktio as an example, the socket file descriptor is
stored in th pktio struct.
In other words, the
On 12/29/2015 21:36, Agrawal Hemant wrote:
Will it be merged for 1.6?
I think no.
1. Petri review is needed for api change.
2. To include it I need to execute all tests again and rework changelog.
That takes some significant time.
So I think it will be scheduled for 1.7
Maxim.
From: Hongbo Zhang
v1->v2 changes:
- don't create arch/arm/ since there isn't implementation now, use arch/linux
as default choice
- create symlink to arch/linux/odp_cpu_cycles.c for powerpc, if absent this
arch cannot be compiled
- add some clean-ups patches 8~11,
From: Hongbo Zhang
This is a pre-step for separating platform specific codes into their own
arch files.
Value of cpu_arch_str will be set later in separate arch parser code.
Signed-off-by: Hongbo Zhang
---
From: Hongbo Zhang
v1->v2 changes:
v1 notes:
This patch set separates the CPU info codes into their own platform sepcific
files.
It is common sence that the top general layer call an uniform interface to
initialize some plarform specific data structures, and this
From: Hongbo Zhang
This function will be moved to their own platform specific files too,
it is better to be named with a odp_ prefix.
Signed-off-by: Hongbo Zhang
---
platform/linux-generic/odp_system_info.c | 12 ++--
1 file changed, 6
From: Hongbo Zhang
It is common way that the general top layer calls a uniform interface to
initialize data structrure for different plarform, this patch introduces
odp_sysinfo_parser() instead of using different interfaces for each
platform, and also removes using
From: Hongbo Zhang
This patch moves the x86 system info codes into the newly added x86
specific platform file.
Signed-off-by: Hongbo Zhang
---
platform/linux-generic/Makefile.am | 6 +-
From: Hongbo Zhang
This patch moves the PowerPC system info codes into the newly added PowerPC
specific platform file.
This patch also creates syslink to arch/linux/odp_cpu_cycles.c for PowerPC.
Signed-off-by: Hongbo Zhang
---
configure.ac
From: Hongbo Zhang
This patch moves the MIPS system info codes into the newly added MIPS
specific platform file.
Signed-off-by: Hongbo Zhang
---
platform/linux-generic/Makefile.am | 1 +
From: Hongbo Zhang
This patch moves the ARM system info codes into the general arch file,
ARM sysinfo isn't implemented now and there is no need to create specific
arch file for it, but this can serve as the default implementaton instead.
Signed-off-by: Hongbo Zhang
From: Hongbo Zhang
The values of cpu_arch_str were deleted due to codes reorganization, this
patch adds values to them again.
Signed-off-by: Hongbo Zhang
---
platform/linux-generic/arch/mips64/odp_sysinfo_parse.c | 1 +
From: Hongbo Zhang
All the CPU frequency API return value type is uint64_t, this patch makes
them return 0 on failure instead of -1.
Signed-off-by: Hongbo Zhang
---
include/odp/api/cpu.h | 4
From: Hongbo Zhang
When per-CPU framework was introduced, it was only implemented on x86,
for other platforms, only the model_str[0] and cpu_hz[0] are set to pass
compile, this patch set all values for model_str[] and cpu_hz[] on the
MIPS platform.
Signed-off-by: Hongbo
From: Hongbo Zhang
When per-CPU framework was introduced, it was only implemented on x86,
for other platforms, only the model_str[0] and cpu_hz[0] are set to pass
compile, this patch set all values for model_str[] and cpu_hz[] on the
PowerPC platform.
Signed-off-by:
This is an API change, so the patch/git log subject should be something like
(instead of linux-generic: sysinfo: ...):
api: cpu: frequency calls return 0 on failure
The API change is OK.
-Petri
> -Original Message-
> From: EXT hongbo.zh...@linaro.org
On 29 December 2015 at 17:22, Savolainen, Petri (Nokia - FI/Espoo)
wrote:
> This is an API change, so the patch/git log subject should be something like
> (instead of linux-generic: sysinfo: ...):
>
>
> api: cpu: frequency calls return 0 on failure
>
>
Sure, my
> -Original Message-
> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT Maxim
> Uvarov
> Sent: Monday, December 28, 2015 6:40 PM
> To: lng-odp@lists.linaro.org
> Subject: [lng-odp] [API-NEXT PATCHv2 2/4] helper: define
> odph_pktio_wailt_linkup() helper to wait link
Memory barriers should go into v1.6. Those are called in validation tests and
additional test cases would not be trivial to add (how to validate at 100%
accuracy that a memory barrier did its job).
api: sync: removed odp_sync_stores
api: barrier: added memory barriers
> + - api:
From: Hongbo Zhang
This patch moves the ARM system info codes into the general arch file,
ARM sysinfo isn't implemented now and there is no need to create specific
arch file for it, but this can serve as the default implementaton instead.
Signed-off-by: Hongbo Zhang
From: Hongbo Zhang
This patch moves the MIPS system info codes into the newly added MIPS
specific platform file.
Signed-off-by: Hongbo Zhang
---
platform/linux-generic/Makefile.am | 1 +
On 12/29/2015 15:14, Savolainen, Petri (Nokia - FI/Espoo) wrote:
-Original Message-
From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT
Maxim Uvarov
Sent: Tuesday, December 29, 2015 1:46 PM
To: Elo, Matias (Nokia - FI/Espoo); lng-odp@lists.linaro.org
Subject: Re:
Application cannot dequeue from sink queues, it can only enqueue.
This queue type can be used for packet output, but also for other
ODP features that will consume all enqueued events.
Signed-off-by: Petri Savolainen
---
include/odp/api/queue.h
On 12/29/2015 11:06, Elo, Matias (Nokia - FI/Espoo) wrote:
-Original Message-
From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT Maxim
Uvarov
Sent: Monday, December 28, 2015 6:40 PM
To: lng-odp@lists.linaro.org
Subject: [lng-odp] [API-NEXT PATCHv2 2/4] helper:
Improved errno specification. Specified that errno may be
set (on failure) also when API spec does not demand it. Errno
values are implementation specific.
Signed-off-by: Petri Savolainen
---
include/odp/api/errno.h | 37 +++--
1 file
On 2015-12-29 18:01, Maxim Uvarov wrote:
> Signed-off-by: Maxim Uvarov
> ---
> pkg/rpm/odp.spec | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/pkg/rpm/odp.spec b/pkg/rpm/odp.spec
> index e76fe58..00e1a61 100644
> --- a/pkg/rpm/odp.spec
> +++
On 12/29/2015 14:30, Elo, Matias (Nokia - FI/Espoo) wrote:
-Original Message-
From: EXT Maxim Uvarov [mailto:maxim.uva...@linaro.org]
Sent: Tuesday, December 29, 2015 12:56 PM
To: Elo, Matias (Nokia - FI/Espoo) ; lng-
o...@lists.linaro.org
Subject: Re: [lng-odp]
There are also those max freq API calls. Maybe it's just for diagnostics or
logging purposes, but it's valuable information if the system you are running
on is setup for maximum frequency, or if it's scaling the frequency.
These are exactly the figures I'd want to see (sampled) in my log.
Signed-off-by: Maxim Uvarov
---
pkg/debian/changelog | 6 ++
1 file changed, 6 insertions(+)
diff --git a/pkg/debian/changelog b/pkg/debian/changelog
index 7680cf1..4a9a7eb 100644
--- a/pkg/debian/changelog
+++ b/pkg/debian/changelog
@@ -1,3 +1,9 @@
+opendataplane
Signed-off-by: Maxim Uvarov
---
pkg/rpm/odp.spec | 2 ++
1 file changed, 2 insertions(+)
diff --git a/pkg/rpm/odp.spec b/pkg/rpm/odp.spec
index e76fe58..00e1a61 100644
--- a/pkg/rpm/odp.spec
+++ b/pkg/rpm/odp.spec
@@ -68,6 +68,8 @@ and guides in HTMLformats.
%post -p
so what is summary for that api?
odp_cpu_hz_current()
odp_cpu_hz()
very unstable on platforms like my x86 laptop. Example:
cat /proc/cpuinfo |grep "cpu MHz"
cpu MHz: 800.000
cpu MHz: 800.000
cpu MHz: 2501.000
cpu MHz: 800.000
cpu MHz: 800.000
cpu MHz
From: Hongbo Zhang
This function will be moved to their own platform specific files too,
it is better to be named with a odp_ prefix.
Signed-off-by: Hongbo Zhang
---
platform/linux-generic/odp_system_info.c | 12 ++--
1 file changed, 6
From: Hongbo Zhang
This patch moves the x86 system info codes into the newly added x86
specific platform file.
Signed-off-by: Hongbo Zhang
---
platform/linux-generic/Makefile.am | 6 +-
From: Hongbo Zhang
It is common way that the general top layer calls a uniform interface to
initialize data structrure for different plarform, this patch introduces
odp_sysinfo_parser() instead of using different interfaces for each
platform, and also removes using
Merged,
Maxim.
On 12/28/2015 20:27, Bill Fischofer wrote:
On Wed, Dec 23, 2015 at 3:33 AM, Balasubramanian Manoharan
> wrote:
fixes: https://bugs.linaro.org/show_bug.cgi?id=1830
Signed-off-by: Balasubramanian Manoharan
On 12/29/2015 16:43, Savolainen, Petri (Nokia - FI/Espoo) wrote:
There are also those max freq API calls. Maybe it's just for diagnostics or
logging purposes, but it's valuable information if the system you are running
on is setup for maximum frequency, or if it's scaling the frequency.
These
Send a quick v3 with only changing of patch 8/11 title since I'll be
off and back on Jan 6.
On 29 December 2015 at 18:31, wrote:
> From: Hongbo Zhang
>
> All the CPU frequency API return value type is uint64_t, this patch makes
> them return 0
From: Hongbo Zhang
The values of cpu_arch_str were deleted due to codes reorganization, this
patch adds values to them again.
Signed-off-by: Hongbo Zhang
---
platform/linux-generic/arch/mips64/odp_sysinfo_parse.c | 1 +
From: Hongbo Zhang
When per-CPU framework was introduced, it was only implemented on x86,
for other platforms, only the model_str[0] and cpu_hz[0] are set to pass
compile, this patch set all values for model_str[] and cpu_hz[] on the
PowerPC platform.
Signed-off-by:
On 12/29/2015 11:25, Bala Manoharan wrote:
The following api changes can also be merged to 1.6
api/linux-generic: classification: rename odp_drop_e to odp_cls_drop_t
api/validation/linux-generic: classification: implement class of
service create api
Regards,
Bala
Just checked that is included
Aligned pktin and pktout modes with renamed queue types.
ODP_PKTIN_MODE_RECV => ODP_PKTIN_MODE_DIRECT
ODP_PKTIN_MODE_SCHED => ODP_PKTIN_MODE_SCHED_QUEUE
ODP_PKTIN_MODE_POLL => ODP_PKTIN_MODE_DIRECT_QUEUE
ODP_PKTOUT_MODE_SEND => ODP_PKTOUT_MODE_DIRECT
Direct pktio modes (_RECV/_SEND) are
Term "poll" does not describe queue type accurately. Many resources
can be polled. For example, application polls the scheduler to
dequeue from "sched" type queues. Term "direct queues" describe better
that application directly dequeues from these queues (instead of using
the scheduler).
v2: included and excluded patches specified by Petri
CI also passed.
Maxim.
Maxim Uvarov (4):
changelog: update from 1.5 to 1.6
debian changelog: update from 1.5 to 1.6
update version number from v1.5.0.0 to v1.6.0.0
rpm changelog: update from 1.5 to 1.6
CHANGELOG | 104
Signed-off-by: Maxim Uvarov
---
include/odp/api/version.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/odp/api/version.h b/include/odp/api/version.h
index 5498c9f..95fac3c 100644
--- a/include/odp/api/version.h
+++
Signed-off-by: Maxim Uvarov
---
CHANGELOG | 104 ++
1 file changed, 104 insertions(+)
diff --git a/CHANGELOG b/CHANGELOG
index 4e9db28..bd5b998 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,3 +1,107 @@
From: Hongbo Zhang
v2->v3 changes:
use "api: cpu:" tag in patch 8/11 title instead of "linux-generic: sysinfo"
v1->v2 changes:
- don't create arch/arm/ since there isn't implementation now, use arch/linux
as default choice
- create symlink to
From: Hongbo Zhang
This is a pre-step for separating platform specific codes into their own
arch files.
Value of cpu_arch_str will be set later in separate arch parser code.
Signed-off-by: Hongbo Zhang
---
On 12/29/2015 11:13, Savolainen, Petri (Nokia - FI/Espoo) wrote:
Memory barriers should go into v1.6. Those are called in validation tests and
additional test cases would not be trivial to add (how to validate at 100%
accuracy that a memory barrier did its job).
api: sync: removed
Added packet output mode for sink queues (odp_queue_t). This
mode is replacement of usage of default output queue, which
defined ODP_QUEUE_TYPE_PKTOUT. Direct mode gives optimal
packet output performance but sink queue mode enable better
application modularity (a module does not have to know if
This API changing patch:
Reviewed-by: Petri Savolainen
> -Original Message-
> From: EXT hongbo.zh...@linaro.org [mailto:hongbo.zh...@linaro.org]
> Sent: Tuesday, December 29, 2015 12:31 PM
> To: lng-odp@lists.linaro.org
> Cc: petri.savolai...@linaro.org;
> -Original Message-
> From: EXT Maxim Uvarov [mailto:maxim.uva...@linaro.org]
> Sent: Tuesday, December 29, 2015 12:56 PM
> To: Elo, Matias (Nokia - FI/Espoo) ; lng-
> o...@lists.linaro.org
> Subject: Re: [lng-odp] [API-NEXT PATCHv2 2/4] helper: define
>
On 12/29/2015 13:31, hongbo.zh...@linaro.org wrote:
From: Hongbo Zhang
The values of cpu_arch_str were deleted due to codes reorganization, this
patch adds values to them again.
Signed-off-by: Hongbo Zhang
---
I'm proposing these renames before Monarch release. Scheduled "none queues"
and directly accessed "poll queues" are not optimal terms to rescribe how those
queue types are used. "Parallel" and "direct" queues offer better terms. PKTIN
queue type will be removed. PKTOUT type should be replaced with
In practice, this type of queues are referred as "parallel queues"
instead of "none queues". Also "none" type is quite pessimistic
term, since scheduler may perform lots of HW offload also in this mode.
Signed-off-by: Petri Savolainen
---
From: Hongbo Zhang
All the CPU frequency API return value type is uint64_t, this patch makes
them return 0 on failure instead of -1.
Signed-off-by: Hongbo Zhang
---
include/odp/api/cpu.h | 4
From: Hongbo Zhang
This patch moves the PowerPC system info codes into the newly added PowerPC
specific platform file.
This patch also creates syslink to arch/linux/odp_cpu_cycles.c for PowerPC.
Signed-off-by: Hongbo Zhang
---
configure.ac
From: Hongbo Zhang
When per-CPU framework was introduced, it was only implemented on x86,
for other platforms, only the model_str[0] and cpu_hz[0] are set to pass
compile, this patch set all values for model_str[] and cpu_hz[] on the
MIPS platform.
Signed-off-by: Hongbo
> -Original Message-
> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT
> Maxim Uvarov
> Sent: Tuesday, December 29, 2015 1:46 PM
> To: Elo, Matias (Nokia - FI/Espoo); lng-odp@lists.linaro.org
> Subject: Re: [lng-odp] [API-NEXT PATCHv2 2/4] helper: define
>
On 2015-12-29 18:01, Maxim Uvarov wrote:
> Signed-off-by: Maxim Uvarov
> ---
> pkg/debian/changelog | 6 ++
> 1 file changed, 6 insertions(+)
>
> diff --git a/pkg/debian/changelog b/pkg/debian/changelog
> index 7680cf1..4a9a7eb 100644
> --- a/pkg/debian/changelog
>
79 matches
Mail list logo