Re: [PATCH v2 10/13] mtd/ubi/block: add error handling support for add_disk()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Luis Chamberlain <mcgrof@xxxxxxxxxx>
Subject: Re: [PATCH v2 10/13] mtd/ubi/block: add error handling support for add_disk()
From: Christoph Hellwig <hch@xxxxxx>
Date: Wed, 3 Nov 2021 17:07:50 +0100
Cc: axboe@xxxxxxxxx, hch@xxxxxx, penguin-kernel@xxxxxxxxxxxxxxxxxxx, dan.j.williams@xxxxxxxxx, vishal.l.verma@xxxxxxxxx, dave.jiang@xxxxxxxxx, ira.weiny@xxxxxxxxx, richard@xxxxxx, miquel.raynal@xxxxxxxxxxx, vigneshr@xxxxxx, efremov@xxxxxxxxx, song@xxxxxxxxxx, martin.petersen@xxxxxxxxxx, hare@xxxxxxx, jack@xxxxxxx, ming.lei@xxxxxxxxxx, tj@xxxxxxxxxx, linux-mtd@xxxxxxxxxxxxxxxxxxx, linux-scsi@xxxxxxxxxxxxxxx, linux-raid@xxxxxxxxxxxxxxx, linux-block@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <20211103122157.1215783-11-mcgrof@kernel.org>
References: <20211103122157.1215783-1-mcgrof@kernel.org> <20211103122157.1215783-11-mcgrof@kernel.org>
User-agent: Mutt/1.5.17 (2007-11-01)

On Wed, Nov 03, 2021 at 05:21:54AM -0700, Luis Chamberlain wrote:
> We never checked for errors on add_disk() as this function
> returned void. Now that this is fixed, use the shiny new
> error handling.

Looks good,

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

References:
- [PATCH v2 00/13] block: add_disk() error handling stragglers
  - From: Luis Chamberlain
- [PATCH v2 10/13] mtd/ubi/block: add error handling support for add_disk()
  - From: Luis Chamberlain

Prev by Date: Re: [PATCH v2 09/13] block/sunvdc: add error handling support for add_disk()
Next by Date: Re: [PATCH v2 12/13] block: make __register_blkdev() return an error
Previous by thread: [PATCH v2 10/13] mtd/ubi/block: add error handling support for add_disk()
Next by thread: [PATCH v2 04/13] nvdimm/blk: avoid calling del_gendisk() on early failures
Index(es):
- Date
- Thread

[Index of Archives] [Linux RAID] [Linux SCSI] [Linux ATA RAID] [IDE] [Linux Wireless] [Linux Kernel] [ATH6KL] [Linux Bluetooth] [Linux Netdev] [Kernel Newbies] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Device Mapper]