Re: [PATCH v2] Make 83xx perfmon support selectable

2008-03-21 Thread Scott Wood
On Thu, Mar 20, 2008 at 07:59:41PM -0400, Jerry Van Baren wrote: > Per Andy (and my limited reading of the UMs), some 83xx have the PMR > registers and some don't. The compiler either supports the PMR register > or it doesn't. If you make it runtime configurable, people running CPUs > that

Re: [PATCH v2] Make 83xx perfmon support selectable

2008-03-20 Thread Jerry Van Baren
Kumar Gala wrote: On Mar 18, 2008, at 12:05 PM, Scott Wood wrote: On Fri, Mar 07, 2008 at 05:59:03PM -0600, Andy Fleming wrote: Not all e300 cores support the performance monitors, and the ones that don't will be confused by the mf/mtpmr instructions. This allows the support to be optional,

Re: [PATCH v2] Make 83xx perfmon support selectable

2008-03-19 Thread Kumar Gala
On Mar 18, 2008, at 12:05 PM, Scott Wood wrote: On Fri, Mar 07, 2008 at 05:59:03PM -0600, Andy Fleming wrote: Not all e300 cores support the performance monitors, and the ones that don't will be confused by the mf/mtpmr instructions. This allows the support to be optional, so the 8349 can tur

Re: [PATCH v2] Make 83xx perfmon support selectable

2008-03-18 Thread Scott Wood
On Fri, Mar 07, 2008 at 05:59:03PM -0600, Andy Fleming wrote: > Not all e300 cores support the performance monitors, and the ones > that don't will be confused by the mf/mtpmr instructions. This > allows the support to be optional, so the 8349 can turn it off > while the 8379 can turn it on. Sadl

Re: [PATCH v2] Make 83xx perfmon support selectable

2008-03-10 Thread Kumar Gala
On Mar 7, 2008, at 5:59 PM, Andy Fleming wrote: Not all e300 cores support the performance monitors, and the ones that don't will be confused by the mf/mtpmr instructions. This allows the support to be optional, so the 8349 can turn it off while the 8379 can turn it on. Sadly, those aren't co

Re: [PATCH v2] Make 83xx perfmon support selectable

2008-03-08 Thread Jerry Van Baren
Andy Fleming wrote: > Not all e300 cores support the performance monitors, and the ones > that don't will be confused by the mf/mtpmr instructions. This > allows the support to be optional, so the 8349 can turn it off > while the 8379 can turn it on. Sadly, those aren't config options, > so it wi