[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20190917081448.zonghnqa4aoc4bmp@sirius.home.kraxel.org>
Date: Tue, 17 Sep 2019 10:14:48 +0200
From: Gerd Hoffmann <kraxel@...hat.com>
To: Steven Price <steven.price@....com>
Cc: dri-devel@...ts.freedesktop.org,
Tomeu Vizoso <tomeu.vizoso@...labora.com>,
Maxime Ripard <maxime.ripard@...tlin.com>,
Daniel Vetter <daniel.vetter@...ll.ch>,
open list <linux-kernel@...r.kernel.org>,
David Airlie <airlied@...ux.ie>,
Thomas Zimmermann <tzimmermann@...e.de>,
"open list:VIRTIO GPU DRIVER"
<virtualization@...ts.linux-foundation.org>,
Sean Paul <sean@...rly.run>,
Alyssa Rosenzweig <alyssa.rosenzweig@...labora.com>
Subject: Re: [PATCH 2/8] drm/shmem: switch shmem helper to
&drm_gem_object_funcs.mmap
Hi,
> > - /* VM_PFNMAP was set by drm_gem_mmap() */
> > - vma->vm_flags &= ~VM_PFNMAP;
> > - vma->vm_flags |= VM_MIXEDMAP;
> > + vma->vm_flags |= (VM_MIXEDMAP|VM_DONTEXPAND);
>
> I'm finding this a bit hard to follow - but I think here we've lost
> VM_IO and VM_DONTDUMP which used to be set by drm_gem_mmap().
Yep. Intentional, but I think I better split that off to a separate
patch with a commit message explaining things.
> Also it looks like nothing is fiddling vma->vm_page_prot anymore.
Oops, that one was not intentional. Will fix.
cheers,
Gerd
Powered by blists - more mailing lists