On Wed, Nov 21, 2018 at 09:54:08AM -0300, Arnaldo Carvalho de Melo wrote:
> Em Wed, Nov 21, 2018 at 09:51:30AM -0300, Arnaldo Carvalho de Melo escreveu:
> > Em Thu, Nov 15, 2018 at 04:14:42PM +0100, Jiri Olsa escreveu:
> > > On Thu, Nov 15, 2018 at 08:33:27AM -0500, Liang, Kan wrote:
> > > > 
> > > > 
> > > > On 11/15/2018 8:21 AM, Jiri Olsa wrote:
> > > > > On Wed, Nov 14, 2018 at 01:24:16PM -0800, [email protected] 
> > > > > wrote:
> > > > > > From: Kan Liang <[email protected]>
> > > > > > 
> > > > > > Add JSON metrics (based on event list v1) for Cascadelake server
> > > > > 
> > > > > hum mail server probably crippled the email so it won't apply:
> > > > > 
> > > > > patching file pmu-events/arch/x86/cascadelakex/cache.json
> > > > > patching file pmu-events/arch/x86/cascadelakex/clx-metrics.json
> > > > > patching file pmu-events/arch/x86/cascadelakex/floating-point.json
> > > > > patching file pmu-events/arch/x86/cascadelakex/frontend.json
> > > > > patching file pmu-events/arch/x86/cascadelakex/memory.json
> > > > > patching file pmu-events/arch/x86/cascadelakex/other.json
> > > > > patching file pmu-events/arch/x86/cascadelakex/pipeline.json
> > > > > patch: **** malformed patch at line 29879:  ow status bit and resets 
> > > > > the counter to less than MAX. The reset value to the counter is not 
> > > > > clocked immediately so the overflow status bit will flip 'high (1)' 
> > > > > and generate another PMI (if enabled) after which the reset value 
> > > > > gets clocked into the counter. Therefore, software will get the 
> > > > > interrupt, read the overflow status bit '1 for bit 34 while the 
> > > > > counter value is less than MAX. Software should ignore this case.",
> > > > > 
> > > > > I've similar error before and it was because of the 998 chars size
> > > > > limit for line.. I guess you could either post it as attachment or
> > > > > change the transfer encoding
> > > > > 
> > > > 
> > > > The patch is attached. Can it be applied?
> > > 
> > > yep, that one works
> > 
> > Can I have your reviewed-by for this one as well?
> 
> I turned the "yep, that one works" into an Acked-by, please let me know
> if that is enough.

yes, if it applies it's ok for me.. but can't judge the events inside ;-)

jirka

Reply via email to