Re: [PATCH] xfs: kill time.h

xfs

Re: [PATCH] xfs: kill time.h

To:	Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject:	Re: [PATCH] xfs: kill time.h
From:	Dave Chinner <david@xxxxxxxxxxxxx>
Date:	Thu, 2 Oct 2014 08:52:21 +1000
Cc:	xfs@xxxxxxxxxxx
Delivered-to:	xfs@xxxxxxxxxxx
In-reply-to:	<20140929104443.GB10847@xxxxxxxxxxxxx>
References:	<1411954911-12278-1-git-send-email-david@xxxxxxxxxxxxx> <20140929104443.GB10847@xxxxxxxxxxxxx>
User-agent:	Mutt/1.5.21 (2010-09-15)

On Mon, Sep 29, 2014 at 03:44:43AM -0700, Christoph Hellwig wrote:
> On Mon, Sep 29, 2014 at 11:41:51AM +1000, Dave Chinner wrote:
> > From: Dave Chinner <dchinner@xxxxxxxxxx>
> > 
> > The typedef for timespecs and nanotime() are completely unnecessary,
> > and delay() can be moved to fs/xfs/linux.h, which means this file
> > can go away.
> 
> Looks good, but shouldn't we just kill delay(), too? There's just
> 11 callers, and none of them in libxfs.

Eventually, yes. ;)

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
Re: [PATCH] xfs: kill time.h, Dave Chinner <=

Previous by Date:	Re: Test xfs/287 failure, Dave Chinner
Next by Date:	Re: [PATCH] xfs: compat_xfs_bstat does not have forkoff, Dave Chinner
Previous by Thread:	[PATCH] xfs: fix the xfs_iflush_done callback search, Mark Tinguely
Next by Thread:	Re: [PATCH] xfs: compat_xfs_bstat does not have forkoff, Dave Chinner
Indexes:	[Date] [Thread] [Top] [All Lists]