From: Ira Weiny <ira.weiny@intel.com>
To: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Cc: axboe@kernel.dk, ebiggers@kernel.org, damien.lemoal@wdc.com,
tytso@mit.edu, bvanassche@acm.org, hch@infradead.org,
johannes.thumshirn@wdc.com, dave.hansen@intel.com,
linux-f2fs-devel@lists.sourceforge.net,
linux-block@vger.kernel.org, clm@fb.com,
adilger.kernel@dilger.ca, dongli.zhang@oracle.com,
linux-fsdevel@vger.kernel.org, jaegeuk@kernel.org,
dsterba@suse.com, linux-ext4@vger.kernel.org
Subject: Re: [f2fs-dev] [RFC PATCH 2/8] brd: use memcpy_from_page() in copy_from_brd()
Date: Sun, 7 Feb 2021 20:35:46 -0800 [thread overview]
Message-ID: <20210208043545.GF5033@iweiny-DESK2.sc.intel.com> (raw)
In-Reply-To: <20210207190425.38107-3-chaitanya.kulkarni@wdc.com>
On Sun, Feb 07, 2021 at 11:04:19AM -0800, Chaitanya Kulkarni wrote:
> Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
> ---
> drivers/block/brd.c | 9 +++------
> 1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/block/brd.c b/drivers/block/brd.c
> index d41b7d489e9f..c1f6d768a1b3 100644
> --- a/drivers/block/brd.c
> +++ b/drivers/block/brd.c
> @@ -220,7 +220,6 @@ static void copy_from_brd(void *dst, struct brd_device *brd,
> sector_t sector, size_t n)
> {
> struct page *page;
> - void *src;
> unsigned int offset = (sector & (PAGE_SECTORS-1)) << SECTOR_SHIFT;
> size_t copy;
>
> @@ -236,11 +235,9 @@ static void copy_from_brd(void *dst, struct brd_device *brd,
> sector += copy >> SECTOR_SHIFT;
> copy = n - copy;
> page = brd_lookup_page(brd, sector);
> - if (page) {
> - src = kmap_atomic(page);
> - memcpy(dst, src, copy);
> - kunmap_atomic(src);
> - } else
> + if (page)
> + memcpy_from_page(dst, page, offset, copy);
Why 'offset'?
Ira
> + else
> memset(dst, 0, copy);
> }
> }
> --
> 2.22.1
>
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
next prev parent reply other threads:[~2021-02-08 4:36 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-02-07 19:04 [f2fs-dev] [RFC PATCH 0/8] use core page calls instead of kmaps Chaitanya Kulkarni
2021-02-07 19:04 ` [f2fs-dev] [RFC PATCH 1/8] brd: use memcpy_from_page() in copy_from_brd() Chaitanya Kulkarni
2021-02-07 19:04 ` [f2fs-dev] [RFC PATCH 2/8] " Chaitanya Kulkarni
2021-02-08 4:35 ` Ira Weiny [this message]
2021-02-08 5:47 ` Chaitanya Kulkarni
2021-02-07 19:04 ` [f2fs-dev] [RFC PATCH 3/8] null_blk: use memcpy_page() in copy_to_nullb() Chaitanya Kulkarni
2021-02-07 19:04 ` [f2fs-dev] [RFC PATCH 4/8] null_blk: use memcpy_page() in copy_from_nullb() Chaitanya Kulkarni
2021-02-07 19:04 ` [f2fs-dev] [RFC PATCH 5/8] ext4: use memcpy_from_page() in pagecache_read() Chaitanya Kulkarni
2021-02-07 19:04 ` [f2fs-dev] [RFC PATCH 6/8] ext4: use memcpy_to_page() in pagecache_write() Chaitanya Kulkarni
2021-03-25 14:22 ` Theodore Ts'o
2021-03-25 15:09 ` Ira Weiny
2021-03-25 15:10 ` Ira Weiny
2021-02-07 19:04 ` [f2fs-dev] [RFC PATCH 7/8] f2fs: use memcpy_from_page() in pagecache_read() Chaitanya Kulkarni
2021-02-07 19:04 ` [f2fs-dev] [RFC PATCH 8/8] f2fs: use memcpy_to_page() in pagecache_write() Chaitanya Kulkarni
2021-02-10 18:01 ` Eric Biggers
2021-02-10 19:41 ` Chaitanya Kulkarni
2021-02-07 19:10 ` [f2fs-dev] [RFC PATCH 0/8] use core page calls instead of kmaps Chaitanya Kulkarni
2021-02-08 4:42 ` Ira Weiny
2021-02-08 5:47 ` Chaitanya Kulkarni
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=20210208043545.GF5033@iweiny-DESK2.sc.intel.com \
--to=ira.weiny@intel.com \
--cc=adilger.kernel@dilger.ca \
--cc=axboe@kernel.dk \
--cc=bvanassche@acm.org \
--cc=chaitanya.kulkarni@wdc.com \
--cc=clm@fb.com \
--cc=damien.lemoal@wdc.com \
--cc=dave.hansen@intel.com \
--cc=dongli.zhang@oracle.com \
--cc=dsterba@suse.com \
--cc=ebiggers@kernel.org \
--cc=hch@infradead.org \
--cc=jaegeuk@kernel.org \
--cc=johannes.thumshirn@wdc.com \
--cc=linux-block@vger.kernel.org \
--cc=linux-ext4@vger.kernel.org \
--cc=linux-f2fs-devel@lists.sourceforge.net \
--cc=linux-fsdevel@vger.kernel.org \
--cc=tytso@mit.edu \
/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;
as well as URLs for NNTP newsgroup(s).