[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4741ef3a-71e3-7118-55e1-c3545a88f967@suse.de>
Date: Fri, 16 Jul 2021 07:50:11 +0200
From: Hannes Reinecke <hare@...e.de>
To: Luis Chamberlain <mcgrof@...nel.org>, axboe@...nel.dk
Cc: bvanassche@....org, ming.lei@...hat.com, hch@...radead.org,
jack@...e.cz, osandov@...com, linux-block@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [RFC 1/6] genhd: update docs for GENHD_FL_UP
On 7/15/21 10:23 PM, Luis Chamberlain wrote:
> The GENHD_FL_UP is used internally so we can know when we can add and
> remove partitions, so just clarify that. Right now it has this 1980's
> description comparing it to network drivers "ifconfig up" idea, and
> that can easily lead to invalid uses.
>
> Signed-off-by: Luis Chamberlain <mcgrof@...nel.org>
> ---
> include/linux/genhd.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/genhd.h b/include/linux/genhd.h
> index 51f27b9b38b5..dc07a957c9e1 100644
> --- a/include/linux/genhd.h
> +++ b/include/linux/genhd.h
> @@ -61,7 +61,7 @@ struct partition_meta_info {
> * Affects responses to the ``CDROM_GET_CAPABILITY`` ioctl.
> *
> * ``GENHD_FL_UP`` (0x0010): indicates that the block device is "up",
> - * with a similar meaning to network interfaces.
> + * and we can add / remove partitions.
> *
> * ``GENHD_FL_SUPPRESS_PARTITION_INFO`` (0x0020): don't include
> * partition information in ``/proc/partitions`` or in the output of
>
Reviewed-by: Hannes Reinecke <hare@...e.de>
Cheers,
Hannes
--
Dr. Hannes Reinecke Kernel Storage Architect
hare@...e.de +49 911 74053 688
SUSE Software Solutions GmbH, Maxfeldstr. 5, 90409 Nürnberg
HRB 36809 (AG Nürnberg), Geschäftsführer: Felix Imendörffer
Powered by blists - more mailing lists