Hello,
This series lets interested callers ask for an -EAGAIN return from the
chardev backends (only unix and tcp sockets as of now) to implement
their own flow control.
Support for other backend types is easy to add, I'll do that in a
separate series if this is acceptable.
Amit Shah (4):
char
On writing errors, we just returned -1 even if some bytes were already
written out. Ensure we return the number of bytes written before we
return the error (on a subsequent call to qemu_chr_write()).
Signed-off-by: Amit Shah
---
qemu-char.c |9 +++--
1 files changed, 7 insertions(+), 2 d
The initialisation for generic ports and console ports is similar.
Factor out the parts that are the same in a different function that can
be called from each of the initfns.
Signed-off-by: Amit Shah
---
hw/virtio-console.c | 31 ++-
1 files changed, 14 insertions(+
For nonblocking chardevs, we can return -EAGAIN to callers of
qemu_chr_write() and let the caller know that the backend is ready to
accept more data via a callback.
Currently only unix and tcp sockets have the ability to return -EAGAIN
and to provide such a callback.
Update all callers of qemu ch
If the char device we're connected to is overwhelmed with data and it
can't accept any more, signal to the virtio-serial-bus to stop sending
us more data till we tell otherwise.
If the current buffer being processed hasn't been completely written out
to the char device, we have to keep it around a
On Thu, Apr 15, 2010 at 01:32:17AM +0200, Aurelien Jarno wrote:
> My impression is that few persons have interest in linux-user or
> non-KVM targets areas, at least from the development point of view.
> I personally don't care about linux-user code, though I have tried to
> merge patches in this a
On 04/15/2010 02:30 AM, Cam Macdonell wrote:
Sample programs, init scripts and the shared memory server are available
in a
git repo here:
www.gitorious.org/nahanni
Please consider qemu.git/contrib.
Should the compilation be tied into Qemu's regular build with a switch
(e.
On Wed, Apr 14, 2010 at 4:50 PM, Kevin Wolf wrote:
> diff --git a/block/qcow.c b/block/qcow.c
> index c619984..01b1692 100644
> --- a/block/qcow.c
> +++ b/block/qcow.c
[..]
> @@ -169,7 +166,7 @@ static int qcow_open(BlockDriverState *bs, const char
> *filename, int flags)
> qemu_free(s->l2_ca
Jun Koi wrote:
> On Wed, Apr 14, 2010 at 12:28 AM, Jan Kiszka wrote:
>> Alexander Graf wrote:
>>> On 13.04.2010, at 15:36, Jan Kiszka wrote:
>>>
Jun Koi wrote:
> Hi,
>
> I am looking into the singlestep command in monitor interface, and it
> seems that we only take into accoun
Jun Koi a écrit :
> On Wed, Apr 14, 2010 at 12:28 AM, Jan Kiszka wrote:
>> Alexander Graf wrote:
>>> On 13.04.2010, at 15:36, Jan Kiszka wrote:
>>>
Jun Koi wrote:
> Hi,
>
> I am looking into the singlestep command in monitor interface, and it
> seems that we only take into acc
Aurelien Jarno wrote:
> Jun Koi a écrit :
>> On Wed, Apr 14, 2010 at 12:28 AM, Jan Kiszka wrote:
>>> Alexander Graf wrote:
On 13.04.2010, at 15:36, Jan Kiszka wrote:
> Jun Koi wrote:
>> Hi,
>>
>> I am looking into the singlestep command in monitor interface, and it
>>
Jan Kiszka a écrit :
> Aurelien Jarno wrote:
>> Jun Koi a écrit :
>>> On Wed, Apr 14, 2010 at 12:28 AM, Jan Kiszka wrote:
Alexander Graf wrote:
> On 13.04.2010, at 15:36, Jan Kiszka wrote:
>
>> Jun Koi wrote:
>>> Hi,
>>>
>>> I am looking into the singlestep command in
> This series lets interested callers ask for an -EAGAIN return from the
> chardev backends (only unix and tcp sockets as of now) to implement
> their own flow control.
As mentioned previously, I think this is a bad idea. The device has no useful
way of determining when to transmit the rest of t
On (Thu) Apr 15 2010 [13:04:40], Paul Brook wrote:
> > This series lets interested callers ask for an -EAGAIN return from the
> > chardev backends (only unix and tcp sockets as of now) to implement
> > their own flow control.
>
> As mentioned previously, I think this is a bad idea. The device has
Signed-off-by: Stefan Hajnoczi
---
block/qcow2.c | 10 ++
1 files changed, 2 insertions(+), 8 deletions(-)
diff --git a/block/qcow2.c b/block/qcow2.c
index 11ce8d1..9b986ae 100644
--- a/block/qcow2.c
+++ b/block/qcow2.c
@@ -539,14 +539,8 @@ static void run_dependent_requests(QCowL2Meta
Am 15.04.2010 15:11, schrieb Stefan Hajnoczi:
> Signed-off-by: Stefan Hajnoczi
Thanks, applied to the block branch.
Kevin
From: Anthony Liguori
Implement P9_TREMOVE support.
This gets file deletion to work.
[mo...@in.ibm.com: Fix truncate to use the relative path]
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/file-op-9p.h |1 +
hw/virtio-9p-local.c |6 ++
hw/virtio-9p.c
From: Gautham R Shenoy
Currently the commandline to create a virtual-filesystem pass-through between
the guest and the host is as follows:
#qemu -fsdev fstype,id=ID,path=path/to/share \
-device virtio-9p-pci,fsdev=ID,mount_tag=tag \
This patch provides a syntactic short-cut to achieve the
From: Anthony Liguori
Implement P9_TWRITE support.
This gets write to file to work
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/file-op-9p.h |1 +
hw/virtio-9p-local.c |7 +++
hw/virtio-9p.c | 112 +-
From: Anthony Liguori
Don't do anything special for flush.
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/virtio-9p.c | 35 ++-
1 files changed, 2 insertions(+), 33 deletions(-)
diff --git a/hw/virtio-9p.c b/hw/virtio-9p.c
index 94b799
From: Anthony Liguori
Implement P9_TOPEN support.
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/file-op-9p.h |2 +
hw/virtio-9p-local.c | 12
hw/virtio-9p.c | 153 --
3 files changed, 162 insertions
From: Anthony Liguori
Add scatter-gather helper functions.
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/virtio-9p.c | 53 +
1 files changed, 53 insertions(+), 0 deletions(-)
diff --git a/hw/virtio-9p.c b/hw/virtio
From: Anthony Liguori
Implement P9_TCLUNK support.
This patch gets `ls -al` to work.
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/virtio-9p.c | 16 ++--
1 files changed, 14 insertions(+), 2 deletions(-)
diff --git a/hw/virtio-9p.c b/hw/virtio-9p.c
index
From: Anthony Liguori
Implement P9_TVERSION support.
[sripat...@in.ibm.com: Handle unknown 9P versions as per the standards]
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/virtio-9p.c | 15 +--
1 files changed, 13 insertions(+), 2 deletions(-)
diff --git
From: Anthony Liguori
Implement P9_TWALK support.
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/virtio-9p.c | 173 +++-
1 files changed, 171 insertions(+), 2 deletions(-)
diff --git a/hw/virtio-9p.c b/hw/virtio-9p.c
From: Anthony Liguori
Add helpers to obtain file stat and mode details.
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/virtio-9p.c | 186
1 files changed, 186 insertions(+), 0 deletions(-)
diff --git a/hw/virtio
From: Anthony Liguori
Implement P9_TCREATE support.
[jv...@linux.vnet.ibm.com: strdup to qemu_strdup conversion]
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/file-op-9p.h |8 ++
hw/virtio-9p-local.c | 87 +
hw/virtio-9p.c | 264 ++
From: Anthony Liguori
Implement P9_TSTAT support. This get the mount to work on the guest.
[ki...@linux.vnet.ibm.com: malloc to qemu_malloc conversion]
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/virtio-9p.c | 55 +++
From: Anthony Liguori
Helper APIs for FID and QID management.
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/virtio-9p.c | 123
1 files changed, 123 insertions(+), 0 deletions(-)
diff --git a/hw/virtio-9p.c b/hw
From: Anthony Liguori
Implement P9_TWSTAT support.
This gets file and directory creation to work.
[jv...@linux.vnet.ibm.com: strdup to qemu_strdup conversion]
[aneesh.ku...@linux.vnet.ibm.com: v9fs_fix_path]
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/file-op-9p.h
From: Anthony Liguori
Implement P9_TREAD support.
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/file-op-9p.h |6 ++
hw/virtio-9p-local.c | 37
hw/virtio-9p.c | 234 +-
3 files changed, 275 inser
From: Anthony Liguori
Add helpers to do string manipulation.
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/virtio-9p.c | 127
1 files changed, 127 insertions(+), 0 deletions(-)
diff --git a/hw/virtio-9p.c b/hw/
From: Anthony Liguori
Add helpers to process the PDUs.
[ki...@linux.vnet.ibm.com: malloc to qemu_malloc coversion]
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/virtio-9p-debug.h |7 ++
hw/virtio-9p.c | 243 ++
From: Anthony Liguori
Implement P9_TATTACH support.
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/virtio-9p.c | 33 +++--
1 files changed, 31 insertions(+), 2 deletions(-)
diff --git a/hw/virtio-9p.c b/hw/virtio-9p.c
index ccc45a3..4ab4f9
Hi,
This patch series adds VirtFS to QEMU. VirtFS is code named for a 9P filesystem
server in QEMU enabling paravirtual filesystem pass-trhough between KVM host
and guest.
VirtFS is intended to offer an alternative to using NFS/CIFS to share host
filesystems on
guest and provides better perform
From: Anthony Liguori
Add minimal set of FileOperations and the corresponding implementations for
local fstype. These will be required for the FID management patches later on.
[aneesh.ku...@linux.vnet.ibm.com: rpath fix ]
Signed-off-by: Anthony Liguori
Signed-off-by: Aneesh Kumar K.V
---
hw/f
From: Anthony Liguori
This patch doesn't implement the 9p protocol handling
code. It adds a simple device which dump the protocol data.
[jv...@linux.vnet.ibm.com: Little-Endian to host format conversion]
[aneesh.ku...@linux.vnet.ibm.com: Multiple-mounts support]
Signed-off-by: Anthony Liguori
From: Gautham R Shenoy
This patch creates a new command line option named -fsdev to hold any file
system specific information.
The option will currently hold the following attributes:
-fsdev fstype id=id,path=path_to_share
where
fstype: Type of the file system.
id: Identifier used to refer t
Signed-off-by: Miguel Di Ciurcio Filho
---
net.c |9 -
net.h |2 --
2 files changed, 0 insertions(+), 11 deletions(-)
diff --git a/net.c b/net.c
index 46e8873..5dfe24d 100644
--- a/net.c
+++ b/net.c
@@ -168,15 +168,6 @@ int parse_host_port(struct sockaddr_in *saddr, const char
*
Each device is represented by a QDict. The returned QObject is a QList
of all devices.
This commit slightly changes the monitor output when 'info network' is used.
Old output:
(qemu) info network
VLAN 1 devices:
e1000.0: model=e1000,macaddr=52:54:00:12:34:56
VLAN 0 devices:
tap.0: i
Signed-off-by: Miguel Di Ciurcio Filho
---
hw/xen_nic.c |6 --
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/hw/xen_nic.c b/hw/xen_nic.c
index 08055b8..d51839d 100644
--- a/hw/xen_nic.c
+++ b/hw/xen_nic.c
@@ -43,6 +43,8 @@
#include "net/checksum.h"
#include "net/util.h"
Signed-off-by: Miguel Di Ciurcio Filho
---
net/socket.c | 47 ---
1 files changed, 32 insertions(+), 15 deletions(-)
diff --git a/net/socket.c b/net/socket.c
index 1c4e153..fa5d24d 100644
--- a/net/socket.c
+++ b/net/socket.c
@@ -28,6 +28,10 @@
#inc
Signed-off-by: Miguel Di Ciurcio Filho
---
net/vde.c |9 +++--
1 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/net/vde.c b/net/vde.c
index 0b46fa6..f582878 100644
--- a/net/vde.c
+++ b/net/vde.c
@@ -31,6 +31,9 @@
#include "qemu-char.h"
#include "qemu-common.h"
#include "
Signed-off-by: Miguel Di Ciurcio Filho
---
net/dump.c |8 ++--
1 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/net/dump.c b/net/dump.c
index 6db7ecf..dea7f7d 100644
--- a/net/dump.c
+++ b/net/dump.c
@@ -27,6 +27,9 @@
#include "sysemu.h"
#include "qemu-error.h"
#include "
These are simple one line changes to replace qemu_format_nic_info_str.
Signed-off-by: Miguel Di Ciurcio Filho
---
hw/dp8393x.c|2 +-
hw/e1000.c |2 +-
hw/lan9118.c|4 ++--
hw/mcf_fec.c|2 +-
hw/mipsnet.c|2 +-
hw/ne2000-isa.c |
Signed-off-by: Miguel Di Ciurcio Filho
---
net/tap-win32.c |9 ++---
net/tap.c | 18 +-
2 files changed, 19 insertions(+), 8 deletions(-)
diff --git a/net/tap-win32.c b/net/tap-win32.c
index 74348da..a54cd31 100644
--- a/net/tap-win32.c
+++ b/net/tap-win32.c
@@ -3
Signed-off-by: Miguel Di Ciurcio Filho
---
hw/eepro100.c | 10 --
1 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/hw/eepro100.c b/hw/eepro100.c
index 7db6fb5..457bda8 100644
--- a/hw/eepro100.c
+++ b/hw/eepro100.c
@@ -1978,8 +1978,14 @@ static int nic_init(PCIDevice *pci_
Signed-off-by: Miguel Di Ciurcio Filho
---
net/slirp.c |8 ++--
1 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/net/slirp.c b/net/slirp.c
index b41c60a..eb9d261 100644
--- a/net/slirp.c
+++ b/net/slirp.c
@@ -32,6 +32,9 @@
#include "monitor.h"
#include "sysemu.h"
#include
This is a helper function that converts a QDict to a QString, using the format:
key1=value1 SPACE key2=value2 SPACE key3=value3
Handy for debugging and other things.
Signed-off-by: Miguel Di Ciurcio Filho
---
qdict.c | 41 +
qdict.h |3 +++
2 files
There is no standard format when formatting VLANClientState.info_str,
so it is difficult to extract information and transmit it over QMP.
This patch adds info_dict, a QDict to better handle the information
of a NIC.
Patches that convert the devices to use this new function will follow.
Signed-of
The VLANClientState structure has the member info_str, a simple string that
is filled with information about NIC devices and used on monitor calls.
There is no coherent formatting of this string by all the NIC devices,
making it difficult to parse and represent this information over QMP.
Patch 01
On 04/15/2010 07:06 AM, Miguel Di Ciurcio Filho wrote:
Signed-off-by: Miguel Di Ciurcio Filho
---
hw/eepro100.c | 10 --
1 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/hw/eepro100.c b/hw/eepro100.c
index 7db6fb5..457bda8 100644
--- a/hw/eepro100.c
+++ b/hw/eepro100.c
On 4/15/10, David Munday wrote:
> Hello,
> Our lab just did a pull from the qemu repository and now we are unable to
> run a 32-bit sparc target on a 64-bit host. This functionality was working
> before we did the pull. The error message we get is:mmap: No such device or
> address
>
> And the
On 4/15/10, Paolo Bonzini wrote:
> On 04/14/2010 06:52 PM, Blue Swirl wrote:
>
> > On 4/14/10, Gerd Hoffmann wrote:
> >
> > > +static inline void atomic_or(uint32_t *var, uint32_t add)
> > > +{
> > > + __asm__ __volatile__ ("lock; orl %1, %0" : "+m" (*var) : "r" (add)
> : "memory");
> > > +
On 4/15/10, Artyom Tarasenko wrote:
> 2010/4/15 Artyom Tarasenko :
>
> > One of LX's tests crashes pretty hard, causing qemu abort.
> > I've tried to look how does the execution flow works with -d in_asm.
> > Does the address in the log show the guest's PC register?
>
>
> It's probably sort of a
Hi,
we are facing sporadic latency issue in our guests due to the
synchronous nature of bdrv_check_byte_request on raw images:
#0 0x7ff8962e2070 in lseek64 () from /lib64/libpthread.so.0
#1 0x004d7f97 in raw_getlength (bs=0xcab010) at block/raw-posix.c:704
#2 0x004b9d22 in
I think Paolo's approach can work even with vl.c including kvm.h,
provided that kvm.h remains such that it can be used safely.
The situation is a bit fragile though. Some kind of poisoning,
splitting kvm.h into two files, or other means should be used to make
kvm.h more robust.
Blue Swirl (2):
This allows limited use of kvm functions (which will return ENOSYS)
even in once-compiled modules. The patch also improves a bit the error
messages for KVM initialization.
Signed-off-by: Paolo Bonzini
Signed-off-by: Blue Swirl
---
Makefile.target |2 +
kvm-all.c |6 ++-
kvm-stub.
Since kvm.h can be used in files compiled once,
we can partially revert
b33612d03540fda7fa67485f1c20395beb7a2bf0.
Signed-off-by: Blue Swirl
---
Makefile.objs |2 +-
Makefile.target |2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/Makefile.objs b/Makefile.objs
index
Signed-off-by: Blue Swirl
---
Makefile.objs |2 +-
Makefile.target |1 -
2 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/Makefile.objs b/Makefile.objs
index c0fe5e2..c3864d0 100644
--- a/Makefile.objs
+++ b/Makefile.objs
@@ -104,7 +104,7 @@ common-obj-$(CONFIG_VNC_TLS) +=
On 4/15/10, Gabriel Southern wrote:
> On Thu, Apr 15, 2010 at 9:33 AM, Blue Swirl wrote:
> > On 4/15/10, David Munday wrote:
> >> Hello,
> >> Our lab just did a pull from the qemu repository and now we are unable
> to run a 32-bit sparc target on a 64-bit host. This functionality was workin
On 04/15/2010 11:47 AM, Blue Swirl wrote:
On 4/15/10, Paolo Bonzini wrote:
On 04/14/2010 06:52 PM, Blue Swirl wrote:
On 4/14/10, Gerd Hoffmann wrote:
+static inline void atomic_or(uint32_t *var, uint32_t add)
+{
+ __asm__ __volatile__ ("lock; orl %1, %0" : "+m" (*var) : "r" (add)
On Thu, Apr 15, 2010 at 9:33 AM, Blue Swirl wrote:
> On 4/15/10, David Munday wrote:
>> Hello,
>> Our lab just did a pull from the qemu repository and now we are unable to
>> run a 32-bit sparc target on a 64-bit host. This functionality was working
>> before we did the pull. The error message
2010/4/15 Blue Swirl :
> On 4/15/10, Artyom Tarasenko wrote:
>> 2010/4/15 Artyom Tarasenko :
>>
>> > One of LX's tests crashes pretty hard, causing qemu abort.
>> > I've tried to look how does the execution flow works with -d in_asm.
>> > Does the address in the log show the guest's PC register?
One of LX's tests crashes pretty hard, causing qemu abort.
I've tried to look how does the execution flow works with -d in_asm.
Does the address in the log show the guest's PC register?
If so it looks strange:
0x70002d8c: fadds %f1, %f2, %f3
0x70002d90: st %f3, [ %l4 ]
0x70002d94: nop
0x70002
2010/4/15 Artyom Tarasenko :
> One of LX's tests crashes pretty hard, causing qemu abort.
> I've tried to look how does the execution flow works with -d in_asm.
> Does the address in the log show the guest's PC register?
It's probably sort of a "timing" issue.
Can we check exceptions not just on
(Thanks to Jan for comments on the last patch)
Qemu has a command named singlestep, which reduces the translated code
block to be only one instruction.
However, there is one flaw when this command is triggered via monitor
interface: we do not flush all the current TBs, so we will miss
single-step
Hi,
In the TLB entry, why do we need to maintain separately addr_read,
addr_write and addr_code? Is it impossible to have only one for all
the purposes?
As far as I can see in tlb_set_page(), these fields are mutually
exclusive, as they either enable or get invalid value (-1).
Thanks,
Jun
Hi Jan:
The attached addresses the spinning with the usb net device. Now I can
enable the device and ethtool shows a link:
# ifconfig usb0 up
# ethtool usb0
Settings for usb0:
Current message level: 0x0007 (7)
Link detected: yes
Though dhclient still can't get an address. Aft
On Fri, Apr 16, 2010 at 11:49 AM, Jun Koi wrote:
> Hi,
>
> In the TLB entry, why do we need to maintain separately addr_read,
> addr_write and addr_code? Is it impossible to have only one for all
> the purposes?
>
> As far as I can see in tlb_set_page(), these fields are mutually
> exclusive, as t
Hi,
I am writing a small tool to trace all the activities that write to an
area of (virtual) memory in Qemu.
I am currently doing that by putting my code at the top of the below
macro in softmmu_header.h
static inline void glue(glue(st, SUFFIX), MEMSUFFIX)
However, it seems I still miss some
On Fri, 16 Apr 2010, Jun Koi wrote:
> Hi,
>
> I am writing a small tool to trace all the activities that write to an
> area of (virtual) memory in Qemu.
> I am currently doing that by putting my code at the top of the below
> macro in softmmu_header.h
>
> static inline void glue(glue(st, SUFFIX)
72 matches
Mail list logo
