[-next] block: add another struct gendisk to <linux/blkdev.h>

Commit Message

Randy Dunlap May 28, 2020, 5:21 p.m. UTC

From: Randy Dunlap <[e-mail]>

Add a forward declaration for struct gendisk when CONFIG_BLOCK is
not set/enabled to prevent multiple (30 - 50) build warnings.

In file included from ../kernel/sched/sched.h:39:0,
                 from ../kernel/sched/core.c:9:
../include/linux/blkdev.h:1895:41: warning: 'struct gendisk' declared inside parameter list will not be visible outside of this definition or declaration
 unsigned long disk_start_io_acct(struct gendisk *disk, unsigned int sectors,
                                         ^~~~~~~

Signed-off-by: Randy Dunlap <[e-mail]>
Cc: Jens Axboe <[e-mail]>
Cc: [e-mail]
---
 include/linux/blkdev.h |    1 +
 1 file changed, 1 insertion(+)

Comments

Jens Axboe May 28, 2020, 7:44 p.m. UTC | #1

On 5/28/20 11:21 AM, Randy Dunlap wrote:
> From: Randy Dunlap <[e-mail]>
> 
> Add a forward declaration for struct gendisk when CONFIG_BLOCK is
> not set/enabled to prevent multiple (30 - 50) build warnings.
> 
> In file included from ../kernel/sched/sched.h:39:0,
>                  from ../kernel/sched/core.c:9:
> ../include/linux/blkdev.h:1895:41: warning: 'struct gendisk' declared inside parameter list will not be visible outside of this definition or declaration
>  unsigned long disk_start_io_acct(struct gendisk *disk, unsigned int sectors,

Should be fixed already:

https://git.kernel.dk/cgit/linux-block/commit/?h=for-5.8/block&id=dc35ada4251f183137ee3a524543c9329d7a4fa2

Patch

--- linux-next-20200528.orig/include/linux/blkdev.h
+++ linux-next-20200528/include/linux/blkdev.h
@@ -1836,6 +1836,7 @@  static inline bool blk_req_can_dispatch_
 #else /* CONFIG_BLOCK */
 
 struct block_device;
+struct gendisk;
 
 /*
  * stubs for when the block layer is configured out