[PATCH xserver 2/3] os/WaitFor: Use xorg_list_append()
Chris Wilson
chris at chris-wilson.co.uk
Sun Apr 15 14:40:04 UTC 2018
Currently, we use xorg_list_add(new, head->prev) which is functionaly
equivalent to xorg_list_append(), but with more pointer chasing, so
reduce the strain on the reader and compiler by using the simpler
append().
Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
---
os/WaitFor.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/os/WaitFor.c b/os/WaitFor.c
index 7c7b1d2d4..e3b545b93 100644
--- a/os/WaitFor.c
+++ b/os/WaitFor.c
@@ -332,7 +332,7 @@ TimerSet(OsTimerPtr timer, int flags, CARD32 millis,
if ((int) (existing->expires - millis) > 0)
break;
/* This even works at the end of the list -- existing->list will be timers */
- xorg_list_add(&timer->list, existing->list.prev);
+ xorg_list_append(&timer->list, &existing->list);
/* Check to see if the timer is ready to run now */
if ((int) (millis - now) <= 0)
--
2.17.0
More information about the xorg-devel
mailing list