* configure.ac (AC_CHECK_TYPES): Add struct kvm_sregs.
* kvm.c (kvm_ioctl_decode_regs_segment): New function.
(kvm_ioctl) <KVM_SET_SREGS, KVM_GET_SREGS>: Use it.
* linux/arch_kvm.c (arch_print_kvm_sregs): New function.
* linux/x86_64/arch_kvm.c (PRINT_FIELD_KVM_SREGS_STRUCT): New macro.
(arch_print_kvm_sregs): New function.
(kvm_ioctl_decode_regs_dtable): Ditto.
(kvm_ioctl_decode_regs_segment): Ditto.

Changes in v2:
* Decode only if struct kvm_sregs is available.
* Put arch-specific and generic stub decoders to arch_kvm.c.
* Use umove_or_printaddr instead of umove.

    Above 3 items are suggested by ldv.

* Use more const modifiers.

Change in v3:
* Put ifdef guards around codes using kvm_sregs.
  Suggested by ldv.

Highlights in v4:
* Guards the case label with ifdef HAVE_STRUCT_KVM_SREGS.
* Fix a critical typo s/HAVE_STRUCT_KVM_REGS/HAVE_STRUCT_KVM_SREGS/.
* Fix a critical typo s/kvm_regs/kvm_sregs/.
* Introduce PRINT_FIELD_KVM_SREGS_STRUCT macro for simplifying
  the code for decoding sregs.

Changes in v5:
* Don't return RVAL_DECODED if the decoder prints something.
  Suggested by ldv.

Signed-off-by: Masatake YAMATO <yam...@redhat.com>
---
 configure.ac            |  5 ++--
 kvm.c                   | 23 ++++++++++++++++
 linux/arch_kvm.c        | 11 ++++++++
 linux/x86_64/arch_kvm.c | 72 +++++++++++++++++++++++++++++++++++++++++++++++++
 4 files changed, 109 insertions(+), 2 deletions(-)

diff --git a/configure.ac b/configure.ac
index 5103d479..729ef3f7 100644
--- a/configure.ac
+++ b/configure.ac
@@ -542,8 +542,9 @@ AC_CHECK_TYPES([struct statfs64], [
 AC_CHECK_TYPES([struct blk_user_trace_setup],,, [#include 
<linux/blktrace_api.h>])
 
 AC_CHECK_TYPES(m4_normalize([
-       struct kvm_regs
-       struct kvm_userspace_memory_region,
+       struct kvm_regs,
+       struct kvm_sregs,
+       struct kvm_userspace_memory_region
 ]),,, [#include <linux/kvm.h>])
 
 AC_CHECK_HEADERS([linux/btrfs.h], [
diff --git a/kvm.c b/kvm.c
index 5731cc25..987e7815 100644
--- a/kvm.c
+++ b/kvm.c
@@ -84,6 +84,23 @@ kvm_ioctl_decode_regs(struct tcb *const tcp, const unsigned 
int code, const kern
 }
 # endif /* HAVE_STRUCT_KVM_REGS */
 
+# ifdef HAVE_STRUCT_KVM_SREGS
+static int
+kvm_ioctl_decode_sregs(struct tcb *const tcp, const unsigned int code, const 
kernel_ulong_t arg)
+{
+       struct kvm_sregs sregs;
+
+       if (code == KVM_GET_SREGS && entering(tcp))
+               return 0;
+
+       tprints(", ");
+       if (!umove_or_printaddr(tcp, arg, &sregs))
+               arch_print_kvm_sregs(tcp, arg, &sregs);
+
+       return RVAL_IOCTL_DECODED;
+}
+# endif        /* HAVE_STRUCT_KVM_SREGS */
+
 int
 kvm_ioctl(struct tcb *const tcp, const unsigned int code, const kernel_ulong_t 
arg)
 {
@@ -102,6 +119,12 @@ kvm_ioctl(struct tcb *const tcp, const unsigned int code, 
const kernel_ulong_t a
                return kvm_ioctl_decode_regs(tcp, code, arg);
 # endif
 
+# ifdef HAVE_STRUCT_KVM_SREGS
+       case KVM_SET_SREGS:
+       case KVM_GET_SREGS:
+               return kvm_ioctl_decode_sregs(tcp, code, arg);
+# endif
+
        case KVM_CREATE_VM:
                return RVAL_DECODED | RVAL_FD;
        case KVM_RUN:
diff --git a/linux/arch_kvm.c b/linux/arch_kvm.c
index d1292a5b..bb1119a4 100644
--- a/linux/arch_kvm.c
+++ b/linux/arch_kvm.c
@@ -7,3 +7,14 @@ arch_print_kvm_regs(struct tcb *const tcp,
        printaddr(addr);
 }
 #endif /* HAVE_STRUCT_KVM_REGS */
+
+#ifdef HAVE_STRUCT_KVM_SREGS
+static void
+arch_print_kvm_sregs(struct tcb *const tcp,
+                   const kernel_ulong_t addr,
+                   const struct kvm_sregs *const sregs)
+{
+       printaddr(addr);
+}
+#endif /* HAVE_STRUCT_KVM_SREGS */
+
diff --git a/linux/x86_64/arch_kvm.c b/linux/x86_64/arch_kvm.c
index f5ade5d8..d76ca034 100644
--- a/linux/x86_64/arch_kvm.c
+++ b/linux/x86_64/arch_kvm.c
@@ -36,3 +36,75 @@ arch_print_kvm_regs(struct tcb *const tcp,
        tprints("}");
 }
 #endif /* HAVE_STRUCT_KVM_REGS */
+
+#ifdef HAVE_STRUCT_KVM_SREGS
+static void
+kvm_ioctl_decode_regs_segment(const char *prefix,
+                             const struct kvm_segment *const segment)
+{
+       tprints(prefix);
+       PRINT_FIELD_X("={", *segment, base);
+       PRINT_FIELD_U(", ", *segment, limit);
+       PRINT_FIELD_U(", ", *segment, selector);
+       PRINT_FIELD_U(", ", *segment, type);
+       PRINT_FIELD_U(", ", *segment, present);
+       PRINT_FIELD_U(", ", *segment, dpl);
+       PRINT_FIELD_U(", ", *segment, db);
+       PRINT_FIELD_U(", ", *segment, s);
+       PRINT_FIELD_U(", ", *segment, l);
+       PRINT_FIELD_U(", ", *segment, g);
+       PRINT_FIELD_U(", ", *segment, avl);
+       tprints("}");
+}
+
+static void
+kvm_ioctl_decode_regs_dtable(const char *prefix,
+                            const struct kvm_dtable *const dtable)
+{
+       tprints(prefix);
+       PRINT_FIELD_X("={", *dtable, base);
+       PRINT_FIELD_U(", ", *dtable, limit);
+       tprints("}");
+}
+
+# define PRINT_FIELD_KVM_SREGS_STRUCT(prefix_, where_, type_, field_)  \
+  kvm_ioctl_decode_regs_ ## type_(prefix_ #field_, &(where_)->field_)
+
+static void
+arch_print_kvm_sregs(struct tcb *const tcp,
+                    const kernel_ulong_t addr,
+                    const struct kvm_sregs *const sregs)
+{
+       PRINT_FIELD_KVM_SREGS_STRUCT("{", sregs, segment, cs);
+       if (abbrev(tcp)) {
+               tprints(", ...}");
+               return;
+       }
+
+       PRINT_FIELD_KVM_SREGS_STRUCT(", ", sregs, segment, ds);
+       PRINT_FIELD_KVM_SREGS_STRUCT(", ", sregs, segment, es);
+       PRINT_FIELD_KVM_SREGS_STRUCT(", ", sregs, segment, fs);
+       PRINT_FIELD_KVM_SREGS_STRUCT(", ", sregs, segment, gs);
+       PRINT_FIELD_KVM_SREGS_STRUCT(", ", sregs, segment, ss);
+       PRINT_FIELD_KVM_SREGS_STRUCT(", ", sregs, segment, tr);
+       PRINT_FIELD_KVM_SREGS_STRUCT(", ", sregs, segment, ldt);
+       PRINT_FIELD_KVM_SREGS_STRUCT(", ", sregs, dtable, gdt);
+       PRINT_FIELD_KVM_SREGS_STRUCT(", ", sregs, dtable, idt);
+       PRINT_FIELD_U(", ", *sregs, cr0);
+       PRINT_FIELD_U(", ", *sregs, cr2);
+       PRINT_FIELD_U(", ", *sregs, cr3);
+       PRINT_FIELD_U(", ", *sregs, cr4);
+       PRINT_FIELD_U(", ", *sregs, cr8);
+       PRINT_FIELD_U(", ", *sregs, efer);
+       PRINT_FIELD_X(", ", *sregs, apic_base);
+       tprints(", interrupt_bitmap=[");
+
+       unsigned int i;
+       for (i = 0; i < ARRAY_SIZE(sregs->interrupt_bitmap); i++) {
+               if (i != 0)
+                       tprints(", ");
+               tprintf("%#" PRI__x64, sregs->interrupt_bitmap[i]);
+       }
+       tprints("]}");
+}
+#endif /* HAVE_STRUCT_KVM_SREGS */
-- 
2.13.6


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel

Reply via email to