[PATCH] imported patch sync-no_cluster-with-cluster-flag.patch

[PATCH] imported patch sync-no_cluster-with-cluster-flag.patch

[Malahal Naineni]

diff -r 8305d08f6ecf -r 5e7598cf7714 drivers/scsi/scsi_lib.c
--- a/drivers/scsi/scsi_lib.c	Wed Aug 25 13:44:33 2010 -0700
+++ b/drivers/scsi/scsi_lib.c	Wed Aug 25 13:44:33 2010 -0700
@@ -1633,8 +1633,10 @@ struct request_queue *__scsi_alloc_queue
 	blk_queue_max_segment_size(q, dma_get_max_seg_size(dev));
 
 	/* New queue, no concurrency on queue_flags */
-	if (!shost->use_clustering)
+	if (!shost->use_clustering) {
 		queue_flag_clear_unlocked(QUEUE_FLAG_CLUSTER, q);
+		q->limits.no_cluster = 1;
+	}
 
 	/*
 	 * set a reasonable default alignment on word boundaries: the

Re: [PATCH] block: QUEUE_FLAG_CLUSTER and no_cluster are NOT in sync

[Malahal Naineni]

Here is the missing header:

no_cluster is used by device mapper (may be others) resulting in
multipath devices setting QUEUE_FLAG_CLUSTER flag even though the
underlying devices (e.g. open iscsi virtual adapter) don't support it.

Signed-off-by: Malahal Naineni (malahal@us.ibm.com)

Malahal Naineni [malahal@us.ibm.com] wrote:
> diff -r 8305d08f6ecf -r 5e7598cf7714 drivers/scsi/scsi_lib.c
> --- a/drivers/scsi/scsi_lib.c	Wed Aug 25 13:44:33 2010 -0700
> +++ b/drivers/scsi/scsi_lib.c	Wed Aug 25 13:44:33 2010 -0700
> @@ -1633,8 +1633,10 @@ struct request_queue *__scsi_alloc_queue
>  	blk_queue_max_segment_size(q, dma_get_max_seg_size(dev));
> 
>  	/* New queue, no concurrency on queue_flags */
> -	if (!shost->use_clustering)
> +	if (!shost->use_clustering) {
>  		queue_flag_clear_unlocked(QUEUE_FLAG_CLUSTER, q);
> +		q->limits.no_cluster = 1;
> +	}
> 
>  	/*
>  	 * set a reasonable default alignment on word boundaries: the
> 
> --
> dm-devel mailing list
> dm-devel@redhat.com
> https://www.redhat.com/mailman/listinfo/dm-devel

Re: block: QUEUE_FLAG_CLUSTER and no_cluster are NOT in sync

[Mike Snitzer]

Malahal,

This patch should go through the scsi tree.  Please update the patch (so
it includes the patch header) and repost to linux-scsi.

Thanks,
Mike

On Sun, Aug 29 2010 at  4:31pm -0400,
Malahal Naineni <malahal@us.ibm.com> wrote:

> Here is the missing header:
> 
> no_cluster is used by device mapper (may be others) resulting in
> multipath devices setting QUEUE_FLAG_CLUSTER flag even though the
> underlying devices (e.g. open iscsi virtual adapter) don't support it.
> 
> Signed-off-by: Malahal Naineni (malahal@us.ibm.com)
> 
> Malahal Naineni [malahal@us.ibm.com] wrote:
> > diff -r 8305d08f6ecf -r 5e7598cf7714 drivers/scsi/scsi_lib.c
> > --- a/drivers/scsi/scsi_lib.c	Wed Aug 25 13:44:33 2010 -0700
> > +++ b/drivers/scsi/scsi_lib.c	Wed Aug 25 13:44:33 2010 -0700
> > @@ -1633,8 +1633,10 @@ struct request_queue *__scsi_alloc_queue
> >  	blk_queue_max_segment_size(q, dma_get_max_seg_size(dev));
> > 
> >  	/* New queue, no concurrency on queue_flags */
> > -	if (!shost->use_clustering)
> > +	if (!shost->use_clustering) {
> >  		queue_flag_clear_unlocked(QUEUE_FLAG_CLUSTER, q);
> > +		q->limits.no_cluster = 1;
> > +	}
> > 
> >  	/*
> >  	 * set a reasonable default alignment on word boundaries: the
> > 
> > --
> > dm-devel mailing list
> > dm-devel@redhat.com
> > https://www.redhat.com/mailman/listinfo/dm-devel
> 
> --
> dm-devel mailing list
> dm-devel@redhat.com
> https://www.redhat.com/mailman/listinfo/dm-devel