From: Paolo Bonzini <pbonzini@redhat.com>
To: Jianyong Wu <Jianyong.Wu@arm.com>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
"yangbo.lu@nxp.com" <yangbo.lu@nxp.com>,
"john.stultz@linaro.org" <john.stultz@linaro.org>,
"tglx@linutronix.de" <tglx@linutronix.de>,
"sean.j.christopherson@intel.com"
<sean.j.christopherson@intel.com>,
"maz@kernel.org" <maz@kernel.org>,
"richardcochran@gmail.com" <richardcochran@gmail.com>,
Mark Rutland <Mark.Rutland@arm.com>,
"will@kernel.org" <will@kernel.org>,
Suzuki Poulose <Suzuki.Poulose@arm.com>,
Steven Price <Steven.Price@arm.com>
Cc: Justin He <Justin.He@arm.com>,
"kvm@vger.kernel.org" <kvm@vger.kernel.org>,
Steve Capper <Steve.Capper@arm.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Kaly Xin <Kaly.Xin@arm.com>, nd <nd@arm.com>,
"kvmarm@lists.cs.columbia.edu" <kvmarm@lists.cs.columbia.edu>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [RFC PATCH v9 0/8] Enable ptp_kvm for arm64
Date: Tue, 7 Jan 2020 09:15:37 +0100 [thread overview]
Message-ID: <bf333cdc-3455-7c64-89c2-014639614904@redhat.com> (raw)
In-Reply-To: <HE1PR0801MB1676CFC9A06B6CE800052A99F43C0@HE1PR0801MB1676.eurprd08.prod.outlook.com>
On 06/01/20 10:38, Jianyong Wu wrote:
> Ping ...
> Any comments to this patch set?
Marc, Will, can you ack it? Since the sticky point was the detection of
the clocksource and it was solved by Thomas's patch, I don't have any
more problems including it.
Thanks,
Paolo
>> -----Original Message-----
>> From: Jianyong Wu <jianyong.wu@arm.com>
>> Sent: Tuesday, December 10, 2019 11:40 AM
>> To: netdev@vger.kernel.org; yangbo.lu@nxp.com; john.stultz@linaro.org;
>> tglx@linutronix.de; pbonzini@redhat.com; sean.j.christopherson@intel.com;
>> maz@kernel.org; richardcochran@gmail.com; Mark Rutland
>> <Mark.Rutland@arm.com>; will@kernel.org; Suzuki Poulose
>> <Suzuki.Poulose@arm.com>; Steven Price <Steven.Price@arm.com>
>> Cc: linux-kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org;
>> kvmarm@lists.cs.columbia.edu; kvm@vger.kernel.org; Steve Capper
>> <Steve.Capper@arm.com>; Kaly Xin (Arm Technology China)
>> <Kaly.Xin@arm.com>; Justin He (Arm Technology China)
>> <Justin.He@arm.com>; Jianyong Wu (Arm Technology China)
>> <Jianyong.Wu@arm.com>; nd <nd@arm.com>
>> Subject: [RFC PATCH v9 0/8] Enable ptp_kvm for arm64
>>
>> kvm ptp targets to provide high precision time sync between guest and host
>> in virtualization environment. Here, we enable kvm ptp for arm64.
>> This patch set base on [1][2][3]
>>
>> change log:
>>
>> from v8 to v9:
>> (1) move ptp_kvm.h to driver/ptp/
>> (2) replace license declaration of ptp_kvm.h the same with other
>> header files in the same directory.
>>
>> from v7 to v8:
>> (1) separate adding clocksource id for arm_arch_counter as a single patch.
>> (2) update commit message for patch 4/8.
>> (3) refine patch 7/8 and patch 8/8 to make them more independent.
>>
>> from v6 to v7:
>> (1) include the omitted clocksource_id.h in last version.
>> (2) reorder the header file in patch.
>> (3) refine some words in commit message to make it more impersonal.
>>
>> from v5 to v6:
>> (1) apply Mark's patch[4] to get SMCCC conduit.
>> (2) add mechanism to recognize current clocksource by add
>> clocksouce_id value into struct clocksource instead of method in patch-v5.
>> (3) rename kvm_arch_ptp_get_clock_fn into
>> kvm_arch_ptp_get_crosststamp.
>>
>> from v4 to v5:
>> (1) remove hvc delay compensasion as it should leave to userspace.
>> (2) check current clocksource in hvc call service.
>> (3) expose current clocksource by adding it to system_time_snapshot.
>> (4) add helper to check if clocksource is arm_arch_counter.
>> (5) rename kvm_ptp.c to ptp_kvm_common.c
>>
>> from v3 to v4:
>> (1) fix clocksource of ptp_kvm to arch_sys_counter.
>> (2) move kvm_arch_ptp_get_clock_fn into arm_arch_timer.c
>> (3) subtract cntvoff before return cycles from host.
>> (4) use ktime_get_snapshot instead of getnstimeofday and
>> get_current_counterval to return time and counter value.
>> (5) split ktime and counter into two 32-bit block respectively to avoid
>> Y2038-safe issue.
>> (6) set time compensation to device time as half of the delay of hvc call.
>> (7) add ARM_ARCH_TIMER as dependency of ptp_kvm for arm64.
>>
>> from v2 to v3:
>> (1) fix some issues in commit log.
>> (2) add some receivers in send list.
>>
>> from v1 to v2:
>> (1) move arch-specific code from arch/ to driver/ptp/
>> (2) offer mechanism to inform userspace if ptp_kvm service is available.
>> (3) separate ptp_kvm code for arm64 into hypervisor part and guest part.
>> (4) add API to expose monotonic clock and counter value.
>> (5) refine code: remove no necessary part and reconsitution.
>>
>> [1]https://git.kernel.org/pub/scm/linux/kernel/git/will/linux.git/
>> commit/?h=kvm/hvc&id=125ea89e4a21e2fc5235410f966a996a1a7148bf
>> [2]https://git.kernel.org/pub/scm/linux/kernel/git/will/linux.git/
>> commit/?h=kvm/hvc&id=464f5a1741e5959c3e4d2be1966ae0093b4dce06
>> [3]https://git.kernel.org/pub/scm/linux/kernel/git/will/linux.git/
>> commit/?h=kvm/hvc&id=6597490e005d0eeca8ed8c1c1d7b4318ee014681
>> [4]https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git/
>> commit/?h=for-next/smccc-conduit-
>> cleanup&id=6b7fe77c334ae59fed9500140e08f4f896b36871
>>
>> Jianyong Wu (6):
>> psci: let arm_smccc_1_1_invoke available by modules
>> ptp: Reorganize ptp_kvm modules to make it arch-independent.
>> clocksource: Add clocksource id for arm arch counter
>> psci: Add hvc call service for ptp_kvm.
>> ptp: arm64: Enable ptp_kvm for arm64
>> kvm: arm64: Add capability check extension for ptp_kvm
>>
>> Mark Rutland (1):
>> arm/arm64: smccc/psci: add arm_smccc_1_1_get_conduit()
>>
>> Thomas Gleixner (1):
>> time: Add mechanism to recognize clocksource in time_get_snapshot
>>
>> drivers/clocksource/arm_arch_timer.c | 24 ++++++
>> drivers/firmware/psci/psci.c | 16 ++++
>> drivers/ptp/Kconfig | 2 +-
>> drivers/ptp/Makefile | 1 +
>> drivers/ptp/ptp_kvm.h | 11 +++
>> drivers/ptp/ptp_kvm_arm64.c | 53 +++++++++++++
>> drivers/ptp/{ptp_kvm.c => ptp_kvm_common.c} | 77 +++++-------------
>> drivers/ptp/ptp_kvm_x86.c | 87 +++++++++++++++++++++
>> include/linux/arm-smccc.h | 30 ++++++-
>> include/linux/clocksource.h | 6 ++
>> include/linux/clocksource_ids.h | 13 +++
>> include/linux/timekeeping.h | 12 +--
>> include/uapi/linux/kvm.h | 1 +
>> kernel/time/clocksource.c | 3 +
>> kernel/time/timekeeping.c | 1 +
>> virt/kvm/arm/arm.c | 1 +
>> virt/kvm/arm/psci.c | 22 ++++++
>> 17 files changed, 294 insertions(+), 66 deletions(-) create mode 100644
>> drivers/ptp/ptp_kvm.h create mode 100644 drivers/ptp/ptp_kvm_arm64.c
>> rename drivers/ptp/{ptp_kvm.c => ptp_kvm_common.c} (63%) create
>> mode 100644 drivers/ptp/ptp_kvm_x86.c create mode 100644
>> include/linux/clocksource_ids.h
>>
>> --
>> 2.17.1
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-01-07 8:16 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-12-10 3:40 [RFC PATCH v9 0/8] Enable ptp_kvm for arm64 Jianyong Wu
2019-12-10 3:40 ` [RFC PATCH v9 1/8] arm/arm64: smccc/psci: add arm_smccc_1_1_get_conduit() Jianyong Wu
2019-12-10 3:40 ` [RFC PATCH v9 2/8] psci: let arm_smccc_1_1_invoke available by modules Jianyong Wu
2019-12-10 3:40 ` [RFC PATCH v9 3/8] ptp: Reorganize ptp_kvm modules to make it arch-independent Jianyong Wu
2019-12-10 3:40 ` [RFC PATCH v9 4/8] time: Add mechanism to recognize clocksource in time_get_snapshot Jianyong Wu
2019-12-10 3:40 ` [RFC PATCH v9 5/8] clocksource: Add clocksource id for arm arch counter Jianyong Wu
2019-12-10 3:40 ` [RFC PATCH v9 6/8] psci: Add hvc call service for ptp_kvm Jianyong Wu
2020-01-07 9:16 ` Marc Zyngier
2020-01-09 5:45 ` Jianyong Wu
2020-01-09 9:16 ` Marc Zyngier
2020-01-10 9:51 ` Jianyong Wu
2020-01-10 10:56 ` Marc Zyngier
2020-01-13 10:30 ` Jianyong Wu
2020-01-13 11:16 ` Marc Zyngier
2020-01-14 10:34 ` Jianyong Wu
2019-12-10 3:40 ` [RFC PATCH v9 7/8] ptp: arm64: Enable ptp_kvm for arm64 Jianyong Wu
2020-01-07 9:29 ` Marc Zyngier
2020-01-09 5:59 ` Jianyong Wu
2020-01-09 9:24 ` Marc Zyngier
2020-01-09 9:46 ` Marc Zyngier
2020-01-10 10:15 ` Jianyong Wu
2020-01-10 10:15 ` Jianyong Wu
2020-01-10 10:35 ` Marc Zyngier
2020-01-13 10:37 ` Jianyong Wu
2020-01-13 11:21 ` Marc Zyngier
2020-01-14 10:22 ` Jianyong Wu
2019-12-10 3:40 ` [RFC PATCH v9 8/8] kvm: arm64: Add capability check extension for ptp_kvm Jianyong Wu
2020-01-06 9:38 ` [RFC PATCH v9 0/8] Enable ptp_kvm for arm64 Jianyong Wu
2020-01-07 8:15 ` Paolo Bonzini [this message]
2020-01-07 9:33 ` Marc Zyngier
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=bf333cdc-3455-7c64-89c2-014639614904@redhat.com \
--to=pbonzini@redhat.com \
--cc=Jianyong.Wu@arm.com \
--cc=Justin.He@arm.com \
--cc=Kaly.Xin@arm.com \
--cc=Mark.Rutland@arm.com \
--cc=Steve.Capper@arm.com \
--cc=Steven.Price@arm.com \
--cc=Suzuki.Poulose@arm.com \
--cc=john.stultz@linaro.org \
--cc=kvm@vger.kernel.org \
--cc=kvmarm@lists.cs.columbia.edu \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maz@kernel.org \
--cc=nd@arm.com \
--cc=netdev@vger.kernel.org \
--cc=richardcochran@gmail.com \
--cc=sean.j.christopherson@intel.com \
--cc=tglx@linutronix.de \
--cc=will@kernel.org \
--cc=yangbo.lu@nxp.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).