On Thu, Feb 13, 2020 at 03:04:17PM +0100, Hannes Reinecke wrote: > +extern int scsi_host_block(struct Scsi_Host *shost); > +extern int scsi_host_unblock(struct Scsi_Host *shost, int new_state); No need for the externs here. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>