- Mail original -
> De: "Paolo Bonzini"
> À: qemu-devel@nongnu.org
> Cc: "Sebastian Tanase"
> Envoyé: Jeudi 7 Août 2014 15:34:49
> Objet: [PULL v2 10/11] monitor: Add drift info to 'info jit'
>
> From: Sebastian Tanase
>
> Sh
the pty is connected
after having tried to reconnect.
Signed-off-by: Sebastian Tanase
---
To reproduce the bug, launch a qemu image that has a parallel port (say lp0)
and redirect it to a pty (-parallel pty). After the VM is launched,
open the corresponding pty on your host (cat /dev/pts/X) and
- Mail original -
> De: "Paolo Bonzini"
> À: "Sebastian Tanase" , qemu-devel@nongnu.org
> Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell"
> ,
> mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com,
> lc
Show in 'info jit' the current delay between the host clock
and the guest clock. In addition, print the maximum advance
and delay of the guest compared to the host.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
---
cpu-exec.c| 6 ++
cpus.c
-off-by: Sebastian Tanase
Tested-by: Camille Bégué
Conflicts:
cpu-exec.c
---
cpu-exec.c | 32 +++-
1 file changed, 31 insertions(+), 1 deletion(-)
diff --git a/cpu-exec.c b/cpu-exec.c
index 1a725b6..7259c94 100644
--- a/cpu-exec.c
+++ b/cpu-exec.c
@@ -29,6
2 clocks is 6. However, we observe that the IO tests (using fio)
are
slower than the cpu tests (using stress).
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
Signed-off-by: Paolo Bonzini
---
cpu-exec.c | 91
cpus.c
icount_time_shift is used for calculting the delay
qemu has to sleep in order to synchronise the
host and guest clocks. Therefore, we need it in
cpu-exec.c.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
Signed-off-by: Paolo Bonzini
---
cpus.c| 8 ++--
include
The align option is used for activating the align algorithm
in order to synchronise the host clock and the guest clock.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
---
cpus.c| 19 ---
include/qemu-common.h | 1 +
qemu-options.hx | 15
Make icount parameter use QemuOpts style options in order
to easily add other suboptions.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
Signed-off-by: Paolo Bonzini
---
cpus.c| 10 +-
include/qemu-common.h | 3 ++-
qemu-options.hx | 4 ++--
qtest.c
et_clock and cpu_get_icount for showing drift information -> patch 6
Important note: This patch series relies on the bug fix discussed here
http://lists.gnu.org/archive/html/qemu-devel/2014-07/msg03208.html
Sebastian Tanase (6):
icount: Add QemuOpts for icount
icount: Add align option to ic
0 and increment based
on the instruction counter when the vcpu is active or
the qemu_icount_bias value when inactive.
Signed-off-by: Sebastian Tanase
---
cpus.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cpus.c b/cpus.c
index 5e7f2cf..de18ece 100644
--- a/cpus.c
+++ b/c
When using the icount option on ARM, the virtual
clock starts counting at realtime clock but it
should start at 0.
This small fix addresses this issue.
Signed-off-by: Sebastian Tanase
---
cpus.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cpus.c b/cpus.c
index 5e7f2cf
- Mail original -
> De: "Paolo Bonzini"
> À: "Sebastian Tanase"
> Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell"
> ,
> mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com,
> lcapitul...@redhat.com, cr
- Mail original -
> De: "Paolo Bonzini"
> À: "Sebastian Tanase"
> Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell"
> ,
> mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com,
> lcapitul...@redhat.com, cr
- Mail original -
> De: "Paolo Bonzini"
> À: "Sebastian Tanase" , qemu-devel@nongnu.org
> Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell"
> ,
> mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com,
> lc
- Mail original -
> De: "Paolo Bonzini"
> À: "Sebastian Tanase"
> Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell"
> ,
> mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com,
> lcapitul...@redhat.com, cr
- Mail original -
> De: "Paolo Bonzini"
> À: "Sebastian Tanase" , qemu-devel@nongnu.org
> Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell"
> ,
> mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com,
> lc
d of 'nanosleep' for Windows hosts in patch 4
* Remove function pointers from patches 4 and 5
Sebastian Tanase (6):
icount: Add QemuOpts for icount
icount: Add align option to icount
icount: Make icount_time_shift available everywhere
cpu_exec: Add sleeping algorithm
cpu_exec: Prin
Show in 'info jit' the current delay between the host clock
and the guest clock. In addition, print the maximum advance
and delay of the guest compared to the host.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
---
cpu-exec.c| 19 ++---
The align option is used for activating the align algorithm
in order to synchronise the host clock and the guest clock.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
---
cpus.c| 19 ---
include/qemu-common.h | 1 +
qemu-options.hx | 15
-off-by: Sebastian Tanase
Tested-by: Camille Bégué
Conflicts:
cpu-exec.c
---
cpu-exec.c | 39 ++-
1 file changed, 34 insertions(+), 5 deletions(-)
diff --git a/cpu-exec.c b/cpu-exec.c
index 9ed6ac1..b62df15 100644
--- a/cpu-exec.c
+++ b/cpu-exec.c
2 clocks is 6. However, we observe that the IO tests (using fio)
are
slower than the cpu tests (using stress).
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
Signed-off-by: Paolo Bonzini
---
cpu-exec.c | 97 ++
1 file
Make icount parameter use QemuOpts style options in order
to easily add other suboptions.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
Signed-off-by: Paolo Bonzini
---
cpus.c| 10 +-
include/qemu-common.h | 3 ++-
qemu-options.hx | 4 ++--
qtest.c
icount_time_shift is used for calculting the delay
qemu has to sleep in order to synchronise the
host and guest clocks. Therefore, we need it in
cpu-exec.c.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
Signed-off-by: Paolo Bonzini
---
cpus.c| 8 ++--
include
- Mail original -
> De: "Paolo Bonzini"
> À: "Sebastian Tanase" , qemu-devel@nongnu.org
> Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell"
> ,
> mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com,
> lc
- Mail original -
> De: "Frederic Konrad"
> À: "Sebastian Tanase" , qemu-devel@nongnu.org
> Cc: kw...@redhat.com, "peter maydell" ,
> a...@alex.org.uk, wenchaoq...@gmail.com,
> quint...@redhat.com, m...@tls.msk.ru, m...@redhat.com, stefa.
- Mail original -
> De: "Paolo Bonzini"
> À: "Sebastian Tanase" , qemu-devel@nongnu.org
> Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell"
> ,
> mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com,
> lc
- Mail original -
> De: "Paolo Bonzini"
> À: "Sebastian Tanase" , qemu-devel@nongnu.org
> Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell"
> ,
> mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com,
> lc
- Mail original -
> De: "Paolo Bonzini"
> À: "Sebastian Tanase" , qemu-devel@nongnu.org
> Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell"
> ,
> mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com,
> lc
- Mail original -
> De: "Paolo Bonzini"
> À: "Sebastian Tanase" , qemu-devel@nongnu.org
> Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell"
> ,
> mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com,
> lc
d by Paolo Bonzini).
* Change delay algorithm to use structures and function pointers in order to
avoid !defined(CONFIG_USER_ONLY) (as suggested by Paolo Bonzini).
* Limit the number of printed messages and add drift information in the
'info jit' command (as suggested by Paolo Bonzini
The align option is used for activating the align algorithm
in order to synchronise the host clock and the guest clock.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
---
cpus.c| 13 +
include/qemu-common.h | 1 +
qemu-options.hx | 15
2 clocks is 6. However, we observe that the IO tests (using fio)
are
slower than the cpu tests (using stress).
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
---
cpu-exec.c | 112 +
1 file changed, 112 insertions(+)
diff
-off-by: Sebastian Tanase
Tested-by: Camille Bégué
---
cpu-exec.c | 94 +++---
1 file changed, 90 insertions(+), 4 deletions(-)
diff --git a/cpu-exec.c b/cpu-exec.c
index ac741b7..4a4533d 100644
--- a/cpu-exec.c
+++ b/cpu-exec.c
@@ -24,12
icount_time_shift is used for calculting the delay
qemu has to sleep in order to synchronise the
host and guest clocks. Therefore, we need it in
cpu-exec.c.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
---
cpus.c| 8 ++--
include/qemu-common.h | 1 +
2 files
Make icount parameter use QemuOpts style options in order
to easily add other suboptions.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
---
cpus.c| 10 +-
include/qemu-common.h | 3 ++-
qemu-options.hx | 4 ++--
qtest.c | 13
Show in 'info jit' the current delay between the host clock
and the guest clock. In addition, print the maximum advance
and delay of the guest compared to the host.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
---
cpu-exec.c
Hello,
I'll be sending a new version (V3) of the patches on Monday. The patches add
QemuOpts
handling to the "-icount option". If you want I can only send the part of the
patch
that adds QemuOpts support.
Best regards,
Sebastian Tanase
- Mail original -
> De: &q
- Mail original -
> De: "Paolo Bonzini"
> À: "Sebastian Tanase" , qemu-devel@nongnu.org
> Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell"
> ,
> mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com,
> lc
take into account the new configure_icount() (patch 1).
* Surround delay code in cpu-exec.c with !defined(CONFIG_USER_ONLY) (patch 3).
* Clean up delay code in cpu-exec.c as suggested by Paolo Bonzini
and Juan Quintela (patch 3).
* Take into account initial offset between real and virtual clocks (pat
If the align option is enabled, we print to the user whenever
the guest clock is behind the host clock in order for he/she
to have a hint about the actual performance. The maximum
print interval is 2s so as not to spam the console.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
don't have this problem (for example
ARM) the offset will always be 0.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
---
cpu-exec.c| 2 +-
cpus.c| 11 +++
include/qemu-common.h | 1 +
3 files changed, 13 insertions(+), 1 deletion(-)
diff --git
icount_time_shift is used for calculting the delay
qemu has to sleep in order to synchronise the
host and guest clocks. Therefore, we need it in
cpu-exec.c.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
---
cpus.c| 8 ++--
include/qemu-common.h | 1 +
2 files
The align option is used for activating the align algorithm
in order to synchronise the host clock and the guest clock.
Therefore we slightly modified the existing icount option.
Thus, the new way to specify an icount is
'-icount [icount=][N|auto][,align]'
Signed-off-by: Sebastian Tana
-by: Sebastian Tanase
Tested-by: Camille Bégué
---
At first, we tried to approximate at each for loop the real time elapsed
while searching for a TB (generating or retrieving from cache) and
executing it. We would then approximate the virtual time corresponding
to the number of virtual
- Mail original -
> De: "Juan Quintela"
> À: "Sebastian Tanase"
> Cc: qemu-devel@nongnu.org, aligu...@amazon.com, afaer...@suse.de,
> r...@twiddle.net, "peter maydell"
> , mich...@walle.cc, pbonz...@redhat.com,
> a...@alex.org.
- Mail original -
> De: "Paolo Bonzini"
> À: "Sebastian Tanase"
> Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell"
> ,
> mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com, kw...@redhat.com,
> quint...@
-by: Sebastian Tanase
Tested-by: Camille Bégué
---
At first, we tried to approximate at each for loop the real time elapsed
while searching for a TB (generating or retrieving from cache) and
executing it. We would then approximate the virtual time corresponding
to the number of virtual
The align option is used for activating the align algorithm
in order to synchronise the host clock and the guest clock.
Therefore we slightly modified the existing icount option.
Thus, the new way to specify an icount is
'-icount [icount=][N|auto][,align]'
Signed-off-by: Sebastian Tana
every 2 seconds, and we only printif the
align option
is enabled.
Sebastian Tanase (4):
icount: Add 'align' and 'icount' options
icount: Make icount_time_shift available everywhere
cpu_exec: Add sleeping algorithm
cpu_exec: Print t
icount_time_shift is used for calculting the delay
qemu has to sleep in order to synchronise the
host and guest clocks. Therefore, we need it in
cpu-exec.c.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
---
cpus.c| 10 --
include/qemu-common.h | 1 +
2 files
If the align option is enabled, we print to the user whenever
the guest clock is behind the host clock in order for he/she
to have a hint about the actual performance. The maximum
print interval is 2s so as not to spam the console.
Signed-off-by: Sebastian Tanase
Tested-by: Camille Bégué
Hello,
First of all thank you for your feedback.
- Mail original -
> De: "Paolo Bonzini"
> À: "Sebastian Tanase" , qemu-devel@nongnu.org
> Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell"
> ,
> mich...@walle.
53 matches
Mail list logo
