答复: [PATCH xf86-video-amdgpu] Simplify drmmode_handle_uevents
Qu, Jim
Jim.Qu at amd.com
Thu Dec 15 04:39:18 UTC 2016
Reviewed-by: Jim Qu <Jim.Qu at amd.com>
Thanks
JimQu
________________________________________
发件人: amd-gfx <amd-gfx-bounces at lists.freedesktop.org> 代表 Michel Dänzer <michel at daenzer.net>
发送时间: 2016年12月15日 11:47
收件人: amd-gfx at lists.freedesktop.org
主题: [PATCH xf86-video-amdgpu] Simplify drmmode_handle_uevents
From: Michel Dänzer <michel.daenzer at amd.com>
No functional change intended.
Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>
---
src/drmmode_display.c | 14 ++++----------
1 file changed, 4 insertions(+), 10 deletions(-)
diff --git a/src/drmmode_display.c b/src/drmmode_display.c
index a5b43458..63b3b076 100644
--- a/src/drmmode_display.c
+++ b/src/drmmode_display.c
@@ -2528,21 +2528,15 @@ static void drmmode_handle_uevents(int fd, void *closure)
ScrnInfoPtr scrn = drmmode->scrn;
struct udev_device *dev;
Bool received = FALSE;
- struct timeval tv;
+ struct timeval tv = { 0, 0 };
fd_set readfd;
- int ret;
FD_ZERO(&readfd);
FD_SET(fd, &readfd);
- tv.tv_sec = 0;
- tv.tv_usec = 0;
- while (1) {
- ret = select(fd + 1, &readfd, NULL, NULL, &tv);
- /* Check if our file descriptor has received data. */
- if (!(ret > 0 && FD_ISSET(fd, &readfd)))
- break;
- /* Make the call to receive device. select() ensured that this will not be blocked. */
+ while (select(fd + 1, &readfd, NULL, NULL, &tv) > 0 &&
+ FD_ISSET(fd, &readfd)) {
+ /* select() ensured that this will not block */
dev = udev_monitor_receive_device(drmmode->uevent_monitor);
if (dev) {
udev_device_unref(dev);
--
2.11.0
_______________________________________________
amd-gfx mailing list
amd-gfx at lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
More information about the amd-gfx
mailing list