linux-kernel - Re: [PATCH 31/42] docs: scsi: convert scsi_mid_low

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 3 Mar 2020 11:20:22 +1100 (AEDT)
From:   Finn Thain <fthain@...egraphics.com.au>
To:     Mauro Carvalho Chehab <mchehab+huawei@...nel.org>
cc:     Linux Doc Mailing List <linux-doc@...r.kernel.org>,
        linux-kernel@...r.kernel.org, Jonathan Corbet <corbet@....net>
Subject: Re: [PATCH 31/42] docs: scsi: convert scsi_mid_low_api.txt to ReST

On Mon, 2 Mar 2020, Mauro Carvalho Chehab wrote:

> +			slave_alloc()
> +			slave_configure()
> +			    |
> +			slave_alloc()   ***
> +			slave_destroy() ***
> +
> +
> +    *** For scsi devices that the mid level tries to scan but do not
> +	respond, a slave_alloc(), slave_destroy() pair is called.
>  
>  If the LLD wants to adjust the default queue settings, it can invoke
>  scsi_change_queue_depth() in its slave_configure() routine.
>  
> -*** For scsi devices that the mid level tries to scan but do not
> -    respond, a slave_alloc(), slave_destroy() pair is called.
> -

Elsewhere in this series you have replaced asterisks with footnotes, as in 
[#]. Was that overlooked here?

BTW, certain patches in this series produce warnings from 'git am', that 
is, 'space before tab in indent'. Most of that whitespace appears to be 
intentional (for table formatting) or is pre-existing, but some seems to 
have been accidentally added...

$ git diff @~42.. | grep $'^[+].*\x20\x09'

Also, some of the patches in this series, like this one, were not Cc'd to 
the linux-scsi mailing list.