| To: | Christoph Hellwig <hch@xxxxxx> |
|---|---|
| Subject: | Re: XFS_ERROR use - was Re: [PATCH] prevent NULL returns from d_obtain_alias |
| From: | Timothy Shimmin <tes@xxxxxxx> |
| Date: | Mon, 20 Oct 2008 11:49:47 +1100 |
| Cc: | Miklos Szeredi <miklos@xxxxxxxxxx>, xfs-oss <xfs@xxxxxxxxxxx> |
| In-reply-to: | <20081017171000.GC18582@xxxxxx> |
| References: | <20081015192839.GA867@xxxxxx> <E1KqWzC-00087u-TG@xxxxxxxxxxxxxxxxxxx> <20081016180947.GA26285@xxxxxx> <48F7D814.2080705@xxxxxxx> <20081017171000.GC18582@xxxxxx> |
| User-agent: | Thunderbird 2.0.0.17 (Macintosh/20080914) |
Christoph Hellwig wrote: >> Fair enough. >> But XFS_ERROR is used throughout the function. > > Can we leave it the simple way for now? Yep. > I have to revamp that whole > function anyway as it's extremly buggy in many ways, especially when > used to open directories (can lead to multiple dentries for a single > directory - ouch) and then I'll kill the other uses. Oh ok. In userspace, we use it for opening directories on xfsdump via jdm_open in order to do bulkstat driven dirent dumping. We also use it in xfsrestore - though I am not convinced we should - it was initially done for "performance" reasons apparently. --Tim |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: XFS filesystem corruption on the arm(el) architecture, Lennert Buytenhek |
|---|---|
| Next by Date: | XFS slow performance, Muhammad Fakhrul Rozi Bin Mohd Seth |
| Previous by Thread: | Re: XFS_ERROR use - was Re: [PATCH] prevent NULL returns from d_obtain_alias, Christoph Hellwig |
| Next by Thread: | Re: XFS_ERROR use - was Re: [PATCH] prevent NULL returns from d_obtain_alias, Christoph Hellwig |
| Indexes: | [Date] [Thread] [Top] [All Lists] |