Robert, Looks like my check was not needed as it is already taken care of in amd64_get_revision(). False alarm.
On Mon, Feb 16, 2009 at 6:41 PM, stephane eranian <eran...@googlemail.com> wrote: > Robert, > > On Thu, Feb 12, 2009 at 5:20 PM, Robert Richter <robert.rich...@amd.com> > wrote: >> +failed: >> + DPRINT("force failed at: %s\n", str ? str : "<NULL>"); >> + /* force AMD64 = force to Barcelona */ >> + amd64_family = 16; >> + amd64_model = 2; >> + amd64_stepping = 2; >> +done: >> + amd64_revision = amd64_get_revision(amd64_family, amd64_model, >> + amd64_stepping); > I have added this to this code: > > if (amd64_revision == AMD64_CPU_UN) > goto failed; > } > > To handle the case the user passes bogus model or stepping. > ------------------------------------------------------------------------------ Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA -OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise -Strategies to boost innovation and cut costs with open source participation -Receive a $600 discount off the registration fee with the source code: SFAD http://p.sf.net/sfu/XcvMzF8H _______________________________________________ perfmon2-devel mailing list perfmon2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/perfmon2-devel