diff options
author | 2018-08-20 16:00:22 +0000 | |
---|---|---|
committer | 2018-08-20 16:00:22 +0000 | |
commit | 2bd648c0c791046970147cb6db731832ad511740 (patch) | |
tree | 9513fd3a61c273d992777348c936600acb660044 /sys/dev/pci/drm/drm_linux.c | |
parent | Remove unused spllock(). (diff) | |
download | wireguard-openbsd-2bd648c0c791046970147cb6db731832ad511740.tar.xz wireguard-openbsd-2bd648c0c791046970147cb6db731832ad511740.zip |
Reorder checks in the read/write(2) family of syscalls to prepare making
file operations mp-safe.
This change makes it clear that `f_offset' is only accessed in vn_read()
and vn_write(), which will help taking it out of the KERNEL_LOCK().
This refactoring uncovered a race in vn_read() which is now documented
and will be addressed in a later diff.
ok visa@
Diffstat (limited to 'sys/dev/pci/drm/drm_linux.c')
-rw-r--r-- | sys/dev/pci/drm/drm_linux.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sys/dev/pci/drm/drm_linux.c b/sys/dev/pci/drm/drm_linux.c index 18ae95f3c18..a414a2623a7 100644 --- a/sys/dev/pci/drm/drm_linux.c +++ b/sys/dev/pci/drm/drm_linux.c @@ -1,4 +1,4 @@ -/* $OpenBSD: drm_linux.c,v 1.29 2018/08/20 14:59:02 visa Exp $ */ +/* $OpenBSD: drm_linux.c,v 1.30 2018/08/20 16:00:22 mpi Exp $ */ /* * Copyright (c) 2013 Jonathan Gray <jsg@openbsd.org> * Copyright (c) 2015, 2016 Mark Kettenis <kettenis@openbsd.org> @@ -809,13 +809,13 @@ fence_context_alloc(unsigned int num) } int -dmabuf_read(struct file *fp, off_t *poff, struct uio *uio, struct ucred *cred) +dmabuf_read(struct file *fp, struct uio *uio, int fflags) { return (ENXIO); } int -dmabuf_write(struct file *fp, off_t *poff, struct uio *uio, struct ucred *cred) +dmabuf_write(struct file *fp, struct uio *uio, int fflags) { return (ENXIO); } |