[PATCH 3/3] drm/udl: Switch to SHMEM

Gerd Hoffmann kraxel at redhat.com
Tue Nov 5 11:05:21 UTC 2019


On Mon, Oct 28, 2019 at 09:45:49AM +0100, Thomas Zimmermann wrote:
> Udl's GEM code and the generic SHMEM are almost identical. Replace
> the former with SHMEM. The dmabuf support in udl is being removed
> in favor of the generic GEM PRIME functions.
> 
> The main difference is in the caching flags for mmap pages. By
> default, SHMEM always sets (uncached) write combining. In udl's
> memory management code, only imported buffers use write combining.
> Memory pages of locally created buffer objects are mmap'ed with
> caching enabled. To keep the optimization, udl provides its own
> mmap function for GEM objects where it fixes up the mapping flags.

Hmm, couldn't spot anything wrong, but it's rather hard to see the
actual changes here ...

Any chance that having a separate "remove all dead code obsoleted by
shmem helpers" patch helps?

cheers,
  Gerd



More information about the dri-devel mailing list