2010/4/28 陈宇飞 :
> I've tested the latest git HEAD version. But still has the problem.
>
> Investigating Linux kernel execution shows that the function which will clear
> the interrupt is not executed.
>
> Here's the relevant kernel code.
>
> maybe_smp4m_msg:
>GET_PROCESSOR4M_ID(o3)
>
Please send in any agenda items you are interested in covering.
If we have a lack of agenda items I'll cancel the week's call.
thanks,
-chris
Om du har problem med att läsa detta e-postmeddelande, klicka här
(http://www.anp.se/newsletterweb/753296/444059437941455D4B7142445C43) för en
webb-version.
Vårt nyhetsbrev skickas automatiskt till våra kunder och intressenter. Vill du
inte ha detta nyhetsbrev framöver, klicka här för att avpre
From: Kirill A. Shutemov
CCblock/vvfat.o
cc1: warnings being treated as errors
block/vvfat.c: In function 'commit_one_file':
block/vvfat.c:2259: error: ignoring return value of 'ftruncate', declared with
attribute warn_unused_result
make: *** [block/vvfat.o] Error 1
CCblock/vvfat.o
In
Alexander Graf wrote:
> On 22.06.2010, at 00:13, Jan Kiszka wrote:
>
>> Jan Kiszka wrote:
>>> And there is some race that cause a lock up in qemu_mutex_lock_iothread
>>> after a while (the cpu_unlink_tb seems to race with the linking - just a
>>> guess so far).
>> This seems to fix a long-standing
Marcelo Tosatti wrote:
> On Mon, Jun 21, 2010 at 10:58:32PM +0200, Jan Kiszka wrote:
>> Jan Kiszka wrote:
>>> Marcelo Tosatti wrote:
Clear exit_request when iothread grabs the global lock.
Signed-off-by: Marcelo Tosatti
diff --git a/cpu-exec.c b/cpu-exec.c
index 0269
On 2010-6-22, at 下午1:46, Artyom Tarasenko wrote:
> 2010/4/28 陈宇飞 :
>> I've tested the latest git HEAD version. But still has the problem.
>>
>> Investigating Linux kernel execution shows that the function which will
>> clear the interrupt is not executed.
>>
>> Here's the relevant kernel code
Am 21.06.2010 17:34, schrieb Anthony Liguori:
> On 06/21/2010 09:01 AM, Kevin Wolf wrote:
>>
>> No, what I'm saying is that even in your model
>>
>>-blockdev format=qcow2,file=image.qcow2,id=blk1
>>
>> becomes qcow2 -> file automatically, whereas
>>
>>-blockdev format=vvfat,file=/tmp/dir/,
Amit Shah wrote:
> From: Kirill A. Shutemov
>
> CCblock/vvfat.o
> cc1: warnings being treated as errors
> block/vvfat.c: In function 'commit_one_file':
> block/vvfat.c:2259: error: ignoring return value of 'ftruncate', declared
> with attribute warn_unused_result
> make: *** [block/vvfat.o]
Am 21.06.2010 17:56, schrieb Markus Armbruster:
> Kevin Wolf writes:
>
>> Am 21.06.2010 15:37, schrieb Anthony Liguori:
>>> On 06/21/2010 08:30 AM, Kevin Wolf wrote:
Am 21.06.2010 15:09, schrieb Anthony Liguori:
> On 06/21/2010 03:19 AM, Kevin Wolf wrote:
>
>> Am 2
Am 21.06.2010 18:21, schrieb Markus Armbruster:
> Christoph Hellwig writes:
>
>> On Mon, Jun 21, 2010 at 09:51:23AM -0500, Anthony Liguori wrote:
>>> I can appreciate the desire to keep protocols and formats as an internal
>>> distinction but as a user visible concept, I think your two examples
This patch factors out bar offset calculation.
Later the calculation logic will be enhanced.
Signed-off-by: Isaku Yamahata
---
src/pciinit.c | 20 ++--
1 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/src/pciinit.c b/src/pciinit.c
index a65c58d..9109739 100644
-
make pci memory space assignment 64bit aware.
If 64bit memory space is found while assigning pci memory space,
clear higher bit and skip to next bar.
This patch is preparation for q35 chipset initialization which
has 64bit bar.
Signed-off-by: Isaku Yamahata
---
src/pciinit.c | 19
This patch makes pci bar offset calculation pci bridge aware.
The offset of pci bridge rom is different from normal device.
Signed-off-by: Isaku Yamahata
---
src/pciinit.c |6 +-
1 files changed, 5 insertions(+), 1 deletions(-)
diff --git a/src/pciinit.c b/src/pciinit.c
index 9109739..2
initialize pci bridge filtering registers.
Signed-off-by: Isaku Yamahata
---
chages v1 -> v2.
- use c99 initialization.
---
src/pciinit.c | 106 +++--
1 files changed, 103 insertions(+), 3 deletions(-)
diff --git a/src/pciinit.c b/src/pciini
This patch introduces foreachpci_in_bus() helper macro for
depth first recursion. foreachpci() is for width first recursion.
The macro will be used later to initialize pci bridge
that requires depth first recursion.
Signed-off-by: Isaku Yamahata
---
changes v1 -> v2:
- simplified according to Ke
pci bridge bus initialization.
Signed-off-by: Isaku Yamahata
---
src/pciinit.c | 70 +
1 files changed, 70 insertions(+), 0 deletions(-)
diff --git a/src/pciinit.c b/src/pciinit.c
index 23b79bc..d22ee10 100644
--- a/src/pciinit.c
+++ b/s
Make pci bar assigner preferchable memory aware.
This is needed for PCI bridge support because memory space and
prefetchable memory space is filtered independently based on
memory base/limit and prefetchable memory base/limit of pci bridge.
On bus 0, such a distinction isn't necessary so keep exist
factor out pci bar region allocation logic.
Signed-off-by: Isaku Yamahata
---
src/pciinit.c | 84 -
1 files changed, 47 insertions(+), 37 deletions(-)
diff --git a/src/pciinit.c b/src/pciinit.c
index 0556ee2..488c77b 100644
--- a/src/pci
KAWAKATSU Noritaka writes:
> Hi,
Hi,
>
> I have built u-boot binary for 'qemu-system-mips -M mips'.
> It is fine to run ths u-boot binary.
> But the same u-boot binary does not run on 'qemu-system-mips64 -M mips'.
>
> I do not understand what happends on the qemu-mips64 execution.
> Is this a b
Markus Armbruster wrote:
> Jan Kiszka writes:
>
>> Luiz Capitulino wrote:
>>> On Fri, 18 Jun 2010 13:26:27 -0300
>>> Miguel Di Ciurcio Filho wrote:
>>>
These commands show the information about active backend network devices.
Signed-off-by: Miguel Di Ciurcio Filho
---
Changes v1 -> v2:
- simplified foreachpci_in_bus()
- add overlap check during pci bar assignemnt.
- use c99 initialization.
Patch description:
This patch set allows seabios to initialize multi pci bus and 64bit BAR.
Currently seabios is able to initialize only pci root bus.
However multi pci bus
allow qemu to load dsdt as external acpi table.
Signed-off-by: Isaku Yamahata
---
changes v1 -> v2.
- load qemu table first and they try default dsdt table if
qemu doesn't supply dsdt.
Thus unnecessary malloc_high()/free() is avoided.
---
src/acpi.c | 49 ++
On 06/08/2010 02:54 PM, Markus Armbruster wrote:
Restrict IDs to letters, digits, '-', '.', '_', starting with a
letter.
This takes care of '/' in qdev IDs breaking qbus_find().
Broke autotest (used leading numeric char). Patch posted.
--
error compiling committee.c: too many arguments
Thank you for your reply.
> so EPC is set to bfc0ee94 which is :
> movn s1,v1,v0
>
>> From what I understand when reading the mips IV manual, the movn ins is
> only for mips IV / 32bit and not 64bit. Looks like the qemu code has
> been written following this. (This also explains why
Kevin Wolf (2):
qcow2: Fix qemu-img check segfault on corrupted images
qcow2: Don't try to check tables that couldn't be loaded
block/qcow2-refcount.c | 15 ---
1 files changed, 12 insertions(+), 3 deletions(-)
With corrupted images, we can easily get an cluster index that exceeds the
array size of the temporary refcount table.
Signed-off-by: Kevin Wolf
---
block/qcow2-refcount.c | 14 +++---
1 files changed, 11 insertions(+), 3 deletions(-)
diff --git a/block/qcow2-refcount.c b/block/qcow2-
Trying to check them leads to a second error message which is more confusing
than helpful:
Can't get refcount for cluster 0: Invalid argument
ERROR cluster 0 refcount=-22 reference=1
Signed-off-by: Kevin Wolf
---
block/qcow2-refcount.c |1 +
1 files changed, 1 insertions(+), 0 delet
Hi,
This is the v5 of the patch-series to have a generic asynchronous task
offloading framework (called threadlets) within qemu.
V4 can be found here:
http://lists.gnu.org/archive/html/qemu-devel/2010-06/msg02152.html
Changes from V4:
=
From: Aneesh Kumar K.V
This patch creates a generic asynchronous-task-offloading infrastructure named
threadlets. The core idea has been borrowed from the threading framework that
is being used by paio.
The reason for creating this generic infrastructure is so that other subsystems,
such as virt
This patch makes the paio subsystem use the threadlet framework thereby
decoupling asynchronous threading framework portion out of posix-aio-compat.c
The patch has been tested with fstress.
Signed-off-by: Gautham R Shenoy
---
posix-aio-compat.c | 152 ---
On Tue, Jun 22, 2010 at 11:55:35AM +0900, Isaku Yamahata wrote:
> make pci hotplug callback return value to caller.
> And when returning error, allocated resources are freed.
>
> Signed-off-by: Isaku Yamahata
> ---
> hw/pci.c | 15 +++
> 1 files changed, 11 insertions(+), 4 deletio
Jan Kiszka writes:
> Markus Armbruster wrote:
>> Jan Kiszka writes:
>>
>>> Luiz Capitulino wrote:
On Fri, 18 Jun 2010 13:26:27 -0300
Miguel Di Ciurcio Filho wrote:
> These commands show the information about active backend network devices.
>
> Signed-off-by: Miguel D
Chris Wright wrote:
> Please send in any agenda items you are interested in covering.
>
> If we have a lack of agenda items I'll cancel the week's call.
As we have no agenda. This week call gets cancelled.
Thanks, Juan.
> thanks,
> -chris
On 06/16/2010 07:58 PM, Glauber Costa wrote:
It is, and the fix would be to create the APIC memory slot as
sharable across forks (should be easy to fix in the kernel).
Kernel pages are already shared across fork, no?
What are kernel pages in this context?
The APIC access page is a
On Tue, Jun 22, 2010 at 11:55:32AM +0900, Isaku Yamahata wrote:
> pci qdev exit code patch(pci_unregister_device) doesn't release
> all the resource allocated by pci qdev initialization.(pci_qdev_init)
> This patch series sorts it out and make pci::hotplug error code to the caller.
>
> Isaku Yamah
On 06/22/2010 03:10 AM, Kevin Wolf wrote:
Am 21.06.2010 17:34, schrieb Anthony Liguori:
On 06/21/2010 09:01 AM, Kevin Wolf wrote:
No, what I'm saying is that even in your model
-blockdev format=qcow2,file=image.qcow2,id=blk1
becomes qcow2 -> file automatically, whereas
-b
On 06/22/2010 07:04 AM, Juan Quintela wrote:
Chris Wright wrote:
Please send in any agenda items you are interested in covering.
If we have a lack of agenda items I'll cancel the week's call.
As we have no agenda. This week call gets cancelled.
Could we send the agenda note o
From: Jes Sorensen
Hi,
This set of patches fixes building qemu-kvm for non KVM targets, as
reported in
http://sourceforge.net/tracker/?func=detail&atid=893831&aid=2984626&group_id=180599
One of the main problem is that we have a tendency to move things from
Makefile.objs to Makefile.target in o
From: Jes Sorensen
time_drift_fix is only present on x86, not just on !ia64. This allows
the code to build for other archs, like MIPS.
Signed-off-by: Jes Sorensen
---
hw/i8259.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/hw/i8259.c b/hw/i8259.c
index a8ae069..e89
From: Jes Sorensen
This avoids breaking the build for non KVM targets where
kvm_set_irqfd() is not defined.
Signed-off-by: Jes Sorensen
---
hw/virtio-pci.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/hw/virtio-pci.c b/hw/virtio-pci.c
index d497eae..938937e 10064
From: Jes Sorensen
pcspk.o i8254.o acpi.o acpi_piix4.o are all required for MIPS as well,
add them to Makefile.target accordingly.
Signed-off-by: Jes Sorensen
---
Makefile.target |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/Makefile.target b/Makefile.target
index b
On 06/22/2010 01:18 PM, Gautham R Shenoy wrote:
From: Aneesh Kumar K.V
This patch creates a generic asynchronous-task-offloading infrastructure named
threadlets. The core idea has been borrowed from the threading framework that
is being used by paio.
The reason for creating this generic infrast
From: Jes Sorensen
We need to declare 'int no_hpet' for all targets to avoid build
failure on no x86 builds.
Signed-off-by: Jes Sorensen
---
vl.c |2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/vl.c b/vl.c
index 79f91d5..c2e99fc 100644
--- a/vl.c
+++ b/vl.c
@@ -207,9
From: Jes Sorensen
Avoid build conflicts and move prototype out of CONFIG_KVM to make the
stub in kvm-stub.c visible.
Signed-off-by: Jes Sorensen
---
qemu-kvm.h | 29 -
1 files changed, 12 insertions(+), 17 deletions(-)
diff --git a/qemu-kvm.h b/qemu-kvm.h
index
From: Jes Sorensen
This fixes build breakage for target MIPS etc.
Signed-off-by: Jes Sorensen
---
vl.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/vl.c b/vl.c
index 0ee963c..79f91d5 100644
--- a/vl.c
+++ b/vl.c
@@ -2459,6 +2459,7 @@ int main(int argc, char **argv,
From: Jes Sorensen
Signed-off-by: Jes Sorensen
---
qemu-kvm.h |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/qemu-kvm.h b/qemu-kvm.h
index c9b93dc..6dd3a01 100644
--- a/qemu-kvm.h
+++ b/qemu-kvm.h
@@ -835,6 +835,7 @@ void qemu_kvm_cpuid_on_env(CPUState *env);
void kv
Am 22.06.2010 14:39, schrieb Anthony Liguori:
> On 06/22/2010 03:10 AM, Kevin Wolf wrote:
>> Am 21.06.2010 17:34, schrieb Anthony Liguori:
>>
>>> On 06/21/2010 09:01 AM, Kevin Wolf wrote:
>>>
No, what I'm saying is that even in your model
-blockdev format=qcow2,file=ima
Hrm, the way you've sent this patch makes Thunderbird unhappy. It
appears the whole thing is treated as an attachment. In the future, I'd
suggest avoiding the Content-Disposition tag
On 06/21/2010 11:01 PM, Mahesh Salgaonkar wrote:
Qemu userspace tool to filter out guest OS memory from qemu
On Mon, 21 Jun 2010 17:31:28 -0300
Miguel Di Ciurcio Filho wrote:
> These commands show the information about active backend network devices.
>
> Changes from v2
> ---
> - Got rid of vlans, a separate command for them will be created if needed
> - Removed socket as a type (it is used
On 06/22/2010 07:57 AM, Kevin Wolf wrote:
and it will be turned into something sensible automagically (namely
adding a file blockdev underneath and passing the file parameter to that
one), but if you want to change an option, you need to specify both?
-blockdev id=foo,format=qcow2,parent=foo
On Tue, Jun 22, 2010 at 10:06 AM, Luiz Capitulino
wrote:
> On Mon, 21 Jun 2010 17:31:28 -0300
> Miguel Di Ciurcio Filho wrote:
>
>> These commands show the information about active backend network devices.
>>
>> Changes from v2
>> ---
>> - Got rid of vlans, a separate command for them
On Tue, 2010-06-22 at 07:49 -0500, Anthony Liguori wrote:
> On 06/22/2010 07:04 AM, Juan Quintela wrote:
> > Chris Wright wrote:
> >
> >> Please send in any agenda items you are interested in covering.
> >>
> >> If we have a lack of agenda items I'll cancel the week's call.
> >>
> > As w
Alex Williamson wrote:
> On Tue, 2010-06-22 at 07:49 -0500, Anthony Liguori wrote:
>
>> On 06/22/2010 07:04 AM, Juan Quintela wrote:
>>
>>> Chris Wright wrote:
>>>
>>>
Please send in any agenda items you are interested in covering.
If we have a lack of agenda item
Signed-off-by: Huacai Chen
---
Makefile.target |1 +
default-configs/mips64el-softmmu.mak |1 +
hw/bonito.c | 809 ++
hw/mips.h|3 +
4 files changed, 814 insertions(+), 0 deletio
Signed-off-by: Huacai Chen
---
hw/usb-uhci.c | 29 +
hw/usb-uhci.h |1 +
2 files changed, 30 insertions(+), 0 deletions(-)
diff --git a/hw/usb-uhci.c b/hw/usb-uhci.c
index 624d55b..accfe2e 100644
--- a/hw/usb-uhci.c
+++ b/hw/usb-uhci.c
@@ -1149,6 +1149,25 @@ sta
Signed-off-by: Huacai Chen
---
Makefile.target |2 +-
hw/pci_ids.h|8 +
hw/vt82c686.c | 569 +++
hw/vt82c686.h | 11 +
4 files changed, 589 insertions(+), 1 deletions(-)
create mode 100644 hw/vt82c686.c
create mode 100644 hw
Signed-off-by: Huacai Chen
---
Makefile.objs|1 +
default-configs/mips64el-softmmu.mak |1 +
hw/ide.h |1 +
hw/ide/via.c | 191 ++
4 files changed, 194 insertions(+), 0 deletio
Signed-off-by: Huacai Chen
---
target-mips/mips-defs.h |4
target-mips/translate_init.c | 35 +++
2 files changed, 39 insertions(+), 0 deletions(-)
diff --git a/target-mips/mips-defs.h b/target-mips/mips-defs.h
index a7f4697..bf094a3 100644
--- a/t
On 06/22/10 15:23, Alexander Graf wrote:
> Alex Williamson wrote:
>>> Could we send the agenda note out a little bit earlier?
>>>
>>> The note arrived at 12:55am CST and we decided we had no agenda at
>>> 7:04am CST. That's a pretty short window and I imagine a lot of people
>>> in the US
Signed-off-by: Huacai Chen
---
Makefile.target|2 +-
hw/mips_fulong2e.c | 416
2 files changed, 417 insertions(+), 1 deletions(-)
create mode 100644 hw/mips_fulong2e.c
diff --git a/Makefile.target b/Makefile.target
index 4b23e6d..a4c
Changes from V4:
Doesn't include the PMON binary to avoid license issues.
--
Huacai Chen
From: Nicholas Bellinger
This patch updates hw/scsi-bus.c to add the PERSISTENT_RESERVE_OUT cdb
case in scsi_req_xfer_mode() to set SCSI_XFER_TO_DEV for outgoing WRITE data.
Signed-off-by: Nicholas A. Bellinger
Acked-by: Gerd Hoffmann
Signed-off-by: Kevin Wolf
---
hw/scsi-bus.c |1 +
1 f
Add new functions that write and flush the written data to disk immediately.
This is what needs to be used for image format metadata to maintain integrity
for cache=... modes that don't use O_DSYNC. (Actually, we only need barriers,
and therefore the functions are defined as such, but flushes is wh
From: Nicholas Bellinger
This patch updates hw/scsi-bus.c to add MAINTENANCE_IN and MAINTENANCE_OUT case
in
scsi_req_length() for TYPE_ROM with MMC commands. It also adds the
MAINTENANCE_OUT
case in scsi_req_xfer_mode() to set SCSI_XFER_TO_DEV for outgoing write data.
Signed-off-by: Nicholas
From: Christoph Hellwig
Both SCSI and virtio expect the physical block size relative to the
logical block size. So get the factor first before calculating the
log2.
Reported-by: Mike Cao
Signed-off-by: Christoph Hellwig
Signed-off-by: Kevin Wolf
---
block_int.h |4 +++-
1 files changed,
With corrupted images, we can easily get an cluster index that exceeds the
array size of the temporary refcount table.
Signed-off-by: Kevin Wolf
---
block/qcow2-refcount.c | 14 +++---
1 files changed, 11 insertions(+), 3 deletions(-)
diff --git a/block/qcow2-refcount.c b/block/qcow2-
The following changes since commit cada87c432e3b9cd55869a24055baf2b7bc0b70c:
Blue Swirl (1):
Update OpenBIOS images
are available in the git repository at:
git://repo.or.cz/qemu/kevin.git for-anthony
Christoph Hellwig (1):
block: fix physical_block_size calculation
Eduardo Hab
Use bdrv_(p)write_sync to ensure metadata integrity in case of a crash.
Signed-off-by: Kevin Wolf
---
block/vmdk.c | 10 +-
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/block/vmdk.c b/block/vmdk.c
index e659908..2d4ba42 100644
--- a/block/vmdk.c
+++ b/block/vmdk.c
@@
From: Eduardo Habkost
This changes the monitor eject_device() function to not check for
bdrv_is_inserted().
Example run where the bug manifests itself:
(output of 'info block' is stripped to include only the CD-ROM device)
(qemu) info block
ide1-cd0: type=cdrom removable=1 locked=0 [not in
From: Yoshiaki Tamura
Although it is really rare to get in to the while loop, the list
operation in the loop is obviously wrong.
Signed-off-by: Yoshiaki Tamura
Signed-off-by: Kevin Wolf
---
hw/virtio-blk.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/hw/virtio-blk
Use bdrv_(p)write_sync to ensure metadata integrity in case of a crash.
While at it, correct the wrong usage of errno.
Signed-off-by: Kevin Wolf
---
block/cow.c | 20 +++-
1 files changed, 11 insertions(+), 9 deletions(-)
diff --git a/block/cow.c b/block/cow.c
index d146434..e
Use bdrv_(p)write_sync to ensure metadata integrity in case of a crash.
Signed-off-by: Kevin Wolf
---
block/qcow.c | 18 ++
1 files changed, 10 insertions(+), 8 deletions(-)
diff --git a/block/qcow.c b/block/qcow.c
index 449858f..816103d 100644
--- a/block/qcow.c
+++ b/block/q
Use bdrv_(p)write_sync to ensure metadata integrity in case of a crash.
Signed-off-by: Kevin Wolf
---
block/vpc.c |9 +
1 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/block/vpc.c b/block/vpc.c
index f1f73e2..e50509e 100644
--- a/block/vpc.c
+++ b/block/vpc.c
@@ -261,7
From: MORITA Kazutaka
Some block drivers use an aio handler and do I/O completion routines
in it. However, the handler is not invoked if we only do
aio_read/write, because registered fds are not checked at all.
This patch registers an aio handler of STDIO to checks whether we can
read a command
Trying to check them leads to a second error message which is more confusing
than helpful:
Can't get refcount for cluster 0: Invalid argument
ERROR cluster 0 refcount=-22 reference=1
Signed-off-by: Kevin Wolf
---
block/qcow2-refcount.c |1 +
1 files changed, 1 insertions(+), 0 delet
Use bdrv_(p)write_sync to ensure metadata integrity in case of a crash.
Signed-off-by: Kevin Wolf
---
block/qcow2-cluster.c | 24
block/qcow2-refcount.c | 24
block/qcow2-snapshot.c | 23 +++
block/qcow2.c |
Alexander Graf wrote:
> Alex Williamson wrote:
>> On Tue, 2010-06-22 at 07:49 -0500, Anthony Liguori wrote:
>>
>>> On 06/22/2010 07:04 AM, Juan Quintela wrote:
>>>
Chris Wright wrote:
> Please send in any agenda items you are interested in covering.
>
>>>
Anthony Liguori wrote:
> On 06/22/2010 09:10 AM, Juan Quintela wrote:
> I think for the past month or so, whether the meeting gets cancelled
> is based on a race condition between me finishing my first cup of
> coffee and you sending out the cancellation note ;-)
This is my 1st cancellation note
Kevin Wolf writes:
> Am 21.06.2010 18:21, schrieb Markus Armbruster:
>> Christoph Hellwig writes:
>>
[...]
>>> The user basically can specify two things:
>>>
>>> - a transport protocol. Normally this is just the filesystem
>>>interface, but it can also be nbd, http or for really sick peop
On 06/16/2010 04:46 AM, Markus Armbruster wrote:
A number of changes to qdev paths have been proposed in various threads.
It's becoming harder to keep track of them, so let me sum them up in one
place. Please correct me if I misrepresent your ideas.
Honestly, I think we've gone off the dee
On 06/22/2010 09:10 AM, Juan Quintela wrote:
Alexander Graf wrote:
Alex Williamson wrote:
On Tue, 2010-06-22 at 07:49 -0500, Anthony Liguori wrote:
On 06/22/2010 07:04 AM, Juan Quintela wrote:
Chris Wright wrote:
Please send in any agenda items
People were wondering why qemu-img check failed after they tried to preallocate
a large qcow2 file and ran out of disk space.
Signed-off-by: Kevin Wolf
---
block/qcow2.c | 15 +--
1 files changed, 9 insertions(+), 6 deletions(-)
diff --git a/block/qcow2.c b/block/qcow2.c
index d29
Christoph Hellwig wrote:
> On Mon, Jun 21, 2010 at 09:51:23AM -0500, Anthony Liguori wrote:
> > I can appreciate the desire to keep protocols and formats as an internal
> > distinction but as a user visible concept, I think your two examples
> > highlight why exposing protocols as formats make se
Kevin Wolf wrote:
> > The "protocol" parlance breaks down when we move away from the simple
> > stuff. For instance, qcow2 needs two children: the block driver
> > providing the delta bits (in qcow2 format), and the block driver
> > providing the base bits (whose configuration happens to be stored
Markus Armbruster wrote:
> A possible reason why we currently expose format and protocol at the
> user interface is to avoid stacking there.
Pragmatic solution?: A few generic flags in each stacking module
("format/protocol/transport"), which govern which other modules are
allowed to stack on top
** Attachment added: "udp-pong.c"
http://launchpadlibrarian.net/50751155/udp-pong.c
--
Slow UDP performance with virtio device
https://bugs.launchpad.net/bugs/597351
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
Status in QEMU: Ne
Public bug reported:
I'm working on an app that is very sensitive to round-trip latency
between the guest and host, and qemu/kvm seems to be significantly
slower than it needs to be.
The attached program is a ping/pong over UDP. Call it with a single
argument to start a listener/echo server on t
On Tue, Jun 22, 2010 at 05:40:02PM +0100, Jamie Lokier wrote:
> Kevin Wolf wrote:
> > > The "protocol" parlance breaks down when we move away from the simple
> > > stuff. For instance, qcow2 needs two children: the block driver
> > > providing the delta bits (in qcow2 format), and the block driver
Public bug reported:
Debugging with gdb-stub does not work with qemu-system-sparc target
Qemu compiled from current git tree.
execution string: qemu-system-sparc.exe -s -S -m 256 -L Bios -hda
sparc.img -boot c
connect with telnet localhost 1234
enter '$s#73' (without quotes, this is single step
Next commit will introduce a new QDict iteration API which
returns QDictEntry entries, but we don't want users to directly
access its members since QDictEntry should be private to QDict.
In the near future this kind of data type will be turned into a
forward reference.
Signed-off-by: Luiz Capitul
Current QMP's client argument checker code is more complex than it should be
and has a flaw: it ignores unknown arguments.
This series solves both problems by introducing a new, simple and ultra-poweful
argument checker. This wasn't trivial to get right due to the number of errors
combinations, so
A missing key is not an error.
Signed-off-by: Luiz Capitulino
---
qdict.c |6 +++---
qdict.h |2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/qdict.c b/qdict.c
index 175bc17..c974d6f 100644
--- a/qdict.c
+++ b/qdict.c
@@ -272,16 +272,16 @@ const char *qdict_get_str(c
This is similar to qmp_check_client_args(), but it checks if
the input object follows the specification (QMP/qmp-spec.txt
section 2.3).
As we're limited to three keys, the work here is quite simple:
we iterate over the input object, checking each time if the
current argument complies to the specif
Signed-off-by: Luiz Capitulino
---
check-qdict.c | 31 +++
1 files changed, 31 insertions(+), 0 deletions(-)
diff --git a/check-qdict.c b/check-qdict.c
index 1b070f4..6afce5a 100644
--- a/check-qdict.c
+++ b/check-qdict.c
@@ -194,6 +194,36 @@ START_TEST(qobject_to_q
Let's call a 'hash' only what is returned by our hash function,
anything else is a 'bucket'.
This helps avoiding confusion with regard to how we traverse
our table.
Signed-off-by: Luiz Capitulino
---
check-qdict.c |2 +-
qdict.c | 24
qdict.h |4 ++
Signed-off-by: Luiz Capitulino
---
qerror.c |4
qerror.h |3 +++
2 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/qerror.c b/qerror.c
index 44d0bf8..8a10557 100644
--- a/qerror.c
+++ b/qerror.c
@@ -177,6 +177,10 @@ static const QErrorStringTable qerror_table[] = {
It's composed of functions qdict_first() and qdict_next(), plus
functions to access QDictEntry values.
This API was suggested by Markus Armbruster and
it offers full control over the iteration process.
The usage is simple, the following example prints all keys in 'qdict'
(it's hopefully better t
Previous two commits added qmp_check_client_args(), which
fully replaces this code and is way better.
It's important to note that the new checker doesn't support
the '/' arg type. As we don't have any of those handlers
converted to QMP, this is just dead code.
Signed-off-by: Luiz Capitulino
---
This commit introduces the second (and last) part of QMP's new
argument checker.
The job is done by check_client_args_type(), it iterates over
the client's argument qdict and for for each argument it checks
if it exists and if its type is valid.
It's important to observe the following changes fro
1 - 100 of 116 matches
Mail list logo
