Xilinx zynq-7000 machine model. Also includes device model for the zynq-specific
system level control register (SLCR) module.
Signed-off-by: Peter A. G. Crosthwaite
---
changes since v2:
removed 2 compile warnings from zynq_arm_sysctl.c
changes since v1:
Added gem init function
remowed WDT instan
Device model for cadence gem ethernet controller.
Signed-off-by: Peter A. G. Crosthwaite
Signed-off-by: John Linn
---
changes from v1:
removed global init function
marked vmsd as unmigratable
cleaned up debug messages
Makefile.target |1 +
hw/cadence_gem.c | 1229 +
Implemented cadence Triple Timer Counter (TCC)
Signed-off-by: Peter A. G. Crosthwaite
Signed-off-by: John Linn
---
changed from v2
changed ptimer to QEMUTimer (Fixed skew/drift issue in timer delays)
changes from v1
refactored event driven code
marked vmsd as unmigratable
Makefile.target |
Implemented cadence UART serial controller
Signed-off-by: Peter A. G. Crosthwaite
Signed-off-by: John Linn
---
changes from v1:
converted register file to array
added vmsd state save/load support
removed read side effects from CISR register
Makefile.target |1 +
hw/cadence_uart.c | 561
This is a suite of Device models and a machine model for the Xilinx Zynq-7000
Extensible Processing Platform:
http://www.xilinx.com/products/silicon-devices/epp/zynq-7000/index.htm
This is an ARM based platform featuring embedded SoC peripherals. This patch
series includes a minimal set of devi
Am 10.02.2012 10:51, schrieb Jay Foad:
> On 30 Oct 2011, Avi Kivity wrote:
>> The memory API supports 64-bit buses (e.g. PCI). A size on such a bus cannot
>> be represented with a 64-bit data type, if both 0 and the entire address
>> space size are to be represented. Futhermore, any address arith
On Sat, 11 Feb 2012, Andreas F?rber wrote:
> Am 10.02.2012 11:26, schrieb ???:
> > On Fri, Feb 10, 2012 at 08:14:41AM +, Stefan Hajnoczi wrote:
> >> On Thu, Feb 09, 2012 at 06:33:16PM +0800, ??? wrote:
> >>> I am running a tiny OS on QEMU and debugging it with gdbstub. The tiny OS
> >>> will
> Am 11.02.2012 00:25, schrieb Paul Brook:
> >> ii) Some tracing of mine indicates QEMU has a highly dynamic memory
> >> usage during runtime, be it due to network layer, block layer or
> >> whatever exactly.
> >
> > We do? Significant compared to the size of guest ram? That sounds like a
> > bug
Am 10.02.2012 11:26, schrieb 陳韋任:
> On Fri, Feb 10, 2012 at 08:14:41AM +, Stefan Hajnoczi wrote:
>> On Thu, Feb 09, 2012 at 06:33:16PM +0800, 陳韋任 wrote:
>>> I am running a tiny OS on QEMU and debugging it with gdbstub. The tiny OS
>>> will
>>> fork process 1, 2, ... and so on. I want to follow
Am 11.02.2012 00:25, schrieb Paul Brook:
>> ii) Some tracing of mine indicates QEMU has a highly dynamic memory
>> usage during runtime, be it due to network layer, block layer or
>> whatever exactly.
>
> We do? Significant compared to the size of guest ram? That sounds like a bug.
Attached is a
> I am running this tiny OS on QEMU then using GDB to connect it.
>
> I want to follow task 1 after the forking, but it seems that GDB
> stick with task 0 and cannot follow task 1 even I do `set follow-fork-mode
> child`.
You have exactly one CPU. That's what the qemu GDB stub exposes. Multipl
> +#ifdef CONFIG_SLIRP
> +static inline void slirp_update_timeout(uint32_t *timeout)
> +{
> +*timeout = MIN(1000, *timeout);
> +}
> +#else
> +static inline void slirp_update_timeout(uint32_t *timeout) { }
> +#endif
Shouldn't we be testing whether slirp is actually in use? I doubt many people
> There's two slightly different scenarios to consider here:
>
> i) User specifies command line options that cannot possibly work.
> => Ideally, yeah, we should just provide an understandable error message
> and exit with error code.
>
> ii) Some tracing of mine indicates QEMU has a highly dynami
> > Make MIPS N32 be a variant of mips64, rather than a new architecture in
> > its own right. This matches how sparc/ppc work, and makes TARGET_MIPS64
> > the right thing to check for a 64-bit mips core.
> >
> > As side-effect of this is that linux-user/mipsn32 gets merged into
> > linux-user/mi
A "linux-user: " prefix for the subject would be nice, but let's wait
for other comments first.
Am 09.02.2012 20:05, schrieb Paul Brook:
> host_to_target_stat64 is used for both stat64 and newfstatat syscalls.
> In the latter case we don't actually have a struct stat64.
> The current TARGET_ABI_BI
Am 10.02.2012 12:23, schrieb Paul Brook:
> Make MIPS N32 be a variant of mips64, rather than a new architecture in
> its own right. This matches how sparc/ppc work, and makes TARGET_MIPS64
> the right thing to check for a 64-bit mips core.
>
> As side-effect of this is that linux-user/mipsn32 get
Am 09.02.2012 17:49, schrieb Peter Maydell:
> On 9 February 2012 14:20, Andreas Färber wrote:
>> Replace device_init() with generalized type_init().
>>
>> While at it, unify naming convention: type_init([$prefix_]register_types)
>> Also, type_init() is a function, so add preceding blank line where
Remove the OpenBSD workaround for the curses probe. This has not been
necessary for 5 releases now.
Signed-off-by: Brad Smith
diff --git a/configure b/configure
index 763db24..c9729f8 100755
--- a/configure
+++ b/configure
@@ -1900,9 +1900,6 @@ if test "$curses" != "no" ; then
curses_found=no
Sounds like your compiler may be to blame. Can you try on a different
system? Or maybe try rebuilding your compiler without any *CFLAGS set?
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/928204
Title
On 02/10/2012 08:39 PM, Paolo Bonzini wrote:
1. added commands blockdev-tray-open, blockdev-tray-close,
blockdev-medium-insert,
blockdev-medium-remove
I think this slightly overengineering. eject and change work well
enough, we do not need blockdev-medium-insert and blockdev-medium-remov
Get the file-descriptor from 'cur_mon', will be used by the QAPI
converted migration command.
Signed-off-by: Anthony Liguori
Signed-off-by: Luiz Capitulino
---
monitor.c |5 +
monitor.h |1 +
2 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/monitor.c b/monitor.c
index
Signed-off-by: Luiz Capitulino
---
error.c | 12
error.h |5 +
2 files changed, 17 insertions(+), 0 deletions(-)
diff --git a/error.c b/error.c
index 990050f..d85e136 100644
--- a/error.c
+++ b/error.c
@@ -43,6 +43,18 @@ void error_set(Error **errp, const char *fmt, ...)
On 02/10/2012 06:55 PM, Kevin Wolf wrote:
Am 10.02.2012 18:04, schrieb Luiz Capitulino:
This reminds me about an earlier try where I did the following, iirc:
1. added commands blockdev-tray-open, blockdev-tray-close,
blockdev-medium-insert,
blockdev-medium-remove
I think this slightly o
The migrate command is one of those commands where HMP and QMP completely
mix up together. This made the conversion to the QAPI (which separates the
command into QMP and HMP parts) a bit difficult.
The first important change to be noticed is that this commit completes the
removal of the Monitor ob
The Monitor object is passed back and forth within the migration/savevm
code so that it can print errors and progress to the user.
However, that approach assumes a HMP monitor, being completely invalid
in QMP.
This commit drops almost every single usage of the Monitor object, all
monitor_printf()
Signed-off-by: Luiz Capitulino
---
check-qdict.c | 29 +
qdict.c | 18 ++
qdict.h |1 +
3 files changed, 48 insertions(+), 0 deletions(-)
diff --git a/check-qdict.c b/check-qdict.c
index fc0d276..12c73ab 100644
--- a/check-qdict.c
+
The new errors are QERR_MIGRATION_ACTIVE and QERR_MIGRATION_NOT_SUPPORTED,
which are going to be used by the QAPI converted migration command.
Signed-off-by: Luiz Capitulino
---
qerror.c |8
qerror.h |6 ++
2 files changed, 14 insertions(+), 0 deletions(-)
diff --git a/qerr
This is a rebase of Anthony's conversion, from his glib branch; and this is
also the beginning of the conversion of complex commands to the qapi.
There are two important changes that should be observed:
1. patch 5/6 purges the 'mon' object from migration code. One of the
consequences is that
As we have thread-local cpu_single_env now and KVM uses exactly one
thread per VCPU, we can drop the cpu_single_env updates from the loop
and initialize this variable only once during setup.
Signed-off-by: Jan Kiszka
---
cpus.c|1 +
kvm-all.c |5 -
2 files changed, 1 insertions(+
In order to perform critical manipulations on the VM state in the
context of a VCPU, specifically code patching, stopping and resuming of
all VCPUs may be necessary. resume_all_vcpus is already compatible, now
enable pause_all_vcpus for this use case by stopping the calling context
before starting
Use OPTION_ROM_START/END from the common header file, add comment to
init code.
Signed-off-by: Jan Kiszka
---
pc-bios/optionrom/kvmvapic.S | 18 --
1 files changed, 8 insertions(+), 10 deletions(-)
diff --git a/pc-bios/optionrom/kvmvapic.S b/pc-bios/optionrom/kvmvapic.S
index
This will allow the APIC core to file a TPR access report. Depending on
the accelerator and kernel irqchip mode, it will either be delivered
right away or queued for later reporting.
In TCG mode, we can restart the triggering instruction and can therefore
forward the event directly. KVM does not a
This enables acceleration for MMIO-based TPR registers accesses of
32-bit Windows guest systems. It is mostly useful with KVM enabled,
either on older Intel CPUs (without flexpriority feature, can also be
manually disabled for testing) or any current AMD processor.
The approach introduced here is
This imports and builds the original VAPIC option ROM of qemu-kvm.
Its interaction with QEMU is described in the commit that introduces the
corresponding device model.
Signed-off-by: Jan Kiszka
---
.gitignore |1 +
Makefile |2 +-
pc-bios/optionrom/M
The CH registers is only written, never read. So we can remove these
operations and, in case of up_set_tpr, also the ECX push/pop.
Signed-off-by: Jan Kiszka
---
pc-bios/optionrom/kvmvapic.S |6 +-
1 files changed, 1 insertions(+), 5 deletions(-)
diff --git a/pc-bios/optionrom/kvmvapic.S
On 02/10/2012 07:41 AM, Daniel P. Berrange wrote:
>> @@ -80,7 +80,7 @@ void *qemu_oom_check(void *ptr)
>> {
>> if (ptr == NULL) {
>> fprintf(stderr, "Failed to allocate memory: %s\n", strerror(errno));
>> -abort();
>> +exit(EXIT_FAILURE);
>
> exit() will call any at
Always add a byte before the final 512-bytes alignment to reserve the
space for the ROM checksum.
Signed-off-by: Jan Kiszka
---
pc-bios/optionrom/optionrom.h |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/pc-bios/optionrom/optionrom.h b/pc-bios/optionrom/optionrom.h
i
Here is v2 of the TPR access optimization. Changes:
- plug race between patching and running VCPUs accessing the same TPR
instruction by stopping VCPUs during patch process
- realized forward/backward check in evaluate_tpr_instruction via a
table but kept patch_instruction as is (too much v
On 02/10/2012 12:08 PM, Kevin Wolf wrote:
Am 10.02.2012 18:22, schrieb Anthony Liguori:
On 02/10/2012 11:09 AM, Amit Shah wrote:
On (Fri) 10 Feb 2012 [07:28:19], Anthony Liguori wrote:
On 02/10/2012 07:19 AM, Amit Shah wrote:
Instead of passing each handler in the qemu_add_handlers() function
Am 10.02.2012 18:22, schrieb Anthony Liguori:
> On 02/10/2012 11:09 AM, Amit Shah wrote:
>> On (Fri) 10 Feb 2012 [07:28:19], Anthony Liguori wrote:
>>> On 02/10/2012 07:19 AM, Amit Shah wrote:
Instead of passing each handler in the qemu_add_handlers() function,
create a struct of handlers
Am 10.02.2012 18:04, schrieb Luiz Capitulino:
> This reminds me about an earlier try where I did the following, iirc:
>
> 1. added commands blockdev-tray-open, blockdev-tray-close,
> blockdev-medium-insert,
> blockdev-medium-remove
> 2. added the events: BLOCK_TRAY_OPEN, BLOCK_TRAY_CLOSE, B
On 02/09/2012 10:02 PM, malc wrote:
On Fri, 10 Feb 2012, Evgeny Voevodin wrote:
On 02/09/2012 06:59 PM, Andreas F?rber wrote:
Disable warnings for spaces before opening parenthesis in
hw/{ac97,adlib,cs4231a,es1370,gus,sb16}.c.
Why audio files are such a special thing?
Because they are cons
On Fri, 10 Feb 2012 12:30:21 +0100
Gerd Hoffmann wrote:
> On 02/07/12 15:38, Daniel P. Berrange wrote:
> > From: "Daniel P. Berrange"
> >
> > This is a followup to
> >
> > http://patchwork.ozlabs.org/patch/121004/
> >
> > With the acceptance of some new APIs to libspice-server.so it
> > is
On Tue, 7 Feb 2012 14:38:40 +
"Daniel P. Berrange" wrote:
> From: "Daniel P. Berrange"
>
> This is a followup to
>
> http://patchwork.ozlabs.org/patch/121004/
>
> With the acceptance of some new APIs to libspice-server.so it
> is possible to add support for SPICE to the 'add_client'
>
This allows us to use ipv4/ipv6 for migration addresses.
Once there, it also uses /etc/services names (it came free).
Signed-off-by: Juan Quintela
Signed-off-by: Amos Kong
---
migration-tcp.c | 60 ---
net.c | 108
On 02/10/2012 11:09 AM, Amit Shah wrote:
On (Fri) 10 Feb 2012 [07:28:19], Anthony Liguori wrote:
On 02/10/2012 07:19 AM, Amit Shah wrote:
Instead of passing each handler in the qemu_add_handlers() function,
create a struct of handlers that can be passed to the function instead.
Signed-off-by:
On Fri, 10 Feb 2012 10:27:21 +0100
Markus Armbruster wrote:
> Luiz Capitulino writes:
>
> > On Thu, 09 Feb 2012 16:01:21 +0100
> > Markus Armbruster wrote:
> >
> >> Luiz Capitulino writes:
> >>
> >> > I've tried to implement a BLOCK_MEDIUM_EJECT event that, as we
> >> > discussed[1],
> >> >
Remove use of parse_host_port.
More SO_SOCKADDR changes.
Signed-off-by: Juan Quintela
Signed-off-by: Amos Kong
---
net/socket.c | 60 +++---
1 files changed, 11 insertions(+), 49 deletions(-)
diff --git a/net/socket.c b/net/socket.c
index d
Those four patches make migration of IPv6 address work.
Use getaddrinfo() to socket addresses infomation.
---
Amos Kong (4):
Use getaddrinfo for migration
net/socket: allow ipv6 for net_socket_listen_init and socket_connect_init
net: split hostname and service by last colon
On Thu, 9 Feb 2012 13:26:40 -0600
Adam Litke wrote:
> On Wed, Feb 08, 2012 at 06:30:40PM -0200, Luiz Capitulino wrote:
> > This commit adds a QMP API for the guest provided memory statistics
> > (long disabled by commit 07b0403dfc2b2ac179ae5b48105096cc2d03375a).
> >
> > The approach taken by the
On (Fri) 10 Feb 2012 [07:28:19], Anthony Liguori wrote:
> On 02/10/2012 07:19 AM, Amit Shah wrote:
> >Instead of passing each handler in the qemu_add_handlers() function,
> >create a struct of handlers that can be passed to the function instead.
> >
> >Signed-off-by: Amit Shah
>
> Why?
It's a goo
On Fri, 10 Feb 2012 10:36:06 +0100
Markus Armbruster wrote:
> Paolo Bonzini writes:
>
> > On 02/09/2012 04:01 PM, Markus Armbruster wrote:
> >> Your GUEST_MEDIUM_EJECTED does*not* track my open<-> closed. I think
> >> it's more complex than a straight open<-> closed event. Evidence: your
>
On Fri, 10 Feb 2012, Jan Kiszka wrote:
> On 2012-02-10 12:18, Paolo Bonzini wrote:
> > On 02/10/2012 12:19 PM, Stefano Stabellini wrote:
> >> I think you are right and the right thing to do would be blocking
> >> indefinitely.
> >> However if slirp doesn't support it, we could have a timeout of 100
Hi,
Recently I observed performance regression regarding virtio-blk,
especially different IO bandwidths between qemu-kvm 0.14.1 and 1.0.
So I want to share the benchmark results, and ask you what the reason
would be.
1. Test condition
- On host, ramdisk-backed block device (/dev/ram0)
- qemu-k
On Fri, Feb 10, 2012 at 08:14:41AM +, Stefan Hajnoczi wrote:
> On Thu, Feb 09, 2012 at 06:33:16PM +0800, 陳韋任 wrote:
> > The question is not so related to QEMU itself, but I want to give it a
> > try.
> > I am running a tiny OS on QEMU and debugging it with gdbstub. The tiny OS
> > will
> >
That method of representing an IPv6 address with a port is
discouraged because of its ambiguity. Referencing to RFC5952,
the recommended format is:
[2312::8274]:5200
test status: Successed
listen side: qemu-kvm -incoming tcp:[2312::8274]:5200
client side: qemu-kvm ...
(qemu
IPv6 address contains colons, parse will be wrong.
[2312::8274]:5200
Signed-off-by: Amos Kong
---
net.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/net.c b/net.c
index f63014c..9e1ef9e 100644
--- a/net.c
+++ b/net.c
@@ -84,7 +84,7 @@ static int get_str_sep(char
Am 10.02.2012 11:36, schrieb Stefan Hajnoczi:
> On Fri, Feb 10, 2012 at 10:35 AM, Stefan Hajnoczi wrote:
>> On Fri, Feb 10, 2012 at 9:47 AM, Zhi Yong Wu wrote:
>>> Today i tried to create one VM with the option "-m 4000", and found it
>>> failed with the following errors:
>>>
>>> Failed to alloca
Am 10.02.2012 12:23, schrieb Zhi Yong Wu:
> On Fri, Feb 10, 2012 at 7:10 PM, Stefan Hajnoczi wrote:
>> On Fri, Feb 10, 2012 at 11:00 AM, Zhi Yong Wu wrote:
>>> On Fri, Feb 10, 2012 at 6:35 PM, Stefan Hajnoczi wrote:
On Fri, Feb 10, 2012 at 9:47 AM, Zhi Yong Wu wrote:
> Today i tried to
Am 10.02.2012 16:13, schrieb Zhi Yong Wu:
On Fri, Feb 10, 2012 at 10:41 PM, Daniel P. Berrange
wrote:
On Fri, Feb 10, 2012 at 10:34:13PM +0800, Zhi Yong Wu wrote:
From: Zhi Yong Wu
Signed-off-by: Zhi Yong Wu
---
oslib-posix.c |4 ++--
oslib-win32.c |4 ++--
2 files changed, 4 inser
On 02/10/2012 03:56 PM, Markus Armbruster wrote:
Paolo Bonzini writes:
On 02/10/2012 01:49 PM, Markus Armbruster wrote:
With manage_door off:
* cdrom_lock_medium() does nothing. Thus, guest OS operating the
virtual door lock does not affect the physical door.
* cdrom_eject() does nothing
From: Zhi Yong Wu
Signed-off-by: Zhi Yong Wu
---
oslib-posix.c |4 ++--
oslib-win32.c |2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/oslib-posix.c b/oslib-posix.c
index b6a3c7f..3b7cd9d 100644
--- a/oslib-posix.c
+++ b/oslib-posix.c
@@ -80,7 +80,7 @@ void *qemu_oo
On Fri, Feb 10, 2012 at 10:41 PM, Daniel P. Berrange
wrote:
> On Fri, Feb 10, 2012 at 10:34:13PM +0800, Zhi Yong Wu wrote:
>> From: Zhi Yong Wu
>>
>> Signed-off-by: Zhi Yong Wu
>> ---
>> oslib-posix.c | 4 ++--
>> oslib-win32.c | 4 ++--
>> 2 files changed, 4 insertions(+), 4 deletions(-)
Paolo Bonzini writes:
> On 02/10/2012 01:49 PM, Markus Armbruster wrote:
>> With manage_door off:
>>
>> * cdrom_lock_medium() does nothing. Thus, guest OS operating the
>> virtual door lock does not affect the physical door.
>>
>> * cdrom_eject() does nothing. Thus, guest OS moving the virtua
Signed-off-by: Lluís Vilanova
---
scripts/tracetool.py | 26 --
trace/stderr.c | 35 +++
2 files changed, 15 insertions(+), 46 deletions(-)
diff --git a/scripts/tracetool.py b/scripts/tracetool.py
index 98a8ca8..a3135b8 100755
---
On Fri, Feb 10, 2012 at 10:34:13PM +0800, Zhi Yong Wu wrote:
> From: Zhi Yong Wu
>
> Signed-off-by: Zhi Yong Wu
> ---
> oslib-posix.c |4 ++--
> oslib-win32.c |4 ++--
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/oslib-posix.c b/oslib-posix.c
> index b6a3c7f..f97
On Fri, Feb 10, 2012 at 10:08 PM, Stefan Hajnoczi wrote:
> On Fri, Feb 10, 2012 at 11:53 AM, Zhi Yong Wu wrote:
>> By the way, need we still to do one clean failure exit?
>
> I find the abort(3) usage odd but it's not critical to change it.
> Since we printed an error message I would have expecte
On 02/09/2012 06:54 PM, carlo.bra...@libero.it wrote:
Hello, in your opinion, is it possible to add some #ifdef and two
options to configure script for activating/deactivating the
compilation of supported video and ethernet interfaces, in a similar
manner it had been done with audio cards and "--
From: Zhi Yong Wu
Signed-off-by: Zhi Yong Wu
---
oslib-posix.c |4 ++--
oslib-win32.c |4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/oslib-posix.c b/oslib-posix.c
index b6a3c7f..f978d56 100644
--- a/oslib-posix.c
+++ b/oslib-posix.c
@@ -80,7 +80,7 @@ void *qemu_
Signed-off-by: Lluís Vilanova
---
monitor.c | 15 ---
1 files changed, 12 insertions(+), 3 deletions(-)
diff --git a/monitor.c b/monitor.c
index 5e099bd..010f659 100644
--- a/monitor.c
+++ b/monitor.c
@@ -617,10 +617,19 @@ static void do_trace_event_set_state(Monitor *mon, const
Q
Signed-off-by: Lluís Vilanova
---
scripts/tracetool.py | 29 ++---
trace/simple.c | 33 ++---
trace/simple.h |6 +-
trace/stderr.h | 11 ---
4 files changed, 17 insertions(+), 62 deletions(-)
delete mode
Signed-off-by: Lluís Vilanova
---
trace/default.c |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/trace/default.c b/trace/default.c
index c9b27a2..12db18d 100644
--- a/trace/default.c
+++ b/trace/default.c
@@ -18,11 +18,10 @@ void trace_print_events(FILE *stream, fprint
This interface decouples event obtention from interaction.
Event can be obtained through three different methods:
* identifier
* name
* simple wildcard pattern
Signed-off-by: Lluís Vilanova
---
docs/tracing.txt | 41 +
trace/control-internal.h | 60 +++
Uses tracetool to generate a backend-independent tracing event description.
Signed-off-by: Lluís Vilanova
---
Makefile |5 +++
Makefile.objs | 15 +++-
scripts/tracetool.py | 94
trace/event-internal.h | 28 +
NOTE: Applies on top of the tracetool-handling trivial changes.
Provides a generic event state description and a more detailed event control and
query interface.
Signed-off-by: Lluís Vilanova
---
Changes in v3:
* Add some assertions.
* Remove debugging printf's.
* Improve documentation.
* M
On 02/10/2012 01:51 PM, Markus Armbruster wrote:
> We need to use O_EXCL in order to suppress event generation in the
> kernel. However, O_EXCL by definition fails when the CD-ROM drive
> is mounted. Automatically unmount it when it is passed through to
> the guest.
I'm not sure automatic unmo
On 02/10/2012 03:01 PM, Stefan Hajnoczi wrote:
The virtio-scsi portions look good.
Thanks, I'll rebase and resubmit!
Paolo
On Fri, Feb 10, 2012 at 11:53 AM, Zhi Yong Wu wrote:
> By the way, need we still to do one clean failure exit?
I find the abort(3) usage odd but it's not critical to change it.
Since we printed an error message I would have expected
exit(EXIT_FAILURE), not abort(3) which is usually used for stran
On Mon, Jan 16, 2012 at 4:30 PM, Paolo Bonzini wrote:
> Here is v2 (three counting the RFC) of the virtio-scsi driver.
> Unlike previous versions, it includes migration support.
>
> The only change in the spec has been s/UNDERRUN/OVERRUN/g.
>
> Paolo Bonzini (13):
> dma-helpers: make QEMUSGList t
On 02/10/2012 01:49 PM, Markus Armbruster wrote:
With manage_door off:
* cdrom_lock_medium() does nothing. Thus, guest OS operating the
virtual door lock does not affect the physical door.
* cdrom_eject() does nothing. Thus, guest OS moving the virtual tray
does not affect the physical tr
Add usb bus op which is called whenever a usb endpoint becomes ready,
so the host adapter emulation can react on that event.
Signed-off-by: Gerd Hoffmann
---
hw/usb.c |4
hw/usb.h |1 +
2 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/hw/usb.c b/hw/usb.c
index 0572547.
Signed-off-by: Lluís Vilanova
---
scripts/tracetool.py | 192 --
1 files changed, 91 insertions(+), 101 deletions(-)
diff --git a/scripts/tracetool.py b/scripts/tracetool.py
index b185724..cd1c29d 100755
--- a/scripts/tracetool.py
+++ b/scripts/tr
From: Ronnie Sahlberg
This patch adds configuration variables for iSCSI to set
initiator-name to use when logging in to the target,
which type of header-digest to negotiate with the target
and username and password for CHAP authentication.
This allows specifying a initiator-name either from the
Add usb_find_device(). This function will check whenever a device with
a specific address is connected to the specified port. Usually this
will just check state and address of the device hooked up to the port,
but in case of a hub it will ask the hub to check all hub ports for a
matching device.
Signed-off-by: Lluís Vilanova
---
Makefile.objs|6 -
Makefile.target |3
scripts/tracetool.py | 357 --
3 files changed, 230 insertions(+), 136 deletions(-)
diff --git a/Makefile.objs b/Makefile.objs
index 2b68739..941386b 10
The OS is allowed to make the UHCI Controller run in circles. That is
usually done to serve multiple connected USB devices in a robin-round
fashion, so the available USB bandwidth is evenly distributed between
devices.
The uhci emulation handles this in a very poor way though. When it
figures it
Add enum to track the status of USBPackets, use that instead of the
owner pointer to figure whenever a usb packet is currently in flight
or not. Add some more packet status sanity checks. Also rename the
USBEndpoint pointer from "owner" to "ep".
Signed-off-by: Gerd Hoffmann
---
hw/usb-ehci.c |
Signed-off-by: Gerd Hoffmann
---
hw/usb-xhci.c |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/hw/usb-xhci.c b/hw/usb-xhci.c
index b274b80..27bdc2b 100644
--- a/hw/usb-xhci.c
+++ b/hw/usb-xhci.c
@@ -2672,7 +2672,10 @@ static void usb_xhci_init(XHCIState *xhci, DeviceS
Signed-off-by: Lluís Vilanova
---
scripts/tracetool.py | 27 +++
1 files changed, 15 insertions(+), 12 deletions(-)
diff --git a/scripts/tracetool.py b/scripts/tracetool.py
index 1134544..7053a74 100755
--- a/scripts/tracetool.py
+++ b/scripts/tracetool.py
@@ -12,6 +12,
Signed-off-by: Lluís Vilanova
---
Makefile.objs|6 -
Makefile.target |3
scripts/tracetool.py | 357 --
3 files changed, 230 insertions(+), 136 deletions(-)
diff --git a/Makefile.objs b/Makefile.objs
index 2b68739..941386b 10
Add a field to XHCITransfer to correctly keep track of NAK'ed usb
packets. Retry transfers when the endpoint is kicked again. Implement
wakeup_endpoint bus op so we can kick the endpoint when needed.
With this patch applied the emulated hid devices are working correctly
when hooked up to xhci.
Signed-off-by: Lluís Vilanova
---
scripts/tracetool.py | 35 ++-
1 files changed, 10 insertions(+), 25 deletions(-)
diff --git a/scripts/tracetool.py b/scripts/tracetool.py
index cd1c29d..91e7620 100755
--- a/scripts/tracetool.py
+++ b/scripts/tracetool.py
@@ -3
From: Stefan Hajnoczi
Per-request attributes like read/write are currently implemented as bool
fields in the QEDAIOCB struct. This becomes unwiedly as the number of
attributes grows. For example, the qed_aio_setup() function would have
to take multiple bool arguments and at call sites it would
From: MORITA Kazutaka
The co_recv coroutine has two things that will try to enter it:
1. The select(2) read callback on the sheepdog socket.
2. The aio_add_request() blocking operations, including a coroutine
mutex.
This patch fixes it by setting NULL to co_recv before sending data.
I
From: Alexander Graf
bdrv_aio_cancel() can trigger bdrv_aio_flush() which makes all aio
that is currently in flight finish. So what we do is:
port reset
detect ncq in flight
cancel ncq
delete ncq sg list
at which point we have double freed the sg list. Instead, with this
patch we do:
Paolo Bonzini writes:
> We need to use O_EXCL in order to suppress event generation in the
> kernel. However, O_EXCL by definition fails when the CD-ROM drive
> is mounted. Automatically unmount it when it is passed through to
> the guest.
I'm not sure automatic unmount is worth the trouble, a
Implement the find_device callback for the usb hub. It'll loop over all
ports, calling usb_find_device for all enabled ports until it finds a
matching device.
Signed-off-by: Gerd Hoffmann
---
hw/usb-hub.c | 21 +
1 files changed, 21 insertions(+), 0 deletions(-)
diff --gi
Signed-off-by: Lluís Vilanova
---
scripts/tracetool.py | 27 +++
1 files changed, 15 insertions(+), 12 deletions(-)
diff --git a/scripts/tracetool.py b/scripts/tracetool.py
index 1134544..7053a74 100755
--- a/scripts/tracetool.py
+++ b/scripts/tracetool.py
@@ -12,6 +12,
Switch over UHCI to use the new usb_find_device()
function for device lookup.
Signed-off-by: Gerd Hoffmann
---
hw/usb-uhci.c | 45 +++--
1 files changed, 15 insertions(+), 30 deletions(-)
diff --git a/hw/usb-uhci.c b/hw/usb-uhci.c
index 407e0f3..a1f597a
From: Benjamin MARSILI
Signed-off-by: Benjamin MARSILI
Signed-off-by: Stefan Hajnoczi
---
net/socket.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/net/socket.c b/net/socket.c
index d4c2002..0bcf229 100644
--- a/net/socket.c
+++ b/net/socket.c
@@ -664,8 +664,
1 - 100 of 213 matches
Mail list logo