bug-make
[Top][All Lists]
Advanced

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

Re: xfstests can't be installed by running make install


From: Zorro Lang
Subject: Re: xfstests can't be installed by running make install
Date: Mon, 16 Jul 2018 15:30:51 +0800
User-agent: Mutt/1.9.3 (2018-01-21)

On Sun, Jul 15, 2018 at 03:11:10PM +0800, Eryu Guan wrote:
> On Sun, Jul 15, 2018 at 03:43:20PM +1000, Dave Chinner wrote:
> > On Thu, Jul 12, 2018 at 12:39:21AM +0800, Eryu Guan wrote:
> > > On Wed, Jul 11, 2018 at 10:33:14PM +0800, Zorro Lang wrote:
> > > > Hi,
> > > > 
> > > > xfstests can't be installed now, if I run "make install", it fails as 
> > > > [1].
> > > > I don't what has been changed, and I'm not really so familar with these
> > > > building things ... Is there anyone knows what's wrong with it? I tested
> > > > on Fedora 28 with GNU Make 4.2.1.
> > > > 
> > > > Thanks,
> > > > Zorro
> > > > 
> > > > [1]
> > > > # git pull
> > > > Already up to date.
> > > > # make
> > > > # make install
> > > > Building include
> > > > Building lib
> > > > Building ltp
> > > > Building src
> > > > Building log-writes
> > > > Building perf
> > > > Building aio-dio-regress
> > > > Building m4
> > > > Building common                                
> > > > Building tests
> > > > /usr/bin/gmake --no-print-directory Q=@ -C include install
> > > > gmake[1]: Nothing to be done for 'install'.
> > > > /usr/bin/gmake --no-print-directory Q=@ -C lib install
> > > > gmake[1]: Nothing to be done for 'install'.
> > > > /usr/bin/gmake --no-print-directory Q=@ -C ltp install
> > > > ../install-sh -o root -g root -m 755 -d /var/lib/xfstests/ltp
> > > > /bin/sh ../libtool --quiet --mode=install ../install-sh -o root -g root 
> > > > -m 755 doio fsstress fsx growfiles iogen aio-stress 
> > > > /var/lib/xfstests/ltp
> > > > ../install-sh -o root -g root -m 755 rwtest.sh /var/lib/xfstests/ltp
> > > > /usr/bin/gmake --no-print-directory Q=@ -C src install                  
> > > >          
> > > > Building log-writes
> > > > Building perf
> > > > Building aio-dio-regress
> > > > /usr/bin/gmake -C log-writes install
> > > > ../../install-sh -o root -g root -m 755 -d 
> > > > /var/lib/xfstests/src/log-writes
> > > > ../../install-sh -o root -g root -m 755 replay-log 
> > > > /var/lib/xfstests/src/log-writes
> > > > /usr/bin/gmake -C perf install
> > > > ../../install-sh -o root -g root -m 755 -d /var/lib/xfstests/src/perf
> > > > ../../install-sh -o root -g root -m 755 ResultData.py 
> > > > fio-insert-and-compare.py FioResultDecoder.py FioCompare.py 
> > > > generate-schema.py fio-results.sql /var/lib/xfstests/src/perf
> > > > /usr/bin/gmake -C aio-dio-regress install
> > > > ../../install-sh -o root -g root -m 755 -d 
> > > > /var/lib/xfstests/src/aio-dio-regress
> > > > ../../install-sh -o root -g root -m 755 aio-dio-fcntl-race 
> > > > aio-dio-subblock-eof-read aio-dio-append-write-read-race 
> > > > aio-dio-invalidate-readahead aio-dio-hole-filling-race 
> > > > aio-free-ring-with-bogus-nr-pages aio-last-ref-held-by-io 
> > > > aio-dio-cycle-write aio-dio-extend-stat 
> > > > aio-io-setup-with-nonwritable-context-pointer 
> > > > aio-dio-invalidate-failure aiodio_sparse2 aiocp aio-dio-cow-race 
> > > > aio-dio-eof-race /var/lib/xfstests/src/aio-dio-regress
> > > > ../install-sh -o root -g root -m 755 -d /var/lib/xfstests/src
> > > > /bin/sh ../libtool --quiet --mode=install ../install-sh -o root -g root 
> > > > -m 755 dirstress fill fill2 getpagesize holes lstat64 nametest permname 
> > > > randholes runas truncfile usemem mmapcat append_reader append_writer 
> > > > dirperf metaperf devzero feature alloc fault fstest t_access_root 
> > > > godown resvtest writemod writev_on_pagefault makeextents itrash rename 
> > > > multi_open_unlink dmiperf unwritten_sync genhashnames t_holes 
> > > > t_mmap_writev t_truncate_cmtime dirhash_collide t_rename_overwrite 
> > > > holetest t_truncate_self t_mmap_dio af_unix t_mmap_stale_pmd 
> > > > t_mmap_cow_race t_mmap_fallocate fsync-err t_mmap_write_ro 
> > > > t_ext4_dax_journal_corruption t_ext4_dax_inline_corruption t_ofd_locks 
> > > > t_locks_execve xfsctl bstat t_mtab getdevicesize 
> > > > preallo_rw_pattern_reader preallo_rw_pattern_writer ftrunc trunc 
> > > > fs_perms testx looptest locktest unwritten_mmap bulkstat_unlink_test 
> > > > bulkstat_unlink_test_modified t_dir_offset t_futimens t_immutable 
> > > > stale_handle pwrite_mmap_blocked t_dir_offset2 seek_sanity_test 
> > > > seek_copy_test t_readdir_1 t_readdir_2 fsync-tester nsexec cloner 
> > > > renameat2 t_getcwd e4compact test-nextquota punch-alternating 
> > > > attr-list-by-handle-cursor-test listxattr dio-interleaved t_dir_type 
> > > > dio-invalidate-cache stat_test t_encrypted_d_revalidate 
> > > > attr_replace_test swapon mkswap fiemap-tester t_stripealign 
> > > > open_by_handle dbtest fssum /var/lib/xfstests/src
> > > > /bin/sh ../libtool --quiet --mode=install ../install-sh -o root -g root 
> > > > -m 755 dmerror fill2attr fill2fs fill2fs_check scaleread.sh 
> > > > /var/lib/xfstests/src
> > > > /bin/sh ../libtool --quiet --mode=install ../install-sh -o root -g root 
> > > > -m 644 dumpfile /var/lib/xfstests/src
> > > > /usr/bin/gmake --no-print-directory Q=@ -C m4 install
> > > > gmake[1]: Nothing to be done for 'install'.
> > > > /usr/bin/gmake --no-print-directory Q=@ -C common install
> > > > ../install-sh -o root -g root -m 755 -d /var/lib/xfstests/common
> > > > ../install-sh -o root -g root -m 644 * /var/lib/xfstests/common
> > > > /usr/bin/gmake --no-print-directory Q=@ -C tests install
> > > > /usr/bin/gmake --no-print-directory Q=@ -C 
> > > > /home/git/xfstests-dev/tests/ install
> > > > /usr/bin/gmake --no-print-directory Q=@ -C 
> > > > /home/git/xfstests-dev/tests/ install
> > > > /usr/bin/gmake --no-print-directory Q=@ -C 
> > > > /home/git/xfstests-dev/tests/ install
> > > > /usr/bin/gmake --no-print-directory Q=@ -C 
> > > > /home/git/xfstests-dev/tests/ install
> > > > /usr/bin/gmake --no-print-directory Q=@ -C 
> > > > /home/git/xfstests-dev/tests/ install
> > > > /usr/bin/gmake --no-print-directory Q=@ -C 
> > > > /home/git/xfstests-dev/tests/ install
> > > > ......
> > > > ......
> > > > (again and again and again ....)
> > > 
> > > This looks like a bug of make to me.
> > > 
> > > TESTS_SUBDIRS = $(sort $(dir $(wildcard $(CURDIR)/[a-z]*/)))
> > 
> > Is this a regression because of commit 103258a566b4 ("build: fix
> > install target using sudo")?
> 
> No, it fails even with that commit reverted.
> 
> > 
> > > The "wildcard" part is supposed to find all subdirs in tests dir, e.g.
> > > "tests/ext4 tests/generic tests/xfs ...", files shouldn't be included.
> > > So we get subdir list and go into each subdir and do install there.
> > > 
> > > But the same "wildcard" expression returns files too on fedora 28, e.g.
> > > "tests/Makefile tests/ext4 tests/generic tests/xfs ...", as a result,
> > 
> > Should *never* return tests/Makefile, because that does not match
> > the [a-z]* regex - it's a lowercase first character match, not
> > uppercase. And the *only* things that should be in tests/ is the
> > Makefile and all the test subdirs, so it shouldn't be matching the
> > wrong thing. i.e. how are we getting tests/ as a result in the
> > TESTS_SUBDIRS set?
> 
> That's why I think it's a bug of make. I did the following test on
> Fedora 28 with make-4.2.1-6.fc28.x86_64.
> 
> address@hidden tmp]# rpm -q make
> make-4.2.1-6.fc28.x86_64
> address@hidden tmp]# pwd
> /root/tmp
> address@hidden tmp]# ls -l
> total 4
> -rw-r--r--. 1 root root 206 Jul 15 14:58 Makefile
> drwxr-xr-x. 1 root root   0 Jul 15 14:59 testdir
> address@hidden tmp]# cat Makefile 
> STRING1 = $(wildcard $(CURDIR)/[a-z]*/)
> STRING2 = $(wildcard ./[a-z]*/)
> default:
>         @echo STRING1="$(STRING1)"
>         @echo STRING2="$(STRING2)"
> address@hidden tmp]# make
> STRING1=/root/tmp/testdir/ /root/tmp/Makefile
> STRING2=./testdir/ ./Makefile
> address@hidden tmp]#
> 
> So 'Makefile' is matched and returned (and the usa ofCURDIR doesn't
> matter), and that explains why we have "tests/" in TESTS_SUBDIRS.
> 
> The same test on RHEL7.4 works all as expected, 'Makefile' didn't match
> the pattern.
> 
> > 
> > > "tests/" is included in subdirs again, and we go to "tests/" to do
> > > install there again and enter a infinite loop.
> > 
> > Again, is this a regression? If so, we need to understand why using
> > $(CURDIR) is returning different results from $(PWD)/$(TESTS_DIR)/
> > 
> > > I can workaround the problem by appending a dot, i.e.
> > 
> > At this point, this seems more like hacking a bandaid over whatever
> > problem is occurring rather than trying to solve/understand why this
> > is now happening.
> 
> Agreed, I just expressed the fact that adding an ending dot could change
> the behavior, I didn't think it's a proper fix until we know exactly
> what causes it.

Thanks all your reviewing. I think it's time to CC address@hidden, to get
more help from make expert about what has been changed in latest make project.

Thanks,
Zorro

> 
> Thanks,
> Eryu
> --
> To unsubscribe from this list: send the line "unsubscribe fstests" in
> the body of a message to address@hidden
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



reply via email to

[Prev in Thread] Current Thread [Next in Thread]