From: Dave N. <dc...@us...> - 2007-02-01 17:26:36
Attachments:
970MP_cvs01_31.patch
|
The PPC970MP has a different set of hardware counters and must be treated as a distinct target from the rest of the PPC970 family. This patch supports the new target. Maynard can commit this patch if it looks OK to you. Along with the usual testing I also ran 'make distcheck' on a PPC970MP. -- Dave Nomura LTC Linux Power Toolchain |
From: John L. <le...@mo...> - 2007-02-01 17:32:26
|
On Thu, Feb 01, 2007 at 09:22:18AM -0800, Dave Nomura wrote: > check_event_mapping_data() > { > > - if [ "$CPUTYPE" = "ppc64/power4" -o "$CPUTYPE" = "ppc64/power5" -o "$CPUTYPE" = "ppc64/power5+" -o "$CPUTYPE" = "ppc64/970" -o "$CPUTYPE" = "ppc64/power6" ]; then > + if [ "$CPUTYPE" = "ppc64/power4" -o "$CPUTYPE" = "ppc64/power5" -o "$CPUTYPE" = "ppc64/power5+" -o "$CPUTYPE" = "ppc64/970" -o "$CPUTYPE" = "ppc64/970MP" -o "$CPUTYPE" = "ppc64/power6" ]; then Wow. Madness :) Please create a little shell function that does a case check on CPUTYPE and returns 1 or whatever if it's power, then use it here. Otherwise fine to commit regards john |
From: Dave N. <dc...@us...> - 2007-02-02 00:01:26
Attachments:
970MP_cvs01_31.patch
|
Maynard, here is the revised patch with the change John suggested to opcontrol:check_event_mapping_data() John Levon wrote: > On Thu, Feb 01, 2007 at 09:22:18AM -0800, Dave Nomura wrote: > > >> check_event_mapping_data() >> { >> >> - if [ "$CPUTYPE" = "ppc64/power4" -o "$CPUTYPE" = "ppc64/power5" -o "$CPUTYPE" = "ppc64/power5+" -o "$CPUTYPE" = "ppc64/970" -o "$CPUTYPE" = "ppc64/power6" ]; then >> + if [ "$CPUTYPE" = "ppc64/power4" -o "$CPUTYPE" = "ppc64/power5" -o "$CPUTYPE" = "ppc64/power5+" -o "$CPUTYPE" = "ppc64/970" -o "$CPUTYPE" = "ppc64/970MP" -o "$CPUTYPE" = "ppc64/power6" ]; then >> > > Wow. Madness :) > > Please create a little shell function that does a case check on CPUTYPE > and returns 1 or whatever if it's power, then use it here. Otherwise > fine to commit > > regards > john > |