Re: [lng-odp] [API-NEXT PATCH] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-10 Thread Ivan Khoronzhuk
On 10.12.15 08:42, Maxim Uvarov wrote: Going to apply this patch to unblock builds in new distros. Any objections? Maxim. I worry about change of timespec struct can break linux-generic time API. I believe it will not happen, but that's why I like Maxim`s approach a little more. [lng-odp]

Re: [lng-odp] [API-NEXT PATCH] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-10 Thread Savolainen, Petri (Nokia - FI/Espoo)
> Signed-off-by: Bill Fischofer > --- > .../linux-generic/include/odp/plat/time_types.h| 5 +++- > platform/linux-generic/odp_time.c | 27 + > - > 2 files changed, 20 insertions(+), 12 deletions(-) > > diff --git

Re: [lng-odp] [API-NEXT PATCH] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-10 Thread Savolainen, Petri (Nokia - FI/Espoo)
From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT Ola Liljedahl Sent: Wednesday, December 09, 2015 6:26 PM To: Bill Fischofer Cc: LNG ODP Mailman List Subject: Re: [lng-odp] [API-NEXT PATCH] linux-generic: time: remove posix bleed through on odp_time_t On 9 December

Re: [lng-odp] [API-NEXT PATCH] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-10 Thread Maxim Uvarov
On 12/10/2015 11:21, Savolainen, Petri (Nokia - FI/Espoo) wrote: Signed-off-by: Bill Fischofer --- .../linux-generic/include/odp/plat/time_types.h| 5 +++- platform/linux-generic/odp_time.c | 27 + - 2 files changed, 20

Re: [lng-odp] [API-NEXT PATCH] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-10 Thread Savolainen, Petri (Nokia - FI/Espoo)
From: EXT Ola Liljedahl [mailto:ola.liljed...@linaro.org] Sent: Thursday, December 10, 2015 12:40 PM To: Savolainen, Petri (Nokia - FI/Espoo) Cc: Bill Fischofer; LNG ODP Mailman List Subject: Re: [lng-odp] [API-NEXT PATCH] linux-generic: time: remove posix bleed through on odp_time_t On 10

Re: [lng-odp] [API-NEXT PATCH] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-10 Thread Ola Liljedahl
On 10 December 2015 at 09:40, Maxim Uvarov wrote: > On 12/10/2015 11:21, Savolainen, Petri (Nokia - FI/Espoo) wrote: > >> Signed-off-by: Bill Fischofer >>> --- >>> .../linux-generic/include/odp/plat/time_types.h| 5 +++- >>>

Re: [lng-odp] [API-NEXT PATCH] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-10 Thread Ola Liljedahl
Bill Fischofer > *Cc:* LNG ODP Mailman List > *Subject:* Re: [lng-odp] [API-NEXT PATCH] linux-generic: time: remove > posix bleed through on odp_time_t > > > > On 9 December 2015 at 16:53, Bill Fischofer <bill.fischo...@linaro.org> > wrote: > > The li

Re: [lng-odp] [API-NEXT PATCH] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-10 Thread Bill Fischofer
On Thu, Dec 10, 2015 at 2:05 AM, Ivan Khoronzhuk wrote: > > > On 10.12.15 08:42, Maxim Uvarov wrote: > >> Going to apply this patch to unblock builds in new distros. Any >> objections? >> >> Maxim. >> > > I worry about change of timespec struct can break

Re: [lng-odp] [API-NEXT PATCH] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-10 Thread Mike Holmes
Is anyone following the OFP project, they are also discussing this struct from a user of ODPs perspctive on a couple of threads like http://www.openfastpath.org/pipermail/openfastpath/2015-December/48.html On 10 December 2015 at 08:07, Bill Fischofer wrote: > > >

Re: [lng-odp] [API-NEXT PATCH] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-10 Thread Bill Fischofer
Thanks. I'll pop over to that thread to take a look. However odp_time_t should be opaque from a user perspective--that's the purpose of having abstract types in the first place. On Thu, Dec 10, 2015 at 9:00 AM, Mike Holmes wrote: > Is anyone following the OFP project,

Re: [lng-odp] [API-NEXT PATCH] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-09 Thread Ola Liljedahl
On 9 December 2015 at 16:53, Bill Fischofer wrote: > The linux-generic implementation of odp_time_t makes use of POSIX > APIs that are sensitive to the _POSIX_C_SOURCE level. Use an indirection > mechanism so that these dependencies do not "bleed through" the ODP API.

[lng-odp] [API-NEXT PATCH] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-09 Thread Bill Fischofer
The linux-generic implementation of odp_time_t makes use of POSIX APIs that are sensitive to the _POSIX_C_SOURCE level. Use an indirection mechanism so that these dependencies do not "bleed through" the ODP API. This means that ODP applications can be independent of _POSIX_C_SOURCE level.

Re: [lng-odp] [API-NEXT PATCH] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-09 Thread Maxim Uvarov
Going to apply this patch to unblock builds in new distros. Any objections? Maxim. On 12/09/2015 19:25, Ola Liljedahl wrote: On 9 December 2015 at 16:53, Bill Fischofer > wrote: The linux-generic implementation of odp_time_t