diff mbox series

[v2] multipathd: Set CPUWeight to 1000 and LimitRTPRIO to 10

Message ID 20240415234306.1984762-1-bmarzins@redhat.com (mailing list archive)
State Not Applicable, archived
Delegated to: christophe varoqui
Headers show
Series [v2] multipathd: Set CPUWeight to 1000 and LimitRTPRIO to 10 | expand

Commit Message

Benjamin Marzinski April 15, 2024, 11:43 p.m. UTC
If multipathd doesn't become a real time process, it was scheduled as a
normal process, without any priority increase. Bump up the CPUWeight so
that even as a normal process, it will still run with increased
priority.

If multipathd did become a real time process, it set itself to the
highest priority, which is excessive. A priority of 10 is plenty.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
---

Notes:
    v2: Lowered LimitRTPRIO as suggested by Martin Wilck.

 multipathd/multipathd.service.in | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Martin Wilck April 16, 2024, 9:59 a.m. UTC | #1
On Mon, 2024-04-15 at 19:43 -0400, Benjamin Marzinski wrote:
> If multipathd doesn't become a real time process, it was scheduled as
> a
> normal process, without any priority increase. Bump up the CPUWeight
> so
> that even as a normal process, it will still run with increased
> priority.
> 
> If multipathd did become a real time process, it set itself to the
> highest priority, which is excessive. A priority of 10 is plenty.
> 
> Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>

Reviewed-by: Martin Wilck <mwilck@suse.com>

> ---
> 
> Notes:
>     v2: Lowered LimitRTPRIO as suggested by Martin Wilck.
> 
>  multipathd/multipathd.service.in | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/multipathd/multipathd.service.in
> b/multipathd/multipathd.service.in
> index 18bb367e..a63ddd9a 100644
> --- a/multipathd/multipathd.service.in
> +++ b/multipathd/multipathd.service.in
> @@ -19,7 +19,8 @@ NotifyAccess=main
>  ExecStart=/sbin/multipathd -d -s
>  ExecReload=/sbin/multipathd reconfigure
>  TasksMax=infinity
> -LimitRTPRIO=infinity
> +LimitRTPRIO=10
> +CPUWeight=1000
>  
>  [Install]
>  WantedBy=sysinit.target
diff mbox series

Patch

diff --git a/multipathd/multipathd.service.in b/multipathd/multipathd.service.in
index 18bb367e..a63ddd9a 100644
--- a/multipathd/multipathd.service.in
+++ b/multipathd/multipathd.service.in
@@ -19,7 +19,8 @@  NotifyAccess=main
 ExecStart=/sbin/multipathd -d -s
 ExecReload=/sbin/multipathd reconfigure
 TasksMax=infinity
-LimitRTPRIO=infinity
+LimitRTPRIO=10
+CPUWeight=1000
 
 [Install]
 WantedBy=sysinit.target