Hi everyone,
My question might look very stupid but the answer would really help me.
I am working on Android emulator which is using QEMU. I need to print out the
value of env->cp15.c13_fcse everytime the
voidtlb_flush(CPUState*env,intflush_global) in exec.c is called. The problem is
when co
On Wed, Nov 13, 2013 at 12:04:18PM +0800, Fam Zheng wrote:
> We were lucky to pass qemu-iotests 048 (qemu-img compare case) but when
> I tried to run with TEST_DIR=/tmp (tmpfs), it fails with a very weird
> mismatch offset. This fixes the bug.
>
> In the if branch, setting ret to 1 before using it
On Wed, Nov 13, 2013 at 12:27:10PM +1000, Richard Henderson wrote:
> On 11/13/2013 08:53 AM, Paolo Bonzini wrote:
> > Il 12/11/2013 19:54, Richard Henderson ha scritto:
> >> For what it's worth, I think BOTH of the patches that have been posted
> >> should be applied. That is, the patch that does
Il 13/11/2013 03:27, Richard Henderson ha scritto:
> I think it's also worthwhile to implement the kvm api in kvm-stub.c,
> unnecessary or not. If you really want compile-time feedback on those that
> ought to have been removed by optimization, you could elide them from the stub
> file depending o
On 11/11/2013 11:48 AM, Amos Kong wrote:
> mac_table was always cleaned up first in handling
> VIRTIO_NET_CTRL_MAC_TABLE_SET command, and we din't recover
> mac_table content in error state, it's not correct.
>
> This patch makes all the changes in temporal variables,
> only update the real mac_tab
On 2013年11月13日 14:32, Fam Zheng wrote:
On 2013年11月13日 10:39, Xu Wang wrote:
Backing file loop should be checked before qemu-img create command
execution. If loop is found, qemu-img create should be stopped and
an error printed.
Signed-off-by: Xu Wang
---
block.c | 9 +
1 file change
On 2013年11月13日 10:39, Xu Wang wrote:
If there is loop exists in the backing file chain, many problems
could be caused by it, such as no response and segment fault during
system boot. Hence stopping backing file loop appear is very necessary.
These patches refine and export loop checking function
On 2013年11月13日 10:39, Xu Wang wrote:
Backing file loop should be checked before qemu-img create command
execution. If loop is found, qemu-img create should be stopped and
an error printed.
Signed-off-by: Xu Wang
---
block.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff
Am 13.11.2013 um 07:27 schrieb Peter Lieven :
>
> Am 12.11.2013 um 16:49 schrieb Paolo Bonzini :
>
>> Writing zeroes to a file can be done by punching a hole if MAY_UNMAP
>> is set.
>>
>> Note that in this case handle_aiocb_discard's ENOTSUP return code
>> is not ignored, but makes the block l
On 2013年11月13日 10:39, Xu Wang wrote:
If there is a loop in the backing file chain, it could cause problems
such as no response or a segfault during system boot. Hence detecting a
backing file loop is necessary. This patch extracts the loop check from
collect_image_info_list() in block.c into inde
Am 12.11.2013 um 16:49 schrieb Paolo Bonzini :
> Writing zeroes to a file can be done by punching a hole if MAY_UNMAP
> is set.
>
> Note that in this case handle_aiocb_discard's ENOTSUP return code
> is not ignored, but makes the block layer fall back to the generic
> implementation.
>
> Signed
Wouldn't it be good to add bdi->can_write_zeroes_with_unmap here as well?
This would automatically avoid full allocation when converting something to a
host device
supporting BLKDISCARDZEROES.
Peter
Am 12.11.2013 um 16:49 schrieb Paolo Bonzini :
> See the next commit for the description of the
Later both qapi-types.py and qapi-visit.py need a common function
for enum name generation.
Signed-off-by: Wenchao Xia
Reviewed-by: Eric Blake
---
scripts/qapi-types.py | 10 --
scripts/qapi.py | 10 ++
2 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/s
Test for inherit and complex union.
Signed-off-by: Wenchao Xia
---
tests/qapi-schema/qapi-schema-test.json | 27 ++
tests/qapi-schema/qapi-schema-test.out | 11 +++
tests/test-qmp-input-visitor.c | 120 +
tests/test-qmp-output-visitor.c | 149 +
By default, any union will automatically generate a enum type as
"[UnionName]Kind" in C code, and it is duplicated when the discriminator
is specified as a pre-defined enum type in schema. After this patch,
the pre-defined enum type will be really used as the switch case
condition in generated C c
Later other scripts will need to check the enum values.
Signed-off-by: Wenchao Xia
Reviewed-by: Eric Blake
---
scripts/qapi.py| 18 ++
tests/qapi-schema/comments.out |2 +-
tests/qapi-schema/qapi-schema-test.out |4 +++-
3 files changed,
Now "enum AIOContext" will generate AIO_CONTEXT instead of A_I_O_CONTEXT,
"X86CPU" will generate X86_CPU instead of X86_C_P_U.
Signed-off-by: Wenchao Xia
---
include/qapi/qmp/qerror.h |2 +-
scripts/qapi.py | 26 +++---
target-i386/cpu.c |2 +-
3 f
It will check whether the values specified are written correctly,
and whether all enum values are covered, when discriminator is a
pre-defined enum type
Signed-off-by: Wenchao Xia
---
scripts/qapi-visit.py | 17 +
scripts/qapi.py | 31 +++
2
This series is respined from RFC series at:
http://lists.nongnu.org/archive/html/qemu-devel/2013-11/msg00363.html
Patch 1-6 add support for enum as discriminator.
Patch 7 improve enum name generation, now AIOContext->AIO_CONTEXT, X86CPU->
X86_CPU.
Patch 8 are the test cases.
Changes from RFC:
M
One function one rule, so the enum string generating have same
behavior for different caller. If multiple caller exist for one
enum define in schema, it is for sure the generated string is
identical.
Note before the patch qapi-visit.py used custom function to
generate the string in union visit, al
It is bad that same key was specified twice, especially when a union have
two branches with same condition. This patch can prevent it.
Signed-off-by: Wenchao Xia
---
scripts/qapi.py |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/scripts/qapi.py b/scripts/qapi.py
index
Am 12.11.2013 um 16:49 schrieb Paolo Bonzini :
> The WRITE SAME command is implemented incorrectly. WRITE SAME with the
> UNMAP bit set should _not_ unmap the sectors unless the written data
> matches the payload of the WRITE SAME command; currently, QEMU is not
> looking at the payload at all.
于 11/12/2013 10:52 PM, Andreas Färber 写道:
Resending yesterday's message since it hasn't arrived on qemu-devel...
Am 11.11.2013 04:58, schrieb 赵小强:
于 11/05/2013 04:51 PM, 赵小强 写道:
于 2013年11月05日 16:25, Chen Fan 写道:
On Tue, 2013-11-05 at 15:55 +0800, xiaoqiang zhao wrote:
[...]
diff --git a/inc
Am 12.11.2013 um 16:49 schrieb Paolo Bonzini :
> bdrv_co_discard is only covering drivers which have a .bdrv_co_discard()
> implementation, but not those with .bdrv_aio_discard(). Not very nice,
> and easy to avoid.
>
> Suggested-by: Kevin Wolf
> Signed-off-by: Paolo Bonzini
> ---
> block.c |
I also have this issue. USB pass-through didn't work on windows 8. I try
to use "virt-mamanger", and set USB interface to USB 2.0. Then
everything works well. The default one would be USB 1.0.
I don't know how to transform virt-manager's configuration to QEMU's
command line arguments. Hope this h
What is your use case for this seris? QCOW2 creation or converting
anything to QCOW2? For the later case you could use "qemu-img convert -S 0
..."
starting in 1.8.
Peter
Hu Tao wrote:
> This adds a preallocation=full mode to qcow2 image creation, which
> creates a non-sparse image file.
>
> Signe
Am 12.11.2013 23:28, schrieb Mike Levin:
> Hello QEMU developers,
>
> I'm trying to do the closes thing to a static linked compile with
> curses enabled of qemu-system32.exe for the Windows platform as possible.
>
> I've resisted mailing this developer mailing list for a year before I
> decided to
Currently the output error is always:
strerror(-4) -> Unknown error -4
This patch moves ret assignment after reporting original error.
Signed-off-by: Amos Kong
---
qemu-img.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/qemu-img.c b/qemu-img.c
index 926f0a0..1fd664f 100
We were lucky to pass qemu-iotests 048 (qemu-img compare case) but when
I tried to run with TEST_DIR=/tmp (tmpfs), it fails with a very weird
mismatch offset. This fixes the bug.
In the if branch, setting ret to 1 before using it makes dead code in
the next line: pnum is never added to mismatch of
Backing file loop should be checked before qemu-img create command
execution. If loop is found, qemu-img create should be stopped and
an error printed.
Signed-off-by: Xu Wang
---
block.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/block.c b/block.c
index 3c43179.
If there is loop exists in the backing file chain, many problems
could be caused by it, such as no response and segment fault during
system boot. Hence stopping backing file loop appear is very necessary.
These patches refine and export loop checking function from collect_image_
info_list() to bloc
Backing file loop should be checked before calling change_backing_
file(). If loop appeared, this calling should be stopped and an
error printed.
Signed-off-by: Xu Wang
---
block.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/block.c b/block.c
index c2ed6ef..861b133 100644
--- a/bloc
If there is a loop in the backing file chain, it could cause problems
such as no response or a segfault during system boot. Hence detecting a
backing file loop is necessary. This patch extracts the loop check from
collect_image_info_list() in block.c into independent functions
bdrv_backing_chain_ok
On Tue, Nov 12, 2013 at 11:31:03AM +0100, Kevin Wolf wrote:
> Am 12.11.2013 um 08:47 hat Hu Tao geschrieben:
> > This adds a preallocation=full mode to qcow2 image creation, which
> > creates a non-sparse image file.
> >
> > Signed-off-by: Hu Tao
> > ---
> > block/qcow2.c | 28 ++
On Tue, Nov 12, 2013 at 11:28:05AM +0100, Kevin Wolf wrote:
> Am 12.11.2013 um 08:47 hat Hu Tao geschrieben:
> > Implement bdrv_zero_init using posix_fallocate.
> >
> > Signed-off-by: Hu Tao
> > ---
> > block/raw-posix.c | 13 +
> > 1 file changed, 13 insertions(+)
> >
> > diff --gi
On 11/11/2013 10:57 PM, Andreas Färber wrote:
> Am 11.11.2013 08:44, schrieb Alexey Kardashevskiy:
>> This adds a macro to calculate the highest bit set.
>
> Isn't that already available as ffs / clz GCC builtin with wrapper in
> qemu/bitops.h? What's the difference to your macro? CC'ing Paolo.
于 2013/11/13 2:12, Eric Blake 写道:
On 11/06/2013 12:33 PM, Wenchao Xia wrote:
It will check whether the values specified are written correctly when
discriminator is a pre-defined enum type, which help check whether the
schema is in good form.
It is allowed, that not every value in enum is used,
On 11/13/2013 08:53 AM, Paolo Bonzini wrote:
> Il 12/11/2013 19:54, Richard Henderson ha scritto:
>> For what it's worth, I think BOTH of the patches that have been posted
>> should be applied. That is, the patch that does (X || 1) -> (1 || X),
>> and the patch that adds the stub.
>>
>> Frankly I'
On 11/13/2013 12:11 AM, Igor Mammedov wrote:
> On Tue, 12 Nov 2013 23:39:27 +1100
> Alexey Kardashevskiy wrote:
>
>> On 12.11.2013 20:58, Igor Mammedov wrote:
>>> On Tue, 12 Nov 2013 10:49:58 +1100
>>> Alexey Kardashevskiy wrote:
>>>
On 11/12/2013 01:25 AM, Igor Mammedov wrote:
> On Mon
On 11/12/2013 11:45 PM, Andreas Färber wrote:
> Am 12.11.2013 13:39, schrieb Alexey Kardashevskiy:
>> On 12.11.2013 20:58, Igor Mammedov wrote:
>>> PS:
>>> extending QemuOpts to parsing +/-opts format, seems like good workaround
>>> above problem. But I was under impression that general movement wa
bdrv_get_info could fail. Add check before using the returned value.
Signed-off-by: Fam Zheng
---
block/mirror.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/block/mirror.c b/block/mirror.c
index 7b95acf..c0c321b 100644
--- a/block/mirror.c
+++ b/block/mirror.c
@@ -320
This will return cluster_size and is_compressed to caller, if all the
extents has the same value (or there's only one extent). Otherwise
return -ENOTSUP.
Signed-off-by: Fam Zheng
---
block/vmdk.c | 19 +++
tests/qemu-iotests/059.out | 1 +
2 files changed, 20 inser
VMDK could have big cluster_size for monolithicFlat. It implements
.bdrv_get_info now, a 32 bit field is likely to overflow.
Signed-off-by: Fam Zheng
---
include/block/block.h | 2 +-
qemu-img.c| 3 ++-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/include/block/bloc
If target block driver forces compression, qemu-img convert needs to
write by cluster size as well as "-c" option.
Particularly, this applies for converting to VMDK streamOptimized
format.
Signed-off-by: Fam Zheng
---
include/block/block.h | 1 +
qemu-img.c| 5 -
2 files changed
Add a wrapper function to support "compressed" path in qemu-img convert.
Only support streamOptimized subformat case for now (num_extents == 1
and extent compression is true).
Signed-off-by: Fam Zheng
---
block/vmdk.c | 14 ++
1 file changed, 14 insertions(+)
diff --git a/block/vmdk
Previouly, "qemu-img convert" from ISO to VMDK with subformat=streamOptimized
fails:
$ ./qemu-img convert -O vmdk -o subformat=streamOptimized foo.iso bar.vmdk
VMDK: can't write to allocated cluster for streamOptimized
qemu-img: error while writing sector 64: Input/output error
Becaus
Il 12/11/2013 23:10, Michael S. Tsirkin ha scritto:
> This was already discussed on previous revisions of this patch.
> Have you seen that discussion?
Obviously not. :)
Paolo
Il 12/11/2013 19:54, Richard Henderson ha scritto:
> For what it's worth, I think BOTH of the patches that have been posted
> should be applied. That is, the patch that does (X || 1) -> (1 || X),
> and the patch that adds the stub.
>
> Frankly I'd have thought this was obvious
It's not that obvi
Hello QEMU developers,
I'm trying to do the closes thing to a static linked compile with curses
enabled of qemu-system32.exe for the Windows platform as possible.
I've resisted mailing this developer mailing list for a year before I
decided to turn here for help. I've made a self-contained QEMU/L
On Tue, Nov 12, 2013 at 07:26:02PM +0100, Paolo Bonzini wrote:
> Il 12/11/2013 14:58, Igor Mammedov ha scritto:
> > 'etc/reserved-memory-end' will allow QEMU to tell BIOS where PCI
> > BARs mapping could safely start in high memory.
> >
> > Allowing BIOS to start mapping 64-bit PCI BARs at address
On Thu, Oct 10, 2013 at 11:14:22PM +0100, Gabriel Kerneis wrote:
> We were so excited about our QEMU/CPC GSoC project that we have decided
> to write a 12-page article to share our experience
Our "QEMU/CPC" paper has been accepted for publication at PEPM'14:
http://www.program-transformation.org/P
v2: condition enablement of new mapping to new machine types (Paolo)
v3: fix changelog
v4: rebase
v5: ensure alignment of piecetwo on 2MB GPA (Igor)
do not register zero-sized piece-one(Igor)
v6: fix memory leak (Igor)
fix integer overflow(Ig
On Tue, Nov 12, 2013 at 01:45:51PM +0100, Igor Mammedov wrote:
> On Sun, 10 Nov 2013 18:47:53 -0200
> Marcelo Tosatti wrote:
>
> [...]
>
> > @@ -1177,10 +1182,50 @@ FWCfgState *pc_memory_init(MemoryRegion
> > *system_memory,
> > e820_add_entry(0, below_4g_mem_size, E820_RAM);
> > if (
On Tue, 12 Nov 2013 19:26:02 +0100
Paolo Bonzini wrote:
> Il 12/11/2013 14:58, Igor Mammedov ha scritto:
> > 'etc/reserved-memory-end' will allow QEMU to tell BIOS where PCI
> > BARs mapping could safely start in high memory.
> >
> > Allowing BIOS to start mapping 64-bit PCI BARs at address wher
On 11/10/2013 10:48 PM, Amos Kong wrote:
mac_table was always cleaned up first in handling
VIRTIO_NET_CTRL_MAC_TABLE_SET command, and we din't recover
mac_table content in error state, it's not correct.
This patch makes all the changes in temporal variables,
only update the real mac_table if eve
On 11/08/2013 10:43 PM, Amos Kong wrote:
On Fri, Nov 08, 2013 at 02:42:27PM -0500, Vlad Yasevich wrote:
What about this approach? This only updates the monitory when all the
bits have been written to.
Hi Vlad,
Looks good to me.
Using this patch, we don't need to care the writing order.
If w
On Nov 12, 2013 9:23 PM, "Eric Blake" wrote:
>
> On 11/12/2013 04:10 AM, Avi Kivity wrote:
> > Out-of-the-box, 'make install' sets up an unusable qemu-bridge-helper
since
> > it doesn't have the required capabilities.
> >
> > Fix by adding them.
> >
> > Note: this may break installing as non-root.
On 11/12/2013 04:10 AM, Avi Kivity wrote:
> Out-of-the-box, 'make install' sets up an unusable qemu-bridge-helper since
> it doesn't have the required capabilities.
>
> Fix by adding them.
>
> Note: this may break installing as non-root. This is actually the right
> thing to do, since not settin
- Original Message -
> I haven't been there at the original creation of this functionality, but
> I tend to agree with you. For analyzing the vmcore with gdb or crash,
> the alignment doesn't seem to be important, so it was probably ignored.
With respect to the crash utility, the p_alig
Am 12.11.2013 19:57, schrieb Peter Maydell:
> On 12 November 2013 18:54, Richard Henderson wrote:
>> For what it's worth, I think BOTH of the patches that have been posted
>> should be applied. That is, the patch that does (X || 1) -> (1 || X),
>> and the patch that adds the stub.
> I think that
Out-of-the-box, 'make install' sets up an unusable qemu-bridge-helper since
it doesn't have the required capabilities.
Fix by adding them.
Note: this may break installing as non-root. This is actually the right
thing to do, since not setting up the capability would result in a broken
setup. Per
Public bug reported:
Description of problem:
In case of power failure disk images that were active and created in qcow2
format can become logically corrupt so that they actually appear as unused
(full of zeroes).
Data seems to be there, but at this moment i cannot find any reliable method to
re
On 12 November 2013 18:54, Richard Henderson wrote:
> For what it's worth, I think BOTH of the patches that have been posted
> should be applied. That is, the patch that does (X || 1) -> (1 || X),
> and the patch that adds the stub.
I think that makes sense and would be happy with that as a reso
On 11/13/2013 03:04 AM, Anthony Liguori wrote:
> On Tue, Nov 12, 2013 at 8:08 AM, Peter Maydell
> wrote:
>> On 12 November 2013 15:58, Paolo Bonzini wrote:
>>> I don't really see a reason why QEMU should give clang more weight than
>>> Windows or Mac OS X.
>>
>> I'm not asking for more weight (a
When an assigned device is initialized it copies the device config
space into the emulated config space. Unfortunately multifunction is
setup prior to the device initfn and gets clobbered. We need to
restore it just like pci-assign does.
Signed-off-by: Alex Williamson
Cc: qemu-sta...@nongnu.org
Signed-off-by: Luiz Capitulino
---
MAINTAINERS | 4
1 file changed, 4 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 77edacf..02b85ee 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -699,6 +699,7 @@ S: Supported
F: monitor.c
F: hmp.c
F: hmp-commands.hx
+T: git git://repo.or.cz/
From: Richard Henderson
We say we support python 2.4, but python 2.4.3 does not
support the "expr if test else expr" syntax used here.
This allows QEMU to compile on RHEL 5.3, the last release for ia64.
Signed-off-by: Richard Henderson
Reviewed-by: Michael Roth
Signed-off-by: Luiz Capitulino
The following changes since commit 964668b03d26f0b5baa5e5aff0c966f4fcb76e9e:
Update version for 1.7.0-rc0 release (2013-11-06 21:49:39 -0800)
are available in the git repository at:
git://repo.or.cz/qemu/qmp-unstable.git queue/qmp
for you to fetch changes up to dce07e0b2b172ccce43955fdee214
On 11/12/2013 11:29 PM, Claudio Fontana wrote:
> provide a skeleton for a64 instruction decoding in translate-a64.c,
> by dividing instructions into the classes defined by the
> ARM Architecture Reference Manual(DDI0487A_a) C3
>
> Signed-off-by: Claudio Fontana
> Signed-off-by: Alex Bennée
> Rev
Il 12/11/2013 14:58, Igor Mammedov ha scritto:
> 'etc/reserved-memory-end' will allow QEMU to tell BIOS where PCI
> BARs mapping could safely start in high memory.
>
> Allowing BIOS to start mapping 64-bit PCI BARs at address where it
> wouldn't conflict with other mappings QEMU might place before
On 11/06/2013 12:33 PM, Wenchao Xia wrote:
> Later both qapi-types.py and qapi-visit.py need a common function
> for enum name generation.
>
> Signed-off-by: Wenchao Xia
> ---
> scripts/qapi-types.py | 10 --
> scripts/qapi.py | 10 ++
> 2 files changed, 10 insertions(+
On 11/06/2013 12:33 PM, Wenchao Xia wrote:
> It will check whether the values specified are written correctly when
> discriminator is a pre-defined enum type, which help check whether the
> schema is in good form.
>
> It is allowed, that not every value in enum is used, so does not check
> that ca
On 12 November 2013 17:04, Anthony Liguori wrote:
> QEMU has always been intimately tied to GCC. Heck, it all started as
> a giant GCC hack relying on entirely undocumented behavior (dyngen's
> disassembly of functions).
It has historically. Blue Swirl put in a lot of work to
remove those depend
On 11/12/2013 10:43 AM, Eric Blake wrote:
> On 11/12/2013 09:33 AM, Corey Minyard wrote:
>> Allow a socket that connects to reconnect on a periodic basis if it
>> fails to connect at startup or if the connection drops while in use.
>>
>> Signed-off-by: Corey Minyard
>> ---
>> include/sysemu/char.
On Tue, Nov 12, 2013 at 8:08 AM, Peter Maydell wrote:
> On 12 November 2013 15:58, Paolo Bonzini wrote:
>> I don't really see a reason why QEMU should give clang more weight than
>> Windows or Mac OS X.
>
> I'm not asking for more weight (and actually my main
> reason for caring about clang is ex
Am 12.11.2013 04:02, schrieb Chen Fan:
> On Tue, 2013-11-12 at 09:54 +0800, 赵小强 wrote:
>> He asked me to drop the parent_realize. so in v2, I just replace the
>> 'init' with 'realize'.
>>
> Hmm,I'm confused, Go through the entire QEMU source code, there was no
> DeviceRealize realize field in leaf
This adds an interface for IPMI that connects to a remote
BMC over a chardev (generally a TCP socket). The OpenIPMI
lanserv simulator describes this interface, see that for
interface details.
Signed-off-by: Corey Minyard
---
default-configs/i386-softmmu.mak | 1 +
default-configs/x86_64-sof
On 11/06/2013 12:33 PM, Wenchao Xia wrote:
> It is possible to reach default case, when an union have a enum
> discriminator, so don't abort() but report the error message.
>
> Signed-off-by: Wenchao Xia
> ---
> scripts/qapi-visit.py | 14 --
> 1 files changed, 12 insertions(+), 2
Allow a socket that connects to reconnect on a periodic basis if it
fails to connect at startup or if the connection drops while in use.
Signed-off-by: Corey Minyard
---
include/sysemu/char.h | 3 ++
qemu-char.c | 88 ---
qemu-options.hx
On 11/05/2013 08:27 PM, 宫文超 wrote:
> hello evryone,who can give me a detail explain of the qemu code of the online
> snapshot?THX
Top-posting and replying to a digest message (especially without
changing the subject line) is poor netiquette. Better would have been
starting a new thread.
Your qu
This provides a minimal local BMC, basically enough to comply with the
spec and provide a complete watchdog timer (including a sensor, SDR,
and event).
Signed-off-by: Corey Minyard
---
default-configs/i386-softmmu.mak |1 +
default-configs/x86_64-softmmu.mak |1 +
hw/ipmi/Makefile.objs
On 11/12/2013 09:33 AM, Corey Minyard wrote:
> Allow a socket that connects to reconnect on a periodic basis if it
> fails to connect at startup or if the connection drops while in use.
>
> Signed-off-by: Corey Minyard
> ---
> include/sysemu/char.h | 3 ++
> qemu-char.c | 88
> ++
Signed-off-by: Corey Minyard io_length = 3;
memory_region_init_io(&s->io, OBJECT(s), &ipmi_bt_io_ops, bt, "ipmi-bt",
3);
+vmstate_register(NULL, 0, &vmstate_ipmi_bt, bt);
return 0;
}
diff --git a/hw/ipmi/ipmi_extern.c b/hw/ipmi/ipmi_extern.c
index 1cd7c11..d66797d 100644
--- a/h
Since we report ANC_SUP==0 in VPD page B2h, we need to return
an error (ILLEGAL REQUEST/INVALID FIELD IN CDB) for all WRITE SAME
requests with ANCHOR==1.
Inspired by a similar patch to the LIO in-kernel target.
Signed-off-by: Paolo Bonzini
---
hw/scsi/scsi-disk.c | 12 +++-
1 file chang
Add some basic documentation for the IPMI device.
Signed-off-by: Corey Minyard
---
qemu-options.hx | 35 +++
1 file changed, 35 insertions(+)
diff --git a/qemu-options.hx b/qemu-options.hx
index 5bcfaa0..500d7c8 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
Add an IPMI table entry to the SMBIOS.
---
hw/ipmi/isa_ipmi.c | 29 +
include/hw/i386/smbios.h | 14 ++
2 files changed, 43 insertions(+)
diff --git a/hw/ipmi/isa_ipmi.c b/hw/ipmi/isa_ipmi.c
index b38c846..e40ca90 100644
--- a/hw/ipmi/isa_ipmi.c
+++ b
There was no way to directly add a table entry to the SMBIOS table,
even though the BIOS supports this. So add a function to do this.
This is in preparation for the IPMI handler adding it's SMBIOS table
entry.
Signed-off-by: Corey Minyard
---
hw/i386/smbios.c | 27 +++
1
This provides the simulation of the BT hardware interface for
IPMI.
Signed-off-by: Corey Minyard
---
default-configs/i386-softmmu.mak | 1 +
default-configs/x86_64-softmmu.mak | 1 +
hw/ipmi/Makefile.objs | 1 +
hw/ipmi/ipmi_bt.c | 367 ++
Loaded all ACPI tables from guest, making
a good environment for further unit tests.
Checked that ACPI tables are corrected pointed
within the ACPI tree using their signatures.
Verified checksum for all the tables.
Signed-off-by: Marcel Apfelbaum
---
To be applied on top of: [PATCH v2 2/2] acpi-
This provides the base infrastructure to tie IPMI low-level
interfaces into a PC ISA bus.
Signed-off-by: Corey Minyard
---
default-configs/i386-softmmu.mak | 1 +
default-configs/x86_64-softmmu.mak | 1 +
hw/ipmi/Makefile.objs | 1 +
hw/ipmi/isa_ipmi.c | 148
Test the KCS interface with a local BMC and a BT interface with an
external BMC.
Signed-off-by: Corey Minyard
---
tests/Makefile| 4 +
tests/ipmi-bt-test.c | 440 ++
tests/ipmi-kcs-test.c | 294 +
3 files
This provides the simulation of the KCS hardware interface.
Signed-off-by: Corey Minyard
---
default-configs/i386-softmmu.mak | 1 +
default-configs/x86_64-softmmu.mak | 1 +
hw/ipmi/Makefile.objs | 1 +
hw/ipmi/ipmi_kcs.c | 345 ++
Add the basic IPMI types and infrastructure to QEMU. Low-level
interfaces and simulation interfaces will register with this; it's
kind of the go-between to tie them together.
Signed-off-by: Corey Minyard
---
default-configs/i386-softmmu.mak | 1 +
default-configs/x86_64-softmmu.mak | 1 +
This allocates the CharDriverState structure and passes it in to the
open routine. This allows a coming option to automatically attempt to
reconnect a chardev if the connection fails. The chardev has to be
kept around so a reconnect can be done on it.
Signed-off-by: Corey Minyard
---
backends/
Postpone the addition of the ACPI and SMBIOS tables until after
device initialization. This allows devices to add entries to these
tables.
Signed-off-by: Corey Minyard
---
hw/i386/pc.c | 38 ++
1 file changed, 30 insertions(+), 8 deletions(-)
diff --git a/hw
This will result in a double free on close, because it's freed
in qemu_chr_delete() right after calling the close function.
Signed-off-by: Corey Minyard
---
qemu-char.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/qemu-char.c b/qemu-char.c
index 23d7647..935066d 100644
--- a/qemu-char.c
++
The chardev backends that used qemu_chr_open_fd did not get their
file descriptors closed at end of file or when the chardev was closed.
This could result in a file descriptor leak.
Signed-off-by: Corey Minyard
---
qemu-char.c | 35 +--
1 file changed, 29 insertio
There are two (sets of) patches to the general code beyond the IPMI
device addition.
One set adds an option to qemu-char net devices to automatically try to
reconnect if the connection disconnects. This lets the IPMI device
connect to a remote BMC and recover if that BMC fails.
The other set all
On 11/12/13 14:58, Igor Mammedov wrote:
> From: "Michael S. Tsirkin"
>
> With a help of negative memory region priority PCI address space
> is mapped underneath RAM regions effectively catching every access
> to addresses not mapped by any other region.
> It simplifies PCI address space mapping i
On 11/12/13 17:05, Igor Mammedov wrote:
> On Tue, 12 Nov 2013 16:11:49 +0100
> Laszlo Ersek wrote:
>
>
>> Second, I tested the patch under the following circumstances:
>> - 3.10-based host kernel,
>> - qemu v1.7.0-rc0, with additional patches that shrink the "pci-hole"
>> memory range to just
1 - 100 of 178 matches
Mail list logo