diff mbox

[kvm-unit-tests,v2,03/11] s390x: set initital stack pointer to stackptr, not stacktop

Message ID 20180112143417.16743-4-david@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

David Hildenbrand Jan. 12, 2018, 2:34 p.m. UTC 
... was wrong from the beginning and remained unnoticed until we started
playing with dynamic memory allocation.

Signed-off-by: David Hildenbrand <david@redhat.com>
---
 s390x/cstart64.S | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Thomas Huth Jan. 12, 2018, 4:04 p.m. UTC | #1 
On 12.01.2018 15:34, David Hildenbrand wrote:
> ... was wrong from the beginning and remained unnoticed until we started
> playing with dynamic memory allocation.
> 
> Signed-off-by: David Hildenbrand <david@redhat.com>
> ---
>  s390x/cstart64.S | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/s390x/cstart64.S b/s390x/cstart64.S
> index 4d0c877..9a26ed3 100644
> --- a/s390x/cstart64.S
> +++ b/s390x/cstart64.S
> @@ -18,7 +18,7 @@
>  	.globl start
>  start:
>  	/* setup stack */
> -	larl	%r15, stacktop
> +	larl	%r15, stackptr
>  	/* setup initial PSW mask + control registers*/
>  	larl	%r1, initital_psw
>  	lpswe	0(%r1)
> 

Reviewed-by: Thomas Huth <thuth@redhat.com>
diff mbox

Patch

diff --git a/s390x/cstart64.S b/s390x/cstart64.S
index 4d0c877..9a26ed3 100644
--- a/s390x/cstart64.S
+++ b/s390x/cstart64.S
@@ -18,7 +18,7 @@ 
 	.globl start
 start:
 	/* setup stack */
-	larl	%r15, stacktop
+	larl	%r15, stackptr
 	/* setup initial PSW mask + control registers*/
 	larl	%r1, initital_psw
 	lpswe	0(%r1)