Re: [PATCH v2 00/22] Yet Another path checker refactor

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Sep 13, 2024 at 11:30:23AM +0200, Martin Wilck wrote:
> On Thu, 2024-09-12 at 17:49 -0400, Benjamin Marzinski wrote:
> > This patchset cleans up some of the ugly code from my last refactor,
> > and
> > speeds up multipathd path checking, by refactoring how paths are
> > checked.
> > 
> 
> I haven't started reviewing this yet, but I'm seeing a CI problem with
> the sysfs test, only with clang 18 on Fedora 40:
> https://github.com/openSUSE/multipath-tools/actions/runs/10845899189


I'm not sure what this is and I've been entirely unable to reproduce it
using Fedora 40 and clang 18.1.8 (the current f40 version of clang). It
seems pretty obvious that I has nothing to do with my patchset. It
looks like it's related to the renaming open() issues. Given the

"%s() has remaining non-returned values."

and

"__wrap___open64_2: '%s' parameter still has values that haven't been
checked."

messages, I'm guessing something other than __open64_2() was called. But
like I said, I can't reproduce this.

-Ben

> 
> Martin





[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux