Re: [PATCH v3 1/6] block: add disk sequence number

From: Christoph Hellwig
Date: Wed Jun 23 2021 - 07:49:10 EST


On Wed, Jun 23, 2021 at 12:58:53PM +0200, Matteo Croce wrote:
> +void inc_diskseq(struct gendisk *disk)
> +{
> + static atomic64_t diskseq;

Please don't hide file scope variables in functions.

Can you explain a little more why we need a global sequence count vs
a per-disk one here?