diff mbox series

target/i386: HAX: Enable ROM/ROM device memory region support

Message ID 1561528815-4912-1-git-send-email-hang.yuan@linux.intel.com (mailing list archive)
State New, archived
Headers show
Series target/i386: HAX: Enable ROM/ROM device memory region support | expand

Commit Message

hang.yuan@linux.intel.com June 26, 2019, 6 a.m. UTC
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.


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


no-reply@patchew.org June 26, 2019, 8:55 a.m. UTC | #1
Patchew URL: https://patchew.org/QEMU/1561528815-4912-1-git-send-email-hang.yuan@linux.intel.com/


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

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..

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

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
=== OUTPUT END ===

Test command exited with code: 1

The full log is available at
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
diff mbox series


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))
-    }
     /* Adjust start_pa and size so that they are page-aligned. (Cf
      * kvm_set_phys_mem() in kvm-all.c).