[Top] [All Lists]

Re: [PATCH 24/49] xfs: kill xfs_vnodeops.[ch]

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH 24/49] xfs: kill xfs_vnodeops.[ch]
From: Mark Tinguely <tinguely@xxxxxxx>
Date: Sat, 27 Jul 2013 13:58:34 -0500
Cc: xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20130727015522.GU13468@dastard>
References: <1374215120-7271-1-git-send-email-david@xxxxxxxxxxxxx> <1374215120-7271-25-git-send-email-david@xxxxxxxxxxxxx> <51F2CB91.3020801@xxxxxxx> <20130727015522.GU13468@dastard>
User-agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:9.0) Gecko/20120122 Thunderbird/9.0
On 07/26/13 20:55, Dave Chinner wrote:
On Fri, Jul 26, 2013 at 02:18:41PM -0500, Mark Tinguely wrote:
On 07/19/13 01:24, Dave Chinner wrote:
From: Dave Chinner<dchinner@xxxxxxxxxx>

Now we have xfs_inode.c for holding kernel-only XFS inode
operations, move all the inode operations from xfs_vnodeops.c to
this new file as it holds another set of kernel-only inode
operations. The name of this file traces back to the days of Irix
and it's vnodes which we don't have anymore.

Essentially this move consolidates the inode locking functions
and a bunch of XFS inode operations into the one file. Eventually
the high level functions will be merged into the VFS interface
functions in xfs_iops.c.

This leaves only internal preallocation, EOF block manipulation and
hole punching functions in vnodeops.c. Move these to xfs_bmap_util.c
where we are already consolidating various in-kernel physical extent
manipulation and querying functions.

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

As already mentioned, xfs_inode_fork.c still has a unwanted include
of now removed xfs_vnodeops.h.

A xfs_qm_dqrele(pdqp) does not exist in the copied code:

Yes, already fixed that here. It was causing test failures on



Okay, then considered it:

Reviewed-by: Mark Tinguely <tinguely@xxxxxxx>

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