* [PATCH 3/3] alleviate time drift with HPET periodic timers [not found] <1225303441.421235.1300462107604.JavaMail.root@zmail07.collab.prod.int.phx2.redhat.com> @ 2011-03-18 15:55 ` Ulrich Obergfell 0 siblings, 0 replies; 8+ messages in thread From: Ulrich Obergfell @ 2011-03-18 15:55 UTC (permalink / raw) To: qemu-devel; +Cc: kvm, gcosta, Anthony Liguori, aliguori, Jan Kiszka Part 3 of the patch implements the following options for the 'configure' script. --disable-hpet-driftfix --enable-hpet-driftfix Signed-off-by: Ulrich Obergfell <uobergfe@redhat.com> diff -up ./configure.orig3 ./configure --- ./configure.orig3 2011-02-18 22:48:06.000000000 +0100 +++ ./configure 2011-03-13 12:43:44.870966181 +0100 @@ -140,6 +140,7 @@ linux_aio="" attr="" vhost_net="" xfs="" +hpet_driftfix="no" gprof="no" debug_tcg="no" @@ -749,6 +750,10 @@ for opt do ;; --enable-rbd) rbd="yes" ;; + --disable-hpet-driftfix) hpet_driftfix="no" + ;; + --enable-hpet-driftfix) hpet_driftfix="yes" + ;; *) echo "ERROR: unknown option $opt"; show_help="yes" ;; esac @@ -2602,6 +2607,7 @@ echo "Trace output file $trace_file-<pid echo "spice support $spice" echo "rbd support $rbd" echo "xfsctl support $xfs" +echo "HPET drift fix $hpet_driftfix" if test $sdl_too_old = "yes"; then echo "-> Your SDL version is too old - please upgrade to have SDL support" @@ -2893,6 +2899,10 @@ if test "$rbd" = "yes" ; then echo "CONFIG_RBD=y" >> $config_host_mak fi +if test "$hpet_driftfix" = "yes" ; then + echo "CONFIG_HPET_DRIFTFIX=y" >> $config_host_mak +fi + # USB host support case "$usb" in linux) ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Qemu-devel] [PATCH 3/3] alleviate time drift with HPET periodic timers @ 2011-03-18 15:55 ` Ulrich Obergfell 0 siblings, 0 replies; 8+ messages in thread From: Ulrich Obergfell @ 2011-03-18 15:55 UTC (permalink / raw) To: qemu-devel; +Cc: Jan Kiszka, aliguori, kvm, gcosta Part 3 of the patch implements the following options for the 'configure' script. --disable-hpet-driftfix --enable-hpet-driftfix Signed-off-by: Ulrich Obergfell <uobergfe@redhat.com> diff -up ./configure.orig3 ./configure --- ./configure.orig3 2011-02-18 22:48:06.000000000 +0100 +++ ./configure 2011-03-13 12:43:44.870966181 +0100 @@ -140,6 +140,7 @@ linux_aio="" attr="" vhost_net="" xfs="" +hpet_driftfix="no" gprof="no" debug_tcg="no" @@ -749,6 +750,10 @@ for opt do ;; --enable-rbd) rbd="yes" ;; + --disable-hpet-driftfix) hpet_driftfix="no" + ;; + --enable-hpet-driftfix) hpet_driftfix="yes" + ;; *) echo "ERROR: unknown option $opt"; show_help="yes" ;; esac @@ -2602,6 +2607,7 @@ echo "Trace output file $trace_file-<pid echo "spice support $spice" echo "rbd support $rbd" echo "xfsctl support $xfs" +echo "HPET drift fix $hpet_driftfix" if test $sdl_too_old = "yes"; then echo "-> Your SDL version is too old - please upgrade to have SDL support" @@ -2893,6 +2899,10 @@ if test "$rbd" = "yes" ; then echo "CONFIG_RBD=y" >> $config_host_mak fi +if test "$hpet_driftfix" = "yes" ; then + echo "CONFIG_HPET_DRIFTFIX=y" >> $config_host_mak +fi + # USB host support case "$usb" in linux) ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 3/3] alleviate time drift with HPET periodic timers 2011-03-18 15:55 ` [Qemu-devel] " Ulrich Obergfell @ 2011-03-19 9:44 ` Jan Kiszka -1 siblings, 0 replies; 8+ messages in thread From: Jan Kiszka @ 2011-03-19 9:44 UTC (permalink / raw) To: Ulrich Obergfell; +Cc: qemu-devel, kvm, gcosta, Anthony Liguori, aliguori [-- Attachment #1: Type: text/plain, Size: 404 bytes --] On 2011-03-18 16:55, Ulrich Obergfell wrote: > > Part 3 of the patch implements the following options for the 'configure' script. > > --disable-hpet-driftfix > --enable-hpet-driftfix 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. Jan [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 259 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Qemu-devel] Re: [PATCH 3/3] alleviate time drift with HPET periodic timers @ 2011-03-19 9:44 ` Jan Kiszka 0 siblings, 0 replies; 8+ messages in thread From: Jan Kiszka @ 2011-03-19 9:44 UTC (permalink / raw) To: Ulrich Obergfell; +Cc: aliguori, qemu-devel, kvm, gcosta [-- Attachment #1: Type: text/plain, Size: 404 bytes --] On 2011-03-18 16:55, Ulrich Obergfell wrote: > > Part 3 of the patch implements the following options for the 'configure' script. > > --disable-hpet-driftfix > --enable-hpet-driftfix 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. Jan [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 259 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 3/3] alleviate time drift with HPET periodic timers 2011-03-19 9:44 ` [Qemu-devel] " Jan Kiszka @ 2011-03-22 10:03 ` Ulrich Obergfell -1 siblings, 0 replies; 8+ messages in thread From: Ulrich Obergfell @ 2011-03-22 10:03 UTC (permalink / raw) To: Jan Kiszka; +Cc: qemu-devel, kvm, gcosta, Anthony Liguori, aliguori >> Part 3 of the patch implements the following options for the >> 'configure' script. >> >> --disable-hpet-driftfix >> --enable-hpet-driftfix > > 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 ? Regards, Uli ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Qemu-devel] Re: [PATCH 3/3] alleviate time drift with HPET periodic timers @ 2011-03-22 10:03 ` Ulrich Obergfell 0 siblings, 0 replies; 8+ messages in thread From: Ulrich Obergfell @ 2011-03-22 10:03 UTC (permalink / raw) To: Jan Kiszka; +Cc: aliguori, qemu-devel, kvm, gcosta >> Part 3 of the patch implements the following options for the >> 'configure' script. >> >> --disable-hpet-driftfix >> --enable-hpet-driftfix > > 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 ? Regards, Uli ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 3/3] alleviate time drift with HPET periodic timers 2011-03-22 10:03 ` [Qemu-devel] " Ulrich Obergfell @ 2011-03-22 10:08 ` Jan Kiszka -1 siblings, 0 replies; 8+ messages in thread From: Jan Kiszka @ 2011-03-22 10:08 UTC (permalink / raw) To: Ulrich Obergfell; +Cc: qemu-devel, kvm, gcosta, Anthony Liguori, aliguori On 2011-03-22 11:03, Ulrich Obergfell wrote: > >>> Part 3 of the patch implements the following options for the >>> 'configure' script. >>> >>> --disable-hpet-driftfix >>> --enable-hpet-driftfix >> >> 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. Jan -- Siemens AG, Corporate Technology, CT T DE IT 1 Corporate Competence Center Embedded Linux ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Qemu-devel] Re: [PATCH 3/3] alleviate time drift with HPET periodic timers @ 2011-03-22 10:08 ` Jan Kiszka 0 siblings, 0 replies; 8+ messages in thread From: Jan Kiszka @ 2011-03-22 10:08 UTC (permalink / raw) To: Ulrich Obergfell; +Cc: aliguori, qemu-devel, kvm, gcosta On 2011-03-22 11:03, Ulrich Obergfell wrote: > >>> Part 3 of the patch implements the following options for the >>> 'configure' script. >>> >>> --disable-hpet-driftfix >>> --enable-hpet-driftfix >> >> 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. Jan -- Siemens AG, Corporate Technology, CT T DE IT 1 Corporate Competence Center Embedded Linux ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2011-03-22 10:08 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1225303441.421235.1300462107604.JavaMail.root@zmail07.collab.prod.int.phx2.redhat.com>
2011-03-18 15:55 ` [PATCH 3/3] alleviate time drift with HPET periodic timers Ulrich Obergfell
2011-03-18 15:55 ` [Qemu-devel] " Ulrich Obergfell
2011-03-19 9:44 ` Jan Kiszka
2011-03-19 9:44 ` [Qemu-devel] " Jan Kiszka
2011-03-22 10:03 ` Ulrich Obergfell
2011-03-22 10:03 ` [Qemu-devel] " Ulrich Obergfell
2011-03-22 10:08 ` Jan Kiszka
2011-03-22 10:08 ` [Qemu-devel] " Jan Kiszka
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.