[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080613210257.GA529@elte.hu>
Date: Fri, 13 Jun 2008 23:02:57 +0200
From: Ingo Molnar <mingo@...e.hu>
To: David Brownell <david-b@...bell.net>
Cc: Lior Dotan <liodot@...il.com>, Adrian Bunk <bunk@...nel.org>,
"Rafael J. Wysocki" <rjw@...k.pl>, a.zummo@...ertech.it,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
bugme-daemon@...zilla.kernel.org, rtc-linux@...glegroups.com
Subject: Re: [Bug 10866] /dev/rtc was missing until I disabled
CONFIG_RTC_CLASS
* David Brownell <david-b@...bell.net> wrote:
> On Friday 13 June 2008, Lior Dotan wrote:
> > I don't know if it helps you to decide but the way I got to this
> > configuration is by copying my working 2.6.25 .config file and running
> > make oldconfig. I think this scenario is common when upgrading to a
> > newer version so you should make sure it doesn't generate an invalid
> > configuration.
>
> It doesn't! "No /dev/rtc" is a perfectly valid config. [...]
The bug scenario is rather simple: /dev/rtc existed with that config in
prior kernels and now, after "make oldconfig" it does not exist, it's a
plain regression and must be fixed.
> And it's not uncommon that new kernels require config tweaks.
that's wrong - 'make oldconfig' must work smoothly and in an expected
way.
If you argue that the new kernel should come up with a non-existent
/dev/rtc, then you are wrong and you go against established regression
handling policies of the kernel. Smooth migration via "make oldconfig"
is a must, otherwise we'd lose testers and users.
The old /dev/rtc might indeed have been unbelievably "bad" in various
ways and you dont even want to think about that code now, but it's your
code now that is used so you might as well think about the other 98% of
users who used the old /dev/rtc with old kernels. (not because they
wanted to use bad code, but because simply that was the default)
Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists