[PATCH] powerpc: Fix size of st_nlink on 64bit

STALE5246d

5 messages, 3 authors, 2012-06-03 · open the first message on its own page

[PATCH] powerpc: Fix size of st_nlink on 64bit

From: Anton Blanchard <hidden>
Date: 2012-06-02 11:34:52

commit e57f93cc53b7 (powerpc: get rid of nlink_t uses, switch to
explicitly-sized type) changed the size of st_nlink on ppc64 from
a long to a short, resulting in boot failures.

Signed-off-by: Anton Blanchard <redacted>
---

Index: linux-build/arch/powerpc/include/asm/stat.h
===================================================================
--- linux-build.orig/arch/powerpc/include/asm/stat.h	2012-06-02 21:25:50.322275743 +1000
+++ linux-build/arch/powerpc/include/asm/stat.h	2012-06-02 21:26:35.183130538 +1000
@@ -30,7 +30,7 @@ struct stat {
 	unsigned long	st_dev;
 	ino_t		st_ino;
 #ifdef __powerpc64__
-	unsigned short	st_nlink;
+	unsigned long	st_nlink;
 	mode_t		st_mode;
 #else
 	mode_t		st_mode;

Re: [PATCH] powerpc: Fix size of st_nlink on 64bit

From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: 2012-06-02 21:19:37

On Sat, 2012-06-02 at 21:34 +1000, Anton Blanchard wrote:
commit e57f93cc53b7 (powerpc: get rid of nlink_t uses, switch to
explicitly-sized type) changed the size of st_nlink on ppc64 from
a long to a short, resulting in boot failures.

Signed-off-by: Anton Blanchard <redacted>
Acked-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Al, did you CC me on that ? I don't remember seeing it but it could be
my fault...

Cheers,
Ben.
quoted hunk
---

Index: linux-build/arch/powerpc/include/asm/stat.h
===================================================================
--- linux-build.orig/arch/powerpc/include/asm/stat.h	2012-06-02 21:25:50.322275743 +1000
+++ linux-build/arch/powerpc/include/asm/stat.h	2012-06-02 21:26:35.183130538 +1000
@@ -30,7 +30,7 @@ struct stat {
 	unsigned long	st_dev;
 	ino_t		st_ino;
 #ifdef __powerpc64__
-	unsigned short	st_nlink;
+	unsigned long	st_nlink;
 	mode_t		st_mode;
 #else
 	mode_t		st_mode;

Re: [PATCH] powerpc: Fix size of st_nlink on 64bit

From: Stephen Rothwell <hidden>
Date: 2012-06-03 02:28:59

Hi Anton,

On Sat, 2 Jun 2012 21:34:52 +1000 Anton Blanchard [off-list ref] wrote:
commit e57f93cc53b7 (powerpc: get rid of nlink_t uses, switch to
explicitly-sized type) changed the size of st_nlink on ppc64 from
a long to a short, resulting in boot failures.

Signed-off-by: Anton Blanchard <redacted>
Would this affect my (early user mode) boot problems reported yesterday;

/init: 71: mknod: Permission denied
/init: 88: mknod: Permission denied
/init: 88: mknod: Permission denied

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au

Re: [PATCH] powerpc: Fix size of st_nlink on 64bit

From: Anton Blanchard <hidden>
Date: 2012-06-03 03:48:36

Hi Stephen,
quoted
commit e57f93cc53b7 (powerpc: get rid of nlink_t uses, switch to
explicitly-sized type) changed the size of st_nlink on ppc64 from
a long to a short, resulting in boot failures.

Signed-off-by: Anton Blanchard <redacted>
Would this affect my (early user mode) boot problems reported
yesterday;

/init: 71: mknod: Permission denied
/init: 88: mknod: Permission denied
/init: 88: mknod: Permission denied
Very similar to the errors I was seeing so I think the patch will fix
it.

Anton

Re: [PATCH] powerpc: Fix size of st_nlink on 64bit

From: Stephen Rothwell <hidden>
Date: 2012-06-03 05:30:57

Hi Anton,

On Sun, 3 Jun 2012 13:48:36 +1000 Anton Blanchard [off-list ref] wrote:
quoted
quoted
commit e57f93cc53b7 (powerpc: get rid of nlink_t uses, switch to
explicitly-sized type) changed the size of st_nlink on ppc64 from
a long to a short, resulting in boot failures.

Signed-off-by: Anton Blanchard <redacted>
Would this affect my (early user mode) boot problems reported
yesterday;

/init: 71: mknod: Permission denied
/init: 88: mknod: Permission denied
/init: 88: mknod: Permission denied
Very similar to the errors I was seeing so I think the patch will fix
it.
Great.  One less thing to bisect tomorrow :-)

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help