public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Jeff Law <law@redhat.com>
To: Richard Biener <richard.guenther@gmail.com>
Cc: Segher Boessenkool <segher@kernel.crashing.org>,
	andrew@codesourcery.com, Jakub Jelinek <jakub@redhat.com>,
	GCC Patches <gcc-patches@gcc.gnu.org>
Subject: Re: [PATCH] Delete powerpcspe
Date: Fri, 14 Dec 2018 16:02:00 -0000	[thread overview]
Message-ID: <f3130062-38f0-e27d-590a-6ae62c61698f@redhat.com> (raw)
In-Reply-To: <CAFiYyc0MwwMNuuNypg=zk8KBkFm=60Yit1=-owemKbJ6rBKynQ@mail.gmail.com>

On 12/14/18 2:52 AM, Richard Biener wrote:
> On Thu, Dec 13, 2018 at 5:49 PM Jeff Law <law@redhat.com> wrote:
>>
>> On 12/12/18 10:33 AM, Segher Boessenkool wrote:
>>> On Wed, Dec 12, 2018 at 11:36:29AM +0100, Richard Biener wrote:
>>>> On Tue, Dec 11, 2018 at 2:37 PM Jeff Law <law@redhat.com> wrote:
>>>>> One way to deal with these problems is to create a fake simulator that
>>>>> always returns success.  That's what my tester does for the embedded
>>>>> targets.  That allows us to do reliable compile-time tests as well as
>>>>> the various scan-whatever tests.
>>>>>
>>>>> It would be trivial to start sending those results to gcc-testresults.
>>>>
>>>> I think it would be more useful if the execute testing would be
>>>> reported as UNSUPPORTED rather than simply PASS w/o being
>>>> sure it does.
>>>
>>> Yes.
>> Yes, but I don't think we've got a reasonable way to do that in the
>> existing dejagnu framework.
>>
>>
>>>
>>>> But while posting to gcc-testresults is a sign of testing tracking
>>>> regressions (and progressions!) in bugzilla and caring for those
>>>> bugs is far more important...
>>>
>>> If results are posted to gcc-testresults then other people can get a
>>> feel whether the port is detoriating, and at what rate.  If no results
>>> are posted we just have to assume the worst.  Most people do not have
>>> the time (or setup) to test it for themselves.
>> Yup.  I wish I had the time to extract more of the data the tester is
>> gathering and produce this kind of info.
>>
>> I have not made it a priority to try and address all the issues I've
>> seen in the tester.  We have some ports that are incredibly flaky
>> (epiphany for example), and many that have a lot of failures, but are
>> stable in their set of failures.
>>
>> My goal to date has mostly been to identify regressions.  I'm not even
>> able to keep up with that.  For example s390/s390x have been failing for
>> about a week with their kernel builds.    sparc, i686, aarch64 are
>> consistently tripping over regressions.  ia64 hasn't worked since we put
>> in qsort consistency checking, etc etc.
> 
> Yeah :/
> 
> I wonder if we could set up auto-(simulator)-testing for all supported
> archs (and build testing for all supported configs) on the CF
> (with the required scripting in contrib/ so it's easy to replicate).  I'd
> simply test only released snapshots to keep the load reasonable
> and besides posting to gcc-testresults also post testresults
> differences to gcc-regression?
It's certainly possible.  Though I've found that managing this kind of
thing with Jenkins is far easier than rolling our own.  I'd be happy to
move an instance out into the CF.

> 
> That said, can we document how to simulator-test $target in
> a structural way somewhere?  Either my means of (a) script(s)
> in contrib/ or by simple documentation in a new gcc/testing.texi
> or on the wiki?
It should be possible. Sometimes it's just using the right
--target_board.    Other times there isn't one so you write your own
glue code :(  That glue code is part of dejagnu.



> 
> You at least seem to have some sort of scripting for some targets?
> Esp. having target boards and simulator configs would be nice
> (and pointers where to look for simulators).
Well, since I'm using a fake simulator no mapping is needed.  Though
I've got plumbing in to use the simulator from gdb in place.  The plan
was to turn that on once things using the fake simulator were stable.

Jeff

  reply	other threads:[~2018-12-14 16:02 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-03 20:50 Segher Boessenkool
2018-12-03 21:40 ` Jeff Law
2018-12-03 21:49   ` Jakub Jelinek
2018-12-10 18:25     ` Andrew Jenner
2018-12-10 20:13       ` Segher Boessenkool
2018-12-11  8:44         ` Richard Biener
2018-12-11 13:37           ` Jeff Law
2018-12-12 10:36             ` Richard Biener
2018-12-12 17:33               ` Segher Boessenkool
2018-12-13 16:49                 ` Jeff Law
2018-12-14  8:21                   ` Segher Boessenkool
2018-12-14  8:43                     ` Iain Sandoe
2018-12-14 16:08                     ` Jeff Law
2018-12-14  9:52                   ` Richard Biener
2018-12-14 16:02                     ` Jeff Law [this message]
2018-12-14 18:41                       ` Joseph Myers
2018-12-14 21:15                         ` Jeff Law

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=f3130062-38f0-e27d-590a-6ae62c61698f@redhat.com \
    --to=law@redhat.com \
    --cc=andrew@codesourcery.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=jakub@redhat.com \
    --cc=richard.guenther@gmail.com \
    --cc=segher@kernel.crashing.org \
    /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).