BTW, why is the name of riscv32-unknown-elf-gcc different from others?
Sep 5 2020
Sep 4 2020
Why is sys/contrib/dev/ice/README a binary file?
Sep 3 2020
Sep 2 2020
Would it make more sense to create sys/x86/include/x86_ieeefp.h and include it in amd64/include/ieeefp.h and i386/include/ieeefp.h?
Sep 1 2020
Aug 25 2020
Aug 24 2020
Aug 23 2020
I wasn't sure about the name for the option so I haven't added a description yet, will do in the next version.
I also assumed this knob could be used for other tests that are only useful for "full CI builds" rather than checking for regressions.
I believe the default should be off until the excessive compile time is fixed. These tests are unlikely to catch any regressions but add about 20 minutes of compile time to *all* builds. If the default isn't off I might as well just keep our downstream change that disables them completely and not bother upstreaming it. Having the option to turn these tests on for people who care makes more sense to me.
Aug 21 2020
Aug 20 2020
Aug 19 2020
Aug 18 2020
Aug 17 2020
Aug 15 2020
I probably wasn't clear enough in my message, it's not running the tests that's slow, it's building them (even for x86). The problem is that this seems to happen towards the end of the libraries build so it's mostly serial even when using something like -j72
Aug 14 2020
I think marking these test as EXTRA_TESTS is a good idea, and I don't strongly object setting them off. However maybe we can also check why these tests are slow, and maybe only disabled on !x86 architectures. If we disabled these tests by default, I think it is still worth to run them at least daily. (This is the planned frequency of !x86 test jobs on ci.freebsd.org)
Aug 13 2020
Looks fine other than the Tn thing. I think in general we are not adding them and maybe removing them, but it seems orthogonal to the rest of the change.
But I actually did remove them (those .Tn). Should they be added back, you think? What is the rule of thumb? To keep old and/deprecated macros?
Aug 12 2020
Aug 11 2020
Should we also mention this in src/lib/geom/part/gpart.8?