All of lore.kernel.org
 help / color / mirror / Atom feed
* Added number of PMRs to cpu_specs
@ 2004-11-01  5:42 Kumar Gala
  2004-11-01 17:23 ` Dan Malek
  0 siblings, 1 reply; 2+ messages in thread
From: Kumar Gala @ 2004-11-01  5:42 UTC (permalink / raw)
  To: Paul Mackerras; +Cc: linuxppc-dev list, anton

Paul,

Andy F. is adding 'proper' oprofile support to PPC32.  I was wondering 
if you had any issue if we added the # of PMRs a cpu has to the 
cpu_specs table.  I figured it better that we keep all such info in a 
single place, rather than reproducing PVR based checks in multiple 
places.

Note, PPC64 currently does a PVR based check in arch/ppc64/common.c.  I 
would recommend that model->num_counters come from cpu_specs and 
oprof_ppc64_ops.cpu_type be composed from cpu_specs.cpu_name.

- kumar

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Added number of PMRs to cpu_specs
  2004-11-01  5:42 Added number of PMRs to cpu_specs Kumar Gala
@ 2004-11-01 17:23 ` Dan Malek
  0 siblings, 0 replies; 2+ messages in thread
From: Dan Malek @ 2004-11-01 17:23 UTC (permalink / raw)
  To: Kumar Gala; +Cc: linuxppc-dev list, anton, Paul Mackerras


On Nov 1, 2004, at 12:42 AM, Kumar Gala wrote:

> ..... I figured it better that we keep all such info in a single 
> place, rather than reproducing PVR based checks in multiple places.

I think if a PVR test causes lots of variations to be tested, we should 
add
the information to the cpu tables.  If a PVR test is trivial (like is 
it a 601),
then we just read the PVR and test it.


	-- Dan

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2004-11-01 17:21 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-11-01  5:42 Added number of PMRs to cpu_specs Kumar Gala
2004-11-01 17:23 ` Dan Malek

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.