lkml.org 
[lkml]   [2009]   [Sep]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [RFC][PATCH 10/14] x86: generic aperf/mperf code.
From
Date
On Fri, 2009-09-04 at 11:25 +0200, Peter Zijlstra wrote:
> On Fri, 2009-09-04 at 11:19 +0200, Thomas Renninger wrote:
> > You still use struct perf_pair split/hi/lo members in #ifdef __i386__
> > case which you deleted above.
>
> > > shift_count = fls(h);
> > >
> > > - cur.aperf.whole >>= shift_count;
> > > - cur.mperf.whole >>= shift_count;
> > > + cur.aperf >>= shift_count;
> > > + cur.mperf >>= shift_count;
> > > }
> > >
> > > if (((unsigned long)(-1) / 100) < cur.aperf.split.lo) {
> > Same here, possibly still elsewhere.
> > Is this only x86_64 compile tested?
>
> Of course, who still has 32bit only hardware anyway ;-)
>
> Will fix, thanks for spotting that.

Hrmm, on that, does it really make sense to maintain the i386 code path?

How frequently is that code called and what i386 only chips support
aperf/mperf, atom?


\
 
 \ /
  Last update: 2009-09-04 11:29    [W:0.082 / U:0.152 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site