From: <tony.nguyen@bt.com>
To: <qemu-devel@nongnu.org>
Cc: fam@euphon.net, peter.maydell@linaro.org, walling@linux.ibm.com,
cohuck@redhat.com, sagark@eecs.berkeley.edu, david@redhat.com,
jasowang@redhat.com, palmer@sifive.com,
mark.cave-ayland@ilande.co.uk, i.mitsyanko@gmail.com,
keith.busch@intel.com, jcmvbkbc@gmail.com,
frederic.konrad@adacore.com, dmitry.fleytman@gmail.com,
kraxel@redhat.com, edgar.iglesias@gmail.com, gxt@mprc.pku.edu.cn,
pburton@wavecomp.com, xiaoguangrong.eric@gmail.com,
peter.chubb@nicta.com.au, philmd@redhat.com, robh@kernel.org,
hare@suse.com, sstabellini@kernel.org, berto@igalia.com,
chouteau@adacore.com, qemu-block@nongnu.org,
arikalo@wavecomp.com, jslaby@suse.cz, deller@gmx.de,
mst@redhat.com, magnus.damm@gmail.com, jcd@tribudubois.net,
pasic@linux.ibm.com, borntraeger@de.ibm.com, mreitz@redhat.com,
hpoussin@reactos.org, joel@jms.id.au, anthony.perard@citrix.com,
xen-devel@lists.xenproject.org, david@gibson.dropbear.id.au,
lersek@redhat.com, green@moxielogic.com, atar4qemu@gmail.com,
antonynpavlov@gmail.com, marex@denx.de, jiri@resnulli.us,
ehabkost@redhat.com, minyard@acm.org, qemu-s390x@nongnu.org,
sw@weilnetz.de, alistair@alistair23.me, yuval.shaia@oracle.com,
b.galvani@gmail.com, eric.auger@redhat.com,
alex.williamson@redhat.com, qemu-arm@nongnu.org,
jan.kiszka@web.de, clg@kaod.org, stefanha@redhat.com,
marcandre.lureau@redhat.com, shorne@gmail.com, jsnow@redhat.com,
rth@twiddle.net, kwolf@redhat.com, qemu-riscv@nongnu.org,
proljc@gmail.com, pbonzini@redhat.com, andrew@aj.id.au,
kbastian@mail.uni-paderborn.de, crwulff@gmail.com,
laurent@vivier.eu, Andrew.Baumann@microsoft.com,
sundeep.lkml@gmail.com, andrew.smirnov@gmail.com,
michael@walle.cc, paul.durrant@citrix.com, qemu-ppc@nongnu.org,
huth@tuxfamily.org, amarkovic@wavecomp.com, imammedo@redhat.com,
aurelien@aurel32.net, stefanb@linux.ibm.com
Subject: [Qemu-devel] [PATCH v6 25/26] target/sparc: Add TLB entry with attributes
Date: Wed, 7 Aug 2019 08:35:12 +0000 [thread overview]
Message-ID: <1565166911751.16655@bt.com> (raw)
In-Reply-To: <45ec4924e0b34a3d9124e2db06af75b4@tpw09926dag18e.domain1.systemhost.net>
Append MemTxAttrs to interfaces so we can pass along up coming Invert
Endian TTE bit on SPARC64.
Signed-off-by: Tony Nguyen <tony.nguyen@bt.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
---
target/sparc/mmu_helper.c | 32 ++++++++++++++++++--------------
1 file changed, 18 insertions(+), 14 deletions(-)
diff --git a/target/sparc/mmu_helper.c b/target/sparc/mmu_helper.c
index cbd1e91..826e14b 100644
--- a/target/sparc/mmu_helper.c
+++ b/target/sparc/mmu_helper.c
@@ -88,7 +88,7 @@ static const int perm_table[2][8] = {
};
static int get_physical_address(CPUSPARCState *env, hwaddr *physical,
- int *prot, int *access_index,
+ int *prot, int *access_index, MemTxAttrs *attrs,
target_ulong address, int rw, int mmu_idx,
target_ulong *page_size)
{
@@ -219,6 +219,7 @@ bool sparc_cpu_tlb_fill(CPUState *cs, vaddr address, int size,
target_ulong vaddr;
target_ulong page_size;
int error_code = 0, prot, access_index;
+ MemTxAttrs attrs = {};
/*
* TODO: If we ever need tlb_vaddr_to_host for this target,
@@ -229,7 +230,7 @@ bool sparc_cpu_tlb_fill(CPUState *cs, vaddr address, int size,
assert(!probe);
address &= TARGET_PAGE_MASK;
- error_code = get_physical_address(env, &paddr, &prot, &access_index,
+ error_code = get_physical_address(env, &paddr, &prot, &access_index, &attrs,
address, access_type,
mmu_idx, &page_size);
vaddr = address;
@@ -490,8 +491,8 @@ static inline int ultrasparc_tag_match(SparcTLBEntry *tlb,
return 0;
}
-static int get_physical_address_data(CPUSPARCState *env,
- hwaddr *physical, int *prot,
+static int get_physical_address_data(CPUSPARCState *env, hwaddr *physical,
+ int *prot, MemTxAttrs *attrs,
target_ulong address, int rw, int mmu_idx)
{
CPUState *cs = env_cpu(env);
@@ -608,8 +609,8 @@ static int get_physical_address_data(CPUSPARCState *env,
return 1;
}
-static int get_physical_address_code(CPUSPARCState *env,
- hwaddr *physical, int *prot,
+static int get_physical_address_code(CPUSPARCState *env, hwaddr *physical,
+ int *prot, MemTxAttrs *attrs,
target_ulong address, int mmu_idx)
{
CPUState *cs = env_cpu(env);
@@ -686,7 +687,7 @@ static int get_physical_address_code(CPUSPARCState *env,
}
static int get_physical_address(CPUSPARCState *env, hwaddr *physical,
- int *prot, int *access_index,
+ int *prot, int *access_index, MemTxAttrs *attrs,
target_ulong address, int rw, int mmu_idx,
target_ulong *page_size)
{
@@ -716,11 +717,11 @@ static int get_physical_address(CPUSPARCState *env, hwaddr *physical,
}
if (rw == 2) {
- return get_physical_address_code(env, physical, prot, address,
+ return get_physical_address_code(env, physical, prot, attrs, address,
mmu_idx);
} else {
- return get_physical_address_data(env, physical, prot, address, rw,
- mmu_idx);
+ return get_physical_address_data(env, physical, prot, attrs, address,
+ rw, mmu_idx);
}
}
@@ -734,10 +735,11 @@ bool sparc_cpu_tlb_fill(CPUState *cs, vaddr address, int size,
target_ulong vaddr;
hwaddr paddr;
target_ulong page_size;
+ MemTxAttrs attrs = {};
int error_code = 0, prot, access_index;
address &= TARGET_PAGE_MASK;
- error_code = get_physical_address(env, &paddr, &prot, &access_index,
+ error_code = get_physical_address(env, &paddr, &prot, &access_index, &attrs,
address, access_type,
mmu_idx, &page_size);
if (likely(error_code == 0)) {
@@ -747,7 +749,8 @@ bool sparc_cpu_tlb_fill(CPUState *cs, vaddr address, int size,
env->dmmu.mmu_primary_context,
env->dmmu.mmu_secondary_context);
- tlb_set_page(cs, vaddr, paddr, prot, mmu_idx, page_size);
+ tlb_set_page_with_attrs(cs, vaddr, paddr, attrs, prot, mmu_idx,
+ page_size);
return true;
}
if (probe) {
@@ -849,9 +852,10 @@ static int cpu_sparc_get_phys_page(CPUSPARCState *env, hwaddr *phys,
{
target_ulong page_size;
int prot, access_index;
+ MemTxAttrs attrs = {};
- return get_physical_address(env, phys, &prot, &access_index, addr, rw,
- mmu_idx, &page_size);
+ return get_physical_address(env, phys, &prot, &access_index, &attrs, addr,
+ rw, mmu_idx, &page_size);
}
#if defined(TARGET_SPARC64)
--
1.8.3.1
?
next prev parent reply other threads:[~2019-08-07 8:37 UTC|newest]
Thread overview: 61+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-07 8:23 [Qemu-devel] [PATCH v6 00/26] Invert Endian bit in SPARCv9 MMU TTE tony.nguyen
2019-08-07 8:25 ` [Qemu-devel] [PATCH v6 01/26] configure: Define TARGET_ALIGNED_ONLY tony.nguyen
2019-08-08 14:08 ` Cornelia Huck
2019-08-07 8:26 ` [Qemu-devel] [PATCH v6 02/26] tcg: TCGMemOp is now accelerator independent MemOp tony.nguyen
2019-08-08 14:14 ` Cornelia Huck
2019-08-07 8:26 ` [Qemu-devel] [PATCH v6 03/26] memory: Introduce size_memop tony.nguyen
2019-08-07 15:30 ` Richard Henderson
2019-08-07 8:27 ` [Qemu-devel] [PATCH v6 04/26] target/mips: Access MemoryRegion with MemOp tony.nguyen
2019-08-07 12:50 ` tony.nguyen
2019-08-07 8:27 ` [Qemu-devel] [PATCH v6 05/26] hw/s390x: " tony.nguyen
2019-08-08 14:31 ` Cornelia Huck
2019-08-07 8:27 ` [Qemu-devel] [PATCH v6 06/26] hw/intc/armv7m_nic: " tony.nguyen
2019-08-07 12:54 ` tony.nguyen
2019-08-07 8:28 ` [Qemu-devel] [PATCH v6 07/26] hw/virtio: " tony.nguyen
2019-08-08 14:32 ` Cornelia Huck
2019-08-07 8:28 ` [Qemu-devel] [PATCH v6 08/26] hw/vfio: " tony.nguyen
2019-08-08 14:34 ` Cornelia Huck
2019-08-07 8:28 ` [Qemu-devel] [PATCH v6 09/26] exec: " tony.nguyen
2019-08-07 12:56 ` tony.nguyen
2019-08-07 8:29 ` [Qemu-devel] [PATCH v6 10/26] cputlb: " tony.nguyen
2019-08-07 12:45 ` tony.nguyen
2019-08-07 15:32 ` Richard Henderson
2019-08-07 8:29 ` [Qemu-devel] [PATCH v6 11/26] memory: " tony.nguyen
2019-08-07 15:38 ` Richard Henderson
2019-08-07 8:30 ` [Qemu-devel] [PATCH v6 12/26] hw/s390x: Hard code size with MO_{8|16|32|64} tony.nguyen
2019-08-07 15:47 ` Richard Henderson
2019-08-08 14:44 ` Cornelia Huck
2019-08-07 8:30 ` [Qemu-devel] [PATCH v6 13/26] target/mips: " tony.nguyen
2019-08-07 15:47 ` Richard Henderson
2019-08-07 8:30 ` [Qemu-devel] [PATCH v6 14/26] exec: " tony.nguyen
2019-08-07 15:48 ` Richard Henderson
2019-08-07 8:31 ` [Qemu-devel] [PATCH v6 15/26] build: Correct non-common common-obj-* to obj-* tony.nguyen
2019-08-07 10:42 ` Paolo Bonzini
2019-08-07 8:31 ` [Qemu-devel] [PATCH v6 16/26] exec: Map device_endian onto MemOp tony.nguyen
2019-08-07 15:55 ` Richard Henderson
2019-08-07 15:59 ` Richard Henderson
2019-08-07 16:06 ` Richard Henderson
2019-08-07 8:31 ` [Qemu-devel] [PATCH v6 17/26] exec: Replace device_endian with MemOp tony.nguyen
2019-08-07 16:23 ` Richard Henderson
2019-08-09 0:35 ` David Gibson
2019-08-07 8:32 ` [Qemu-devel] [PATCH v6 18/26] exec: Delete device_endian tony.nguyen
2019-08-07 16:23 ` Richard Henderson
2019-08-07 8:32 ` [Qemu-devel] [PATCH v6 19/26] exec: Delete DEVICE_HOST_ENDIAN tony.nguyen
2019-08-07 10:22 ` Paolo Bonzini
2019-08-07 15:03 ` Richard Henderson
2019-08-07 8:33 ` [Qemu-devel] [PATCH v6 20/26] memory: Access MemoryRegion with endianness tony.nguyen
2019-08-07 10:27 ` Paolo Bonzini
2019-08-07 17:49 ` Richard Henderson
2019-08-07 18:00 ` Paolo Bonzini
2019-08-07 18:23 ` Richard Henderson
2019-08-07 8:33 ` [Qemu-devel] [PATCH v6 21/26] cputlb: Replace size and endian operands for MemOp tony.nguyen
2019-08-07 17:38 ` Richard Henderson
2019-08-07 8:34 ` [Qemu-devel] [PATCH v6 22/26] memory: Single byte swap along the I/O path tony.nguyen
2019-08-07 8:34 ` [Qemu-devel] [PATCH v6 23/26] cpu: TLB_FLAGS_MASK bit to force memory slow path tony.nguyen
2019-08-07 8:34 ` [Qemu-devel] [PATCH v6 24/26] cputlb: Byte swap memory transaction attribute tony.nguyen
2019-08-07 8:35 ` tony.nguyen [this message]
2019-08-07 8:35 ` [Qemu-devel] [PATCH v6 26/26] target/sparc: sun4u Invert Endian TTE bit tony.nguyen
2019-08-07 13:04 ` tony.nguyen
2019-08-07 10:37 ` [Qemu-devel] [PATCH v6 00/26] Invert Endian bit in SPARCv9 MMU TTE Philippe Mathieu-Daudé
2019-08-07 12:41 ` tony.nguyen
2019-08-07 12:54 ` 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=1565166911751.16655@bt.com \
--to=tony.nguyen@bt.com \
--cc=Andrew.Baumann@microsoft.com \
--cc=alex.williamson@redhat.com \
--cc=alistair@alistair23.me \
--cc=amarkovic@wavecomp.com \
--cc=andrew.smirnov@gmail.com \
--cc=andrew@aj.id.au \
--cc=anthony.perard@citrix.com \
--cc=antonynpavlov@gmail.com \
--cc=arikalo@wavecomp.com \
--cc=atar4qemu@gmail.com \
--cc=aurelien@aurel32.net \
--cc=b.galvani@gmail.com \
--cc=berto@igalia.com \
--cc=borntraeger@de.ibm.com \
--cc=chouteau@adacore.com \
--cc=clg@kaod.org \
--cc=cohuck@redhat.com \
--cc=crwulff@gmail.com \
--cc=david@gibson.dropbear.id.au \
--cc=david@redhat.com \
--cc=deller@gmx.de \
--cc=dmitry.fleytman@gmail.com \
--cc=edgar.iglesias@gmail.com \
--cc=ehabkost@redhat.com \
--cc=eric.auger@redhat.com \
--cc=fam@euphon.net \
--cc=frederic.konrad@adacore.com \
--cc=green@moxielogic.com \
--cc=gxt@mprc.pku.edu.cn \
--cc=hare@suse.com \
--cc=hpoussin@reactos.org \
--cc=huth@tuxfamily.org \
--cc=i.mitsyanko@gmail.com \
--cc=imammedo@redhat.com \
--cc=jan.kiszka@web.de \
--cc=jasowang@redhat.com \
--cc=jcd@tribudubois.net \
--cc=jcmvbkbc@gmail.com \
--cc=jiri@resnulli.us \
--cc=joel@jms.id.au \
--cc=jslaby@suse.cz \
--cc=jsnow@redhat.com \
--cc=kbastian@mail.uni-paderborn.de \
--cc=keith.busch@intel.com \
--cc=kraxel@redhat.com \
--cc=kwolf@redhat.com \
--cc=laurent@vivier.eu \
--cc=lersek@redhat.com \
--cc=magnus.damm@gmail.com \
--cc=marcandre.lureau@redhat.com \
--cc=marex@denx.de \
--cc=mark.cave-ayland@ilande.co.uk \
--cc=michael@walle.cc \
--cc=minyard@acm.org \
--cc=mreitz@redhat.com \
--cc=mst@redhat.com \
--cc=palmer@sifive.com \
--cc=pasic@linux.ibm.com \
--cc=paul.durrant@citrix.com \
--cc=pbonzini@redhat.com \
--cc=pburton@wavecomp.com \
--cc=peter.chubb@nicta.com.au \
--cc=peter.maydell@linaro.org \
--cc=philmd@redhat.com \
--cc=proljc@gmail.com \
--cc=qemu-arm@nongnu.org \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=qemu-riscv@nongnu.org \
--cc=qemu-s390x@nongnu.org \
--cc=robh@kernel.org \
--cc=rth@twiddle.net \
--cc=sagark@eecs.berkeley.edu \
--cc=shorne@gmail.com \
--cc=sstabellini@kernel.org \
--cc=stefanb@linux.ibm.com \
--cc=stefanha@redhat.com \
--cc=sundeep.lkml@gmail.com \
--cc=sw@weilnetz.de \
--cc=walling@linux.ibm.com \
--cc=xen-devel@lists.xenproject.org \
--cc=xiaoguangrong.eric@gmail.com \
--cc=yuval.shaia@oracle.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).