xfs
[Top] [All Lists]

Re: [PATCH 29/55] xfs: introduce xfs_inode_buf.c for inode buffer operat

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH 29/55] xfs: introduce xfs_inode_buf.c for inode buffer operations
From: Mark Tinguely <tinguely@xxxxxxx>
Date: Thu, 05 Sep 2013 17:27:37 -0500
Cc: xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <1378332359-14737-30-git-send-email-david@xxxxxxxxxxxxx>
References: <1378332359-14737-1-git-send-email-david@xxxxxxxxxxxxx> <1378332359-14737-30-git-send-email-david@xxxxxxxxxxxxx>
User-agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:9.0) Gecko/20120122 Thunderbird/9.0
On 09/04/13 17:05, Dave Chinner wrote:
From: Dave Chinner<dchinner@xxxxxxxxxx>

The only thing remaining in xfs_inode.[ch] are the operations that
read, write or verify physical inodes in their underlying buffers.
Move all this code to xfs_inode_buf.[ch] and so we can stop sharing
xfs_inode.[ch] with userspace.

Signed-off-by: Dave Chinner<dchinner@xxxxxxxxxx>
---

user equivalent of the for Linux 3.12 kernel patch 13/50.

Did not change from previous posting. I am more okay with the patch
this time.

Moves the last shared inode items to xfs_trans.h and removes the
kernel-only xfs_inode.h.

Sources do not have icreate patch (kernel commit cca9f93a) which is not
appropriate for xfsprogs...mentioned because it is a code difference.

The uuid_copy being renamed to platform_uuid_copy() is not a biggy, xfs.h does the same in a define.

Reviewed-by: Mark Tinguely <tinguely@xxxxxxx>

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