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.
>
> [root@fedoravm tmp]# rpm -q make
> make-4.2.1-6.fc28.x86_64
> [root@fedoravm tmp]# pwd
> /root/tmp
> [root@fedoravm 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
> [root@fedoravm tmp]# cat Makefile
> STRING1 = $(wildcard $(CURDIR)/[a-z]*/)
> STRING2 = $(wildcard ./[a-z]*/)
> default:
> @echo STRING1="$(STRING1)"
> @echo STRING2="$(STRING2)"
> [root@fedoravm tmp]# make
> STRING1=/root/tmp/testdir/ /root/tmp/Makefile
> STRING2=./testdir/ ./Makefile
> [root@fedoravm 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 bug-make@xxxxxxx, 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 majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe fstests" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
