Hi Aubrey, Looks very good! Only found one nit.
At line 61 of turbo.c, need to insert _turbo_supported = B_FALSE; Either that or just set it to false first thing on entry to the routine and only set it to true at the bottom. Otherwise it looks great. Mark Li, Aubrey wrote: > The prototype of turbo observability support is almost done. > webrev is here: > http://cr.opensolaris.org/~aubrey/turbo-support/ > > It includes both kernel part and application part. > The screenshot is here: > http://www.opensolaris.org/os/project/tesla/Work/Powertop/turbo.JPG > > Welcome any comments and suggestions! > > Thanks, > -Aubrey >
