Fix page_check_range() wrap-around check when len=0.
write(1,"",0) on linux-user emulation should be OK, but fails.
This is a regression brought by 376a7909.
This patch fixes it at the last of the calling path shown below,
do_syscall:write -> access_ok() -> page_check_range(),
as linux-kernel d
On Saturday 27 March 2010 07:32:41 Aurelien Jarno wrote:
> On Fri, Mar 26, 2010 at 04:47:51PM -0500, Rob Landley wrote:
> > On Friday 26 March 2010 14:00:00 Aurelien Jarno wrote:
> > > I am pretty fine applying a correct patch if you send a new one.
> >
> > By which you mean rip out the whole #ifde
Signed-off-by: Blue Swirl
---
Makefile.target |2 +-
cpus.c | 773 +++
cpus.h | 18 ++
vl.c| 749 +-
4 files changed, 795 insertions(+), 747 deletions(-)
These will be moved later.
Signed-off-by: Blue Swirl
---
vl.c | 72 +++--
1 files changed, 47 insertions(+), 25 deletions(-)
diff --git a/vl.c b/vl.c
index d9fc0cc..a1bb6e1 100644
--- a/vl.c
+++ b/vl.c
@@ -2945,6 +2945,22 @@ static v
Signed-off-by: Blue Swirl
---
hw/xen_machine_pv.c |3 ---
kvm-all.c |2 --
vl.c|4
3 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/hw/xen_machine_pv.c b/hw/xen_machine_pv.c
index 162f88d..586214d 100644
--- a/hw/xen_machine_pv.c
+++ b/hw/x
Let cpu_exec_init_all() be called from non-CPU code.
Signed-off-by: Blue Swirl
---
cpu-all.h |1 -
qemu-common.h |2 ++
2 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/cpu-all.h b/cpu-all.h
index f281a91..9942d49 100644
--- a/cpu-all.h
+++ b/cpu-all.h
@@ -756,7 +756,6
Allow balloon.h, gdbstub.h and kvm.h to be included from
non-CPU code.
Signed-off-by: Blue Swirl
---
balloon.h |2 --
gdbstub.h | 12 +---
kvm.h |7 ---
3 files changed, 13 insertions(+), 8 deletions(-)
diff --git a/balloon.h b/balloon.h
index c3a1ad3..8c019eb 100644
-
With this set of patches against HEAD, I'm finally able to compile vl.c
only once.
Comments? Objections?
The patches can be found in:
git://repo.or.cz/qemu/blueswirl.git
http://repo.or.cz/r/qemu/blueswirl.git
Blue Swirl (7):
Adjust debug handling
Allow various header files to be included fro
Signed-off-by: Blue Swirl
---
vl.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/vl.c b/vl.c
index 8a73235..a2869b8 100644
--- a/vl.c
+++ b/vl.c
@@ -2920,7 +2920,7 @@ static bool tcg_cpu_exec(void)
if (ret == EXCP_DEBUG) {
gdb_set_stop_cpu(en
Fix to Markus comments.
---
console.h |6 +++
input.c | 119 ++
monitor.c |8
qemu-monitor.hx | 17
4 files changed, 141 insertions(+), 9 deletions(-)
diff --git a/console.h b/console.h
index 7d1940
On 03/26/2010 07:14 PM, Cam Macdonell wrote:
I'm not familiar with the uio internals, but for the interface, an ioctl()
on the fd to assign an eventfd to an MSI vector. Similar to ioeventfd, but
instead of mapping a doorbell to an eventfd, it maps a real MSI to an
eventfd.
uio will neve
pushpara...@gmail.com
On Fri, Mar 26, 2010 at 10:46:14AM +0100, Markus Armbruster wrote:
> > +QEMUPutKbdEntry *qemu_add_kbd_event_handler(QEMUPutKBDEvent *func,
> > +void *opaque,
> > +const char *name)
> > +{
> > +QEMUPutKbdEntr
On Mon, Mar 22, 2010 at 10:28:59AM +0100, Markus Armbruster wrote:
> Cleaner integration of location tracking with qemu-tool.c. Move
> qerror_report() where it belongs.
>
> v2: Remove an assertion that unreachable code can't be reached, at
> Blue Swirl's request. Rebased.
>
> Markus Armbruster
On Thu, Mar 25, 2010 at 10:35:03PM +0100, Juergen Lock wrote:
> Signed-off-by: Juergen Lock
Thanks applied.
> --- a/qemu-timer.c
> +++ b/qemu-timer.c
> @@ -35,6 +35,9 @@
> #include
> #include
> #include
> +#ifdef __FreeBSD__
> +#include
> +#endif
>
> #ifdef __linux__
> #include
>
>
Aurelien Jarno wrote:
> On Tue, Mar 16, 2010 at 11:01:46AM +0300, Michael Tokarev wrote:
>> The error message qemu gives when hugetlbfs is not
>> accessible is cryptic at best:
>>
>> mkstemp: Permission denied
>>
>> Make it a bit more specific instead:
>>
>> unable to create backing store for hu
Hello,
Am 26.03.2010 um 17:06 schrieb Riku Voipio:
> From: Juha Riihimäki
>
> remove help menu, fix fullscreen mode mouse handling, add confirmation
> dialog to quit menu command, add support for undocumented alt-grab and
> ctrl-grab options, add qemu version printout in about panel.
> - fix bu
On Fri, Mar 19, 2010 at 02:21:13PM -0700, Richard Henderson wrote:
> The current default stack limit of 512kB is far too small; a fair
> number of gcc testsuite failures (for all guests) are directly
> attributable to this. Using the -s option in every invocation of
> the emulator is annoying to s
On Sat, Mar 20, 2010 at 02:28:03AM +0530, Rabin Vincent wrote:
> Don't set PAGE_EXEC for XN pages, to avoid a bypass of XN protection
> checking if the page is already in the TLB.
Thanks, applied.
> Signed-off-by: Rabin Vincent
> ---
> target-arm/helper.c | 10 +++---
> 1 files changed, 7
On Fri, Mar 19, 2010 at 11:30:35AM +0100, Paolo Bonzini wrote:
> The period for Win32 timers is very short and always the same
> independent of dynticks, so it's possible that the timer fires
> before qemu_run_all_timers has reset alarm_timer->pending to zero.
> Reset alarm_timer->pending before re
On Sat, Mar 20, 2010 at 03:23:23PM +0900, Ryota Ozaki wrote:
> errx takes the exit status of a process as the first
> argument. Passing errno to it is wrong. Instead the
> patch lets errx take EXIT_FAILURE.
Thanks, applied.
> Signed-off-by: Ryota Ozaki
> ---
> qemu-nbd.c | 34 +++
On Sat, Mar 20, 2010 at 03:23:22PM +0900, Ryota Ozaki wrote:
> bdrv_open may return -errno so we have to check
> if the return value is '< 0', not '== -1'.
Thanks, applied.
> Signed-off-by: Ryota Ozaki
> ---
> qemu-nbd.c |2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --
On Sat, Mar 20, 2010 at 03:23:24PM +0900, Ryota Ozaki wrote:
> - use err(3) instead of errx(3) if errno is available
> to report why failed
> - let fail prior to daemon(3) if opening a nbd file
> is likely to fail after daemonizing to avoid silent
> failure exit
>
> Signed-off-by: Ryota Ozak
On Sat, Mar 20, 2010 at 04:08:38PM +0900, Ryota Ozaki wrote:
> bdrv_open may return -errno so we have to check
> if the return value is '< 0', not '== -1'.
Thanks, applied.
> Signed-off-by: Ryota Ozaki
> ---
> qemu-io.c |2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --g
On Thu, Mar 25, 2010 at 08:58:17AM -0500, Adam Litke wrote:
> Beginning with its introduction, the virtio balloon has had an overflow error
> that causes 'info balloon' to misreport the actual memory size when the
> balloon
> itself becomes larger than 4G. Use a cast when converting dev->actual f
On Wed, Mar 24, 2010 at 11:12:05PM +0800, Amos Kong wrote:
>
> When input some invialid word 'unknowcmd' through QMP port, qemu outputs this
> error message:
> "parse error: invalid keyword `%s'"
> This patch makes qemu output the content of invalid keyword, like:
> "parse error: invalid keyword `
On Fri, Mar 19, 2010 at 12:59:24PM +0800, TeLeMan wrote:
> The "params" is never NULL and the usb hid devices have no params.
This looks plainly wrong. With your patch, usb devices which don't
accept parameters, will accept and ignore them.
What are you trying to fix here?
> Signed-off-by: TeLeM
On Fri, Mar 12, 2010 at 01:52:31PM +0100, Kevin Wolf wrote:
> Now that we output an error message according to the returned error code in
> qemu-img, let's return the real error codes. "Input/output error" for
> everything isn't helpful.
>
> Signed-off-by: Kevin Wolf
Thanks, applied.
> ---
> b
On Tue, Mar 16, 2010 at 07:12:58PM +0100, Christoph Hellwig wrote:
> Fix a few typos in the helptexts for the various aio commands.
This is clearly an improvement, but it introduces a typo (see below).
> Signed-off-by: Christoph Hellwig
>
> Index: qemu/qemu-io.c
> ==
On Tue, Mar 16, 2010 at 11:01:46AM +0300, Michael Tokarev wrote:
> The error message qemu gives when hugetlbfs is not
> accessible is cryptic at best:
>
> mkstemp: Permission denied
>
> Make it a bit more specific instead:
>
> unable to create backing store for hugepages: Permission denied
>
On Wed, Mar 03, 2010 at 03:32:43PM +0100, Jens Osterkamp wrote:
> If the user wants to create a chardev of type socket but forgets to give a
> host= option, qemu_opt_get returns NULL. This NULL pointer is then fed into
> strlen a few lines below without a check which results in a segfault.
> This f
On Fri, Mar 26, 2010 at 04:47:51PM -0500, Rob Landley wrote:
> On Friday 26 March 2010 14:00:00 Aurelien Jarno wrote:
> > > I'm not asking anyone to care about me personally, I'm asking them to
> > > care about specific technical issues. If those issues don't interest
> > > you, they don't interes
This is very much work in progress, it does not compile yet. However,
the only target dependencies remaining are QEMU_OPTION_xxx enums and
the target specific option handling in the option parsing loop. I
haven't got a good plan for that, except maybe QEMUOption structure
could have a field that sp
On Sat, Mar 27, 2010 at 09:20:51AM +0300, coo...@gmail.com wrote:
> My question is : how memory access microoperations are now
> implemented in qemu 0.12.x ?
We heard you the first time. :-)
Cheers,
--
Stuart Brady
[ Sorry for sending twice, forgot to reply to the list ]
On Fri, Mar 26, 2010 at 11:23:30PM +0300, coo...@gmail.com wrote:
> Hello.
> in qemu 0.9.x there was a special file with micro-operations, which
> implemented access to memory. For example for arm architecture it
> was op_mem.h file. I was a
Hello,
I'm having trouble booting from SCSI adapter 53C895a and e.g. INT13h OS
like MS-DOS 6.22.
I downloaded and installed the option ROM with -option-rom 8xx_64.rom:
http://www.lsi.com/DistributionSystem/AssetDocument/files/support/ssp/sdms/Bios/lsi_bios.zip
I'm seeing that Harddisks are in
On 3/26/10, Zachary Amsden wrote:
> For some reason, this uses CONFIG_TARGET_I386 instead of TARGET_I386, so
> the code is dead.
The code is also broken: it references undefined variable 'buf'
instead of 'value'.
Hello,
I'm having problems with QEMU 0.12.3 (and even older versions) that Alt-GR
doesn't work: e.g. Alt-GR q should be @ but is still q.
Tested with e.g. Knoppix Live CD 6.2 and MS-DOS 6.22.
In general I'm having also keyboard problems with
upper/lowercase/Shift state under MS-DOS 6.22 (KEY
On (Fri) Mar 26 2010 [14:52:36], Luiz Capitulino wrote:
> > > My suggestion for the immediate term is to do what we have been doing so
> > > far, ie. call it VIRTIO_SERIAL_ADD. Worst case here is: we add a new way
> > > to group events which requires a new VIRTIO_SERIAL event, in this case we
> >
On 3/27/10, Aurelien Jarno wrote:
> On Fri, Mar 26, 2010 at 11:37:08PM +0200, Blue Swirl wrote:
> > On 3/26/10, Aurelien Jarno wrote:
> > > On Thu, Mar 25, 2010 at 10:24:12PM +0200, Blue Swirl wrote:
> > > > Move CPU specific declarations to a separate file.
> > >
> > >
> > > This basicall
40 matches
Mail list logo
