Re: Preempt-RT patch for 2.6.25

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Thomas Gleixner
Date: Monday, May 5, 2008 - 2:01 pm

Daniel,

On Mon, 5 May 2008, Daniel Walker wrote:

Dropping architectures just for the sake of bisectability does more
harm than good.

We never dropped architecture support due to a rewrite in course of
the whole preempt-rt series. We never dropped architecture support
when we made parts of the code ready for upstream and made it
bisectable.

Keeping the architecture ports even in a maybe disfunctional state in
the queue is important as we move them along and make the obvious
changes to them so anyone interested to bring them up to date has not
to start from scratch again, which is a major PITA (we just did one
from scratch)


Yes, and if you care to look at the code which was merged into
mainline, then you might notice that it was always made
bisectable. Usually it was rewritten pretty much as well.


With half of the functionality dropped, renames along the line so it
can not be verified whether it ends up to be the same code as it was
before. If you really want to make it bisectable then apply the
necessary rename patches to the queue first, make sure that it does
not result in a code change and then refactor the whole thing.


We have been there before. kernel development does not follow the "we
want _now_" principle at all. Have you ever tried to yell at Linus "we
want XYZ _now_" ? If you decide to try it, please keep me on CC - I
want to enjoy the show.


Sure, we are happy to trade a queue which has major updates of code
close to mainline integration and has preserved the existing
architecture support for some bisectable artifact.

We went through this kind of discussion several times in the past and
you still seem to believe that you can impose your POV on project
maintainers.

Again, that's not the way it works. 

Nobody will object the refactoring of the queue when it is done in a
cooperative way. By creating a fact and trying to enforce it by any
means you'll only reap controversy and attention, not any real
progress.


Exactly, for the majority of problems with preempt-rt, reporting the
bug and waiting for the person who is able to decode it is the only
choice. The hard to decode problems like subtle races are not
decodable by bisection.

Bisectable problems are pretty rare, because most of the problems are
with PREEMPT_RT, which is a disruptive new feature that only gets
activated late in the queue.

Again, we are of course not opposed to a cooperative effort to make
the queue fully bisectable - as long as it has no drawbacks. That
means gradual steps, which is not rocket science.

Thanks,

	tglx
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Preempt-RT patch for 2.6.25, Remy Bohmer, (Fri May 2, 11:02 am)
Re: Preempt-RT patch for 2.6.25, Sven-Thorsten Dietrich, (Fri May 2, 11:34 am)
Re: Preempt-RT patch for 2.6.25, Steven Rostedt, (Fri May 2, 11:45 am)
Re: Preempt-RT patch for 2.6.25, Sven-Thorsten Dietrich, (Fri May 2, 11:54 am)
Re: Preempt-RT patch for 2.6.25, Steven Rostedt, (Fri May 2, 12:02 pm)
Re: Preempt-RT patch for 2.6.25, Sven-Thorsten Dietrich, (Fri May 2, 12:14 pm)
Re: Preempt-RT patch for 2.6.25, Steven Rostedt, (Fri May 2, 7:44 pm)
Re: Preempt-RT patch for 2.6.25, Daniel Walker, (Mon May 5, 7:21 am)
Re: Preempt-RT patch for 2.6.25, Daniel Walker, (Mon May 5, 9:01 am)
Re: Preempt-RT patch for 2.6.25, Steven Rostedt, (Mon May 5, 9:11 am)
Re: Preempt-RT patch for 2.6.25, Daniel Walker, (Mon May 5, 9:17 am)
Re: Preempt-RT patch for 2.6.25, Daniel Walker, (Mon May 5, 9:19 am)
Re: Preempt-RT patch for 2.6.25, Steven Rostedt, (Mon May 5, 9:21 am)
Re: Preempt-RT patch for 2.6.25, Daniel Walker, (Mon May 5, 9:31 am)
Re: Preempt-RT patch for 2.6.25, Steven Rostedt, (Mon May 5, 9:44 am)
Re: Preempt-RT patch for 2.6.25, Daniel Walker, (Mon May 5, 10:04 am)
Re: Preempt-RT patch for 2.6.25, Steven Rostedt, (Mon May 5, 11:32 am)
Re: Preempt-RT patch for 2.6.25, Daniel Walker, (Mon May 5, 11:58 am)
Re: Preempt-RT patch for 2.6.25, Thomas Gleixner, (Mon May 5, 2:01 pm)
Re: Preempt-RT patch for 2.6.25, Daniel Walker, (Mon May 5, 3:12 pm)
Re: Preempt-RT patch for 2.6.25, Ingo Molnar, (Mon May 5, 4:47 pm)
Re: Preempt-RT patch for 2.6.25, Daniel Walker, (Mon May 5, 5:11 pm)
Re: Preempt-RT patch for 2.6.25, Thomas Gleixner, (Mon May 5, 5:13 pm)
Re: Preempt-RT patch for 2.6.25, Thomas Gleixner, (Mon May 5, 6:30 pm)
Re: Preempt-RT patch for 2.6.25, Daniel Walker, (Mon May 5, 6:43 pm)
Re: Preempt-RT patch for 2.6.25, Thomas Gleixner, (Mon May 5, 6:54 pm)
Re: Preempt-RT patch for 2.6.25, Daniel Walker, (Mon May 5, 7:10 pm)
Re: Preempt-RT patch for 2.6.25, Thomas Gleixner, (Tue May 6, 1:19 am)
Re: Preempt-RT patch for 2.6.25, Ingo Molnar, (Tue May 6, 1:43 am)
Re: Preempt-RT patch for 2.6.25, Steven Rostedt, (Tue May 6, 3:40 am)
Re: Preempt-RT patch for 2.6.25, Daniel Walker, (Tue May 6, 9:01 am)
Re: Preempt-RT patch for 2.6.25, Daniel Walker, (Tue May 6, 9:05 am)
Re: Preempt-RT patch for 2.6.25, Steven Rostedt, (Tue May 6, 9:32 am)
Re: Preempt-RT patch for 2.6.25, Daniel Walker, (Tue May 6, 10:06 am)
Re: Preempt-RT patch for 2.6.25, Steven Rostedt, (Tue May 6, 1:59 pm)