[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v11 5/6] KVM: arm64: ioctl to fetch/store tags in a guest
From: |
Catalin Marinas |
Subject: |
Re: [PATCH v11 5/6] KVM: arm64: ioctl to fetch/store tags in a guest |
Date: |
Tue, 27 Apr 2021 18:58:44 +0100 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Fri, Apr 16, 2021 at 04:43:08PM +0100, Steven Price wrote:
> diff --git a/arch/arm64/include/uapi/asm/kvm.h
> b/arch/arm64/include/uapi/asm/kvm.h
> index 24223adae150..2b85a047c37d 100644
> --- a/arch/arm64/include/uapi/asm/kvm.h
> +++ b/arch/arm64/include/uapi/asm/kvm.h
> @@ -184,6 +184,20 @@ struct kvm_vcpu_events {
> __u32 reserved[12];
> };
>
> +struct kvm_arm_copy_mte_tags {
> + __u64 guest_ipa;
> + __u64 length;
> + union {
> + void __user *addr;
> + __u64 padding;
> + };
> + __u64 flags;
> + __u64 reserved[2];
> +};
I know Marc asked for some reserved space in here but I'm not sure it's
the right place. And what's with the union of a 64-bit pointer and
64-bit padding, it doesn't change any layout? Maybe add the two reserved
values to the union in case we want to store something else in the
future.
Or maybe I'm missing something, I haven't checked how other KVM ioctls
work.
--
Catalin
- [PATCH v11 0/6] MTE support for KVM guest, Steven Price, 2021/04/16
- [PATCH v11 1/6] arm64: mte: Sync tags for pages where PTE is untagged, Steven Price, 2021/04/16
- [PATCH v11 4/6] arm64: kvm: Expose KVM_ARM_CAP_MTE, Steven Price, 2021/04/16
- [PATCH v11 2/6] arm64: kvm: Introduce MTE VM feature, Steven Price, 2021/04/16
- [PATCH v11 5/6] KVM: arm64: ioctl to fetch/store tags in a guest, Steven Price, 2021/04/16
- Re: [PATCH v11 5/6] KVM: arm64: ioctl to fetch/store tags in a guest,
Catalin Marinas <=
- [PATCH v11 6/6] KVM: arm64: Document MTE capability and ioctl, Steven Price, 2021/04/16
- [PATCH v11 3/6] arm64: kvm: Save/restore MTE registers, Steven Price, 2021/04/16