2010-04-01 19:44:57 +07:00
|
|
|
/*
|
|
|
|
* Copyright (c) Red Hat Inc.
|
|
|
|
|
|
|
|
* Permission is hereby granted, free of charge, to any person obtaining a
|
|
|
|
* copy of this software and associated documentation files (the "Software"),
|
|
|
|
* to deal in the Software without restriction, including without limitation
|
|
|
|
* the rights to use, copy, modify, merge, publish, distribute, sub license,
|
|
|
|
* and/or sell copies of the Software, and to permit persons to whom the
|
|
|
|
* Software is furnished to do so, subject to the following conditions:
|
|
|
|
*
|
|
|
|
* The above copyright notice and this permission notice (including the
|
|
|
|
* next paragraph) shall be included in all copies or substantial portions
|
|
|
|
* of the Software.
|
|
|
|
*
|
|
|
|
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
|
|
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
|
|
* FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT. IN NO EVENT SHALL
|
|
|
|
* THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
|
|
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
|
|
|
|
* FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
|
|
|
|
* DEALINGS IN THE SOFTWARE.
|
|
|
|
*
|
|
|
|
* Authors: Dave Airlie <airlied@redhat.com>
|
|
|
|
* Jerome Glisse <jglisse@redhat.com>
|
|
|
|
* Pauli Nieminen <suokkos@gmail.com>
|
|
|
|
*/
|
|
|
|
|
|
|
|
/* simple list based uncached page pool
|
|
|
|
* - Pool collects resently freed pages for reuse
|
|
|
|
* - Use page->lru to keep a free list
|
|
|
|
* - doesn't track currently in use pages
|
|
|
|
*/
|
2012-03-17 11:43:50 +07:00
|
|
|
|
|
|
|
#define pr_fmt(fmt) "[TTM] " fmt
|
|
|
|
|
2010-04-01 19:44:57 +07:00
|
|
|
#include <linux/list.h>
|
|
|
|
#include <linux/spinlock.h>
|
|
|
|
#include <linux/highmem.h>
|
|
|
|
#include <linux/mm_types.h>
|
2010-04-01 19:44:58 +07:00
|
|
|
#include <linux/module.h>
|
2010-04-01 19:44:57 +07:00
|
|
|
#include <linux/mm.h>
|
2010-04-08 05:42:04 +07:00
|
|
|
#include <linux/seq_file.h> /* for seq_printf */
|
2010-04-08 10:42:03 +07:00
|
|
|
#include <linux/slab.h>
|
2010-11-30 01:52:18 +07:00
|
|
|
#include <linux/dma-mapping.h>
|
2010-04-01 19:44:57 +07:00
|
|
|
|
2011-07-27 06:09:06 +07:00
|
|
|
#include <linux/atomic.h>
|
2010-04-01 19:44:57 +07:00
|
|
|
|
2012-10-03 00:01:07 +07:00
|
|
|
#include <drm/ttm/ttm_bo_driver.h>
|
|
|
|
#include <drm/ttm/ttm_page_alloc.h>
|
2018-07-26 15:49:36 +07:00
|
|
|
#include <drm/ttm/ttm_set_memory.h>
|
2010-04-01 19:44:57 +07:00
|
|
|
|
|
|
|
#define NUM_PAGES_TO_ALLOC (PAGE_SIZE/sizeof(struct page *))
|
|
|
|
#define SMALL_ALLOCATION 16
|
|
|
|
#define FREE_ALL_PAGES (~0U)
|
|
|
|
/* times are in msecs */
|
|
|
|
#define PAGE_FREE_INTERVAL 1000
|
|
|
|
|
|
|
|
/**
|
|
|
|
* struct ttm_page_pool - Pool to reuse recently allocated uc/wc pages.
|
|
|
|
*
|
|
|
|
* @lock: Protects the shared pool from concurrnet access. Must be used with
|
|
|
|
* irqsave/irqrestore variants because pool allocator maybe called from
|
|
|
|
* delayed work.
|
|
|
|
* @fill_lock: Prevent concurrent calls to fill.
|
|
|
|
* @list: Pool of free uc/wc pages for fast reuse.
|
|
|
|
* @gfp_flags: Flags to pass for alloc_page.
|
|
|
|
* @npages: Number of pages in pool.
|
|
|
|
*/
|
|
|
|
struct ttm_page_pool {
|
|
|
|
spinlock_t lock;
|
|
|
|
bool fill_lock;
|
|
|
|
struct list_head list;
|
2010-09-22 23:45:45 +07:00
|
|
|
gfp_t gfp_flags;
|
2010-04-01 19:44:57 +07:00
|
|
|
unsigned npages;
|
2010-04-01 19:44:58 +07:00
|
|
|
char *name;
|
|
|
|
unsigned long nfrees;
|
|
|
|
unsigned long nrefills;
|
2017-11-21 08:37:52 +07:00
|
|
|
unsigned int order;
|
2010-04-01 19:44:57 +07:00
|
|
|
};
|
|
|
|
|
2010-04-01 19:45:03 +07:00
|
|
|
/**
|
|
|
|
* Limits for the pool. They are handled without locks because only place where
|
|
|
|
* they may change is in sysfs store. They won't have immediate effect anyway
|
2010-05-26 21:21:04 +07:00
|
|
|
* so forcing serialization to access them is pointless.
|
2010-04-01 19:45:03 +07:00
|
|
|
*/
|
|
|
|
|
2010-04-01 19:44:57 +07:00
|
|
|
struct ttm_pool_opts {
|
|
|
|
unsigned alloc_size;
|
|
|
|
unsigned max_size;
|
|
|
|
unsigned small;
|
|
|
|
};
|
|
|
|
|
2017-10-05 19:27:34 +07:00
|
|
|
#define NUM_POOLS 6
|
2010-04-01 19:44:57 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* struct ttm_pool_manager - Holds memory pools for fst allocation
|
|
|
|
*
|
|
|
|
* Manager is read only object for pool code so it doesn't need locking.
|
|
|
|
*
|
|
|
|
* @free_interval: minimum number of jiffies between freeing pages from pool.
|
|
|
|
* @page_alloc_inited: reference counting for pool allocation.
|
|
|
|
* @work: Work that is used to shrink the pool. Work is only run when there is
|
|
|
|
* some pages to free.
|
|
|
|
* @small_allocation: Limit in number of pages what is small allocation.
|
|
|
|
*
|
|
|
|
* @pools: All pool objects in use.
|
|
|
|
**/
|
|
|
|
struct ttm_pool_manager {
|
2010-04-01 19:45:03 +07:00
|
|
|
struct kobject kobj;
|
2010-04-01 19:44:57 +07:00
|
|
|
struct shrinker mm_shrink;
|
|
|
|
struct ttm_pool_opts options;
|
|
|
|
|
|
|
|
union {
|
|
|
|
struct ttm_page_pool pools[NUM_POOLS];
|
|
|
|
struct {
|
|
|
|
struct ttm_page_pool wc_pool;
|
|
|
|
struct ttm_page_pool uc_pool;
|
|
|
|
struct ttm_page_pool wc_pool_dma32;
|
|
|
|
struct ttm_page_pool uc_pool_dma32;
|
2017-10-05 19:27:34 +07:00
|
|
|
struct ttm_page_pool wc_pool_huge;
|
|
|
|
struct ttm_page_pool uc_pool_huge;
|
2010-04-01 19:44:57 +07:00
|
|
|
} ;
|
|
|
|
};
|
|
|
|
};
|
|
|
|
|
2010-04-01 19:45:03 +07:00
|
|
|
static struct attribute ttm_page_pool_max = {
|
|
|
|
.name = "pool_max_size",
|
|
|
|
.mode = S_IRUGO | S_IWUSR
|
|
|
|
};
|
|
|
|
static struct attribute ttm_page_pool_small = {
|
|
|
|
.name = "pool_small_allocation",
|
|
|
|
.mode = S_IRUGO | S_IWUSR
|
|
|
|
};
|
|
|
|
static struct attribute ttm_page_pool_alloc_size = {
|
|
|
|
.name = "pool_allocation_size",
|
|
|
|
.mode = S_IRUGO | S_IWUSR
|
|
|
|
};
|
|
|
|
|
|
|
|
static struct attribute *ttm_pool_attrs[] = {
|
|
|
|
&ttm_page_pool_max,
|
|
|
|
&ttm_page_pool_small,
|
|
|
|
&ttm_page_pool_alloc_size,
|
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
static void ttm_pool_kobj_release(struct kobject *kobj)
|
|
|
|
{
|
|
|
|
struct ttm_pool_manager *m =
|
|
|
|
container_of(kobj, struct ttm_pool_manager, kobj);
|
2010-07-04 09:03:07 +07:00
|
|
|
kfree(m);
|
2010-04-01 19:45:03 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
static ssize_t ttm_pool_store(struct kobject *kobj,
|
|
|
|
struct attribute *attr, const char *buffer, size_t size)
|
|
|
|
{
|
|
|
|
struct ttm_pool_manager *m =
|
|
|
|
container_of(kobj, struct ttm_pool_manager, kobj);
|
|
|
|
int chars;
|
|
|
|
unsigned val;
|
|
|
|
chars = sscanf(buffer, "%u", &val);
|
|
|
|
if (chars == 0)
|
|
|
|
return size;
|
|
|
|
|
|
|
|
/* Convert kb to number of pages */
|
|
|
|
val = val / (PAGE_SIZE >> 10);
|
|
|
|
|
|
|
|
if (attr == &ttm_page_pool_max)
|
|
|
|
m->options.max_size = val;
|
|
|
|
else if (attr == &ttm_page_pool_small)
|
|
|
|
m->options.small = val;
|
|
|
|
else if (attr == &ttm_page_pool_alloc_size) {
|
|
|
|
if (val > NUM_PAGES_TO_ALLOC*8) {
|
2012-03-17 11:43:50 +07:00
|
|
|
pr_err("Setting allocation size to %lu is not allowed. Recommended size is %lu\n",
|
2010-05-26 21:21:04 +07:00
|
|
|
NUM_PAGES_TO_ALLOC*(PAGE_SIZE >> 7),
|
|
|
|
NUM_PAGES_TO_ALLOC*(PAGE_SIZE >> 10));
|
2010-04-01 19:45:03 +07:00
|
|
|
return size;
|
|
|
|
} else if (val > NUM_PAGES_TO_ALLOC) {
|
2012-03-17 11:43:50 +07:00
|
|
|
pr_warn("Setting allocation size to larger than %lu is not recommended\n",
|
|
|
|
NUM_PAGES_TO_ALLOC*(PAGE_SIZE >> 10));
|
2010-04-01 19:45:03 +07:00
|
|
|
}
|
|
|
|
m->options.alloc_size = val;
|
|
|
|
}
|
|
|
|
|
|
|
|
return size;
|
|
|
|
}
|
|
|
|
|
|
|
|
static ssize_t ttm_pool_show(struct kobject *kobj,
|
|
|
|
struct attribute *attr, char *buffer)
|
|
|
|
{
|
|
|
|
struct ttm_pool_manager *m =
|
|
|
|
container_of(kobj, struct ttm_pool_manager, kobj);
|
|
|
|
unsigned val = 0;
|
|
|
|
|
|
|
|
if (attr == &ttm_page_pool_max)
|
|
|
|
val = m->options.max_size;
|
|
|
|
else if (attr == &ttm_page_pool_small)
|
|
|
|
val = m->options.small;
|
|
|
|
else if (attr == &ttm_page_pool_alloc_size)
|
|
|
|
val = m->options.alloc_size;
|
|
|
|
|
|
|
|
val = val * (PAGE_SIZE >> 10);
|
|
|
|
|
|
|
|
return snprintf(buffer, PAGE_SIZE, "%u\n", val);
|
|
|
|
}
|
|
|
|
|
|
|
|
static const struct sysfs_ops ttm_pool_sysfs_ops = {
|
|
|
|
.show = &ttm_pool_show,
|
|
|
|
.store = &ttm_pool_store,
|
|
|
|
};
|
|
|
|
|
|
|
|
static struct kobj_type ttm_pool_kobj_type = {
|
|
|
|
.release = &ttm_pool_kobj_release,
|
|
|
|
.sysfs_ops = &ttm_pool_sysfs_ops,
|
|
|
|
.default_attrs = ttm_pool_attrs,
|
|
|
|
};
|
|
|
|
|
2010-07-04 09:03:07 +07:00
|
|
|
static struct ttm_pool_manager *_manager;
|
2010-04-01 19:44:57 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* Select the right pool or requested caching state and ttm flags. */
|
2017-10-05 19:27:34 +07:00
|
|
|
static struct ttm_page_pool *ttm_get_pool(int flags, bool huge,
|
|
|
|
enum ttm_caching_state cstate)
|
2010-04-01 19:44:57 +07:00
|
|
|
{
|
|
|
|
int pool_index;
|
|
|
|
|
|
|
|
if (cstate == tt_cached)
|
|
|
|
return NULL;
|
|
|
|
|
|
|
|
if (cstate == tt_wc)
|
|
|
|
pool_index = 0x0;
|
|
|
|
else
|
|
|
|
pool_index = 0x1;
|
|
|
|
|
2017-10-05 19:27:34 +07:00
|
|
|
if (flags & TTM_PAGE_FLAG_DMA32) {
|
|
|
|
if (huge)
|
|
|
|
return NULL;
|
2010-04-01 19:44:57 +07:00
|
|
|
pool_index |= 0x2;
|
|
|
|
|
2017-10-05 19:27:34 +07:00
|
|
|
} else if (huge) {
|
|
|
|
pool_index |= 0x4;
|
|
|
|
}
|
|
|
|
|
2010-07-04 09:03:07 +07:00
|
|
|
return &_manager->pools[pool_index];
|
2010-04-01 19:44:57 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
/* set memory back to wb and free the pages. */
|
2017-11-21 13:24:48 +07:00
|
|
|
static void ttm_pages_put(struct page *pages[], unsigned npages,
|
|
|
|
unsigned int order)
|
2010-04-01 19:44:57 +07:00
|
|
|
{
|
2017-11-21 13:24:48 +07:00
|
|
|
unsigned int i, pages_nr = (1 << order);
|
|
|
|
|
|
|
|
if (order == 0) {
|
2018-07-26 15:49:36 +07:00
|
|
|
if (ttm_set_pages_array_wb(pages, npages))
|
2017-11-21 13:24:48 +07:00
|
|
|
pr_err("Failed to set %d pages to wb!\n", npages);
|
|
|
|
}
|
|
|
|
|
|
|
|
for (i = 0; i < npages; ++i) {
|
|
|
|
if (order > 0) {
|
2018-07-26 15:49:36 +07:00
|
|
|
if (ttm_set_pages_wb(pages[i], pages_nr))
|
2017-11-21 13:24:48 +07:00
|
|
|
pr_err("Failed to set %d pages to wb!\n", pages_nr);
|
|
|
|
}
|
|
|
|
__free_pages(pages[i], order);
|
|
|
|
}
|
2010-04-01 19:44:57 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
static void ttm_pool_update_free_locked(struct ttm_page_pool *pool,
|
|
|
|
unsigned freed_pages)
|
|
|
|
{
|
|
|
|
pool->npages -= freed_pages;
|
2010-04-01 19:44:58 +07:00
|
|
|
pool->nfrees += freed_pages;
|
2010-04-01 19:44:57 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Free pages from pool.
|
|
|
|
*
|
|
|
|
* To prevent hogging the ttm_swap process we only free NUM_PAGES_TO_ALLOC
|
|
|
|
* number of pages in one go.
|
|
|
|
*
|
|
|
|
* @pool: to free the pages from
|
|
|
|
* @free_all: If set to true will free all pages in pool
|
drm/ttm: Avoid memory allocation from shrinker functions.
Andrew Morton wrote:
> On Wed, 12 Nov 2014 13:08:55 +0900 Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp> wrote:
>
> > Andrew Morton wrote:
> > > Poor ttm guys - this is a bit of a trap we set for them.
> >
> > Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid deadlock.")
> > changed to use sc->gfp_mask rather than GFP_KERNEL.
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *),
> > - GFP_KERNEL);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> >
> > But this bug is caused by sc->gfp_mask containing some flags which are not
> > in GFP_KERNEL, right? Then, I think
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp & GFP_KERNEL);
> >
> > would hide this bug.
> >
> > But I think we should use GFP_ATOMIC (or drop __GFP_WAIT flag)
>
> Well no - ttm_page_pool_free() should stop calling kmalloc altogether.
> Just do
>
> struct page *pages_to_free[16];
>
> and rework the code to free 16 pages at a time. Easy.
Well, ttm code wants to process 512 pages at a time for performance.
Memory footprint increased by 512 * sizeof(struct page *) buffer is
only 4096 bytes. What about using static buffer like below?
----------
>From d3cb5393c9c8099d6b37e769f78c31af1541fe8c Mon Sep 17 00:00:00 2001
From: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Date: Thu, 13 Nov 2014 22:21:54 +0900
Subject: [PATCH] drm/ttm: Avoid memory allocation from shrinker functions.
Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid
deadlock.") caused BUG_ON() due to sc->gfp_mask containing flags
which are not in GFP_KERNEL.
https://bugzilla.kernel.org/show_bug.cgi?id=87891
Changing from sc->gfp_mask to (sc->gfp_mask & GFP_KERNEL) would
avoid the BUG_ON(), but avoiding memory allocation from shrinker
function is better and reliable fix.
Shrinker function is already serialized by global lock, and
clean up function is called after shrinker function is unregistered.
Thus, we can use static buffer when called from shrinker function
and clean up function.
Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Cc: stable <stable@kernel.org> [2.6.35+]
Signed-off-by: Dave Airlie <airlied@redhat.com>
2014-11-13 20:43:23 +07:00
|
|
|
* @use_static: Safe to use static buffer
|
2010-04-01 19:44:57 +07:00
|
|
|
**/
|
2014-08-03 18:02:31 +07:00
|
|
|
static int ttm_page_pool_free(struct ttm_page_pool *pool, unsigned nr_free,
|
drm/ttm: Avoid memory allocation from shrinker functions.
Andrew Morton wrote:
> On Wed, 12 Nov 2014 13:08:55 +0900 Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp> wrote:
>
> > Andrew Morton wrote:
> > > Poor ttm guys - this is a bit of a trap we set for them.
> >
> > Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid deadlock.")
> > changed to use sc->gfp_mask rather than GFP_KERNEL.
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *),
> > - GFP_KERNEL);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> >
> > But this bug is caused by sc->gfp_mask containing some flags which are not
> > in GFP_KERNEL, right? Then, I think
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp & GFP_KERNEL);
> >
> > would hide this bug.
> >
> > But I think we should use GFP_ATOMIC (or drop __GFP_WAIT flag)
>
> Well no - ttm_page_pool_free() should stop calling kmalloc altogether.
> Just do
>
> struct page *pages_to_free[16];
>
> and rework the code to free 16 pages at a time. Easy.
Well, ttm code wants to process 512 pages at a time for performance.
Memory footprint increased by 512 * sizeof(struct page *) buffer is
only 4096 bytes. What about using static buffer like below?
----------
>From d3cb5393c9c8099d6b37e769f78c31af1541fe8c Mon Sep 17 00:00:00 2001
From: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Date: Thu, 13 Nov 2014 22:21:54 +0900
Subject: [PATCH] drm/ttm: Avoid memory allocation from shrinker functions.
Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid
deadlock.") caused BUG_ON() due to sc->gfp_mask containing flags
which are not in GFP_KERNEL.
https://bugzilla.kernel.org/show_bug.cgi?id=87891
Changing from sc->gfp_mask to (sc->gfp_mask & GFP_KERNEL) would
avoid the BUG_ON(), but avoiding memory allocation from shrinker
function is better and reliable fix.
Shrinker function is already serialized by global lock, and
clean up function is called after shrinker function is unregistered.
Thus, we can use static buffer when called from shrinker function
and clean up function.
Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Cc: stable <stable@kernel.org> [2.6.35+]
Signed-off-by: Dave Airlie <airlied@redhat.com>
2014-11-13 20:43:23 +07:00
|
|
|
bool use_static)
|
2010-04-01 19:44:57 +07:00
|
|
|
{
|
drm/ttm: Avoid memory allocation from shrinker functions.
Andrew Morton wrote:
> On Wed, 12 Nov 2014 13:08:55 +0900 Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp> wrote:
>
> > Andrew Morton wrote:
> > > Poor ttm guys - this is a bit of a trap we set for them.
> >
> > Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid deadlock.")
> > changed to use sc->gfp_mask rather than GFP_KERNEL.
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *),
> > - GFP_KERNEL);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> >
> > But this bug is caused by sc->gfp_mask containing some flags which are not
> > in GFP_KERNEL, right? Then, I think
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp & GFP_KERNEL);
> >
> > would hide this bug.
> >
> > But I think we should use GFP_ATOMIC (or drop __GFP_WAIT flag)
>
> Well no - ttm_page_pool_free() should stop calling kmalloc altogether.
> Just do
>
> struct page *pages_to_free[16];
>
> and rework the code to free 16 pages at a time. Easy.
Well, ttm code wants to process 512 pages at a time for performance.
Memory footprint increased by 512 * sizeof(struct page *) buffer is
only 4096 bytes. What about using static buffer like below?
----------
>From d3cb5393c9c8099d6b37e769f78c31af1541fe8c Mon Sep 17 00:00:00 2001
From: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Date: Thu, 13 Nov 2014 22:21:54 +0900
Subject: [PATCH] drm/ttm: Avoid memory allocation from shrinker functions.
Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid
deadlock.") caused BUG_ON() due to sc->gfp_mask containing flags
which are not in GFP_KERNEL.
https://bugzilla.kernel.org/show_bug.cgi?id=87891
Changing from sc->gfp_mask to (sc->gfp_mask & GFP_KERNEL) would
avoid the BUG_ON(), but avoiding memory allocation from shrinker
function is better and reliable fix.
Shrinker function is already serialized by global lock, and
clean up function is called after shrinker function is unregistered.
Thus, we can use static buffer when called from shrinker function
and clean up function.
Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Cc: stable <stable@kernel.org> [2.6.35+]
Signed-off-by: Dave Airlie <airlied@redhat.com>
2014-11-13 20:43:23 +07:00
|
|
|
static struct page *static_buf[NUM_PAGES_TO_ALLOC];
|
2010-04-01 19:44:57 +07:00
|
|
|
unsigned long irq_flags;
|
|
|
|
struct page *p;
|
|
|
|
struct page **pages_to_free;
|
|
|
|
unsigned freed_pages = 0,
|
|
|
|
npages_to_free = nr_free;
|
|
|
|
|
|
|
|
if (NUM_PAGES_TO_ALLOC < nr_free)
|
|
|
|
npages_to_free = NUM_PAGES_TO_ALLOC;
|
|
|
|
|
drm/ttm: Avoid memory allocation from shrinker functions.
Andrew Morton wrote:
> On Wed, 12 Nov 2014 13:08:55 +0900 Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp> wrote:
>
> > Andrew Morton wrote:
> > > Poor ttm guys - this is a bit of a trap we set for them.
> >
> > Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid deadlock.")
> > changed to use sc->gfp_mask rather than GFP_KERNEL.
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *),
> > - GFP_KERNEL);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> >
> > But this bug is caused by sc->gfp_mask containing some flags which are not
> > in GFP_KERNEL, right? Then, I think
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp & GFP_KERNEL);
> >
> > would hide this bug.
> >
> > But I think we should use GFP_ATOMIC (or drop __GFP_WAIT flag)
>
> Well no - ttm_page_pool_free() should stop calling kmalloc altogether.
> Just do
>
> struct page *pages_to_free[16];
>
> and rework the code to free 16 pages at a time. Easy.
Well, ttm code wants to process 512 pages at a time for performance.
Memory footprint increased by 512 * sizeof(struct page *) buffer is
only 4096 bytes. What about using static buffer like below?
----------
>From d3cb5393c9c8099d6b37e769f78c31af1541fe8c Mon Sep 17 00:00:00 2001
From: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Date: Thu, 13 Nov 2014 22:21:54 +0900
Subject: [PATCH] drm/ttm: Avoid memory allocation from shrinker functions.
Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid
deadlock.") caused BUG_ON() due to sc->gfp_mask containing flags
which are not in GFP_KERNEL.
https://bugzilla.kernel.org/show_bug.cgi?id=87891
Changing from sc->gfp_mask to (sc->gfp_mask & GFP_KERNEL) would
avoid the BUG_ON(), but avoiding memory allocation from shrinker
function is better and reliable fix.
Shrinker function is already serialized by global lock, and
clean up function is called after shrinker function is unregistered.
Thus, we can use static buffer when called from shrinker function
and clean up function.
Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Cc: stable <stable@kernel.org> [2.6.35+]
Signed-off-by: Dave Airlie <airlied@redhat.com>
2014-11-13 20:43:23 +07:00
|
|
|
if (use_static)
|
|
|
|
pages_to_free = static_buf;
|
|
|
|
else
|
treewide: kmalloc() -> kmalloc_array()
The kmalloc() function has a 2-factor argument form, kmalloc_array(). This
patch replaces cases of:
kmalloc(a * b, gfp)
with:
kmalloc_array(a * b, gfp)
as well as handling cases of:
kmalloc(a * b * c, gfp)
with:
kmalloc(array3_size(a, b, c), gfp)
as it's slightly less ugly than:
kmalloc_array(array_size(a, b), c, gfp)
This does, however, attempt to ignore constant size factors like:
kmalloc(4 * 1024, gfp)
though any constants defined via macros get caught up in the conversion.
Any factors with a sizeof() of "unsigned char", "char", and "u8" were
dropped, since they're redundant.
The tools/ directory was manually excluded, since it has its own
implementation of kmalloc().
The Coccinelle script used for this was:
// Fix redundant parens around sizeof().
@@
type TYPE;
expression THING, E;
@@
(
kmalloc(
- (sizeof(TYPE)) * E
+ sizeof(TYPE) * E
, ...)
|
kmalloc(
- (sizeof(THING)) * E
+ sizeof(THING) * E
, ...)
)
// Drop single-byte sizes and redundant parens.
@@
expression COUNT;
typedef u8;
typedef __u8;
@@
(
kmalloc(
- sizeof(u8) * (COUNT)
+ COUNT
, ...)
|
kmalloc(
- sizeof(__u8) * (COUNT)
+ COUNT
, ...)
|
kmalloc(
- sizeof(char) * (COUNT)
+ COUNT
, ...)
|
kmalloc(
- sizeof(unsigned char) * (COUNT)
+ COUNT
, ...)
|
kmalloc(
- sizeof(u8) * COUNT
+ COUNT
, ...)
|
kmalloc(
- sizeof(__u8) * COUNT
+ COUNT
, ...)
|
kmalloc(
- sizeof(char) * COUNT
+ COUNT
, ...)
|
kmalloc(
- sizeof(unsigned char) * COUNT
+ COUNT
, ...)
)
// 2-factor product with sizeof(type/expression) and identifier or constant.
@@
type TYPE;
expression THING;
identifier COUNT_ID;
constant COUNT_CONST;
@@
(
- kmalloc
+ kmalloc_array
(
- sizeof(TYPE) * (COUNT_ID)
+ COUNT_ID, sizeof(TYPE)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(TYPE) * COUNT_ID
+ COUNT_ID, sizeof(TYPE)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(TYPE) * (COUNT_CONST)
+ COUNT_CONST, sizeof(TYPE)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(TYPE) * COUNT_CONST
+ COUNT_CONST, sizeof(TYPE)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(THING) * (COUNT_ID)
+ COUNT_ID, sizeof(THING)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(THING) * COUNT_ID
+ COUNT_ID, sizeof(THING)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(THING) * (COUNT_CONST)
+ COUNT_CONST, sizeof(THING)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(THING) * COUNT_CONST
+ COUNT_CONST, sizeof(THING)
, ...)
)
// 2-factor product, only identifiers.
@@
identifier SIZE, COUNT;
@@
- kmalloc
+ kmalloc_array
(
- SIZE * COUNT
+ COUNT, SIZE
, ...)
// 3-factor product with 1 sizeof(type) or sizeof(expression), with
// redundant parens removed.
@@
expression THING;
identifier STRIDE, COUNT;
type TYPE;
@@
(
kmalloc(
- sizeof(TYPE) * (COUNT) * (STRIDE)
+ array3_size(COUNT, STRIDE, sizeof(TYPE))
, ...)
|
kmalloc(
- sizeof(TYPE) * (COUNT) * STRIDE
+ array3_size(COUNT, STRIDE, sizeof(TYPE))
, ...)
|
kmalloc(
- sizeof(TYPE) * COUNT * (STRIDE)
+ array3_size(COUNT, STRIDE, sizeof(TYPE))
, ...)
|
kmalloc(
- sizeof(TYPE) * COUNT * STRIDE
+ array3_size(COUNT, STRIDE, sizeof(TYPE))
, ...)
|
kmalloc(
- sizeof(THING) * (COUNT) * (STRIDE)
+ array3_size(COUNT, STRIDE, sizeof(THING))
, ...)
|
kmalloc(
- sizeof(THING) * (COUNT) * STRIDE
+ array3_size(COUNT, STRIDE, sizeof(THING))
, ...)
|
kmalloc(
- sizeof(THING) * COUNT * (STRIDE)
+ array3_size(COUNT, STRIDE, sizeof(THING))
, ...)
|
kmalloc(
- sizeof(THING) * COUNT * STRIDE
+ array3_size(COUNT, STRIDE, sizeof(THING))
, ...)
)
// 3-factor product with 2 sizeof(variable), with redundant parens removed.
@@
expression THING1, THING2;
identifier COUNT;
type TYPE1, TYPE2;
@@
(
kmalloc(
- sizeof(TYPE1) * sizeof(TYPE2) * COUNT
+ array3_size(COUNT, sizeof(TYPE1), sizeof(TYPE2))
, ...)
|
kmalloc(
- sizeof(TYPE1) * sizeof(THING2) * (COUNT)
+ array3_size(COUNT, sizeof(TYPE1), sizeof(TYPE2))
, ...)
|
kmalloc(
- sizeof(THING1) * sizeof(THING2) * COUNT
+ array3_size(COUNT, sizeof(THING1), sizeof(THING2))
, ...)
|
kmalloc(
- sizeof(THING1) * sizeof(THING2) * (COUNT)
+ array3_size(COUNT, sizeof(THING1), sizeof(THING2))
, ...)
|
kmalloc(
- sizeof(TYPE1) * sizeof(THING2) * COUNT
+ array3_size(COUNT, sizeof(TYPE1), sizeof(THING2))
, ...)
|
kmalloc(
- sizeof(TYPE1) * sizeof(THING2) * (COUNT)
+ array3_size(COUNT, sizeof(TYPE1), sizeof(THING2))
, ...)
)
// 3-factor product, only identifiers, with redundant parens removed.
@@
identifier STRIDE, SIZE, COUNT;
@@
(
kmalloc(
- (COUNT) * STRIDE * SIZE
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
|
kmalloc(
- COUNT * (STRIDE) * SIZE
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
|
kmalloc(
- COUNT * STRIDE * (SIZE)
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
|
kmalloc(
- (COUNT) * (STRIDE) * SIZE
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
|
kmalloc(
- COUNT * (STRIDE) * (SIZE)
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
|
kmalloc(
- (COUNT) * STRIDE * (SIZE)
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
|
kmalloc(
- (COUNT) * (STRIDE) * (SIZE)
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
|
kmalloc(
- COUNT * STRIDE * SIZE
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
)
// Any remaining multi-factor products, first at least 3-factor products,
// when they're not all constants...
@@
expression E1, E2, E3;
constant C1, C2, C3;
@@
(
kmalloc(C1 * C2 * C3, ...)
|
kmalloc(
- (E1) * E2 * E3
+ array3_size(E1, E2, E3)
, ...)
|
kmalloc(
- (E1) * (E2) * E3
+ array3_size(E1, E2, E3)
, ...)
|
kmalloc(
- (E1) * (E2) * (E3)
+ array3_size(E1, E2, E3)
, ...)
|
kmalloc(
- E1 * E2 * E3
+ array3_size(E1, E2, E3)
, ...)
)
// And then all remaining 2 factors products when they're not all constants,
// keeping sizeof() as the second factor argument.
@@
expression THING, E1, E2;
type TYPE;
constant C1, C2, C3;
@@
(
kmalloc(sizeof(THING) * C2, ...)
|
kmalloc(sizeof(TYPE) * C2, ...)
|
kmalloc(C1 * C2 * C3, ...)
|
kmalloc(C1 * C2, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(TYPE) * (E2)
+ E2, sizeof(TYPE)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(TYPE) * E2
+ E2, sizeof(TYPE)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(THING) * (E2)
+ E2, sizeof(THING)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(THING) * E2
+ E2, sizeof(THING)
, ...)
|
- kmalloc
+ kmalloc_array
(
- (E1) * E2
+ E1, E2
, ...)
|
- kmalloc
+ kmalloc_array
(
- (E1) * (E2)
+ E1, E2
, ...)
|
- kmalloc
+ kmalloc_array
(
- E1 * E2
+ E1, E2
, ...)
)
Signed-off-by: Kees Cook <keescook@chromium.org>
2018-06-13 03:55:00 +07:00
|
|
|
pages_to_free = kmalloc_array(npages_to_free,
|
|
|
|
sizeof(struct page *),
|
|
|
|
GFP_KERNEL);
|
2010-04-01 19:44:57 +07:00
|
|
|
if (!pages_to_free) {
|
2017-11-03 22:00:35 +07:00
|
|
|
pr_debug("Failed to allocate memory for pool free operation\n");
|
2010-04-01 19:44:57 +07:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
restart:
|
|
|
|
spin_lock_irqsave(&pool->lock, irq_flags);
|
|
|
|
|
|
|
|
list_for_each_entry_reverse(p, &pool->list, lru) {
|
|
|
|
if (freed_pages >= npages_to_free)
|
|
|
|
break;
|
|
|
|
|
|
|
|
pages_to_free[freed_pages++] = p;
|
|
|
|
/* We can only remove NUM_PAGES_TO_ALLOC at a time. */
|
|
|
|
if (freed_pages >= NUM_PAGES_TO_ALLOC) {
|
|
|
|
/* remove range of pages from the pool */
|
|
|
|
__list_del(p->lru.prev, &pool->list);
|
|
|
|
|
|
|
|
ttm_pool_update_free_locked(pool, freed_pages);
|
|
|
|
/**
|
|
|
|
* Because changing page caching is costly
|
|
|
|
* we unlock the pool to prevent stalling.
|
|
|
|
*/
|
|
|
|
spin_unlock_irqrestore(&pool->lock, irq_flags);
|
|
|
|
|
2017-11-21 13:24:48 +07:00
|
|
|
ttm_pages_put(pages_to_free, freed_pages, pool->order);
|
2010-04-01 19:44:57 +07:00
|
|
|
if (likely(nr_free != FREE_ALL_PAGES))
|
|
|
|
nr_free -= freed_pages;
|
|
|
|
|
|
|
|
if (NUM_PAGES_TO_ALLOC >= nr_free)
|
|
|
|
npages_to_free = nr_free;
|
|
|
|
else
|
|
|
|
npages_to_free = NUM_PAGES_TO_ALLOC;
|
|
|
|
|
|
|
|
freed_pages = 0;
|
|
|
|
|
|
|
|
/* free all so restart the processing */
|
|
|
|
if (nr_free)
|
|
|
|
goto restart;
|
|
|
|
|
2011-06-09 00:06:15 +07:00
|
|
|
/* Not allowed to fall through or break because
|
2010-04-01 19:44:57 +07:00
|
|
|
* following context is inside spinlock while we are
|
|
|
|
* outside here.
|
|
|
|
*/
|
|
|
|
goto out;
|
|
|
|
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/* remove range of pages from the pool */
|
|
|
|
if (freed_pages) {
|
|
|
|
__list_del(&p->lru, &pool->list);
|
|
|
|
|
|
|
|
ttm_pool_update_free_locked(pool, freed_pages);
|
|
|
|
nr_free -= freed_pages;
|
|
|
|
}
|
|
|
|
|
|
|
|
spin_unlock_irqrestore(&pool->lock, irq_flags);
|
|
|
|
|
|
|
|
if (freed_pages)
|
2017-11-21 13:24:48 +07:00
|
|
|
ttm_pages_put(pages_to_free, freed_pages, pool->order);
|
2010-04-01 19:44:57 +07:00
|
|
|
out:
|
drm/ttm: Avoid memory allocation from shrinker functions.
Andrew Morton wrote:
> On Wed, 12 Nov 2014 13:08:55 +0900 Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp> wrote:
>
> > Andrew Morton wrote:
> > > Poor ttm guys - this is a bit of a trap we set for them.
> >
> > Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid deadlock.")
> > changed to use sc->gfp_mask rather than GFP_KERNEL.
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *),
> > - GFP_KERNEL);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> >
> > But this bug is caused by sc->gfp_mask containing some flags which are not
> > in GFP_KERNEL, right? Then, I think
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp & GFP_KERNEL);
> >
> > would hide this bug.
> >
> > But I think we should use GFP_ATOMIC (or drop __GFP_WAIT flag)
>
> Well no - ttm_page_pool_free() should stop calling kmalloc altogether.
> Just do
>
> struct page *pages_to_free[16];
>
> and rework the code to free 16 pages at a time. Easy.
Well, ttm code wants to process 512 pages at a time for performance.
Memory footprint increased by 512 * sizeof(struct page *) buffer is
only 4096 bytes. What about using static buffer like below?
----------
>From d3cb5393c9c8099d6b37e769f78c31af1541fe8c Mon Sep 17 00:00:00 2001
From: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Date: Thu, 13 Nov 2014 22:21:54 +0900
Subject: [PATCH] drm/ttm: Avoid memory allocation from shrinker functions.
Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid
deadlock.") caused BUG_ON() due to sc->gfp_mask containing flags
which are not in GFP_KERNEL.
https://bugzilla.kernel.org/show_bug.cgi?id=87891
Changing from sc->gfp_mask to (sc->gfp_mask & GFP_KERNEL) would
avoid the BUG_ON(), but avoiding memory allocation from shrinker
function is better and reliable fix.
Shrinker function is already serialized by global lock, and
clean up function is called after shrinker function is unregistered.
Thus, we can use static buffer when called from shrinker function
and clean up function.
Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Cc: stable <stable@kernel.org> [2.6.35+]
Signed-off-by: Dave Airlie <airlied@redhat.com>
2014-11-13 20:43:23 +07:00
|
|
|
if (pages_to_free != static_buf)
|
|
|
|
kfree(pages_to_free);
|
2010-04-01 19:44:57 +07:00
|
|
|
return nr_free;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
2010-05-26 21:21:04 +07:00
|
|
|
* Callback for mm to request pool to reduce number of page held.
|
drivers: convert shrinkers to new count/scan API
Convert the driver shrinkers to the new API. Most changes are compile
tested only because I either don't have the hardware or it's staging
stuff.
FWIW, the md and android code is pretty good, but the rest of it makes me
want to claw my eyes out. The amount of broken code I just encountered is
mind boggling. I've added comments explaining what is broken, but I fear
that some of the code would be best dealt with by being dragged behind the
bike shed, burying in mud up to it's neck and then run over repeatedly
with a blunt lawn mower.
Special mention goes to the zcache/zcache2 drivers. They can't co-exist
in the build at the same time, they are under different menu options in
menuconfig, they only show up when you've got the right set of mm
subsystem options configured and so even compile testing is an exercise in
pulling teeth. And that doesn't even take into account the horrible,
broken code...
[glommer@openvz.org: fixes for i915, android lowmem, zcache, bcache]
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Glauber Costa <glommer@openvz.org>
Acked-by: Mel Gorman <mgorman@suse.de>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Kent Overstreet <koverstreet@google.com>
Cc: John Stultz <john.stultz@linaro.org>
Cc: David Rientjes <rientjes@google.com>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: "Theodore Ts'o" <tytso@mit.edu>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Cc: Arve Hjønnevåg <arve@android.com>
Cc: Carlos Maiolino <cmaiolino@redhat.com>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Chuck Lever <chuck.lever@oracle.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: David Rientjes <rientjes@google.com>
Cc: Gleb Natapov <gleb@redhat.com>
Cc: Greg Thelen <gthelen@google.com>
Cc: J. Bruce Fields <bfields@redhat.com>
Cc: Jan Kara <jack@suse.cz>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: John Stultz <john.stultz@linaro.org>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Cc: Kent Overstreet <koverstreet@google.com>
Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Cc: Marcelo Tosatti <mtosatti@redhat.com>
Cc: Mel Gorman <mgorman@suse.de>
Cc: Steven Whitehouse <swhiteho@redhat.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: Trond Myklebust <Trond.Myklebust@netapp.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
2013-08-28 07:18:11 +07:00
|
|
|
*
|
|
|
|
* XXX: (dchinner) Deadlock warning!
|
|
|
|
*
|
|
|
|
* This code is crying out for a shrinker per pool....
|
2010-04-01 19:44:57 +07:00
|
|
|
*/
|
drivers: convert shrinkers to new count/scan API
Convert the driver shrinkers to the new API. Most changes are compile
tested only because I either don't have the hardware or it's staging
stuff.
FWIW, the md and android code is pretty good, but the rest of it makes me
want to claw my eyes out. The amount of broken code I just encountered is
mind boggling. I've added comments explaining what is broken, but I fear
that some of the code would be best dealt with by being dragged behind the
bike shed, burying in mud up to it's neck and then run over repeatedly
with a blunt lawn mower.
Special mention goes to the zcache/zcache2 drivers. They can't co-exist
in the build at the same time, they are under different menu options in
menuconfig, they only show up when you've got the right set of mm
subsystem options configured and so even compile testing is an exercise in
pulling teeth. And that doesn't even take into account the horrible,
broken code...
[glommer@openvz.org: fixes for i915, android lowmem, zcache, bcache]
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Glauber Costa <glommer@openvz.org>
Acked-by: Mel Gorman <mgorman@suse.de>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Kent Overstreet <koverstreet@google.com>
Cc: John Stultz <john.stultz@linaro.org>
Cc: David Rientjes <rientjes@google.com>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: "Theodore Ts'o" <tytso@mit.edu>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Cc: Arve Hjønnevåg <arve@android.com>
Cc: Carlos Maiolino <cmaiolino@redhat.com>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Chuck Lever <chuck.lever@oracle.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: David Rientjes <rientjes@google.com>
Cc: Gleb Natapov <gleb@redhat.com>
Cc: Greg Thelen <gthelen@google.com>
Cc: J. Bruce Fields <bfields@redhat.com>
Cc: Jan Kara <jack@suse.cz>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: John Stultz <john.stultz@linaro.org>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Cc: Kent Overstreet <koverstreet@google.com>
Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Cc: Marcelo Tosatti <mtosatti@redhat.com>
Cc: Mel Gorman <mgorman@suse.de>
Cc: Steven Whitehouse <swhiteho@redhat.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: Trond Myklebust <Trond.Myklebust@netapp.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
2013-08-28 07:18:11 +07:00
|
|
|
static unsigned long
|
|
|
|
ttm_pool_shrink_scan(struct shrinker *shrink, struct shrink_control *sc)
|
2010-04-01 19:44:57 +07:00
|
|
|
{
|
2014-08-03 18:02:03 +07:00
|
|
|
static DEFINE_MUTEX(lock);
|
|
|
|
static unsigned start_pool;
|
2010-04-01 19:44:57 +07:00
|
|
|
unsigned i;
|
2014-08-03 18:02:03 +07:00
|
|
|
unsigned pool_offset;
|
2010-04-01 19:44:57 +07:00
|
|
|
struct ttm_page_pool *pool;
|
2011-05-25 07:12:27 +07:00
|
|
|
int shrink_pages = sc->nr_to_scan;
|
drivers: convert shrinkers to new count/scan API
Convert the driver shrinkers to the new API. Most changes are compile
tested only because I either don't have the hardware or it's staging
stuff.
FWIW, the md and android code is pretty good, but the rest of it makes me
want to claw my eyes out. The amount of broken code I just encountered is
mind boggling. I've added comments explaining what is broken, but I fear
that some of the code would be best dealt with by being dragged behind the
bike shed, burying in mud up to it's neck and then run over repeatedly
with a blunt lawn mower.
Special mention goes to the zcache/zcache2 drivers. They can't co-exist
in the build at the same time, they are under different menu options in
menuconfig, they only show up when you've got the right set of mm
subsystem options configured and so even compile testing is an exercise in
pulling teeth. And that doesn't even take into account the horrible,
broken code...
[glommer@openvz.org: fixes for i915, android lowmem, zcache, bcache]
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Glauber Costa <glommer@openvz.org>
Acked-by: Mel Gorman <mgorman@suse.de>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Kent Overstreet <koverstreet@google.com>
Cc: John Stultz <john.stultz@linaro.org>
Cc: David Rientjes <rientjes@google.com>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: "Theodore Ts'o" <tytso@mit.edu>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Cc: Arve Hjønnevåg <arve@android.com>
Cc: Carlos Maiolino <cmaiolino@redhat.com>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Chuck Lever <chuck.lever@oracle.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: David Rientjes <rientjes@google.com>
Cc: Gleb Natapov <gleb@redhat.com>
Cc: Greg Thelen <gthelen@google.com>
Cc: J. Bruce Fields <bfields@redhat.com>
Cc: Jan Kara <jack@suse.cz>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: John Stultz <john.stultz@linaro.org>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Cc: Kent Overstreet <koverstreet@google.com>
Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Cc: Marcelo Tosatti <mtosatti@redhat.com>
Cc: Mel Gorman <mgorman@suse.de>
Cc: Steven Whitehouse <swhiteho@redhat.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: Trond Myklebust <Trond.Myklebust@netapp.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
2013-08-28 07:18:11 +07:00
|
|
|
unsigned long freed = 0;
|
2017-11-21 08:37:52 +07:00
|
|
|
unsigned int nr_free_pool;
|
2010-04-01 19:44:57 +07:00
|
|
|
|
2014-08-03 18:02:03 +07:00
|
|
|
if (!mutex_trylock(&lock))
|
|
|
|
return SHRINK_STOP;
|
|
|
|
pool_offset = ++start_pool % NUM_POOLS;
|
2010-04-01 19:44:57 +07:00
|
|
|
/* select start pool in round robin fashion */
|
|
|
|
for (i = 0; i < NUM_POOLS; ++i) {
|
|
|
|
unsigned nr_free = shrink_pages;
|
2017-11-21 15:47:16 +07:00
|
|
|
unsigned page_nr;
|
|
|
|
|
2010-04-01 19:44:57 +07:00
|
|
|
if (shrink_pages == 0)
|
|
|
|
break;
|
2017-11-21 08:37:52 +07:00
|
|
|
|
2010-07-04 09:03:07 +07:00
|
|
|
pool = &_manager->pools[(i + pool_offset)%NUM_POOLS];
|
2017-11-21 15:47:16 +07:00
|
|
|
page_nr = (1 << pool->order);
|
drm/ttm: Avoid memory allocation from shrinker functions.
Andrew Morton wrote:
> On Wed, 12 Nov 2014 13:08:55 +0900 Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp> wrote:
>
> > Andrew Morton wrote:
> > > Poor ttm guys - this is a bit of a trap we set for them.
> >
> > Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid deadlock.")
> > changed to use sc->gfp_mask rather than GFP_KERNEL.
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *),
> > - GFP_KERNEL);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> >
> > But this bug is caused by sc->gfp_mask containing some flags which are not
> > in GFP_KERNEL, right? Then, I think
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp & GFP_KERNEL);
> >
> > would hide this bug.
> >
> > But I think we should use GFP_ATOMIC (or drop __GFP_WAIT flag)
>
> Well no - ttm_page_pool_free() should stop calling kmalloc altogether.
> Just do
>
> struct page *pages_to_free[16];
>
> and rework the code to free 16 pages at a time. Easy.
Well, ttm code wants to process 512 pages at a time for performance.
Memory footprint increased by 512 * sizeof(struct page *) buffer is
only 4096 bytes. What about using static buffer like below?
----------
>From d3cb5393c9c8099d6b37e769f78c31af1541fe8c Mon Sep 17 00:00:00 2001
From: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Date: Thu, 13 Nov 2014 22:21:54 +0900
Subject: [PATCH] drm/ttm: Avoid memory allocation from shrinker functions.
Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid
deadlock.") caused BUG_ON() due to sc->gfp_mask containing flags
which are not in GFP_KERNEL.
https://bugzilla.kernel.org/show_bug.cgi?id=87891
Changing from sc->gfp_mask to (sc->gfp_mask & GFP_KERNEL) would
avoid the BUG_ON(), but avoiding memory allocation from shrinker
function is better and reliable fix.
Shrinker function is already serialized by global lock, and
clean up function is called after shrinker function is unregistered.
Thus, we can use static buffer when called from shrinker function
and clean up function.
Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Cc: stable <stable@kernel.org> [2.6.35+]
Signed-off-by: Dave Airlie <airlied@redhat.com>
2014-11-13 20:43:23 +07:00
|
|
|
/* OK to use static buffer since global mutex is held. */
|
2017-11-21 15:47:16 +07:00
|
|
|
nr_free_pool = roundup(nr_free, page_nr) >> pool->order;
|
2017-11-21 08:37:52 +07:00
|
|
|
shrink_pages = ttm_page_pool_free(pool, nr_free_pool, true);
|
2017-11-21 15:47:16 +07:00
|
|
|
freed += (nr_free_pool - shrink_pages) << pool->order;
|
|
|
|
if (freed >= sc->nr_to_scan)
|
|
|
|
break;
|
2017-12-01 17:21:34 +07:00
|
|
|
shrink_pages <<= pool->order;
|
2010-04-01 19:44:57 +07:00
|
|
|
}
|
2014-08-03 18:02:03 +07:00
|
|
|
mutex_unlock(&lock);
|
drivers: convert shrinkers to new count/scan API
Convert the driver shrinkers to the new API. Most changes are compile
tested only because I either don't have the hardware or it's staging
stuff.
FWIW, the md and android code is pretty good, but the rest of it makes me
want to claw my eyes out. The amount of broken code I just encountered is
mind boggling. I've added comments explaining what is broken, but I fear
that some of the code would be best dealt with by being dragged behind the
bike shed, burying in mud up to it's neck and then run over repeatedly
with a blunt lawn mower.
Special mention goes to the zcache/zcache2 drivers. They can't co-exist
in the build at the same time, they are under different menu options in
menuconfig, they only show up when you've got the right set of mm
subsystem options configured and so even compile testing is an exercise in
pulling teeth. And that doesn't even take into account the horrible,
broken code...
[glommer@openvz.org: fixes for i915, android lowmem, zcache, bcache]
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Glauber Costa <glommer@openvz.org>
Acked-by: Mel Gorman <mgorman@suse.de>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Kent Overstreet <koverstreet@google.com>
Cc: John Stultz <john.stultz@linaro.org>
Cc: David Rientjes <rientjes@google.com>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: "Theodore Ts'o" <tytso@mit.edu>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Cc: Arve Hjønnevåg <arve@android.com>
Cc: Carlos Maiolino <cmaiolino@redhat.com>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Chuck Lever <chuck.lever@oracle.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: David Rientjes <rientjes@google.com>
Cc: Gleb Natapov <gleb@redhat.com>
Cc: Greg Thelen <gthelen@google.com>
Cc: J. Bruce Fields <bfields@redhat.com>
Cc: Jan Kara <jack@suse.cz>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: John Stultz <john.stultz@linaro.org>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Cc: Kent Overstreet <koverstreet@google.com>
Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Cc: Marcelo Tosatti <mtosatti@redhat.com>
Cc: Mel Gorman <mgorman@suse.de>
Cc: Steven Whitehouse <swhiteho@redhat.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: Trond Myklebust <Trond.Myklebust@netapp.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
2013-08-28 07:18:11 +07:00
|
|
|
return freed;
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
static unsigned long
|
|
|
|
ttm_pool_shrink_count(struct shrinker *shrink, struct shrink_control *sc)
|
|
|
|
{
|
|
|
|
unsigned i;
|
|
|
|
unsigned long count = 0;
|
2017-11-21 08:37:52 +07:00
|
|
|
struct ttm_page_pool *pool;
|
drivers: convert shrinkers to new count/scan API
Convert the driver shrinkers to the new API. Most changes are compile
tested only because I either don't have the hardware or it's staging
stuff.
FWIW, the md and android code is pretty good, but the rest of it makes me
want to claw my eyes out. The amount of broken code I just encountered is
mind boggling. I've added comments explaining what is broken, but I fear
that some of the code would be best dealt with by being dragged behind the
bike shed, burying in mud up to it's neck and then run over repeatedly
with a blunt lawn mower.
Special mention goes to the zcache/zcache2 drivers. They can't co-exist
in the build at the same time, they are under different menu options in
menuconfig, they only show up when you've got the right set of mm
subsystem options configured and so even compile testing is an exercise in
pulling teeth. And that doesn't even take into account the horrible,
broken code...
[glommer@openvz.org: fixes for i915, android lowmem, zcache, bcache]
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Glauber Costa <glommer@openvz.org>
Acked-by: Mel Gorman <mgorman@suse.de>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Kent Overstreet <koverstreet@google.com>
Cc: John Stultz <john.stultz@linaro.org>
Cc: David Rientjes <rientjes@google.com>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: "Theodore Ts'o" <tytso@mit.edu>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Cc: Arve Hjønnevåg <arve@android.com>
Cc: Carlos Maiolino <cmaiolino@redhat.com>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Chuck Lever <chuck.lever@oracle.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: David Rientjes <rientjes@google.com>
Cc: Gleb Natapov <gleb@redhat.com>
Cc: Greg Thelen <gthelen@google.com>
Cc: J. Bruce Fields <bfields@redhat.com>
Cc: Jan Kara <jack@suse.cz>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: John Stultz <john.stultz@linaro.org>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Cc: Kent Overstreet <koverstreet@google.com>
Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Cc: Marcelo Tosatti <mtosatti@redhat.com>
Cc: Mel Gorman <mgorman@suse.de>
Cc: Steven Whitehouse <swhiteho@redhat.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: Trond Myklebust <Trond.Myklebust@netapp.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
2013-08-28 07:18:11 +07:00
|
|
|
|
2017-11-21 08:37:52 +07:00
|
|
|
for (i = 0; i < NUM_POOLS; ++i) {
|
|
|
|
pool = &_manager->pools[i];
|
|
|
|
count += (pool->npages << pool->order);
|
|
|
|
}
|
drivers: convert shrinkers to new count/scan API
Convert the driver shrinkers to the new API. Most changes are compile
tested only because I either don't have the hardware or it's staging
stuff.
FWIW, the md and android code is pretty good, but the rest of it makes me
want to claw my eyes out. The amount of broken code I just encountered is
mind boggling. I've added comments explaining what is broken, but I fear
that some of the code would be best dealt with by being dragged behind the
bike shed, burying in mud up to it's neck and then run over repeatedly
with a blunt lawn mower.
Special mention goes to the zcache/zcache2 drivers. They can't co-exist
in the build at the same time, they are under different menu options in
menuconfig, they only show up when you've got the right set of mm
subsystem options configured and so even compile testing is an exercise in
pulling teeth. And that doesn't even take into account the horrible,
broken code...
[glommer@openvz.org: fixes for i915, android lowmem, zcache, bcache]
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Glauber Costa <glommer@openvz.org>
Acked-by: Mel Gorman <mgorman@suse.de>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Kent Overstreet <koverstreet@google.com>
Cc: John Stultz <john.stultz@linaro.org>
Cc: David Rientjes <rientjes@google.com>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: "Theodore Ts'o" <tytso@mit.edu>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Cc: Arve Hjønnevåg <arve@android.com>
Cc: Carlos Maiolino <cmaiolino@redhat.com>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Chuck Lever <chuck.lever@oracle.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: David Rientjes <rientjes@google.com>
Cc: Gleb Natapov <gleb@redhat.com>
Cc: Greg Thelen <gthelen@google.com>
Cc: J. Bruce Fields <bfields@redhat.com>
Cc: Jan Kara <jack@suse.cz>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: John Stultz <john.stultz@linaro.org>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Cc: Kent Overstreet <koverstreet@google.com>
Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Cc: Marcelo Tosatti <mtosatti@redhat.com>
Cc: Mel Gorman <mgorman@suse.de>
Cc: Steven Whitehouse <swhiteho@redhat.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: Trond Myklebust <Trond.Myklebust@netapp.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
2013-08-28 07:18:11 +07:00
|
|
|
|
|
|
|
return count;
|
2010-04-01 19:44:57 +07:00
|
|
|
}
|
|
|
|
|
2018-01-11 11:57:58 +07:00
|
|
|
static int ttm_pool_mm_shrink_init(struct ttm_pool_manager *manager)
|
2010-04-01 19:44:57 +07:00
|
|
|
{
|
drivers: convert shrinkers to new count/scan API
Convert the driver shrinkers to the new API. Most changes are compile
tested only because I either don't have the hardware or it's staging
stuff.
FWIW, the md and android code is pretty good, but the rest of it makes me
want to claw my eyes out. The amount of broken code I just encountered is
mind boggling. I've added comments explaining what is broken, but I fear
that some of the code would be best dealt with by being dragged behind the
bike shed, burying in mud up to it's neck and then run over repeatedly
with a blunt lawn mower.
Special mention goes to the zcache/zcache2 drivers. They can't co-exist
in the build at the same time, they are under different menu options in
menuconfig, they only show up when you've got the right set of mm
subsystem options configured and so even compile testing is an exercise in
pulling teeth. And that doesn't even take into account the horrible,
broken code...
[glommer@openvz.org: fixes for i915, android lowmem, zcache, bcache]
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Glauber Costa <glommer@openvz.org>
Acked-by: Mel Gorman <mgorman@suse.de>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Kent Overstreet <koverstreet@google.com>
Cc: John Stultz <john.stultz@linaro.org>
Cc: David Rientjes <rientjes@google.com>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: "Theodore Ts'o" <tytso@mit.edu>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Cc: Arve Hjønnevåg <arve@android.com>
Cc: Carlos Maiolino <cmaiolino@redhat.com>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Chuck Lever <chuck.lever@oracle.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: David Rientjes <rientjes@google.com>
Cc: Gleb Natapov <gleb@redhat.com>
Cc: Greg Thelen <gthelen@google.com>
Cc: J. Bruce Fields <bfields@redhat.com>
Cc: Jan Kara <jack@suse.cz>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: John Stultz <john.stultz@linaro.org>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Cc: Kent Overstreet <koverstreet@google.com>
Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Cc: Marcelo Tosatti <mtosatti@redhat.com>
Cc: Mel Gorman <mgorman@suse.de>
Cc: Steven Whitehouse <swhiteho@redhat.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: Trond Myklebust <Trond.Myklebust@netapp.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
2013-08-28 07:18:11 +07:00
|
|
|
manager->mm_shrink.count_objects = ttm_pool_shrink_count;
|
|
|
|
manager->mm_shrink.scan_objects = ttm_pool_shrink_scan;
|
2010-04-01 19:44:57 +07:00
|
|
|
manager->mm_shrink.seeks = 1;
|
2018-01-11 11:57:58 +07:00
|
|
|
return register_shrinker(&manager->mm_shrink);
|
2010-04-01 19:44:57 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
static void ttm_pool_mm_shrink_fini(struct ttm_pool_manager *manager)
|
|
|
|
{
|
|
|
|
unregister_shrinker(&manager->mm_shrink);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int ttm_set_pages_caching(struct page **pages,
|
|
|
|
enum ttm_caching_state cstate, unsigned cpages)
|
|
|
|
{
|
|
|
|
int r = 0;
|
|
|
|
/* Set page caching */
|
|
|
|
switch (cstate) {
|
|
|
|
case tt_uncached:
|
2018-07-26 15:49:36 +07:00
|
|
|
r = ttm_set_pages_array_uc(pages, cpages);
|
2010-04-01 19:44:57 +07:00
|
|
|
if (r)
|
2012-03-17 11:43:50 +07:00
|
|
|
pr_err("Failed to set %d pages to uc!\n", cpages);
|
2010-04-01 19:44:57 +07:00
|
|
|
break;
|
|
|
|
case tt_wc:
|
2018-07-26 15:49:36 +07:00
|
|
|
r = ttm_set_pages_array_wc(pages, cpages);
|
2010-04-01 19:44:57 +07:00
|
|
|
if (r)
|
2012-03-17 11:43:50 +07:00
|
|
|
pr_err("Failed to set %d pages to wc!\n", cpages);
|
2010-04-01 19:44:57 +07:00
|
|
|
break;
|
|
|
|
default:
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
return r;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Free pages the pages that failed to change the caching state. If there is
|
|
|
|
* any pages that have changed their caching state already put them to the
|
|
|
|
* pool.
|
|
|
|
*/
|
|
|
|
static void ttm_handle_caching_state_failure(struct list_head *pages,
|
|
|
|
int ttm_flags, enum ttm_caching_state cstate,
|
|
|
|
struct page **failed_pages, unsigned cpages)
|
|
|
|
{
|
|
|
|
unsigned i;
|
2010-05-26 21:21:04 +07:00
|
|
|
/* Failed pages have to be freed */
|
2010-04-01 19:44:57 +07:00
|
|
|
for (i = 0; i < cpages; ++i) {
|
|
|
|
list_del(&failed_pages[i]->lru);
|
|
|
|
__free_page(failed_pages[i]);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Allocate new pages with correct caching.
|
|
|
|
*
|
|
|
|
* This function is reentrant if caller updates count depending on number of
|
|
|
|
* pages returned in pages array.
|
|
|
|
*/
|
2010-09-22 23:45:45 +07:00
|
|
|
static int ttm_alloc_new_pages(struct list_head *pages, gfp_t gfp_flags,
|
2017-10-05 19:27:34 +07:00
|
|
|
int ttm_flags, enum ttm_caching_state cstate,
|
|
|
|
unsigned count, unsigned order)
|
2010-04-01 19:44:57 +07:00
|
|
|
{
|
|
|
|
struct page **caching_array;
|
|
|
|
struct page *p;
|
|
|
|
int r = 0;
|
2017-10-05 19:27:34 +07:00
|
|
|
unsigned i, j, cpages;
|
|
|
|
unsigned npages = 1 << order;
|
2017-12-01 17:23:56 +07:00
|
|
|
unsigned max_cpages = min(count << order, (unsigned)NUM_PAGES_TO_ALLOC);
|
2010-04-01 19:44:57 +07:00
|
|
|
|
|
|
|
/* allocate array for page caching change */
|
treewide: kmalloc() -> kmalloc_array()
The kmalloc() function has a 2-factor argument form, kmalloc_array(). This
patch replaces cases of:
kmalloc(a * b, gfp)
with:
kmalloc_array(a * b, gfp)
as well as handling cases of:
kmalloc(a * b * c, gfp)
with:
kmalloc(array3_size(a, b, c), gfp)
as it's slightly less ugly than:
kmalloc_array(array_size(a, b), c, gfp)
This does, however, attempt to ignore constant size factors like:
kmalloc(4 * 1024, gfp)
though any constants defined via macros get caught up in the conversion.
Any factors with a sizeof() of "unsigned char", "char", and "u8" were
dropped, since they're redundant.
The tools/ directory was manually excluded, since it has its own
implementation of kmalloc().
The Coccinelle script used for this was:
// Fix redundant parens around sizeof().
@@
type TYPE;
expression THING, E;
@@
(
kmalloc(
- (sizeof(TYPE)) * E
+ sizeof(TYPE) * E
, ...)
|
kmalloc(
- (sizeof(THING)) * E
+ sizeof(THING) * E
, ...)
)
// Drop single-byte sizes and redundant parens.
@@
expression COUNT;
typedef u8;
typedef __u8;
@@
(
kmalloc(
- sizeof(u8) * (COUNT)
+ COUNT
, ...)
|
kmalloc(
- sizeof(__u8) * (COUNT)
+ COUNT
, ...)
|
kmalloc(
- sizeof(char) * (COUNT)
+ COUNT
, ...)
|
kmalloc(
- sizeof(unsigned char) * (COUNT)
+ COUNT
, ...)
|
kmalloc(
- sizeof(u8) * COUNT
+ COUNT
, ...)
|
kmalloc(
- sizeof(__u8) * COUNT
+ COUNT
, ...)
|
kmalloc(
- sizeof(char) * COUNT
+ COUNT
, ...)
|
kmalloc(
- sizeof(unsigned char) * COUNT
+ COUNT
, ...)
)
// 2-factor product with sizeof(type/expression) and identifier or constant.
@@
type TYPE;
expression THING;
identifier COUNT_ID;
constant COUNT_CONST;
@@
(
- kmalloc
+ kmalloc_array
(
- sizeof(TYPE) * (COUNT_ID)
+ COUNT_ID, sizeof(TYPE)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(TYPE) * COUNT_ID
+ COUNT_ID, sizeof(TYPE)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(TYPE) * (COUNT_CONST)
+ COUNT_CONST, sizeof(TYPE)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(TYPE) * COUNT_CONST
+ COUNT_CONST, sizeof(TYPE)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(THING) * (COUNT_ID)
+ COUNT_ID, sizeof(THING)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(THING) * COUNT_ID
+ COUNT_ID, sizeof(THING)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(THING) * (COUNT_CONST)
+ COUNT_CONST, sizeof(THING)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(THING) * COUNT_CONST
+ COUNT_CONST, sizeof(THING)
, ...)
)
// 2-factor product, only identifiers.
@@
identifier SIZE, COUNT;
@@
- kmalloc
+ kmalloc_array
(
- SIZE * COUNT
+ COUNT, SIZE
, ...)
// 3-factor product with 1 sizeof(type) or sizeof(expression), with
// redundant parens removed.
@@
expression THING;
identifier STRIDE, COUNT;
type TYPE;
@@
(
kmalloc(
- sizeof(TYPE) * (COUNT) * (STRIDE)
+ array3_size(COUNT, STRIDE, sizeof(TYPE))
, ...)
|
kmalloc(
- sizeof(TYPE) * (COUNT) * STRIDE
+ array3_size(COUNT, STRIDE, sizeof(TYPE))
, ...)
|
kmalloc(
- sizeof(TYPE) * COUNT * (STRIDE)
+ array3_size(COUNT, STRIDE, sizeof(TYPE))
, ...)
|
kmalloc(
- sizeof(TYPE) * COUNT * STRIDE
+ array3_size(COUNT, STRIDE, sizeof(TYPE))
, ...)
|
kmalloc(
- sizeof(THING) * (COUNT) * (STRIDE)
+ array3_size(COUNT, STRIDE, sizeof(THING))
, ...)
|
kmalloc(
- sizeof(THING) * (COUNT) * STRIDE
+ array3_size(COUNT, STRIDE, sizeof(THING))
, ...)
|
kmalloc(
- sizeof(THING) * COUNT * (STRIDE)
+ array3_size(COUNT, STRIDE, sizeof(THING))
, ...)
|
kmalloc(
- sizeof(THING) * COUNT * STRIDE
+ array3_size(COUNT, STRIDE, sizeof(THING))
, ...)
)
// 3-factor product with 2 sizeof(variable), with redundant parens removed.
@@
expression THING1, THING2;
identifier COUNT;
type TYPE1, TYPE2;
@@
(
kmalloc(
- sizeof(TYPE1) * sizeof(TYPE2) * COUNT
+ array3_size(COUNT, sizeof(TYPE1), sizeof(TYPE2))
, ...)
|
kmalloc(
- sizeof(TYPE1) * sizeof(THING2) * (COUNT)
+ array3_size(COUNT, sizeof(TYPE1), sizeof(TYPE2))
, ...)
|
kmalloc(
- sizeof(THING1) * sizeof(THING2) * COUNT
+ array3_size(COUNT, sizeof(THING1), sizeof(THING2))
, ...)
|
kmalloc(
- sizeof(THING1) * sizeof(THING2) * (COUNT)
+ array3_size(COUNT, sizeof(THING1), sizeof(THING2))
, ...)
|
kmalloc(
- sizeof(TYPE1) * sizeof(THING2) * COUNT
+ array3_size(COUNT, sizeof(TYPE1), sizeof(THING2))
, ...)
|
kmalloc(
- sizeof(TYPE1) * sizeof(THING2) * (COUNT)
+ array3_size(COUNT, sizeof(TYPE1), sizeof(THING2))
, ...)
)
// 3-factor product, only identifiers, with redundant parens removed.
@@
identifier STRIDE, SIZE, COUNT;
@@
(
kmalloc(
- (COUNT) * STRIDE * SIZE
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
|
kmalloc(
- COUNT * (STRIDE) * SIZE
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
|
kmalloc(
- COUNT * STRIDE * (SIZE)
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
|
kmalloc(
- (COUNT) * (STRIDE) * SIZE
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
|
kmalloc(
- COUNT * (STRIDE) * (SIZE)
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
|
kmalloc(
- (COUNT) * STRIDE * (SIZE)
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
|
kmalloc(
- (COUNT) * (STRIDE) * (SIZE)
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
|
kmalloc(
- COUNT * STRIDE * SIZE
+ array3_size(COUNT, STRIDE, SIZE)
, ...)
)
// Any remaining multi-factor products, first at least 3-factor products,
// when they're not all constants...
@@
expression E1, E2, E3;
constant C1, C2, C3;
@@
(
kmalloc(C1 * C2 * C3, ...)
|
kmalloc(
- (E1) * E2 * E3
+ array3_size(E1, E2, E3)
, ...)
|
kmalloc(
- (E1) * (E2) * E3
+ array3_size(E1, E2, E3)
, ...)
|
kmalloc(
- (E1) * (E2) * (E3)
+ array3_size(E1, E2, E3)
, ...)
|
kmalloc(
- E1 * E2 * E3
+ array3_size(E1, E2, E3)
, ...)
)
// And then all remaining 2 factors products when they're not all constants,
// keeping sizeof() as the second factor argument.
@@
expression THING, E1, E2;
type TYPE;
constant C1, C2, C3;
@@
(
kmalloc(sizeof(THING) * C2, ...)
|
kmalloc(sizeof(TYPE) * C2, ...)
|
kmalloc(C1 * C2 * C3, ...)
|
kmalloc(C1 * C2, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(TYPE) * (E2)
+ E2, sizeof(TYPE)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(TYPE) * E2
+ E2, sizeof(TYPE)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(THING) * (E2)
+ E2, sizeof(THING)
, ...)
|
- kmalloc
+ kmalloc_array
(
- sizeof(THING) * E2
+ E2, sizeof(THING)
, ...)
|
- kmalloc
+ kmalloc_array
(
- (E1) * E2
+ E1, E2
, ...)
|
- kmalloc
+ kmalloc_array
(
- (E1) * (E2)
+ E1, E2
, ...)
|
- kmalloc
+ kmalloc_array
(
- E1 * E2
+ E1, E2
, ...)
)
Signed-off-by: Kees Cook <keescook@chromium.org>
2018-06-13 03:55:00 +07:00
|
|
|
caching_array = kmalloc_array(max_cpages, sizeof(struct page *),
|
|
|
|
GFP_KERNEL);
|
2010-04-01 19:44:57 +07:00
|
|
|
|
|
|
|
if (!caching_array) {
|
2017-11-03 22:00:35 +07:00
|
|
|
pr_debug("Unable to allocate table for new pages\n");
|
2010-04-01 19:44:57 +07:00
|
|
|
return -ENOMEM;
|
|
|
|
}
|
|
|
|
|
|
|
|
for (i = 0, cpages = 0; i < count; ++i) {
|
2017-10-05 19:27:34 +07:00
|
|
|
p = alloc_pages(gfp_flags, order);
|
2010-04-01 19:44:57 +07:00
|
|
|
|
|
|
|
if (!p) {
|
2017-11-03 22:00:35 +07:00
|
|
|
pr_debug("Unable to get page %u\n", i);
|
2010-04-01 19:44:57 +07:00
|
|
|
|
|
|
|
/* store already allocated pages in the pool after
|
|
|
|
* setting the caching state */
|
|
|
|
if (cpages) {
|
2010-05-26 21:21:04 +07:00
|
|
|
r = ttm_set_pages_caching(caching_array,
|
|
|
|
cstate, cpages);
|
2010-04-01 19:44:57 +07:00
|
|
|
if (r)
|
|
|
|
ttm_handle_caching_state_failure(pages,
|
|
|
|
ttm_flags, cstate,
|
|
|
|
caching_array, cpages);
|
|
|
|
}
|
|
|
|
r = -ENOMEM;
|
|
|
|
goto out;
|
|
|
|
}
|
|
|
|
|
2017-10-05 19:27:34 +07:00
|
|
|
list_add(&p->lru, pages);
|
|
|
|
|
2010-04-01 19:44:57 +07:00
|
|
|
#ifdef CONFIG_HIGHMEM
|
|
|
|
/* gfp flags of highmem page should never be dma32 so we
|
|
|
|
* we should be fine in such case
|
|
|
|
*/
|
2017-10-05 19:27:34 +07:00
|
|
|
if (PageHighMem(p))
|
|
|
|
continue;
|
|
|
|
|
2010-04-01 19:44:57 +07:00
|
|
|
#endif
|
2017-10-05 19:27:34 +07:00
|
|
|
for (j = 0; j < npages; ++j) {
|
|
|
|
caching_array[cpages++] = p++;
|
2010-04-01 19:44:57 +07:00
|
|
|
if (cpages == max_cpages) {
|
|
|
|
|
|
|
|
r = ttm_set_pages_caching(caching_array,
|
|
|
|
cstate, cpages);
|
|
|
|
if (r) {
|
|
|
|
ttm_handle_caching_state_failure(pages,
|
|
|
|
ttm_flags, cstate,
|
|
|
|
caching_array, cpages);
|
|
|
|
goto out;
|
|
|
|
}
|
|
|
|
cpages = 0;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
if (cpages) {
|
|
|
|
r = ttm_set_pages_caching(caching_array, cstate, cpages);
|
|
|
|
if (r)
|
|
|
|
ttm_handle_caching_state_failure(pages,
|
|
|
|
ttm_flags, cstate,
|
|
|
|
caching_array, cpages);
|
|
|
|
}
|
|
|
|
out:
|
|
|
|
kfree(caching_array);
|
|
|
|
|
|
|
|
return r;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
2011-06-09 00:06:15 +07:00
|
|
|
* Fill the given pool if there aren't enough pages and the requested number of
|
2010-04-01 19:44:57 +07:00
|
|
|
* pages is small.
|
|
|
|
*/
|
2017-10-05 19:27:34 +07:00
|
|
|
static void ttm_page_pool_fill_locked(struct ttm_page_pool *pool, int ttm_flags,
|
|
|
|
enum ttm_caching_state cstate,
|
|
|
|
unsigned count, unsigned long *irq_flags)
|
2010-04-01 19:44:57 +07:00
|
|
|
{
|
|
|
|
struct page *p;
|
|
|
|
int r;
|
|
|
|
unsigned cpages = 0;
|
|
|
|
/**
|
|
|
|
* Only allow one pool fill operation at a time.
|
|
|
|
* If pool doesn't have enough pages for the allocation new pages are
|
|
|
|
* allocated from outside of pool.
|
|
|
|
*/
|
|
|
|
if (pool->fill_lock)
|
|
|
|
return;
|
|
|
|
|
|
|
|
pool->fill_lock = true;
|
|
|
|
|
2011-06-09 00:06:15 +07:00
|
|
|
/* If allocation request is small and there are not enough
|
|
|
|
* pages in a pool we fill the pool up first. */
|
2010-07-04 09:03:07 +07:00
|
|
|
if (count < _manager->options.small
|
2010-04-01 19:44:57 +07:00
|
|
|
&& count > pool->npages) {
|
|
|
|
struct list_head new_pages;
|
2010-07-04 09:03:07 +07:00
|
|
|
unsigned alloc_size = _manager->options.alloc_size;
|
2010-04-01 19:44:57 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* Can't change page caching if in irqsave context. We have to
|
|
|
|
* drop the pool->lock.
|
|
|
|
*/
|
|
|
|
spin_unlock_irqrestore(&pool->lock, *irq_flags);
|
|
|
|
|
|
|
|
INIT_LIST_HEAD(&new_pages);
|
|
|
|
r = ttm_alloc_new_pages(&new_pages, pool->gfp_flags, ttm_flags,
|
2017-10-05 19:27:34 +07:00
|
|
|
cstate, alloc_size, 0);
|
2010-04-01 19:44:57 +07:00
|
|
|
spin_lock_irqsave(&pool->lock, *irq_flags);
|
|
|
|
|
|
|
|
if (!r) {
|
|
|
|
list_splice(&new_pages, &pool->list);
|
2010-04-01 19:44:58 +07:00
|
|
|
++pool->nrefills;
|
2010-04-01 19:44:57 +07:00
|
|
|
pool->npages += alloc_size;
|
|
|
|
} else {
|
2017-11-03 22:00:35 +07:00
|
|
|
pr_debug("Failed to fill pool (%p)\n", pool);
|
2010-04-01 19:44:57 +07:00
|
|
|
/* If we have any pages left put them to the pool. */
|
2017-08-16 13:25:51 +07:00
|
|
|
list_for_each_entry(p, &new_pages, lru) {
|
2010-04-01 19:44:57 +07:00
|
|
|
++cpages;
|
|
|
|
}
|
|
|
|
list_splice(&new_pages, &pool->list);
|
|
|
|
pool->npages += cpages;
|
|
|
|
}
|
|
|
|
|
|
|
|
}
|
|
|
|
pool->fill_lock = false;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
2017-09-21 16:28:25 +07:00
|
|
|
* Allocate pages from the pool and put them on the return list.
|
2010-04-01 19:44:57 +07:00
|
|
|
*
|
2017-09-21 16:28:25 +07:00
|
|
|
* @return zero for success or negative error code.
|
2010-04-01 19:44:57 +07:00
|
|
|
*/
|
2017-09-21 16:28:25 +07:00
|
|
|
static int ttm_page_pool_get_pages(struct ttm_page_pool *pool,
|
|
|
|
struct list_head *pages,
|
|
|
|
int ttm_flags,
|
|
|
|
enum ttm_caching_state cstate,
|
2017-10-05 19:27:34 +07:00
|
|
|
unsigned count, unsigned order)
|
2010-04-01 19:44:57 +07:00
|
|
|
{
|
|
|
|
unsigned long irq_flags;
|
|
|
|
struct list_head *p;
|
|
|
|
unsigned i;
|
2017-09-21 16:28:25 +07:00
|
|
|
int r = 0;
|
2010-04-01 19:44:57 +07:00
|
|
|
|
|
|
|
spin_lock_irqsave(&pool->lock, irq_flags);
|
2017-10-05 19:27:34 +07:00
|
|
|
if (!order)
|
|
|
|
ttm_page_pool_fill_locked(pool, ttm_flags, cstate, count,
|
|
|
|
&irq_flags);
|
2010-04-01 19:44:57 +07:00
|
|
|
|
|
|
|
if (count >= pool->npages) {
|
|
|
|
/* take all pages from the pool */
|
|
|
|
list_splice_init(&pool->list, pages);
|
|
|
|
count -= pool->npages;
|
|
|
|
pool->npages = 0;
|
|
|
|
goto out;
|
|
|
|
}
|
|
|
|
/* find the last pages to include for requested number of pages. Split
|
2011-06-09 00:06:15 +07:00
|
|
|
* pool to begin and halve it to reduce search space. */
|
2010-04-01 19:44:57 +07:00
|
|
|
if (count <= pool->npages/2) {
|
|
|
|
i = 0;
|
|
|
|
list_for_each(p, &pool->list) {
|
|
|
|
if (++i == count)
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
} else {
|
|
|
|
i = pool->npages + 1;
|
|
|
|
list_for_each_prev(p, &pool->list) {
|
|
|
|
if (--i == count)
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
2011-06-09 00:06:15 +07:00
|
|
|
/* Cut 'count' number of pages from the pool */
|
2010-04-01 19:44:57 +07:00
|
|
|
list_cut_position(pages, &pool->list, p);
|
|
|
|
pool->npages -= count;
|
|
|
|
count = 0;
|
|
|
|
out:
|
|
|
|
spin_unlock_irqrestore(&pool->lock, irq_flags);
|
2017-09-21 16:28:25 +07:00
|
|
|
|
|
|
|
/* clear the pages coming from the pool if requested */
|
|
|
|
if (ttm_flags & TTM_PAGE_FLAG_ZERO_ALLOC) {
|
|
|
|
struct page *page;
|
|
|
|
|
|
|
|
list_for_each_entry(page, pages, lru) {
|
|
|
|
if (PageHighMem(page))
|
|
|
|
clear_highpage(page);
|
|
|
|
else
|
|
|
|
clear_page(page_address(page));
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/* If pool didn't have enough pages allocate new one. */
|
|
|
|
if (count) {
|
|
|
|
gfp_t gfp_flags = pool->gfp_flags;
|
|
|
|
|
|
|
|
/* set zero flag for page allocation if required */
|
|
|
|
if (ttm_flags & TTM_PAGE_FLAG_ZERO_ALLOC)
|
|
|
|
gfp_flags |= __GFP_ZERO;
|
|
|
|
|
2017-12-22 20:12:40 +07:00
|
|
|
if (ttm_flags & TTM_PAGE_FLAG_NO_RETRY)
|
|
|
|
gfp_flags |= __GFP_RETRY_MAYFAIL;
|
|
|
|
|
2017-09-21 16:28:25 +07:00
|
|
|
/* ttm_alloc_new_pages doesn't reference pool so we can run
|
|
|
|
* multiple requests in parallel.
|
|
|
|
**/
|
|
|
|
r = ttm_alloc_new_pages(pages, gfp_flags, ttm_flags, cstate,
|
2017-10-05 19:27:34 +07:00
|
|
|
count, order);
|
2017-09-21 16:28:25 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
return r;
|
2010-04-01 19:44:57 +07:00
|
|
|
}
|
|
|
|
|
2011-11-10 05:15:26 +07:00
|
|
|
/* Put all pages in pages list to correct pool to wait for reuse */
|
|
|
|
static void ttm_put_pages(struct page **pages, unsigned npages, int flags,
|
|
|
|
enum ttm_caching_state cstate)
|
|
|
|
{
|
2017-10-05 19:27:34 +07:00
|
|
|
struct ttm_page_pool *pool = ttm_get_pool(flags, false, cstate);
|
2017-10-12 18:25:08 +07:00
|
|
|
#ifdef CONFIG_TRANSPARENT_HUGEPAGE
|
2017-10-05 19:27:34 +07:00
|
|
|
struct ttm_page_pool *huge = ttm_get_pool(flags, true, cstate);
|
2017-10-12 18:25:08 +07:00
|
|
|
#endif
|
2011-11-10 05:15:26 +07:00
|
|
|
unsigned long irq_flags;
|
|
|
|
unsigned i;
|
|
|
|
|
|
|
|
if (pool == NULL) {
|
|
|
|
/* No pool for this memory type so free the pages */
|
2017-09-20 20:06:12 +07:00
|
|
|
i = 0;
|
|
|
|
while (i < npages) {
|
2017-10-13 00:28:42 +07:00
|
|
|
#ifdef CONFIG_TRANSPARENT_HUGEPAGE
|
|
|
|
struct page *p = pages[i];
|
|
|
|
#endif
|
|
|
|
unsigned order = 0, j;
|
2017-09-20 20:06:12 +07:00
|
|
|
|
|
|
|
if (!pages[i]) {
|
|
|
|
++i;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2017-10-13 00:28:42 +07:00
|
|
|
#ifdef CONFIG_TRANSPARENT_HUGEPAGE
|
2019-04-02 14:26:52 +07:00
|
|
|
if (!(flags & TTM_PAGE_FLAG_DMA32) &&
|
|
|
|
(npages - i) >= HPAGE_PMD_NR) {
|
2019-04-02 14:29:35 +07:00
|
|
|
for (j = 1; j < HPAGE_PMD_NR; ++j)
|
2019-04-10 16:43:43 +07:00
|
|
|
if (++p != pages[i + j])
|
2017-11-23 09:12:17 +07:00
|
|
|
break;
|
2017-10-13 00:28:42 +07:00
|
|
|
|
2017-11-23 09:12:17 +07:00
|
|
|
if (j == HPAGE_PMD_NR)
|
|
|
|
order = HPAGE_PMD_ORDER;
|
|
|
|
}
|
2017-10-13 00:28:42 +07:00
|
|
|
#endif
|
|
|
|
|
2017-09-20 20:06:12 +07:00
|
|
|
if (page_count(pages[i]) != 1)
|
|
|
|
pr_err("Erroneous page count. Leaking pages.\n");
|
|
|
|
__free_pages(pages[i], order);
|
|
|
|
|
2017-10-13 00:28:42 +07:00
|
|
|
j = 1 << order;
|
|
|
|
while (j) {
|
2017-09-20 20:06:12 +07:00
|
|
|
pages[i++] = NULL;
|
2017-10-13 00:28:42 +07:00
|
|
|
--j;
|
2011-11-10 05:15:26 +07:00
|
|
|
}
|
|
|
|
}
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2017-10-05 19:27:34 +07:00
|
|
|
i = 0;
|
|
|
|
#ifdef CONFIG_TRANSPARENT_HUGEPAGE
|
|
|
|
if (huge) {
|
|
|
|
unsigned max_size, n2free;
|
|
|
|
|
|
|
|
spin_lock_irqsave(&huge->lock, irq_flags);
|
2019-04-02 14:26:52 +07:00
|
|
|
while ((npages - i) >= HPAGE_PMD_NR) {
|
2017-10-05 19:27:34 +07:00
|
|
|
struct page *p = pages[i];
|
|
|
|
unsigned j;
|
|
|
|
|
|
|
|
if (!p)
|
|
|
|
break;
|
|
|
|
|
2019-04-02 14:29:35 +07:00
|
|
|
for (j = 1; j < HPAGE_PMD_NR; ++j)
|
2019-04-10 16:43:43 +07:00
|
|
|
if (++p != pages[i + j])
|
2017-10-05 19:27:34 +07:00
|
|
|
break;
|
|
|
|
|
|
|
|
if (j != HPAGE_PMD_NR)
|
|
|
|
break;
|
|
|
|
|
|
|
|
list_add_tail(&pages[i]->lru, &huge->list);
|
|
|
|
|
|
|
|
for (j = 0; j < HPAGE_PMD_NR; ++j)
|
|
|
|
pages[i++] = NULL;
|
|
|
|
huge->npages++;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Check that we don't go over the pool limit */
|
|
|
|
max_size = _manager->options.max_size;
|
|
|
|
max_size /= HPAGE_PMD_NR;
|
|
|
|
if (huge->npages > max_size)
|
|
|
|
n2free = huge->npages - max_size;
|
|
|
|
else
|
|
|
|
n2free = 0;
|
|
|
|
spin_unlock_irqrestore(&huge->lock, irq_flags);
|
|
|
|
if (n2free)
|
|
|
|
ttm_page_pool_free(huge, n2free, false);
|
|
|
|
}
|
|
|
|
#endif
|
|
|
|
|
2011-11-10 05:15:26 +07:00
|
|
|
spin_lock_irqsave(&pool->lock, irq_flags);
|
2017-10-05 19:27:34 +07:00
|
|
|
while (i < npages) {
|
2011-11-10 05:15:26 +07:00
|
|
|
if (pages[i]) {
|
|
|
|
if (page_count(pages[i]) != 1)
|
2012-03-17 11:43:50 +07:00
|
|
|
pr_err("Erroneous page count. Leaking pages.\n");
|
2011-11-10 05:15:26 +07:00
|
|
|
list_add_tail(&pages[i]->lru, &pool->list);
|
|
|
|
pages[i] = NULL;
|
|
|
|
pool->npages++;
|
|
|
|
}
|
2017-10-05 19:27:34 +07:00
|
|
|
++i;
|
2011-11-10 05:15:26 +07:00
|
|
|
}
|
|
|
|
/* Check that we don't go over the pool limit */
|
|
|
|
npages = 0;
|
|
|
|
if (pool->npages > _manager->options.max_size) {
|
|
|
|
npages = pool->npages - _manager->options.max_size;
|
|
|
|
/* free at least NUM_PAGES_TO_ALLOC number of pages
|
|
|
|
* to reduce calls to set_memory_wb */
|
|
|
|
if (npages < NUM_PAGES_TO_ALLOC)
|
|
|
|
npages = NUM_PAGES_TO_ALLOC;
|
|
|
|
}
|
|
|
|
spin_unlock_irqrestore(&pool->lock, irq_flags);
|
|
|
|
if (npages)
|
drm/ttm: Avoid memory allocation from shrinker functions.
Andrew Morton wrote:
> On Wed, 12 Nov 2014 13:08:55 +0900 Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp> wrote:
>
> > Andrew Morton wrote:
> > > Poor ttm guys - this is a bit of a trap we set for them.
> >
> > Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid deadlock.")
> > changed to use sc->gfp_mask rather than GFP_KERNEL.
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *),
> > - GFP_KERNEL);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> >
> > But this bug is caused by sc->gfp_mask containing some flags which are not
> > in GFP_KERNEL, right? Then, I think
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp & GFP_KERNEL);
> >
> > would hide this bug.
> >
> > But I think we should use GFP_ATOMIC (or drop __GFP_WAIT flag)
>
> Well no - ttm_page_pool_free() should stop calling kmalloc altogether.
> Just do
>
> struct page *pages_to_free[16];
>
> and rework the code to free 16 pages at a time. Easy.
Well, ttm code wants to process 512 pages at a time for performance.
Memory footprint increased by 512 * sizeof(struct page *) buffer is
only 4096 bytes. What about using static buffer like below?
----------
>From d3cb5393c9c8099d6b37e769f78c31af1541fe8c Mon Sep 17 00:00:00 2001
From: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Date: Thu, 13 Nov 2014 22:21:54 +0900
Subject: [PATCH] drm/ttm: Avoid memory allocation from shrinker functions.
Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid
deadlock.") caused BUG_ON() due to sc->gfp_mask containing flags
which are not in GFP_KERNEL.
https://bugzilla.kernel.org/show_bug.cgi?id=87891
Changing from sc->gfp_mask to (sc->gfp_mask & GFP_KERNEL) would
avoid the BUG_ON(), but avoiding memory allocation from shrinker
function is better and reliable fix.
Shrinker function is already serialized by global lock, and
clean up function is called after shrinker function is unregistered.
Thus, we can use static buffer when called from shrinker function
and clean up function.
Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Cc: stable <stable@kernel.org> [2.6.35+]
Signed-off-by: Dave Airlie <airlied@redhat.com>
2014-11-13 20:43:23 +07:00
|
|
|
ttm_page_pool_free(pool, npages, false);
|
2011-11-10 05:15:26 +07:00
|
|
|
}
|
|
|
|
|
2010-04-01 19:44:57 +07:00
|
|
|
/*
|
|
|
|
* On success pages list will hold count number of correctly
|
|
|
|
* cached pages.
|
|
|
|
*/
|
2011-11-10 05:15:26 +07:00
|
|
|
static int ttm_get_pages(struct page **pages, unsigned npages, int flags,
|
|
|
|
enum ttm_caching_state cstate)
|
2010-04-01 19:44:57 +07:00
|
|
|
{
|
2017-10-05 19:27:34 +07:00
|
|
|
struct ttm_page_pool *pool = ttm_get_pool(flags, false, cstate);
|
2017-10-12 18:25:08 +07:00
|
|
|
#ifdef CONFIG_TRANSPARENT_HUGEPAGE
|
2017-10-05 19:27:34 +07:00
|
|
|
struct ttm_page_pool *huge = ttm_get_pool(flags, true, cstate);
|
2017-10-12 18:25:08 +07:00
|
|
|
#endif
|
2011-11-11 06:24:09 +07:00
|
|
|
struct list_head plist;
|
2010-04-01 19:44:57 +07:00
|
|
|
struct page *p = NULL;
|
2017-12-04 17:17:54 +07:00
|
|
|
unsigned count, first;
|
2010-04-01 19:44:57 +07:00
|
|
|
int r;
|
|
|
|
|
|
|
|
/* No pool for cached pages */
|
|
|
|
if (pool == NULL) {
|
2017-09-21 16:28:25 +07:00
|
|
|
gfp_t gfp_flags = GFP_USER;
|
2017-10-12 18:25:08 +07:00
|
|
|
unsigned i;
|
|
|
|
#ifdef CONFIG_TRANSPARENT_HUGEPAGE
|
|
|
|
unsigned j;
|
|
|
|
#endif
|
2017-09-20 20:06:12 +07:00
|
|
|
|
2017-09-21 16:28:25 +07:00
|
|
|
/* set zero flag for page allocation if required */
|
|
|
|
if (flags & TTM_PAGE_FLAG_ZERO_ALLOC)
|
|
|
|
gfp_flags |= __GFP_ZERO;
|
|
|
|
|
2017-12-22 20:12:40 +07:00
|
|
|
if (flags & TTM_PAGE_FLAG_NO_RETRY)
|
|
|
|
gfp_flags |= __GFP_RETRY_MAYFAIL;
|
|
|
|
|
2010-04-01 19:44:57 +07:00
|
|
|
if (flags & TTM_PAGE_FLAG_DMA32)
|
|
|
|
gfp_flags |= GFP_DMA32;
|
|
|
|
else
|
2010-05-26 21:21:03 +07:00
|
|
|
gfp_flags |= GFP_HIGHUSER;
|
2010-04-01 19:44:57 +07:00
|
|
|
|
2017-09-20 20:06:12 +07:00
|
|
|
i = 0;
|
|
|
|
#ifdef CONFIG_TRANSPARENT_HUGEPAGE
|
2017-11-23 09:12:17 +07:00
|
|
|
if (!(gfp_flags & GFP_DMA32)) {
|
|
|
|
while (npages >= HPAGE_PMD_NR) {
|
|
|
|
gfp_t huge_flags = gfp_flags;
|
2017-09-20 20:06:12 +07:00
|
|
|
|
2018-04-25 22:32:10 +07:00
|
|
|
huge_flags |= GFP_TRANSHUGE_LIGHT | __GFP_NORETRY |
|
|
|
|
__GFP_KSWAPD_RECLAIM;
|
2017-11-23 09:12:17 +07:00
|
|
|
huge_flags &= ~__GFP_MOVABLE;
|
|
|
|
huge_flags &= ~__GFP_COMP;
|
|
|
|
p = alloc_pages(huge_flags, HPAGE_PMD_ORDER);
|
|
|
|
if (!p)
|
|
|
|
break;
|
2017-09-20 20:06:12 +07:00
|
|
|
|
2017-11-23 09:12:17 +07:00
|
|
|
for (j = 0; j < HPAGE_PMD_NR; ++j)
|
|
|
|
pages[i++] = p++;
|
2017-09-20 20:06:12 +07:00
|
|
|
|
2017-11-23 09:12:17 +07:00
|
|
|
npages -= HPAGE_PMD_NR;
|
|
|
|
}
|
2017-09-20 20:06:12 +07:00
|
|
|
}
|
|
|
|
#endif
|
|
|
|
|
2017-12-04 17:17:54 +07:00
|
|
|
first = i;
|
2017-09-20 20:06:12 +07:00
|
|
|
while (npages) {
|
2011-04-13 06:15:09 +07:00
|
|
|
p = alloc_page(gfp_flags);
|
2010-04-01 19:44:57 +07:00
|
|
|
if (!p) {
|
2017-11-03 22:00:35 +07:00
|
|
|
pr_debug("Unable to allocate page\n");
|
2010-04-01 19:44:57 +07:00
|
|
|
return -ENOMEM;
|
|
|
|
}
|
2011-04-13 06:15:09 +07:00
|
|
|
|
2017-12-04 17:17:54 +07:00
|
|
|
/* Swap the pages if we detect consecutive order */
|
|
|
|
if (i > first && pages[i - 1] == p - 1)
|
|
|
|
swap(p, pages[i - 1]);
|
|
|
|
|
2017-09-20 20:06:12 +07:00
|
|
|
pages[i++] = p;
|
|
|
|
--npages;
|
2010-04-01 19:44:57 +07:00
|
|
|
}
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2017-10-05 19:27:34 +07:00
|
|
|
count = 0;
|
|
|
|
|
|
|
|
#ifdef CONFIG_TRANSPARENT_HUGEPAGE
|
|
|
|
if (huge && npages >= HPAGE_PMD_NR) {
|
|
|
|
INIT_LIST_HEAD(&plist);
|
|
|
|
ttm_page_pool_get_pages(huge, &plist, flags, cstate,
|
|
|
|
npages / HPAGE_PMD_NR,
|
|
|
|
HPAGE_PMD_ORDER);
|
|
|
|
|
|
|
|
list_for_each_entry(p, &plist, lru) {
|
|
|
|
unsigned j;
|
|
|
|
|
|
|
|
for (j = 0; j < HPAGE_PMD_NR; ++j)
|
|
|
|
pages[count++] = &p[j];
|
|
|
|
}
|
|
|
|
}
|
|
|
|
#endif
|
|
|
|
|
2011-11-11 06:24:09 +07:00
|
|
|
INIT_LIST_HEAD(&plist);
|
2017-10-05 19:27:34 +07:00
|
|
|
r = ttm_page_pool_get_pages(pool, &plist, flags, cstate,
|
|
|
|
npages - count, 0);
|
2017-09-21 16:28:25 +07:00
|
|
|
|
2017-12-04 17:26:14 +07:00
|
|
|
first = count;
|
|
|
|
list_for_each_entry(p, &plist, lru) {
|
|
|
|
struct page *tmp = p;
|
|
|
|
|
|
|
|
/* Swap the pages if we detect consecutive order */
|
|
|
|
if (count > first && pages[count - 1] == tmp - 1)
|
|
|
|
swap(tmp, pages[count - 1]);
|
|
|
|
pages[count++] = tmp;
|
|
|
|
}
|
2010-04-01 19:44:57 +07:00
|
|
|
|
2017-09-21 16:28:25 +07:00
|
|
|
if (r) {
|
|
|
|
/* If there is any pages in the list put them back to
|
|
|
|
* the pool.
|
|
|
|
*/
|
2017-11-03 22:00:35 +07:00
|
|
|
pr_debug("Failed to allocate extra pages for large request\n");
|
2017-09-21 16:28:25 +07:00
|
|
|
ttm_put_pages(pages, count, flags, cstate);
|
|
|
|
return r;
|
2010-04-01 19:44:57 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2014-07-22 09:55:39 +07:00
|
|
|
static void ttm_page_pool_init_locked(struct ttm_page_pool *pool, gfp_t flags,
|
2017-11-21 08:37:52 +07:00
|
|
|
char *name, unsigned int order)
|
2010-04-01 19:44:57 +07:00
|
|
|
{
|
|
|
|
spin_lock_init(&pool->lock);
|
|
|
|
pool->fill_lock = false;
|
|
|
|
INIT_LIST_HEAD(&pool->list);
|
2010-04-01 19:44:58 +07:00
|
|
|
pool->npages = pool->nfrees = 0;
|
2010-04-01 19:44:57 +07:00
|
|
|
pool->gfp_flags = flags;
|
2010-04-01 19:44:58 +07:00
|
|
|
pool->name = name;
|
2017-11-21 08:37:52 +07:00
|
|
|
pool->order = order;
|
2010-04-01 19:44:57 +07:00
|
|
|
}
|
|
|
|
|
2010-04-01 19:45:03 +07:00
|
|
|
int ttm_page_alloc_init(struct ttm_mem_global *glob, unsigned max_pages)
|
2010-04-01 19:44:57 +07:00
|
|
|
{
|
2010-04-01 19:45:03 +07:00
|
|
|
int ret;
|
2017-11-21 08:37:52 +07:00
|
|
|
#ifdef CONFIG_TRANSPARENT_HUGEPAGE
|
|
|
|
unsigned order = HPAGE_PMD_ORDER;
|
|
|
|
#else
|
|
|
|
unsigned order = 0;
|
|
|
|
#endif
|
2010-07-04 09:03:07 +07:00
|
|
|
|
|
|
|
WARN_ON(_manager);
|
2010-04-01 19:44:57 +07:00
|
|
|
|
2012-03-17 11:43:50 +07:00
|
|
|
pr_info("Initializing pool allocator\n");
|
2010-04-01 19:44:57 +07:00
|
|
|
|
2010-07-04 09:03:07 +07:00
|
|
|
_manager = kzalloc(sizeof(*_manager), GFP_KERNEL);
|
2018-01-02 20:24:55 +07:00
|
|
|
if (!_manager)
|
|
|
|
return -ENOMEM;
|
2010-04-01 19:44:57 +07:00
|
|
|
|
2017-11-21 08:37:52 +07:00
|
|
|
ttm_page_pool_init_locked(&_manager->wc_pool, GFP_HIGHUSER, "wc", 0);
|
2010-04-01 19:44:57 +07:00
|
|
|
|
2017-11-21 08:37:52 +07:00
|
|
|
ttm_page_pool_init_locked(&_manager->uc_pool, GFP_HIGHUSER, "uc", 0);
|
2010-04-01 19:44:57 +07:00
|
|
|
|
2010-07-04 09:03:07 +07:00
|
|
|
ttm_page_pool_init_locked(&_manager->wc_pool_dma32,
|
2017-11-21 08:37:52 +07:00
|
|
|
GFP_USER | GFP_DMA32, "wc dma", 0);
|
2010-04-01 19:44:57 +07:00
|
|
|
|
2010-07-04 09:03:07 +07:00
|
|
|
ttm_page_pool_init_locked(&_manager->uc_pool_dma32,
|
2017-11-21 08:37:52 +07:00
|
|
|
GFP_USER | GFP_DMA32, "uc dma", 0);
|
2010-04-01 19:44:57 +07:00
|
|
|
|
2017-10-05 19:27:34 +07:00
|
|
|
ttm_page_pool_init_locked(&_manager->wc_pool_huge,
|
2018-04-25 22:32:10 +07:00
|
|
|
(GFP_TRANSHUGE_LIGHT | __GFP_NORETRY |
|
|
|
|
__GFP_KSWAPD_RECLAIM) &
|
|
|
|
~(__GFP_MOVABLE | __GFP_COMP),
|
2017-11-21 08:37:52 +07:00
|
|
|
"wc huge", order);
|
2017-10-05 19:27:34 +07:00
|
|
|
|
|
|
|
ttm_page_pool_init_locked(&_manager->uc_pool_huge,
|
2018-04-25 22:32:10 +07:00
|
|
|
(GFP_TRANSHUGE_LIGHT | __GFP_NORETRY |
|
|
|
|
__GFP_KSWAPD_RECLAIM) &
|
|
|
|
~(__GFP_MOVABLE | __GFP_COMP)
|
2017-11-21 08:37:52 +07:00
|
|
|
, "uc huge", order);
|
2017-10-05 19:27:34 +07:00
|
|
|
|
2010-07-04 09:03:07 +07:00
|
|
|
_manager->options.max_size = max_pages;
|
|
|
|
_manager->options.small = SMALL_ALLOCATION;
|
|
|
|
_manager->options.alloc_size = NUM_PAGES_TO_ALLOC;
|
|
|
|
|
|
|
|
ret = kobject_init_and_add(&_manager->kobj, &ttm_pool_kobj_type,
|
|
|
|
&glob->kobj, "pool");
|
2018-01-11 11:57:58 +07:00
|
|
|
if (unlikely(ret != 0))
|
|
|
|
goto error;
|
2010-04-01 19:44:57 +07:00
|
|
|
|
2018-01-11 11:57:58 +07:00
|
|
|
ret = ttm_pool_mm_shrink_init(_manager);
|
|
|
|
if (unlikely(ret != 0))
|
|
|
|
goto error;
|
2010-04-01 19:44:57 +07:00
|
|
|
return 0;
|
2018-01-11 11:57:58 +07:00
|
|
|
|
|
|
|
error:
|
|
|
|
kobject_put(&_manager->kobj);
|
|
|
|
_manager = NULL;
|
|
|
|
return ret;
|
2010-04-01 19:44:57 +07:00
|
|
|
}
|
|
|
|
|
2010-09-22 23:45:45 +07:00
|
|
|
void ttm_page_alloc_fini(void)
|
2010-04-01 19:44:57 +07:00
|
|
|
{
|
|
|
|
int i;
|
|
|
|
|
2012-03-17 11:43:50 +07:00
|
|
|
pr_info("Finalizing pool allocator\n");
|
2010-07-04 09:03:07 +07:00
|
|
|
ttm_pool_mm_shrink_fini(_manager);
|
2010-04-01 19:44:57 +07:00
|
|
|
|
drm/ttm: Avoid memory allocation from shrinker functions.
Andrew Morton wrote:
> On Wed, 12 Nov 2014 13:08:55 +0900 Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp> wrote:
>
> > Andrew Morton wrote:
> > > Poor ttm guys - this is a bit of a trap we set for them.
> >
> > Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid deadlock.")
> > changed to use sc->gfp_mask rather than GFP_KERNEL.
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *),
> > - GFP_KERNEL);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> >
> > But this bug is caused by sc->gfp_mask containing some flags which are not
> > in GFP_KERNEL, right? Then, I think
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp & GFP_KERNEL);
> >
> > would hide this bug.
> >
> > But I think we should use GFP_ATOMIC (or drop __GFP_WAIT flag)
>
> Well no - ttm_page_pool_free() should stop calling kmalloc altogether.
> Just do
>
> struct page *pages_to_free[16];
>
> and rework the code to free 16 pages at a time. Easy.
Well, ttm code wants to process 512 pages at a time for performance.
Memory footprint increased by 512 * sizeof(struct page *) buffer is
only 4096 bytes. What about using static buffer like below?
----------
>From d3cb5393c9c8099d6b37e769f78c31af1541fe8c Mon Sep 17 00:00:00 2001
From: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Date: Thu, 13 Nov 2014 22:21:54 +0900
Subject: [PATCH] drm/ttm: Avoid memory allocation from shrinker functions.
Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid
deadlock.") caused BUG_ON() due to sc->gfp_mask containing flags
which are not in GFP_KERNEL.
https://bugzilla.kernel.org/show_bug.cgi?id=87891
Changing from sc->gfp_mask to (sc->gfp_mask & GFP_KERNEL) would
avoid the BUG_ON(), but avoiding memory allocation from shrinker
function is better and reliable fix.
Shrinker function is already serialized by global lock, and
clean up function is called after shrinker function is unregistered.
Thus, we can use static buffer when called from shrinker function
and clean up function.
Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Cc: stable <stable@kernel.org> [2.6.35+]
Signed-off-by: Dave Airlie <airlied@redhat.com>
2014-11-13 20:43:23 +07:00
|
|
|
/* OK to use static buffer since global mutex is no longer used. */
|
2010-04-01 19:44:57 +07:00
|
|
|
for (i = 0; i < NUM_POOLS; ++i)
|
drm/ttm: Avoid memory allocation from shrinker functions.
Andrew Morton wrote:
> On Wed, 12 Nov 2014 13:08:55 +0900 Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp> wrote:
>
> > Andrew Morton wrote:
> > > Poor ttm guys - this is a bit of a trap we set for them.
> >
> > Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid deadlock.")
> > changed to use sc->gfp_mask rather than GFP_KERNEL.
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *),
> > - GFP_KERNEL);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> >
> > But this bug is caused by sc->gfp_mask containing some flags which are not
> > in GFP_KERNEL, right? Then, I think
> >
> > - pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp);
> > + pages_to_free = kmalloc(npages_to_free * sizeof(struct page *), gfp & GFP_KERNEL);
> >
> > would hide this bug.
> >
> > But I think we should use GFP_ATOMIC (or drop __GFP_WAIT flag)
>
> Well no - ttm_page_pool_free() should stop calling kmalloc altogether.
> Just do
>
> struct page *pages_to_free[16];
>
> and rework the code to free 16 pages at a time. Easy.
Well, ttm code wants to process 512 pages at a time for performance.
Memory footprint increased by 512 * sizeof(struct page *) buffer is
only 4096 bytes. What about using static buffer like below?
----------
>From d3cb5393c9c8099d6b37e769f78c31af1541fe8c Mon Sep 17 00:00:00 2001
From: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Date: Thu, 13 Nov 2014 22:21:54 +0900
Subject: [PATCH] drm/ttm: Avoid memory allocation from shrinker functions.
Commit a91576d7916f6cce ("drm/ttm: Pass GFP flags in order to avoid
deadlock.") caused BUG_ON() due to sc->gfp_mask containing flags
which are not in GFP_KERNEL.
https://bugzilla.kernel.org/show_bug.cgi?id=87891
Changing from sc->gfp_mask to (sc->gfp_mask & GFP_KERNEL) would
avoid the BUG_ON(), but avoiding memory allocation from shrinker
function is better and reliable fix.
Shrinker function is already serialized by global lock, and
clean up function is called after shrinker function is unregistered.
Thus, we can use static buffer when called from shrinker function
and clean up function.
Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Cc: stable <stable@kernel.org> [2.6.35+]
Signed-off-by: Dave Airlie <airlied@redhat.com>
2014-11-13 20:43:23 +07:00
|
|
|
ttm_page_pool_free(&_manager->pools[i], FREE_ALL_PAGES, true);
|
2010-04-01 19:45:03 +07:00
|
|
|
|
2010-07-04 09:03:07 +07:00
|
|
|
kobject_put(&_manager->kobj);
|
|
|
|
_manager = NULL;
|
2010-04-01 19:44:57 +07:00
|
|
|
}
|
2010-04-01 19:44:58 +07:00
|
|
|
|
2018-01-19 14:17:27 +07:00
|
|
|
static void
|
|
|
|
ttm_pool_unpopulate_helper(struct ttm_tt *ttm, unsigned mem_count_update)
|
|
|
|
{
|
2018-02-22 01:02:06 +07:00
|
|
|
struct ttm_mem_global *mem_glob = ttm->bdev->glob->mem_glob;
|
2018-01-19 14:17:27 +07:00
|
|
|
unsigned i;
|
|
|
|
|
|
|
|
if (mem_count_update == 0)
|
|
|
|
goto put_pages;
|
|
|
|
|
|
|
|
for (i = 0; i < mem_count_update; ++i) {
|
|
|
|
if (!ttm->pages[i])
|
|
|
|
continue;
|
|
|
|
|
2018-02-22 01:02:06 +07:00
|
|
|
ttm_mem_global_free_page(mem_glob, ttm->pages[i], PAGE_SIZE);
|
2018-01-19 14:17:27 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
put_pages:
|
|
|
|
ttm_put_pages(ttm->pages, ttm->num_pages, ttm->page_flags,
|
|
|
|
ttm->caching_state);
|
|
|
|
ttm->state = tt_unpopulated;
|
|
|
|
}
|
|
|
|
|
2017-12-21 16:42:50 +07:00
|
|
|
int ttm_pool_populate(struct ttm_tt *ttm, struct ttm_operation_ctx *ctx)
|
2011-11-03 10:59:28 +07:00
|
|
|
{
|
2018-02-22 01:02:06 +07:00
|
|
|
struct ttm_mem_global *mem_glob = ttm->bdev->glob->mem_glob;
|
2011-11-03 10:59:28 +07:00
|
|
|
unsigned i;
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
if (ttm->state != tt_unpopulated)
|
|
|
|
return 0;
|
|
|
|
|
2018-02-05 16:57:07 +07:00
|
|
|
if (ttm_check_under_lowerlimit(mem_glob, ttm->num_pages, ctx))
|
|
|
|
return -ENOMEM;
|
|
|
|
|
2017-09-19 20:20:42 +07:00
|
|
|
ret = ttm_get_pages(ttm->pages, ttm->num_pages, ttm->page_flags,
|
|
|
|
ttm->caching_state);
|
|
|
|
if (unlikely(ret != 0)) {
|
2018-01-19 14:17:27 +07:00
|
|
|
ttm_pool_unpopulate_helper(ttm, 0);
|
2017-09-19 20:20:42 +07:00
|
|
|
return ret;
|
|
|
|
}
|
2011-11-03 10:59:28 +07:00
|
|
|
|
2017-09-19 20:20:42 +07:00
|
|
|
for (i = 0; i < ttm->num_pages; ++i) {
|
2017-07-04 21:56:24 +07:00
|
|
|
ret = ttm_mem_global_alloc_page(mem_glob, ttm->pages[i],
|
2017-12-21 16:42:50 +07:00
|
|
|
PAGE_SIZE, ctx);
|
2011-11-03 10:59:28 +07:00
|
|
|
if (unlikely(ret != 0)) {
|
2018-01-19 14:17:27 +07:00
|
|
|
ttm_pool_unpopulate_helper(ttm, i);
|
2011-11-03 10:59:28 +07:00
|
|
|
return -ENOMEM;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
if (unlikely(ttm->page_flags & TTM_PAGE_FLAG_SWAPPED)) {
|
|
|
|
ret = ttm_tt_swapin(ttm);
|
|
|
|
if (unlikely(ret != 0)) {
|
|
|
|
ttm_pool_unpopulate(ttm);
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
ttm->state = tt_unbound;
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(ttm_pool_populate);
|
|
|
|
|
|
|
|
void ttm_pool_unpopulate(struct ttm_tt *ttm)
|
|
|
|
{
|
2018-01-19 14:17:27 +07:00
|
|
|
ttm_pool_unpopulate_helper(ttm, ttm->num_pages);
|
2011-11-03 10:59:28 +07:00
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(ttm_pool_unpopulate);
|
|
|
|
|
2017-12-21 16:42:50 +07:00
|
|
|
int ttm_populate_and_map_pages(struct device *dev, struct ttm_dma_tt *tt,
|
|
|
|
struct ttm_operation_ctx *ctx)
|
2017-08-18 21:04:57 +07:00
|
|
|
{
|
2017-09-20 19:07:02 +07:00
|
|
|
unsigned i, j;
|
2017-08-18 21:04:57 +07:00
|
|
|
int r;
|
|
|
|
|
2017-12-21 16:42:50 +07:00
|
|
|
r = ttm_pool_populate(&tt->ttm, ctx);
|
2017-08-18 21:04:57 +07:00
|
|
|
if (r)
|
|
|
|
return r;
|
|
|
|
|
2017-09-20 19:07:02 +07:00
|
|
|
for (i = 0; i < tt->ttm.num_pages; ++i) {
|
|
|
|
struct page *p = tt->ttm.pages[i];
|
|
|
|
size_t num_pages = 1;
|
|
|
|
|
|
|
|
for (j = i + 1; j < tt->ttm.num_pages; ++j) {
|
|
|
|
if (++p != tt->ttm.pages[j])
|
|
|
|
break;
|
|
|
|
|
|
|
|
++num_pages;
|
|
|
|
}
|
|
|
|
|
2017-08-18 21:04:57 +07:00
|
|
|
tt->dma_address[i] = dma_map_page(dev, tt->ttm.pages[i],
|
2017-09-20 19:07:02 +07:00
|
|
|
0, num_pages * PAGE_SIZE,
|
2017-08-18 21:04:57 +07:00
|
|
|
DMA_BIDIRECTIONAL);
|
|
|
|
if (dma_mapping_error(dev, tt->dma_address[i])) {
|
|
|
|
while (i--) {
|
|
|
|
dma_unmap_page(dev, tt->dma_address[i],
|
|
|
|
PAGE_SIZE, DMA_BIDIRECTIONAL);
|
|
|
|
tt->dma_address[i] = 0;
|
|
|
|
}
|
|
|
|
ttm_pool_unpopulate(&tt->ttm);
|
|
|
|
return -EFAULT;
|
|
|
|
}
|
2017-09-20 19:07:02 +07:00
|
|
|
|
|
|
|
for (j = 1; j < num_pages; ++j) {
|
|
|
|
tt->dma_address[i + 1] = tt->dma_address[i] + PAGE_SIZE;
|
|
|
|
++i;
|
|
|
|
}
|
2017-08-18 21:04:57 +07:00
|
|
|
}
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(ttm_populate_and_map_pages);
|
|
|
|
|
|
|
|
void ttm_unmap_and_unpopulate_pages(struct device *dev, struct ttm_dma_tt *tt)
|
|
|
|
{
|
2017-09-20 19:07:02 +07:00
|
|
|
unsigned i, j;
|
|
|
|
|
|
|
|
for (i = 0; i < tt->ttm.num_pages;) {
|
|
|
|
struct page *p = tt->ttm.pages[i];
|
|
|
|
size_t num_pages = 1;
|
|
|
|
|
|
|
|
if (!tt->dma_address[i] || !tt->ttm.pages[i]) {
|
|
|
|
++i;
|
|
|
|
continue;
|
2017-08-18 21:04:57 +07:00
|
|
|
}
|
2017-09-20 19:07:02 +07:00
|
|
|
|
|
|
|
for (j = i + 1; j < tt->ttm.num_pages; ++j) {
|
|
|
|
if (++p != tt->ttm.pages[j])
|
|
|
|
break;
|
|
|
|
|
|
|
|
++num_pages;
|
|
|
|
}
|
|
|
|
|
|
|
|
dma_unmap_page(dev, tt->dma_address[i], num_pages * PAGE_SIZE,
|
|
|
|
DMA_BIDIRECTIONAL);
|
|
|
|
|
|
|
|
i += num_pages;
|
2017-08-18 21:04:57 +07:00
|
|
|
}
|
|
|
|
ttm_pool_unpopulate(&tt->ttm);
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(ttm_unmap_and_unpopulate_pages);
|
|
|
|
|
2010-04-01 19:44:58 +07:00
|
|
|
int ttm_page_alloc_debugfs(struct seq_file *m, void *data)
|
|
|
|
{
|
|
|
|
struct ttm_page_pool *p;
|
|
|
|
unsigned i;
|
|
|
|
char *h[] = {"pool", "refills", "pages freed", "size"};
|
2010-07-04 09:03:07 +07:00
|
|
|
if (!_manager) {
|
2010-04-01 19:44:58 +07:00
|
|
|
seq_printf(m, "No pool allocator running.\n");
|
|
|
|
return 0;
|
|
|
|
}
|
2017-10-05 19:27:34 +07:00
|
|
|
seq_printf(m, "%7s %12s %13s %8s\n",
|
2010-04-01 19:44:58 +07:00
|
|
|
h[0], h[1], h[2], h[3]);
|
|
|
|
for (i = 0; i < NUM_POOLS; ++i) {
|
2010-07-04 09:03:07 +07:00
|
|
|
p = &_manager->pools[i];
|
2010-04-01 19:44:58 +07:00
|
|
|
|
2017-10-05 19:27:34 +07:00
|
|
|
seq_printf(m, "%7s %12ld %13ld %8d\n",
|
2010-04-01 19:44:58 +07:00
|
|
|
p->name, p->nrefills,
|
|
|
|
p->nfrees, p->npages);
|
|
|
|
}
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(ttm_page_alloc_debugfs);
|