thanks, it is applied to my vhost_scsi git tree
git://github.com/mdroth/qemu.git vhost-scsi
On Thu, Jul 12, 2012 at 4:55 AM, Nicholas A. Bellinger
wrote:
> From: Nicholas Bellinger
>
> This QEMU patch sets VirtIOSCSIConfig->max_target=0 for vhost-scsi operation
> to restrict virtio-scsi LLD gue
Improve the tests for the LBA to cover more cases, the new test looks like this
if (r->req.cmd.lba > r->req.cmd.lba + len
|| r->req.cmd.lba + len > s->qdev.max_lba) {
For the 16 byte opcodes, the lba is a uint64, so the first check is to make
sure that we do not wrap.
For example
Paolo, List
Please find a small patch to the scsi emulation.
This patch improves the checkign that the requested lbas are all available.
We check both that lba+len is not going past the end of the device
but also iflba+len < lba
This second condition could occur for deviously crafted scsi pa
On Tue, Jul 10, 2012 at 03:15:51PM +0200, Igor Mammedov wrote:
> /* pc.c */
> -int cpu_is_bsp(CPUX86State *env);
Why remove it instead of modifying it to check BSP bit in apic base?
I think it will make the patch smaller and open code the check does not
look nice.
--
Gleb
Il 12/07/2012 02:19, Ronnie Sahlberg ha scritto:
> The QEMU SCSI emulation does not support protection information,
> so any READ/WRITE/VERIFY commands that has the protect bits set to
> non-zero should fail with ILLEGAL_REQUEST/INVALID_FIELD_IN_CDB
>
> From SCSI SBC :
> If the logical unit does n
Il 12/07/2012 07:34, Zhi Yong Wu ha scritto:
> HI,
>
> Do we need to maintain one QEMU branch to collect all useful latest
> patches for tcm_vhost support? You know, those patches will not get
> merged into qemu.git/master.
Never say never, but the answer to your question is yes: please apply
th
On 12/07/12 15:29, Alex Williamson wrote:
> On Thu, 2012-07-12 at 14:58 +1000, Alexey Kardashevskiy wrote:
>> On 12/07/12 14:43, Alex Williamson wrote:
>>> On Thu, 2012-07-12 at 14:38 +1000, Alexey Kardashevskiy wrote:
On 12/07/12 14:31, Alex Williamson wrote:
> On Thu, 2012-07-12 at 14:16
HI,
Do we need to maintain one QEMU branch to collect all useful latest
patches for tcm_vhost support? You know, those patches will not get
merged into qemu.git/master.
On Thu, Jul 12, 2012 at 4:55 AM, Nicholas A. Bellinger
wrote:
> From: Nicholas Bellinger
>
> This QEMU patch sets VirtIOSCSI
On Thu, 2012-07-12 at 14:58 +1000, Alexey Kardashevskiy wrote:
> On 12/07/12 14:43, Alex Williamson wrote:
> > On Thu, 2012-07-12 at 14:38 +1000, Alexey Kardashevskiy wrote:
> >> On 12/07/12 14:31, Alex Williamson wrote:
> >>> On Thu, 2012-07-12 at 14:16 +1000, Alexey Kardashevskiy wrote:
> On
On 12/07/12 14:43, Alex Williamson wrote:
> On Thu, 2012-07-12 at 14:38 +1000, Alexey Kardashevskiy wrote:
>> On 12/07/12 14:31, Alex Williamson wrote:
>>> On Thu, 2012-07-12 at 14:16 +1000, Alexey Kardashevskiy wrote:
On 12/07/12 12:54, Alex Williamson wrote:
> On Wed, 2012-07-11 at 12:25
On Thu, 2012-07-12 at 14:38 +1000, Alexey Kardashevskiy wrote:
> On 12/07/12 14:31, Alex Williamson wrote:
> > On Thu, 2012-07-12 at 14:16 +1000, Alexey Kardashevskiy wrote:
> >> On 12/07/12 12:54, Alex Williamson wrote:
> >>> On Wed, 2012-07-11 at 12:25 +1000, Alexey Kardashevskiy wrote:
> On
On 12/07/12 14:31, Alex Williamson wrote:
> On Thu, 2012-07-12 at 14:16 +1000, Alexey Kardashevskiy wrote:
>> On 12/07/12 12:54, Alex Williamson wrote:
>>> On Wed, 2012-07-11 at 12:25 +1000, Alexey Kardashevskiy wrote:
On 11/07/12 02:57, Alex Williamson wrote:
> On Tue, 2012-07-10 at 15:51
On Thu, 2012-07-12 at 14:16 +1000, Alexey Kardashevskiy wrote:
> On 12/07/12 12:54, Alex Williamson wrote:
> > On Wed, 2012-07-11 at 12:25 +1000, Alexey Kardashevskiy wrote:
> >> On 11/07/12 02:57, Alex Williamson wrote:
> >>> On Tue, 2012-07-10 at 15:51 +1000, Alexey Kardashevskiy wrote:
> Th
On 12/07/12 12:54, Alex Williamson wrote:
> On Wed, 2012-07-11 at 12:25 +1000, Alexey Kardashevskiy wrote:
>> On 11/07/12 02:57, Alex Williamson wrote:
>>> On Tue, 2012-07-10 at 15:51 +1000, Alexey Kardashevskiy wrote:
The two patches in this set are supposed to add VFIO support for POWER.
>>>
On Sat, Apr 21, 2012 at 05:28:36PM -0400, Xin Tong wrote:
> On Mon, Apr 16, 2012 at 10:51 PM, 陳韋任 wrote:
> >> what does the inline sequence look like ? what kind of things (other
> >> than refill tlb) performed in callout but not the inlined sequence ?
> >
> > What do you mean by the inline seque
qemu-img not only suports k/K/M/G/T/b, but also supports m/g/t/B. So correct
it in help message.
Signed-off-by: Dong Xu Wang
CC: riegama...@gmail.com
---
v1->v2: also correct error reporting.
qemu-img.c |9 +
1 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/qemu-img.c
On Wed, 2012-07-11 at 12:54 +1000, Alexey Kardashevskiy wrote:
> On 11/07/12 02:55, Alex Williamson wrote:
> > On Tue, 2012-07-10 at 15:51 +1000, Alexey Kardashevskiy wrote:
> >> The patch enables VFIO on POWER.
> >>
> >> It literally does the following:
> >>
> >> 1. POWERPC IOMMU support (the kern
2012/7/12 Dong Xu Wang :
> qemu-img not only suports k/K/M/G/T/b, but also supports m/g/t/B. So correct
> it in help message.
>
> Signed-off-by: Dong Xu Wang
> ---
> qemu-img.c |5 +++--
> 1 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/qemu-img.c b/qemu-img.c
> index 80cfb9
On Wed, 2012-07-11 at 12:25 +1000, Alexey Kardashevskiy wrote:
> On 11/07/12 02:57, Alex Williamson wrote:
> > On Tue, 2012-07-10 at 15:51 +1000, Alexey Kardashevskiy wrote:
> >> The two patches in this set are supposed to add VFIO support for POWER.
> >>
> >> The first one adds one more step in th
qemu-img not only suports k/K/M/G/T/b, but also supports m/g/t/B. So correct
it in help message.
Signed-off-by: Dong Xu Wang
---
qemu-img.c |5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/qemu-img.c b/qemu-img.c
index 80cfb9b..533eac2 100644
--- a/qemu-img.c
+++ b/qe
On Wed, Jul 11, 2012 at 8:27 PM, Igor Mammedov wrote:
> On 07/11/2012 09:35 AM, liu ping fan wrote:
>>
>> On Tue, Jul 10, 2012 at 9:35 PM, Igor Mammedov
>> wrote:
>>>
>>> On 07/09/2012 02:57 PM, Andreas Färber wrote:
Am 09.07.2012 12:59, schrieb igor:
>
>
> On 06/20/201
On Tue, Jul 10, 2012 at 11:32 PM, Peter Maydell
wrote:
> Add support for reading device tree properties (both generic
> and single-cell ones) to QEMU's convenience wrapper layer.
>
> Signed-off-by: Peter Maydell
> ---
> Here's a v2:
> * added qemu_devtree_get_one_cell_from_prop() which reads a s
The QEMU SCSI emulation does not support protection information,
so any READ/WRITE/VERIFY commands that has the protect bits set to
non-zero should fail with ILLEGAL_REQUEST/INVALID_FIELD_IN_CDB
>From SCSI SBC :
If the logical unit does not support protection information,
then the device server sh
List, Paolo,
Please find a small patch for the scsi emulation in qemu.
Since QEMU scsi emulation does not implement/support protection information
any READ/WRITE/VERIFY command that specifies a non-zero value for the
rd/vr/wr-protect field in the CDB should fail with
ILLEGAL_REQUEST/INVALID_FIEL
On 09.07.2012, at 10:50, Markus Armbruster wrote:
> Alexander Graf writes:
>
>> We've had support for creating AHCI devices using -device for a while now,
>> but it's cumbersome to users. We really should provide an easier way for
>> them to leverage the power of AHCI!
>>
>> So let's introduce
On 25.06.2012, at 19:32, Alexander Graf wrote:
> When forwarding a segmentation fault into the guest process, we were passing
> the host's address directly into the guest process's signal descriptor.
>
> That obviously confused the guest process, since it didn't know what to make
> of the (usual
From: Nicholas Bellinger
This QEMU patch sets VirtIOSCSIConfig->max_target=0 for vhost-scsi operation
to restrict virtio-scsi LLD guest scanning to max_id=0 (a single target ID
instance) when connected to individual tcm_vhost endpoints as requested by
Paolo.
This ensures that virtio-scsi LLD onl
Luiz Capitulino wrote:
> On Wed, 11 Jul 2012 12:25:38 -0600
> Eric Blake wrote:
>
>> On 07/11/2012 12:08 PM, Luiz Capitulino wrote:
>> > On Fri, 29 Jun 2012 18:43:57 +0200
>> > Juan Quintela wrote:
>> >
>> >> We add time spent for migration to the output of "info migrate"
>> >> command. 'total
On Fri, 22 Jun 2012 14:36:09 -0400
Corey Bryant wrote:
> Signed-off-by: Corey Bryant
I've cherry-picked this one into the qmp tree.
> ---
> v2:
> -Convert getfd and closefd to QAPI (lcapitul...@redhat.com)
> -Remove changes that returned fd from getfd (lcapitul...@redhat.com)
> -Wrap hmp_*
Il 15/06/2012 23:12, Eric Blake ha scritto:
>> > +#
>> > # Returns: nothing on success
>> > # If @device is not a valid block device, DeviceNotFound
>> > # If @target can't be opened, OpenFileFailed
>> > @@ -1389,7 +1393,8 @@
>> > { 'command': 'drive-mirror',
>> >'data': {
Il 16/06/2012 01:02, Eric Blake ha scritto:
>> > +unsigned long *levels[HBITMAP_LEVELS];
> and at this point, I decided reading the .h first makes more sense.
> Also, this is a high-level first-impressions review, not a line-by-line
> algorithmic accuracy review. Did you invent this yourself,
On Wed, 11 Jul 2012 12:25:38 -0600
Eric Blake wrote:
> On 07/11/2012 12:08 PM, Luiz Capitulino wrote:
> > On Fri, 29 Jun 2012 18:43:57 +0200
> > Juan Quintela wrote:
> >
> >> We add time spent for migration to the output of "info migrate"
> >> command. 'total_time' means time since the start f
Orit Wasserman wrote:
> On 07/01/2012 01:20 PM, Juan Quintela wrote:
>> Orit Wasserman wrote:
>>> On 06/28/2012 10:22 PM, Juan Quintela wrote:
Notice that the live migration users never unregister, so no problem
about freeing the ops structure.
void unregister_savevm(DeviceState *
At the moment i'm using version qemu 0.12.3+noroms-0ubuntu9.18 of my
ubuntu distribution. I'm triing to compile the latest upstream version
during the next two weeks to verify if this is still an issue.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is sub
On 07/11/2012 12:08 PM, Luiz Capitulino wrote:
> On Fri, 29 Jun 2012 18:43:57 +0200
> Juan Quintela wrote:
>
>> We add time spent for migration to the output of "info migrate"
>> command. 'total_time' means time since the start fo migration if
>> migration is 'active', and total time of migratio
On 06.07.2012, at 17:58, Jan Kiszka wrote:
> On 2012-07-06 17:36, Alexander Graf wrote:
>>
>> On 02.07.2012, at 09:24, Jan Kiszka wrote:
>>
>>> On 2012-07-02 06:28, Alexey Kardashevskiy wrote:
Ping?
On 22/06/12 11:15, Alexey Kardashevskiy wrote:
> Added (msi|msix)_set_
On Fri, 29 Jun 2012 18:43:57 +0200
Juan Quintela wrote:
> We add time spent for migration to the output of "info migrate"
> command. 'total_time' means time since the start fo migration if
> migration is 'active', and total time of migration if migration is
> completed. As we are also intereste
Crístian Viana writes:
> Windows 2008+ is very sensitive to missed ticks. The RTC is used by default as
> the time source. If driftfix is not enabled, Windows is prone to
> blue screening.
>
> Signed-off-by: Crístian Viana
Reviewed-by: Anthony Liguori
Regards,
Anthony Liguori
> ---
> hw/mc
Crístian Viana writes:
> The current value for the -rtc driftfix option is 'none'. This patch
> makes sure that the old machines configuration will work the same way
> even after that option changes its default value.
>
> Signed-off-by: Crístian Viana
Reviewed-by: Anthony Liguori
Regards,
An
Crístian Viana writes:
> If a GlobalProperty has already been registered, it won't have its
> value overwritten.
>
> Signed-off-by: Crístian Viana
> ---
>
> Changes since v4:
> - Only register a GlobalProperty if it hasn't already been registered. That
> avoids the property values to be overw
On 07/11/2012 08:01 PM, Juan Quintela wrote:
> Orit Wasserman wrote:
>> On 07/01/2012 01:20 PM, Juan Quintela wrote:
>>> Orit Wasserman wrote:
On 06/28/2012 10:22 PM, Juan Quintela wrote:
> Notice that the live migration users never unregister, so no problem
> about freeing the ops s
Il 11/07/2012 13:35, Hannes Reinecke ha scritto:
> trace_megasas_dcmd_dump_frame() takes 9 arguments, which is
> rather much. Plus the trace infrastructure doesn't support
> it anymore.
> As we can get the information via other means it's pointless
> to have it in the driver, so rather use some pro
Hi,
On Wed, Jul 11, 2012 at 05:55:25PM +0300, Avi Kivity wrote:
> On 07/11/2012 01:32 PM, Vasilis Liaskovitis wrote:
> > Implement batch dimm creation command line options. These could be useful
> > for
> > not bloating the command line with a large number of dimms.
>
> IMO this is unneeded. Wi
Hi,
On Wed, Jul 11, 2012 at 09:14:29AM -0600, Eric Blake wrote:
> On 07/11/2012 04:32 AM, Vasilis Liaskovitis wrote:
> > Returns total memory of guest in bytes, including hotplugged memory.
> >
> > Signed-off-by: Vasilis Liaskovitis
>
> Should this instead be merged with query-balloon output, s
Hi,
On Wed, Jul 11, 2012 at 01:56:19PM +0200, Gerd Hoffmann wrote:
> On 07/11/12 12:31, Vasilis Liaskovitis wrote:
> > In order to hotplug memory between RamSize and BUILD_PCIMEM_START, the pci
> > window needs to start at BUILD_PCIMEM_START (0xe000).
> > Otherwise, the guest cannot online new
Hi,
On Wed, Jul 11, 2012 at 08:59:03AM -0600, Eric Blake wrote:
> On 07/11/2012 04:31 AM, Vasilis Liaskovitis wrote:
> > Guest can respond to ACPI hotplug events e.g. with _EJ or _OST method.
> > This patch implements a tail queue to store guest notifications for memory
> > hot-add and hot-remove
Hi,
On Wed, Jul 11, 2012 at 06:48:38PM +0800, Wen Congyang wrote:
> > +if (enabled)
> > +add_e820(mem_base, mem_len, E820_RAM);
>
> add_e820() is declared in memmap.h. You should include this header file,
> otherwise, seabios cannot be built.
thanks. you had the same comment
Il 15/06/2012 22:12, Eric Blake ha scritto:
> On 06/15/2012 09:05 AM, Paolo Bonzini wrote:
>> > This adds the monitor commands that start the mirroring job.
>> >
>> > Signed-off-by: Paolo Bonzini
>> > ---
>> > blockdev.c | 120
>> > --
>
Il 15/06/2012 23:14, Eric Blake ha scritto:
> On 06/15/2012 09:05 AM, Paolo Bonzini wrote:
>> Signed-off-by: Paolo Bonzini
>> ---
>> qapi-schema.json |4
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/qapi-schema.json b/qapi-schema.json
>> index 2ee988b..250edf2 100644
>> --- a/qa
Il 15/06/2012 19:58, Eric Blake ha scritto:
> On 06/15/2012 09:05 AM, Paolo Bonzini wrote:
>> This patch adds support for error management to streaming.
>>
>> Signed-off-by: Paolo Bonzini
>> ---
>> block/stream.c | 28 +++-
>> block_int.h |3 ++-
>> blockdev.c
Il 15/06/2012 19:50, Eric Blake ha scritto:
>> > { 'type': 'BlockJobInfo',
>> >'data': {'type': 'str', 'device': 'str', 'len': 'int',
>> > - 'offset': 'int', 'paused': 'bool', 'speed': 'int'} }
>> > + 'offset': 'int', 'paused': 'bool', 'speed': 'int',
>> > + 'io-s
Il 15/06/2012 19:45, Eric Blake ha scritto:
>> > +++ b/qapi-schema.json
>> > @@ -429,7 +429,7 @@
>> > #
>> > # Since: 1.0
>> > ##
>> > -{ 'enum': 'BlockDeviceIoStatus', 'data': [ 'ok', 'failed', 'nospace' ] }
>> > +{ 'enum': 'BlockDeviceIoStatus', 'data': [ 'ok', 'nospace', 'failed' ] }
> Isn't
Il 15/06/2012 19:32, Eric Blake ha scritto:
> Just so I'm clear, am I right that:
>
> calling block-job-pause multiple times means the subsequent calls are
> successful no-ops; likewise, calling block-job-resume on a job not
> paused (whether by calling multiple times, or if the job never paused i
On Wed, Jul 11, 2012 at 05:28:26PM +0200, Paolo Bonzini wrote:
> Il 11/07/2012 17:25, Michael Roth ha scritto:
> > > No reason to leave them out, and it will ensure that the dependencies
> > > are picked up. Later we can perhaps move the files to another
> > > directory to avoid ../ usage.
> >
>
Il 15/06/2012 18:45, Eric Blake ha scritto:
> Pre-existing, but do we have any policy on US (canceled) vs. UK
> (cancelled) spelling? Obviously, our API spelling is locked into UK
> spelling, but if the docs generally prefer US spelling, do we need to
> adjust things here? But any such adjustment
Am 10.07.2012 11:12, schrieb Markus Armbruster:
> 29 patches may look discouraging, but most patches are small, and the
> ones that aren't just move code around.
>
> Goals of this series:
>
> 1. One more step towards a clean separation block device host and
>guest part.
>
> 2. Purge CHS geom
On 07/11/2012 10:57 AM, Paolo Bonzini wrote:
Il 11/07/2012 16:56, Fabien Chouteau ha scritto:
So indeed you could also use SIOCDEVPRIVATE / BRCTL_ADD_IF if you were
inclined to do so...
Unfortunately I don't have time to go deep into the code. It's not a
feature I plan to use, so the quicke
Il 11/07/2012 17:25, Michael Roth ha scritto:
> > No reason to leave them out, and it will ensure that the dependencies
> > are picked up. Later we can perhaps move the files to another
> > directory to avoid ../ usage.
>
> Couldn't we accomplish that by just adding the dependencies to qga-obj-y
On Wed, Jul 11, 2012 at 04:40:20PM +0200, Paolo Bonzini wrote:
> No reason to leave them out, and it will ensure that the dependencies
> are picked up. Later we can perhaps move the files to another
> directory to avoid ../ usage.
Couldn't we accomplish that by just adding the dependencies to qga
Il 11/07/2012 16:56, Fabien Chouteau ha scritto:
>>> So indeed you could also use SIOCDEVPRIVATE / BRCTL_ADD_IF if you were
>>> inclined to do so...
>>>
>>> >>
>>> >> Unfortunately I don't have time to go deep into the code. It's not a
>>> >> feature I plan to use, so the quickest so
On 07/11/2012 04:32 AM, Vasilis Liaskovitis wrote:
> Returns total memory of guest in bytes, including hotplugged memory.
>
> Signed-off-by: Vasilis Liaskovitis
Should this instead be merged with query-balloon output, so that we have
a single command that shows all aspects of memory usage (both
2012/7/11 Jing Huang :
> This patch fix ping issues for linux-user guest.
>
> * The do_setsockopts function in linux-user does not support SOL_RAW
> socket which is used in ping net tool.
>
> * The recvmsg in main_loop of ping could not fetch
> sockaddr_in struct. That is because do_sendrecvmsg in
On 07/11/2012 04:31 AM, Vasilis Liaskovitis wrote:
> Guest can respond to ACPI hotplug events e.g. with _EJ or _OST method.
> This patch implements a tail queue to store guest notifications for memory
> hot-add and hot-remove requests.
>
> Guest responses for memory hotplug command on a per-dimm b
On 07/11/2012 04:22 PM, Corey Bryant wrote:
>
>
> On 07/10/2012 09:37 AM, Fabien Chouteau wrote:
>> On 07/10/2012 01:06 PM, Paolo Bonzini wrote:
>>> Il 10/07/2012 13:02, Fabien Chouteau ha scritto:
Bridge helper uses ioctl's not available on old Linux versions, we add
this flag
On 07/11/2012 01:32 PM, Vasilis Liaskovitis wrote:
> Implement batch dimm creation command line options. These could be useful for
> not bloating the command line with a large number of dimms.
IMO this is unneeded. With a management tool there is no problem
generating a long command line; from th
Thanks for this patch. Review comments below...
(cc'ing the linux-user maintainer)
On 11 July 2012 14:56, Jing Huang wrote:
> This patch fix ping issues for linux-user guest.
>
> * The do_setsockopts function in linux-user does not support SOL_RAW
> socket which is used in ping net tool.
>
> * T
After commit dcff25f2cd8c11a9368cc2369aeb0319c32d9e26, Dependency file
are taken from the directories that have a Makefile.objs file. This is
not enough, since files can be included from other directories.
So, pick them from directories that have an object file in them.
Signed-off-by: Paolo Bonzi
No reason to leave them out, and it will ensure that the dependencies
are picked up. Later we can perhaps move the files to another
directory to avoid ../ usage.
Signed-off-by: Paolo Bonzini
---
Makefile |2 +-
qga/Makefile.objs |2 ++
2 files changed, 3 insertions(+), 1 deleti
Avi and Jan experienced more problem with dependencies. This series
should fix them.
Paolo Bonzini (2):
build: include qapi-generated/ files in qga/Makefile.objs
build: get dependency file directories from object file names
build: do not use hw/kvm/
Makefile |4 ++--
Make
Kevin Wolf writes:
> Am 10.07.2012 11:12, schrieb Markus Armbruster:
>> Commit f3d54fc4 factored it out of hw/ide.c for reuse. Sensible,
>> except it was put into block.c. Device-specific functionality should
>> be kept in device code, not the block layer. Move it to
>> hw/hd-geometry.c, and m
On 07/10/2012 09:37 AM, Fabien Chouteau wrote:
On 07/10/2012 01:06 PM, Paolo Bonzini wrote:
Il 10/07/2012 13:02, Fabien Chouteau ha scritto:
Bridge helper uses ioctl's not available on old Linux versions, we add
this flag to disable the build.
Which ioctls? Please detect them, so that we c
On Sat, 16 Jun 2012 02:17:30 +0530
Supriya Kannery wrote:
> New command "block_set_hostcache" added for dynamically changing
> host pagecache setting of a block device.
>
> Usage:
> block_set_hostcache
> = block device
> = on/off
>
> Example:
> (qemu) block_set_hostcache ide0-hd0
Bridge helper uses ioctl's not available on old Linux versions, we add
this flag to disable the build.
Signed-off-by: Fabien Chouteau
---
Makefile |2 +-
configure | 42 ++
2 files changed, 43 insertions(+), 1 deletion(-)
diff --git a/Makefile b/Ma
On 07/06/2012 08:11 AM, Markus Armbruster wrote:
>
> It seems there could be more knobs to control than just "dump guest
> state yes/no". Therefore, extensible command line syntax like
> "--core-dump guest-ram=off" seems to be advisable. We have too many
> -dont-do-FOO options already.
-m 2T,du
On Mon, 09 Jul 2012 16:43:40 +0200
Kevin Wolf wrote:
> Am 15.06.2012 23:07, schrieb Eric Blake:
> > On 06/15/2012 02:47 PM, Supriya Kannery wrote:
> >> Enhance "info block" to display hostcache setting for each
> >> block device.
> >>
> >
> >> ##
> >> { 'type': 'BlockInfo',
> >>'data': {'d
This patch fix ping issues for linux-user guest.
* The do_setsockopts function in linux-user does not support SOL_RAW
socket which is used in ping net tool.
* The recvmsg in main_loop of ping could not fetch
sockaddr_in struct. That is because do_sendrecvmsg in linux-user does
not pass the msg->m
Am 10.07.2012 11:12, schrieb Markus Armbruster:
> Commit f3d54fc4 factored it out of hw/ide.c for reuse. Sensible,
> except it was put into block.c. Device-specific functionality should
> be kept in device code, not the block layer. Move it to
> hw/hd-geometry.c, and make stylistic changes requi
On 07/11/2012 08:15 AM, Paolo Bonzini wrote:
Il 11/07/2012 14:39, Kevin Wolf ha scritto:
Am 11.07.2012 07:25, schrieb Michael Tokarev:
On 11.07.2012 09:09, Stefan Weil wrote:
Commit 25e5e4c7 broke compilation for non POSIX hosts (e.g. MinGW)
because it partially replaced "ret" by "count".
It
Signed-off-by: Markus Armbruster
---
hw/block-common.c | 40
hw/block-common.h |2 ++
hw/ide/core.c | 24
hw/ide/qdev.c | 19 ++-
hw/scsi-disk.c| 31 +++
hw/virti
Il 11/07/2012 14:39, Kevin Wolf ha scritto:
> Am 11.07.2012 07:25, schrieb Michael Tokarev:
>> On 11.07.2012 09:09, Stefan Weil wrote:
>>> Commit 25e5e4c7 broke compilation for non POSIX hosts (e.g. MinGW)
>>> because it partially replaced "ret" by "count".
>>>
>>> It also changed the handling of E
On Wed, Jul 11, 2012 at 02:09:53PM +0100, Peter Maydell wrote:
> On 11 July 2012 13:54, Kevin Wolf wrote:
> > Does this patch improve anything? Last time I checked, qemu only
> > compiled on gcc anyway.
>
> It would be nice to be able to compile with LLVM/Clang; however
> since Clang supports the
Kevin Wolf writes:
> Am 05.07.2012 18:39, schrieb Markus Armbruster:
>> Markus Armbruster writes:
>>
>>> Kevin Wolf writes:
>>>
Am 29.06.2012 17:34, schrieb Markus Armbruster:
> New limits straight from ATA4 6.2 Register delivered data transfer
> command sector addressing.
>
>
On 11 July 2012 13:54, Kevin Wolf wrote:
> Does this patch improve anything? Last time I checked, qemu only
> compiled on gcc anyway.
It would be nice to be able to compile with LLVM/Clang; however
since Clang supports the ?: gcc extension this patch doesn't
move us any further in that direction.
Signed-off-by: Markus Armbruster
---
hw/Makefile.objs |2 +-
hw/block-common.c| 24
hw/block-common.h|3 +++
hw/ide/qdev.c| 12 ++--
hw/scsi-disk.c |8 +---
hw/usb/dev-storage.c | 10 ++
hw/virtio-blk.c
This stuff doesn't belong to block layer, and was put there only
because a better home didn't exist then. Now it does.
Signed-off-by: Markus Armbruster
---
block.h | 45 -
hw/block-common.h | 45 ++
Applies on top of "[PATCH v3 00/29] Disk geometry cleanup".
Markus Armbruster (4):
hw/block-common: Move BlockConf & friends from block.h
hw/block-common: Factor out fall back to legacy -drive serial=...
blockdev: Don't limit DriveInfo serial to 20 characters
hw/block-common: Factor out fa
All current users (IDE, SCSI and virtio-blk) happen to share this 20
characters limit. Still, it should be left to device models. They
already enforce their limits. They have to, as the DriveInfo limit
only affects legacy -drive serial=..., not the qdev properties.
usb-storage, which doesn't li
Am 10.07.2012 20:06, schrieb MORITA Kazutaka:
> At Tue, 10 Jul 2012 16:12:27 +0200,
> Christoph Hellwig wrote:
>>
>> Only buffers that map to unallocated blocks need to be zeroed.
>>
>> Signed-off-by: Christoph Hellwig
>>
>> ---
>> block/sheepdog.c | 37 ++---
>>
Am 08.07.2012 14:09, schrieb Andreas Schwab:
> blauwir...@gmail.com writes:
>
>> +pstrcpy(bs->backing_format, sizeof(bs->backing_format),
>> +backing_fmt ? backing_file : "");
>
> s/backing_file/backing_fmt/
Which is why such changes are probably a bad idea. Even more so if they
On Wed, 11 Jul 2012 09:00:59 +0200
Markus Armbruster wrote:
> Luiz Capitulino writes:
>
> > It allows for specifying an alias for each option name, see next commits
> > examples.
> >
> > Signed-off-by: Luiz Capitulino
> > ---
> > qemu-option.c | 9 -
> > qemu-option.h | 1 +
> > 2 fil
Am 08.07.2012 21:22, schrieb blauwir...@gmail.com:
> From: Blue Swirl
>
> Use 'unsigned int' for bit numbers instead of 'unsigned long' or
> 'int'. Adjust asserts.
>
> Signed-off-by: Blue Swirl
I haven't followed the original discussion and therefore don't know what
the controversy is about (n
Il 11/07/2012 14:08, Avi Kivity ha scritto:
> specific command line or guest?
>>>
>>> qemu-system-x86_64
>>> >>
>>> >> Just did the same, but it's all fine here.
>> >
>> > Ok, I'll debug it. Probably something stupid like a miscompile.
> Indeed, a simple clean build fixed it up.
Am 11.07.2012 07:25, schrieb Michael Tokarev:
> On 11.07.2012 09:09, Stefan Weil wrote:
>> Commit 25e5e4c7 broke compilation for non POSIX hosts (e.g. MinGW)
>> because it partially replaced "ret" by "count".
>>
>> It also changed the handling of EINTR in a wrong way.
>>
>> The patch restores the o
On 07/11/2012 03:17 AM, liu ping fan wrote:
On Tue, Jul 10, 2012 at 6:12 PM, Andreas Färber wrote:
Am 10.07.2012 10:41, schrieb Paolo Bonzini:
Il 10/07/2012 08:16, Liu Ping Fan ha scritto:
This will give the embeded logic module, such as apic has the
opportunity to reset.
Signed-off-by: Liu
On 07/11/2012 09:35 AM, liu ping fan wrote:
On Tue, Jul 10, 2012 at 9:35 PM, Igor Mammedov wrote:
On 07/09/2012 02:57 PM, Andreas Färber wrote:
Am 09.07.2012 12:59, schrieb igor:
On 06/20/2012 03:35 PM, Andreas Färber wrote:
Am 20.06.2012 14:59, schrieb Igor Mammedov:
It's not correct t
This allows qemu to receive notifications from the guest OS on success or
failure of a memory hotplug request. The guest OS needs to implement the _OST
functionality for this to work (linux-next: http://lkml.org/lkml/2012/6/25/321)
Also add new _OST registers in docs/specs/acpi_hotplug.txt
Signed-
Add support for _OST method. _OST method will write into the correct I/O byte to
signal success / failure of hot-add or hot-remove to qemu.
Signed-off-by: Vasilis Liaskovitis
---
src/acpi-dsdt.dsl | 46 ++
src/ssdt-mem.dsl |4
2 files chan
This is v2 of the ACPI memory hotplug prototype for x86_64 target.
Changes v1->v2
- memory map is automatically calculated for hotplug dimms. Dimms are added from
top-of-memory skipping the pci hole at [PCI_HOLE_START, 4G).
- Renamed from "-memslot" to "-dimm". Commands changed to "dimm_add",
"d
On 07/11/2012 02:55 PM, Jan Kiszka wrote:
> On 2012-07-11 13:46, Avi Kivity wrote:
>> Current qemu-kvm master merged with latest upstream fails on startup:
>>
>> (gdb) bt
>> #0 0x7fdcd4a047a0 in kvm_vcpu_ioctl (env=0x0, type=-1071075694) at
>> /home/tlv/akivity/qemu/kvm-all.c:1602
>> #1 0x00
Guest can respond to ACPI hotplug events e.g. with _EJ or _OST method.
This patch implements a tail queue to store guest notifications for memory
hot-add and hot-remove requests.
Guest responses for memory hotplug command on a per-dimm basis can be detected
with the new hmp command "info memhp" or
1 - 100 of 150 matches
Mail list logo
