diff for duplicates of <1564123618147.19868@bt.com> diff --git a/a/2.bin b/a/2.bin deleted file mode 100644 index aa4ef45..0000000 --- a/a/2.bin +++ /dev/null @@ -1,108 +0,0 @@ -<html> -<head> -<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"> -<style type="text/css" style="display:none"><!-- P { margin-top: 0px; margin-bottom: 0px; } .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left-width: 2px; border-left-style: solid; border-left-color: rgb(128, 0, 0); }--></style> -</head> -<body dir="ltr" style="font-size:12pt;color:#000000;background-color:#FFFFFF;font-family:Calibri,Arial,Helvetica,sans-serif;"> -<p></p> -<div><span style="font-size: 12pt;">No-op MEMOP_SIZE and SIZE_MEMOP macros allows us to later easily</span><br> -</div> -<div>convert memory_region_dispatch_{read|write} paramter "unsigned size"</div> -<div>into a size+sign+endianness encoded "MemOp op".</div> -<div><br> -</div> -<div>Being a no-op macro, this patch does not introduce any logical change.</div> -<div><br> -</div> -<div>Signed-off-by: Tony Nguyen <tony.nguyen@bt.com></div> -<div>---</div> -<div> accel/tcg/cputlb.c | 21 ++++++++++-----------</div> -<div> 1 file changed, 10 insertions(+), 11 deletions(-)</div> -<div><br> -</div> -<div>diff --git a/accel/tcg/cputlb.c b/accel/tcg/cputlb.c</div> -<div>index 523be4c..5d88cec 100644</div> -<div>--- a/accel/tcg/cputlb.c</div> -<div>+++ b/accel/tcg/cputlb.c</div> -<div>@@ -881,7 +881,7 @@ static void tlb_fill(CPUState *cpu, target_ulong addr, int size,</div> -<div> </div> -<div> static uint64_t io_readx(CPUArchState *env, CPUIOTLBEntry *iotlbentry,</div> -<div> int mmu_idx, target_ulong addr, uintptr_t retaddr,</div> -<div>- MMUAccessType access_type, int size)</div> -<div>+ MMUAccessType access_type, MemOp op)</div> -<div> {</div> -<div> CPUState *cpu = env_cpu(env);</div> -<div> hwaddr mr_offset;</div> -<div>@@ -906,14 +906,13 @@ static uint64_t io_readx(CPUArchState *env, CPUIOTLBEntry *iotlbentry,</div> -<div> qemu_mutex_lock_iothread();</div> -<div> locked = true;</div> -<div> }</div> -<div>- r = memory_region_dispatch_read(mr, mr_offset,</div> -<div>- &val, size, iotlbentry->attrs);</div> -<div>+ r = memory_region_dispatch_read(mr, mr_offset, &val, op, iotlbentry->attrs);</div> -<div> if (r != MEMTX_OK) {</div> -<div> hwaddr physaddr = mr_offset +</div> -<div> section->offset_within_address_space -</div> -<div> section->offset_within_region;</div> -<div> </div> -<div>- cpu_transaction_failed(cpu, physaddr, addr, size, access_type,</div> -<div>+ cpu_transaction_failed(cpu, physaddr, addr, MEMOP_SIZE(op), access_type,</div> -<div> mmu_idx, iotlbentry->attrs, r, retaddr);</div> -<div> }</div> -<div> if (locked) {</div> -<div>@@ -925,7 +924,7 @@ static uint64_t io_readx(CPUArchState *env, CPUIOTLBEntry *iotlbentry,</div> -<div> </div> -<div> static void io_writex(CPUArchState *env, CPUIOTLBEntry *iotlbentry,</div> -<div> int mmu_idx, uint64_t val, target_ulong addr,</div> -<div>- uintptr_t retaddr, int size)</div> -<div>+ uintptr_t retaddr, MemOp op)</div> -<div> {</div> -<div> CPUState *cpu = env_cpu(env);</div> -<div> hwaddr mr_offset;</div> -<div>@@ -947,15 +946,15 @@ static void io_writex(CPUArchState *env, CPUIOTLBEntry *iotlbentry,</div> -<div> qemu_mutex_lock_iothread();</div> -<div> locked = true;</div> -<div> }</div> -<div>- r = memory_region_dispatch_write(mr, mr_offset,</div> -<div>- val, size, iotlbentry->attrs);</div> -<div>+ r = memory_region_dispatch_write(mr, mr_offset, val, op, iotlbentry->attrs);</div> -<div> if (r != MEMTX_OK) {</div> -<div> hwaddr physaddr = mr_offset +</div> -<div> section->offset_within_address_space -</div> -<div> section->offset_within_region;</div> -<div> </div> -<div>- cpu_transaction_failed(cpu, physaddr, addr, size, MMU_DATA_STORE,</div> -<div>- mmu_idx, iotlbentry->attrs, r, retaddr);</div> -<div>+ cpu_transaction_failed(cpu, physaddr, addr, MEMOP_SIZE(op),</div> -<div>+ MMU_DATA_STORE, mmu_idx, iotlbentry->attrs, r,</div> -<div>+ retaddr);</div> -<div> }</div> -<div> if (locked) {</div> -<div> qemu_mutex_unlock_iothread();</div> -<div>@@ -1306,7 +1305,7 @@ load_helper(CPUArchState *env, target_ulong addr, TCGMemOpIdx oi,</div> -<div> }</div> -<div> </div> -<div> res = io_readx(env, &env_tlb(env)->d[mmu_idx].iotlb[index],</div> -<div>- mmu_idx, addr, retaddr, access_type, size);</div> -<div>+ mmu_idx, addr, retaddr, access_type, SIZE_MEMOP(size));</div> -<div> return handle_bswap(res, size, big_endian);</div> -<div> }</div> -<div> </div> -<div>@@ -1555,7 +1554,7 @@ store_helper(CPUArchState *env, target_ulong addr, uint64_t val,</div> -<div> </div> -<div> io_writex(env, &env_tlb(env)->d[mmu_idx].iotlb[index], mmu_idx,</div> -<div> handle_bswap(val, size, big_endian),</div> -<div>- addr, retaddr, size);</div> -<div>+ addr, retaddr, SIZE_MEMOP(size));</div> -<div> return;</div> -<div> }</div> -<div> </div> -<div>-- </div> -<div>1.8.3.1</div> -<div><br> -<br> -</div> -<p><br> -</p> -</body> -</html> diff --git a/a/2.hdr b/a/2.hdr deleted file mode 100644 index e54d0ae..0000000 --- a/a/2.hdr +++ /dev/null @@ -1,2 +0,0 @@ -Content-Type: text/html; charset="iso-8859-1" -Content-Transfer-Encoding: quoted-printable diff --git a/a/content_digest b/N1/content_digest index 80ceb73..8563455 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,38 +1,38 @@ "ref\03106a3c959c4498fad13a5799c89ba7b@tpw09926dag18e.domain1.systemhost.net\0" "From\0<tony.nguyen@bt.com>\0" - "Subject\0[Qemu-riscv] [Qemu-devel] [PATCH v5 09/15] cputlb: Access MemoryRegion with MemOp\0" + "Subject\0[Qemu-devel] [PATCH v5 09/15] cputlb: Access MemoryRegion with MemOp\0" "Date\0Fri, 26 Jul 2019 06:46:58 +0000\0" "To\0<qemu-devel@nongnu.org>\0" - "Cc\0<peter.maydell@linaro.org>" - <walling@linux.ibm.com> - <sagark@eecs.berkeley.edu> - <david@redhat.com> - <palmer@sifive.com> - <mark.cave-ayland@ilande.co.uk> - <Alistair.Francis@wdc.com> - <edgar.iglesias@gmail.com> - <arikalo@wavecomp.com> - <mst@redhat.com> - <pasic@linux.ibm.com> - <borntraeger@de.ibm.com> - <rth@twiddle.net> - <atar4qemu@gmail.com> - <ehabkost@redhat.com> - <alex.williamson@redhat.com> - <qemu-arm@nongnu.org> - <stefanha@redhat.com> - <shorne@gmail.com> - <david@gibson.dropbear.id.au> - <qemu-riscv@nongnu.org> - <qemu-s390x@nongnu.org> - <kbastian@mail.uni-paderborn.de> - <cohuck@redhat.com> - <laurent@vivier.eu> - <qemu-ppc@nongnu.org> - <amarkovic@wavecomp.com> - <pbonzini@redhat.com> - " <aurelien@aurel32.net>\0" - "\01:1\0" + "Cc\0peter.maydell@linaro.org" + walling@linux.ibm.com + sagark@eecs.berkeley.edu + mst@redhat.com + palmer@sifive.com + mark.cave-ayland@ilande.co.uk + laurent@vivier.eu + Alistair.Francis@wdc.com + edgar.iglesias@gmail.com + arikalo@wavecomp.com + david@redhat.com + pasic@linux.ibm.com + borntraeger@de.ibm.com + rth@twiddle.net + atar4qemu@gmail.com + ehabkost@redhat.com + qemu-s390x@nongnu.org + qemu-arm@nongnu.org + stefanha@redhat.com + shorne@gmail.com + david@gibson.dropbear.id.au + qemu-riscv@nongnu.org + kbastian@mail.uni-paderborn.de + cohuck@redhat.com + alex.williamson@redhat.com + qemu-ppc@nongnu.org + amarkovic@wavecomp.com + pbonzini@redhat.com + " aurelien@aurel32.net\0" + "\00:1\0" "b\0" "No-op MEMOP_SIZE and SIZE_MEMOP macros allows us to later easily\n" "convert memory_region_dispatch_{read|write} paramter \"unsigned size\"\n" @@ -124,115 +124,5 @@ "\n" "--\n" 1.8.3.1 - "\01:2\0" - "b\0" - "<html>\r\n" - "<head>\r\n" - "<meta http-equiv=\"Content-Type\" content=\"text/html; charset=iso-8859-1\">\r\n" - "<style type=\"text/css\" style=\"display:none\"><!-- P { margin-top: 0px; margin-bottom: 0px; } .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left-width: 2px; border-left-style: solid; border-left-color: rgb(128, 0, 0); }--></style>\r\n" - "</head>\r\n" - "<body dir=\"ltr\" style=\"font-size:12pt;color:#000000;background-color:#FFFFFF;font-family:Calibri,Arial,Helvetica,sans-serif;\">\r\n" - "<p></p>\r\n" - "<div><span style=\"font-size: 12pt;\">No-op MEMOP_SIZE and SIZE_MEMOP macros allows us to later easily</span><br>\r\n" - "</div>\r\n" - "<div>convert memory_region_dispatch_{read|write} paramter "unsigned size"</div>\r\n" - "<div>into a size+sign+endianness encoded "MemOp op".</div>\r\n" - "<div><br>\r\n" - "</div>\r\n" - "<div>Being a no-op macro, this patch does not introduce any logical change.</div>\r\n" - "<div><br>\r\n" - "</div>\r\n" - "<div>Signed-off-by: Tony Nguyen <tony.nguyen@bt.com></div>\r\n" - "<div>---</div>\r\n" - "<div> accel/tcg/cputlb.c | 21 ++++++++++-----------</div>\r\n" - "<div> 1 file changed, 10 insertions(+), 11 deletions(-)</div>\r\n" - "<div><br>\r\n" - "</div>\r\n" - "<div>diff --git a/accel/tcg/cputlb.c b/accel/tcg/cputlb.c</div>\r\n" - "<div>index 523be4c..5d88cec 100644</div>\r\n" - "<div>--- a/accel/tcg/cputlb.c</div>\r\n" - "<div>+++ b/accel/tcg/cputlb.c</div>\r\n" - "<div>@@ -881,7 +881,7 @@ static void tlb_fill(CPUState *cpu, target_ulong addr, int size,</div>\r\n" - "<div> </div>\r\n" - "<div> static uint64_t io_readx(CPUArchState *env, CPUIOTLBEntry *iotlbentry,</div>\r\n" - "<div> int mmu_idx, target_ulong addr, uintptr_t retaddr,</div>\r\n" - "<div>- MMUAccessType access_type, int size)</div>\r\n" - "<div>+ MMUAccessType access_type, MemOp op)</div>\r\n" - "<div> {</div>\r\n" - "<div> CPUState *cpu = env_cpu(env);</div>\r\n" - "<div> hwaddr mr_offset;</div>\r\n" - "<div>@@ -906,14 +906,13 @@ static uint64_t io_readx(CPUArchState *env, CPUIOTLBEntry *iotlbentry,</div>\r\n" - "<div> qemu_mutex_lock_iothread();</div>\r\n" - "<div> locked = true;</div>\r\n" - "<div> }</div>\r\n" - "<div>- r = memory_region_dispatch_read(mr, mr_offset,</div>\r\n" - "<div>- &val, size, iotlbentry->attrs);</div>\r\n" - "<div>+ r = memory_region_dispatch_read(mr, mr_offset, &val, op, iotlbentry->attrs);</div>\r\n" - "<div> if (r != MEMTX_OK) {</div>\r\n" - "<div> hwaddr physaddr = mr_offset +</div>\r\n" - "<div> section->offset_within_address_space -</div>\r\n" - "<div> section->offset_within_region;</div>\r\n" - "<div> </div>\r\n" - "<div>- cpu_transaction_failed(cpu, physaddr, addr, size, access_type,</div>\r\n" - "<div>+ cpu_transaction_failed(cpu, physaddr, addr, MEMOP_SIZE(op), access_type,</div>\r\n" - "<div> mmu_idx, iotlbentry->attrs, r, retaddr);</div>\r\n" - "<div> }</div>\r\n" - "<div> if (locked) {</div>\r\n" - "<div>@@ -925,7 +924,7 @@ static uint64_t io_readx(CPUArchState *env, CPUIOTLBEntry *iotlbentry,</div>\r\n" - "<div> </div>\r\n" - "<div> static void io_writex(CPUArchState *env, CPUIOTLBEntry *iotlbentry,</div>\r\n" - "<div> int mmu_idx, uint64_t val, target_ulong addr,</div>\r\n" - "<div>- uintptr_t retaddr, int size)</div>\r\n" - "<div>+ uintptr_t retaddr, MemOp op)</div>\r\n" - "<div> {</div>\r\n" - "<div> CPUState *cpu = env_cpu(env);</div>\r\n" - "<div> hwaddr mr_offset;</div>\r\n" - "<div>@@ -947,15 +946,15 @@ static void io_writex(CPUArchState *env, CPUIOTLBEntry *iotlbentry,</div>\r\n" - "<div> qemu_mutex_lock_iothread();</div>\r\n" - "<div> locked = true;</div>\r\n" - "<div> }</div>\r\n" - "<div>- r = memory_region_dispatch_write(mr, mr_offset,</div>\r\n" - "<div>- val, size, iotlbentry->attrs);</div>\r\n" - "<div>+ r = memory_region_dispatch_write(mr, mr_offset, val, op, iotlbentry->attrs);</div>\r\n" - "<div> if (r != MEMTX_OK) {</div>\r\n" - "<div> hwaddr physaddr = mr_offset +</div>\r\n" - "<div> section->offset_within_address_space -</div>\r\n" - "<div> section->offset_within_region;</div>\r\n" - "<div> </div>\r\n" - "<div>- cpu_transaction_failed(cpu, physaddr, addr, size, MMU_DATA_STORE,</div>\r\n" - "<div>- mmu_idx, iotlbentry->attrs, r, retaddr);</div>\r\n" - "<div>+ cpu_transaction_failed(cpu, physaddr, addr, MEMOP_SIZE(op),</div>\r\n" - "<div>+ MMU_DATA_STORE, mmu_idx, iotlbentry->attrs, r,</div>\r\n" - "<div>+ retaddr);</div>\r\n" - "<div> }</div>\r\n" - "<div> if (locked) {</div>\r\n" - "<div> qemu_mutex_unlock_iothread();</div>\r\n" - "<div>@@ -1306,7 +1305,7 @@ load_helper(CPUArchState *env, target_ulong addr, TCGMemOpIdx oi,</div>\r\n" - "<div> }</div>\r\n" - "<div> </div>\r\n" - "<div> res = io_readx(env, &env_tlb(env)->d[mmu_idx].iotlb[index],</div>\r\n" - "<div>- mmu_idx, addr, retaddr, access_type, size);</div>\r\n" - "<div>+ mmu_idx, addr, retaddr, access_type, SIZE_MEMOP(size));</div>\r\n" - "<div> return handle_bswap(res, size, big_endian);</div>\r\n" - "<div> }</div>\r\n" - "<div> </div>\r\n" - "<div>@@ -1555,7 +1554,7 @@ store_helper(CPUArchState *env, target_ulong addr, uint64_t val,</div>\r\n" - "<div> </div>\r\n" - "<div> io_writex(env, &env_tlb(env)->d[mmu_idx].iotlb[index], mmu_idx,</div>\r\n" - "<div> handle_bswap(val, size, big_endian),</div>\r\n" - "<div>- addr, retaddr, size);</div>\r\n" - "<div>+ addr, retaddr, SIZE_MEMOP(size));</div>\r\n" - "<div> return;</div>\r\n" - "<div> }</div>\r\n" - "<div> </div>\r\n" - "<div>-- </div>\r\n" - "<div>1.8.3.1</div>\r\n" - "<div><br>\r\n" - "<br>\r\n" - "</div>\r\n" - "<p><br>\r\n" - "</p>\r\n" - "</body>\r\n" - "</html>\r\n" -b71bca06cd52167da31079519b4838ed1c9923223e70bd9b5e2d43480f5a335b +a852804189b5062e7c001af1413c06237112ab7350465f97102b6bbc5d5f719e
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.