On Sun, 25 Apr 2021 05:56:53 +0000
Ruifeng Wang <[email protected]> wrote:

> Instead of polling for tail to be updated, use wfe instruction.
> 
> Signed-off-by: Gavin Hu <[email protected]>
> Signed-off-by: Ruifeng Wang <[email protected]>
> Reviewed-by: Steve Capper <[email protected]>
> Reviewed-by: Ola Liljedahl <[email protected]>
> Reviewed-by: Honnappa Nagarahalli <[email protected]>
> Acked-by: Konstantin Ananyev <[email protected]>

Looks ok to me, but it does raise an interesting question.
Shouldn't the original code have been using atomic load to look at ht->tail.

This another place where "volatile considered harmful" applies.

Reply via email to