mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Thomas Gleixner <tglx@linutronix.de>
To: Daniel Walker <dwalker@mvista.com>
Cc: Ingo Molnar <mingo@elte.hu>, Steven Rostedt <rostedt@goodmis.org>,
	Sven-Thorsten Dietrich <sven@thebigcorporation.com>,
	Remy Bohmer <linux@bohmer.net>,
	LKML <linux-kernel@vger.kernel.org>,
	RT <linux-rt-users@vger.kernel.org>,
	Jon Masters <jonathan@jonmasters.org>
Subject: Re: Preempt-RT patch for 2.6.25
Date: Tue, 6 May 2008 03:30:46 +0200 (CEST)	[thread overview]
Message-ID: <alpine.LFD.1.10.0805060258360.3318@apollo.tec.linutronix.de> (raw)
In-Reply-To: <1210032701.17132.98.camel@localhost.localdomain>

On Mon, 5 May 2008, Daniel Walker wrote:
> On Tue, 2008-05-06 at 01:47 +0200, Ingo Molnar wrote:
> > * Daniel Walker <dwalker@mvista.com> wrote:
> > 
> > > I think dropping ports (temporarily) is perfectly reasonable. There is 
> > > no reason to hamper forward development just to keep old architecture 
> > > ports in the tree.
> > 
> > You are missing the point: a lot of people (those who wrote the brunt of 
> > the -rt tree and who maintained it over the years and who maintain it 
> > today) think it's not reasonable and have stated it very clearly to you 
> > that it's a bug. Keeping things alive is not preventing forward 
> > development.
> 
> That has always been my intention. I've never said the arch code would
> be permanently gone. 

Get it. Dropping it means bitrot.

The responsible maintainers keep that (maybe stale) code at least in
sync as far as the obvious fixups are concerned.

Your way of chosing the least effort approach and justifying it with
handwaving arguments is just disgusting.
 
> > Since it's code that you regard stale it shouldnt be all that hard to 
> > fix it up - in general it's much easier to fix a bug than to talk it out 
> > of existence, even if you disagree with a maintainer about how 
> > significant a bug is.
> 
> It shouldn't be hard, but it's too much to do all in one go. It's better
> from my perspective to do it in parts, given that there is significant
> refactoring to do per arch, if we solidify x86 then we can better
> refactor the other architectures. Better to do the split up once vs.
> many times.

Ah, it's too much in one go. But we should accept an "all in one go"
refactoring of the code we maintained over years including your
decision to drop fully functional ports (FYI, I've verified it) just
because you define that it's the best way to go.

Again, there is a choice.

1) Work with us - the maintainers and major contributors to preempt-rt -
   in a way which is useful for the project and the community

2) Hold on to your private "superior" uber queue and STFU.

> Not to mention this has not been discussed until now. My bisect tree was
> all but ignored until just recently.

It will be ignored for ever until you decide to work with us instead
of requesting that we drop our work in favour of your incomplete and
buggy hackery.

> > This issue is clearly not central to your refactoring (it cannot be, 
> > it's all about stale code), so by inflexibly insisting on your opinion 
> > against the (well-explained) opinion of the maintainers you'll just 
> > waste their time and make it more difficult for them to work with you, 
> > for no good reason.
> 
> I'm simply making sure their argument is widely known and potentially
> discussed on list instead of in a back room..

I'm impressed by your selfless efforts to save the world from the
ongoing preempt-rt conspiracy.

Thanks,

	tglx

  reply	other threads:[~2008-05-06  1:32 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-02 18:02 Remy Bohmer
2008-05-02 18:34 ` Sven-Thorsten Dietrich
2008-05-02 18:45   ` Steven Rostedt
2008-05-02 18:54     ` Sven-Thorsten Dietrich
2008-05-02 19:02       ` Steven Rostedt
2008-05-02 19:14         ` Sven-Thorsten Dietrich
2008-05-03  2:44           ` Steven Rostedt
2008-05-05 14:21             ` Daniel Walker
2008-05-05 16:01         ` Daniel Walker
2008-05-05 16:11           ` Steven Rostedt
2008-05-05 16:19             ` Daniel Walker
2008-05-05 16:44               ` Steven Rostedt
2008-05-05 17:04                 ` Daniel Walker
2008-05-05 18:32                   ` Steven Rostedt
2008-05-05 18:58                     ` Daniel Walker
2008-05-05 21:01                       ` Thomas Gleixner
2008-05-05 22:12                         ` Daniel Walker
2008-05-05 23:47                           ` Ingo Molnar
2008-05-06  0:11                             ` Daniel Walker
2008-05-06  1:30                               ` Thomas Gleixner [this message]
2008-05-06  1:43                                 ` Daniel Walker
2008-05-06  8:43                               ` Ingo Molnar
2008-05-06 16:01                                 ` Daniel Walker
2008-05-06  0:13                           ` Thomas Gleixner
2008-05-06  1:54                           ` Thomas Gleixner
2008-05-06  2:10                             ` Daniel Walker
2008-05-06  8:19                               ` Thomas Gleixner
2008-05-06 10:40                           ` Steven Rostedt
2008-05-06 16:05                             ` Daniel Walker
2008-05-06 16:32                               ` Steven Rostedt
2008-05-06 17:06                                 ` Daniel Walker
2008-05-06 20:59                                   ` Steven Rostedt
2008-05-05 16:17     ` Daniel Walker
2008-05-05 16:21       ` Steven Rostedt
2008-05-05 16:31         ` Daniel Walker

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=alpine.LFD.1.10.0805060258360.3318@apollo.tec.linutronix.de \
    --to=tglx@linutronix.de \
    --cc=dwalker@mvista.com \
    --cc=jonathan@jonmasters.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-rt-users@vger.kernel.org \
    --cc=linux@bohmer.net \
    --cc=mingo@elte.hu \
    --cc=rostedt@goodmis.org \
    --cc=sven@thebigcorporation.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

all inboxes | Powered by JetHome®