malc writes:
> On Mon, 7 Dec 2009, Markus Armbruster wrote:
>
>> malc writes:
>>
>> > On Mon, 30 Nov 2009, Markus Armbruster wrote:
>> >
>> >> Commit a7d27b53 made zero-sized allocations a fatal error, deviating
>> >> from ISO C's malloc() & friends. Revert that, but take care never to
>> >> r
Anthony Liguori writes:
> Markus Armbruster wrote:
>> Commit a7d27b53 made zero-sized allocations a fatal error, deviating
>> from ISO C's malloc() & friends. Revert that, but take care never to
>> return a null pointer, like malloc() & friends may do (it's
>> implementation defined), because th
Split out pci device initialization from pc_init1() into pc_pci_device_init().
and removed unnecessary braces.
Signed-off-by: Isaku Yamahata
---
hw/pc.c | 35 +++
1 files changed, 19 insertions(+), 16 deletions(-)
diff --git a/hw/pc.c b/hw/pc.c
index 7d7c800..b
This time, I fixed up the patch series.
- fixed up copyright notice.
- make rtc_xxx functions accept/return ISADevice.
- move up ioapic initalization patch.
Although Gerd suggested to use DeviceState, I used ISADevice.
Gerd, do you want me to use DeviceState instead of ISADevice?
patch seri
By introducing a registering function, make pc_init1() not refer to
ferr_irq directly in order to make ferr_irq piix independent.
Later pc_init1() will be split out into another file keeping ferr_irq
static.
Signed-off-by: Isaku Yamahata
---
hw/pc.c |8 +++-
hw/pc.h |2 ++
2 files ch
add acpi constants from linux header files and
replace the old constants with them.
The acpi constants will be used by other file.
Signed-off-by: Isaku Yamahata
---
hw/acpi.c | 56 +++
hw/acpi.h | 78
Split out vga initialization which is independent of piix
from pc_init1() as pc_vga_init().
Later it will be used.
Signed-off-by: Isaku Yamahata
---
hw/pc.c | 41 +++--
1 files changed, 23 insertions(+), 18 deletions(-)
diff --git a/hw/pc.c b/hw/pc.c
index
Remove unnecessary reference to a global variable, RTCState *rtc_state,
by passing it as function argument.
Only the cmos_set_s3_resume_init() needs it global.
So introduce a registering function and use it.
Signed-off-by: Isaku Yamahata
Cc: Paolo Bonzini
---
hw/pc.c | 25
make cpu_smm_update() generic to be independent on i440fx by
registering a callback.
Signed-off-by: Isaku Yamahata
---
hw/pc.c | 18 +++---
hw/pc.h |8 +++-
hw/piix_pci.c |5 -
3 files changed, 26 insertions(+), 5 deletions(-)
diff --git a/hw/pc.c b/hw/
Move rtc_xxx declarations from pc.h into mc146818rtc.h.
Signed-off-by: Isaku Yamahata
---
hw/mc146818rtc.h | 10 ++
hw/pc.h | 10 ++
2 files changed, 12 insertions(+), 8 deletions(-)
create mode 100644 hw/mc146818rtc.h
diff --git a/hw/mc146818rtc.h b/hw/mc146818rtc
Split out apm register emulation for acpi.c into pc_apm.c.
The apm emulation will be used later.
Signed-off-by: Isaku Yamahata
---
Makefile.target |4 +-
hw/acpi.c | 57 ++--
hw/pc_apm.c | 85 ++
remove unnecessary global static variables, pit.
Make it local.
Signed-off-by: Isaku Yamahata
---
hw/pc.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/hw/pc.c b/hw/pc.c
index fc7ee8a..c728486 100644
--- a/hw/pc.c
+++ b/hw/pc.c
@@ -66,7 +66,6 @@
static fdctrl_t *fl
Finally, we can safely split out the piix specific part from pc.c
into pc_piix.c.
Signed-off-by: Isaku Yamahata
---
Makefile.target |2 +-
hw/pc.c | 256 +-
hw/pc.h | 32 +++
hw/pc_piix.c| 244 +++
Remove a global variable, floppy_controller.
Since it is unnecessarily global, make it local and pass it as
a function argument.
Signed-off-by: Isaku Yamahata
---
hw/pc.c |8 +---
1 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/hw/pc.c b/hw/pc.c
index c728486..dcf7eeb 1006
Introduce a function, pc_allocate_cpu_irq(), to allocate cpu irq
in order to make pic_irq_request() piix independent.
Later piix code will be split out to another file keeping pic_irq_request()
static.
Signed-off-by: Isaku Yamahata
---
hw/pc.c |7 ++-
1 files changed, 6 insertions(+), 1
Split out memory allocation and rom/bios loading which doesn't depend
on piix from pc_init1() into pc_memory_init().
Later it will be used.
Signed-off-by: Isaku Yamahata
---
hw/pc.c | 67 +++---
1 files changed, 42 insertions(+), 25 delet
The changeset of 2c8d9340203c7f19265fd4cb2341f568217a3af6
prevents isa_irq_handler() from NULL refering of IsaIrqState::ioapic.
However it would be better to initialize the member before reference.
Signed-off-by: Isaku Yamahata
---
hw/pc.c |6 +++---
1 files changed, 3 insertions(+), 3 delet
remove unused variable in get_pmsts().
Signed-off-by: Isaku Yamahata
---
hw/acpi_piix4.c |2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/hw/acpi_piix4.c b/hw/acpi_piix4.c
index 53efa2a..887c82f 100644
--- a/hw/acpi_piix4.c
+++ b/hw/acpi_piix4.c
@@ -63,8 +63,6 @@ static
split out cpu initialization which is piix independent from pc_init1()
into pc_cpus_init(). Later it will be used.
Signed-off-by: Isaku Yamahata
---
hw/pc.c | 32 +++-
1 files changed, 19 insertions(+), 13 deletions(-)
diff --git a/hw/pc.c b/hw/pc.c
index d0ef06b..
To match rtc_xxx with qdev, make rtc_xxx accept and return ISADevice
instead of RTCState.
Signed-off-by: Isaku Yamahata
---
hw/mc146818rtc.c | 26 +++---
hw/mc146818rtc.h |8
hw/mips_jazz.c |1 +
hw/mips_malta.c |3 ++-
hw/mips_r4k.c|3 ++-
h
Split out basic device, i.e. legacy devices like floppy, initialization
from pc_init1() into pc_basic_device_init().
Later it will be used.
Signed-off-by: Isaku Yamahata
---
hw/pc.c | 89 +++---
1 files changed, 50 insertions(+), 39 delet
On 12/07/2009 04:06 PM, Anthony Liguori wrote:
For GPFS-like shared filesystems (:-P) we would always use lock=off. It
wouldn't protect us from a non-cluster aware writer.
Management tools can always do this the right way---even with
partition-grained locking if needed.
Paolo
Split out pc smbus routines from acpi.c into pc_smbus.c and
use it.
The split out smbus emulation will be used later.
Signed-off-by: Isaku Yamahata
---
Makefile.target |2 +
hw/acpi.c | 164 +++
hw/pc_smbus.c | 178 +++
Split acpi.c into the common part and the piix4 specific part.
The common part will be used later.
Signed-off-by: Isaku Yamahata
---
Makefile.target |4 +-
hw/acpi.c | 557 ---
hw/{acpi.c => acpi_piix4.c} | 172 +
Am 07.12.2009 18:09, schrieb malc:
> On Mon, 7 Dec 2009, Anthony Liguori wrote:
>
>> malc wrote:
Does anyone object to this moving forward?
>>>
>>> Yeah, i object to the split production/development qemu_malloc[z].
>>>
>>
>> It's clear to me that there are still improper callers
Hi,
Doesn't make sense. The list is not about *creating* default devices, but
about *not* creating them.
Why do we need a negative list? Wouldn't it be a lot more useful to have
positive lists? Maybe I'm just missing the whole point of your patchset though.
Probably the latter ;)
The s
Am 07.12.2009 15:53, schrieb Anthony Liguori:
> Daniel P. Berrange wrote:
>> It is safe if you assume that no one else has tried to modify the disk
>> since you opened it, otherwise you'd be commiting changes against a
>> base state which no longer exists.
>
> 1) first user opens cow1.qcow, acqui
Rusty Russell wrote on 12/08/2009 07:48:00 AM:
> Avishay; this would be the total sectors in an I/O, as separate from
SIZE_MAX
> (maximum size of any single scatterlist entry) and SEG_MAX (maximum
number of
> scatterlist entries)?
Correct. In the guest virtblk driver, it changes the call to
blk_
On 12/08/2009 11:44 AM, Avishay Traeger1 wrote:
Thanks!
Rusty.
Sure. Avi - do you want me to resubmit the kvm and qemu patches?
You mean the virtio and qemu patches. That's up to their maintainers
(Rusty and Anthony).
--
error compiling committee.c: too many arguments to funct
Am 07.12.2009 15:16, schrieb Richard W.M. Jones:
> Here's a second go at this patch. The only change is that we add a
> second parameter (backinglock=none|shared|exclusive) which can be used
> to control locking on the first level (only) of backing disk.
And what about the backing file of the bac
Am 07.12.2009 15:31, schrieb Richard W.M. Jones:
> On Mon, Dec 07, 2009 at 08:22:24AM -0600, Anthony Liguori wrote:
>> Richard W.M. Jones wrote:
>>> On Mon, Dec 07, 2009 at 07:39:11AM -0600, Anthony Liguori wrote:
>>>
Richard W.M. Jones wrote:
> Also if we only acquire the loc
On Tue, Dec 08, 2009 at 10:48:17AM +0100, Kevin Wolf wrote:
> Am 07.12.2009 15:31, schrieb Richard W.M. Jones:
> > So to be clear, the use case is that all the other VMs must be shut
> > down, then the VM which wants to commit will upgrade its lock and
> > commit, and then all the other VMs will re
On Tue, 8 Dec 2009, Markus Armbruster wrote:
> malc writes:
>
[..snip..]
> glibc frees since 1999. From glibc/ChangeLog.10:
>
> 1999-04-28 Andreas Jaeger
>
> * malloc/malloc.c (REALLOC_ZERO_BYTES_FREES): Define it to follow
> ISO C9x and Unix98.
Problem is - this is NOT what
On Tue, Dec 08, 2009 at 11:00:17AM +0100, Kevin Wolf wrote:
> Am 07.12.2009 15:16, schrieb Richard W.M. Jones:
> > Here's a second go at this patch. The only change is that we add a
> > second parameter (backinglock=none|shared|exclusive) which can be used
> > to control locking on the first level
On 12/07/09 19:34, Daniel P. Berrange wrote:
On Sun, Dec 06, 2009 at 04:43:03PM -0600, Anthony Liguori wrote:
It can be downloaded from Savannah at:
http://download.savannah.gnu.org/releases/qemu/qemu-0.12.0-rc1.tar.gz
Please send testing feedback (positive or negative) to qemu-devel and
file
On 12/07/09 22:54, Anthony Liguori wrote:
Daniel P. Berrange wrote:
SCSI controllers have no trouble existing without any attached
disks. This could be achieved with the (legacy) monitor syntax
pci_add pci_addr=auto storage if=scsi
This is now denied with
scsi requires a backing file/device.
On Tue, Dec 08, 2009 at 11:40:54AM +0100, Gerd Hoffmann wrote:
> On 12/07/09 19:34, Daniel P. Berrange wrote:
> >On Sun, Dec 06, 2009 at 04:43:03PM -0600, Anthony Liguori wrote:
> >>
> >>It can be downloaded from Savannah at:
> >>
> >>http://download.savannah.gnu.org/releases/qemu/qemu-0.12.0-rc1.t
On Mon, 7 Dec 2009 21:36:58 +0100
Markus Armbruster wrote:
> This patch series converts monitor commands eject, change, closefd,
> getfd to QError.
Thanks a lot for this work.
Although the series is already merged, I have some comments and will
reply the patches.
On Mon, 7 Dec 2009 21:37:10 +0100
Markus Armbruster wrote:
> +#define QERR_FD_NOT_FOUND \
> +"{ 'class': 'fd_not_found', 'data': { 'name': %s } }"
> +
I think this came from your previous series of not using CamelCase?
Also happens in next patches..
On Mon, 7 Dec 2009 21:37:16 +0100
Markus Armbruster wrote:
> -{ "error": { "class": json-string, "data": json-value }, "id": json-value }
> +{ "error": { "class": json-string, "data": json-value, "desc": json-string },
> + "id": json-value }
>
> Where,
>
> - The "class" member contains t
Hi,
Qemu creates a bunch of default devices (serial, parallel, vga, ...) if
the user didn't specify one on the command line. Unfortunaly this
doesn't work well with the qdev way of doing things because this logic
is tied to the -serial, -parallel, ... command line switches. Devices
created via
i.e. -global PCI.= will set a default property for all
PCI devices. Also works for the compat properties used by machine
types.
Signed-off-by: Gerd Hoffmann
---
hw/qdev-properties.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/hw/qdev-properties.c b/hw/qdev-propert
The patch decuples the -chardev switch and the actual chardev
initialization. Without this patch qemu ignores chardev entries
coming via -readconfig.
Signed-off-by: Gerd Hoffmann
---
vl.c | 16 +---
1 files changed, 13 insertions(+), 3 deletions(-)
diff --git a/vl.c b/vl.c
index
This reverts commit adcb181afe5a951c521411c7a8e9d9b791aa6742.
Conflicts:
monitor.h
Signed-off-by: Gerd Hoffmann
---
monitor.c | 18 --
monitor.h |1 -
qemu-options.hx |5 ++---
vl.c| 11 ---
4 files changed, 6 insertions(+), 2
This patch renames the compat properties into global properties and
makes them more generic. The compatibility stuff is only one of
multiple possible users now.
Signed-off-by: Gerd Hoffmann
---
hw/boards.h |2 +-
hw/pc.c |2 +-
hw/qdev-properties.c | 22 +
Make the 'vc' chardev backend print a title line with the chardev name
after initialization, using CharDriverState->label.
This replaces the banner printing code in vl.c.
Signed-off-by: Gerd Hoffmann
---
console.c |8
vl.c | 24
2 files changed, 8 in
This reverts commit 93d434b4aec0702b87ebf52449a3cdf2c3596825.
Signed-off-by: Gerd Hoffmann
---
vl.c | 26 --
1 files changed, 0 insertions(+), 26 deletions(-)
diff --git a/vl.c b/vl.c
index 11eda45..172828a 100644
--- a/vl.c
+++ b/vl.c
@@ -4720,20 +4720,6 @@ int main(i
This patch adds infrastructure and command line option for setting
global defaults for device properties, i.e. you can for example use
-global virtio-blk-pci.vectors=0
to turn off msi by default for all virtio block devices. The config
file syntax is:
[global]
driver = "virtio-blk-pci"
pr
Signed-off-by: Gerd Hoffmann
---
console.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/console.c b/console.c
index 2aeb5b3..8086bd6 100644
--- a/console.c
+++ b/console.c
@@ -1388,8 +1388,10 @@ static void text_console_do_init(CharDriverState *chr,
DisplayState *ds
Qemu creates a default serial line for you in case you didn't specify
one on the command line. Right now this is tied to the '-serial
' command line switch, which in turn causes trouble if you are
creating your serial line via '-device isa-serial,'.
This patch adds a variable default_serial which
Add global command line option to disable default devices.
Signed-off-by: Gerd Hoffmann
---
qemu-options.hx |5 +
vl.c|6 ++
2 files changed, 11 insertions(+), 0 deletions(-)
diff --git a/qemu-options.hx b/qemu-options.hx
index 420b7d8..e05b2a0 100644
--- a/qemu-opti
Qemu creates a default parallel port for you in case you didn't specify
one on the command line. Right now this is tied to the '-parallel
' command line switch, which in turn causes trouble if you are
creating your parallel port via '-device isa-parallel,'.
This patch adds a variable default_para
The logic in this code obviously predates the multiple monitor
capability of qemu and looks increasingly silly these days.
I think the intention of this piece of code is to get a reasonable
default for the -nographic case: have monitor and serial line muxed
on stdio.
With the new default_serial a
This patch makes the monitor default device configuration work like the
default serial and parallel port devices. It adds a variable
default_monitor which says whenever a default monitor should be added.
It is enabled by default. It is cleared when qemu finds '-monitor' on
the command line.
Sign
Qemu creates a vga display for you in case you didn't specify one on the
command line. Right now this is tied to the '-vga ' command line
switch, which in turn causes trouble if you are creating your gfx card
using '-device VGA,'.
This patch adds a variable default_vga which says whenever a defau
Signed-off-by: Gerd Hoffmann
---
qemu-char.c |2 +-
qemu-char.h |1 +
2 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/qemu-char.c b/qemu-char.c
index da5c15c..c6008c3 100644
--- a/qemu-char.c
+++ b/qemu-char.c
@@ -2231,7 +2231,7 @@ static CharDriverState *qemu_chr_open_soc
Add -mon switch which maps pretty straight forward into the QemuOpts
internal representation:
-mon chardev=[,mode=[control|readline]][,[no]default]
Via config file:
[mon]
chardev = ""
mode = "readline"
default = "on"
Signed-off-by: Gerd Hoffmann
---
qemu-options.hx |7 +++
Add a default_net variable which specified whenever a default network
should be created. It is cleared in case any -net option is specified
and it is also added to the new -nodefaults switch.
Signed-off-by: Gerd Hoffmann
---
net.c |5 -
net.h |1 +
vl.c |1 +
3 files changed, 6
This patch adds a variable default_virtcon which says whenever a default
virtio console should be added. It is disabled by default, followup
patch will enable it for s390. It is cleared when qemu finds
'-virtiocon', '-device virtio-console-s390' or '-device
virtio-console-pci' on the command line
All "normal" system emulation targets in qemu I'm aware of display
output on either VGA or serial output.
Our S390x virtio machine doesn't have such kind of legacy hardware. So
instead we need to default to a virtio console.
Add flags to QEMUMachine to indicate which kind of default devices make
Acts like -monitor but switched into qmp mode.
Signed-off-by: Gerd Hoffmann
---
qemu-options.hx |2 ++
vl.c| 14 +-
2 files changed, 11 insertions(+), 5 deletions(-)
diff --git a/qemu-options.hx b/qemu-options.hx
index 7234447..b8cc375 100644
--- a/qemu-options.hx
Add a default_drive variable which specified whenever the default drives
(cdrom, floppy, sd) should be created. It is cleared when the new
-nodefaults switch is specified on the command line.
Signed-off-by: Gerd Hoffmann
---
vl.c | 16 ++--
1 files changed, 10 insertions(+), 6 del
This patch reworks the -monitor handling:
- It adds a new "mon" QemuOpts list for the monitor(s).
- It adds a monitor_parse() function to parse the -monitor switch.
- It adds a mon_init function to initialize the monitor(s) from the
"mon" QemuOpts list.
- It winds up everything and removes
On Tue, Dec 08, 2009 at 10:11:48AM -0200, Luiz Capitulino wrote:
> On Mon, 7 Dec 2009 21:37:16 +0100
> Markus Armbruster wrote:
>
> > -{ "error": { "class": json-string, "data": json-value }, "id": json-value }
> > +{ "error": { "class": json-string, "data": json-value, "desc": json-string
> >
Luiz Capitulino writes:
> On Mon, 7 Dec 2009 21:37:10 +0100
> Markus Armbruster wrote:
>
>> +#define QERR_FD_NOT_FOUND \
>> +"{ 'class': 'fd_not_found', 'data': { 'name': %s } }"
>> +
>
> I think this came from your previous series of not using CamelCase?
>
> Also happens in next patches.
Commits c7c338c4, 41471a23, 7a046f5f and a488be27 used
lower_case_with_underscores for class values. Existing usage
CamelCase. ChangeToThatForConsistency.
Signed-off-by: Markus Armbruster
---
qerror.h |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/qerror.h b/qe
On 12/08/2009 01:11 PM, Luiz Capitulino wrote:
Not to mention that those strings can't be modified when the protocol
becomes stable and we're probably talking about dozens if not a hundred
of strings.
I would say that there is _explicitly no promise_ of keeping these
stable. You can serve
On 12/07/2009 01:42 PM, Gerd Hoffmann wrote:
Add global command line option to disable default devices.
Should -readconfig imply this?
Paolo
Luiz Capitulino writes:
> On Mon, 7 Dec 2009 21:37:16 +0100
> Markus Armbruster wrote:
>
>> -{ "error": { "class": json-string, "data": json-value }, "id": json-value }
>> +{ "error": { "class": json-string, "data": json-value, "desc": json-string
>> },
>> + "id": json-value }
>>
>> Where
On 12/08/09 11:52, Daniel P. Berrange wrote:
On Tue, Dec 08, 2009 at 11:40:54AM +0100, Gerd Hoffmann wrote:
"worked" as in "guest actually sees the new device (without reboot)" or
"worked" as in "qemu didn't abort" ?
The latter. The guest does not see it, but it at least does not abort.
It is
Paolo Bonzini writes:
> On 12/08/2009 01:11 PM, Luiz Capitulino wrote:
>> Not to mention that those strings can't be modified when the protocol
>> becomes stable and we're probably talking about dozens if not a hundred
>> of strings.
>
> I would say that there is _explicitly no promise_ of keep
On 12/08/09 13:52, Gerd Hoffmann wrote:
The latter. The guest does not see it, but it at least does not abort.
It is the 'does not abort' behaviour I'm interested in - quite OK with
this returning an error to the monitor client when acpi is disabled.
Does the attached patch fix it for you?
On
Luiz Capitulino wrote:
On Mon, 7 Dec 2009 21:37:16 +0100
Markus Armbruster wrote:
-{ "error": { "class": json-string, "data": json-value }, "id": json-value }
+{ "error": { "class": json-string, "data": json-value, "desc": json-string },
+ "id": json-value }
Where,
- The "class" me
On hosts without high-res timers it is impossible to inject rtc interrupt
faster then 1kHz. Windows sometimes configures RTC to generate 1kHz
interrupts, so we can't inject missed interrupts when running on such
hosts. Always injecting an interrupt on REG_C read is also not an option
since Windows
The disk image I created from my old laptop disk with VBoxManage
internalcommand converthd obviously was not a multiple of 1MB as when
created from scratch. This fixes QEMU refusing it. We still require the
size to be a multiple of sector size though.
It then boots correctly.
Signed-off-by:
Al
On 1 oct. 2009, at 18:13, Blue Swirl wrote:
> On Thu, Oct 1, 2009 at 6:30 PM, G 3 wrote:
>> On Sep 30, 2009, at 3:16 PM, Anthony Liguori wrote:
>>
>> G 3 wrote:
>>
>> This patch fixes a problem in the file cutils.c that prevents qemu from
>> being built on Mac OS X. This patch makes sure fsync
On Tue, 08 Dec 2009 07:18:11 -0600
Anthony Liguori wrote:
> Luiz Capitulino wrote:
> > On Mon, 7 Dec 2009 21:37:16 +0100
> > Markus Armbruster wrote:
> >
> >
> >> -{ "error": { "class": json-string, "data": json-value }, "id": json-value
> >> }
> >> +{ "error": { "class": json-string, "data
Luiz Capitulino writes:
> On Tue, 8 Dec 2009 12:25:13 +
> "Daniel P. Berrange" wrote:
>
>> On Tue, Dec 08, 2009 at 10:11:48AM -0200, Luiz Capitulino wrote:
>> > On Mon, 7 Dec 2009 21:37:16 +0100
>> > Markus Armbruster wrote:
>> >
>> > > -{ "error": { "class": json-string, "data": json-val
Am 07.12.2009 16:00, schrieb Kevin Wolf:
> Am 07.12.2009 15:16, schrieb Jan Kiszka:
>>> Likely not. What I did was nothing special, and I did not noticed such a
>>> crash in the last months.
>>
>> And now it happened again (qemu-kvm head, during kernel installation
>> from network onto local qcow2-
On Tue, 8 Dec 2009 12:25:13 +
"Daniel P. Berrange" wrote:
> On Tue, Dec 08, 2009 at 10:11:48AM -0200, Luiz Capitulino wrote:
> > On Mon, 7 Dec 2009 21:37:16 +0100
> > Markus Armbruster wrote:
> >
> > > -{ "error": { "class": json-string, "data": json-value }, "id":
> > > json-value }
> >
On 12/08/09 13:46, Paolo Bonzini wrote:
On 12/07/2009 01:42 PM, Gerd Hoffmann wrote:
Add global command line option to disable default devices.
Should -readconfig imply this?
Hmm, not sure. Why do you think this would be useful?
cheers,
Gerd
This patchset adds MIPS16 support to the MIPS backend. MIPS16 is a
compact encoding of a subset of the MIPS integer instructions, similar
to ARM's Thumb mode. Mode switching occurs when either a special
instruction (JALX) is executed, or when a jump-to-register instruction
is executed; the instru
As promised, here is my small writeup on which setups I feel
are important in the long run for server-type guests. This
does not cover -net user, which is really for desktop kinds
of applications where you do not want to connect into the
guest from another IP address.
I can see four separate setup
On Wed, Nov 25, 2009 at 12:33:03AM +0100, Jan Kiszka wrote:
> This patch extends the qemu-kvm state sync logic with support for
> KVM_GET/SET_VCPU_EVENTS, giving access to yet missing exception,
> interrupt and NMI states.
>
> Signed-off-by: Jan Kiszka
> ---
> kvm-all.c | 11 ++
> > > Add global command line option to disable default devices.
> >
> > Should -readconfig imply this?
>
> Hmm, not sure. Why do you think this would be useful?
Here is my thinking: if you used -writeconfig, your machine description
is (at least in theory) entirely included in the config file i
Signed-off-by: Nathan Froyd
---
target-mips/translate_init.c | 27 ++-
1 files changed, 18 insertions(+), 9 deletions(-)
diff --git a/target-mips/translate_init.c b/target-mips/translate_init.c
index c950eab..6f50555 100644
--- a/target-mips/translate_init.c
+++ b/targ
We need to stash the operating mode into the low bit of the error PC and
restore it on return from interrupts.
Signed-off-by: Nathan Froyd
---
target-mips/helper.c| 51 +++---
target-mips/op_helper.c | 17 --
2 files changed, 44 inserti
The only thing to do here is to expose the current processor mode to GDB
and to set the processor mode properly when we change the PC.
Signed-off-by: Nathan Froyd
---
gdbstub.c | 18 +++---
1 files changed, 15 insertions(+), 3 deletions(-)
diff --git a/gdbstub.c b/gdbstub.c
index
Marcelo Tosatti wrote:
> On Wed, Nov 25, 2009 at 12:33:03AM +0100, Jan Kiszka wrote:
>> This patch extends the qemu-kvm state sync logic with support for
>> KVM_GET/SET_VCPU_EVENTS, giving access to yet missing exception,
>> interrupt and NMI states.
>>
>> Signed-off-by: Jan Kiszka
>> ---
>> kvm-
The following fixes a class of long-standing bugs in qemu:
when kvm is enabled, guest might access device structures
in memory while they are updated by qemu on another CPU.
In this scenario, memory barriers are necessary to prevent
host CPU from reordering memory accesses, which might confuse
the
With qemu-kvm and with iothread, guest might read in-memory structures
on one host CPU while it is being updated on another one, and vice
versa. Therefore we must use memory barriers to ensure proper memory
read/write ordering.
The following implements memory barriers with API matching
that of th
include barriers.h and remove a non-portable
wmb implementation from virtio.c (it will work
for intel but not for other architectures).
Signed-off-by: Michael S. Tsirkin
---
hw/virtio.c | 10 +-
1 files changed, 1 insertions(+), 9 deletions(-)
diff --git a/hw/virtio.c b/hw/virtio.c
in
Add missing memory barriers in virtio:
- before checking interrupt enabled bit, make sure
data has been written to memory
- make sure ring index has been read before reading ring data
Signed-off-by: Michael S. Tsirkin
---
hw/virtio.c |8 +++-
1 files changed, 7 insertions(+), 1 deletio
On Tue, Dec 08, 2009 at 11:11:29AM -0500, Paolo Bonzini wrote:
> > > > Add global command line option to disable default devices.
> > >
> > > Should -readconfig imply this?
> >
> > Hmm, not sure. Why do you think this would be useful?
>
> Here is my thinking: if you used -writeconfig, your machi
On 12/08/09 17:11, Paolo Bonzini wrote:
Add global command line option to disable default devices.
Should -readconfig imply this?
Hmm, not sure. Why do you think this would be useful?
Here is my thinking: if you used -writeconfig, your machine description
is (at least in theory) entirely i
> > Here is my thinking: if you used -writeconfig, your machine description
> > is (at least in theory) entirely included in the config file including
> > (again in theory) the default devices.
>
> The default devices are *not* included by in the file created by
> -writeconfig.
Yes, that's why I
We create separate masks for the "basic" branch hflags and the
"extended" branch hflags and define MIPS_HFLAG_BMASK as the logical or
of those two. This is done to avoid churning the codebase in lots of
different places.
We also make the execution mode an hflag under MIPS_HFLAG_TMASK
Signed-off-
There's no good way to add this incrementally, so we do it all at once.
The only changes to shared code are in handle_delay_slot. We need to
flip ISAMode when doing a jump-and-exchange. We also need to set
ISAMode the low bit of the target address for jump-to-register.
Also, since we're now addi
As KVM now makes use of exception_index to keep pending exceptions, we
have to save&restore this field as well.
NOTE: We have to nail the arch-independent exception_index down to a
certain bit width for proper vmstate processing, namely to 32 bit.
Signed-off-by: Jan Kiszka
---
cpu-defs.h
This is a common pattern in existing code. We'll also use it to
implement the mips16 SAVE/RESTORE instructions.
Signed-off-by: Nathan Froyd
---
target-mips/translate.c | 40
1 files changed, 16 insertions(+), 24 deletions(-)
diff --git a/target-mips/t
1 - 100 of 116 matches
Mail list logo
