From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: Ingo Molnar <mingo@elte.hu>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>,
mingo@redhat.com, hpa@zytor.com, linux-kernel@vger.kernel.org,
roland@redhat.com, suresh.b.siddha@intel.com, tglx@linutronix.de,
hjl.tools@gmail.com, Andrew Morton <akpm@linux-foundation.org>,
Linus <torvalds@linux-foundation.org>
Subject: Re: linux-next requirements
Date: Sat, 27 Feb 2010 13:23:14 +0100 [thread overview]
Message-ID: <201002271323.14402.rjw@sisk.pl> (raw)
In-Reply-To: <20100227093948.GB31794@elte.hu>
On Saturday 27 February 2010, Ingo Molnar wrote:
>
> * Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> > [I have removed linux-tip-commits from the cc list]
> >
> > Hi Ingo,
> >
> > On Tue, 23 Feb 2010 09:45:52 +0100 Ingo Molnar <mingo@elte.hu> wrote:
> > >
> > > Developers simply cannot be expected to build for 22 architectures, and
> > > they shouldnt be.
> >
> > I have agreed with this point of yours several times. Why do you keep
> > stating it?
>
> If you agree with me then why do you put so much focus on cross-arch build
> failures, versus other, more relevant forms of testing?
I don't really know what this is all about. Stephen does what he can and
that's generally appreciated very much. It helps to make sure the code builds
correctly on the architectures it's supposed to build on and there's nothing
wrong with that IMO.
> > > The thing is, last i checked you didnt even _test_ x86 as the first step
> > > in your linux-next build tests. Most of your generic build bug reports are
> > > against PowerPC. They create the appearance that x86 is a second class
> > > citizen in linux-next.
> >
> > Lets see. Over the last 60 days, I have reported 37 build errors. Of
> > these, 16 were reported against x86, 14 against ppc, 7 against other archs.
>
> So only 43% of them were even relevant on the platform that 95+% of the Linux
> testers use? Seems to support the points i made.
Well, I hope you don't mean that because the majority of bug reporters (vs
testers, the number of whom is unknown to me at least) use x86, we are free
to break the other architectures. ;-)
> > Of the ppc reports, 10 would not affect x86 builds (due to being ppc
> > specific problems or dependencies on implicit includes that do happen on
> > x86). None of the reports against other arches would affect x86 builds.
> >
> > I also reported 31 warnings. 15 against x86, 15 against ppc and 1 against
> > both. Of those only reported against ppc, 13 did not affect x86.
> >
> > So of my "generic" reports, 4 errors and 2 warnings were reported against
> > ppc, 16 errors and 15 warnings again x86.
> >
> > Also, I am not sure how reports of 37 build errors and 32 warnings over 60
> > days can tax the resources of our developer base. [...]
>
> Note that out of those 37 build errors only a small minority were caused by
> any tree i co-maintain. (i dont have the precise numbers but it's below 5)
>
> Why? Because i cross-build before pushing to linux-next. I bug people about
> cross-arch build failures, and about the patch flow delays and hickups this
> causes. Without that you'd see twice that many cross-build failures.
>
> Which in itself is not bad of course (any fix is a good fix) - except the
> forced prioritization and its place in the workflow: it sends the wrong
> testing message.
>
> It sends the message that building on N architectures is more important than
> for the code to work for real people. I've had good developers waste their
> time trying to set up cross-build testing environments and complain to me how
> this complicates their testing.
That's the kind of task linux-next is really good at AFAICT. Before linux-next
I used to have a cross-build testing environment like this, but I don't need it
any more, because I know linux-next will catch the cross-build problems for
me and I appreciate that very much, because it saves a lot of my time.
Rafael
next prev parent reply other threads:[~2010-02-27 12:22 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-02-11 19:50 [patch v3 0/2] updated ptrace/core-dump patches for supporting xstate - v3 Suresh Siddha
2010-02-11 19:50 ` [patch v3 1/2] x86, ptrace: regset extensions to support xstate Suresh Siddha
2010-02-11 23:18 ` [tip:x86/ptrace] " tip-bot for Suresh Siddha
2010-02-12 3:45 ` [patch v3 1/2] " Roland McGrath
2010-02-12 17:31 ` Oleg Nesterov
2010-02-11 19:51 ` [patch v3 2/2] ptrace: Add support for generic PTRACE_GETREGSET/PTRACE_SETREGSET Suresh Siddha
2010-02-11 23:19 ` [tip:x86/ptrace] " tip-bot for Suresh Siddha
2010-02-22 9:07 ` Ingo Molnar
2010-02-22 9:33 ` linux-next requiements (Was: Re: [tip:x86/ptrace] ptrace: Add support for generic PTRACE_GETREGSET/PTRACE_SETREGSET) Stephen Rothwell
2010-02-22 10:27 ` Ingo Molnar
2010-02-22 11:47 ` linux-next requirements " Stephen Rothwell
2010-02-22 22:57 ` H. Peter Anvin
2010-02-22 23:59 ` Stephen Rothwell
2010-02-23 20:20 ` Roland McGrath
2010-02-23 20:49 ` H. Peter Anvin
2010-02-23 22:54 ` Stephen Rothwell
2010-02-23 8:45 ` Ingo Molnar
2010-02-23 19:52 ` Al Viro
2010-02-23 19:57 ` Al Viro
2010-02-24 7:25 ` linux-next requirements Stephen Rothwell
2010-02-27 1:53 ` Grant Likely
2010-02-27 8:53 ` Geert Uytterhoeven
2010-02-27 9:09 ` Jaswinder Singh Rajput
2010-02-27 9:39 ` Ingo Molnar
2010-02-27 12:23 ` Rafael J. Wysocki [this message]
2010-02-27 12:47 ` Ingo Molnar
2010-02-27 19:07 ` Rafael J. Wysocki
2010-02-27 21:50 ` Geert Uytterhoeven
2010-02-27 22:31 ` Rafael J. Wysocki
2010-02-28 7:06 ` Ingo Molnar
2010-02-28 12:22 ` Rafael J. Wysocki
2010-02-28 7:14 ` Ingo Molnar
2010-02-28 7:37 ` Stephen Rothwell
2010-02-28 7:51 ` Ingo Molnar
2010-02-28 8:19 ` Al Viro
2010-02-28 8:53 ` Ingo Molnar
2010-02-28 10:26 ` Stephen Rothwell
2010-02-28 7:23 ` Ingo Molnar
2010-03-01 15:13 ` Nick Bowler
2010-03-03 21:53 ` Pavel Machek
2010-03-04 0:35 ` Thomas Gleixner
2010-03-04 0:42 ` Andrew Morton
2010-03-04 1:17 ` Thomas Gleixner
2010-03-04 2:48 ` Ingo Molnar
2010-02-22 18:37 ` [tip:x86/ptrace] ptrace: Add support for generic PTRACE_GETREGSET/PTRACE_SETREGSET Roland McGrath
2010-02-23 18:36 ` [tip:x86/ptrace] parisc: Disable CONFIG_HAVE_ARCH_TRACEHOOK tip-bot for Roland McGrath
2010-02-12 3:56 ` [patch v3 2/2] ptrace: Add support for generic PTRACE_GETREGSET/PTRACE_SETREGSET Roland McGrath
2010-02-12 15:59 ` Oleg Nesterov
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=201002271323.14402.rjw@sisk.pl \
--to=rjw@sisk.pl \
--cc=akpm@linux-foundation.org \
--cc=hjl.tools@gmail.com \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=mingo@redhat.com \
--cc=roland@redhat.com \
--cc=sfr@canb.auug.org.au \
--cc=suresh.b.siddha@intel.com \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.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