v4 submitted to correct that.  Thanks.

On Tue, Nov 11, 2014 at 8:00 AM, Taras Kondratiuk <
[email protected]> wrote:

> On 11/11/2014 03:47 PM, Bill Fischofer wrote:
>
>> v3 patch posted to correct 2nd problem (ret_buf returning to
>> blk_freelist instead of buf_freelist).
>>
>> The pool bufcount is decremented as part of ret_buf processing.  It
>> reflects the number of allocated buffers and will return to 0 naturally
>> when the last allocated buffer is freed, so this should not be set to 0
>> unconditionally.
>>
>
> It should, because what currently happens during pool creation:
> - bufcount is set to 0
> - new buffers are pushed to a free list via ret_buf()
> - ret_buf() decrements bufcount -> bufcount underflows
>
> So after buffer is created its bufcount is not zero. As a result buffer
> can't be destroyed.
>
_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to