Kevin, thanks for the patch. I corrected the warning now.
On Wed, Jun 14, 2006 at 02:47:44PM -0500, Kevin Corry wrote: > On Tue June 13 2006 4:52 pm, Stephane Eranian wrote: > > Hello, > > > > I have released another version of the perfmon new code base package. > > This version of the kernel patch is relative to 2.6.17-rc6. > > Got a compile warning with the new perfmon2 patches. Here's a patch in > case you haven't already caught this one. > > -- > Kevin Corry > [EMAIL PROTECTED] > http://www.ibm.com/linux/ > http://evms.sourceforge.net/ > > > Fix compiler warning: > > perfmon/perfmon.c: In function `__pfm_read_pmds': > perfmon/perfmon.c: 1399: warning: too few arguments for format > > "act_set" would appear to be the id of the "active" set. Couldn't figure > out what "acc" is supposed to be, so just removed that from the message > string. > > Signed-Off-By: Kevin Corry <[EMAIL PROTECTED]> > > Index: 2.6.17-rc6-perfmon2/perfmon/perfmon.c > =================================================================== > --- 2.6.17-rc6-perfmon2.orig/perfmon/perfmon.c > +++ 2.6.17-rc6-perfmon2/perfmon/perfmon.c > @@ -1396,11 +1396,12 @@ int __pfm_read_pmds(struct pfm_context * > val = hw_val; > } > > - PFM_DBG("set%u pmd%u=0x%llx switch_thres=%llu acc=%d > act_set=%u", > + PFM_DBG("set%u pmd%u=0x%llx switch_thres=%llu act_set=%u", > set->id, > cnum, > (unsigned long long)val, > - (unsigned long long)sw_cnt); > + (unsigned long long)sw_cnt, > + active_set->id); > > pfm_retflag_set(req->reg_flags, 0); > req->reg_value = val; -- -Stephane _______________________________________________ perfmon mailing list [email protected] http://www.hpl.hp.com/hosted/linux/mail-archives/perfmon/
