linux-mips
[Top] [All Lists]

Re: [PATCH 25/30] loongson: Hibernation Support in mips system

To: Wu Zhangjin <wuzhangjin@gmail.com>
Subject: Re: [PATCH 25/30] loongson: Hibernation Support in mips system
From: Pavel Machek <pavel@ucw.cz>
Date: Sun, 24 May 2009 21:41:25 +0200
Cc: linux-mips@linux-mips.org, Ralf Baechle <ralf@linux-mips.org>, linux-kernel@vger.kernel.org, Arnaud Patard <apatard@mandriva.com>, loongson-dev@googlegroups.com, zhangfx@lemote.com, yanh@lemote.com, Philippe Vachon <philippe@cowpig.ca>, Zhang Le <r0bertz@gentoo.org>, Erwan Lerale <erwan@thiscow.com>
In-reply-to: <1242426488.10164.173.camel@falcon>
Original-recipient: rfc822;linux-mips@linux-mips.org
References: <1242426488.10164.173.camel@falcon>
Sender: linux-mips-bounce@linux-mips.org
User-agent: Mutt/1.5.18 (2008-05-17)
Hi!

> >From d4776f4891b9be96d357910f62d9ebaf898a3015 Mon Sep 17 00:00:00 2001
> From: Wu Zhangjin <wuzhangjin@gmail.com>
> Date: Sat, 16 May 2009 04:51:26 +0800
> Subject: [PATCH 25/30] loongson: Hibernation Support in mips system

> diff --git a/arch/mips/Makefile b/arch/mips/Makefile
> index d73f084..8bde363 100644
> --- a/arch/mips/Makefile
> +++ b/arch/mips/Makefile
> @@ -677,6 +677,9 @@ core-y                    += arch/mips/kernel/ 
> arch/mips/mm/
> arch/mips/math-emu/
>  
>  drivers-$(CONFIG_OPROFILE)   += arch/mips/oprofile/
>  
> +# suspend and hibernation support
> +drivers-$(CONFIG_PM) += arch/mips/power/
> +

Do all config combinations compile?

> @@ -3,4 +3,6 @@
>  
>  /* Somewhen...  Maybe :-)  */
>  
> +static inline int arch_prepare_suspend(void) { return 0; }
> +

And kill the somewhen comment?

> @@ -326,3 +327,15 @@ void output_octeon_cop2_state_defines(void)
>       BLANK();
>  }
>  #endif
> +
> +#ifdef CONFIG_HIBERNATION
> +void output_pbe_defines(void)
> +{
> +     COMMENT(" Linux struct pbe offsets. ");
> +     OFFSET(PBE_ADDRESS , pbe, address);
> +     OFFSET(PBE_ORIG_ADDRESS  , pbe, orig_address);
> +     OFFSET(PBE_NEXT  , pbe, next);
> +     DEFINE(PBE_SIZE  , sizeof(struct pbe));
> +     BLANK();
> +}
> +#endif

What is this? please delete spaces before ,.




> diff --git a/arch/mips/power/hibernate.S b/arch/mips/power/hibernate.S
> new file mode 100644
> index 0000000..e45ec45
> --- /dev/null
> +++ b/arch/mips/power/hibernate.S
> @@ -0,0 +1,78 @@
> +#incldue <linux/linkage.h>
> +#include <asm/asm-offsets.h>
> +#include <asm/regdef.h>
> +#include <asm/asm.h>
> +
> +.text 
> +LEAF(swsusp_arch_suspend)
> +      PTR_LA t0, saved_ra
> +      PTR_S ra, (t0)
> +      PTR_LA t0, saved_sp
> +      PTR_S sp, (t0)
> +      PTR_LA t0, saved_fp
> +      PTR_S fp, (t0)
> +      PTR_LA t0, saved_gp
> +      PTR_S gp, (t0)
> +      PTR_LA t0, saved_s0
> +      PTR_S s0, (t0) 
> +      PTR_LA t0, saved_s1
> +      PTR_S s1, (t0)
> +      PTR_LA t0, saved_s2
> +      PTR_S s2, (t0)
> +      PTR_LA t0, saved_s3
> +      PTR_S s3, (t0)
> +      PTR_LA t0, saved_s4
> +      PTR_S s4, (t0)
> +      PTR_LA t0, saved_s5
> +      PTR_S s5, (t0)
> +      PTR_LA t0, saved_s6
> +      PTR_S s6, (t0)
> +      PTR_LA t0, saved_s7
> +      PTR_S s7, (t0)
> +      PTR_LA t0, saved_a0
> +      PTR_S a0, (t0)
> +      PTR_LA t0, saved_a1
> +      PTR_S a1, (t0)
> +      PTR_LA t0, saved_a2
> +      PTR_S a2, (t0)
> +      PTR_LA t0, saved_v1
> +      PTR_S v1, (t0)
> +      j swsusp_save
> +      nop
> +END(swsusp_arch_suspend)
> +
> +LEAF(swsusp_arch_resume)
> +     PTR_L t0, restore_pblist
> +0: 
> +     PTR_L t1, PBE_ADDRESS(t0)   /* source */
> +     PTR_L t2, PBE_ORIG_ADDRESS(t0) /* destination */
> +     PTR_ADDIU t3, t1, _PAGE_SIZE
> +1: 
> +     REG_L t8, (t1)
> +     REG_S t8, (t2)
> +     PTR_ADDIU t1, t1, SZREG
> +     PTR_ADDIU t2, t2, SZREG
> +     bne t1, t3, 1b
> +     PTR_L t0, PBE_NEXT(t0)
> +     bnez t0, 0b
> +     //flush cache and tlb. no need?I am not sure.

Avoid c++ comments... and yes, I guess you should flush cache/tlb.

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

<Prev in Thread] Current Thread [Next in Thread>