* [PATCH] scsi: switch to scsi-mq by default
@ 2018-07-04 8:53 Johannes Thumshirn
2018-07-11 2:52 ` Martin K. Petersen
0 siblings, 1 reply; 5+ messages in thread
From: Johannes Thumshirn @ 2018-07-04 8:53 UTC (permalink / raw)
To: Martin K . Petersen
Cc: Hannes Reinecke, Bart Van Assche, Ming Lei,
Linux SCSI Mailinglist, Linux Kernel Mailinglist,
Johannes Thumshirn
It has been more than one year since we tried to change the default
from legacy to multi queue in SCSI with commit c279bd9e406 ("scsi:
default to scsi-mq"). But due to issues with suspend/resume and
performance problems it had been reverted again with commit
cbe7dfa26eee ("Revert "scsi: default to scsi-mq"").
In the meantime there have been a substantial amount of performance
improvements and suspend/resume got fixed as well, thus we can
re-enable scsi-mq without a significant performance penalty.
Signed-off-by: Johannes Thumshirn <jthumshirn@suse.de>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Reviewed-by: Ming Lei <ming.lei@redhat.com>
Acked-by: John Garry <john.garry@huawei.com>
---
Changes since RFC:
- Improve Changelog as Bart suggested
- Collect Acks and Reviews
---
drivers/scsi/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig
index 35c909bbf8ba..bd115bab162e 100644
--- a/drivers/scsi/Kconfig
+++ b/drivers/scsi/Kconfig
@@ -49,6 +49,7 @@ config SCSI_NETLINK
config SCSI_MQ_DEFAULT
bool "SCSI: use blk-mq I/O path by default"
+ default y
depends on SCSI
---help---
This option enables the new blk-mq based I/O path for SCSI
--
2.16.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] scsi: switch to scsi-mq by default
2018-07-04 8:53 [PATCH] scsi: switch to scsi-mq by default Johannes Thumshirn
@ 2018-07-11 2:52 ` Martin K. Petersen
2018-07-11 8:31 ` Adrian Hunter
0 siblings, 1 reply; 5+ messages in thread
From: Martin K. Petersen @ 2018-07-11 2:52 UTC (permalink / raw)
To: Johannes Thumshirn
Cc: Martin K . Petersen, Hannes Reinecke, Bart Van Assche, Ming Lei,
Linux SCSI Mailinglist, Linux Kernel Mailinglist
Johannes,
> It has been more than one year since we tried to change the default
> from legacy to multi queue in SCSI with commit c279bd9e406 ("scsi:
> default to scsi-mq"). But due to issues with suspend/resume and
> performance problems it had been reverted again with commit
> cbe7dfa26eee ("Revert "scsi: default to scsi-mq"").
>
> In the meantime there have been a substantial amount of performance
> improvements and suspend/resume got fixed as well, thus we can
> re-enable scsi-mq without a significant performance penalty.
Applied to 4.19/scsi-queue, thanks!
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] scsi: switch to scsi-mq by default
2018-07-11 2:52 ` Martin K. Petersen
@ 2018-07-11 8:31 ` Adrian Hunter
2018-07-11 16:35 ` Ming Lei
0 siblings, 1 reply; 5+ messages in thread
From: Adrian Hunter @ 2018-07-11 8:31 UTC (permalink / raw)
To: Martin K. Petersen, Johannes Thumshirn
Cc: Hannes Reinecke, Bart Van Assche, Ming Lei,
Linux SCSI Mailinglist, Linux Kernel Mailinglist
On 11/07/18 05:52, Martin K. Petersen wrote:
>
> Johannes,
>
>> It has been more than one year since we tried to change the default
>> from legacy to multi queue in SCSI with commit c279bd9e406 ("scsi:
>> default to scsi-mq"). But due to issues with suspend/resume and
>> performance problems it had been reverted again with commit
>> cbe7dfa26eee ("Revert "scsi: default to scsi-mq"").
>>
>> In the meantime there have been a substantial amount of performance
>> improvements and suspend/resume got fixed as well, thus we can
>> re-enable scsi-mq without a significant performance penalty.
>
> Applied to 4.19/scsi-queue, thanks!
>
But there is no runtime pm support, is there? I tried enabling runtime PM
for UFS SCSI devices with scsi-mq and it didn't work, which is a regression.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] scsi: switch to scsi-mq by default
2018-07-11 8:31 ` Adrian Hunter
@ 2018-07-11 16:35 ` Ming Lei
2018-07-12 6:17 ` Adrian Hunter
0 siblings, 1 reply; 5+ messages in thread
From: Ming Lei @ 2018-07-11 16:35 UTC (permalink / raw)
To: Adrian Hunter
Cc: Martin K. Petersen, Johannes Thumshirn, Hannes Reinecke,
Bart Van Assche, Linux SCSI Mailinglist,
Linux Kernel Mailinglist
On Wed, Jul 11, 2018 at 4:31 PM, Adrian Hunter <adrian.hunter@intel.com> wrote:
> On 11/07/18 05:52, Martin K. Petersen wrote:
>>
>> Johannes,
>>
>>> It has been more than one year since we tried to change the default
>>> from legacy to multi queue in SCSI with commit c279bd9e406 ("scsi:
>>> default to scsi-mq"). But due to issues with suspend/resume and
>>> performance problems it had been reverted again with commit
>>> cbe7dfa26eee ("Revert "scsi: default to scsi-mq"").
>>>
>>> In the meantime there have been a substantial amount of performance
>>> improvements and suspend/resume got fixed as well, thus we can
>>> re-enable scsi-mq without a significant performance penalty.
>>
>> Applied to 4.19/scsi-queue, thanks!
>>
>
> But there is no runtime pm support, is there? I tried enabling runtime PM
> for UFS SCSI devices with scsi-mq and it didn't work, which is a regression.
I just posted one RFC patchset for supporting runtime PM on scsi_mq, and
looks it works in my test on usb-storage.
https://marc.info/?l=linux-block&m=153132657922233&w=2
thanks,
Ming Lei
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] scsi: switch to scsi-mq by default
2018-07-11 16:35 ` Ming Lei
@ 2018-07-12 6:17 ` Adrian Hunter
0 siblings, 0 replies; 5+ messages in thread
From: Adrian Hunter @ 2018-07-12 6:17 UTC (permalink / raw)
To: Ming Lei
Cc: Martin K. Petersen, Johannes Thumshirn, Hannes Reinecke,
Bart Van Assche, Linux SCSI Mailinglist,
Linux Kernel Mailinglist
On 11/07/18 19:35, Ming Lei wrote:
> On Wed, Jul 11, 2018 at 4:31 PM, Adrian Hunter <adrian.hunter@intel.com> wrote:
>> On 11/07/18 05:52, Martin K. Petersen wrote:
>>>
>>> Johannes,
>>>
>>>> It has been more than one year since we tried to change the default
>>>> from legacy to multi queue in SCSI with commit c279bd9e406 ("scsi:
>>>> default to scsi-mq"). But due to issues with suspend/resume and
>>>> performance problems it had been reverted again with commit
>>>> cbe7dfa26eee ("Revert "scsi: default to scsi-mq"").
>>>>
>>>> In the meantime there have been a substantial amount of performance
>>>> improvements and suspend/resume got fixed as well, thus we can
>>>> re-enable scsi-mq without a significant performance penalty.
>>>
>>> Applied to 4.19/scsi-queue, thanks!
>>>
>>
>> But there is no runtime pm support, is there? I tried enabling runtime PM
>> for UFS SCSI devices with scsi-mq and it didn't work, which is a regression.
>
> I just posted one RFC patchset for supporting runtime PM on scsi_mq, and
> looks it works in my test on usb-storage.
>
> https://marc.info/?l=linux-block&m=153132657922233&w=2
Cool, please cc me on the next revision of your patch set.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-07-12 6:19 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-04 8:53 [PATCH] scsi: switch to scsi-mq by default Johannes Thumshirn
2018-07-11 2:52 ` Martin K. Petersen
2018-07-11 8:31 ` Adrian Hunter
2018-07-11 16:35 ` Ming Lei
2018-07-12 6:17 ` Adrian Hunter
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).