qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Philippe Mathieu-Daudé" <f4bug@amsat.org>
To: qemu-devel@nongnu.org, Paolo Bonzini <pbonzini@redhat.com>
Cc: "Laurent Vivier" <lvivier@redhat.com>,
	"Peter Maydell" <peter.maydell@linaro.org>,
	"Thomas Huth" <thuth@redhat.com>,
	"Stephen Checkoway" <stephen.checkoway@oberlin.edu>,
	"Mark Cave-Ayland" <mark.cave-ayland@ilande.co.uk>,
	"Markus Armbruster" <armbru@redhat.com>,
	"Eduardo Habkost" <ehabkost@redhat.com>,
	"Hervé Poussineau" <hpoussin@reactos.org>,
	"Edgar E . Iglesias" <edgar.iglesias@gmail.com>,
	"Artyom Tarasenko" <atar4qemu@gmail.com>,
	"Richard Henderson" <rth@twiddle.net>
Subject: Re: [RFC PATCH 0/9] hw/misc: Add support for interleaved memory accesses
Date: Sat, 22 Aug 2020 17:15:59 +0200	[thread overview]
Message-ID: <c21cd155-31e2-5911-f5c3-7362e0819f75@amsat.org> (raw)
In-Reply-To: <20200817161853.593247-1-f4bug@amsat.org>

On 8/17/20 6:18 PM, Philippe Mathieu-Daudé wrote:
> This is a follow up on last year patch from Stephen Checkoway:
> "block/pflash_cfi02: Implement intereleaved flash devices"
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg613077.html
> 
> Instead of making the PFLASH device more complex, I went
> implementing a generic interleaver device (that can be re-used
> for other MMIO devices, probably pointless, still useful for
> simple testing).
> 
> The series is available on [1] and is organized as:
> 
> 1/ Fix a pair of issues in memory.c as we are not exercing much
> the memory_region_[read/write]_with_attrs_accessor functions (see
> also [2]).
> 
> 2/ Allow to use qtest helpers with the 'none' machine (without any
> CPU).
> 
> 3/ Add the interleaver device
> 
> 4/ Add a device to test the interleaver device (later I'll expand
> it to other MMIO accesses, such access_with_adjusted_size, endianess
> and unaligned accesses). Let the 'none' machine create this device
> from command line.
> 
> 5/ Add qtests using 4/ to test 3/.
> 
> 6/ Add a way to display the interleaved memory (sub)regions in
> the HMP 'info mtree' command.
> 
> Based-on: <20200816173051.16274-1-f4bug@amsat.org>
> [1] https://gitlab.com/philmd/qemu/-/tree/interleaved_device-v1
> [2] https://www.mail-archive.com/qemu-devel@nongnu.org/msg730279.html

I followed Paolo's suggestion to use qtest_memread/qtest_memwrite to
avoid endianess swapping, and now cross-endianess works :)

Review still welcome, but I'll respin in September.

Regards,

Phil.


  parent reply	other threads:[~2020-08-22 15:17 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-17 16:18 [RFC PATCH 0/9] hw/misc: Add support for interleaved memory accesses Philippe Mathieu-Daudé
2020-08-17 16:18 ` [RFC PATCH 1/9] memory: Initialize MemoryRegionOps for RAM memory regions Philippe Mathieu-Daudé
2020-08-18  9:52   ` P J P
2020-08-18  9:58     ` Philippe Mathieu-Daudé
2020-08-17 16:18 ` [RFC PATCH 2/9] qtest: Add local qtest_mem_as() getter Philippe Mathieu-Daudé
2020-08-17 16:18 ` [RFC PATCH 3/9] qtest: Directly use global address_space_memory when no CPU available Philippe Mathieu-Daudé
2020-08-17 16:18 ` [RFC PATCH 4/9] hw/misc: Add interleaver device to make interleaved memory accesses Philippe Mathieu-Daudé
2020-08-17 16:18 ` [RFC PATCH 5/9] hw/misc: Add MMIO test device Philippe Mathieu-Daudé
2020-08-17 16:18 ` [RFC PATCH 6/9] hw/core/null-machine: Allow to use the MMIO 'test' device Philippe Mathieu-Daudé
2020-08-17 16:18 ` [RFC PATCH 7/9] tests/qtest: Add generic MMIO tests Philippe Mathieu-Daudé
2020-08-17 16:18 ` [RFC PATCH 8/9] memory: Allow memory region to display its subregions own descriptions Philippe Mathieu-Daudé
2020-08-17 16:18 ` [RFC PATCH 9/9] hw/misc/interleaver: Display subregions in 'info mtree' Philippe Mathieu-Daudé
2020-08-17 20:13 ` [RFC PATCH 0/9] hw/misc: Add support for interleaved memory accesses no-reply
2020-08-22 15:15 ` Philippe Mathieu-Daudé [this message]
2020-08-22 15:17 ` Philippe Mathieu-Daudé

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=c21cd155-31e2-5911-f5c3-7362e0819f75@amsat.org \
    --to=f4bug@amsat.org \
    --cc=armbru@redhat.com \
    --cc=atar4qemu@gmail.com \
    --cc=edgar.iglesias@gmail.com \
    --cc=ehabkost@redhat.com \
    --cc=hpoussin@reactos.org \
    --cc=lvivier@redhat.com \
    --cc=mark.cave-ayland@ilande.co.uk \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    --cc=stephen.checkoway@oberlin.edu \
    --cc=thuth@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).