mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Cong Wang <amwang@redhat.com>
To: linux-kernel@vger.kernel.org
Cc: akpm@linux-foundation.org, Cong Wang <amwang@redhat.com>,
	Hugh Dickins <hughd@google.com>,
	Paul Gortmaker <paul.gortmaker@windriver.com>,
	David Vrabel <david.vrabel@citrix.com>,
	Jens Axboe <axboe@kernel.dk>, Minchan Kim <minchan.kim@gmail.com>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	Al Viro <viro@zeniv.linux.org.uk>,
	Andrea Arcangeli <aarcange@redhat.com>,
	Mel Gorman <mel@csn.ul.ie>, David Rientjes <rientjes@google.com>,
	Michel Lespinasse <walken@google.com>,
	Cesar Eduardo Barros <cesarb@cesarb.net>,
	Eric B Munson <emunson@mgebm.net>,
	Pekka Enberg <penberg@kernel.org>, Joe Perches <joe@perches.com>,
	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>,
	linux-mm@kvack.org
Subject: [PATCH 52/62] mm: remove the second argument of k[un]map_atomic()
Date: Sun, 27 Nov 2011 13:27:32 +0800	[thread overview]
Message-ID: <1322371662-26166-53-git-send-email-amwang@redhat.com> (raw)
In-Reply-To: <1322371662-26166-1-git-send-email-amwang@redhat.com>


Signed-off-by: Cong Wang <amwang@redhat.com>
---
 mm/bounce.c   |    4 ++--
 mm/filemap.c  |    8 ++++----
 mm/ksm.c      |   12 ++++++------
 mm/memory.c   |    4 ++--
 mm/shmem.c    |    4 ++--
 mm/swapfile.c |   30 +++++++++++++++---------------
 mm/vmalloc.c  |    8 ++++----
 7 files changed, 35 insertions(+), 35 deletions(-)

diff --git a/mm/bounce.c b/mm/bounce.c
index 4e9ae72..d1be02c 100644
--- a/mm/bounce.c
+++ b/mm/bounce.c
@@ -50,9 +50,9 @@ static void bounce_copy_vec(struct bio_vec *to, unsigned char *vfrom)
 	unsigned char *vto;
 
 	local_irq_save(flags);
-	vto = kmap_atomic(to->bv_page, KM_BOUNCE_READ);
+	vto = kmap_atomic(to->bv_page);
 	memcpy(vto + to->bv_offset, vfrom, to->bv_len);
-	kunmap_atomic(vto, KM_BOUNCE_READ);
+	kunmap_atomic(vto);
 	local_irq_restore(flags);
 }
 
