> +#ifndef CONFIG_ARCH_NO_SG_CHAIN > #define SCSI_INLINE_PROT_SG_CNT 1 > - > #define SCSI_INLINE_SG_CNT 2 > +#else > +#define SCSI_INLINE_PROT_SG_CNT 0 > +#define SCSI_INLINE_SG_CNT 0 > +#endif Please avoid the double negative and just use an ifdef. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>