diff mbox series

udmabuf: set read/write flag when exporting

Message ID 20181114122029.16766-1-kraxel@redhat.com (mailing list archive)
State New, archived
Headers show
Series udmabuf: set read/write flag when exporting | expand

Commit Message

Gerd Hoffmann Nov. 14, 2018, 12:20 p.m. UTC
Otherwise, mmap fails when done with PROT_WRITE.

Suggested-by: Gurchetan Singh <gurchetansingh@chromium.org>
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 drivers/dma-buf/udmabuf.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Gurchetan Singh Nov. 16, 2018, 1:46 a.m. UTC | #1
Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>
Tested-by: Gurchetan Singh <gurchetansingh@chromium.org>
On Wed, Nov 14, 2018 at 4:20 AM Gerd Hoffmann <kraxel@redhat.com> wrote:
>
> Otherwise, mmap fails when done with PROT_WRITE.
>
> Suggested-by: Gurchetan Singh <gurchetansingh@chromium.org>
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
>  drivers/dma-buf/udmabuf.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/dma-buf/udmabuf.c b/drivers/dma-buf/udmabuf.c
> index e70328ab7e..d9ff246093 100644
> --- a/drivers/dma-buf/udmabuf.c
> +++ b/drivers/dma-buf/udmabuf.c
> @@ -189,6 +189,7 @@ static long udmabuf_create(const struct udmabuf_create_list *head,
>         exp_info.ops  = &udmabuf_ops;
>         exp_info.size = ubuf->pagecount << PAGE_SHIFT;
>         exp_info.priv = ubuf;
> +       exp_info.flags = O_RDWR;
>
>         buf = dma_buf_export(&exp_info);
>         if (IS_ERR(buf)) {
> --
> 2.9.3
>
Gerd Hoffmann Nov. 16, 2018, 7:52 a.m. UTC | #2
On Thu, Nov 15, 2018 at 05:46:30PM -0800, Gurchetan Singh wrote:
> Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>
> Tested-by: Gurchetan Singh <gurchetansingh@chromium.org>

Thanks.  Pushed to drm-misc-fixes.

cheers,
  Gerd
diff mbox series

Patch

diff --git a/drivers/dma-buf/udmabuf.c b/drivers/dma-buf/udmabuf.c
index e70328ab7e..d9ff246093 100644
--- a/drivers/dma-buf/udmabuf.c
+++ b/drivers/dma-buf/udmabuf.c
@@ -189,6 +189,7 @@  static long udmabuf_create(const struct udmabuf_create_list *head,
 	exp_info.ops  = &udmabuf_ops;
 	exp_info.size = ubuf->pagecount << PAGE_SHIFT;
 	exp_info.priv = ubuf;
+	exp_info.flags = O_RDWR;
 
 	buf = dma_buf_export(&exp_info);
 	if (IS_ERR(buf)) {