[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 3/3] alleviate time drift with HPET periodic tim
[Qemu-devel] Re: [PATCH 3/3] alleviate time drift with HPET periodic timers
Tue, 22 Mar 2011 11:08:10 +0100
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:22.214.171.124) Gecko/20080226 SUSE/126.96.36.199-1.1 Thunderbird/188.8.131.52 Mnenhy/0.7.5.666
On 2011-03-22 11:03, Ulrich Obergfell wrote:
>>> Part 3 of the patch implements the following options for the
>>> 'configure' script.
>> I see no benefit in this configurability. Just make the driftfix
>> unconditionally available, runtime-disabled by default for now until it
>> matured and there is no downside in enabling it all the time.
> Many Thanks Jan,
> I enclosed the code in '#ifdef CONFIG_HPET_DRIFTFIX ... #endif'
> so that it can be easily identified (and removed if the generic API
> would be implemented some day). Since the ifdef's are already there
> I added the configuration option for convenience. As you don't see
> any benefit in this option, I can remove that part of the patch.
> However, I'd suggest to keep the ifdef's and do the following:
> - Rename to '#ifdef HPET_DRIFTFIX ... #endif' to make it clear that
> this is not controlled via a configuration option.
> - Add '#define HPET_DRIFTFIX' to hw/hpet_emul.h.
> Do you agree ?
Thanks to versioning control and feature-oriented commits, it's not very
hard to identify what code changes relate to which feature additions. So
I still don't see a need for that.
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux