From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Sandeen Date: Mon, 02 Nov 2009 21:22:52 +0000 Subject: Re: [PATCH] fs/ext4: test the i_flags field of the right structure Message-Id: <4AEF4DAC.9080905@redhat.com> List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Julia Lawall Cc: cmm@us.ibm.com, Theodore Ts'o , Andreas Dilger , linux-ext4@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org Julia Lawall wrote: > From: Julia Lawall > > Elsewhere, EXT4_EXTENTS_FL is combined with EXT4_I(inode)->i_flags, rather > than inode->i_flags. > > Signed-off-by: Julia Lawall Reviewed-by: Eric Sandeen > > --- > fs/ext4/inode.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c > index 5c5bc5d..c0e34ab 100644 > --- a/fs/ext4/inode.c > +++ b/fs/ext4/inode.c > @@ -2787,7 +2787,7 @@ static int ext4_da_writepages_trans_blocks(struct inode *inode) > * number of contiguous block. So we will limit > * number of contiguous block to a sane value > */ > - if (!(inode->i_flags & EXT4_EXTENTS_FL) && > + if (!(EXT4_I(inode)->i_flags & EXT4_EXTENTS_FL) && > (max_blocks > EXT4_MAX_TRANS_DATA)) > max_blocks = EXT4_MAX_TRANS_DATA; > > -- > To unsubscribe from this list: send the line "unsubscribe linux-ext4" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Sandeen Subject: Re: [PATCH] fs/ext4: test the i_flags field of the right structure Date: Mon, 02 Nov 2009 15:22:52 -0600 Message-ID: <4AEF4DAC.9080905@redhat.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: cmm@us.ibm.com, "Theodore Ts'o" , Andreas Dilger , linux-ext4@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org To: Julia Lawall Return-path: Received: from mx1.redhat.com ([209.132.183.28]:41034 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755410AbZKBVW4 (ORCPT ); Mon, 2 Nov 2009 16:22:56 -0500 In-Reply-To: Sender: linux-ext4-owner@vger.kernel.org List-ID: Julia Lawall wrote: > From: Julia Lawall > > Elsewhere, EXT4_EXTENTS_FL is combined with EXT4_I(inode)->i_flags, rather > than inode->i_flags. > > Signed-off-by: Julia Lawall Reviewed-by: Eric Sandeen > > --- > fs/ext4/inode.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c > index 5c5bc5d..c0e34ab 100644 > --- a/fs/ext4/inode.c > +++ b/fs/ext4/inode.c > @@ -2787,7 +2787,7 @@ static int ext4_da_writepages_trans_blocks(struct inode *inode) > * number of contiguous block. So we will limit > * number of contiguous block to a sane value > */ > - if (!(inode->i_flags & EXT4_EXTENTS_FL) && > + if (!(EXT4_I(inode)->i_flags & EXT4_EXTENTS_FL) && > (max_blocks > EXT4_MAX_TRANS_DATA)) > max_blocks = EXT4_MAX_TRANS_DATA; > > -- > To unsubscribe from this list: send the line "unsubscribe linux-ext4" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html