> I've tried redo series with passing alloc list as first argument,
> looks ugly as hell
I tried too. Not too bad at all. See below:
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index f66da5d..820504a 100644
--- a/hw/i386/acpi-build.c
+++ b/hw/i386/acpi-build.c
@@ -491,14 +491,14 @@ s
From: Tang Chen
Memory and CPU hot unplug are both asynchronous procedures.
When the unplug operation happens, unplug request cb is called first.
And when guest OS finished handling unplug, unplug cb will be called
to do the real removal of device.
This patch adds hotunplug cb to piix4, which me
From: Tang Chen
Memory and CPU hot unplug are both asynchronous procedures.
When the unplug operation happens, unplug request cb is called first.
And when guest OS finished handling unplug, unplug cb will be called
to do the real removal of device.
This patch adds hotunplug cb to pc machine, whi
From: Tang Chen
Memory and CPU hot unplug are both asynchronous procedures.
They both need unplug request callback to initiate unplug operation.
Add unplug handler to pc machine that will be used by following
CPU and memory unplug patches.
Reviewed-by: Igor Mammedov
Signed-off-by: Tang Chen
S
From: Tang Chen
Memory and CPU hot unplug are both asynchronous procedures.
They both need unplug request cb when the unplug operation happens.
This patch adds hotunplug request cb for ich9, and memory and CPU
hot unplug will share it.
Reviewed-by: Igor Mammedov
Signed-off-by: Tang Chen
Signe
From: Tang Chen
Memory and CPU hot unplug are both asynchronous procedures.
When the unplug operation happens, unplug request cb is called first.
And when guest OS finished handling unplug, unplug cb will be called
to do the real removal of device.
This patch adds hotunplug cb to ich9, which mem
Memory and CPU hot unplug are both asynchronous procedures.
When the unplug operation happens, unplug request cb is called first.
And when guest OS finished handling unplug, unplug cb will be called
to do the real removal of device.
They both need pc-machine, piix4 and ich9 unplug and unplug reque
On Fri, Dec 19, 2014 at 11:47:02AM +, Igor Mammedov wrote:
> Use build_append_namestring() instead of build_append_nameseg()
> So user won't have to care whether name is NameSeg, NamePath or
> NameString.
>
> See for refernce ACPI 5.0: 20.2.2 Name Objects Encoding
>
> Signed-off-by: Igor Mamm
On Thu, Jan 22, 2015 at 02:49:44PM +, Igor Mammedov wrote:
> Git tree for playing with:
> https://github.com/imammedo/qemu/commits/ASL_API_v2
>
> Igor Mammedov (47):
> acpi: introduce AML composer aml_append()
> acpi: add acpi_scope() term
> acpi: add acpi_device() term
> acpi: add
On Tue, Jan 27, 2015 at 11:29:09PM +0100, Igor Mammedov wrote:
> On Mon, 26 Jan 2015 18:17:55 +0200
> "Michael S. Tsirkin" wrote:
>
> > On Mon, Jan 26, 2015 at 04:34:01PM +0100, Andrew Jones wrote:
> > > On Mon, Jan 26, 2015 at 04:09:20PM +0100, Igor Mammedov wrote:
> > > > On Mon, 26 Jan 2015 10
On Tue, Jan 27, 2015 at 11:15:41 -0700, Eric Blake wrote:
> On 01/26/2015 08:31 AM, Peter Krempa wrote:
> > Produce more human readable error messages and fix few spelling
> > mistakes.
> >
> > Also remove a redundant check for the max memory size.
> >
> > Signed-off-by: Peter Krempa
> > ---
> >
On 01/27/2015 06:55 PM, John Snow wrote:
>
>
> On 01/27/2015 07:48 AM, Hannes Reinecke wrote:
>> On 01/23/2015 06:21 PM, Stefan Hajnoczi wrote:
>>> Dear libvirt, KVM, and QEMU contributors,
>>> The Google Summer of Code season begins soon and it's time to
>>> collect
>>> our thoughts for mentorin
On 12/27/2014 11:23 PM, Paolo Bonzini wrote:
>
>
> On 26/12/2014 04:31, Yang Hongyang wrote:
>> Please feel free to comment.
>> We want comments/feedbacks as many as possiable please, thanks in advance.
>
> Hi Yang,
>
> I think it's possible to build COLO block replication from many basic
> blo
I am not sure whether I have understood your request completely. QEMU
can access raw partitions if the OS supports them. MS Windows supports
physical partitions and calls them \\.\PhysicalDrive0,
\\.\PhysicalDrive1 and so on. Admin rights are required to access these
devices. Depending on your shel
On 27/01/15 21:05, Max Reitz wrote:
On 2015-01-27 at 08:51, Denis V. Lunev wrote:
fallocate() works fine and could handle properly with arbitrary size
requests.
Maybe "could properly handle arbitrary size requests" (or
"...arbitrarily sized requests")?
There is no sense to reduce the amoun
On 2015/1/27 18:30, Igor Mammedov wrote:
> On Tue, 27 Jan 2015 14:47:44 +0800
> Shannon Zhao wrote:
>
>> On 2015/1/26 18:19, Igor Mammedov wrote:
>>> On Sat, 24 Jan 2015 17:21:11 +0800
>>> Shannon Zhao wrote:
>>>
Introduce a preliminary framework in virt-acpi-build.c with the main
ACPI
On Thu, 01/22 15:47, Paolo Bonzini wrote:
> Note that even after this patch, most callers of address_space_*
> functions must still be under the big QEMU lock, otherwise the memory
> region returned by address_space_translate can disappear as soon as
> address_space_translate returns. This will be
Quoting Bharata B Rao (2015-01-27 22:19:56)
> On Thu, Jan 22, 2015 at 04:16:01PM -0600, Michael Roth wrote:
> > Quoting Bharata B Rao (2015-01-08 00:10:13)
> > > +static void spapr_machine_device_plug(HotplugHandler *hotplug_dev,
> > > + DeviceState *dev, Error
On Tue, Dec 23, 2014 at 07:41:22AM +0100, Alexander Graf wrote:
>
>
>
> > Am 23.12.2014 um 04:56 schrieb David Gibson :
> >
> >> On Tue, Dec 23, 2014 at 01:30:11AM +0100, Alexander Graf wrote:
> >>
> >>
> >>> On 23.12.14 01:17, David Gibson wrote:
> >>> The initial creation of the PAPR RTC qd
On Tue, Jan 27, 2015 at 10:04:31AM +, Dr. David Alan Gilbert wrote:
> * David Gibson (da...@gibson.dropbear.id.au) wrote:
> > On Wed, Dec 17, 2014 at 06:21:34PM +, Dr. David Alan Gilbert wrote:
> > > * David Gibson (da...@gibson.dropbear.id.au) wrote:
> > > > On Fri, Oct 03, 2014 at 06:47:4
On Tue, Dec 23, 2014 at 07:33:30AM +0100, Alexander Graf wrote:
>
>
>
> > Am 23.12.2014 um 04:14 schrieb David Gibson :
> >
> >> On Tue, Dec 23, 2014 at 01:26:58AM +0100, Alexander Graf wrote:
> >>
> >>
> >>> On 23.12.14 01:17, David Gibson wrote:
> >>> On x86, the guest's RTC can be read wit
On Tue, Jan 27, 2015 at 09:40:12AM +, Dr. David Alan Gilbert wrote:
> * David Gibson (da...@gibson.dropbear.id.au) wrote:
> > On Wed, Jan 14, 2015 at 08:13:27PM +, Dr. David Alan Gilbert wrote:
> > > * David Gibson (da...@gibson.dropbear.id.au) wrote:
> > > > On Fri, Oct 03, 2014 at 06:47:4
On 1/28/15 00:11, Michael Tokarev wrote:
> 25.01.2015 14:03, Chen Gang S wrote:
>> start/end_exclusive() need be pairs, except the start_exclusive() in
>
> "need TO be pairs", or "should be pairs" or "should be called in pairs".
>
>> stop_all_tasks() which is only used by force_sig(), which will
> -Original Message-
> From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com]
> Sent: Tuesday, January 20, 2015 1:19 AM
> To: Xu, Quan
> Cc: qemu-devel@nongnu.org; xen-de...@lists.xen.org;
> stefano.stabell...@eu.citrix.com
> Subject: Re: [v3 3/5] Qemu-Xen-vTPM: Register Xen s
On Thu, Jan 22, 2015 at 04:16:01PM -0600, Michael Roth wrote:
> Quoting Bharata B Rao (2015-01-08 00:10:13)
> > +static void spapr_machine_device_plug(HotplugHandler *hotplug_dev,
> > + DeviceState *dev, Error **errp)
> > +{
> > +if (object_dynamic_cast(OBJE
On Mon, Jan 26, 2015 at 10:26 PM, Michael Roth
wrote:
> Quoting David Gibson (2015-01-18 22:31:23)
>> On Tue, Dec 23, 2014 at 06:30:22AM -0600, Michael Roth wrote:
>> > +
>> > +static void spapr_hotplug_req_event(sPAPRDRConnector *drc, uint8_t
>> > hp_action)
>> > +{
>> > +struct hp_log_full
I think I mistakenly got put on the CC list for this. Can the next replyer
please remove j...@redhat.com? Thanks.
--
John Priddy
Solutions Architect
Red Hat
+1 (917) 576-8023 - Mobile
j...@redhat.com
- Original Message -
> From: "Eric Blake"
> To: "Max Reitz" , qemu-devel@nongnu.org
>
I have two Ubuntu 14.04 KVM instances (kernel version
3.16.0-29-generic). (Running under qemu 2.2.0) They're connected using
qemu's 'multicast' network type, and basic connectivity works fine. I
can ping between the guests, they can transfer data, etc.
I'll refer to them this way:
guest1 -
Public bug reported:
I'm using a windows tablet that makes imposible usb booting. It would be nice
to have access to raw partitions in order to run linux installers using qemu. I
can successfully install several boot loaders using uefi, so I gues this
feature would be very helpful.
Thanks!
**
Firstly, thank you for your work about the related patches and replying
so much details.
On 1/27/15 19:04, Michael Tokarev wrote:
> 23.01.2015 13:01, Chen Gang S wrote:
>> When failure occurs during allocating vec[i], also need free all
>> allocated vec[i] in failure processing code block before
This will allow test cases to run command in qtest protocol. It's
write-only for now.
Signed-off-by: Fam Zheng
Reviewed-by: Max Reitz
---
tests/qemu-iotests/iotests.py | 16 ++--
1 file changed, 14 insertions(+), 2 deletions(-)
diff --git a/tests/qemu-iotests/iotests.py b/tests/qem
This will enable accounting of aio requests issued from qemu-io aio
read/write commands.
Signed-off-by: Fam Zheng
Reviewed-by: Max Reitz
---
qemu-io-cmds.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/qemu-io-cmds.c b/qemu-io-cmds.c
index e708552..29377cd 100644
--- a/qemu-i
This case utilizes qemu-io command "aio_{read,write} -q" to verify the
effectiveness of IO throttling options.
It's implemented by driving the vm timer from qtest protocol, so the
throttling timers are signaled with determinied time duration. Then we
verify the completed IO requests are within 10%
This adds scripts/qtest.py as a python library for qtest protocol.
This is a skeleton with a basic "cmd" method to execute a command,
reading and parsing of qtest output could be added later on demand.
Signed-off-by: Fam Zheng
Reviewed-by: Max Reitz
---
scripts/qtest.py | 71 ++
v6: Less resource demanding patch 5. (Max)
Add rev-by of Max to other patches.
v5: Rebase and improve the test. Please review again.
Patch dependencies:
This test depends on the qtest timer fix to run correctly.
http://lists.gnu.org/archive/html/qemu-devel/2015-01/msg01865.html
QMP command "block_set_io_throttle" expects underscores in parameters
instead of dashes: {iops,bps}_{rd,wr,max}.
Add optional argument conv_keys (defaults to True, backward compatible),
it will be used in IO throttling test case.
Reviewed-by: Benoit Canet
Signed-off-by: Fam Zheng
Reviewed-by: M
On Mon, Jan 26, 2015 at 6:18 PM, Sebastian Macke wrote:
> Hi Jia,
>
>
> On 1/26/2015 10:50 AM, Jia Liu wrote:
>>
>> Hi Sebastian, Christian
>>
>> On Sun, Jan 25, 2015 at 6:25 PM, Sebastian Macke
>> wrote:
>>>
>>> From: Christian Svensson
>>>
>>> This patch adds support for atomic locks
>>> and i
On Tue, 01/27 11:14, Max Reitz wrote:
> On 2015-01-26 at 22:03, Fam Zheng wrote:
> >On Mon, 01/26 15:45, Max Reitz wrote:
> >>On 2015-01-16 at 03:46, Fam Zheng wrote:
> >>>This case utilizes qemu-io command "aio_{read,write} -q" to verify the
> >>>effectiveness of IO throttling options.
> >>>
> >>>
The commit 533ffb17a that removed qed_aiocb_info.cancel said to remove
this but didn't do it.
Signed-off-by: Fam Zheng
---
block/qed.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/block/qed.h b/block/qed.h
index d3934a0..615e676 100644
--- a/block/qed.h
+++ b/block/qed.h
@@ -133,7 +133,6 @
On 2015/1/27 18:49, Dinar Valeev wrote:
> On 01/27/2015 10:18 AM, Gonglei wrote:
>> On 2015/1/27 16:57, Dinar Valeev wrote:
>>
>>> On 01/27/2015 03:51 AM, Gonglei wrote:
On 2015/1/27 7:52, dval...@suse.de wrote:
> From: Dinar Valeev
>
> on sPAPR we need to update boot_order
Call blk_is_available() before using blk_bs() to obtain the root
BlockDriverState behind the BlockBackend.
Signed-off-by: Max Reitz
---
blockdev.c | 13 ++---
1 file changed, 10 insertions(+), 3 deletions(-)
diff --git a/blockdev.c b/blockdev.c
index 9801a7e..74d26a6 100644
--- a/blockd
This structure will store some of the state of the root BDS if the BDS
tree is removed, so that state can be restored once a new BDS tree is
inserted.
Signed-off-by: Max Reitz
---
block/block-backend.c | 26 ++
include/block/block_int.h | 9 +
inclu
Change abandoned by Tony Breeds (t...@bakeyournoodle.com) on branch: master
Review: https://review.openstack.org/123957
Reason: The main distros we care about have landed or are in progress.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to Q
In the drive-backup transaction, call blk_is_available() before using
blk_bs() to obtain the root BlockDriverState behind the BlockBackend.
Signed-off-by: Max Reitz
---
blockdev.c | 15 ++-
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/blockdev.c b/blockdev.c
index 0
Quoting Tyrel Datwyler (2015-01-27 16:05:52)
> On 01/27/2015 01:36 PM, Michael Roth wrote:
> > Quoting David Gibson (2015-01-26 23:24:11)
> >> On Sun, Jan 25, 2015 at 11:21:26PM -0600, Michael Roth wrote:
> >>> Quoting David Gibson (2015-01-16 00:21:55)
> On Tue, Dec 23, 2014 at 06:30:17AM -06
On 2015/1/27 22:40, Ian Jackson wrote:
Chen, Tiejun writes ("Re: [Qemu-devel] [RFC][PATCH 1/1] libxl: add one machine
property to support IGD GFX passthrough"):
On 2015/1/23 8:43, Chen, Tiejun wrote:
On 2015/1/22 8:51, Chen, Tiejun wrote:
At this point I just concern here if we still use 'gfx
On Mon, 26 Jan 2015, Leon Alrae wrote:
> BadVAddr is supposed to capture the most recent address that caused
> the exception. Currently this is not happening as translation is not stopped
> and BadVAddr is updated with subsequent addresses.
>
> Signed-off-by: Leon Alrae
> ---
I think this dese
On Mon, 26 Jan 2015, Leon Alrae wrote:
> Signed-off-by: Leon Alrae
> ---
Reviewed-by: Maciej W. Rozycki
Maciej
On Mon, 26 Jan 2015, Leon Alrae wrote:
> The test is supposed to terminate TB if the end of the page is reached.
> However, with current implementation it may never succeed for microMIPS or
> mips16.
>
> Reported-by: Richard Henderson
> Signed-off-by: Leon Alrae
> ---
I'm not sure if you need
Added machvirt parsing of feature keywords added to the -cpu command line
option. Parsing occurs during machine initialization.
Signed-off-by: Greg Bellows
---
v1 -> v2
- Fix multiple property handling
---
hw/arm/virt.c | 20 +---
1 file changed, 17 insertions(+), 3 deletions(
Add AArch32 to AArch64 register sychronization functions.
Replace manual register synchronization with new functions in
aarch64_cpu_do_interrupt() and HELPER(exception_return)().
Signed-off-by: Greg Bellows
---
v2 -> v3
- Conditionalize interrupt handler update of aarch64.
---
target-arm/helpe
Add 32-bit to/from 64-bit register synchronization on register gets and puts.
Set EL1_32BIT feature flag passed to KVM
Signed-off-by: Greg Bellows
---
v2 -> v3
- Conditionalize sync of 32-bit and 64-bit registers
---
target-arm/kvm64.c | 33 +
1 file changed, 29
Adds registration and get/set functions for enabling/disabling the AArch64
execution state on AArch64 CPUs. By default AArch64 execution state is enabled
on AArch64 CPUs, setting the property to off, will disable the execution state.
The below QEMU invocation would have AArch64 execution state dis
Added support for running an AArch32 guest on a AArch64 KVM host. Support has
only been added to the QEMU machvirt machine. The addition of CPU properties
specifiable from the command line were added to allow disablement of AArch64
execution state hereby forcing EL1 to be AArch32. The new CPU co
On 01/27/2015 12:45 PM, Max Reitz wrote:
> In the drive-backup transaction, call blk_is_available() before using
> blk_bs() to obtain the root BlockDriverState behind the BlockBackend.
Took me a while to figure that TA meant transaction in the subject.
Either spell it out, or just drop 'TA' entire
available in the git repository at:
>
> git://git.linaro.org/people/riku.voipio/qemu.git
> tags/pull-linux-user-20150127
>
> for you to fetch changes up to 30b8b68eb574fd68060eebcc4da790fdfe18d668:
>
> linux-user: support targe
On 01/27/2015 12:45 PM, Max Reitz wrote:
> Call blk_is_available() before using blk_bs() to obtain the root
> BlockDriverState behind the BlockBackend.
>
> Signed-off-by: Max Reitz
> ---
> blockdev.c | 10 ++
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
Reviewed-by: Eric Blake
On Wed, Jan 28, 2015 at 4:37 AM, Peter Maydell wrote:
> On 24 January 2015 at 08:02, Alistair Francis wrote:
>> This patch moves the memory region init code from the
>> armv7m_init function to the stellaris_init function
>>
>> Signed-off-by: Alistair Francis
>> Reviewed-by: Peter Crosthwaite
>>
On Mon, 26 Jan 2015 18:17:55 +0200
"Michael S. Tsirkin" wrote:
> On Mon, Jan 26, 2015 at 04:34:01PM +0100, Andrew Jones wrote:
> > On Mon, Jan 26, 2015 at 04:09:20PM +0100, Igor Mammedov wrote:
> > > On Mon, 26 Jan 2015 10:57:21 +0100
> > > Igor Mammedov wrote:
> > >
> > > > On Sat, 24 Jan 2015
From: Felix Janda
The resource argument is translated from host to target for
[gs]etprlimit but not for prlimit64. Fix this.
Signed-off-by: Felix Janda
Signed-off-by: Riku Voipio
---
linux-user/syscall.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/linux-user/syscall.
From: Peter Maydell
The function copy_siginfo_to_user() just calls tswap_siginfo(), so
call the latter function directly and delete the wrapper function.
The wrapper is actually misleading since it implies that the
semantics are like the kernel function with the same name which
copies the data to
On Tue, Jan 27, 2015 at 3:21 PM, Christoffer Dall <
christoffer.d...@linaro.org> wrote:
> On Wed, Jan 21, 2015 at 12:49:53PM -0600, Greg Bellows wrote:
> > Add 32-bit to/from 64-bit register synchronization on register gets and
> puts.
> > Set EL1_32BIT feature flag passed to KVM
> >
> > Signed-of
From: Peter Maydell
The function end_exclusive() isn't used on all targets; mark it as
such to avoid a clang warning.
Signed-off-by: Peter Maydell
Signed-off-by: Riku Voipio
---
linux-user/main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/linux-user/main.c b/linux-use
From: Ed Swierk
linux-user passes the cmd argument of the ioctl syscall as a signed long,
but compares it to an unsigned int when iterating through the ioctl_entries
list. When the cmd is a large value like 0x80047476 (TARGET_TIOCSWINSZ on
mips64) it gets sign-extended to 0x80047476, cau
On 2015-01-27 at 16:59, Eric Blake wrote:
On 01/27/2015 12:45 PM, Max Reitz wrote:
When preparing a blockdev-backup transaction, the BlockBackend should be
used because there may be no medium associated to the BB (which would
make bdrv_find() fail, whereas blk_by_name() does not).
This does not
On 01/27/2015 12:45 PM, Max Reitz wrote:
> Check whether the BlockBackend is actually available at the start of
> snapshot-delete-internal-sync.
Cute abbreviation in the subject line.
>
> Signed-off-by: Max Reitz
> ---
> blockdev.c | 18 +-
> 1 file changed, 13 insertions(+), 5
o/qemu.git tags/pull-linux-user-20150127
for you to fetch changes up to 30b8b68eb574fd68060eebcc4da790fdfe18d668:
linux-user: support target-to-host SCM_CREDENTIALS (2015-01-27 22:33:57 +0200)
linux-user updates since last pu
From: Peter Maydell
The start_exclusive() infrastructure is used on all target
architectures, even if only to do the "stop all CPUs before
dumping core" in force_sig(), so be consistent and call
cpu_exec_start/end in the main loop of every target.
Signed-off-by: Peter Maydell
Signed-off-by: Rik
On 01/27/2015 01:36 PM, Michael Roth wrote:
> Quoting David Gibson (2015-01-26 23:24:11)
>> On Sun, Jan 25, 2015 at 11:21:26PM -0600, Michael Roth wrote:
>>> Quoting David Gibson (2015-01-16 00:21:55)
On Tue, Dec 23, 2014 at 06:30:17AM -0600, Michael Roth wrote:
> From: Nathan Fontenot
>>
On 01/27/2015 12:45 PM, Max Reitz wrote:
> When preparing a blockdev-backup transaction, the BlockBackend should be
> used because there may be no medium associated to the BB (which would
> make bdrv_find() fail, whereas blk_by_name() does not).
>
> This does not make a real difference because blo
On 01/27/2015 12:45 PM, Max Reitz wrote:
> Signed-off-by: Max Reitz
> ---
> block/qapi.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
Reviewed-by: Eric Blake
--
Eric Blake eblake redhat com+1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.a
On 01/27/2015 12:45 PM, Max Reitz wrote:
> blk_by_name() may return a BlockBackend for which blk_bs() returns NULL.
> In this case, an error should be returned (instead of just returning
> NULL without modifying *errp).
>
> Signed-off-by: Max Reitz
> ---
> block.c | 5 +
> 1 file changed, 5
From: Peter Maydell
The m68k signal frame setup code which writes the signal return
trampoline code to the stack was assuming that a 'long' was 32 bits;
on 64 bit systems this meant we would end up writing the 32 bit
(2 insn) trampoline sequence to retaddr+4,retaddr+6 instead of
the intended reta
From: Peter Maydell
Remove the function current_exec_domain_sig(), which always returns
its argument. This was intended as a stub for supporting the kernel's
exec_domain handling, but:
* we don't have any of the other code for execution domains
* in the kernel this handling is architecture-spec
On Tue, Jan 27, 2015 at 3:20 PM, Christoffer Dall <
christoffer.d...@linaro.org> wrote:
> On Wed, Jan 21, 2015 at 12:49:52PM -0600, Greg Bellows wrote:
> > Add AArch32 to AArch64 register sychronization functions.
> > Replace manual register synchronization with new functions in
> > aarch64_cpu_do
On 2015-01-27 at 16:36, Eric Blake wrote:
On 01/27/2015 12:45 PM, Max Reitz wrote:
There are several BlockBackend functions which, in theory, cannot fail.
This patch makes them cope with the BlockDriverState pointer being NULL
by making them fall back to some default action like ignoring the val
Quoting David Gibson (2015-01-26 23:24:11)
> On Sun, Jan 25, 2015 at 11:21:26PM -0600, Michael Roth wrote:
> > Quoting David Gibson (2015-01-16 00:21:55)
> > > On Tue, Dec 23, 2014 at 06:30:17AM -0600, Michael Roth wrote:
> > > > From: Nathan Fontenot
> > > >
> > > > Signed-off-by: Nathan Fonteno
From: Ed Swierk
Without this fix, qemu segfaults when emulating the sigaltstack syscall,
because it incorrectly treats the ss_flags field as 64 bits rather than 32
bits.
Signed-off-by: Ed Swierk
Signed-off-by: Riku Voipio
---
linux-user/mips64/target_signal.h | 2 +-
1 file changed, 1 inserti
On 01/27/2015 12:45 PM, Max Reitz wrote:
> There are several BlockBackend functions which, in theory, cannot fail.
> This patch makes them cope with the BlockDriverState pointer being NULL
> by making them fall back to some default action like ignoring the value
> in setters and returning the defau
On 01/27/2015 12:45 PM, Max Reitz wrote:
> If there is no BlockDriverState in a BlockBackend or if the tray of the
> guest device is open, fail all requests (where that is possible) with
> -ENOMEDIUM.
>
> The reason the status of the guest device is taken into account is
> because once the guest d
El Mon, 19 Jan 2015 15:15:54 -0500
John Snow escribió:
> To avoid the architecture-specific implementations of the generic
> qtest allocator having to know about fields within the allocator, add
> a page_size setter method for users or arch specializations to use.
> The allocator will assume a def
On 01/27/2015 05:13 PM, Alexey Kardashevskiy wrote:
> Instead of tweaking a TCE table device by adding there a bypass flag,
> let's add an alias to RAM and IOMMU memory region, and enable/disable
> those according to the selected bypass mode.
> This way IOMMU memory region can have size of the actu
On Wed, Jan 21, 2015 at 12:49:53PM -0600, Greg Bellows wrote:
> Add 32-bit to/from 64-bit register synchronization on register gets and puts.
> Set EL1_32BIT feature flag passed to KVM
>
> Signed-off-by: Greg Bellows
> ---
> target-arm/kvm64.c | 21 +
> 1 file changed, 17 ins
On Wed, Jan 21, 2015 at 12:49:52PM -0600, Greg Bellows wrote:
> Add AArch32 to AArch64 register sychronization functions.
> Replace manual register synchronization with new functions in
> aarch64_cpu_do_interrupt() and HELPER(exception_return)().
>
> Signed-off-by: Greg Bellows
> ---
> target-ar
On 01/27/2015 12:45 PM, Max Reitz wrote:
> If there is no BDS tree attached to a BlockBackend, functions that can
> do so should fall back to the BlockBackendRootState structure (which are
> blk_is_read_only() and blk_get_flags(), because the read-only status and
> the "open flags" are part of the
From: Peter Maydell
For historical reasons, the define for the shmat() syscall on Alpha is
NR_osf_shmat; however it has the same semantics as this syscall does
on all other architectures, so define TARGET_NR_shmat as well so that
QEMU's code for the syscall is enabled.
This patch brings our beha
From: Alex Suykov
When passing ancillary data through a unix socket, handle
credentials properly instead of doing a simple copy and
issuing a warning.
Signed-off-by: Alex Suykov
Signed-off-by: Riku Voipio
---
linux-user/syscall.c | 18 ++
1 file changed, 14 insertions(+), 4 de
From: Peter Maydell
The aCC array in fpopcode.c is completely unused in QEMU; delete
it (silencing a clang warning).
Signed-off-by: Peter Maydell
Signed-off-by: Riku Voipio
---
linux-user/arm/nwfpe/fpopcode.c | 22 --
1 file changed, 22 deletions(-)
diff --git a/linux-use
Ping: Waiting for reviews on patches #6 (Marc?), and re-reviews for
patches #12 and #15 (Paolo?)
Thanks
--JS
On 01/19/2015 03:15 PM, John Snow wrote:
This series aims to do two main things:
(1) Eliminate global state out of the ahci-test file so that
the tests are more functional. This w
On 01/27/2015 12:45 PM, Max Reitz wrote:
> This structure will store some of the state of the root BDS if the BDS
> tree is removed, so that state can be restored once a new BDS tree is
> inserted.
>
> Signed-off-by: Max Reitz
> ---
> block/block-backend.c | 26 +
On 27 January 2015 at 20:04, Marc Marí wrote:
> The problem was mine, I specified (because I did not change if from
> previous boots) 1G as memory for the Versatile machine, which only has
> 256 MB of RAM
>
> Should this be a bug, or is undefined behaviour? I think it is not very
> difficult to ch
This function removes the BlockDriverState associated with the given
BlockBackend from that BB and sets the BDS pointer in the BB to NULL.
Signed-off-by: Max Reitz
---
block/block-backend.c | 22 +-
include/sysemu/block-backend.h | 1 +
2 files changed, 22 insertion
On 01/27/2015 12:45 PM, Max Reitz wrote:
> These options are only relevant for the user of a whole BDS tree (like a
> guest device or a block job) and should thus be moved into the
> BlockBackend.
>
> Signed-off-by: Max Reitz
> ---
> block.c| 125
> --
Call blk_is_available() before using blk_bs() to obtain the root
BlockDriverState behind the BlockBackend.
Signed-off-by: Max Reitz
---
blockdev.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/blockdev.c b/blockdev.c
index 858d181..012c603 100644
--- a/blockdev.c
Add an option to qmp_blockdev_change_medium() which allows changing the
read-only status of the block device whose medium is changed.
Some drives do not have a inherently fixed read-only status; for
instance, floppy disks can be set read-only or writable independently of
the drive. Some users may
If the "id" field is missing from the options given to blockdev-add,
just omit the BlockBackend and create the BlockDriverState tree alone.
Signed-off-by: Max Reitz
---
blockdev.c | 38 +-
tests/qemu-iotests/087 | 20
t
If there is no BDS tree attached to a BlockBackend, functions that can
do so should fall back to the BlockBackendRootState structure (which are
blk_is_read_only() and blk_get_flags(), because the read-only status and
the "open flags" are part of the BBRS).
Signed-off-by: Max Reitz
---
block/bloc
[replies to earlier patches in the other thread; moving the rest of my
review here for the sake of improved cc]
On 01/27/2015 12:45 PM, Max Reitz wrote:
> As the comment above bdrv_get_stats() says, BlockAcctStats is something
> which belongs to the device instead of each BlockDriverState. This pa
Check whether the BlockBackend is actually available at the start of
snapshot-delete-internal-sync.
Signed-off-by: Max Reitz
---
blockdev.c | 18 +-
1 file changed, 13 insertions(+), 5 deletions(-)
diff --git a/blockdev.c b/blockdev.c
index ae1137f..858d181 100644
--- a/blockdev
On 27 January 2015 at 19:49, Greg Bellows wrote:
> On Tue, Jan 27, 2015 at 12:12 PM, Peter Maydell
> wrote:
>> g_assert_not_reached(), but it didn't seem worth cluttering
>> the switch with a bunch of extra labels just to assert that
>> they weren't reachable.
>>
>
> I see how it could clutter t
1 - 100 of 329 matches
Mail list logo