linux-kernel - Re: [PATCH v2 4/5] scsi: rename SCSI_MAX_{SG, SG_CHAIN}

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:	Mon, 4 Apr 2016 07:22:03 -0700
From:	Bart Van Assche <bart.vanassche@...disk.com>
To:	Ming Lin <mlin@...nel.org>, <linux-kernel@...r.kernel.org>,
	<linux-scsi@...r.kernel.org>
CC:	Christoph Hellwig <hch@....de>
Subject: Re: [PATCH v2 4/5] scsi: rename SCSI_MAX_{SG, SG_CHAIN}_SEGMENTS

On 03/22/16 15:03, Ming Lin wrote:
> From: Ming Lin <ming.l@....samsung.com>
>
> Rename SCSI_MAX_SG_SEGMENTS to SG_CHUNK_SIZE, which means the amount
> we fit into a single scatterlist chunk.
>
> Rename SCSI_MAX_SG_CHAIN_SEGMENTS to SG_MAX_SEGMENTS.
>
> Will move these 2 generic definitions to scatterlist.h later.
>
> Signed-off-by: Ming Lin <ming.l@....samsung.com>
> ---
>   drivers/ata/pata_icside.c           |  2 +-
>   drivers/infiniband/ulp/srp/ib_srp.c |  4 ++--
>   drivers/scsi/arm/cumana_2.c         |  2 +-
>   drivers/scsi/arm/eesox.c            |  2 +-
>   drivers/scsi/arm/powertec.c         |  2 +-
>   drivers/scsi/esas2r/esas2r_main.c   |  4 ++--
>   drivers/scsi/hisi_sas/hisi_sas.h    |  2 +-
>   drivers/scsi/mpt3sas/mpt3sas_base.c |  4 ++--
>   drivers/scsi/mpt3sas/mpt3sas_base.h |  2 +-
>   drivers/scsi/scsi_debug.c           |  2 +-
>   drivers/scsi/scsi_lib.c             | 34 +++++++++++++++++-----------------
>   drivers/usb/storage/scsiglue.c      |  2 +-
>   include/scsi/scsi.h                 |  8 ++++----
>   include/scsi/scsi_host.h            |  2 +-
>   14 files changed, 36 insertions(+), 36 deletions(-)

Hello Ming,

The ib_srp changes in this patch look fine to me.

Bart.