From: Jianyong Wu <Jianyong.Wu@arm.com>
To: "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>,
"pbonzini@redhat.com" <pbonzini@redhat.com>,
"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>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
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 v10 0/9] Enable ptp_kvm for arm/arm64
Date: Fri, 27 Mar 2020 09:35:07 +0000 [thread overview]
Message-ID: <HE1PR0801MB167659B867202B6D4B1695BFF4CC0@HE1PR0801MB1676.eurprd08.prod.outlook.com> (raw)
In-Reply-To: <HE1PR0801MB1676DA6F1AE78462CB524D6AF4160@HE1PR0801MB1676.eurprd08.prod.outlook.com>
Hi ,
Ping ...
Is there any comments.
Need some comments to move forward.
Thanks
Jianyong
> -----Original Message-----
> From: Jianyong Wu <Jianyong.Wu@arm.com>
> Sent: Monday, February 17, 2020 10:29 AM
> To: Jianyong Wu <Jianyong.Wu@arm.com>; 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 <Kaly.Xin@arm.com>; Justin He
> <Justin.He@arm.com>; nd <nd@arm.com>
> Subject: RE: [RFC PATCH v10 0/9] Enable ptp_kvm for arm/arm64
>
> Hi,
> As this version change a lot, expect more comments. thanks.
>
> Thanks
> Jianyong
> > -----Original Message-----
> > From: Jianyong Wu <jianyong.wu@arm.com>
> > Sent: Monday, February 10, 2020 4:49 PM
> > 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 <Kaly.Xin@arm.com>; Justin He
> > <Justin.He@arm.com>; Jianyong Wu <Jianyong.Wu@arm.com>; nd
> > <nd@arm.com>
> > Subject: [RFC PATCH v10 0/9] Enable ptp_kvm for arm/arm64
> >
> > kvm ptp targets to provide high precision time sync between guest and
> > host in virtualization environment. Here, we enable kvm ptp for arm64.
> >
> > change log:
> > from v10 to v9
> > (1) change code base to v5.5.
> > (2) enable ptp_kvm both for arm32 and arm64.
> > (3) let user choose which of virtual counter or physical
> > counter should return when using crosstimestamp mode of ptp_kvm for
> arm/arm64.
> > (4) extend input argument for getcrosstimestamp API.
> >
> > 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.
> >
> >
> > Jianyong Wu (8):
> > psci: export psci conduit get helper.
> > ptp: Reorganize ptp_kvm modules to make it arch-independent.
> > clocksource: Add clocksource id for arm arch counter
> > psci: Add hypercall service for ptp_kvm.
> > ptp: arm/arm64: Enable ptp_kvm for arm/arm64
> > ptp: extend input argument for getcrosstimestamp API
> > arm/arm64: add mechanism to let user choose which counter to return
> > arm/arm64: Add kvm capability check extension for ptp_kvm
> >
> > Thomas Gleixner (1):
> > time: Add mechanism to recognize clocksource in time_get_snapshot
> >
> > drivers/clocksource/arm_arch_timer.c | 33 ++++++++
> > drivers/firmware/psci/psci.c | 1 +
> > drivers/net/ethernet/intel/e1000e/ptp.c | 3 +-
> > drivers/ptp/Kconfig | 2 +-
> > drivers/ptp/Makefile | 5 ++
> > drivers/ptp/ptp_chardev.c | 8 +-
> > drivers/ptp/ptp_kvm.h | 11 +++
> > drivers/ptp/ptp_kvm_arm.c | 53 +++++++++++++
> > drivers/ptp/{ptp_kvm.c => ptp_kvm_common.c} | 83 ++++++--------------
> > drivers/ptp/ptp_kvm_x86.c | 87 +++++++++++++++++++++
> > include/linux/arm-smccc.h | 21 +++++
> > include/linux/clocksource.h | 6 ++
> > include/linux/clocksource_ids.h | 13 +++
> > include/linux/ptp_clock_kernel.h | 3 +-
> > include/linux/timekeeping.h | 12 +--
> > include/uapi/linux/kvm.h | 1 +
> > include/uapi/linux/ptp_clock.h | 4 +-
> > kernel/time/clocksource.c | 3 +
> > kernel/time/timekeeping.c | 1 +
> > virt/kvm/arm/arm.c | 1 +
> > virt/kvm/arm/hypercalls.c | 44 ++++++++++-
> > 21 files changed, 323 insertions(+), 72 deletions(-) create mode
> > 100644 drivers/ptp/ptp_kvm.h create mode 100644
> > drivers/ptp/ptp_kvm_arm.c rename drivers/ptp/{ptp_kvm.c =>
> > ptp_kvm_common.c} (60%) create mode
> > 100644 drivers/ptp/ptp_kvm_x86.c create mode 100644
> > include/linux/clocksource_ids.h
> >
> > --
> > 2.17.1
_______________________________________________
kvmarm mailing list
kvmarm@lists.cs.columbia.edu
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
next prev parent reply other threads:[~2020-03-27 9:35 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-10 8:48 [RFC PATCH v10 0/9] Enable ptp_kvm for arm/arm64 Jianyong Wu
2020-02-10 8:48 ` [RFC PATCH 1/9] psci: export psci conduit get helper Jianyong Wu
2020-02-10 8:48 ` [RFC PATCH 2/9] ptp: Reorganize ptp_kvm modules to make it arch-independent Jianyong Wu
2020-02-10 8:49 ` [RFC PATCH 3/9] time: Add mechanism to recognize clocksource in time_get_snapshot Jianyong Wu
2020-02-10 8:49 ` [RFC PATCH 4/9] clocksource: Add clocksource id for arm arch counter Jianyong Wu
2020-02-10 8:49 ` [RFC PATCH 5/9] psci: Add hypercall service for ptp_kvm Jianyong Wu
2020-02-10 8:49 ` [RFC PATCH 6/9] ptp: arm/arm64: Enable ptp_kvm for arm/arm64 Jianyong Wu
2020-02-10 8:49 ` [RFC PATCH 7/9] ptp: extend input argument for getcrosstimestamp API Jianyong Wu
2020-02-10 8:49 ` [RFC PATCH 8/9] arm/arm64: add mechanism to let user choose which counter to return Jianyong Wu
2020-02-10 8:49 ` [RFC PATCH 9/9] arm/arm64: Add kvm capability check extension for ptp_kvm Jianyong Wu
2020-02-17 2:28 ` [RFC PATCH v10 0/9] Enable ptp_kvm for arm/arm64 Jianyong Wu
2020-03-27 9:35 ` Jianyong Wu [this message]
2020-03-27 9:36 ` 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=HE1PR0801MB167659B867202B6D4B1695BFF4CC0@HE1PR0801MB1676.eurprd08.prod.outlook.com \
--to=jianyong.wu@arm.com \
--cc=Justin.He@arm.com \
--cc=Mark.Rutland@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=pbonzini@redhat.com \
--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).