diff options
| author | Owain Ainsworth <oga@cvs.openbsd.org> | 2008-10-07 22:25:13 +0000 |
|---|---|---|
| committer | Owain Ainsworth <oga@cvs.openbsd.org> | 2008-10-07 22:25:13 +0000 |
| commit | 80c9e2d60d9935ec71deb4cdadcd95ed1eab7bd5 (patch) | |
| tree | 427cd8c404be603a4becfdf03099d5d11bcbf8c1 /sys/dev/pci/drm/drmP.h | |
| parent | fe8d8e4b6ee985aaaffe6e1231eae6166fa58582 (diff) | |
Kill the linux-ready negative return codes in ``shared'' code. We handle
them wrong in several cases that i've noticed and Merging when needed is
still fairly simple, anyway. This shaves another 500 bytes from an amd64
kernel due to not having to flip the sign on some things. It also stops
my eyes bleeding.
Tested by a few along with the last diff that went in.
Diffstat (limited to 'sys/dev/pci/drm/drmP.h')
| -rw-r--r-- | sys/dev/pci/drm/drmP.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sys/dev/pci/drm/drmP.h b/sys/dev/pci/drm/drmP.h index 6bf354fa31b..73260123d0b 100644 --- a/sys/dev/pci/drm/drmP.h +++ b/sys/dev/pci/drm/drmP.h @@ -237,13 +237,12 @@ do { \ } \ } while (0) -/* Returns -errno to shared code */ #define DRM_WAIT_ON( ret, queue, timeout, condition ) \ DRM_SPINLOCK(&dev->irq_lock); \ while ( ret == 0 ) { \ if (condition) \ break; \ - ret = -msleep(&(queue), &dev->irq_lock, \ + ret = msleep(&(queue), &dev->irq_lock, \ PZERO | PCATCH, "drmwtq", (timeout)); \ } \ DRM_SPINUNLOCK(&dev->irq_lock) |