Re: [PATCH 1/4] selftests: Fix kselftest O=objdir build from cluttering top level objdir

[Shuah Khan]

On 5/14/20 8:27 PM, Randy Dunlap wrote:
> On 3/11/20 4:31 PM, Shuah Khan wrote:
> > On 3/11/20 4:58 PM, Kevin Hilman wrote:
> > > Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx> writes:
> > >
> > > > make kselftest-all O=objdir builds create generated objects in objdir.
> > > > This clutters the top level directory with kselftest objects. Fix it
> > > > to create sub-directory under objdir for kselftest objects.
> > > >
> > > > Signed-off-by: Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx>
> > >
> > > Only somewhat related to this patch, another problem that wasn't in your
> > > doci is that the current O= support doesn't support relative paths.
> >
> > Yes I am aware of it and it is in the document as something that will
> > be addressed later.
> >
> > "Note: Relative paths donât work - supporting relative paths breaks work-flows e.g:
> > powerpc. Explore fix. Compile work-flows. Not planning to support at the moment."

I attempted to fix it once and had to revert the patch. This relative
problem needs to be fixed and being tracked as a bug.

> > I am looking to address build and install issues first.

As I said above, there were higher priority test build and install
failures in cross-build and native build use-cases when I made this
call to prioritize fixing them first. I fixed all the known issues.

As of Linux 5.7-rc5 all of the know issues related test build and
install failures have been fixed.

With tha done, it is time to work on the relative path fix. I have
my reverted patch to start with and get it to work with the
workflows so it doesn't break them.

I also have the following patch you gave me as reference for fixing
the relative path problem:

https://lore.kernel.org/lkml/158351957799.3363.15269768530697526765.stgit@devnote2/

In any case, based on my previous experience fixing this problem,
I plan to take the fix in rc1 to allow enough time to fix any
problems that show up. It will be fixed for sure.

Hope this helps addresses any concerns you may have on whether or
not this problem will be fixed.

thanks,
-- Shuah