| To: | Felix Janda <felix.janda@xxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 2/6] libxfs/linux.c: Replace use of ustat by stat |
| From: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
| Date: | Tue, 12 Jan 2016 23:55:01 -0800 |
| Cc: | xfs@xxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <20160112195945.GC568@nyan> |
| References: | <cover.1452627740.git.felix.janda@xxxxxxxxx> <20160112195945.GC568@nyan> |
| User-agent: | Mutt/1.5.24 (2015-08-30) |
On Tue, Jan 12, 2016 at 08:59:45PM +0100, Felix Janda wrote: > ustat has been used to check whether a device file is mounted. > The function is deprecated and not supported by uclibc and musl. > Now do the check using the *mntent functions. > > Based on patch by Natanael Copa <ncopa@xxxxxxxxxxxxxxx>. > > Signed-off-by: Felix Janda <felix.janda@xxxxxxxxx> ustat is a lot more efficient in checking for the mountedness than this loop, so I'd really prefer not to do this unconditionally. As far as Linux is concerned the best thing we could do is to use O_EXCL for any open of a device file, as Linux overloadds O_EXCL behavior for block devices so that any other O_EXCL open will be rejected if one has the file open. But that might be a major pain to find into the cross platform framework.. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH 1/6] Move from __uint*_t types to uint*_t and likewise for __int*_t, Christoph Hellwig |
|---|---|
| Next by Date: | Re: [PATCH 3/6] fsr/xfs_fsr.c: Include <paths.h> for _PATH_MOUNTED, Christoph Hellwig |
| Previous by Thread: | Re: [PATCH 2/6] libxfs/linux.c: Replace use of ustat by stat, Felix Janda |
| Next by Thread: | Re: [PATCH 2/6] libxfs/linux.c: Replace use of ustat by stat, Felix Janda |
| Indexes: | [Date] [Thread] [Top] [All Lists] |