On Thu, 2012-04-19 at 19:20 -0500, Anthony Liguori wrote:
> Hi Nicholas,
>
> On 04/19/2012 06:53 PM, Nicholas A. Bellinger wrote:
> > On Thu, 2012-04-19 at 07:30 -0500, Anthony Liguori wrote:
> >> However, for storage, be it scsi or direct access, the same problem really
> >> doesn't exist. There
Il 19/04/2012 21:34, Luiz Capitulino ha scritto:
> If this turns out to be a problem for libvirt, I'd say it's a libvirt bug.
FWIW, I agree completely.
Paolo
On Fri, Apr 20, 2012 at 5:00 PM, Nicholas A. Bellinger
wrote:
> On Thu, 2012-04-19 at 19:20 -0500, Anthony Liguori wrote:
>> Hi Nicholas,
>>
>> On 04/19/2012 06:53 PM, Nicholas A. Bellinger wrote:
>> > On Thu, 2012-04-19 at 07:30 -0500, Anthony Liguori wrote:
>> >> However, for storage, be it scsi
On Fri, 2012-04-20 at 17:50 +1000, ronnie sahlberg wrote:
> On Fri, Apr 20, 2012 at 5:00 PM, Nicholas A. Bellinger
> wrote:
> > On Thu, 2012-04-19 at 19:20 -0500, Anthony Liguori wrote:
> >> Hi Nicholas,
> >>
> > The argument that a SCSI target for virtual machines is so complex that
> > it can
On 04/20/2012 07:40 AM, Amos Kong wrote:
> Add a bool argument to inet_connect() to assign if set socket
> to block/nonblock, and delete original argument 'socktype'
> that is unused.
> Add a new argument to inet_connect()/inet_connect_opts(),
> to pass back connect error by error class.
>
> Retry
On 04/20/2012 07:43 AM, Amos Kong wrote:
> Use help functions in qemu-socket.c for tcp migration,
> which already support ipv6 addresses.
>
> Currently errp will be set to UNDEFINED_ERROR when migration fails,
> qemu would output "migration failed: ...", and current user can
> see a message("An un
Am 19.04.2012 23:18, schrieb Marcelo Tosatti:
> On Thu, Apr 19, 2012 at 05:14:20PM -0300, Marcelo Tosatti wrote:
>>> There is one intended change in functionality in this patch, which is
>>> that it allocates new clusters even when it could satisfy the first part
>>> of the request with already all
Il 20/04/2012 09:00, Nicholas A. Bellinger ha scritto:
> On Thu, 2012-04-19 at 19:20 -0500, Anthony Liguori wrote:
>> TCM runs in the absolute most privileged context possible. When you're
>> dealing
>> with extremely hostile input, it's pretty obvious that you want to run it in
>> the
>> lowe
Am 13.04.2012 11:17, schrieb Liu Yuan:
> From: Liu Yuan
>
> The 'qemu-img convert -h' advertise that the default cache mode is
> 'writeback', while in fact it is 'unsafe'.
>
> This patch 1) fix the help manual and 2) let bdrv_close() call bdrv_flush()
>
> 2) is needed because some backend stora
Hi. Sorry for the slow follow-up.
Stefan Hajnoczi writes:
> Yes, it's odd that QEMU changes make the issue go away but tcpdump
> suggests the packet is not being sent from the bridge to the tap
> device.
Indeed. I really don't understand how to the two could possible interact!
> e1000 and rtl8
[...]
>> [Patch 5]
>> Track-alignment-explicitly
>> Almost the same as the previous, just changed priority from r->align to
>> r->sum when setting start address of root regions.
>>
>> I guess there are more chances to fit memory regions if we try place regions
>> with higher r->sum like it was b
Am 20.04.2012 02:14, schrieb Michael Roth:
> On Wed, Apr 18, 2012 at 01:11:13AM +0200, Andreas Färber wrote:
>> Signed-off-by: Andreas Färber
>> ---
>> target-i386/cpu.c | 14 +-
>> 1 files changed, 13 insertions(+), 1 deletions(-)
>>
>> diff --git a/target-i386/cpu.c b/target-i386/
From: Liu Yuan
The 'qemu-img convert -h' advertise that the default cache mode is
'writeback', while in fact it is 'unsafe'.
This patch 1) fix the help manual and 2) let bdrv_close() call bdrv_flush()
2) is needed because some backend storage doesn't have a self-flush
mechanism(for e.g., sheepd
Am 20.04.2012 11:10, schrieb Liu Yuan:
> From: Liu Yuan
>
> The 'qemu-img convert -h' advertise that the default cache mode is
> 'writeback', while in fact it is 'unsafe'.
>
> This patch 1) fix the help manual and 2) let bdrv_close() call bdrv_flush()
>
> 2) is needed because some backend stora
On Fri, Apr 20, 2012 at 12:21 AM, andrzej zaborowski wrote:
> On 18 April 2012 14:35, Stefan Hajnoczi wrote:
>> Recently there have been new SD card emulation patches so I want to
>> raise the issue of synchronous I/O while there is focus on the SD
>> subsystem. Maybe some of the people who are
Am 24.02.2012 19:24, schrieb Anthony Liguori:
> On 02/24/2012 12:09 PM, Andreas Färber wrote:
>> Am 24.02.2012 18:27, schrieb Anthony Liguori:
>>> On 02/24/2012 11:11 AM, Gerhard Wiesinger wrote:
Hello,
I'm having compile/link errors on
85f38553031b1a6e07f786c9ab0d403af7252b4f:
These patches remove timer code which is no longer needed
and try to improve the remaining code.
[PATCH 1/5] qemu-timer: Remove redundant include statements
[PATCH 2/5] qemu-timer: Remove unused function qemu_alarm_pending
[PATCH 3/5] qemu-timer: Use bool, false, true for boolean values
[PATCH 4/5
Some time ago, the last time which did not have dynticks was removed,
so now all timers have dynticks.
I also removed a misleading error message for the dynticks timer.
If timer_create fails, there is already an error message, and
QEMU will use the unix timer which also provides dynamic ticks,
the
The last user of this function was removed by commit
12d4536f7d911b6d87a766ad7300482ea663cea2.
Signed-off-by: Stefan Weil
---
qemu-timer.c |5 -
qemu-timer.h |1 -
2 files changed, 0 insertions(+), 6 deletions(-)
diff --git a/qemu-timer.c b/qemu-timer.c
index 87d9636..38adb0d 100644
On 2012-04-19 22:03, Eduardo Habkost wrote:
> Jan/Avi: ping?
>
> I would like to get this ABI detail clarified so it can be implemented
> the right way on Qemu and KVM.
>
> My proposal is to simply add tsc-deadline to the data returned by
> GET_SUPPORTED_CPUID, making KVM_CAP_TSC_DEADLINE_TIMER u
Il 20/04/2012 12:03, Stefan Weil ha scritto:
> Some time ago, the last time which did not have dynticks was removed,
> so now all timers have dynticks.
>
> I also removed a misleading error message for the dynticks timer.
> If timer_create fails, there is already an error message, and
> QEMU will
Am 13.03.2012 03:23, schrieb Anthony Liguori:
> On 03/12/2012 10:19 AM, Kevin Wolf wrote:
>> The following changes since commit
>> a348f108842fb928563865c9918642900cd0d477:
>>
>>Add missing const attributes for MemoryRegionOps (2012-03-11
>> 11:40:15 +)
>>
>> are available in the git reposi
Am 20.04.2012 12:16, schrieb Andreas Färber:
> Am 13.03.2012 03:23, schrieb Anthony Liguori:
>> On 03/12/2012 10:19 AM, Kevin Wolf wrote:
>>> The following changes since commit
>>> a348f108842fb928563865c9918642900cd0d477:
>>>
>>>Add missing const attributes for MemoryRegionOps (2012-03-11
>>>
qemu-timer.h includes qemu-common.h which already includes time.h,
sys/time.h, windows.h, unistd.h, fcntl.h, errno.h and signal.h.
Therefore those include statements are redundant and can be removed.
Signed-off-by: Stefan Weil
---
qemu-timer.c | 11 ++-
qemu-timer.h |6 --
2 f
On 04/19/2012 04:08 PM, Vasilis Liaskovitis wrote:
The numa_fw_cfg paravirt interface is extended to include SRAT information for
all hotplug-able memslots. There are 3 words for each hotplug-able memory
slot,
denoting start address, size and node proximity. nb_numa_nodes is set to 1 by
Am 20.04.2012 12:16, schrieb Andreas Färber:
> Am 13.03.2012 03:23, schrieb Anthony Liguori:
>> On 03/12/2012 10:19 AM, Kevin Wolf wrote:
>>> The following changes since commit
>>> a348f108842fb928563865c9918642900cd0d477:
>>>
>>>Add missing const attributes for MemoryRegionOps (2012-03-11
>>>
This avoids conversions between int and bool / char.
It also makes the code more readable.
Signed-off-by: Stefan Weil
---
qemu-timer.c | 31 ---
qemu-timer.h |6 +++---
2 files changed, 19 insertions(+), 18 deletions(-)
diff --git a/qemu-timer.c b/qemu-timer.c
Am 20.04.2012 06:16, schrieb David Gibson:
> I'm really hoping I can get some extra review of this code path. I
> believe it's correct, but it's not straightforward to test, since it
> will not be exercise by correct guest software.
Sounds like a good case for qtest. Did you consider that?
Kevin
Am 20.04.2012 12:37, schrieb Kevin Wolf:
> Am 20.04.2012 12:16, schrieb Andreas Färber:
>> [...] I've been seeing the following annoying
>> problem on master:
>>
>> $ make check-block
>> /home/andreas/QEMU/qemu/tests/qemu-iotests-quick.sh
>> hostname: Name or service not known
>> hostname: Name or
Remove all holes which were found by pahole on Linux x86_64
(and replace "struct QEMUTimer" by "QEMUTimer").
Signed-off-by: Stefan Weil
---
qemu-timer.c | 14 +++---
1 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/qemu-timer.c b/qemu-timer.c
index 1fbc2df..2546640 100644
Some time ago, the last time which did not have dynticks was removed,
so now all timers have dynticks.
I also removed a misleading error message for the dynticks timer.
If timer_create fails, there is already an error message, and
QEMU will use the unix timer which also provides dynamic ticks,
the
Am 20.04.2012 12:38, schrieb Andreas Färber:
> Am 20.04.2012 12:37, schrieb Kevin Wolf:
>> Am 20.04.2012 12:16, schrieb Andreas Färber:
>>> [...] I've been seeing the following annoying
>>> problem on master:
>>> However `hostname` returns ".site" just fine, and I don't
>>> understand why a user's
`hostname -s` may output an errror:
hostname: Name or service not known
This causes all tests to fail for `make check-block`.
Suppress such error messages, letting the tests succeed.
Signed-off-by: Andreas Färber
---
tests/qemu-iotests/common.config |2 +-
1 files changed, 1 insertions(+),
On 04/19/2012 04:08 PM, Vasilis Liaskovitis wrote:
Extend the DSDT to include methods for handling memory hot-add and hot-remove
notifications and memory device status requests. These functions are called
from the memory device SSDT methods.
Eject has only been tested with level gpe event, but w
Am 20.04.2012 12:50, schrieb Andreas Färber:
> `hostname -s` may output an errror:
> hostname: Name or service not known
> This causes all tests to fail for `make check-block`.
>
> Suppress such error messages, letting the tests succeed.
>
> Signed-off-by: Andreas Färber
Thanks, applied to the
This patch adds a function which creates unique serial numbers for usb
devices and puts it into use. Windows guests tend to become unhappy if
they find two identical usb devices in the system. Effects range from
non-functional devices (with yellow exclamation mark in device manager)
to BSODs. Ha
Fix a small copy and paste error in logging.
Signed-off-by: Hans de Goede
---
hw/ac97.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/ac97.c b/hw/ac97.c
index dd4917b..05ae63b 100644
--- a/hw/ac97.c
+++ b/hw/ac97.c
@@ -339,7 +339,7 @@ static uint16_t mixer_load (AC97L
The Linux ac97 driver tries to see if optional things like video input
volume control are available in 2 ways:
1) See if the mute bit is set after reset, if it is no further tests are done
2) If the mute bit is not set it does a write/read test of the mute bit
This patch changes our ac97 to confor
After commit 19677a380a70348134ed7650b294522617eb03fc:
"hw/ac97: add support for volume control"
We are (correctly) using AC97_Record_Gain_Mute and not AC97_Line_In_Volume_Mute
for recording volume, but various places in hw/ac97 were still assumimg that
we are using AC97_Line_In_Volume_Mute for re
The Linux ac97 drivers does a number of register read/write tests to
see how much resolution a volume control actually has.
This patch takes this into account by masking out any bits written to
a volume control reg which should not be there according to the spec.
After this the Linux ac97 driver
The Linux AC97 driver tests this bit to decide wether or not to show
an External amplifier toggle control.
This patch was also tested with a Windows XP guest without any issues.
Signed-off-by: Hans de Goede
---
hw/ac97.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/
On Fri, Apr 20, 2012 at 8:46 AM, Paolo Bonzini wrote:
> Il 20/04/2012 09:00, Nicholas A. Bellinger ha scritto:
>> On Thu, 2012-04-19 at 19:20 -0500, Anthony Liguori wrote:
>>> TCM runs in the absolute most privileged context possible. When you're
>>> dealing
>>> with extremely hostile input, it'
On Fri, Apr 20, 2012 at 3:12 AM, Peter A. G. Crosthwaite
wrote:
> +static inline void flash_sync_area(struct flash *s, int64_t off, int64_t len)
> +{
> + int64_t start, end;
> +
> + if (!s->bdrv) {
> + return;
> + }
> +
> + start = off / 512;
> + end = (off + len) / 512;
> +
On 17 April 2012 06:41, Evgeny Voevodin wrote:
> +static Property exynos4210_irq_gate_properties[] = {
> + DEFINE_PROP_UINT32("n_in", Exynos4210IRQGateState, n_in, 1),
> + DEFINE_PROP_END_OF_LIST(),
> +};
> +
> static const VMStateDescription vmstate_exynos4210_irq_gate = {
> .name = "e
Am 20.04.2012 00:34, schrieb Michael Roth:
> On Wed, Apr 18, 2012 at 01:11:08AM +0200, Andreas Färber wrote:
>> Add the property early in the initfn so that it can be used in helpers
>> such as mce_init().
>>
>> Signed-off-by: Andreas Färber
>> ---
>> target-i386/cpu.c | 38
On 20 April 2012 11:50, Stefan Hajnoczi wrote:
> On Fri, Apr 20, 2012 at 12:21 AM, andrzej zaborowski
> wrote:
>> Yes, controllers would be affected, but there are various ways to go
>> about it. Some could be simple to implement (looking at
>> pxa2xx_mmci.c). First of all the SD specification
On 2 April 2012 06:20, Peter A. G. Crosthwaite
wrote:
> Added linux smp support for the xilinx zynq platform (2x cpus are supported)
>
> Signed-off-by: Peter A. G. Crosthwaite
> ---
> hw/xilinx_zynq.c | 64 -
> 1 files changed, 53 insertions(
Am 20.04.2012 12:59, schrieb Kevin Wolf:
Am 20.04.2012 12:50, schrieb Andreas Färber:
`hostname -s` may output an errror:
hostname: Name or service not known
This causes all tests to fail for `make check-block`.
Suppress such error messages, letting the tests succeed.
Signed-off-by: Andreas Fä
On 20.04.2012 15:13, Peter Maydell wrote:
On 17 April 2012 06:41, Evgeny Voevodin wrote:
+static Property exynos4210_irq_gate_properties[] = {
+DEFINE_PROP_UINT32("n_in", Exynos4210IRQGateState, n_in, 1),
+DEFINE_PROP_END_OF_LIST(),
+};
+
static const VMStateDescription vmstate_exynos
On Thu, Apr 19, 2012 at 10:49:16AM -0500, Michael Roth wrote:
> On Tue, Apr 17, 2012 at 12:07:38PM -0500, Michael Roth wrote:
> > When linux-specific commands (including guest-fsfreeze-*) were consolidated
> > under defined(__linux__), we forgot to account for the case where
> > defined(__linux__)
On Fri, Apr 20, 2012 at 12:39:36PM +0200, Kevin Wolf wrote:
> Am 20.04.2012 06:16, schrieb David Gibson:
> > I'm really hoping I can get some extra review of this code path. I
> > believe it's correct, but it's not straightforward to test, since it
> > will not be exercise by correct guest softwar
On 19.04.2012 21:34, Luiz Capitulino wrote:
> On Thu, 19 Apr 2012 13:57:48 -0500
> Michael Roth wrote:
>
>> On Thu, Apr 19, 2012 at 02:36:53PM -0300, Luiz Capitulino wrote:
>>> Michael,
>>>
>>> I'm going to revive this topic one more time. I was working on v2 of my
>>> fixes
>>> to the suspend r
On Tue, Apr 17, 2012 at 07:58:13PM +0200, Stefan Weil wrote:
> The spelling 'iff' is sometimes used for 'if and only if'.
> Even if that meaning could be applied here, it is not used
> consistently. It is also quite unusual to use 'if and only if'
> in technical documentation. Therefore a simple 'i
Hi,
Looks good, ack.
Regards,
Hans
On 04/20/2012 12:55 PM, Gerd Hoffmann wrote:
This patch adds a function which creates unique serial numbers for usb
devices and puts it into use. Windows guests tend to become unhappy if
they find two identical usb devices in the system. Effects range fro
On Fri, Apr 20, 2012 at 11:40:24AM +1000, David Gibson wrote:
> The new autotests in tests/ generate a number of files, both
> executable and source, which are not caught by the existing .gitignore
> files. This patch adds a new .gitignore in tests/ which covers these.
>
> Signed-off-by: David Gi
On Wed, Apr 18, 2012 at 07:28:34AM +0200, Stefan Weil wrote:
> This was reported by https://bugs.launchpad.net/qemu/+bug/984476.
>
> I also changed the case for 'error'.
>
> Signed-off-by: Stefan Weil
> ---
> hw/e1000.c |2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
Thanks, appl
On Fri, Apr 20, 2012 at 9:24 AM, Kevin Wolf wrote:
> Am 19.04.2012 23:18, schrieb Marcelo Tosatti:
>> On Thu, Apr 19, 2012 at 05:14:20PM -0300, Marcelo Tosatti wrote:
There is one intended change in functionality in this patch, which is
that it allocates new clusters even when it could s
On Fri, Apr 13, 2012 at 03:17:21PM +0100, Stefan Hajnoczi wrote:
> Signed-off-by: Stefan Hajnoczi
> ---
> .gitignore |1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/.gitignore b/.gitignore
> index 9859c7d..db74219 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -89,3 +89,4 @@ cscop
On 16.04.2012, at 21:46, Blue Swirl wrote:
> On Mon, Apr 16, 2012 at 19:28, Peter Maydell wrote:
>> On 16 April 2012 20:12, Blue Swirl wrote:
>>> On Mon, Apr 16, 2012 at 16:53, Alexander Graf wrote:
If you can update it on top of ppc-next and add patch descriptions
in every patch (mo
Il 20/04/2012 14:07, Michal Privoznik ha scritto:
> Just to make this clear: this "report-only-error" behavior concerns only
> guest-suspend-* and guest-shutdown commands, right? Because otherwise,
> if we enable such behavior for all commands (e.g. fsfreeze) I think we
> are entering the world of
From: Eric Bénard
Signed-off-by: Eric Bénard
Reviewed-by: Peter Maydell
Signed-off-by: Stefan Hajnoczi
---
hw/pflash_cfi01.c |1 -
1 file changed, 1 deletion(-)
diff --git a/hw/pflash_cfi01.c b/hw/pflash_cfi01.c
index b03f623..d1c7423 100644
--- a/hw/pflash_cfi01.c
+++ b/hw/pflash_cfi01.
From: Eduardo Elias Ferreira
Signed-off-by: Eduardo Elias Ferreira
Signed-off-by: Stefan Hajnoczi
---
spice-qemu-char.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/spice-qemu-char.c b/spice-qemu-char.c
index 1e735ff..09aa22d 100644
--- a/spice-qemu-char.c
+++ b/spice
From: Paolo Bonzini
Signed-off-by: Paolo Bonzini
Signed-off-by: Stefan Hajnoczi
---
hmp-commands.hx |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hmp-commands.hx b/hmp-commands.hx
index a6f5a84..461fa59 100644
--- a/hmp-commands.hx
+++ b/hmp-commands.hx
@@ -92,8 +9
From: Stefan Weil
val is an uint64_t, therefore %d was not correct.
Signed-off-by: Stefan Weil
Acked-by: Gerd Hoffmann
Signed-off-by: Stefan Hajnoczi
---
hw/qxl.c |2 +-
hw/qxl.h |2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/qxl.c b/hw/qxl.c
index db2318e..
From: Stefan Weil
This was reported by https://bugs.launchpad.net/qemu/+bug/984476.
I also changed the case for 'error'.
Signed-off-by: Stefan Weil
Signed-off-by: Stefan Hajnoczi
---
hw/e1000.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/e1000.c b/hw/e1000.c
ind
From: Amos Kong
It is invalid to return a value from a function
returning void.
[C99 6.8.6.4 says "A return statement with an expression shall not
appear in a function whose return type is void" but gcc 4.6.3 with QEMU
compile flags does not complain. It's still worth fixing this. Stefan]
Sig
The following changes since commit 51006bbc45bc74977ae538190a53df2af534acb9:
Merge remote-tracking branch 'origin/master' into staging (2012-04-18
10:06:09 -0500)
are available in the git repository at:
git://github.com/stefanha/qemu.git trivial-patches
for you to fetch changes up to fecc
From: David Gibson
The new autotests in tests/ generate a number of files, both
executable and source, which are not caught by the existing .gitignore
files. This patch adds a new .gitignore in tests/ which covers these.
[Changed 'rtc-test' to '*-test' so future tests do not need to be added
to
On Fri, Apr 20, 2012 at 1:27 PM, Stefan Hajnoczi wrote:
> On Fri, Apr 20, 2012 at 9:24 AM, Kevin Wolf wrote:
>> Am 19.04.2012 23:18, schrieb Marcelo Tosatti:
>>> On Thu, Apr 19, 2012 at 05:14:20PM -0300, Marcelo Tosatti wrote:
> There is one intended change in functionality in this patch, whi
Il 20/04/2012 04:36, Zhi Hui Li ha scritto:
> If I add the code :
>if ((0 == (d->mask & mask)) && (0 != (d->status & (mask << 4 {
>channel_run (icont, ichan);
>}
> Because function DMA_set_return is called in fdctrl_read_DMA_cb and
> fdctrl_write_DMA_cb, the fdctrl_stop_transfer
Am 20.04.2012 15:06, schrieb Stefan Hajnoczi:
> On Fri, Apr 20, 2012 at 1:27 PM, Stefan Hajnoczi wrote:
>> On Fri, Apr 20, 2012 at 9:24 AM, Kevin Wolf wrote:
>>> Am 19.04.2012 23:18, schrieb Marcelo Tosatti:
On Thu, Apr 19, 2012 at 05:14:20PM -0300, Marcelo Tosatti wrote:
>> There is one
Anthony,
please pull:
git://xenbits.xen.org/people/sstabellini/qemu-dm.git build_fix
this small patch series fixes the build breakage introduced by
f1dbf015dfb0aa7f66f710a1f1bc58b662951de2 with Xen < 4.2.
The problem is that xc_hvm_inject_msi is only defined from Xen 4.2
onwards so we need to p
On Fri, 20 Apr 2012 14:07:16 +0200
Michal Privoznik wrote:
> >> But, I think if we tell users we'll *only* send response on error,
> >> we should do our part to *not* send the responses, rather than relying
> >> on them having implemented the reset mechanism to throw them away after
> >> guest wa
Refcount block allocation and refcount table growth rely on
s->free_cluster_index pointing to somewhere after the current
allocation. Change qcow2_allocate_cluster_at() to fulfill this
assumption.
Without this change it could happen that a newly allocated refcount
block and the allocated data bloc
Hi,
On Fri, Apr 20, 2012 at 12:55:24PM +0200, Igor Mammedov wrote:
> >+/* Memory eject notify method */
> >+OperationRegion(MEMJ, SystemIO, 0xaf40, 32)
> >+Field (MEMJ, ByteAcc, NoLock, Preserve)
> >+{
> >+MPE, 256
> >+}
> >+
> >+Method (
On Thu, Apr 19, 2012 at 04:08:38PM +0200, Vasilis Liaskovitis wrote:
>
> series is based on uq/master for qemu-kvm, and master for seabios. Can be
> found
> also at:
forgot to paste the repo links in the original coverletter, here they are if
someone wants them:
https://github.com/vliaskov/qemu-
Am 15.04.2012 01:18, schrieb Peter Maydell:
> On 14 April 2012 18:39, Andreas Färber wrote:
>> Am 14.04.2012 18:42, schrieb Peter Maydell:
>>> Register subclasses for each ARM CPU implementation (with the
>>> exception of "pxa270", which is an alias for "pxa270-a0").
>>
>> This is no longer accura
> -Original Message-
> From: Peter Crosthwaite [mailto:peter.crosthwa...@petalogix.com]
> Sent: Thursday, April 19, 2012 11:18 PM
> To: John Linn
> Cc: qemu-devel@nongnu.org; p...@codesourcery.com;
> peter.mayd...@linaro.org; edgar.igles...@gmail.com; Duy Le;
> john.willi...@petalogix.com
>
>From the Department of the Redundancy Department:
remove the extra pthread switch which might be there
from the package config check for gthreads.
Signed-off-by: Peter Portante
---
configure | 11 ++-
1 files changed, 10 insertions(+), 1 deletions(-)
diff --git a/configure b/conf
Am 14.04.2012 18:42, schrieb Peter Maydell:
> Move the setting of the feature bits from cpu_reset_model_id()
> to each CPU's instance init function. This requires us to move
> the features field in CPUARMState so that it is not cleared
> on reset.
>
> Signed-off-by: Peter Maydell
> ---
[...]
> di
Am 14.04.2012 18:42, schrieb Peter Maydell:
> Move the reset FPSID to the ARMCPU struct, and set it in the
> per-implementation instance init function. At reset we then
> just copy the reset value into the CPUARMState field.
>
> Signed-off-by: Peter Maydell
> ---
> target-arm/cpu-qom.h |1 +
On 20 April 2012 15:43, Andreas Färber wrote:
> Am 14.04.2012 18:42, schrieb Peter Maydell:
>> static void pxa270c5_initfn(Object *obj)
>> {
>> ARMCPU *cpu = ARM_CPU(obj);
>> + set_feature(&cpu->env, ARM_FEATURE_V7);
>> + set_feature(&cpu->env, ARM_FEATURE_VFP4);
>> + set_feature(&
On Fri, Apr 20, 2012 at 12:12:38PM +0200, Jan Kiszka wrote:
> On 2012-04-19 22:03, Eduardo Habkost wrote:
> > Jan/Avi: ping?
> >
> > I would like to get this ABI detail clarified so it can be implemented
> > the right way on Qemu and KVM.
> >
> > My proposal is to simply add tsc-deadline to the d
Am 14.04.2012 18:42, schrieb Peter Maydell:
> Move the MVFR* VFP feature register values to ARMCPU,
> so they are set up by the implementation-specific instance
> init functions rather than in cpu_reset_model_id().
>
> Signed-off-by: Peter Maydell
Acked-by: Andreas Färber
/-F
--
SUSE LINUX P
Am 14.04.2012 18:42, schrieb Peter Maydell:
> Move CTR (cache type register) value to an ARMCPU field
> set up by per-cpu init fns.
>
> Signed-off-by: Peter Maydell
Acked-by: Andreas Färber
/-F
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild,
On 2012-04-20 17:00, Eduardo Habkost wrote:
> On Fri, Apr 20, 2012 at 12:12:38PM +0200, Jan Kiszka wrote:
>> On 2012-04-19 22:03, Eduardo Habkost wrote:
>>> Jan/Avi: ping?
>>>
>>> I would like to get this ABI detail clarified so it can be implemented
>>> the right way on Qemu and KVM.
>>>
>>> My pr
Am 14.04.2012 18:42, schrieb Peter Maydell:
> Move the reset value of SCTLR to ARMCPU, initialised in
> the per-cpu init functions. It can then be reset by a
> simple copy, and we can drop the code from cpu_reset_model_id().
>
> Signed-off-by: Peter Maydell
Acked-by: Andreas Färber
/-F
--
SU
On 16 April 2012 16:02, Eric Bénard wrote:
> - add support for the 64MB NOR CFI01 flash available at
> 0x3400 on the versatilepb board
> http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.dui0225d/BBAJIHEC.html
>
> - tested with barebox bootloader
>
> Signed-off-by: Eric Bénard
Revi
On Fri, Apr 20, 2012 at 05:19:17PM +0200, Jan Kiszka wrote:
> On 2012-04-20 17:00, Eduardo Habkost wrote:
> > On Fri, Apr 20, 2012 at 12:12:38PM +0200, Jan Kiszka wrote:
> >> On 2012-04-19 22:03, Eduardo Habkost wrote:
> >>> Jan/Avi: ping?
> >>>
> >>> I would like to get this ABI detail clarified s
Signed-off-by: Peter Portante
---
hw/spapr_hcall.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/hw/spapr_hcall.c b/hw/spapr_hcall.c
index 634763e..94bb504 100644
--- a/hw/spapr_hcall.c
+++ b/hw/spapr_hcall.c
@@ -482,7 +482,7 @@ static target_ulong register_dtl(CPUPPCS
Public bug reported:
Looks like there is a simple misspelling of the env parameter, as "emv",
in hw/spapr_hcall.c:485:
static target_ulong deregister_dtl(CPUPPCState *emv, target_ulong
addr)
** Affects: qemu
Importance: Undecided
Status: New
--
You received this bug notificatio
Am 14.04.2012 18:42, schrieb Peter Maydell:
> Move feature register value setup to per-CPU init functions.
>
> Signed-off-by: Peter Maydell
> ---
> target-arm/cpu-qom.h | 14
> target-arm/cpu.c | 91
> ++
> target-arm/helper.c |
On 20 April 2012 16:50, Andreas Färber wrote:
> Missing cpu->id_mmfr3 = 0x11;
[etc]
Whoops, that was a bit sloppy. Fixed and new version pushed
to git://git.linaro.org/people/pmaydell/qemu-arm.git drop-reset-model-id
-- PMM
Am 14.04.2012 18:42, schrieb Peter Maydell:
> Move cache ID register reset out of cpu_reset_model_id() by
> creating a field for the reset value in ARMCPU and setting it
> up in the cpu specific init functions.
>
> Signed-off-by: Peter Maydell
> ---
> target-arm/cpu-qom.h |5 +
> target-
On 20 April 2012 17:00, Andreas Färber wrote:
> Am 14.04.2012 18:42, schrieb Peter Maydell:
>> + cpu->clidr = (1 << 27) | (2 << 24) | 3;
>
> Copy&paste, should be (1 << 27) | (1 << 24) | 3.
Fixed and pushed, sigh.
-- PMM
Am 14.04.2012 18:42, schrieb Peter Maydell:
> cpu_reset_model_id() is now empty and we can remove it.
>
> Signed-off-by: Peter Maydell
Acked-by: Andreas Färber
However, ...
> diff --git a/target-arm/helper.c b/target-arm/helper.c
> index 5cbc7e0..653885a 100644
> --- a/target-arm/helper.c
> +
Il 20/04/2012 12:03, Stefan Weil ha scritto:
> These patches remove timer code which is no longer needed
> and try to improve the remaining code.
>
> [PATCH 1/5] qemu-timer: Remove redundant include statements
> [PATCH 2/5] qemu-timer: Remove unused function qemu_alarm_pending
> [PATCH 3/5] qemu-t
On 20 April 2012 17:14, Andreas Färber wrote:
> Am 14.04.2012 18:42, schrieb Peter Maydell:
>> /* TODO Move contents into arm_cpu_reset() in cpu.c,
>> * once cpu_reset_model_id() is eliminated,
>
> ...if you were to drop this comment line it would be cleaner. There is
> no cpu_reset_model_
Am 20.04.2012 18:05, schrieb Peter Maydell:
> On 20 April 2012 17:00, Andreas Färber wrote:
>> Am 14.04.2012 18:42, schrieb Peter Maydell:
>>> +cpu->clidr = (1 << 27) | (2 << 24) | 3;
>>
>> Copy&paste, should be (1 << 27) | (1 << 24) | 3.
>
> Fixed and pushed, sigh.
Acked-by: Andreas Färber
On Fri, Apr 20, 2012 at 12:33:57PM +0200, Igor Mammedov wrote:
> On 04/19/2012 04:08 PM, Vasilis Liaskovitis wrote:
> >-numa_fw_cfg = g_malloc0((1 + max_cpus + nb_numa_nodes) * 8);
> >+numa_fw_cfg = g_malloc0((2 + max_cpus + nb_numa_nodes + 3 *
> >nb_hp_memslots) * 8);
> > numa_fw_cfg
1 - 100 of 188 matches
Mail list logo
