linux-toradex.git/mm/zbud.c, branch v3.19-rc1

mm/zbud: init user ops only when it is needed

2014-12-13T20:42:51+00:00

When zbud is initialized through the zpool wrapper, pool->ops which
points to user-defined operations is always set regardless of whether it
is specified from the upper layer. This causes zbud_reclaim_page() to
iterate its loop for evicting pool pages out without any gain.

This patch sets the user-defined ops only when it is needed, so that
zbud_reclaim_page() can bail out the reclamation loop earlier if there
is no user-defined operations specified.

Signed-off-by: Heesub Shin 
Acked-by: Dan Streetman 
Cc: Seth Jennings 
Cc: Sunae Seo 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

Merge Linus' tree to be be to apply submitted patches to newer code than

2014-11-20T13:42:02+00:00

current trivial.git base

zbud, zswap: change module author email

2014-11-13T08:05:47+00:00

Old email no longer viable.

Signed-off-by: Seth Jennings 
Signed-off-by: Jiri Kosina

zbud: avoid accessing last unused freelist

2014-10-10T02:26:03+00:00

For now, there are NCHUNKS of 64 freelists in zbud_pool, the last
unbuddied[63] freelist linked with all zbud pages which have free chunks
of 63.  Calculating according to context of num_free_chunks(), our max
chunk number of unbuddied zbud page is 62, so none of zbud pages will be
added/removed in last freelist, but still we will try to find an unbuddied
zbud page in the last unused freelist, it is unneeded.

This patch redefines NCHUNKS to 63 as free chunk number in one zbud page,
hence we can decrease size of zpool and avoid accessing the last unused
freelist whenever failing to allocate zbud from freelist in zbud_alloc.

Signed-off-by: Chao Yu 
Cc: Seth Jennings 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

mm/zpool: use prefixed module loading

2014-08-29T23:28:16+00:00

To avoid potential format string expansion via module parameters, do not
use the zpool type directly in request_module() without a format string.
Additionally, to avoid arbitrary modules being loaded via zpool API
(e.g.  via the zswap_zpool_type module parameter) add a "zpool-" prefix
to the requested module, as well as module aliases for the existing
zpool types (zbud and zsmalloc).

Signed-off-by: Kees Cook 
Cc: Seth Jennings 
Cc: Minchan Kim 
Cc: Nitin Gupta 
Acked-by: Dan Streetman 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

mm/zpool: zbud/zsmalloc implement zpool

2014-08-07T01:01:23+00:00

Update zbud and zsmalloc to implement the zpool api.

[fengguang.wu@intel.com: make functions static]
Signed-off-by: Dan Streetman 
Tested-by: Seth Jennings 
Cc: Minchan Kim 
Cc: Nitin Gupta 
Cc: Weijie Yang 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

mm/zbud: change zbud_alloc size type to size_t

2014-08-07T01:01:23+00:00

Change the type of the zbud_alloc() size param from unsigned int to
size_t.

Technically, this should not make any difference, as the zbud
implementation already restricts the size to well within either type's
limits; but as zsmalloc (and kmalloc) use size_t, and zpool will use
size_t, this brings the size parameter type in line with zsmalloc/zpool.

Signed-off-by: Dan Streetman 
Acked-by: Seth Jennings 
Tested-by: Seth Jennings 
Cc: Weijie Yang 
Cc: Minchan Kim 
Cc: Nitin Gupta 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

mm/zbud.c: make size unsigned like unique callsite

2014-06-04T23:54:13+00:00

zbud_alloc is only called by zswap_frontswap_store with unsigned int len.
Change function parameter + update >= 0 check.

Signed-off-by: Fabian Frederick 
Acked-by: Seth Jennings 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

mm/zbud: fix some trivial typos in comments

2013-09-11T22:57:35+00:00

Signed-off-by: Jianguo Wu 
Cc: Seth Jennings 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

mm: zbud: fix condition check on allocation size

2013-07-31T21:41:03+00:00

zbud_alloc() incorrectly verifies the size of allocation limit.  It
should deny the allocation request greater than (PAGE_SIZE -
ZHDR_SIZE_ALIGNED - CHUNK_SIZE), not (PAGE_SIZE - ZHDR_SIZE_ALIGNED)
which has no remaining spaces for its buddy.  There is no point in
spending the entire zbud page storing only a single page, since we don't
have any benefits.

Signed-off-by: Heesub Shin 
Acked-by: Seth Jennings 
Cc: Bob Liu 
Cc: Dongjun Shin 
Cc: Sunae Seo 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds