Hi Stephane,
> -----Original Message-----
> From: [EMAIL PROTECTED]
> [mailto:[EMAIL PROTECTED] On Behalf Of
> Stephane Eranian
> Sent: Thursday, August 30, 2007 5:02 PM
> To: azuma?$B!JEl ?$B9'<y!K
> Cc: [EMAIL PROTECTED]
> Subject: Re: [perfmon] [PATCH] 0/2: Cell support for Perfmon2
> onlinux-2.6.22(direct PM register access)
>
> Hello,
>
> On Thu, Aug 30, 2007 at 12:28:51PM +0900, azuma?$B!JEl
> ?$B9'<y!K wrote:
> >
> > This patch directly accesses the MMIO registers and sets PM
> signal groups
> > for Cell platform.
> >
> > My previous patch was modified to correspond to git-tree.
> > In addition, I changed some coding style and error processing etc.
> > (no functional change)
> >
> > However, this patch doesn't correspond to SMP.
>
> What do you mean here?
This is my mistake. Please ignore.
(This is written for cbe_regs.c(3of3) which was currently sent Cell-linux
ML.(cbe-oss-dev))
I made some confusions. So I re-send 0of2, 1of2 and 2of2 again.
(Sorry for increasing the numbers of mails.)
P.S. I posted the message from the mail address with Japanese.
My address in perfmon2 mail-archives might be garbled.
Please ignore, or if possible, please delete that message in mail-archives.
> > Moreover "PPE signal group" is included only this time.
> >
>
> > I posted the patch to cbe_regs.[ch] (as 3of3).
> > However, the patch is posted to PPC/Cell kernel now.
> >
> Ok, so they will eventually show up in the mainline then.
> I will include the 3 parts in my tree then when patch 3of3
> shows up in mainline we'll drop it. This way the perfmon2
> tree will compile on Cell right waway.
>
> Thanks for your contributions.
>
> > 1/2
> > The patch replaces rtas_reset_signals() and rtas_activate_signals()
> > with pfm_cell_reset_signals() and
> pfm_cell_activate_signals() respectively
> > according to the platform identification. Moreover, there
> is a restriction
> > in the combination in PM signal group of Cell. The
> pfm_cell_pmc_check()
> > for the combination check is registered at the same time.
> >
> > 2/2
> > It is a main body of pfm_cell_reset_signals() and
> pfm_cell_activate_signals().
> > (in the new file: perfmon_cell_signals.c) Moreover, this
> patch includes
> > pfm_cell_pmc_check() too.
> >
> > Thanks,
> >
> > -- Takaki Azuma
> > _______________________________________________
> > perfmon mailing list
> > [email protected]
> > http://www.hpl.hp.com/hosted/linux/mail-archives/perfmon/
>
> --
>
> -Stephane
> _______________________________________________
> perfmon mailing list
> [email protected]
> http://www.hpl.hp.com/hosted/linux/mail-archives/perfmon/
-- Takaki Azuma
_______________________________________________
perfmon mailing list
[email protected]
http://www.hpl.hp.com/hosted/linux/mail-archives/perfmon/
