xfs
[Top] [All Lists]

Re: [PATCH] fix xfs kernel so 2GiB logs work

To: Michael Nishimoto <miken@xxxxxxxxx>
Subject: Re: [PATCH] fix xfs kernel so 2GiB logs work
From: David Chinner <dgc@xxxxxxx>
Date: Thu, 24 Apr 2008 10:45:56 +1000
Cc: XFS Mailing List <xfs@xxxxxxxxxxx>
In-reply-to: <480FC7AA.5060401@agami.com>
References: <480FC7AA.5060401@agami.com>
Sender: xfs-bounce@xxxxxxxxxxx
User-agent: Mutt/1.4.2.1i
On Wed, Apr 23, 2008 at 04:35:06PM -0700, Michael Nishimoto wrote:
> I hope that I've got everything now.

Almost ;)

> Ensure that 2 GiB xfs logs work properly.
> 
> We found this while experimenting with 2GiB xfs logs.
> The previous code never assumed that xfs logs would
> ever get so large.
> 
> Signed-off-by: Michael Nishimoto <miken@xxxxxxxxx>
> ===========================================================
> --- xfs_log.2.c 2008-04-01 11:55:45.000000000 -0700
> +++ xfs_log.3.c 2008-04-01 11:56:53.000000000 -0700

We normally diff from the root of the kernel tree between a
pristine tree and a modified tree such that:

--- 2.6.x-xfs-new.orig/fs/xfs/xfs_log.c 2008-04-23 09:31:29.000000000 +1000
+++ 2.6.x-xfs-new/fs/xfs/xfs_log.c      2008-04-24 10:39:15.696830268 +1000
@@ -226,20 +226,24 @@ xlog_grant_sub_space(struct log *log, in
 static void
 xlog_grant_add_space_write(struct log *log, int bytes)
....

is the path in the patch and you apply it via 'patch -p1 < file'
in the root of the kernel tree.

> @@ -230,20 +230,24 @@
>  static void
>  xlog_grant_add_space_write(struct log *log, int bytes)
>  {
> -       log->l_grant_write_bytes += bytes;
> -       if (log->l_grant_write_bytes > log->l_logsize) {
   ^^^^^^^

And it looks like your mailer has converted tabs to spaces.

Don't worry about either of those things for this patch, I've
fixed them up when applying them to my test tree.

Cheers,

Dave.
-- 
Dave Chinner
Principal Engineer
SGI Australian Software Group


<Prev in Thread] Current Thread [Next in Thread>