* [PATCH 1/1] cciss: set max scatter gather entries to 32 on P600
@ 2013-08-14 20:52 Mike Miller
2013-08-14 21:27 ` James Bottomley
0 siblings, 1 reply; 5+ messages in thread
From: Mike Miller @ 2013-08-14 20:52 UTC (permalink / raw)
To: Andrew Morton, Jens Axboe; +Cc: LKML-scsi, LKML, thenzl, bubrown, scameron
Patch 1/1
From: Mike Miller <mike.miller@hp.com>
At one time we used to set the maximum number of scatter gather elements on
all Smart Array controllers to 32. At some point in time the firmware began
to write the "appropriate" value for each controller into the config table.
The cciss driver would then read that and set h->maxsgentries.
h->maxsgentries = readl(&(h->cfgtable->MaxSGElements);
On the P600 that value is 544. Under some workloads a significant
performance reduction may result. This patch forces the P600 to use only 32
scatter gather elements. Other controllers are not affected.
Signed-off-by: Mike Miller <mike.miller@hp.com>
Signed-off-by: Dwight (Bud) Brown <bubrown@redhat.com>
Signed-off-by: Tomas Henzl <thenzl@redhat.com>
Acked-by: Stephen M. Cameron <steve.cameron@hp.com>
---
drivers/block/cciss.c | 7 +++++++
1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/drivers/block/cciss.c b/drivers/block/cciss.c
index 62b6c2c..d2d95ff 100644
--- a/drivers/block/cciss.c
+++ b/drivers/block/cciss.c
@@ -4258,6 +4258,13 @@ static void cciss_find_board_params(ctlr_info_t *h)
h->nr_cmds = h->max_commands - 4 - cciss_tape_cmds;
h->maxsgentries = readl(&(h->cfgtable->MaxSGElements));
/*
+ * The P600 may exhibit poor performnace under some workloads
+ * if we use the value in the configuration table. Limit this
+ * controller to MAXSGENTRIES (32) instead.
+ */
+ if (h->board_id == 0x3225103C)
+ h->maxsgentries = MAXSGENTRIES;
+ /*
* Limit in-command s/g elements to 32 save dma'able memory.
* Howvever spec says if 0, use 31
*/
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] cciss: set max scatter gather entries to 32 on P600
2013-08-14 20:52 [PATCH 1/1] cciss: set max scatter gather entries to 32 on P600 Mike Miller
@ 2013-08-14 21:27 ` James Bottomley
2013-08-14 21:41 ` Miller, Mike (OS Dev)
0 siblings, 1 reply; 5+ messages in thread
From: James Bottomley @ 2013-08-14 21:27 UTC (permalink / raw)
To: mike.miller, mikem
Cc: Andrew Morton, Jens Axboe, LKML-scsi, LKML, thenzl, bubrown, scameron
On Wed, 2013-08-14 at 15:52 -0500, Mike Miller wrote:
> Patch 1/1
>
> From: Mike Miller <mike.miller@hp.com>
>
> At one time we used to set the maximum number of scatter gather elements on
> all Smart Array controllers to 32. At some point in time the firmware began
> to write the "appropriate" value for each controller into the config table.
> The cciss driver would then read that and set h->maxsgentries.
>
> h->maxsgentries = readl(&(h->cfgtable->MaxSGElements);
>
> On the P600 that value is 544. Under some workloads a significant
> performance reduction may result. This patch forces the P600 to use only 32
> scatter gather elements. Other controllers are not affected.
>
> Signed-off-by: Mike Miller <mike.miller@hp.com>
> Signed-off-by: Dwight (Bud) Brown <bubrown@redhat.com>
> Signed-off-by: Tomas Henzl <thenzl@redhat.com>
> Acked-by: Stephen M. Cameron <steve.cameron@hp.com>
I don't quite understand the signoff chain on this patch. For a one
line patch, are you saying it has three authors?
James
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH 1/1] cciss: set max scatter gather entries to 32 on P600
2013-08-14 21:27 ` James Bottomley
@ 2013-08-14 21:41 ` Miller, Mike (OS Dev)
2013-08-15 11:21 ` Tomas Henzl
0 siblings, 1 reply; 5+ messages in thread
From: Miller, Mike (OS Dev) @ 2013-08-14 21:41 UTC (permalink / raw)
To: James Bottomley, mikem
Cc: Andrew Morton, Jens Axboe, LKML-scsi, LKML, thenzl, bubrown, scameron
-----Original Message-----
From: James Bottomley [mailto:James.Bottomley@HansenPartnership.com]
Sent: Wednesday, August 14, 2013 4:27 PM
To: Miller, Mike (OS Dev); mikem@thumper.usa.hp.com
Cc: Andrew Morton; Jens Axboe; LKML-scsi; LKML; thenzl@redhat.com; bubrown@redhat.com; scameron@beardog.cce.hp.com
Subject: Re: [PATCH 1/1] cciss: set max scatter gather entries to 32 on P600
On Wed, 2013-08-14 at 15:52 -0500, Mike Miller wrote:
> Patch 1/1
>
> From: Mike Miller <mike.miller@hp.com>
>
> At one time we used to set the maximum number of scatter gather
> elements on all Smart Array controllers to 32. At some point in time
> the firmware began to write the "appropriate" value for each controller into the config table.
> The cciss driver would then read that and set h->maxsgentries.
>
> h->maxsgentries = readl(&(h->cfgtable->MaxSGElements);
>
> On the P600 that value is 544. Under some workloads a significant
> performance reduction may result. This patch forces the P600 to use
> only 32 scatter gather elements. Other controllers are not affected.
>
> Signed-off-by: Mike Miller <mike.miller@hp.com>
> Signed-off-by: Dwight (Bud) Brown <bubrown@redhat.com>
> Signed-off-by: Tomas Henzl <thenzl@redhat.com>
> Acked-by: Stephen M. Cameron <steve.cameron@hp.com>
I don't quite understand the signoff chain on this patch. For a one line patch, are you saying it has three authors?
James
The patch origin is unknown. I got it from Tomas and Bud who think it may have originated from HP. I cleaned it up, compile tested it, and sent it on. I'll review my procedures for future patches.
-- mikem
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] cciss: set max scatter gather entries to 32 on P600
2013-08-14 21:41 ` Miller, Mike (OS Dev)
@ 2013-08-15 11:21 ` Tomas Henzl
2013-08-15 20:11 ` Bud Brown
0 siblings, 1 reply; 5+ messages in thread
From: Tomas Henzl @ 2013-08-15 11:21 UTC (permalink / raw)
To: Miller, Mike (OS Dev)
Cc: James Bottomley, mikem, Andrew Morton, Jens Axboe, LKML-scsi,
LKML, bubrown, scameron
On 08/14/2013 11:41 PM, Miller, Mike (OS Dev) wrote:
>
> -----Original Message-----
> From: James Bottomley [mailto:James.Bottomley@HansenPartnership.com]
> Sent: Wednesday, August 14, 2013 4:27 PM
> To: Miller, Mike (OS Dev); mikem@thumper.usa.hp.com
> Cc: Andrew Morton; Jens Axboe; LKML-scsi; LKML; thenzl@redhat.com; bubrown@redhat.com; scameron@beardog.cce.hp.com
> Subject: Re: [PATCH 1/1] cciss: set max scatter gather entries to 32 on P600
>
> On Wed, 2013-08-14 at 15:52 -0500, Mike Miller wrote:
>> Patch 1/1
>>
>> From: Mike Miller <mike.miller@hp.com>
>>
>> At one time we used to set the maximum number of scatter gather
>> elements on all Smart Array controllers to 32. At some point in time
>> the firmware began to write the "appropriate" value for each controller into the config table.
>> The cciss driver would then read that and set h->maxsgentries.
>>
>> h->maxsgentries = readl(&(h->cfgtable->MaxSGElements);
>>
>> On the P600 that value is 544. Under some workloads a significant
>> performance reduction may result. This patch forces the P600 to use
>> only 32 scatter gather elements. Other controllers are not affected.
>>
>> Signed-off-by: Mike Miller <mike.miller@hp.com>
>> Signed-off-by: Dwight (Bud) Brown <bubrown@redhat.com>
>> Signed-off-by: Tomas Henzl <thenzl@redhat.com>
>> Acked-by: Stephen M. Cameron <steve.cameron@hp.com>
> I don't quite understand the signoff chain on this patch. For a one line patch, are you saying it has three authors?
It look a bit funny, but from what I know everybody from that list^ was somehow involved in that issue, debugging..testing, etc,
so seeing the names there is not that bad. But I'm also not the direct author of the patch so feel free to remove my name from the list.
Tomas
>
> James
>
> The patch origin is unknown. I got it from Tomas and Bud who think it may have originated from HP. I cleaned it up, compile tested it, and sent it on. I'll review my procedures for future patches.
>
> -- mikem
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] cciss: set max scatter gather entries to 32 on P600
2013-08-15 11:21 ` Tomas Henzl
@ 2013-08-15 20:11 ` Bud Brown
0 siblings, 0 replies; 5+ messages in thread
From: Bud Brown @ 2013-08-15 20:11 UTC (permalink / raw)
To: Tomas Henzl
Cc: Mike Miller (OS Dev),
James Bottomley, mikem, Andrew Morton, Jens Axboe, LKML-scsi,
LKML, scameron
>
> >
> > James
> >
> > The patch origin is unknown. I got it from Tomas and Bud who think it may
> > have originated from HP. I cleaned it up, compile tested it, and sent it
> > on. I'll review my procedures for future patches.
> >
> > -- mikem
> >
Patch originated from me, although Tomas did the heavy lifting via kernel bisect
to localize the specific set of changes causing the performance regression.
Its modeled on the other P600'ism in the driver.
bud
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-08-15 20:13 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-08-14 20:52 [PATCH 1/1] cciss: set max scatter gather entries to 32 on P600 Mike Miller
2013-08-14 21:27 ` James Bottomley
2013-08-14 21:41 ` Miller, Mike (OS Dev)
2013-08-15 11:21 ` Tomas Henzl
2013-08-15 20:11 ` Bud Brown
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.