From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-15.2 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3FE71C6379D for ; Thu, 26 Nov 2020 17:47:59 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id D083A207BC for ; Thu, 26 Nov 2020 17:47:58 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D083A207BC Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=suse.cz Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 403146B005D; Thu, 26 Nov 2020 12:47:58 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 38B606B0078; Thu, 26 Nov 2020 12:47:58 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 22DBA6B007B; Thu, 26 Nov 2020 12:47:58 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0196.hostedemail.com [216.40.44.196]) by kanga.kvack.org (Postfix) with ESMTP id 00F396B005D for ; Thu, 26 Nov 2020 12:47:57 -0500 (EST) Received: from smtpin20.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id BCFE28249980 for ; Thu, 26 Nov 2020 17:47:57 +0000 (UTC) X-FDA: 77527302594.20.oven92_03140b227381 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin20.hostedemail.com (Postfix) with ESMTP id A06C0180C07A3 for ; Thu, 26 Nov 2020 17:47:57 +0000 (UTC) X-HE-Tag: oven92_03140b227381 X-Filterd-Recvd-Size: 4524 Received: from mx2.suse.de (mx2.suse.de [195.135.220.15]) by imf33.hostedemail.com (Postfix) with ESMTP for ; Thu, 26 Nov 2020 17:47:56 +0000 (UTC) X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.221.27]) by mx2.suse.de (Postfix) with ESMTP id 14588ACA9; Thu, 26 Nov 2020 17:47:55 +0000 (UTC) Received: by quack2.suse.cz (Postfix, from userid 1000) id AC5821E10D0; Thu, 26 Nov 2020 18:47:54 +0100 (CET) Date: Thu, 26 Nov 2020 18:47:54 +0100 From: Jan Kara To: Christoph Hellwig Cc: Jens Axboe , Tejun Heo , Josef Bacik , Coly Li , Mike Snitzer , Greg Kroah-Hartman , Jan Kara , Johannes Thumshirn , dm-devel@redhat.com, Jan Kara , linux-block@vger.kernel.org, linux-bcache@vger.kernel.org, linux-mtd@lists.infradead.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org Subject: Re: [PATCH 39/44] block: pass a block_device to blk_alloc_devt Message-ID: <20201126174754.GY422@quack2.suse.cz> References: <20201126130422.92945-1-hch@lst.de> <20201126130422.92945-40-hch@lst.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20201126130422.92945-40-hch@lst.de> User-Agent: Mutt/1.10.1 (2018-07-13) X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Thu 26-11-20 14:04:17, Christoph Hellwig wrote: > Pass the block_device actually needed instead of the hd_struct. > > Signed-off-by: Christoph Hellwig Looks good. You can add: Reviewed-by: Jan Kara Honza > --- > block/blk.h | 2 +- > block/genhd.c | 14 +++++++------- > block/partitions/core.c | 2 +- > 3 files changed, 9 insertions(+), 9 deletions(-) > > diff --git a/block/blk.h b/block/blk.h > index d5bf8f3a078186..9657c6da7c770c 100644 > --- a/block/blk.h > +++ b/block/blk.h > @@ -350,7 +350,7 @@ static inline void blk_queue_free_zone_bitmaps(struct request_queue *q) {} > > struct block_device *disk_map_sector_rcu(struct gendisk *disk, sector_t sector); > > -int blk_alloc_devt(struct hd_struct *part, dev_t *devt); > +int blk_alloc_devt(struct block_device *part, dev_t *devt); > void blk_free_devt(dev_t devt); > char *disk_name(struct gendisk *hd, int partno, char *buf); > #define ADDPART_FLAG_NONE 0 > diff --git a/block/genhd.c b/block/genhd.c > index a85ffd7385718d..89cd0ba8e3b84a 100644 > --- a/block/genhd.c > +++ b/block/genhd.c > @@ -559,8 +559,8 @@ static int blk_mangle_minor(int minor) > } > > /** > - * blk_alloc_devt - allocate a dev_t for a partition > - * @part: partition to allocate dev_t for > + * blk_alloc_devt - allocate a dev_t for a block device > + * @bdev: block device to allocate dev_t for > * @devt: out parameter for resulting dev_t > * > * Allocate a dev_t for block device. > @@ -572,14 +572,14 @@ static int blk_mangle_minor(int minor) > * CONTEXT: > * Might sleep. > */ > -int blk_alloc_devt(struct hd_struct *part, dev_t *devt) > +int blk_alloc_devt(struct block_device *bdev, dev_t *devt) > { > - struct gendisk *disk = part_to_disk(part); > + struct gendisk *disk = bdev->bd_disk; > int idx; > > /* in consecutive minor range? */ > - if (part->bdev->bd_partno < disk->minors) { > - *devt = MKDEV(disk->major, disk->first_minor + part->bdev->bd_partno); > + if (bdev->bd_partno < disk->minors) { > + *devt = MKDEV(disk->major, disk->first_minor + bdev->bd_partno); > return 0; > } > > @@ -735,7 +735,7 @@ static void __device_add_disk(struct device *parent, struct gendisk *disk, > > disk->flags |= GENHD_FL_UP; > > - retval = blk_alloc_devt(disk->part0->bd_part, &devt); > + retval = blk_alloc_devt(disk->part0, &devt); > if (retval) { > WARN_ON(1); > return; > diff --git a/block/partitions/core.c b/block/partitions/core.c > index ecc3228a086956..4f823c4c733518 100644 > --- a/block/partitions/core.c > +++ b/block/partitions/core.c > @@ -392,7 +392,7 @@ static struct hd_struct *add_partition(struct gendisk *disk, int partno, > pdev->type = &part_type; > pdev->parent = ddev; > > - err = blk_alloc_devt(p, &devt); > + err = blk_alloc_devt(bdev, &devt); > if (err) > goto out_bdput; > pdev->devt = devt; > -- > 2.29.2 > -- Jan Kara SUSE Labs, CR