[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Some serious issues with the new -O option
Re: Some serious issues with the new -O option
Tue, 30 Apr 2013 16:04:33 +0200
On 04/30/2013 03:37 PM, Paul Smith wrote:
> Just to be clear, you're saying that the testsuite runs as one long
> operation, updating one target, and the recipe invokes one test script,
No; the testsuite runs as a recursive make invocation (yes, this is sadly
truly needed in order to support all the features offered by the
Automake parallel testsuite harness --- sorry), but each test script (and
there can be hundreds of them, as is the case for GNU coreutils or GNU
automake itself) is run as a separate target, explicit for tests
which have no extension and pattern-based for tests that have an extension.
> I can see how that environment might be problematic for this new
> feature. It works much better for lots of smaller targets.
See above: we are speaking of hundreds of little targets here ...
> However, you could avoid this issue if you specify that the target to be
> built is a sub-make,
The problem here is exactly the fact that the output of the sub-make is
not displayed until its execution has ended.
> by prefixing it with a "+" operator for example.
BTW, the "+" operator is not portable soem other make implementations
still targeted by Automake (e.g., Solaris make).
> If this is true and you use -Otarget or -Ojob then make will not capture
> the output from that job.
> Of course this also has the unfortunate side-effect that running "make
> -n" will still try to run the test scripts.
This would be horrific of course ;-)
> On Tue, 2013-04-30 at 11:48 +0200, Stefano Lattarini wrote:
>> So please don't ever make that option the default; if you really
>> really want to, at least put in place some smart checks that only
>> enable '-O' when the output is *not* a tty, so that we can have the
>> best of both worlds (useful feedback for interactive runs, more
>> readable logs for batch runs).
> This is a possibility for sure.
> I have to say that my experience with parallel builds hasn't been as
> wonderful as others here.
OTOH, mine has been very good, and I almost alway use "make -j8" or higher
when building GNU software (or other good software the relies on GNU make,
like Git). The only real issues I've experienced so far have been when
I used the '-O' option ...
> I often get output which is corrupted, and
> not just by intermixing whole lines but also by having individual lines
> intermixed (that is the beginning of the line is one job's output, then
> the end of the line is another job's output, etc.)
Most of these issues are solved using silent-rules, as offered by Automake
or implemented ad-hoc by build systems of other software (Git, Linux).
I think the best ways to use parallel GNU make are:
- with silent rules when the output goes to a tty (it makes the output
visually scannable, and makes warnings and error messages form compilers
and most other tools stand out much more clearly).
- with verbose rules and possibly the -O option enabled when the output
goes to a log, to simplify post-build debugging and analysis (especially
useful with CI systems and the like).
> This kind of corruption is often completely unrecoverable and I simply
> re-run the build without parallelism enabled.
> I think it depends on how much output jobs emit and how many you are
> running at the same time.
Indeed; this corresponds to my experiences so far.
> It could also be that Windows is better about
> avoiding interrupted writes to the same device.
Not using windows, I cannot say anything here, sorry.
> Tim Murphy <address@hidden> writes:
>> What I mean is that:
>> ./make -Otarget
>> might be a good interactive default rather than -Omake.
> I always intended that and never suggested -Omake would be the default.
> I think -Omake is only really useful for completely automated,
> background builds. It wouldn't ever be something someone would use if
> they wanted to watch the build interactively.
>> I haven't tested to see if this is how the new feature works or not. I
>> don't think it's completely necessary to keep all output from one
>> submake together. so turning that off might make things more
>> interactive, Per-target syncing is a valid compromise.
> This is the default. If you use -Otarget or (-Ojob) then what is
> supposed to happen is that make uses the same sub-make detection
> algorithms used for jobserver, etc. and if it determines that a job it's
> going to run is a submake it does NOT collect its output.
This appears not to be the case, actually ...
> However, I have suspicions that this is not working properly. I have a
> make-based cross-compilation environment (building gcc + tools) I've
> been using for years, and I tried it with the latest make and -O and saw
> similar problems (sub-make output collected into one large log instead
> of per-target).
Glad to know you can reproduce this.
> Thinking about it right now I think I might know what the problem is,
> actually. I'll look at this later.
> Stefano Lattarini <address@hidden> writes:
>> I wasn't even aware of those differences; as of latest Git commit
>> 'moved-to-git-46-g19a69ba', I don't see them documented in either
>> the help screen, the manpage, the texinfo manual, nor the NEWS file.
> I don't see where that code comes from? There is no Git commit in the
> standard GNU make archive with a SHA g19a69ba.
The SHA is 19a69ba; the leading 'g' is added by the "git describe" command.
Sorry for not saying that explicitly.
> The current HEAD on master is:
> 19a69ba Support dynamic object loading on MS-Windows.
We are referring to the same commit then.
> At any rate, the new option and option arguments are documented in the
Then I must be messing something up locally. I'll see if I can solve this.
> and there is an overview including the option arguments in the
> man page. The NEWS file doesn't discuss the individual option
> arguments, only the option. Ditto the --help output.
Re: Some serious issues with the new -O option, Paul Smith, 2013/04/30
- Re: Some serious issues with the new -O option,
Stefano Lattarini <=