From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755174AbdBPNau (ORCPT ); Thu, 16 Feb 2017 08:30:50 -0500 Received: from mail-ua0-f195.google.com ([209.85.217.195]:33276 "EHLO mail-ua0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754065AbdBPNaq (ORCPT ); Thu, 16 Feb 2017 08:30:46 -0500 MIME-Version: 1.0 In-Reply-To: References: <1487245547-24384-1-git-send-email-tom.leiming@gmail.com> <1487245547-24384-3-git-send-email-tom.leiming@gmail.com> From: Ming Lei Date: Thu, 16 Feb 2017 21:30:45 +0800 Message-ID: Subject: Re: [PATCH 02/17] block: introduce bio_remove_last_page() To: Johannes Thumshirn Cc: Shaohua Li , Jens Axboe , Linux Kernel Mailing List , "open list:SOFTWARE RAID (Multiple Disks) SUPPORT" , linux-block , Christoph Hellwig , NeilBrown Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Feb 16, 2017 at 8:08 PM, Johannes Thumshirn wrote: > On 02/16/2017 12:45 PM, Ming Lei wrote: >> MD need this helper to remove the last added page, so introduce >> it. >> >> Signed-off-by: Ming Lei >> --- >> block/bio.c | 23 +++++++++++++++++++++++ >> include/linux/bio.h | 1 + >> 2 files changed, 24 insertions(+) >> >> diff --git a/block/bio.c b/block/bio.c >> index 5eec5e08417f..0ce7ffcd7939 100644 >> --- a/block/bio.c >> +++ b/block/bio.c >> @@ -837,6 +837,29 @@ int bio_add_pc_page(struct request_queue *q, struct bio *bio, struct page >> EXPORT_SYMBOL(bio_add_pc_page); >> >> /** >> + * bio_remove_last_page - remove the last added page >> + * @bio: destination bio >> + * >> + * Attempt to remove the last added page from the bio_vec maplist. >> + */ >> +void bio_remove_last_page(struct bio *bio) >> +{ >> + /* >> + * cloned bio must not modify vec list >> + */ >> + if (WARN_ON_ONCE(bio_flagged(bio, BIO_CLONED))) >> + return; >> + >> + if (bio->bi_vcnt > 0) { > > In patch 1 you introduce bio_segments_all() with the log message 'So > that we can replace the direct access to .bi_vcnt.' Here you introduce a > new direct access to it (plus the duplicated WARN_ON_ONCE()). > > Maybe use the helper directly here (I admit I haven't gone through the > whole series yet, so I can't see if the change is made later). Firstly MD does need one helper to remove the last added page, as you can see there are three such uses in patch3. Secondly both the two helpers will be changed once multipage bvec is supported, that means we have to change MD too after multipage bvec if just using bio_segments_all() to replace .bi_vcnt for removing the last added page. Thanks, Ming Lei