From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: [PATCH] vhost/scsi: increase VHOST_SCSI_PREALLOC_PROT_SGLS to 2048 Date: Wed, 8 Aug 2018 23:42:22 +0300 Message-ID: <20180808233807-mutt-send-email-mst@kernel.org> References: <20180808192955.1115-1-gedwards@ddn.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20180808192955.1115-1-gedwards@ddn.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: Greg Edwards Cc: pbonzini@redhat.com, virtualization@lists.linux-foundation.org List-Id: virtualization@lists.linuxfoundation.org On Wed, Aug 08, 2018 at 01:29:55PM -0600, Greg Edwards wrote: > The current value of VHOST_SCSI_PREALLOC_PROT_SGLS is too small to > accommodate larger I/Os, e.g. 16-32 MiB, when the VIRTIO_SCSI_F_T10_PI > feature bit is negotiated and the backing store supports T10 PI. > > vhost-scsi rejects the command with errors like: > > [ 59.581317] vhost_scsi_calc_sgls: requested sgl_count: 1820 exceeds pre-allocated max_sgls: 512 > > Signed-off-by: Greg Edwards > --- > drivers/vhost/scsi.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/vhost/scsi.c b/drivers/vhost/scsi.c > index 17fcd3b2e686..8c32cf58d6fa 100644 > --- a/drivers/vhost/scsi.c > +++ b/drivers/vhost/scsi.c > @@ -56,7 +56,7 @@ > #define VHOST_SCSI_DEFAULT_TAGS 256 > #define VHOST_SCSI_PREALLOC_SGLS 2048 > #define VHOST_SCSI_PREALLOC_UPAGES 2048 > -#define VHOST_SCSI_PREALLOC_PROT_SGLS 512 > +#define VHOST_SCSI_PREALLOC_PROT_SGLS 2048 > > struct vhost_scsi_inflight { > /* Wait for the flush operation to finish */ I guess it's ok since PREALLOC_SGLS is already 2K ... or am I missing something. Paolo, any input on this? > -- > 2.17.1