From: Michael Ellerman <michael@ellerman.id.au>
To: Arnd Bergmann <arnd@arndb.de>
Cc: Arnd Bergmann <arnd.bergmann@de.ibm.com>,
linuxppc-dev list <linuxppc-dev@ozlabs.org>,
paulus@samba.org,
Lucio Jose Herculano Correia <luciojhc@br.ibm.com>,
Andre Detsch <adetsch@br.ibm.com>,
cbe-oss-dev@ozlabs.org
Subject: Re: [Cbe-oss-dev] [patch 4/5] cell: saving spus information for kexec crash
Date: Tue, 19 Jun 2007 09:58:39 +1000 [thread overview]
Message-ID: <1182211119.23887.3.camel@concordia.ozlabs.ibm.com> (raw)
In-Reply-To: <20070618224923.498962312@arndb.de>
[-- Attachment #1: Type: text/plain, Size: 4595 bytes --]
On Tue, 2007-06-19 at 00:42 +0200, Arnd Bergmann wrote:
> plain text document attachment (cell-spus-info-kexec-crash-1.diff)
> From: Andre Detsch <adetsch@br.ibm.com>
> This patch adds support for investigating spus information after a
> kernel crash event, through kdump vmcore file.
> Implementation is based on xmon code, but the new functionality was
> kept independent from xmon.
>
> Signed-off-by: Lucio Jose Herculano Correia <luciojhc@br.ibm.com>
> Signed-off-by: Andre Detsch <adetsch@br.ibm.com>
> Signed-off-by: Arnd Bergmann <arnd.bergmann@de.ibm.com>
I'd like to have this patch share the emergency stopping logic with the
xmon code, I'll try and get a patch done in the next few days.
cheers
> Index: linux-2.6/arch/powerpc/kernel/crash.c
> ===================================================================
> --- linux-2.6.orig/arch/powerpc/kernel/crash.c
> +++ linux-2.6/arch/powerpc/kernel/crash.c
> @@ -219,6 +219,70 @@ void crash_kexec_secondary(struct pt_reg
> cpus_in_sr = CPU_MASK_NONE;
> }
> #endif
> +#ifdef CONFIG_SPU_BASE
> +
> +#include <asm/spu.h>
> +#include <asm/spu_priv1.h>
> +
> +struct crash_spu_info {
> + struct spu *spu;
> + u32 saved_spu_runcntl_RW;
> + u32 saved_spu_status_R;
> + u32 saved_spu_npc_RW;
> + u64 saved_mfc_sr1_RW;
> + u64 saved_mfc_dar;
> + u64 saved_mfc_dsisr;
> +};
> +
> +#define CRASH_NUM_SPUS 16 /* Enough for current hardware */
> +static struct crash_spu_info crash_spu_info[CRASH_NUM_SPUS];
> +
> +static void crash_kexec_stop_spus(void)
> +{
> + struct spu *spu;
> + int i;
> + u64 tmp;
> +
> + for (i = 0; i < CRASH_NUM_SPUS; i++) {
> + if (!crash_spu_info[i].spu)
> + continue;
> +
> + spu = crash_spu_info[i].spu;
> +
> + crash_spu_info[i].saved_spu_runcntl_RW =
> + in_be32(&spu->problem->spu_runcntl_RW);
> + crash_spu_info[i].saved_spu_status_R =
> + in_be32(&spu->problem->spu_status_R);
> + crash_spu_info[i].saved_spu_npc_RW =
> + in_be32(&spu->problem->spu_npc_RW);
> +
> + crash_spu_info[i].saved_mfc_dar = spu_mfc_dar_get(spu);
> + crash_spu_info[i].saved_mfc_dsisr = spu_mfc_dsisr_get(spu);
> + tmp = spu_mfc_sr1_get(spu);
> + crash_spu_info[i].saved_mfc_sr1_RW = tmp;
> +
> + tmp &= ~MFC_STATE1_MASTER_RUN_CONTROL_MASK;
> + spu_mfc_sr1_set(spu, tmp);
> +
> + __delay(200);
> + }
> +}
> +
> +void crash_register_spus(struct list_head *list)
> +{
> + struct spu *spu;
> +
> + list_for_each_entry(spu, list, full_list) {
> + if (spu->number >= CRASH_NUM_SPUS) {
> + WARN_ON(1);
> + continue;
> + }
> +
> + crash_spu_info[spu->number].spu = spu;
> + }
> +}
> +
> +#endif /* CONFIG_SPU_BASE */
>
> void default_machine_crash_shutdown(struct pt_regs *regs)
> {
> @@ -254,6 +318,9 @@ void default_machine_crash_shutdown(stru
> crash_save_cpu(regs, crashing_cpu);
> crash_kexec_prepare_cpus(crashing_cpu);
> cpu_set(crashing_cpu, cpus_in_crash);
> +#ifdef CONFIG_SPU_BASE
> + crash_kexec_stop_spus();
> +#endif
> if (ppc_md.kexec_cpu_down)
> ppc_md.kexec_cpu_down(1, 0);
> }
> Index: linux-2.6/include/asm-powerpc/spu.h
> ===================================================================
> --- linux-2.6.orig/include/asm-powerpc/spu.h
> +++ linux-2.6/include/asm-powerpc/spu.h
> @@ -166,6 +166,10 @@ int spu_irq_class_0_bottom(struct spu *s
> int spu_irq_class_1_bottom(struct spu *spu);
> void spu_irq_setaffinity(struct spu *spu, int cpu);
>
> +#ifdef CONFIG_KEXEC
> +void crash_register_spus(struct list_head *list);
> +#endif
> +
> extern void spu_invalidate_slbs(struct spu *spu);
> extern void spu_associate_mm(struct spu *spu, struct mm_struct *mm);
>
> Index: linux-2.6/arch/powerpc/platforms/cell/spu_base.c
> ===================================================================
> --- linux-2.6.orig/arch/powerpc/platforms/cell/spu_base.c
> +++ linux-2.6/arch/powerpc/platforms/cell/spu_base.c
> @@ -610,6 +610,9 @@ static int __init init_spu_base(void)
> }
>
> xmon_register_spus(&spu_full_list);
> +#ifdef CONFIG_KEXEC
> + crash_register_spus(&spu_full_list);
> +#endif
>
> return 0;
>
>
> --
>
> _______________________________________________
> cbe-oss-dev mailing list
> cbe-oss-dev@ozlabs.org
> https://ozlabs.org/mailman/listinfo/cbe-oss-dev
--
Michael Ellerman
OzLabs, IBM Australia Development Lab
wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)
We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 189 bytes --]
next prev parent reply other threads:[~2007-06-18 23:58 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-06-18 22:42 [patch 0/5] cell patches for 2.6.23 Arnd Bergmann
2007-06-18 22:42 ` [patch 1/5] Enable SPU switch notification to detect currently active SPU tasks Arnd Bergmann
2007-06-18 22:42 ` [patch 2/5] Add support to OProfile for profiling Cell/B.E. SPUs Arnd Bergmann
2007-06-20 2:32 ` [Cbe-oss-dev] " mita
2007-06-20 9:35 ` Arnd Bergmann
2007-06-18 22:42 ` [patch 3/5] cell: updated driver for DDR2 memory on AXON Arnd Bergmann
[not found] ` <20070619154812.GA20347@ps3linux.grid.fixstars.com>
2007-06-19 23:03 ` [Cbe-oss-dev] " Arnd Bergmann
2007-06-18 22:42 ` [patch 4/5] cell: saving spus information for kexec crash Arnd Bergmann
2007-06-18 23:58 ` Michael Ellerman [this message]
2007-06-20 18:52 ` [Cbe-oss-dev] " André Detsch
2007-06-18 22:42 ` [patch 5/5] Update cell_defconfig Arnd Bergmann
2007-06-18 23:01 ` [Cbe-oss-dev] [patch 0/5] cell patches for 2.6.23 Arnd Bergmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1182211119.23887.3.camel@concordia.ozlabs.ibm.com \
--to=michael@ellerman.id.au \
--cc=adetsch@br.ibm.com \
--cc=arnd.bergmann@de.ibm.com \
--cc=arnd@arndb.de \
--cc=cbe-oss-dev@ozlabs.org \
--cc=linuxppc-dev@ozlabs.org \
--cc=luciojhc@br.ibm.com \
--cc=paulus@samba.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).