From: Nick Piggin <npiggin@suse.de> To: Andrew Morton <akpm@osdl.org> Cc: Linux Kernel <linux-kernel@vger.kernel.org>, Linux Filesystems <linux-fsdevel@vger.kernel.org>, Nick Piggin <npiggin@suse.de>, Linux Memory Management <linux-mm@kvack.org> Subject: [patch 4/9] mm: generic_file_buffered_write cleanup Date: Mon, 29 Jan 2007 11:32:14 +0100 (CET) [thread overview] Message-ID: <20070129081942.23584.6374.sendpatchset@linux.site> (raw) In-Reply-To: <20070129081905.23584.97878.sendpatchset@linux.site> From: Andrew Morton <akpm@osdl.org> Clean up buffered write code. Rename some variables and fix some types. Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Nick Piggin <npiggin@suse.de> Index: linux-2.6/mm/filemap.c =================================================================== --- linux-2.6.orig/mm/filemap.c +++ linux-2.6/mm/filemap.c @@ -1854,16 +1854,15 @@ generic_file_buffered_write(struct kiocb size_t count, ssize_t written) { struct file *file = iocb->ki_filp; - struct address_space * mapping = file->f_mapping; + struct address_space *mapping = file->f_mapping; const struct address_space_operations *a_ops = mapping->a_ops; struct inode *inode = mapping->host; long status = 0; struct page *page; struct page *cached_page = NULL; - size_t bytes; struct pagevec lru_pvec; const struct iovec *cur_iov = iov; /* current iovec */ - size_t iov_base = 0; /* offset in the current iovec */ + size_t iov_offset = 0; /* offset in the current iovec */ char __user *buf; pagevec_init(&lru_pvec, 0); @@ -1874,31 +1873,33 @@ generic_file_buffered_write(struct kiocb if (likely(nr_segs == 1)) buf = iov->iov_base + written; else { - filemap_set_next_iovec(&cur_iov, &iov_base, written); - buf = cur_iov->iov_base + iov_base; + filemap_set_next_iovec(&cur_iov, &iov_offset, written); + buf = cur_iov->iov_base + iov_offset; } do { - unsigned long index; - unsigned long offset; - unsigned long maxlen; - size_t copied; + pgoff_t index; /* Pagecache index for current page */ + unsigned long offset; /* Offset into pagecache page */ + unsigned long maxlen; /* Bytes remaining in current iovec */ + size_t bytes; /* Bytes to write to page */ + size_t copied; /* Bytes copied from user */ - offset = (pos & (PAGE_CACHE_SIZE -1)); /* Within page */ + offset = (pos & (PAGE_CACHE_SIZE - 1)); index = pos >> PAGE_CACHE_SHIFT; bytes = PAGE_CACHE_SIZE - offset; if (bytes > count) bytes = count; + maxlen = cur_iov->iov_len - iov_offset; + if (maxlen > bytes) + maxlen = bytes; + /* * Bring in the user page that we will copy from _first_. * Otherwise there's a nasty deadlock on copying from the * same page as we're writing to, without it being marked * up-to-date. */ - maxlen = cur_iov->iov_len - iov_base; - if (maxlen > bytes) - maxlen = bytes; fault_in_pages_readable(buf, maxlen); page = __grab_cache_page(mapping,index,&cached_page,&lru_pvec); @@ -1929,7 +1930,7 @@ generic_file_buffered_write(struct kiocb buf, bytes); else copied = filemap_copy_from_user_iovec(page, offset, - cur_iov, iov_base, bytes); + cur_iov, iov_offset, bytes); flush_dcache_page(page); status = a_ops->commit_write(file, page, offset, offset+bytes); if (status == AOP_TRUNCATED_PAGE) { @@ -1947,12 +1948,12 @@ generic_file_buffered_write(struct kiocb buf += status; if (unlikely(nr_segs > 1)) { filemap_set_next_iovec(&cur_iov, - &iov_base, status); + &iov_offset, status); if (count) buf = cur_iov->iov_base + - iov_base; + iov_offset; } else { - iov_base += status; + iov_offset += status; } } }
WARNING: multiple messages have this Message-ID (diff)
From: Nick Piggin <npiggin@suse.de> From: Andrew Morton <akpm@osdl.org> To: Andrew Morton <akpm@osdl.org> Cc: Linux Kernel <linux-kernel@vger.kernel.org>, Linux Filesystems <linux-fsdevel@vger.kernel.org>, Nick Piggin <npiggin@suse.de>, Linux Memory Management <linux-mm@kvack.org> Subject: [patch 4/9] mm: generic_file_buffered_write cleanup Date: Mon, 29 Jan 2007 11:32:14 +0100 (CET) [thread overview] Message-ID: <20070129081942.23584.6374.sendpatchset@linux.site> (raw) In-Reply-To: <20070129081905.23584.97878.sendpatchset@linux.site> Clean up buffered write code. Rename some variables and fix some types. Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Nick Piggin <npiggin@suse.de> Index: linux-2.6/mm/filemap.c =================================================================== --- linux-2.6.orig/mm/filemap.c +++ linux-2.6/mm/filemap.c @@ -1854,16 +1854,15 @@ generic_file_buffered_write(struct kiocb size_t count, ssize_t written) { struct file *file = iocb->ki_filp; - struct address_space * mapping = file->f_mapping; + struct address_space *mapping = file->f_mapping; const struct address_space_operations *a_ops = mapping->a_ops; struct inode *inode = mapping->host; long status = 0; struct page *page; struct page *cached_page = NULL; - size_t bytes; struct pagevec lru_pvec; const struct iovec *cur_iov = iov; /* current iovec */ - size_t iov_base = 0; /* offset in the current iovec */ + size_t iov_offset = 0; /* offset in the current iovec */ char __user *buf; pagevec_init(&lru_pvec, 0); @@ -1874,31 +1873,33 @@ generic_file_buffered_write(struct kiocb if (likely(nr_segs == 1)) buf = iov->iov_base + written; else { - filemap_set_next_iovec(&cur_iov, &iov_base, written); - buf = cur_iov->iov_base + iov_base; + filemap_set_next_iovec(&cur_iov, &iov_offset, written); + buf = cur_iov->iov_base + iov_offset; } do { - unsigned long index; - unsigned long offset; - unsigned long maxlen; - size_t copied; + pgoff_t index; /* Pagecache index for current page */ + unsigned long offset; /* Offset into pagecache page */ + unsigned long maxlen; /* Bytes remaining in current iovec */ + size_t bytes; /* Bytes to write to page */ + size_t copied; /* Bytes copied from user */ - offset = (pos & (PAGE_CACHE_SIZE -1)); /* Within page */ + offset = (pos & (PAGE_CACHE_SIZE - 1)); index = pos >> PAGE_CACHE_SHIFT; bytes = PAGE_CACHE_SIZE - offset; if (bytes > count) bytes = count; + maxlen = cur_iov->iov_len - iov_offset; + if (maxlen > bytes) + maxlen = bytes; + /* * Bring in the user page that we will copy from _first_. * Otherwise there's a nasty deadlock on copying from the * same page as we're writing to, without it being marked * up-to-date. */ - maxlen = cur_iov->iov_len - iov_base; - if (maxlen > bytes) - maxlen = bytes; fault_in_pages_readable(buf, maxlen); page = __grab_cache_page(mapping,index,&cached_page,&lru_pvec); @@ -1929,7 +1930,7 @@ generic_file_buffered_write(struct kiocb buf, bytes); else copied = filemap_copy_from_user_iovec(page, offset, - cur_iov, iov_base, bytes); + cur_iov, iov_offset, bytes); flush_dcache_page(page); status = a_ops->commit_write(file, page, offset, offset+bytes); if (status == AOP_TRUNCATED_PAGE) { @@ -1947,12 +1948,12 @@ generic_file_buffered_write(struct kiocb buf += status; if (unlikely(nr_segs > 1)) { filemap_set_next_iovec(&cur_iov, - &iov_base, status); + &iov_offset, status); if (count) buf = cur_iov->iov_base + - iov_base; + iov_offset; } else { - iov_base += status; + iov_offset += status; } } } -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2007-01-29 10:32 UTC|newest] Thread overview: 61+ messages / expand[flat|nested] mbox.gz Atom feed top 2007-01-29 10:31 [patch 0/9] buffered write deadlock fix Nick Piggin 2007-01-29 10:31 ` Nick Piggin 2007-01-29 10:31 ` [patch 1/9] fs: libfs buffered write leak fix Nick Piggin 2007-01-29 10:31 ` Nick Piggin 2007-02-02 23:52 ` Andrew Morton 2007-02-02 23:52 ` Andrew Morton 2007-02-03 1:33 ` Nick Piggin 2007-02-03 1:33 ` Nick Piggin 2007-02-03 1:58 ` Andrew Morton 2007-02-03 1:58 ` Andrew Morton 2007-02-03 2:09 ` Nick Piggin 2007-02-03 2:09 ` Nick Piggin 2007-02-03 2:19 ` Andrew Morton 2007-02-03 2:19 ` Andrew Morton 2007-02-03 2:28 ` Nick Piggin 2007-02-03 2:28 ` Nick Piggin 2007-02-03 17:49 ` Jörn Engel 2007-02-03 17:49 ` Jörn Engel 2007-02-04 3:55 ` Nick Piggin 2007-02-04 3:55 ` Nick Piggin 2007-02-04 3:55 ` Nick Piggin 2007-01-29 10:31 ` [patch 2/9] mm: revert "generic_file_buffered_write(): handle zero length iovec segments" Nick Piggin 2007-01-29 10:31 ` Nick Piggin, Andrew Morton 2007-01-29 10:32 ` [patch 3/9] mm: revert "generic_file_buffered_write(): deadlock on vectored write" Nick Piggin 2007-01-29 10:32 ` Nick Piggin, Andrew Morton 2007-01-29 10:32 ` Nick Piggin [this message] 2007-01-29 10:32 ` [patch 4/9] mm: generic_file_buffered_write cleanup Nick Piggin, Andrew Morton 2007-01-29 10:32 ` [patch 5/9] mm: debug write deadlocks Nick Piggin 2007-01-29 10:32 ` Nick Piggin 2007-01-29 10:32 ` [patch 6/9] mm: be sure to trim blocks Nick Piggin 2007-01-29 10:32 ` Nick Piggin 2007-01-29 10:32 ` [patch 7/9] mm: cleanup pagecache insertion operations Nick Piggin 2007-01-29 10:32 ` Nick Piggin 2007-01-29 10:32 ` [patch 8/9] mm: generic_file_buffered_write iovec cleanup Nick Piggin 2007-01-29 10:32 ` Nick Piggin 2007-01-29 10:33 ` [patch 9/9] mm: fix pagecache write deadlocks Nick Piggin 2007-01-29 10:33 ` Nick Piggin 2007-01-29 11:11 ` Nick Piggin 2007-01-29 11:11 ` Nick Piggin 2007-02-02 23:53 ` Andrew Morton 2007-02-02 23:53 ` Andrew Morton 2007-02-03 1:38 ` Nick Piggin 2007-02-03 1:38 ` Nick Piggin 2007-01-30 20:55 ` [patch 0/9] buffered write deadlock fix Andrew Morton 2007-01-30 20:55 ` Andrew Morton 2007-01-30 23:21 ` Andrew Morton 2007-01-30 23:21 ` Andrew Morton 2007-01-31 1:31 ` Nick Piggin 2007-01-31 1:31 ` Nick Piggin 2007-01-31 0:32 ` Nick Piggin 2007-01-31 0:32 ` Nick Piggin 2007-02-02 23:52 ` Andrew Morton 2007-02-02 23:52 ` Andrew Morton 2007-02-03 1:22 ` Nick Piggin 2007-02-03 1:22 ` Nick Piggin 2007-02-03 6:43 ` Suparna Bhattacharya 2007-02-03 6:43 ` Suparna Bhattacharya 2007-02-03 15:31 ` Fengguang Wu 2007-02-03 15:31 ` Fengguang Wu 2007-02-03 15:31 ` Fengguang Wu 2007-02-04 8:49 Nick Piggin 2007-02-04 8:50 ` [patch 4/9] mm: generic_file_buffered_write cleanup Nick Piggin 2007-02-04 8:50 ` Nick Piggin, Andrew Morton
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=20070129081942.23584.6374.sendpatchset@linux.site \ --to=npiggin@suse.de \ --cc=akpm@osdl.org \ --cc=linux-fsdevel@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.