diff --git a/mm/filemap.c b/mm/filemap.c
index c0018f2..1c5f8f7 100644
--- a/mm/filemap.c
+++ b/mm/filemap.c
@@ -1332,10 +1332,10 @@ int file_read_actor(read_descriptor_t *desc, struct page *page,
 	 * taking the kmap.
 	 */
 	if (!fault_in_pages_writeable(desc->arg.buf, size)) {
-		kaddr = kmap_atomic(page, KM_USER0);
+		kaddr = kmap_atomic(page);
 		left = __copy_to_user_inatomic(desc->arg.buf,
 						kaddr + offset, size);
-		kunmap_atomic(kaddr, KM_USER0);
+		kunmap_atomic(kaddr);
 		if (left == 0)
 			goto success;
 	}
@@ -2062,7 +2062,7 @@ size_t iov_iter_copy_from_user_atomic(struct page *page,
 	size_t copied;
 
 	BUG_ON(!in_atomic());
-	kaddr = kmap_atomic(page, KM_USER0);
+	kaddr = kmap_atomic(page);
 	if (likely(i->nr_segs == 1)) {
 		int left;
 		char __user *buf = i->iov->iov_base + i->iov_offset;
@@ -2072,7 +2072,7 @@ size_t iov_iter_copy_from_user_atomic(struct page *page,
 		copied = __iovec_copy_from_user_inatomic(kaddr + offset,
 						i->iov, i->iov_offset, bytes);
 	}
-	kunmap_atomic(kaddr, KM_USER0);
+	kunmap_atomic(kaddr);
 
 	return copied;
 }
diff --git a/mm/ksm.c b/mm/ksm.c
index 310544a..a6d3fb7 100644
--- a/mm/ksm.c
+++ b/mm/ksm.c
@@ -672,9 +672,9 @@ error:
 static u32 calc_checksum(struct page *page)
 {
 	u32 checksum;
-	void *addr = kmap_atomic(page, KM_USER0);
+	void *addr = kmap_atomic(page);
 	checksum = jhash2(addr, PAGE_SIZE / 4, 17);
-	kunmap_atomic(addr, KM_USER0);
+	kunmap_atomic(addr);
 	return checksum;
 }
 
@@ -683,11 +683,11 @@ static int memcmp_pages(struct page *page1, struct page *page2)
 	char *addr1, *addr2;
 	int ret;
 
-	addr1 = kmap_atomic(page1, KM_USER0);
-	addr2 = kmap_atomic(page2, KM_USER1);
+	addr1 = kmap_atomic(page1);
+	addr2 = kmap_atomic(page2);
 	ret = memcmp(addr1, addr2, PAGE_SIZE);
-	kunmap_atomic(addr2, KM_USER1);
-	kunmap_atomic(addr1, KM_USER0);
+	kunmap_atomic(addr2);
+	kunmap_atomic(addr1);
 	return ret;
 }
 
diff --git a/mm/memory.c b/mm/memory.c
index 829d437..07ace87 100644
--- a/mm/memory.c
+++ b/mm/memory.c
@@ -2428,7 +2428,7 @@ static inline void cow_user_page(struct page *dst, struct page *src, unsigned lo
 	 * fails, we just zero-fill it. Live with it.
 	 */
 	if (unlikely(!src)) {
-		void *kaddr = kmap_atomic(dst, KM_USER0);
+		void *kaddr = kmap_atomic(dst);
 		void __user *uaddr = (void __user *)(va & PAGE_MASK);
 
 		/*
@@ -2439,7 +2439,7 @@ static inline void cow_user_page(struct page *dst, struct page *src, unsigned lo
 		 */
 		if (__copy_from_user_inatomic(kaddr, uaddr, PAGE_SIZE))
 			clear_page(kaddr);
-		kunmap_atomic(kaddr, KM_USER0);
+		kunmap_atomic(kaddr);
 		flush_dcache_page(dst);
 	} else
 		copy_user_highpage(dst, src, va, vma);
diff --git a/mm/shmem.c b/mm/shmem.c
index d672250..5bc679d 100644
--- a/mm/shmem.c
+++ b/mm/shmem.c
@@ -1631,9 +1631,9 @@ static int shmem_symlink(struct inode *dir, struct dentry *dentry, const char *s
 		}
 		inode->i_mapping->a_ops = &shmem_aops;
 		inode->i_op = &shmem_symlink_inode_operations;
-		kaddr = kmap_atomic(page, KM_USER0);
+		kaddr = kmap_atomic(page);
 		memcpy(kaddr, symname, len);
-		kunmap_atomic(kaddr, KM_USER0);
+		kunmap_atomic(kaddr);
 		set_page_dirty(page);
 		unlock_page(page);
 		page_cache_release(page);
diff --git a/mm/swapfile.c b/mm/swapfile.c
index b1cd120..94d8a62 100644
--- a/mm/swapfile.c
+++ b/mm/swapfile.c
@@ -2426,9 +2426,9 @@ int add_swap_count_continuation(swp_entry_t entry, gfp_t gfp_mask)
 		if (!(count & COUNT_CONTINUED))
 			goto out;
 
-		map = kmap_atomic(list_page, KM_USER0) + offset;
+		map = kmap_atomic(list_page) + offset;
 		count = *map;
-		kunmap_atomic(map, KM_USER0);
+		kunmap_atomic(map);
 
 		/*
 		 * If this continuation count now has some space in it,
@@ -2471,7 +2471,7 @@ static bool swap_count_continued(struct swap_info_struct *si,
 
 	offset &= ~PAGE_MASK;
 	page = list_entry(head->lru.next, struct page, lru);
-	map = kmap_atomic(page, KM_USER0) + offset;
+	map = kmap_atomic(page) + offset;
 
 	if (count == SWAP_MAP_MAX)	/* initial increment from swap_map */
 		goto init_map;		/* jump over SWAP_CONT_MAX checks */
@@ -2481,26 +2481,26 @@ static bool swap_count_continued(struct swap_info_struct *si,
 		 * Think of how you add 1 to 999
 		 */
 		while (*map == (SWAP_CONT_MAX | COUNT_CONTINUED)) {
-			kunmap_atomic(map, KM_USER0);
+			kunmap_atomic(map);
 			page = list_entry(page->lru.next, struct page, lru);
 			BUG_ON(page == head);
-			map = kmap_atomic(page, KM_USER0) + offset;
+			map = kmap_atomic(page) + offset;
 		}
 		if (*map == SWAP_CONT_MAX) {
-			kunmap_atomic(map, KM_USER0);
+			kunmap_atomic(map);
 			page = list_entry(page->lru.next, struct page, lru);
 			if (page == head)
 				return false;	/* add count continuation */
-			map = kmap_atomic(page, KM_USER0) + offset;
+			map = kmap_atomic(page) + offset;
 init_map:		*map = 0;		/* we didn't zero the page */
 		}
 		*map += 1;
-		kunmap_atomic(map, KM_USER0);
+		kunmap_atomic(map);
 		page = list_entry(page->lru.prev, struct page, lru);
 		while (page != head) {
-			map = kmap_atomic(page, KM_USER0) + offset;
+			map = kmap_atomic(page) + offset;
 			*map = COUNT_CONTINUED;
-			kunmap_atomic(map, KM_USER0);
+			kunmap_atomic(map);
 			page = list_entry(page->lru.prev, struct page, lru);
 		}
 		return true;			/* incremented */
@@ -2511,22 +2511,22 @@ init_map:		*map = 0;		/* we didn't zero the page */
 		 */
 		BUG_ON(count != COUNT_CONTINUED);
 		while (*map == COUNT_CONTINUED) {
-			kunmap_atomic(map, KM_USER0);
+			kunmap_atomic(map);
 			page = list_entry(page->lru.next, struct page, lru);
 			BUG_ON(page == head);
-			map = kmap_atomic(page, KM_USER0) + offset;
+			map = kmap_atomic(page) + offset;
 		}
 		BUG_ON(*map == 0);
 		*map -= 1;
 		if (*map == 0)
 			count = 0;
-		kunmap_atomic(map, KM_USER0);
+		kunmap_atomic(map);
 		page = list_entry(page->lru.prev, struct page, lru);
 		while (page != head) {
-			map = kmap_atomic(page, KM_USER0) + offset;
+			map = kmap_atomic(page) + offset;
 			*map = SWAP_CONT_MAX | count;
 			count = COUNT_CONTINUED;
-			kunmap_atomic(map, KM_USER0);
+			kunmap_atomic(map);
 			page = list_entry(page->lru.prev, struct page, lru);
 		}
 		return count == COUNT_CONTINUED;
diff --git a/mm/vmalloc.c b/mm/vmalloc.c
index 3231bf3..1a39dca 100644
--- a/mm/vmalloc.c
+++ b/mm/vmalloc.c
@@ -1879,9 +1879,9 @@ static int aligned_vread(char *buf, char *addr, unsigned long count)
 			 * we can expect USER0 is not used (see vread/vwrite's
 			 * function description)
 			 */
-			void *map = kmap_atomic(p, KM_USER0);
+			void *map = kmap_atomic(p);
 			memcpy(buf, map + offset, length);
-			kunmap_atomic(map, KM_USER0);
+			kunmap_atomic(map);
 		} else
 			memset(buf, 0, length);
 
@@ -1918,9 +1918,9 @@ static int aligned_vwrite(char *buf, char *addr, unsigned long count)
 			 * we can expect USER0 is not used (see vread/vwrite's
 			 * function description)
 			 */
-			void *map = kmap_atomic(p, KM_USER0);
+			void *map = kmap_atomic(p);
 			memcpy(map + offset, buf, length);
-			kunmap_atomic(map, KM_USER0);
+			kunmap_atomic(map);
 		}
 		addr += length;
 		buf += length;
-- 
1.7.4.4


  parent reply	other threads:[~2011-11-27  5:38 UTC|newest]

Thread overview: 121+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-27  5:26 [V2 PATCH 00/62] highmem: remove the second argument of kmap_atomic/kunmap_atomic Cong Wang
2011-11-27  5:26 ` [PATCH 01/62] highmem: mark k[un]map_atomic() with two arguments as deprecated Cong Wang
2011-11-27 12:34   ` Cesar Eduardo Barros
2011-11-28  5:07     ` Cong Wang
2011-11-28  7:46   ` [UPDATED PATCH " Cong Wang
2011-11-27  5:26 ` [PATCH 02/62] include/linux/highmem.h: remove the second argument of k[un]map_atomic() Cong Wang
2011-11-27  5:26 ` [PATCH 03/62] arm: " Cong Wang
2011-12-10 21:46   ` Russell King - ARM Linux
2011-11-27  5:26 ` [PATCH 04/62] mips: " Cong Wang
2011-12-09 16:06   ` Ralf Baechle
2011-12-12  2:52     ` Cong Wang
2011-11-27  5:26 ` [PATCH 05/62] powerpc: " Cong Wang
2011-11-27  5:26 ` [PATCH 06/62] sh: " Cong Wang
2011-11-27  5:26 ` [PATCH 07/62] um: " Cong Wang
2011-11-27  5:26 ` [PATCH 08/62] x86: " Cong Wang
2011-11-28 12:35   ` Avi Kivity
2011-11-28 12:47   ` Herbert Xu
2011-11-27  5:26 ` [PATCH 09/62] crypto: " Cong Wang
2011-11-27  5:26 ` [PATCH 10/62] ata: " Cong Wang
2011-11-27 18:40   ` Jeff Garzik
2011-11-28 10:01   ` Sergei Shtylyov
2011-11-28 11:42     ` Cong Wang
2011-11-28 11:49       ` Cong Wang
2011-11-28 19:00         ` James Bottomley
2011-11-29  3:25           ` Cong Wang
2011-11-27  5:26 ` [PATCH 11/62] block: " Cong Wang
2011-11-27  5:26 ` [PATCH 12/62] crypto: " Cong Wang
2011-11-27  5:26 ` [PATCH 13/62] edac: " Cong Wang
2011-11-27  5:26 ` [PATCH 14/62] drm: " Cong Wang
2011-11-27  5:26 ` [PATCH 15/62] ide: " Cong Wang
2011-11-27  6:12   ` David Miller
2011-11-27  5:26 ` [PATCH 16/62] infiniband: " Cong Wang
2011-11-28  7:38   ` Roland Dreier
2011-11-28  7:45     ` Cong Wang
2011-11-27  5:26 ` [PATCH 17/62] md: " Cong Wang
2011-11-27  6:00   ` NeilBrown
2011-11-27 10:27     ` [dm-devel] " Milan Broz
2011-11-28  5:10       ` Cong Wang
2011-11-28  7:47     ` [UPDATED PATCH " Cong Wang
2011-11-27  5:26 ` [PATCH 18/62] media: " Cong Wang
2011-11-27 23:10   ` Andy Walls
2011-11-27  5:26 ` [PATCH 19/62] memstick: " Cong Wang
2011-11-27  5:27 ` [PATCH 20/62] mmc: " Cong Wang
2011-11-27 21:02   ` Guennadi Liakhovetski
2011-11-30 23:51   ` David Brown
2011-12-01 18:47   ` Chris Ball
2011-12-02  2:10     ` Cong Wang
2011-11-27  5:27 ` [PATCH 21/62] net: " Cong Wang
2011-11-27  6:12   ` David Miller
2011-11-27  8:07   ` Eric Dumazet
2011-11-28 18:06     ` Alexander Duyck
2011-11-28 18:26       ` Eric Dumazet
2011-11-28  7:39   ` Cong Wang
2011-11-28  7:48   ` [UPDATED PATCH " Cong Wang
2011-11-27  5:27 ` [PATCH 23/62] hv: " Cong Wang
2011-11-27  9:34   ` Greg KH
2011-11-27  5:27 ` [PATCH 24/62] pohmelfs: " Cong Wang
2011-11-27  9:34   ` Greg KH
2011-11-28  0:58     ` Evgeniy Polyakov
2011-11-27  5:27 ` [PATCH 25/62] rtl8192u: " Cong Wang
2011-11-27  9:34   ` Greg KH
2011-11-27  5:27 ` [PATCH 26/62] zram: " Cong Wang
2011-11-27  9:34   ` Greg KH
2011-11-27  5:27 ` [PATCH 27/62] target: " Cong Wang
2011-11-27  5:27 ` [PATCH 28/62] vhost: " Cong Wang
2011-11-27  5:27 ` [PATCH 29/62] fs: " Cong Wang
2011-11-28  2:36   ` Benjamin LaHaise
2011-11-27  5:27 ` [PATCH 30/62] btrfs: " Cong Wang
2011-11-27  5:27 ` [PATCH 31/62] ecryptfs: " Cong Wang
2011-11-27  5:27 ` [PATCH 32/62] afs: " Cong Wang
2011-11-27  5:27 ` [PATCH 33/62] exofs: " Cong Wang
2011-11-28 17:35   ` Boaz Harrosh
2011-11-27  5:27 ` [PATCH 34/62] ext2: " Cong Wang
2011-11-28 14:15   ` Jan Kara
2011-11-27  5:27 ` [PATCH 35/62] fuse: " Cong Wang
2011-11-27  5:27 ` [PATCH 36/62] gfs2: " Cong Wang
2011-11-27  5:27 ` [PATCH 37/62] jbd: " Cong Wang
2011-11-28 14:15   ` Jan Kara
2011-11-27  5:27 ` [PATCH 38/62] jbd2: " Cong Wang
2011-11-27  5:27 ` [PATCH 39/62] logfs: " Cong Wang
2011-11-27  5:27 ` [PATCH 40/62] minix: " Cong Wang
2011-11-27  5:27 ` [PATCH 41/62] nfs: " Cong Wang
2011-11-27  5:27 ` [PATCH 42/62] nilfs2: " Cong Wang
2011-11-28 16:14   ` Ryusuke Konishi
2011-11-27  5:27 ` [PATCH 43/62] ntfs: " Cong Wang
2011-11-27  5:27 ` [PATCH 44/62] ocfs2: " Cong Wang
2011-12-01 20:18   ` Joel Becker
2011-12-02  2:04     ` Cong Wang
2011-12-02  2:29       ` Andrew Morton
2011-12-02  2:38         ` Cong Wang
2011-11-27  5:27 ` [PATCH 45/62] reiserfs: " Cong Wang
2011-11-27  5:27 ` [PATCH 46/62] squashfs: " Cong Wang
2011-11-27  5:27 ` [PATCH 47/62] ubifs: " Cong Wang
2011-11-27  5:27 ` [PATCH 48/62] udf: " Cong Wang
2011-11-28 14:16   ` Jan Kara
2011-11-27  5:27 ` [PATCH 49/62] kdb: " Cong Wang
2011-11-27  5:27 ` [PATCH 50/62] power: " Cong Wang
2011-11-27 11:38   ` Rafael J. Wysocki
2011-11-27  5:27 ` [PATCH 51/62] lib: " Cong Wang
2011-11-27  5:27 ` Cong Wang [this message]
2011-11-27  5:27 ` [PATCH 53/62] net: " Cong Wang
2011-11-27  6:12   ` David Miller
2011-11-27  5:27 ` [PATCH 54/62] rds: " Cong Wang
2011-11-27  6:13   ` David Miller
2011-11-27  5:27 ` [PATCH 55/62] sunrpc: " Cong Wang
2011-11-27  5:27 ` [PATCH 56/62] tomoyo: " Cong Wang
2011-11-27  5:27 ` [PATCH 57/62] md: " Cong Wang
2011-11-28  7:50   ` [UPDATED PATCH 57/62] dm: " Cong Wang
2011-11-27  5:27 ` [PATCH 58/62] gma500: " Cong Wang
2011-11-28 11:33   ` Alan Cox
2011-11-27  5:27 ` [PATCH 59/62] zcache: " Cong Wang
2011-11-27  9:34   ` Greg KH
2011-11-27  5:27 ` [PATCH 60/62] drbd: " Cong Wang
2011-11-27  5:27 ` [PATCH 61/62] highmem: kill all __kmap_atomic() Cong Wang
2011-12-09 16:07   ` Ralf Baechle
2011-11-27  5:27 ` [PATCH 62/62] feature-removal-schedule.txt: add the deprecated form of kmap_atomic() Cong Wang
2011-11-27 22:21 ` [V2 PATCH 00/62] highmem: remove the second argument of kmap_atomic/kunmap_atomic Benjamin Herrenschmidt
2011-11-28  4:34   ` Benjamin Herrenschmidt
2011-11-28  5:34     ` Cong Wang
2011-11-28  7:42   ` Cong Wang
2011-11-29  3:36 ` Cong Wang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1322371662-26166-53-git-send-email-amwang@redhat.com \
    --to=amwang@redhat.com \
    --cc=aarcange@redhat.com \
    --cc=akpm@linux-foundation.org \
    --cc=axboe@kernel.dk \
    --cc=cesarb@cesarb.net \
    --cc=david.vrabel@citrix.com \
    --cc=emunson@mgebm.net \
    --cc=hughd@google.com \
    --cc=joe@perches.com \
    --cc=kamezawa.hiroyu@jp.fujitsu.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mel@csn.ul.ie \
    --cc=minchan.kim@gmail.com \
    --cc=paul.gortmaker@windriver.com \
    --cc=paulmck@linux.vnet.ibm.com \
    --cc=penberg@kernel.org \
    --cc=rientjes@google.com \
    --cc=viro@zeniv.linux.org.uk \
    --cc=walken@google.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®