From: Olivier Hainque <hainque@adacore.com>
To: Alexandre Oliva <aoliva@redhat.com>
Cc: Olivier Hainque <hainque@adacore.com>,
GCC Patches <gcc-patches@gcc.gnu.org>,
Nicolas Roche <roche@adacore.com>
Subject: Re: fix libcc1 dependencies in toplevel Makefile
Date: Tue, 27 Jun 2017 16:32:00 -0000 [thread overview]
Message-ID: <EC1C224B-E051-4851-907E-25DA7E3E8533@adacore.com> (raw)
In-Reply-To: <F36A0A1A-EC61-46D5-B343-FA839C4DF287@adacore.com>
Hi Alex,
> On Jun 26, 2017, at 09:16 , Olivier Hainque <hainque@adacore.com> wrote:
>
>> I'd like to understand better what the concurrency problem is with the
>> current build machinery, before we proceed with this change. If you
>> manage to trigger the problem again, could you try to further analyze
>> build logs to check for e.g. concurrent activation of all-gcc in both
>> the top-level Makefile and the recursed-into-for-stage1 Makefile, or
>> somesuch? Something else worth considering is what the make targets
>> specified in the command line were.
>
> The problems were showing pretty rarely, only on certain hosts, in
> certain load conditions. We should still have the logs around and I'll
> look into this. They are regular logs, without -d. I can almost for sure
> fetch the exact "make" command line involved.
This was:
make -j 32 BOOT_LDFLAGS=-Wl,--stack=0x2000000 CC=gcc 'ADAFLAGS=-W -Wall -gnatpg -gnata -gnatws -gnatU -gnatyN' CXXFLAGS=-O2 BOOT_CFLAGS=-O2 CFLAGS=-O2 'LN_S=cp -p' 'BOOT_ADAFLAGS=-gnatpgn -gnatU' 'STAGE1_CFLAGS=-O2 -O0 -g' bootstrap
From the logs of discussions we tracked, the understanding
of the dependency issue was that we *had* (before the patch),
possibilities to have stage_current and maybe-all-gcc targets
built concurrently, via
> configure-target-libquadmath: stage_current
> all-target-libquadmath: configure-target-libquadmath
> maybe-all-target-libquadmath: all-target-libquadmath
> all-target: maybe-all-target-libquadmath
on the one hand,
> all-libcc1: maybe-all-gcc
> maybe-all-libcc1: all-libcc1
> all-host: maybe-all-libcc1
on the other hand.
Does that make sense ?
Thanks for your feedback!
(Note that I'll be away from tomorrow to Monday)
Olivier
next prev parent reply other threads:[~2017-06-27 16:32 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-13 12:58 Olivier Hainque
2017-06-14 11:39 ` Nathan Sidwell
2017-06-14 21:11 ` Olivier Hainque
2017-06-15 12:03 ` Nathan Sidwell
2017-06-15 12:29 ` Olivier Hainque
2017-06-22 12:13 ` Alexandre Oliva
2017-06-26 7:41 ` Olivier Hainque
2017-06-27 16:32 ` Olivier Hainque [this message]
2017-06-27 19:53 ` Alexandre Oliva
2017-07-03 21:05 ` Olivier Hainque
2018-06-03 19:13 ` Alexandre Oliva
2018-06-12 2:50 ` Alexandre Oliva
2018-06-12 8:57 ` Olivier Hainque
2018-06-12 15:31 ` Jeff Law
2018-06-26 5:39 ` Alexandre Oliva
2018-06-27 19:53 ` Olivier Hainque
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=EC1C224B-E051-4851-907E-25DA7E3E8533@adacore.com \
--to=hainque@adacore.com \
--cc=aoliva@redhat.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=roche@adacore.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).