RFR (S): Single-element buffer in thread-local taskqueues
Zhengyu Gu
zgu at redhat.com
Wed Nov 9 15:40:27 UTC 2016
Ok
Thanks,
-Zhengyu
On 11/09/2016 10:02 AM, Aleksey Shipilev wrote:
> On 11/09/2016 03:14 PM, Zhengyu Gu wrote:
>> Would it make sense to incorporate pop_buffer() inside pop_local()?
>> since you are doing the push buffer in push()? The buffered element
>> could be easily forgotten when migrating from OverflowTaskQueue to
>> BufferedOverflowTaskQueue.
> The way taskqueues implement delegation is by introducing new methods,
> apparently: see how OverflowTaskQueue extends the GenericTaskQueue.
> pop_buffer seems to be the natural continuation of
> pop_local/pop_overflow divide...
>
> There is push() that does everything and overrides the super::push, but
> pop_*() are distinct. For consistency, I would keep it this way.
>
> Thanks,
> -Aleksey
>
More information about the shenandoah-dev
mailing list