Re: [RESEND PATCH v2] drm/gem: Mark pinned pages as unevictable

2019-01-17 Thread Eric Anholt
Kuo-Hsin Yang writes: > The gem drivers use shmemfs to allocate backing storage for gem objects. > On Samsung Chromebook Plus, the drm/rockchip driver may call > rockchip_gem_get_pages -> drm_gem_get_pages -> shmem_read_mapping_page > to pin a lot of pages, breaking the page reclaim mechanism

Re: [RESEND PATCH v2] drm/gem: Mark pinned pages as unevictable

2019-01-09 Thread Chris Wilson
Quoting Kuo-Hsin Yang (2019-01-08 07:45:17) > The gem drivers use shmemfs to allocate backing storage for gem objects. > On Samsung Chromebook Plus, the drm/rockchip driver may call > rockchip_gem_get_pages -> drm_gem_get_pages -> shmem_read_mapping_page > to pin a lot of pages, breaking the page

[RESEND PATCH v2] drm/gem: Mark pinned pages as unevictable

2019-01-07 Thread Kuo-Hsin Yang
The gem drivers use shmemfs to allocate backing storage for gem objects. On Samsung Chromebook Plus, the drm/rockchip driver may call rockchip_gem_get_pages -> drm_gem_get_pages -> shmem_read_mapping_page to pin a lot of pages, breaking the page reclaim mechanism and causing oom-killer invocation.

Re: [PATCH v2] drm/gem: Mark pinned pages as unevictable

2018-12-18 Thread Kuo-Hsin Yang
On Tue, Dec 18, 2018 at 9:55 PM Chris Wilson wrote: > > Quoting Kuo-Hsin Yang (2018-12-17 09:04:01) > > > > E.g. when the size of a zone is 3.9 GiB, the inactive_ratio is 5. If > > active_anon / inactive_anon < 5 and all pages in the inactive_anon lru > > are pinned, page reclaim would keep

Re: [PATCH v2] drm/gem: Mark pinned pages as unevictable

2018-12-18 Thread Chris Wilson
Quoting Kuo-Hsin Yang (2018-12-17 09:04:01) > The gem drivers use shmemfs to allocate backing storage for gem objects. > On Samsung Chromebook Plus, the drm/rockchip driver may call > rockchip_gem_get_pages -> drm_gem_get_pages -> shmem_read_mapping_page > to pin a lot of pages, breaking the page

[PATCH v2] drm/gem: Mark pinned pages as unevictable

2018-12-17 Thread Kuo-Hsin Yang
The gem drivers use shmemfs to allocate backing storage for gem objects. On Samsung Chromebook Plus, the drm/rockchip driver may call rockchip_gem_get_pages -> drm_gem_get_pages -> shmem_read_mapping_page to pin a lot of pages, breaking the page reclaim mechanism and causing oom-killer invocation.