All of lore.kernel.org
 help / color / mirror / Atom feed
From: Benjamin Marzinski <bmarzins@redhat.com>
To: Martin Wilck <martin.wilck@suse.com>
Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>,
	device-mapper development <dm-devel@lists.linux.dev>
Subject: Re: [PATCH 19/22] multipathd: make multipath devices manage their path check times
Date: Tue, 16 Jul 2024 14:26:17 -0400	[thread overview]
Message-ID: <Zpa7Sag0jXX0AE6Z@redhat.com> (raw)
In-Reply-To: <bf9ea641dd20ed03fc8ce68a5e1cd6d70a6fa513.camel@suse.com>

On Tue, Jul 16, 2024 at 06:36:35PM +0200, Martin Wilck wrote:
> On Mon, 2024-07-15 at 22:49 +0200, Martin Wilck wrote:
> > On Sat, 2024-07-13 at 02:05 -0400, Benjamin Marzinski wrote:

> > > +	next_idx = (start_secs + pp->tick) % adjust_int;
> > > +	if (next_idx > goal_idx && next_idx - goal_idx < pp-
> > > > checkint)
> > > +		pp->tick--;
> 
> This code raises compilation errors on some architectures, see
> https://github.com/openSUSE/multipath-tools/actions/runs/9942603306/job/27464440204
> 
> Martin

Thanks. Switching to your modulo idea for checking offset will avoid
this.

-Ben


  reply	other threads:[~2024-07-16 18:26 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-07-13  6:04 [PATCH 00/22] path checker refactor and misc fixes Benjamin Marzinski
2024-07-13  6:04 ` [PATCH 01/22] libmultipath: rename dm_map_present_by_wwid() and add outputs Benjamin Marzinski
2024-07-15 10:53   ` Martin Wilck
2024-07-15 15:51     ` Martin Wilck
2024-07-15 15:14   ` Martin Wilck
2024-07-16 14:22     ` Benjamin Marzinski
2024-07-13  6:04 ` [PATCH 02/22] multipathd: make cli_add_map() handle adding maps by WWID correctly Benjamin Marzinski
2024-07-13  6:04 ` [PATCH 03/22] multipathd: remove checker restart optimization Benjamin Marzinski
2024-07-13  6:04 ` [PATCH 04/22] multipathd: refactor path state getting code into a helper Benjamin Marzinski
2024-07-15 15:19   ` Martin Wilck
2024-07-15 16:07     ` Martin Wilck
2024-07-13  6:04 ` [PATCH 05/22] multipathd: handle uninitialized paths in new function Benjamin Marzinski
2024-07-15 15:34   ` Martin Wilck
2024-07-15 15:36     ` Martin Wilck
2024-07-16 14:23       ` Benjamin Marzinski
2024-07-13  6:04 ` [PATCH 06/22] multipathd: make check_path() static Benjamin Marzinski
2024-07-13  6:04 ` [PATCH 07/22] multipathd: remove redundant checks in handle_uninitialized_path() Benjamin Marzinski
2024-07-13  6:04 ` [PATCH 08/22] multipathd: check paths immediately after failing udev initialization Benjamin Marzinski
2024-07-15 15:39   ` Martin Wilck
2024-07-13  6:04 ` [PATCH 09/22] multipathd: set pp->tick = max_checkint in handle_uninitialized_path Benjamin Marzinski
2024-07-15 16:00   ` Martin Wilck
2024-07-16 14:52     ` Benjamin Marzinski
2024-07-16 15:31       ` Martin Wilck
2024-07-13  6:04 ` [PATCH 10/22] multipathd: return 0 from path_check() if that path wasn't checked Benjamin Marzinski
2024-07-15 16:05   ` Martin Wilck
2024-07-16 14:52     ` Benjamin Marzinski
2024-07-13  6:04 ` [PATCH 11/22] multipathd: reorder path state checks Benjamin Marzinski
2024-07-15 16:23   ` Martin Wilck
2024-07-13  6:04 ` [PATCH 12/22] multipathd: adjust when mpp is synced with the kernel Benjamin Marzinski
2024-07-15 16:23   ` Martin Wilck
2024-07-16 15:14     ` Benjamin Marzinski
2024-07-13  6:04 ` [PATCH 13/22] multipathd: resync map after setup_map in ev_remove_path Benjamin Marzinski
2024-07-15 16:32   ` Martin Wilck
2024-07-16 15:15     ` Benjamin Marzinski
2024-07-13  6:04 ` [PATCH 14/22] multipathd: resync map after setup_map in resize_map Benjamin Marzinski
2024-07-13  6:04 ` [PATCH 15/22] multipathd: always resync map in reload_and_sync_map Benjamin Marzinski
2024-07-15 16:40   ` Martin Wilck
2024-07-13  6:05 ` [PATCH 16/22] multipathd: correctly handle paths removed for a wwid change Benjamin Marzinski
2024-07-15 19:16   ` Martin Wilck
2024-07-16 15:16     ` Benjamin Marzinski
2024-07-13  6:05 ` [PATCH 17/22] multipathd: handle changed wwid when adding partial path Benjamin Marzinski
2024-07-15 19:40   ` Martin Wilck
2024-07-16 15:45     ` Benjamin Marzinski
2024-07-13  6:05 ` [PATCH 18/22] multipathd: don't read conf->checkint twice in check_path Benjamin Marzinski
2024-07-15 19:41   ` Martin Wilck
2024-07-13  6:05 ` [PATCH 19/22] multipathd: make multipath devices manage their path check times Benjamin Marzinski
2024-07-15 20:49   ` Martin Wilck
2024-07-16 16:36     ` Martin Wilck
2024-07-16 18:26       ` Benjamin Marzinski [this message]
2024-07-16 18:23     ` Benjamin Marzinski
2024-07-13  6:05 ` [PATCH 20/22] multipathd: factor out actual path checking loop from checkerloop Benjamin Marzinski
2024-07-15 20:54   ` Martin Wilck
2024-07-13  6:05 ` [PATCH 21/22] multipathd: check paths in order by mpp Benjamin Marzinski
2024-07-15 21:19   ` Martin Wilck
2024-07-16 20:00     ` Benjamin Marzinski
2024-07-13  6:05 ` [PATCH 22/22] multipathd: clean up the correct wwid in check_path_wwid_change Benjamin Marzinski
2024-07-15 21:22   ` Martin Wilck
2024-07-16 20:00     ` Benjamin Marzinski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Zpa7Sag0jXX0AE6Z@redhat.com \
    --to=bmarzins@redhat.com \
    --cc=christophe.varoqui@opensvc.com \
    --cc=dm-devel@lists.linux.dev \
    --cc=martin.wilck@suse.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.