LKML Archive on lore.kernel.org help / color / mirror / Atom feed
* [PATCH v2] gpu: drm: udl: Adding new typedef vm_fault_t
@ 2018-04-25 4:59 Souptick Joarder
2018-05-10 13:48 ` Souptick Joarder
0 siblings, 1 reply; 5+ messages in thread
From: Souptick Joarder @ 2018-04-25 4:59 UTC (permalink / raw)
To: airlied, christian.koenig, cakturk, seanpaul, daniel.vetter, mhocko
Cc: dri-devel, linux-kernel, willy
Use new return type vm_fault_t for fault and huge_fault
handler. For now, this is just documenting that the
function returns a VM_FAULT value rather than an errno.
Once all instances are converted, vm_fault_t will become
a distinct type.
Commit 1c8f422059ae ("mm: change return type to vm_fault_t")
Previously vm_insert_page() returns err which driver
mapped into VM_FAULT_* type. The new function vmf_
insert_page() will replace this inefficiency by
returning VM_FAULT_* type.
Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com>
Reviewed-by: Matthew Wilcox <mawilcox@microsoft.com>
---
v2: Updated the change log
drivers/gpu/drm/udl/udl_drv.h | 3 ++-
drivers/gpu/drm/udl/udl_gem.c | 15 ++-------------
2 files changed, 4 insertions(+), 14 deletions(-)
diff --git a/drivers/gpu/drm/udl/udl_drv.h b/drivers/gpu/drm/udl/udl_drv.h
index 2a75ab8..11151c4 100644
--- a/drivers/gpu/drm/udl/udl_drv.h
+++ b/drivers/gpu/drm/udl/udl_drv.h
@@ -16,6 +16,7 @@
#include <linux/usb.h>
#include <drm/drm_gem.h>
+#include <linux/mm_types.h>
#define DRIVER_NAME "udl"
#define DRIVER_DESC "DisplayLink"
@@ -134,7 +135,7 @@ struct drm_gem_object *udl_gem_prime_import(struct drm_device *dev,
int udl_gem_vmap(struct udl_gem_object *obj);
void udl_gem_vunmap(struct udl_gem_object *obj);
int udl_drm_gem_mmap(struct file *filp, struct vm_area_struct *vma);
-int udl_gem_fault(struct vm_fault *vmf);
+vm_fault_t udl_gem_fault(struct vm_fault *vmf);
int udl_handle_damage(struct udl_framebuffer *fb, int x, int y,
int width, int height);
diff --git a/drivers/gpu/drm/udl/udl_gem.c b/drivers/gpu/drm/udl/udl_gem.c
index dee6bd9..cf5fe35 100644
--- a/drivers/gpu/drm/udl/udl_gem.c
+++ b/drivers/gpu/drm/udl/udl_gem.c
@@ -100,13 +100,12 @@ int udl_drm_gem_mmap(struct file *filp, struct vm_area_struct *vma)
return ret;
}
-int udl_gem_fault(struct vm_fault *vmf)
+vm_fault_t udl_gem_fault(struct vm_fault *vmf)
{
struct vm_area_struct *vma = vmf->vma;
struct udl_gem_object *obj = to_udl_bo(vma->vm_private_data);
struct page *page;
unsigned int page_offset;
- int ret = 0;
page_offset = (vmf->address - vma->vm_start) >> PAGE_SHIFT;
@@ -114,17 +113,7 @@ int udl_gem_fault(struct vm_fault *vmf)
return VM_FAULT_SIGBUS;
page = obj->pages[page_offset];
- ret = vm_insert_page(vma, vmf->address, page);
- switch (ret) {
- case -EAGAIN:
- case 0:
- case -ERESTARTSYS:
- return VM_FAULT_NOPAGE;
- case -ENOMEM:
- return VM_FAULT_OOM;
- default:
- return VM_FAULT_SIGBUS;
- }
+ return vmf_insert_page(vma, vmf->address, page);
}
int udl_gem_get_pages(struct udl_gem_object *obj)
--
1.9.1
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v2] gpu: drm: udl: Adding new typedef vm_fault_t 2018-04-25 4:59 [PATCH v2] gpu: drm: udl: Adding new typedef vm_fault_t Souptick Joarder @ 2018-05-10 13:48 ` Souptick Joarder 2018-05-21 4:57 ` Souptick Joarder 0 siblings, 1 reply; 5+ messages in thread From: Souptick Joarder @ 2018-05-10 13:48 UTC (permalink / raw) To: airlied, Christian König, Cihangir Aktürk, Sean Paul, Daniel Vetter, Michal Hocko Cc: dri-devel, linux-kernel, Matthew Wilcox On Wed, Apr 25, 2018 at 10:29 AM, Souptick Joarder <jrdr.linux@gmail.com> wrote: > Use new return type vm_fault_t for fault and huge_fault > handler. For now, this is just documenting that the > function returns a VM_FAULT value rather than an errno. > Once all instances are converted, vm_fault_t will become > a distinct type. > > Commit 1c8f422059ae ("mm: change return type to vm_fault_t") > > Previously vm_insert_page() returns err which driver > mapped into VM_FAULT_* type. The new function vmf_ > insert_page() will replace this inefficiency by > returning VM_FAULT_* type. > > Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com> > Reviewed-by: Matthew Wilcox <mawilcox@microsoft.com> > --- > v2: Updated the change log > > drivers/gpu/drm/udl/udl_drv.h | 3 ++- > drivers/gpu/drm/udl/udl_gem.c | 15 ++------------- > 2 files changed, 4 insertions(+), 14 deletions(-) > > diff --git a/drivers/gpu/drm/udl/udl_drv.h b/drivers/gpu/drm/udl/udl_drv.h > index 2a75ab8..11151c4 100644 > --- a/drivers/gpu/drm/udl/udl_drv.h > +++ b/drivers/gpu/drm/udl/udl_drv.h > @@ -16,6 +16,7 @@ > > #include <linux/usb.h> > #include <drm/drm_gem.h> > +#include <linux/mm_types.h> > > #define DRIVER_NAME "udl" > #define DRIVER_DESC "DisplayLink" > @@ -134,7 +135,7 @@ struct drm_gem_object *udl_gem_prime_import(struct drm_device *dev, > int udl_gem_vmap(struct udl_gem_object *obj); > void udl_gem_vunmap(struct udl_gem_object *obj); > int udl_drm_gem_mmap(struct file *filp, struct vm_area_struct *vma); > -int udl_gem_fault(struct vm_fault *vmf); > +vm_fault_t udl_gem_fault(struct vm_fault *vmf); > > int udl_handle_damage(struct udl_framebuffer *fb, int x, int y, > int width, int height); > diff --git a/drivers/gpu/drm/udl/udl_gem.c b/drivers/gpu/drm/udl/udl_gem.c > index dee6bd9..cf5fe35 100644 > --- a/drivers/gpu/drm/udl/udl_gem.c > +++ b/drivers/gpu/drm/udl/udl_gem.c > @@ -100,13 +100,12 @@ int udl_drm_gem_mmap(struct file *filp, struct vm_area_struct *vma) > return ret; > } > > -int udl_gem_fault(struct vm_fault *vmf) > +vm_fault_t udl_gem_fault(struct vm_fault *vmf) > { > struct vm_area_struct *vma = vmf->vma; > struct udl_gem_object *obj = to_udl_bo(vma->vm_private_data); > struct page *page; > unsigned int page_offset; > - int ret = 0; > > page_offset = (vmf->address - vma->vm_start) >> PAGE_SHIFT; > > @@ -114,17 +113,7 @@ int udl_gem_fault(struct vm_fault *vmf) > return VM_FAULT_SIGBUS; > > page = obj->pages[page_offset]; > - ret = vm_insert_page(vma, vmf->address, page); > - switch (ret) { > - case -EAGAIN: > - case 0: > - case -ERESTARTSYS: > - return VM_FAULT_NOPAGE; > - case -ENOMEM: > - return VM_FAULT_OOM; > - default: > - return VM_FAULT_SIGBUS; > - } > + return vmf_insert_page(vma, vmf->address, page); > } > > int udl_gem_get_pages(struct udl_gem_object *obj) > -- > 1.9.1 > Any comment on this patch ? ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] gpu: drm: udl: Adding new typedef vm_fault_t 2018-05-10 13:48 ` Souptick Joarder @ 2018-05-21 4:57 ` Souptick Joarder 2018-05-23 8:49 ` Daniel Vetter 0 siblings, 1 reply; 5+ messages in thread From: Souptick Joarder @ 2018-05-21 4:57 UTC (permalink / raw) To: airlied, Christian König, Cihangir Aktürk, Sean Paul, Daniel Vetter, Michal Hocko Cc: dri-devel, linux-kernel, Matthew Wilcox On Thu, May 10, 2018 at 7:18 PM, Souptick Joarder <jrdr.linux@gmail.com> wrote: > On Wed, Apr 25, 2018 at 10:29 AM, Souptick Joarder <jrdr.linux@gmail.com> wrote: >> Use new return type vm_fault_t for fault and huge_fault >> handler. For now, this is just documenting that the >> function returns a VM_FAULT value rather than an errno. >> Once all instances are converted, vm_fault_t will become >> a distinct type. >> >> Commit 1c8f422059ae ("mm: change return type to vm_fault_t") >> >> Previously vm_insert_page() returns err which driver >> mapped into VM_FAULT_* type. The new function vmf_ >> insert_page() will replace this inefficiency by >> returning VM_FAULT_* type. >> >> Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com> >> Reviewed-by: Matthew Wilcox <mawilcox@microsoft.com> >> --- >> v2: Updated the change log >> >> drivers/gpu/drm/udl/udl_drv.h | 3 ++- >> drivers/gpu/drm/udl/udl_gem.c | 15 ++------------- >> 2 files changed, 4 insertions(+), 14 deletions(-) >> >> diff --git a/drivers/gpu/drm/udl/udl_drv.h b/drivers/gpu/drm/udl/udl_drv.h >> index 2a75ab8..11151c4 100644 >> --- a/drivers/gpu/drm/udl/udl_drv.h >> +++ b/drivers/gpu/drm/udl/udl_drv.h >> @@ -16,6 +16,7 @@ >> >> #include <linux/usb.h> >> #include <drm/drm_gem.h> >> +#include <linux/mm_types.h> >> >> #define DRIVER_NAME "udl" >> #define DRIVER_DESC "DisplayLink" >> @@ -134,7 +135,7 @@ struct drm_gem_object *udl_gem_prime_import(struct drm_device *dev, >> int udl_gem_vmap(struct udl_gem_object *obj); >> void udl_gem_vunmap(struct udl_gem_object *obj); >> int udl_drm_gem_mmap(struct file *filp, struct vm_area_struct *vma); >> -int udl_gem_fault(struct vm_fault *vmf); >> +vm_fault_t udl_gem_fault(struct vm_fault *vmf); >> >> int udl_handle_damage(struct udl_framebuffer *fb, int x, int y, >> int width, int height); >> diff --git a/drivers/gpu/drm/udl/udl_gem.c b/drivers/gpu/drm/udl/udl_gem.c >> index dee6bd9..cf5fe35 100644 >> --- a/drivers/gpu/drm/udl/udl_gem.c >> +++ b/drivers/gpu/drm/udl/udl_gem.c >> @@ -100,13 +100,12 @@ int udl_drm_gem_mmap(struct file *filp, struct vm_area_struct *vma) >> return ret; >> } >> >> -int udl_gem_fault(struct vm_fault *vmf) >> +vm_fault_t udl_gem_fault(struct vm_fault *vmf) >> { >> struct vm_area_struct *vma = vmf->vma; >> struct udl_gem_object *obj = to_udl_bo(vma->vm_private_data); >> struct page *page; >> unsigned int page_offset; >> - int ret = 0; >> >> page_offset = (vmf->address - vma->vm_start) >> PAGE_SHIFT; >> >> @@ -114,17 +113,7 @@ int udl_gem_fault(struct vm_fault *vmf) >> return VM_FAULT_SIGBUS; >> >> page = obj->pages[page_offset]; >> - ret = vm_insert_page(vma, vmf->address, page); >> - switch (ret) { >> - case -EAGAIN: >> - case 0: >> - case -ERESTARTSYS: >> - return VM_FAULT_NOPAGE; >> - case -ENOMEM: >> - return VM_FAULT_OOM; >> - default: >> - return VM_FAULT_SIGBUS; >> - } >> + return vmf_insert_page(vma, vmf->address, page); >> } >> >> int udl_gem_get_pages(struct udl_gem_object *obj) >> -- >> 1.9.1 >> > > Any comment on this patch ? If no comment, we would like to get this patch in queue for 4.18. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] gpu: drm: udl: Adding new typedef vm_fault_t 2018-05-21 4:57 ` Souptick Joarder @ 2018-05-23 8:49 ` Daniel Vetter 2018-05-23 9:54 ` Souptick Joarder 0 siblings, 1 reply; 5+ messages in thread From: Daniel Vetter @ 2018-05-23 8:49 UTC (permalink / raw) To: Souptick Joarder Cc: airlied, Christian König, Cihangir Aktürk, Sean Paul, Daniel Vetter, Michal Hocko, dri-devel, linux-kernel, Matthew Wilcox On Mon, May 21, 2018 at 10:27:44AM +0530, Souptick Joarder wrote: > On Thu, May 10, 2018 at 7:18 PM, Souptick Joarder <jrdr.linux@gmail.com> wrote: > > On Wed, Apr 25, 2018 at 10:29 AM, Souptick Joarder <jrdr.linux@gmail.com> wrote: > >> Use new return type vm_fault_t for fault and huge_fault > >> handler. For now, this is just documenting that the > >> function returns a VM_FAULT value rather than an errno. > >> Once all instances are converted, vm_fault_t will become > >> a distinct type. > >> > >> Commit 1c8f422059ae ("mm: change return type to vm_fault_t") > >> > >> Previously vm_insert_page() returns err which driver > >> mapped into VM_FAULT_* type. The new function vmf_ > >> insert_page() will replace this inefficiency by > >> returning VM_FAULT_* type. > >> > >> Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com> > >> Reviewed-by: Matthew Wilcox <mawilcox@microsoft.com> > >> --- > >> v2: Updated the change log > >> > >> drivers/gpu/drm/udl/udl_drv.h | 3 ++- > >> drivers/gpu/drm/udl/udl_gem.c | 15 ++------------- > >> 2 files changed, 4 insertions(+), 14 deletions(-) > >> > >> diff --git a/drivers/gpu/drm/udl/udl_drv.h b/drivers/gpu/drm/udl/udl_drv.h > >> index 2a75ab8..11151c4 100644 > >> --- a/drivers/gpu/drm/udl/udl_drv.h > >> +++ b/drivers/gpu/drm/udl/udl_drv.h > >> @@ -16,6 +16,7 @@ > >> > >> #include <linux/usb.h> > >> #include <drm/drm_gem.h> > >> +#include <linux/mm_types.h> > >> > >> #define DRIVER_NAME "udl" > >> #define DRIVER_DESC "DisplayLink" > >> @@ -134,7 +135,7 @@ struct drm_gem_object *udl_gem_prime_import(struct drm_device *dev, > >> int udl_gem_vmap(struct udl_gem_object *obj); > >> void udl_gem_vunmap(struct udl_gem_object *obj); > >> int udl_drm_gem_mmap(struct file *filp, struct vm_area_struct *vma); > >> -int udl_gem_fault(struct vm_fault *vmf); > >> +vm_fault_t udl_gem_fault(struct vm_fault *vmf); > >> > >> int udl_handle_damage(struct udl_framebuffer *fb, int x, int y, > >> int width, int height); > >> diff --git a/drivers/gpu/drm/udl/udl_gem.c b/drivers/gpu/drm/udl/udl_gem.c > >> index dee6bd9..cf5fe35 100644 > >> --- a/drivers/gpu/drm/udl/udl_gem.c > >> +++ b/drivers/gpu/drm/udl/udl_gem.c > >> @@ -100,13 +100,12 @@ int udl_drm_gem_mmap(struct file *filp, struct vm_area_struct *vma) > >> return ret; > >> } > >> > >> -int udl_gem_fault(struct vm_fault *vmf) > >> +vm_fault_t udl_gem_fault(struct vm_fault *vmf) > >> { > >> struct vm_area_struct *vma = vmf->vma; > >> struct udl_gem_object *obj = to_udl_bo(vma->vm_private_data); > >> struct page *page; > >> unsigned int page_offset; > >> - int ret = 0; > >> > >> page_offset = (vmf->address - vma->vm_start) >> PAGE_SHIFT; > >> > >> @@ -114,17 +113,7 @@ int udl_gem_fault(struct vm_fault *vmf) > >> return VM_FAULT_SIGBUS; > >> > >> page = obj->pages[page_offset]; > >> - ret = vm_insert_page(vma, vmf->address, page); > >> - switch (ret) { > >> - case -EAGAIN: > >> - case 0: > >> - case -ERESTARTSYS: > >> - return VM_FAULT_NOPAGE; > >> - case -ENOMEM: > >> - return VM_FAULT_OOM; > >> - default: > >> - return VM_FAULT_SIGBUS; > >> - } > >> + return vmf_insert_page(vma, vmf->address, page); > >> } > >> > >> int udl_gem_get_pages(struct udl_gem_object *obj) > >> -- > >> 1.9.1 > >> > > > > Any comment on this patch ? > > If no comment, we would like to get this patch in queue > for 4.18. 4.18 is done already, queued up for 4.19 in drm-misc-next. Thanks, Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] gpu: drm: udl: Adding new typedef vm_fault_t 2018-05-23 8:49 ` Daniel Vetter @ 2018-05-23 9:54 ` Souptick Joarder 0 siblings, 0 replies; 5+ messages in thread From: Souptick Joarder @ 2018-05-23 9:54 UTC (permalink / raw) To: Souptick Joarder, airlied, Christian König, Cihangir Aktürk, Sean Paul, Michal Hocko, dri-devel, linux-kernel, Matthew Wilcox Cc: Daniel Vetter On Wed, May 23, 2018 at 2:19 PM, Daniel Vetter <daniel@ffwll.ch> wrote: > On Mon, May 21, 2018 at 10:27:44AM +0530, Souptick Joarder wrote: >> On Thu, May 10, 2018 at 7:18 PM, Souptick Joarder <jrdr.linux@gmail.com> wrote: >> > On Wed, Apr 25, 2018 at 10:29 AM, Souptick Joarder <jrdr.linux@gmail.com> wrote: >> >> Use new return type vm_fault_t for fault and huge_fault >> >> handler. For now, this is just documenting that the >> >> function returns a VM_FAULT value rather than an errno. >> >> Once all instances are converted, vm_fault_t will become >> >> a distinct type. >> >> >> >> Commit 1c8f422059ae ("mm: change return type to vm_fault_t") >> >> >> >> Previously vm_insert_page() returns err which driver >> >> mapped into VM_FAULT_* type. The new function vmf_ >> >> insert_page() will replace this inefficiency by >> >> returning VM_FAULT_* type. >> >> >> >> Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com> >> >> Reviewed-by: Matthew Wilcox <mawilcox@microsoft.com> >> >> --- >> >> v2: Updated the change log >> >> >> >> drivers/gpu/drm/udl/udl_drv.h | 3 ++- >> >> drivers/gpu/drm/udl/udl_gem.c | 15 ++------------- >> >> 2 files changed, 4 insertions(+), 14 deletions(-) >> >> >> >> diff --git a/drivers/gpu/drm/udl/udl_drv.h b/drivers/gpu/drm/udl/udl_drv.h >> >> index 2a75ab8..11151c4 100644 >> >> --- a/drivers/gpu/drm/udl/udl_drv.h >> >> +++ b/drivers/gpu/drm/udl/udl_drv.h >> >> @@ -16,6 +16,7 @@ >> >> >> >> #include <linux/usb.h> >> >> #include <drm/drm_gem.h> >> >> +#include <linux/mm_types.h> >> >> >> >> #define DRIVER_NAME "udl" >> >> #define DRIVER_DESC "DisplayLink" >> >> @@ -134,7 +135,7 @@ struct drm_gem_object *udl_gem_prime_import(struct drm_device *dev, >> >> int udl_gem_vmap(struct udl_gem_object *obj); >> >> void udl_gem_vunmap(struct udl_gem_object *obj); >> >> int udl_drm_gem_mmap(struct file *filp, struct vm_area_struct *vma); >> >> -int udl_gem_fault(struct vm_fault *vmf); >> >> +vm_fault_t udl_gem_fault(struct vm_fault *vmf); >> >> >> >> int udl_handle_damage(struct udl_framebuffer *fb, int x, int y, >> >> int width, int height); >> >> diff --git a/drivers/gpu/drm/udl/udl_gem.c b/drivers/gpu/drm/udl/udl_gem.c >> >> index dee6bd9..cf5fe35 100644 >> >> --- a/drivers/gpu/drm/udl/udl_gem.c >> >> +++ b/drivers/gpu/drm/udl/udl_gem.c >> >> @@ -100,13 +100,12 @@ int udl_drm_gem_mmap(struct file *filp, struct vm_area_struct *vma) >> >> return ret; >> >> } >> >> >> >> -int udl_gem_fault(struct vm_fault *vmf) >> >> +vm_fault_t udl_gem_fault(struct vm_fault *vmf) >> >> { >> >> struct vm_area_struct *vma = vmf->vma; >> >> struct udl_gem_object *obj = to_udl_bo(vma->vm_private_data); >> >> struct page *page; >> >> unsigned int page_offset; >> >> - int ret = 0; >> >> >> >> page_offset = (vmf->address - vma->vm_start) >> PAGE_SHIFT; >> >> >> >> @@ -114,17 +113,7 @@ int udl_gem_fault(struct vm_fault *vmf) >> >> return VM_FAULT_SIGBUS; >> >> >> >> page = obj->pages[page_offset]; >> >> - ret = vm_insert_page(vma, vmf->address, page); >> >> - switch (ret) { >> >> - case -EAGAIN: >> >> - case 0: >> >> - case -ERESTARTSYS: >> >> - return VM_FAULT_NOPAGE; >> >> - case -ENOMEM: >> >> - return VM_FAULT_OOM; >> >> - default: >> >> - return VM_FAULT_SIGBUS; >> >> - } >> >> + return vmf_insert_page(vma, vmf->address, page); >> >> } >> >> >> >> int udl_gem_get_pages(struct udl_gem_object *obj) >> >> -- >> >> 1.9.1 >> >> >> > >> > Any comment on this patch ? >> >> If no comment, we would like to get this patch in queue >> for 4.18. > > 4.18 is done already, queued up for 4.19 in drm-misc-next. > > Thanks, Daniel > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch Thanks :) ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-05-23 9:54 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-04-25 4:59 [PATCH v2] gpu: drm: udl: Adding new typedef vm_fault_t Souptick Joarder 2018-05-10 13:48 ` Souptick Joarder 2018-05-21 4:57 ` Souptick Joarder 2018-05-23 8:49 ` Daniel Vetter 2018-05-23 9:54 ` Souptick Joarder
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).