From: Laurent ALFONSI
Note that the support for the command-line argument requires:
1. add the new field "uint8_t rand_bytes[16]" to "struct
image_info" since only the variable "info" lives both in
main() and in create_elf_tables()
2. write a dedicated parser to conver
On Mon, Jun 20, 2011 at 6:37 AM, Fam Zheng wrote:
> Is there any difference between bdrv_getlength and
> bdrv_get_allocated_file_size for bs-file? If not, I can simplify it by
> reusing it in two raw devices.
Yes, the two functions are different:
POSIX sparse files (files with holes) take up les
Is there any difference between bdrv_getlength and
bdrv_get_allocated_file_size for bs-file? If not, I can simplify it by
reusing it in two raw devices.
On Sun, Jun 19, 2011 at 2:35 AM, Stefan Hajnoczi wrote:
> On Sat, Jun 4, 2011 at 1:44 AM, Fam Zheng wrote:
>> The `qemu-img info` results for m
Am 19.06.2011 um 22:57 schrieb Blue Swirl:
These and the stack frame patches can be found in
git://repo.or.cz/qemu/blueswirl.git
Blue Swirl (9):
cpu_loop_exit: avoid using AREG0
sparc: fix coding style of the area to be moved
sparc: move do_interrupt to helper.c
x86: use caller supplied CPU
On Mon, 20 Jun 2011, Blue Swirl wrote:
> Use TCG_REG_CALL_STACK instead of TCG_REG_R1 etc. for consistency.
You spell it TCG_REG_CALL_STACK in the subject/comment but
REG_CALL_STACK in the patch, which suggest that it was never
even compile tested.
[..snip..]
--
mailto:av1...@comtv.ru
Use stack instead of temp_buf array in CPUState for TCG temps.
Signed-off-by: Blue Swirl
---
tcg/mips/tcg-target.c |7 ---
1 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/tcg/mips/tcg-target.c b/tcg/mips/tcg-target.c
index 79e20fa..d536457 100644
--- a/tcg/mips/tcg-target.
The IBM E15 is equivalent to an S3 Vision864.
Lacking S3 SDAC (86C716) support, the DAC indizes are translated
to greyscale colors. This works sufficiently to observe firmware
boot progress.
Cc: Hervé Poussineau
Fixed off-by-one drawing issue.
Replaced hardcoded color for RECT.
Separate I/O deb
Use TCG_REG_CALL_STACK instead of TCG_REG_SP for consistency.
Signed-off-by: Blue Swirl
---
tcg/hppa/tcg-target.c | 30 --
1 files changed, 16 insertions(+), 14 deletions(-)
diff --git a/tcg/hppa/tcg-target.c b/tcg/hppa/tcg-target.c
index 79bca63..bd40954 100644
--
Use stack instead of temp_buf array in CPUState for TCG temps.
Signed-off-by: Blue Swirl
---
tcg/arm/tcg-target.c | 11 +--
1 files changed, 9 insertions(+), 2 deletions(-)
diff --git a/tcg/arm/tcg-target.c b/tcg/arm/tcg-target.c
index 93eb0f1..eacda6b 100644
--- a/tcg/arm/tcg-target.
Am 19.06.2011 um 21:03 schrieb Blue Swirl:
On Sun, Jun 19, 2011 at 9:40 PM, Andreas Färber > wrote:
Some remarks:
* TCX seems to assume that TARGET_HOST_PAGE == 4 * 1024 in the
unrolled loop
in tcx_update_display(). You might want to assert that.
TCX was only used on Sparc32. Though some U
Use stack instead of temp_buf array in CPUState for TCG temps.
On Sparc64, stack pointer is not aligned but there is a fixed bias of 2047,
so don't try to enforce alignment.
Signed-off-by: Blue Swirl
---
tcg/sparc/tcg-target.c |7 ---
tcg/tcg.c |2 ++
2 files changed, 6
Use stack instead of temp_buf array in CPUState for TCG temps.
Signed-off-by: Blue Swirl
---
tcg/ppc/tcg-target.c |7 +--
tcg/ppc64/tcg-target.c |7 +--
2 files changed, 10 insertions(+), 4 deletions(-)
diff --git a/tcg/ppc/tcg-target.c b/tcg/ppc/tcg-target.c
index 729061f..66
Use stack instead of temp_buf array in CPUState for TCG
temps.
Signed-off-by: Blue Swirl
---
tcg/i386/tcg-target.c | 22 --
1 files changed, 12 insertions(+), 10 deletions(-)
diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c
index 4c29ced..7529677 100644
--- a/tcg
Use TCG_REG_CALL_STACK instead of TCG_REG_R15 for consistency.
Signed-off-by: Blue Swirl
---
tcg/s390/tcg-target.c |8 +---
1 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/tcg/s390/tcg-target.c b/tcg/s390/tcg-target.c
index 2fc5646..e112052 100644
--- a/tcg/s390/tcg-target
Use TCG_REG_CALL_STACK instead of TCG_REG_SP for consistency.
Signed-off-by: Blue Swirl
---
tcg/mips/tcg-target.c | 13 +++--
1 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/tcg/mips/tcg-target.c b/tcg/mips/tcg-target.c
index 12ff9d5..79e20fa 100644
--- a/tcg/mips/tcg-ta
Delegate TCG temp_buf setup to targets, so that they can use a stack
frame later instead.
Signed-off-by: Blue Swirl
---
tcg/arm/tcg-target.c |2 ++
tcg/hppa/tcg-target.c |2 ++
tcg/i386/tcg-target.c |2 ++
tcg/ia64/tcg-target.c |2 ++
tcg/mips/tcg-target.c |2 ++
tcg/pp
Remove now unused temp_buf from CPUState. Rename CPU_TEMP_BUF_NLONGS
to TCG_TEMP_BUF_NLONGS.
Signed-off-by: Blue Swirl
---
cpu-defs.h |3 ---
tcg/arm/tcg-target.c |6 +++---
tcg/hppa/tcg-target.c |4 ++--
tcg/i386/tcg-target.c |4 ++--
tcg/ia64/tcg-target.c |
Use TCG_REG_CALL_STACK instead of TCG_REG_R1 etc. for consistency.
Signed-off-by: Blue Swirl
---
tcg/ppc/tcg-target.c | 17 ++---
tcg/ppc64/tcg-target.c | 16 +---
2 files changed, 19 insertions(+), 14 deletions(-)
diff --git a/tcg/ppc/tcg-target.c b/tcg/ppc/tcg-ta
I think x86 and Sparc patches are ready to be applied.
Blue Swirl (18):
Delegate setup of TCG temporaries to targets
TCG: fix negative frame offset calculations
TCG: remove broken stack allocation for call arguments
TCG/x86: use TCG_REG_CALL_STACK instead of TCG_REG_ESP
TCG/x86: use stac
Move functions cpu_has_work() and cpu_pc_from_tb() from exec.h to cpu.h. This is
needed by later patches.
Signed-off-by: Blue Swirl
---
exec-all.h |1 +
target-alpha/cpu.h | 22 +
target-alpha/exec.h | 20 ---
target-arm/cpu.h
Use stack instead of temp_buf array in CPUState for TCG temps.
Signed-off-by: Blue Swirl
---
tcg/ia64/tcg-target.c |9 ++---
1 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index e22e9b6..528ae64 100644
--- a/tcg/ia64/tcg-targe
Use stack instead of temp_buf array in CPUState for TCG temps.
Signed-off-by: Blue Swirl
---
tcg/s390/tcg-target.c |7 ---
1 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/tcg/s390/tcg-target.c b/tcg/s390/tcg-target.c
index e112052..c063509 100644
--- a/tcg/s390/tcg-target.
Now that all targets use common function signature for do_interrupt(),
there is no
need for the #ifdeffery anymore.
Signed-off-by: Blue Swirl
---
cpu-exec.c | 26 --
1 files changed, 0 insertions(+), 26 deletions(-)
diff --git a/cpu-exec.c b/cpu-exec.c
index 64203b3..f
Use TCG_REG_CALL_STACK instead of TCG_REG_R12 for consistency.
Signed-off-by: Blue Swirl
---
tcg/ia64/tcg-target.c |9 ++---
1 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index 6386a5b..e22e9b6 100644
--- a/tcg/ia64/tcg-targe
Except for specific cases where the use of %esp changes the encoding of
the instruction, it's cleaner to use TCG_REG_CALL_STACK instead of
TCG_REG_ESP.
Signed-off-by: Blue Swirl
---
tcg/i386/tcg-target.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/tcg/i386/tcg
Most exec-all.h include directives are now useless, remove them.
Signed-off-by: Blue Swirl
---
bsd-user/main.c |2 +-
cpus.c|1 -
disas.c |1 -
exec.c|1 -
gdbstub.c |2 +-
Several x86 specific functions are called from cpu-exec.c with the
assumption that global env register is valid. This will be changed
later, so make the functions use caller supplied CPUState parameter.
It would be cleaner to move the functions to helper.c, but there are
quite a lot of dependencie
size_t is unsigned, so the frame offset calculations can be incorrect for
negative offsets.
Signed-off-by: Blue Swirl
---
tcg/tcg.c | 10 +++---
1 files changed, 7 insertions(+), 3 deletions(-)
diff --git a/tcg/tcg.c b/tcg/tcg.c
index debf47f..d8bf721 100644
--- a/tcg/tcg.c
+++ b/tcg/tcg.
Use stack instead of temp_buf array in CPUState for TCG temps.
Signed-off-by: Blue Swirl
---
tcg/hppa/tcg-target.c |7 +--
1 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/tcg/hppa/tcg-target.c b/tcg/hppa/tcg-target.c
index bd40954..b900c88 100644
--- a/tcg/hppa/tcg-target.
do_interrupt() was mixing CPUState pointer passed from caller
and global env (AREG0).
Fix by moving the function to helper.c. Introduce a helper for calling
change_pstate() safely from outside of execution context.
Signed-off-by: Blue Swirl
---
target-sparc/cpu.h |4 +
target-sparc/ex
Before the next patch, fix coding style of the areas affected.
Change the type of the return value from cpu_has_work() and
qemu_cpu_has_work() to bool.
Signed-off-by: Blue Swirl
---
cpu-all.h|2 +-
cpu-exec.c |2 +-
target-alpha/exec.h |2 +-
targe
The code for stack allocation for call arguments is way too simplistic
to actually work on targets with non-trivial stack allocation policies,
e.g. ppc64. We've also already allocated TCG_STATIC_CALL_ARGS_SIZE worth
of stack for calls which should be well more than any helper needs.
Remove broken
Pass CPUState to do_interrupt(). This is needed by later patches.
It would be cleaner to move the function to helper.c, but there are
a few dependencies between do_interrupt() and other functions.
Signed-off-by: Blue Swirl
---
cpu-exec.c |4 ++--
target-m68k/cpu.h |3
Before the next patch, fix coding style of the areas affected.
Signed-off-by: Blue Swirl
---
target-sparc/op_helper.c | 31 +--
1 files changed, 17 insertions(+), 14 deletions(-)
diff --git a/target-sparc/op_helper.c b/target-sparc/op_helper.c
index 52a894e..f049e3
Make cpu_loop_exit() take a parameter for CPUState instead of relying
on global env.
Signed-off-by: Blue Swirl
---
cpu-exec.c| 16
exec-all.h|2 +-
target-alpha/op_helper.c |6 +++---
target-arm/op_helper.c|6 +++
These and the stack frame patches can be found in
git://repo.or.cz/qemu/blueswirl.git
Blue Swirl (9):
cpu_loop_exit: avoid using AREG0
sparc: fix coding style of the area to be moved
sparc: move do_interrupt to helper.c
x86: use caller supplied CPUState for interrupt related stuff
m68k:
On Thu, Jun 9, 2011 at 2:39 PM, Jan Kiszka wrote:
> On 2011-06-09 22:00, Anthony Liguori wrote:
>> On 06/09/2011 11:44 AM, Jan Kiszka wrote:
>>> A first step towards getting rid of register_device_unmigratable
>>> (ivshmem and lacking vmstate support in virtio are blocking this):
>>>
>>> Allow to
On Sun, Jun 19, 2011 at 9:40 PM, Andreas Färber wrote:
> Am 19.06.2011 um 15:27 schrieb Blue Swirl:
>
>> On Sun, Jun 19, 2011 at 1:04 PM, Andreas Färber
>> wrote:
>>>
>>> Am 18.06.2011 um 22:42 schrieb Blue Swirl:
>>>
On Thu, Jun 16, 2011 at 3:02 AM, Andreas Färber
wrote:
>
> T
On 06/17/2011 10:25 PM, Luiz Capitulino wrote:
On Fri, 17 Jun 2011 16:25:32 -0500
Michael Roth wrote:
On 06/17/2011 03:13 PM, Luiz Capitulino wrote:
On Fri, 17 Jun 2011 14:21:31 -0500
Michael Roth wrote:
On 06/16/2011 01:42 PM, Luiz Capitulino wrote:
On Tue, 14 Jun 2011 15:06:22 -0500
Mi
Am 19.06.2011 um 15:27 schrieb Blue Swirl:
On Sun, Jun 19, 2011 at 1:04 PM, Andreas Färber > wrote:
Am 18.06.2011 um 22:42 schrieb Blue Swirl:
On Thu, Jun 16, 2011 at 3:02 AM, Andreas Färber >
wrote:
The IBM E15 is equivalent to an S3 Vision864.
Lacking S3 SDAC (86C716) support, the DAC ind
Hi
I am trying to understand the structures that QEMU saves when do_savevm()
is invoked. Can anyone explain to me the fields that are part of the
TimerState structure in qemu-timer.c?
Thanks
Nilay
On 19.06.2011, at 18:04, Avi Kivity wrote:
> On 06/19/2011 06:53 PM, Andreas Färber wrote:
>> Am 19.06.2011 um 17:46 schrieb Avi Kivity:
>>
>>> On 06/19/2011 06:22 PM, Stefan Hajnoczi wrote:
I wonder if this will break non-Linux platforms. Perhaps buffer an
entire row of pixels instea
On 06/17/2011 08:53 AM, Stefan Hajnoczi wrote:
Perhaps someone has been saying this all along but I want to spell it
out that image streaming and live block copy are equivalent in theory.
I just realized this last night. In practice we might choose one
implementation or two different ones for pe
On Tue, Jun 07, 2011 at 12:15:55PM +0200, Jiri Denemark wrote:
> > +}
> > +},
>
> What about using the same form of progress reporting as used by query-migrate?
> That is, instead of
Done.
> One can trivially compute percentage from that but it's impossible to get this
> kind
On 06/19/2011 06:53 PM, Andreas Färber wrote:
Am 19.06.2011 um 17:46 schrieb Avi Kivity:
On 06/19/2011 06:22 PM, Stefan Hajnoczi wrote:
I wonder if this will break non-Linux platforms. Perhaps buffer an
entire row of pixels instead and only fwrite(3) at the end of the
outer loop.
That's how
On 06/18/2011 12:17 PM, Stefan Hajnoczi wrote:
On Sat, Jun 18, 2011 at 10:15 AM, Stefan Hajnoczi wrote:
On Fri, Jun 17, 2011 at 1:31 PM, Marcelo Tosatti wrote:
On Thu, Jun 16, 2011 at 04:30:18PM +0100, Stefan Hajnoczi wrote:
On Thu, Jun 16, 2011 at 11:52:43AM -0300, Marcelo Tosatti wrote:
Th
Am 19.06.2011 um 17:46 schrieb Avi Kivity:
On 06/19/2011 06:22 PM, Stefan Hajnoczi wrote:
I wonder if this will break non-Linux platforms. Perhaps buffer an
entire row of pixels instead and only fwrite(3) at the end of the
outer loop.
That's how I wrote this in the first place. Since the co
On 06/19/2011 06:22 PM, Stefan Hajnoczi wrote:
I wonder if this will break non-Linux platforms. Perhaps buffer an
entire row of pixels instead and only fwrite(3) at the end of the
outer loop.
That's how I wrote this in the first place. Since the consensus is
against these functions, I'll sub
On Sun, Jun 19, 2011 at 3:54 PM, Avi Kivity wrote:
> When running kvm-autotest, fputc() is often the second highest (sometimes #1)
> function showing up in a profile. This is due to fputc() locking the file
> for every byte written.
>
> Optimize by using fputc_unlocked(). Since the file is local
On Sun, 19 Jun 2011, Avi Kivity wrote:
CONFORMING TO
The four functionsgetc_unlocked(), getchar_unlocked(),
putc_unlocked(), putchar_unlocked() are in POSIX.1-2001.
The non-standard *_unlocked() variants occur on a few Unix systems, and
are available in rece
When running kvm-autotest, fputc() is often the second highest (sometimes #1)
function showing up in a profile. This is due to fputc() locking the file
for every byte written.
Optimize by using fputc_unlocked(). Since the file is local to the caller,
clearly no locking is needed. According to t
On Sun, Jun 19, 2011 at 1:04 PM, Andreas Färber wrote:
> Am 18.06.2011 um 22:42 schrieb Blue Swirl:
>
>> On Thu, Jun 16, 2011 at 3:02 AM, Andreas Färber
>> wrote:
>>>
>>> The IBM E15 is equivalent to an S3 Vision864.
>>>
>>> Lacking S3 SDAC (86C716) support, the DAC indizes are translated
>>> to
Add dummy registers for SuperSPARC MXCC MMU counter breakpoints, save
and load all MXCC registers.
Signed-off-by: Blue Swirl
---
v2: add save/load for all MXCC registers, fix width of breakpoint
action register
target-sparc/cpu.h |4 +++-
target-sparc/machine.c | 26 ++
For both working and non-working cases, please provide:
- guest dmesg
- lspci -vvv for each NIC in the guest
- lspci -vvv for each NIC in the host
Also, please test using a fixed address for the device, ex. -device pci-
assign,host=0e:00.1,addr=3 If you reverse the command line order, but
k
On 06/15/2011 04:17 PM, Jan Kiszka wrote:
These FPU states are properly maintained by KVM but not yet by TCG. So
far we unconditionally set them to 0 in the guest which may cause
state corruptions, though not with modern guests.
To avoid breaking backward migration, use a conditional subsection
Andreas Färber a écrit :
Am 18.06.2011 um 22:42 schrieb Blue Swirl:
Please take a look at tcx.c for a 8 bit mode frame buffer with palette
translation. Also VGA_DIRTY bit handling should be added to this loop
to speed it up.
Will look into it.
I doubt this is causing the long delays though.
*
On 06/17/2011 05:00 PM, Joerg Roedel wrote:
To let the user configure the desired tsc frequency for the
guest if running in KVM.
@@ -704,6 +705,14 @@ static int cpu_x86_find_by_name(x86_def_t *x86_cpu_def,
const char *cpu_model)
} else if (!strcmp(featurestr, "model_id")) {
Am 18.06.2011 um 22:42 schrieb Blue Swirl:
On Thu, Jun 16, 2011 at 3:02 AM, Andreas Färber > wrote:
The IBM E15 is equivalent to an S3 Vision864.
Lacking S3 SDAC (86C716) support, the DAC indizes are translated
to greyscale colors. This works sufficiently to observe firmware
boot progress.
Cc:
The following changes since commit 0b862cedf36d927818c50584ddd611b0370673df:
configure: Detect and don't try to use older libcurl (2011-06-13 21:16:27
+0200)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/mst/qemu.git for_anthony
Alex Williamson (1):
From: Jan Kiszka
This fixes e.g. '-vga none -monitor vc:120Cx50C'.
Signed-off-by: Jan Kiszka
---
console.c |9 -
1 files changed, 8 insertions(+), 1 deletions(-)
diff --git a/console.c b/console.c
index 9c6addf..acd8ca1 100644
--- a/console.c
+++ b/console.c
@@ -1349,8 +1349,15 @@
60 matches
Mail list logo
