Re: [lng-odp] [PATCH] linux-generic: timer: use plain buffers for timeouts

2015-06-22 Thread Ivan Khoronzhuk
Hi Ola, Maybe I'm wrong, but let me try to explain change in question. The main reason in this change is the lack of possibility to convert buf_hdr to buf_handle. The function odp_timeout_to_event requires this conversion. By and large it's not required the tmo_handle to be the tmo_header for

Re: [lng-odp] [PATCH] linux-generic: timer: use plain buffers for timeouts

2015-06-01 Thread Ola Liljedahl
If I understand your patch correctly, a timeout event is really a buffer with the timeout-specific header stored just after the buffer header (as before). You don't change the definition of odp_timeout_hdr_t and the implementation of odp_timeout_hdr(odp_buffer_t buf) has just moved from odp_timer_i

Re: [lng-odp] [PATCH] linux-generic: timer: use plain buffers for timeouts

2015-05-30 Thread Taras Kondratiuk
On 05/29/2015 07:26 AM, Ola Liljedahl wrote: > On 13 May 2015 at 17:31, Ola Liljedahl > wrote: > > On 13 May 2015 at 12:19, Maxim Uvarov > wrote: > > Patch looks good. Validation test passed. Need one more review.

Re: [lng-odp] [PATCH] linux-generic: timer: use plain buffers for timeouts

2015-05-29 Thread Ola Liljedahl
On 13 May 2015 at 17:31, Ola Liljedahl wrote: > On 13 May 2015 at 12:19, Maxim Uvarov wrote: > >> Patch looks good. Validation test passed. Need one more review. >> > Thanks. I did this primarily for KS2 being able to use the linux-generic > timer implementation without any changes. I am waiting

Re: [lng-odp] [PATCH] linux-generic: timer: use plain buffers for timeouts

2015-05-13 Thread Ola Liljedahl
On 13 May 2015 at 12:19, Maxim Uvarov wrote: > Patch looks good. Validation test passed. Need one more review. > Thanks. I did this primarily for KS2 being able to use the linux-generic timer implementation without any changes. I am waiting for Taras to verify this. -- Ola > > Thanks, > Maxim.

Re: [lng-odp] [PATCH] linux-generic: timer: use plain buffers for timeouts

2015-05-13 Thread Maxim Uvarov
Patch looks good. Validation test passed. Need one more review. Thanks, Maxim. On 04/24/2015 17:10, Ola Liljedahl wrote: Use plain buffers for timeouts. Store the timeout header in the buffer data area. This simplifies re-use on other platforms where the event header cannot be arbitrarily defin

[lng-odp] [PATCH] linux-generic: timer: use plain buffers for timeouts

2015-04-24 Thread Ola Liljedahl
Use plain buffers for timeouts. Store the timeout header in the buffer data area. This simplifies re-use on other platforms where the event header cannot be arbitrarily defined. Signed-off-by: Ola Liljedahl --- (This document/code contribution attached is provided under the terms of agreement LES