Am 03.06.2016 um 17:01 hat Denis V. Lunev geschrieben:
> On 06/03/2016 03:30 PM, Kevin Wolf wrote:
> >This options allows to flush the image periodically during write tests.
> >
> >Signed-off-by: Kevin Wolf
> This pattern could be a bit different - you wait requests to finish
>
On Wed, 1 Jun 2016 18:57:37 +1000
Alexey Kardashevskiy wrote:
> Every IOMMU has some granularity which MemoryRegionIOMMUOps::translate
> uses when translating, however this information is not available outside
> the translate context for various checks.
>
> This adds a
-Original Message-
From: Alexander Graf [mailto:ag...@suse.de]
Sent: Friday, June 03, 2016 6:26 PM
To: Mihai Claudiu Caraman ; Peter Maydell
> Cc: Bogdan Purcareata ; QEMU Developers
> ;
On 03/06/16 12:18, Emilio G. Cota wrote:
> On Sun, May 29, 2016 at 22:52:27 +0300, Sergey Fedorov wrote:
>> I was just wondering if it could be worthwhile to pass a hash function
>> when initializing a QHT. Then we could have variants of qht_insert(),
>> qht_remove() and qht_lookup() which does
On 06/03/2016 08:16 AM, Sergey Fedorov wrote:
On 03/06/16 04:08, Richard Henderson wrote:
So your suggestion is to generate different TCG opcode sequences
depending on the underlying target architecture? And you are against
forwarding this task further, to the backend code?
Yes, I would prefer
On Wed, 1 Jun 2016 18:57:38 +1000
Alexey Kardashevskiy wrote:
> This makes use of the new "memory registering" feature. The idea is
> to provide the userspace ability to notify the host kernel about pages
> which are going to be used for DMA. Having this information, the host
>
Sector-based limits are awkward to think about; in our on-going
quest to move to byte-based interfaces, convert max_discard and
discard_alignment. Rename them, using 'pdiscard' as an aid to
track which remaining discard interfaces need conversion, and so
that the compiler will help us catch the
Sector-based limits are awkward to think about; in our on-going
quest to move to byte-based interfaces, convert max_transfer_length
and opt_transfer_length. Rename them (dropping the _length suffix)
so that the compiler will help us catch the change in semantics
across any rebased code. Improve
BlockLimits is currently an ugly mix of byte limits vs.
sector limits. Unify it. Fix some bugs I found in
bdrv_aligned_preadv() while at it.
Probably conflicts with Kevin's ongoing work to migrate
bdrv_aligned_preadv() to be byte-based, but I found this
handy before tackling conversion of
It makes more sense to have ALL block size limit constraints
in the same struct. Improve the documentation while at it.
Note that bdrv_refresh_limits() has to keep things alive across
a memset() of BlockLimits.
Signed-off-by: Eric Blake
---
include/block/block_int.h | 12
On 06/03/2016 03:30 PM, Kevin Wolf wrote:
This options allows to flush the image periodically during write tests.
Signed-off-by: Kevin Wolf
This pattern could be a bit different - you wait requests to finish
and after that start flush. In this case there is no gain with my
On 03/06/16 14:41, Emilio G. Cota wrote:
> On Sun, May 29, 2016 at 23:45:23 +0300, Sergey Fedorov wrote:
>> On 25/05/16 04:13, Emilio G. Cota wrote:
>>> diff --git a/tests/qht-bench.c b/tests/qht-bench.c
>>> new file mode 100644
>>> index 000..30d27c8
>>> --- /dev/null
>>> +++
Am 01.06.2016 um 23:10 hat Eric Blake geschrieben:
> Another step on our continuing quest to switch to byte-based
> interfaces.
>
> Signed-off-by: Eric Blake
> ---
> block/raw-posix.c | 34 +-
> trace-events | 2 +-
> 2 files changed, 18
On 15/04/16 17:23, Alex Bennée wrote:
> diff --git a/include/qemu/atomic.h b/include/qemu/atomic.h
> index 5dba7db..94e7110 100644
> --- a/include/qemu/atomic.h
> +++ b/include/qemu/atomic.h
> @@ -123,6 +123,16 @@
> _old; \
>
> -Original Message-
> From: Qemu-devel
> [mailto:qemu-devel-bounces+mihai.caraman=freescale@nongnu.org] On Behalf
> Of Peter Maydell
> Sent: Thursday, June 02, 2016 3:33 PM
> To: Bogdan Purcareata
> Cc: QEMU Developers ; Peter
On 06/02/2016 12:35 PM, Denis V. Lunev wrote:
> The command will work this way:
> qemu-img --trace qcow2* create -f qcow2 1.img 64G
>
> Signed-off-by: Denis V. Lunev
> Suggested by: Daniel P. Berrange
> CC: Eric Blake
> CC: Paolo
On Wed, 1 Jun 2016 18:57:41 +1000
Alexey Kardashevskiy wrote:
> New VFIO_SPAPR_TCE_v2_IOMMU type supports dynamic DMA window management.
> This adds ability to VFIO common code to dynamically allocate/remove
> DMA windows in the host kernel when new VFIO container is
On 06/02/2016 08:01 AM, Sascha Silbe wrote:
> Dear Jianjun,
>
> Jianjun Duan writes:
>
> [include/migration/vmstate.h]
>> @@ -185,6 +185,8 @@ enum VMStateFlags {
>> * to determine the number of entries in the array. Only valid in
>> * combination with one
On 03/06/16 16:45, Jakub Horak wrote:
(Added David as PPC maintainer and qemu-ppc mailing list)
> Hello,
> I think there's a bug in "wait" instruction code generator for PowerPC
> architecture. It doesn't make sense to store a non-initialized register.
>
> Best regards,
> Jakub Horak
>
>
>
On 06/03/2016 03:30 PM, Kevin Wolf wrote:
This extends qemu-img bench with an option that makes it use sequential
writes instead of reads for the test run.
Signed-off-by: Kevin Wolf
---
qemu-img-cmds.hx | 4 ++--
qemu-img.c | 24 +++-
On 03/06/16 15:14, Mark Cave-Ayland wrote:
> On 03/06/16 15:06, Cedric Le Goater wrote:
>
>> On 06/03/2016 04:00 PM, Cédric Le Goater wrote:
>>> Hello Mark,
>>>
>>> On 06/03/2016 03:52 PM, Mark Cave-Ayland wrote:
On 03/06/16 13:11, Cédric Le Goater wrote:
> This is follow up to
2016-06-03 14:09+0800, Haozhong Zhang:
> This patch adds the support to inject SRAR and SRAO as LMCE, i.e. they
> will be injected to only one VCPU rather than broadcast to all
> VCPUs. As KVM reports LMCE support on Intel platforms, this features is
> only available on Intel platforms.
>
>
On Thu, Jun 02, 2016 at 11:15:54PM +0300, Marcel Apfelbaum wrote:
> Use the standard '-device iommu' instead of '-machine,iommu=on'
> to create the IOMMU device.
>
> Signed-off-by: Marcel Apfelbaum
Why can't we keep support for the old flag?
> ---
> hw/core/machine.c |
Not that we pass any flags during reads yet, but we may want to
support BDRV_REQ_FUA on reads in the future. So don't throw
away the input flags.
Signed-off-by: Eric Blake
---
block/io.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/block/io.c
On 06/03/2016 06:06 PM, Eric Blake wrote:
On 06/03/2016 08:05 AM, Vladimir Sementsov-Ogievskiy wrote:
Add target-zeroed flag to allow user specify that target is already
zeroed. With this flag set zeroes which was in source before mirror
start will not be copyed.
With this flag set, any runs
On Tue, May 24, 2016 at 12:37:19PM -0500, miny...@acm.org wrote:
> From: Corey Minyard
>
> This will let things in other files (like IPMI) build SMBIOS tables.
>
> Signed-off-by: Corey Minyard
> ---
> hw/smbios/smbios.c | 70
On Wed, 1 Jun 2016 18:57:43 +1000
Alexey Kardashevskiy wrote:
> The sPAPR TCE tables manage 2 copies when VFIO is using an IOMMU -
> a guest view of the table and a hardware TCE table. If there is no VFIO
> presense in the address space, then just the guest view is used, if
>
Make sure everything is aligned to the passed-in alignment,
not just sectors. Also makes sure that the computation of
max_nb_sectors later in the function will not divide by
zero, by guaranteeing align is at least a sector.
Signed-off-by: Eric Blake
---
block/io.c | 5 +++--
On 03.06.2016 11:07, Fam Zheng wrote:
> Somehow in my locking series, I missed this case where concurrent access
> to an image is performed, perhaps we can remove this case independently.
The patch itself is good, but this commit message will not make much
sense in the git log. I'd be fine with
On 03/06/16 14:01, Emilio G. Cota wrote:
> On Sun, May 29, 2016 at 22:52:27 +0300, Sergey Fedorov wrote:
>>> +/*
>>> + * Find the last valid entry in @head, and swap it with @orig[pos], which
>>> has
>>> + * just been invalidated.
>>> + */
>>> +static inline void qht_bucket_fill_hole(struct
On Wed, 1 Jun 2016 18:57:40 +1000
Alexey Kardashevskiy wrote:
> There are going to be multiple IOMMUs per a container. This moves
> the single host IOMMU parameter set to a list of VFIOHostDMAWindow.
>
> This should cause no behavioral change and will be used later by
> the
On 06/03/2016 08:05 AM, Vladimir Sementsov-Ogievskiy wrote:
> Add target-zeroed flag to allow user specify that target is already
> zeroed. With this flag set zeroes which was in source before mirror
> start will not be copyed.
With this flag set, any runs of zeroes in the source before the
On 06/03/2016 05:16 PM, Mihai Claudiu Caraman wrote:
-Original Message-
From: Peter Maydell [mailto:peter.mayd...@linaro.org]
Sent: Friday, June 03, 2016 5:38 PM
To: Mihai Claudiu Caraman
Cc: Bogdan Purcareata ; QEMU Developers
On Sat, May 28, 2016 at 21:15:06 +0300, Sergey Fedorov wrote:
> On 25/05/16 04:13, Emilio G. Cota wrote:
> (snip)
> > +double qdist_avg(const struct qdist *dist)
> > +{
> > +unsigned long count;
> > +size_t i;
> > +double ret = 0;
> > +
> > +count = qdist_sample_count(dist);
> > +
On Fri, Jun 03, 2016 at 11:50:14AM +0800, Xiao Guangrong wrote:
>
>
> On 05/23/2016 09:46 PM, Paolo Bonzini wrote:
> >
> >
> > On 20/05/2016 23:39, Eduardo Habkost wrote:
> > > Introduce Skylake-Client CPU model, which inherits the features
> > > from Broadwell and supports some additional
On 25 May 2016 at 11:58, xiaoqiang zhao wrote:
> This patch series's main idea is to drop the use of
> qemu_char_get_next_serial. patches 1-5 QOM'ify the related
> device model and the affected board code. Most of them
> are arm related. The patch 6 get rid of the function
From: Aleksandar Markovic
fpu/softfloat-specialize.h is the most critical file in SoftFloat
library, since it handles numerous differences between platforms in
relation to floating point arithmetics. This patch makes the code
in this file more consistent
From: Aleksandar Markovic
New set of helpers for handling nan2008-syle versions of instructions
.., for Mips R6.
All involved instructions have float operand and integer result. Their
core functionality is implemented via
Sergey Fedorov writes:
> On 15/04/16 17:23, Alex Bennée wrote:
>> diff --git a/include/qemu/atomic.h b/include/qemu/atomic.h
>> index 5dba7db..94e7110 100644
>> --- a/include/qemu/atomic.h
>> +++ b/include/qemu/atomic.h
>> @@ -123,6 +123,16 @@
>> _old;
Am 03.06.2016 um 20:20 hat cl...@redhat.com geschrieben:
> From: Colin Lord
>
> Returns negative error codes and accompanying error messages in cases where
> the device has no tray or the tray is locked and isn't forced open. This
> extra information should result in better
On 06/03/2016 11:21 AM, Kevin Wolf wrote:
> Reading from qcow2 images is now byte granularity.
>
> Most of the affected code in qcow2 actually gets simpler with this
> change. The only exception is encryption, which is fixed on 512 bytes
> blocks; in order to keep this working,
On 06/03/2016 01:12 PM, Alex Bennée wrote:
>>> +#define atomic_bool_cmpxchg(ptr, old, new) \
>>> +({ \
>>> +typeof(*ptr) _old = (old), _new = (new);\
>>> +bool r;
On Fri, Jun 03, 2016 at 12:13:18PM +0200, Igor Mammedov wrote:
> On Thu, 2 Jun 2016 14:34:27 -0300
> Eduardo Habkost wrote:
>
> > On Thu, Jun 02, 2016 at 06:56:55PM +0200, Igor Mammedov wrote:
> > > On Thu, 2 Jun 2016 11:38:26 -0300
> > > Eduardo Habkost
On 03/06/16 21:30, Pranith Kumar wrote:
> On Thu, Jun 2, 2016 at 9:08 PM, Richard Henderson wrote:
>> On 06/02/2016 02:37 PM, Sergey Fedorov wrote:
>>>
>>> It would give us three TCG operations for each memory operation instead
>>> of one. But then we might like to combine these
In preparation for implementing .bdrv_co_pwritev in qcow2.
Signed-off-by: Kevin Wolf
---
block/qcow2-cluster.c | 32
block/qcow2.h | 13 +++--
2 files changed, 15 insertions(+), 30 deletions(-)
diff --git
On 06/03/2016 11:03 AM, Eric Blake wrote:
> It makes more sense to have ALL block size limit constraints
> in the same struct. Improve the documentation while at it.
>
> Note that bdrv_refresh_limits() has to keep things alive across
> a memset() of BlockLimits.
>
> Signed-off-by: Eric Blake
On 26 May 2016 at 15:31, Cole Robinson wrote:
> If you try to gic-version=host with TCG on a KVM aarch64 host,
> qemu segfaults, since host requires KVM APIs.
>
> Explicitly reject gic-version=host if KVM is not enabled
>
> https://bugzilla.redhat.com/show_bug.cgi?id=1339977
From: Aleksandar Markovic
This patch modifies SoftFloat library so that it can be configured in
run-time in relation to the meaning of signaling NaN bit, while, at the
same time, strictly preserving its behavior on all existing platforms.
Background:
In
From: Aleksandar Markovic
Only for Mips platform, and only for cases when snan_bit_is_one is 0,
correct the order of argument comparisons in pickNaNMulAdd().
For more info, see [1], page 53, section "3.5.3 NaN Propagation".
[1] "MIPS Architecture for Programmers
On 06/03/2016 11:21 AM, Kevin Wolf wrote:
> This will allow copy on write operations where the overwritten part of
> the cluster is not aligned to sector boundaries.
>
> Signed-off-by: Kevin Wolf
> ---
> block/qcow2-cluster.c | 37 +
> 1
Returns negative error codes and accompanying error messages in cases where
the device has no tray or the tray is locked and isn't forced open. This
extra information should result in better flexibility in functions that
call do_open_tray.
Signed-off-by: Colin Lord
---
v2: fix
This changes qcow2 to implement the byte-based .bdrv_co_pwritev
interface rather than the sector-based old one.
As preallocation uses the same allocation function as normal writes, and
the interface of that function needs to be changed, it is converted in
the same patch.
Signed-off-by: Kevin
This patch changes the units that qcow2_get_cluster_offset() uses
internally, without touching the interface just yet. This will be done
in another patch.
Signed-off-by: Kevin Wolf
---
block/qcow2-cluster.c | 43 ++-
1 file changed, 22
On 05/31/2016 12:36 PM, Laurent Vivier wrote:
> Replace (((n) + (d) - 1) /(d)) by DIV_ROUND_UP(n,d).
>
> This patch is the result of coccinelle script
> scripts/coccinelle/round.cocci
>
> CC: Paolo Bonzini
> Signed-off-by: Laurent Vivier
> ---
>
Address size is 40-bit for the AArch32 stage 2 translation,
and t0sz can be negative (from -8 to 7),
so we need to adjust it to use the existing TTBR selecting logic.
Signed-off-by: Sergey Sorokin
---
target-arm/helper.c | 38 ++
1 file
This series converts qcow2 to the byte-based I/O interfaces. This simplifies
the code by removing many unit conversions, and in the unlikely case of actual
unaligned requests, it even makes the driver work more efficiently by avoiding
read-modify-write.
Kevin Wolf (5):
qcow2: Work with bytes in
On Fri, Jun 03, 2016 at 09:23:28AM +0800, Fam Zheng wrote:
> On Thu, 06/02 16:13, Eduardo Habkost wrote:
> > On Wed, Jun 01, 2016 at 05:45:12PM +0800, Fam Zheng wrote:
> > [...]
> > > +
> > > +CUR_TIME := $(shell date +%Y-%m-%d-%H.%M.%S.)
> > > +# Makes the definition constant after the first
On 03/06/16 20:29, Sergey Fedorov wrote:
> On 03/06/16 20:22, Emilio G. Cota wrote:
>> On Sat, May 28, 2016 at 21:15:06 +0300, Sergey Fedorov wrote:
>>> On 25/05/16 04:13, Emilio G. Cota wrote:
>>> (snip)
+double qdist_avg(const struct qdist *dist)
+{
+unsigned long count;
From: Aleksandar Markovic
This patch modifies SoftFloat library so that it can be configured in
run-time in relation to the meaning of signaling NaN bit, while, at the
same time, strictly preserving its behavior on all existing platforms.
Background:
In
From: Aleksandar Markovic
v7 - preprocesor constants for FCR31 bits removed
- cleanup items in CFC1 and CTC1 handling removed
- initialization of snan_bit_is_one for main FP unit moved
from patch 6 to patch 9
- an instance of "!" corrected to "~"
From: Aleksandar Markovic
v8 - fixed mail recipients
v7 - preprocesor constants for FCR31 bits removed
- cleanup items in CFC1 and CTC1 handling removed
- initialization of snan_bit_is_one for main FP unit moved
from patch 6 to patch 9
- an instance
On 06/03/2016 11:21 AM, Kevin Wolf wrote:
> In preparation for implementing .bdrv_co_pwritev in qcow2.
>
> Signed-off-by: Kevin Wolf
> ---
> block/qcow2-cluster.c | 32
> block/qcow2.h | 13 +++--
> 2 files changed, 15
On 06/03/2016 11:21 AM, Kevin Wolf wrote:
> This patch changes the units that qcow2_get_cluster_offset() uses
> internally, without touching the interface just yet. This will be done
> in another patch.
>
> Signed-off-by: Kevin Wolf
> ---
> block/qcow2-cluster.c | 43
On 06/03/2016 05:47 PM, Mark Cave-Ayland wrote:
> On 03/06/16 15:14, Mark Cave-Ayland wrote:
>
>> On 03/06/16 15:06, Cedric Le Goater wrote:
>>
>>> On 06/03/2016 04:00 PM, Cédric Le Goater wrote:
Hello Mark,
On 06/03/2016 03:52 PM, Mark Cave-Ayland wrote:
> On 03/06/16 13:11,
On 31 May 2016 at 15:18, Cédric Le Goater wrote:
> The Aspeed AST2400 integrates a set of 14 I2C/SMBus bus controllers
> directly connected to the APB bus. They can be programmed as master or
> slave but the propopsed model only supports the master mode.
>
> On the TODO list, we
On 25 May 2016 at 11:52, Edgar E. Iglesias wrote:
> From: "Edgar E. Iglesias"
>
> This series fixes a bunch of issues with KVM runs using the ZynqMP
> machine.
>
> When using KVM, we don't have support for TrustZone, the RPU subsystem
> with
From: Aleksandar Markovic
Missing values EF_MIPS_FP64 and EF_MIPS_NAN2008 added, and the format
of the surrounding code segment adjusted.
Signed-off-by: Thomas Schwinge
Signed-off-by: Maciej W. Rozycki
From: Aleksandar Markovic
Only for Mips platform, and only for cases when snan_bit_is_one is 0,
correct default NaN values (in their 16-, 32-, and 64-bit flavors).
For more info, see [1], page 84, Table 6.3 "Value Supplied When a New
Quiet NaN Is Created", and
This will allow copy on write operations where the overwritten part of
the cluster is not aligned to sector boundaries.
Signed-off-by: Kevin Wolf
---
block/qcow2-cluster.c | 37 +
1 file changed, 17 insertions(+), 20 deletions(-)
diff --git
Hi T,
i can confirm the patch is working . sdl2 switch screen is fixed .
i had been tested it on 2.5.1 and is ok.
I think will be better add this in the qemu git
next step sdl gl ;-)
Many thanks
Luigi
--
You received this bug notification because you are a member of qemu-
devel-ml, which is
On Thu, Jun 2, 2016 at 9:08 PM, Richard Henderson wrote:
> On 06/02/2016 02:37 PM, Sergey Fedorov wrote:
>>
>>
>> It would give us three TCG operations for each memory operation instead
>> of one. But then we might like to combine these barrier operations back
>> with memory
From: Aleksandar Markovic
This patch implements read and write access rules for Mips floating
point control and status register (FCR31). The change can be divided
into following parts:
- Add fields that will keep FCR31's R/W bitmask in procesor
definitions and
On Fri, Jun 03, 2016 at 11:37:49AM +0200, Igor Mammedov wrote:
> On Fri, 3 Jun 2016 09:30:09 +0200
> Peter Krempa wrote:
>
> > On Thu, Jun 02, 2016 at 18:31:04 +0200, Igor Mammedov wrote:
> > > On Thu, 2 Jun 2016 17:05:06 +0200
> > > Peter Krempa wrote:
There are a number of changes that occur at the same time here:
- tb_lock is no longer a NOP for SoftMMU
The tb_lock protects both translation and memory map structures. The
debug assert is updated to reflect this.
- introduce a single vCPU qemu_tcg_cpu_thread_fn
One of these is
This is a current DRAFT of a design proposal for upgrading TCG emulation
to take advantage of modern CPUs by running a thread-per-CPU. The
document goes through the various areas of the code affected by such a
change and proposes design requirements for each part of the solution.
It has been
Testing with Alexander's bare metal syncronisation tests fails in MTTCG
leaving one CPU spinning forever waiting for the second CPU to wake up.
We simply need to kick the vCPU once we have processed the PSCI power on
call.
As the power control API is for system emulation only as is the
On 3 June 2016 at 20:49, Sergey Fedorov wrote:
> I'm not so sure about acq/rel flags. Is there any architecture which has
> explicit acq/rel barriers? I suppose acq/rel memory access instructions
> are always load-link and store-conditional and thus rely on exclusive
>
From: KONRAD Frederic
We know there will be cases where MTTCG won't work until additional work
is done in the front/back ends to support. It will however be useful to
be able to turn it on.
As a result MTTCG will default to off unless the combination is
supported.
From: KONRAD Frederic
This protects TBContext with tb_lock to make tb_* thread safe.
We can still have issue with tb_flush in case of multithread TCG:
another CPU can be executing code during a flush.
This can be fixed later by making all other TCG thread exiting
Lock contention in the hot path of moving between existing patched
TranslationBlocks is the main drag on MTTCG performance. This patch
pushes the tb_lock() usage down to the two places that really need it:
- code generation (tb_gen_code)
- jump patching (tb_add_jump)
The rest of the code
On 06/03/2016 11:49 AM, Eric Blake wrote:
> On 06/03/2016 11:03 AM, Eric Blake wrote:
>> It makes more sense to have ALL block size limit constraints
>> in the same struct. Improve the documentation while at it.
>>
>> Note that bdrv_refresh_limits() has to keep things alive across
>> a memset()
This finally allows TCG to benefit from the iothread introduction: Drop
the global mutex while running pure TCG CPU code. Reacquire the lock
when entering MMIO or PIO emulation, or when leaving the TCG loop.
We have to revert a few optimization for the current TCG threading
model, namely kicking
The only remaining use of the global exit_request flag is now to ensure
we exit the run_loop when we first start to process pending work. This
is just as easily done by setting the first_cpu->exit_request flag.
We lightly re-factor the main vCPU thread to ensure cpu->exit_requests
cause us to
Am 03.06.2016 um 21:39 hat Colin Lord geschrieben:
> Returns negative error codes and accompanying error messages in cases where
> the device has no tray or the tray is locked and isn't forced open. This
> extra information should result in better flexibility in functions that
> call do_open_tray.
..and make the definition local to cpus. In preparation for MTTCG the
concept of a global tcg_current_cpu will no longer make sense. However
we still need to keep track of it in the single-threaded case to be able
to exit quickly when required.
qemu_cpu_kick_no_halt() moves and becomes
From: Sergey Fedorov
First, ensure atomicity of CPU's 'tb_jmp_cache' access by:
* using atomic_read() to look up a TB when not holding 'tb_lock';
* using atomic_write() to remove a TB from each CPU's local cache on
TB invalidation.
Second, add some memory barriers to
While we previously assumed an existing memory lock protected the page
look up in the MTTCG SoftMMU case the memory lock is provided by the
tb_lock. As a result we push the taking of this lock up the call tree.
This requires a slightly different entry for the SoftMMU and user-mode
cases from
On 03/06/16 21:27, Pranith Kumar wrote:
> On Thu, Jun 2, 2016 at 5:18 PM, Richard Henderson wrote:
>> >
>> > Hum. That does seem helpful-ish. But I'm not certain how helpful it is to
>> > complicate the helper functions even further.
>> >
>> > What if we have
On Fri, Jun 03, 2016 at 08:36:21AM +0200, David Hildenbrand wrote:
> > On Thu, Jun 02, 2016 at 10:44:49PM +0200, David Hildenbrand wrote:
> > > > Current CLI option -cpu cpux,features serves as template
> > > > for all created cpus of type: cpux. However QEMU parses
> > > > "features" every time
This adds asserts to check the locking on the various translation
engines structures. There are two sets of structures that are protected
by locks.
The first the l1map and PageDesc structures used to track which
translation blocks are associated with which physical addresses. In
user-mode this is
Currently we rely on the side effect of the main loop grabbing the
iothread_mutex to give any long running basic block chains a kick to
ensure the next vCPU is scheduled. As this code is being re-factored and
rationalised we now do it explicitly here.
Signed-off-by: Alex Bennée
The debug structures are different from many of the other shared ones as
they are modified from two places:
- architectural debug registers
- the gdb stub
The first is usually in the context of vCPU currently running so
modifications are safe. The second is generally only changed while the
From: Paolo Bonzini
softmmu requires more functions to be thread-safe, because translation
blocks can be invalidated from e.g. notdirty callbacks. Probably the
same holds for user-mode emulation, it's just that no one has ever
tried to produce a coherent locking there.
On 03/06/16 23:43, Peter Maydell wrote:
> On 3 June 2016 at 20:49, Sergey Fedorov wrote:
>> I'm not so sure about acq/rel flags. Is there any architecture which has
>> explicit acq/rel barriers? I suppose acq/rel memory access instructions
>> are always load-link and
On 06/03/2016 11:21 AM, Kevin Wolf wrote:
> This changes qcow2 to implement the byte-based .bdrv_co_pwritev
> interface rather than the sector-based old one.
>
> As preallocation uses the same allocation function as normal writes, and
> the interface of that function needs to be changed, it is
On 06/03/2016 01:39 PM, Colin Lord wrote:
> Returns negative error codes and accompanying error messages in cases where
> the device has no tray or the tray is locked and isn't forced open. This
> extra information should result in better flexibility in functions that
> call do_open_tray.
>
>
Make the debug define consistent with the others. The flush operation is
all about invalidating TranslationBlocks on flush events.
Also fix up the commenting on the other DEBUG for the benefit of
checkpatch.
Signed-off-by: Alex Bennée
---
translate-all.c | 8
1
This is the third iteration of the RFC patch set which aims to provide
the basic framework for MTTCG. There have been some considerable
changes since the last extensive review (thanks to all the reviewers).
- many updates to docs/multi-thread-tcg.txt design document
- added assertions for
Signed-off-by: Alex Bennée
Reviewed-by: Sergey Fedorov
---
v3
- add r-b tags
---
cpus.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/cpus.c b/cpus.c
index 665f9bb..c404dd7 100644
--- a/cpus.c
+++ b/cpus.c
@@
This adds calls to the assert_memory_lock for all public APIs which are
documented as holding the mmap_lock for user-mode.
Signed-off-by: Alex Bennée
---
linux-user/elfload.c | 4
translate-all.c | 20
2 files changed, 24 insertions(+)
1 - 100 of 321 matches
Mail list logo