lkml.org 
[lkml]   [2012]   [Aug]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH 1/7] zsmalloc: s/firstpage/page in new copy map funcs
Date
From: Seth Jennings <sjenning@linux.vnet.ibm.com>

firstpage already has precedent and meaning the first page
of a zspage. In the case of the copy mapping functions,
it is the first of a pair of pages needing to be mapped.

This patch just renames the firstpage argument to "page" to
avoid confusion.

Signed-off-by: Seth Jennings <sjenning@linux.vnet.ibm.com>
Signed-off-by: Minchan Kim <minchan@kernel.org>
---
drivers/staging/zsmalloc/zsmalloc-main.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/zsmalloc/zsmalloc-main.c b/drivers/staging/zsmalloc/zsmalloc-main.c
index 8b0bcb6..3c83c65 100644
--- a/drivers/staging/zsmalloc/zsmalloc-main.c
+++ b/drivers/staging/zsmalloc/zsmalloc-main.c
@@ -470,15 +470,15 @@ static struct page *find_get_zspage(struct size_class *class)
return page;
}

-static void zs_copy_map_object(char *buf, struct page *firstpage,
+static void zs_copy_map_object(char *buf, struct page *page,
int off, int size)
{
struct page *pages[2];
int sizes[2];
void *addr;

- pages[0] = firstpage;
- pages[1] = get_next_page(firstpage);
+ pages[0] = page;
+ pages[1] = get_next_page(page);
BUG_ON(!pages[1]);

sizes[0] = PAGE_SIZE - off;
@@ -493,15 +493,15 @@ static void zs_copy_map_object(char *buf, struct page *firstpage,
kunmap_atomic(addr);
}

-static void zs_copy_unmap_object(char *buf, struct page *firstpage,
+static void zs_copy_unmap_object(char *buf, struct page *page,
int off, int size)
{
struct page *pages[2];
int sizes[2];
void *addr;

- pages[0] = firstpage;
- pages[1] = get_next_page(firstpage);
+ pages[0] = page;
+ pages[1] = get_next_page(page);
BUG_ON(!pages[1]);

sizes[0] = PAGE_SIZE - off;
--
1.7.9.5


\
 
 \ /
  Last update: 2012-08-08 08:41    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site