Re: [ACPI] swsusp/s3: Assembly interactions need asmlinkage

Re: [ACPI] swsusp/s3: Assembly interactions need asmlinkage

[Len Brown]

Applied.

thanks,
-Len

On Tue, 2004-02-24 at 08:00, Pavel Machek wrote:
> Hi!
> 
> swsusp/s3 assembly parts, and parts called from assembly are not
> properly marked asmlinkage; that leads to double fault on resume when
> someone compiles kernel with regparm. Thanks go to Stefan Seyfried for
> discovering this. Please apply,
>                                                                 Pavel
> 
> --- tmp/linux/drivers/acpi/hardware/hwsleep.c   2004-02-05
> 01:53:59.000000000 +0100
> +++ linux/drivers/acpi/hardware/hwsleep.c       2004-02-23
> 21:47:23.000000000 +0100
> @@ -205,7 +205,7 @@
>   *
>  
> ******************************************************************************/
>  
> -acpi_status
> +acpi_status asmlinkage
>  acpi_enter_sleep_state (
>         u8                              sleep_state)
>  {
> --- tmp/linux/include/linux/suspend.h   2004-02-24 13:21:40.000000000
> +0100
> +++ linux/include/linux/suspend.h       2004-02-23 20:57:04.000000000
> +0100
> @@ -82,4 +82,10 @@
>  }
>  #endif /* CONFIG_PM */
>  
> +asmlinkage extern void do_magic(int is_resume);
> +asmlinkage extern void do_magic_resume_1(void);
> +asmlinkage extern void do_magic_resume_2(void);
> +asmlinkage extern void do_magic_suspend_1(void);
> +asmlinkage extern void do_magic_suspend_2(void);
> +
>  #endif /* _LINUX_SWSUSP_H */
> 
> -- 
> When do you have a heart between your knees?
> [Johanka's followup: and *two* hearts?]
> 
> 
> -------------------------------------------------------
> SF.Net is sponsored by: Speed Start Your Linux Apps Now.
> Build and deploy apps & Web services for Linux with
> a free DVD software kit from IBM. Click Now!
> http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click
> _______________________________________________
> Acpi-devel mailing list
> Acpi-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/acpi-devel
>