* [PATCH] GFS2: incorrect di_height assignment
@ 2006-11-02 3:45 Alexey Dobriyan
2006-11-02 10:59 ` Christoph Hellwig
0 siblings, 1 reply; 3+ messages in thread
From: Alexey Dobriyan @ 2006-11-02 3:45 UTC (permalink / raw)
To: Andrew Morton; +Cc: Steven Whitehouse, linux-fsdevel
Harmless, but needed for endian-clean GFS2.
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
---
fs/gfs2/inode.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- a/fs/gfs2/inode.c
+++ b/fs/gfs2/inode.c
@@ -691,7 +691,7 @@ static void init_dinode(struct gfs2_inod
di->__pad1 = 0;
di->di_payload_format = cpu_to_be32(0);
- di->di_height = cpu_to_be32(0);
+ di->di_height = cpu_to_be16(0);
di->__pad2 = 0;
di->__pad3 = 0;
di->di_depth = cpu_to_be16(0);
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] GFS2: incorrect di_height assignment
2006-11-02 3:45 [PATCH] GFS2: incorrect di_height assignment Alexey Dobriyan
@ 2006-11-02 10:59 ` Christoph Hellwig
2006-11-02 12:12 ` Steven Whitehouse
0 siblings, 1 reply; 3+ messages in thread
From: Christoph Hellwig @ 2006-11-02 10:59 UTC (permalink / raw)
To: Alexey Dobriyan; +Cc: Andrew Morton, Steven Whitehouse, linux-fsdevel
On Thu, Nov 02, 2006 at 06:45:56AM +0300, Alexey Dobriyan wrote:
> Harmless, but needed for endian-clean GFS2.
>
> Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
> ---
>
> fs/gfs2/inode.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- a/fs/gfs2/inode.c
> +++ b/fs/gfs2/inode.c
> @@ -691,7 +691,7 @@ static void init_dinode(struct gfs2_inod
>
> di->__pad1 = 0;
> di->di_payload_format = cpu_to_be32(0);
> - di->di_height = cpu_to_be32(0);
> + di->di_height = cpu_to_be16(0);
you are allowed to directly assign 0 to __be* variables.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] GFS2: incorrect di_height assignment
2006-11-02 10:59 ` Christoph Hellwig
@ 2006-11-02 12:12 ` Steven Whitehouse
0 siblings, 0 replies; 3+ messages in thread
From: Steven Whitehouse @ 2006-11-02 12:12 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: Alexey Dobriyan, Andrew Morton, linux-fsdevel
[-- Attachment #1: Type: text/plain, Size: 879 bytes --]
Hi,
On Thu, 2006-11-02 at 10:59 +0000, Christoph Hellwig wrote:
> On Thu, Nov 02, 2006 at 06:45:56AM +0300, Alexey Dobriyan wrote:
> > Harmless, but needed for endian-clean GFS2.
> >
> > Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
> > ---
> >
> > fs/gfs2/inode.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > --- a/fs/gfs2/inode.c
> > +++ b/fs/gfs2/inode.c
> > @@ -691,7 +691,7 @@ static void init_dinode(struct gfs2_inod
> >
> > di->__pad1 = 0;
> > di->di_payload_format = cpu_to_be32(0);
> > - di->di_height = cpu_to_be32(0);
> > + di->di_height = cpu_to_be16(0);
>
> you are allowed to directly assign 0 to __be* variables.
>
Alexey, thanks for pointing this out. Attached is a patch which
incorporates Christoph's comments. I've applied it to my -nmw (next
merge window) git tree since its not really a bug fix as such,
Steve.
[-- Attachment #2: Patch to tidy up initialisations in inode.c --]
[-- Type: text/plain, Size: 1923 bytes --]
>From cf768f81b0ae4c20fc96e70beb512ddaf655d624 Mon Sep 17 00:00:00 2001
From: Steven Whitehouse <swhiteho@redhat.com>
Date: Thu, 2 Nov 2006 11:59:28 -0500
Subject: [PATCH] [GFS2] Tidy up 0 initialisations in inode.c
We don't need to use endian conversions for 0 initialisations
when creating a new on-disk inode.
Cc: Christoph Hellwig <hch@infradead.org>
Cc: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
---
fs/gfs2/inode.c | 14 +++++++-------
1 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c
index e467780..faf9b9e 100644
--- a/fs/gfs2/inode.c
+++ b/fs/gfs2/inode.c
@@ -673,15 +673,15 @@ static void init_dinode(struct gfs2_inod
di->di_mode = cpu_to_be32(mode);
di->di_uid = cpu_to_be32(uid);
di->di_gid = cpu_to_be32(gid);
- di->di_nlink = cpu_to_be32(0);
- di->di_size = cpu_to_be64(0);
+ di->di_nlink = 0;
+ di->di_size = 0;
di->di_blocks = cpu_to_be64(1);
di->di_atime = di->di_mtime = di->di_ctime = cpu_to_be64(get_seconds());
di->di_major = cpu_to_be32(MAJOR(dev));
di->di_minor = cpu_to_be32(MINOR(dev));
di->di_goal_meta = di->di_goal_data = cpu_to_be64(inum->no_addr);
di->di_generation = cpu_to_be64(*generation);
- di->di_flags = cpu_to_be32(0);
+ di->di_flags = 0;
if (S_ISREG(mode)) {
if ((dip->i_di.di_flags & GFS2_DIF_INHERIT_JDATA) ||
@@ -699,13 +699,13 @@ static void init_dinode(struct gfs2_inod
di->__pad1 = 0;
di->di_payload_format = cpu_to_be32(S_ISDIR(mode) ? GFS2_FORMAT_DE : 0);
- di->di_height = cpu_to_be32(0);
+ di->di_height = 0;
di->__pad2 = 0;
di->__pad3 = 0;
- di->di_depth = cpu_to_be16(0);
- di->di_entries = cpu_to_be32(0);
+ di->di_depth = 0;
+ di->di_entries = 0;
memset(&di->__pad4, 0, sizeof(di->__pad4));
- di->di_eattr = cpu_to_be64(0);
+ di->di_eattr = 0;
memset(&di->di_reserved, 0, sizeof(di->di_reserved));
brelse(dibh);
--
1.4.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2006-11-02 12:12 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-11-02 3:45 [PATCH] GFS2: incorrect di_height assignment Alexey Dobriyan
2006-11-02 10:59 ` Christoph Hellwig
2006-11-02 12:12 ` Steven Whitehouse
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).