[Qemu-devel] [PATCH] target/i386: HAX: Enable ROM/ROM device memory region support

hang.yuan@linux.intel.com posted 1 patch 4 years, 9 months ago
Test docker-clang@ubuntu passed
Test s390x failed
Test asan passed
Test docker-mingw@fedora passed
Test FreeBSD passed
Test checkpatch failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1561528815-4912-1-git-send-email-hang.yuan@linux.intel.com
Maintainers: Eduardo Habkost <ehabkost@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Richard Henderson <rth@twiddle.net>
There is a newer version of this series
target/i386/hax-mem.c | 12 ++++--------
1 file changed, 4 insertions(+), 8 deletions(-)
[Qemu-devel] [PATCH] target/i386: HAX: Enable ROM/ROM device memory region support
Posted by hang.yuan@linux.intel.com 4 years, 9 months ago
From: Hang Yuan <hang.yuan@linux.intel.com>

Add ROM and ROM device memory region support in HAX. Their memory region is
read only and write access will generate EPT violation. The violation will
be handled in the HAX kernel with the following patch.

https://github.com/intel/haxm/commit/33ceea09a1655fca12c47f1e112b1d269357ff28

Signed-off-by: Hang Yuan <hang.yuan@linux.intel.com>
---
 target/i386/hax-mem.c | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/target/i386/hax-mem.c b/target/i386/hax-mem.c
index 6bb5a24917..685e88cb45 100644
--- a/target/i386/hax-mem.c
+++ b/target/i386/hax-mem.c
@@ -175,15 +175,11 @@ static void hax_process_section(MemoryRegionSection *section, uint8_t flags)
     uint64_t host_va;
     uint32_t max_mapping_size;
 
-    /* We only care about RAM and ROM regions */
-    if (!memory_region_is_ram(mr)) {
-        if (memory_region_is_romd(mr)) {
-            /* HAXM kernel module does not support ROMD yet  */
-            warn_report("Ignoring ROMD region 0x%016" PRIx64 "->0x%016" PRIx64,
-                        start_pa, start_pa + size);
-        }
+    /* We only care about RAM/ROM regions and ROM device */
+    if (memory_region_is_rom(mr) || (memory_region_is_romd(mr)))
+	flags |= HAX_RAM_INFO_ROM;
+    else if (!memory_region_is_ram(mr))
         return;
-    }
 
     /* Adjust start_pa and size so that they are page-aligned. (Cf
      * kvm_set_phys_mem() in kvm-all.c).
-- 
2.21.0


Re: [Qemu-devel] [PATCH] target/i386: HAX: Enable ROM/ROM device memory region support
Posted by no-reply@patchew.org 4 years, 9 months ago
Patchew URL: https://patchew.org/QEMU/1561528815-4912-1-git-send-email-hang.yuan@linux.intel.com/



Hi,

This series seems to have some coding style problems. See output below for
more information:

Message-id: 1561528815-4912-1-git-send-email-hang.yuan@linux.intel.com
Type: series
Subject: [Qemu-devel] [PATCH] target/i386: HAX: Enable ROM/ROM device memory region support

=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===

Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
 * [new tag]         patchew/1561528815-4912-1-git-send-email-hang.yuan@linux.intel.com -> patchew/1561528815-4912-1-git-send-email-hang.yuan@linux.intel.com
 - [tag update]      patchew/20190626024942.29758-1-eblake@redhat.com -> patchew/20190626024942.29758-1-eblake@redhat.com
Switched to a new branch 'test'
9456e09 target/i386: HAX: Enable ROM/ROM device memory region support

=== OUTPUT BEGIN ===
ERROR: braces {} are necessary for all arms of this statement
#32: FILE: target/i386/hax-mem.c:179:
+    if (memory_region_is_rom(mr) || (memory_region_is_romd(mr)))
[...]
+    else if (!memory_region_is_ram(mr))
[...]

ERROR: code indent should never use tabs
#33: FILE: target/i386/hax-mem.c:180:
+^Iflags |= HAX_RAM_INFO_ROM;$

ERROR: braces {} are necessary for all arms of this statement
#34: FILE: target/i386/hax-mem.c:181:
+    else if (!memory_region_is_ram(mr))
[...]

total: 3 errors, 0 warnings, 19 lines checked

Commit 9456e095fae9 (target/i386: HAX: Enable ROM/ROM device memory region support) has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/1561528815-4912-1-git-send-email-hang.yuan@linux.intel.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com