Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 15 Aug 2006 13:04:52 -0700 (PDT)
From:      Kip Macy <kmacy@fsmware.com>
To:        Alexander Leidinger <Alexander@Leidinger.net>
Cc:        Suleiman Souhlal <ssouhlal@freebsd.org>, current@freebsd.org
Subject:   Re: HEADS-UP: starting to commit linuxolator (SoC 2006) changes...
Message-ID:  <20060815130418.I18116@demos.bsdclusters.com>
In-Reply-To: <20060815153451.604d16f1@Magellan.Leidinger.net>
References:  <20060815141151.15ae4349@Magellan.Leidinger.net> <44E1BD03.2030402@FreeBSD.org> <20060815144625.362bf376@Magellan.Leidinger.net> <44E1C3E4.7080508@FreeBSD.org> <20060815153451.604d16f1@Magellan.Leidinger.net>

next in thread | previous in thread | raw e-mail | index | archive | help
What does testing buy us when you've already admitted that futexes et
al. are broken?

 -Kip



On Tue, 15 Aug 2006, Alexander Leidinger wrote:

> Quoting Suleiman Souhlal <ssouhlal@FreeBSD.org> (Tue, 15 Aug 2006 14:53:56 +0200):
>
> I'm doing post-commit checks now (everything is commited and I'm doing
> a kernel build now)...
>
> > Alexander Leidinger wrote:
> >
> > > I already started... and I don't want to commit some parts (the linker
> > > stuff which allows to use the module on amd64).
> >  >
> > > It is also not used by default, as long as we have 2.4.2 for the linux
> > > version, no new functions will be used by glibc. So there is no change
> > > in behavior after the commits. I tested with acroread (which has issues
> > > when run with a 2.6.16 compat.linux.osrelease version, where the new
> > > functions are used by glibc).
> >  >
> > > So this gives us:
> > >  - coverity reports for the code *before the end of the SoC*
> >
> > Why the rush? I'm sure Roman will be around even when the SoC is over.
>
> Yes, but first he will take a vacation, and then he will not be
> available as much as he is now. And maybe Coverity will detect a bug
> which we search currently but can't find (don't know how likely this
> is, but it is at least possible).
>
> > Also, I'm not asking you to wait a month, just a couple of days until
> > more people have had a chance to look at the changes.
>
> I thought about this before deciding to commit it. Most of the code was
> available in P4. Those people with real interest already had a chance
> to look at it, or they didn't had time to do so. If they didn't had
> time to have a look at it, who knows if they have time to do it now?
> There's always someone who says "oh... if you waited 2 days longer...".
>
> That's not the only reason. Tomorrow my holiday is over and I have to
> catch up at work. Today I have plenty of time to commit urgent issues
> (e.g., tinderbox breakage). And at the upcomming WE I may not have
> enough time to do what I do today.
>
> Additionally, the work is a little bit stalled currently, we need more
> testers of the new code. There are a lot of people out there which want
> to test, but don't know how to patch or fear to do something wrong when
> they patch. With the approach I've taken, they just need to run a
> sysctl (compat.linux.osrelease=2.6.16) and they can test. All which
> don't want to test just don't need to care about it.
>
> > It's a bit unreasonable to say "here's a patch, please look at it" and
> > commit it less than a day later.
>
> Yes, it was a little bit fast...
>
> > >  - no change in behavior in the default case, since the new calls
> > >    aren't used by glibs as long as... see following entry
> > >  - easy testing possibility (sysctl compat.linux.osrelease=2.6.16,
> > >    defaults to 2.4.2)
> > >  - more eyes on the code
> >
> > Those are not valid reasons to commit unreviewed and potentially wrong
> > changes.
>
> Uhm... how many percent of code committed to current is reviewed before
> it is commited? How many percent of the code I committed is not
> reviewed and how much is this related to the amount of unreviewed code
> committed in a busy week?
>
> I'm running a kernel with this changes here and since I didn't enabled
> the new code with the sysctl, I don't see any difference in the
> behavior of the linuxolator. The potentially wrong code isn't active.
> It's like loading the linprocfs module but not using it.
>
> Bye,
> Alexander.
>
> --
>    Calvin: Can you make a living playing silly games?  His Dad: Actually, you
> can be among the most overpaid people on the planet.
> http://www.Leidinger.net  Alexander @ Leidinger.net: PGP ID = B0063FE7
> http://www.FreeBSD.org     netchild @ FreeBSD.org  : PGP ID = 72077137
> _______________________________________________
> freebsd-current@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-current
> To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org"
>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20060815130418.I18116>