On Mon, Sep 11, 2023 at 06:38:35PM +0200, mwi...@suse.com wrote:
> From: Martin Wilck
>
> As the assignment of free aliases now relies on the bindings being
> properly sorted, add some unit tests to make sure the sorting algorithm
> works.
On second thought...
> ---
> tests/alias.c | 212
On Mon, Sep 11, 2023 at 06:38:35PM +0200, mwi...@suse.com wrote:
> From: Martin Wilck
>
> As the assignment of free aliases now relies on the bindings being
> properly sorted, add some unit tests to make sure the sorting algorithm
> works.
Missing a Signed-off-by, but otherwise
Reviewed-by:
On Mon, Sep 11, 2023 at 06:38:34PM +0200, mwi...@suse.com wrote:
> From: Martin Wilck
>
> The "unsorted" test fail now, and are removed. The algorithm is now
> better at finding "gaps".
>
Reviewed-by: Benjamin Marzinski
> Signed-off-by: Martin Wilck
> ---
> tests/alias.c | 88
On Mon, Sep 11, 2023 at 06:38:32PM +0200, mwi...@suse.com wrote:
> From: Martin Wilck
>
Reviewed-by: Benjamin Marzinski
> Signed-off-by: Martin Wilck
> ---
> tests/alias.c | 30 --
> 1 file changed, 12 insertions(+), 18 deletions(-)
>
> diff --git a/tests/alias.c
On Mon, Sep 11, 2023 at 06:38:27PM +0200, mwi...@suse.com wrote:
> From: Martin Wilck
>
> Rather than opening the bindings file every time we must retrieve
> a binding, keep the contents in memory and write the file only
> if additions have been made. This simplifies the code, and should speed
On Mon, Sep 11, 2023 at 06:38:31PM +0200, mwi...@suse.com wrote:
> From: Martin Wilck
>
> The current sort order of aliases is alphabetical, which is does not match
> the actual order of aliases, where "mpathaa" > "mpathz". Change the ordering
> as
> follows: first sort by string length, then
On Mon, Sep 11, 2023 at 06:38:16PM +0200, mwi...@suse.com wrote:
> From: Martin Wilck
>
Reviewed-by: Benjamin Marzinski
> Signed-off-by: Martin Wilck
> ---
> tests/alias.c | 441 ++
> 1 file changed, 441 insertions(+)
>
> diff --git
On Mon, Sep 11, 2023 at 06:38:13PM +0200, mwi...@suse.com wrote:
> From: Martin Wilck
>
> If the bindings file is changed in a way that multipathd can't handle
> (e.g. by swapping the aliases of two maps), multipathd must not try
> to re-use an alias that is already used by another map. Creating
On Mon, Sep 11, 2023 at 06:38:11PM +0200, mwi...@suse.com wrote:
> From: Martin Wilck
>
> Factor out a trivial helper function.
>
Reviewed-by: Benjamin Marzinski
> Signed-off-by: Martin Wilck
> ---
> libmultipath/alias.c | 32 +++-
> 1 file changed, 19
On Mon, Sep 11, 2023 at 06:38:33PM +0200, mwi...@suse.com wrote:
> From: Martin Wilck
>
> If we can assume that the bindings array is totally ordered for every
> prefix, which the previous patch guarantees, the search for a free ID can be
> simplified.
>
Reviewed-by: Benjamin Marzinski
>
On Mon, Sep 11, 2023 at 06:38:28PM +0200, mwi...@suse.com wrote:
> From: Martin Wilck
>
> The different implementation of get_user_friendly_alias() and its helpers
> necessitates changes in the unit tests. It would be nice if it didn't, but the
> unit tests are too closely bound to the
Commit ba30585936b0 ("dm: move setting md->type into dm_setup_md_queue")
has removed the only reference to dm_set_md_type(), so remove
dm_set_md_type() now.
Signed-off-by: Li Lingfeng
---
drivers/md/dm.c | 6 --
drivers/md/dm.h | 1 -
2 files changed, 7 deletions(-)
diff --git
On Fri 08-09-23 12:51:03, Zdenek Kabelac wrote:
> Dne 08. 09. 23 v 12:20 Jan Kara napsal(a):
> > On Fri 08-09-23 11:29:40, Zdenek Kabelac wrote:
> > > Dne 08. 09. 23 v 9:32 Jan Kara napsal(a):
> > > > On Thu 07-09-23 14:04:51, Mikulas Patocka wrote:
> > > > > On Thu, 7 Sep 2023, Christian Brauner
From: Martin Wilck
Since "libmultipath: keep bindings in memory", we don't re-read the
bindings file after every modification. Add a notification mechanism
that makes multipathd aware of changes to the bindings file. Because
multipathd itself will change the bindings file, it must compare
14 matches
Mail list logo