Hi Thomas, > -----Original Message----- > From: Thomas Gleixner <tglx@xxxxxxxxxxxxx> > Sent: Friday, May 29, 2020 12:36 AM > To: Jianyong Wu <Jianyong.Wu@xxxxxxx>; netdev@xxxxxxxxxxxxxxx; > yangbo.lu@xxxxxxx; john.stultz@xxxxxxxxxx; pbonzini@xxxxxxxxxx; > sean.j.christopherson@xxxxxxxxx; maz@xxxxxxxxxx; > richardcochran@xxxxxxxxx; Mark Rutland <Mark.Rutland@xxxxxxx>; > will@xxxxxxxxxx; Suzuki Poulose <Suzuki.Poulose@xxxxxxx>; Steven Price > <Steven.Price@xxxxxxx> > Cc: linux-kernel@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; > kvmarm@xxxxxxxxxxxxxxxxxxxxx; kvm@xxxxxxxxxxxxxxx; Steve Capper > <Steve.Capper@xxxxxxx>; Kaly Xin <Kaly.Xin@xxxxxxx>; Justin He > <Justin.He@xxxxxxx>; Wei Chen <Wei.Chen@xxxxxxx>; Jianyong Wu > <Jianyong.Wu@xxxxxxx>; nd <nd@xxxxxxx> > Subject: Re: [RFC PATCH v12 05/11] time: Add mechanism to recognize > clocksource in time_get_snapshot > > Jianyong Wu <jianyong.wu@xxxxxxx> writes: > > From: Thomas Gleixner <tglx@xxxxxxxxxxxxx> diff --git > > a/kernel/time/clocksource.c b/kernel/time/clocksource.c index > > 7cb09c4cf21c..a8f65b3e4ec8 100644 > > --- a/kernel/time/clocksource.c > > +++ b/kernel/time/clocksource.c > > @@ -928,6 +928,9 @@ int __clocksource_register_scale(struct > > clocksource *cs, u32 scale, u32 freq) > > > > clocksource_arch_init(cs); > > > > +if (WARN_ON_ONCE((unsigned int)cs->id >= CSID_MAX)) > > + cs->id = CSID_GENERIC; > > + > > This is white space damaged and certainly not from me. Sorry, I will fix it. Thanks Jianyong