Re: [PATCH v2 1/4] Implement {read,update}_persistent_clock.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello.

Benjamin Herrenschmidt wrote:

Eh... poor you. Tony got clockevent driver reengineered for no apparent reason. And he's introduced the jiffy drift by deleting the main loop from timer_interrupt(). Yet this borken version was preferred to what was known working since about 2.6.18 and included into 2.6.21-rt patchset. I don't like that policy. Will you be pushing fixes from -rt to PowerPC, or will it fall on my shoulders now?

Possibly because whatever implementation existed before was never
provided in a mergeable form

The fact that vDSO calls were removed out of the necessity (because having the broken TOD stuff removed was better than to leave it in place producing inexact results) doesn't mean that everything in the patches provided was broken and of no use. Yet you want everything removed in favor of somewhat dubious implementation.

and pushed to -rt bypassing the powerpc architecture maintainer ?

There was no bypassing, everything was submitted publicly via this list. The -rt patch (via the hrtimers patchset) was the place to merge the hrtimers code at thiat time, and nobody showed any interest in making the code better, i.e. amending vDSO stuff, for months... (The question also is why Tony was submitting his patches also to linux-rt-users while they were known not to apply to the -rt patch.)

Ben.

WBR, Sergei
-
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [RT Stable]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]

  Powered by Linux