Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-22 Thread Dave Chinner
On Fri, Apr 22, 2016 at 11:53:48PM +0200, Florian Margaine wrote: > On Tue, Apr 19, 2016 at 1:06 AM, Dave Chinner wrote: > >> A way to query freeze state might be nice, I think, but yeah, it's > >> racy, so you can't depend on it - but it might be useful in the "huh, > >> IO

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-22 Thread Dave Chinner
On Fri, Apr 22, 2016 at 11:53:48PM +0200, Florian Margaine wrote: > On Tue, Apr 19, 2016 at 1:06 AM, Dave Chinner wrote: > >> A way to query freeze state might be nice, I think, but yeah, it's > >> racy, so you can't depend on it - but it might be useful in the "huh, > >> IO is failing, what's

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-22 Thread Florian Margaine
On Tue, Apr 19, 2016 at 1:06 AM, Dave Chinner wrote: >> A way to query freeze state might be nice, I think, but yeah, it's >> racy, so you can't depend on it - but it might be useful in the "huh, >> IO is failing, what's going on? Oh, it's frozen, ok" scenario... > > So

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-22 Thread Florian Margaine
On Tue, Apr 19, 2016 at 1:06 AM, Dave Chinner wrote: >> A way to query freeze state might be nice, I think, but yeah, it's >> racy, so you can't depend on it - but it might be useful in the "huh, >> IO is failing, what's going on? Oh, it's frozen, ok" scenario... > > So maybe we should just add

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-18 Thread Dave Chinner
On Mon, Apr 18, 2016 at 11:20:22AM -0400, Eric Sandeen wrote: > > > On 4/14/16 10:17 PM, Dave Chinner wrote: > > On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > >> This lets userland get the filesystem freezing status, aka whether the > >> filesystem is frozen or not. This is

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-18 Thread Dave Chinner
On Mon, Apr 18, 2016 at 11:20:22AM -0400, Eric Sandeen wrote: > > > On 4/14/16 10:17 PM, Dave Chinner wrote: > > On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > >> This lets userland get the filesystem freezing status, aka whether the > >> filesystem is frozen or not. This is

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-18 Thread Eric Sandeen
On 4/18/16 1:20 PM, Florian Margaine wrote: > On Mon, 2016-04-18 at 11:20 -0400, Eric Sandeen wrote: >> > >> > On 4/14/16 10:17 PM, Dave Chinner wrote: >>> > > On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > > > This lets userland get the filesystem freezing status, aka

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-18 Thread Eric Sandeen
On 4/18/16 1:20 PM, Florian Margaine wrote: > On Mon, 2016-04-18 at 11:20 -0400, Eric Sandeen wrote: >> > >> > On 4/14/16 10:17 PM, Dave Chinner wrote: >>> > > On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > > > This lets userland get the filesystem freezing status, aka

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-18 Thread Florian Margaine
On Mon, 2016-04-18 at 11:20 -0400, Eric Sandeen wrote: > > On 4/14/16 10:17 PM, Dave Chinner wrote: > > On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > > > This lets userland get the filesystem freezing status, aka > > > whether the > > > filesystem is frozen or not. This is

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-18 Thread Florian Margaine
On Mon, 2016-04-18 at 11:20 -0400, Eric Sandeen wrote: > > On 4/14/16 10:17 PM, Dave Chinner wrote: > > On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > > > This lets userland get the filesystem freezing status, aka > > > whether the > > > filesystem is frozen or not. This is

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-18 Thread Eric Sandeen
On 4/14/16 10:17 PM, Dave Chinner wrote: > On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: >> This lets userland get the filesystem freezing status, aka whether the >> filesystem is frozen or not. This is so that an application can know if >> it should freeze the filesystem or

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-18 Thread Eric Sandeen
On 4/14/16 10:17 PM, Dave Chinner wrote: > On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: >> This lets userland get the filesystem freezing status, aka whether the >> filesystem is frozen or not. This is so that an application can know if >> it should freeze the filesystem or

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-17 Thread Jan Kara
On Sat 16-04-16 14:18:19, Florian Margaine wrote: > On Fri, 2016-04-15 at 12:17 +1000, Dave Chinner wrote: > > On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > > > This lets userland get the filesystem freezing status, aka whether > > > the > > > filesystem is frozen or not.

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-17 Thread Jan Kara
On Sat 16-04-16 14:18:19, Florian Margaine wrote: > On Fri, 2016-04-15 at 12:17 +1000, Dave Chinner wrote: > > On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > > > This lets userland get the filesystem freezing status, aka whether > > > the > > > filesystem is frozen or not.

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-16 Thread Florian Margaine
On Fri, 2016-04-15 at 12:17 +1000, Dave Chinner wrote: > On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > > This lets userland get the filesystem freezing status, aka whether > > the > > filesystem is frozen or not. This is so that an application can > > know if > > it should

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-16 Thread Florian Margaine
On Fri, 2016-04-15 at 12:17 +1000, Dave Chinner wrote: > On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > > This lets userland get the filesystem freezing status, aka whether > > the > > filesystem is frozen or not. This is so that an application can > > know if > > it should

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-14 Thread Dave Chinner
On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > This lets userland get the filesystem freezing status, aka whether the > filesystem is frozen or not. This is so that an application can know if > it should freeze the filesystem or if it isn't necessary when taking a > snapshot.

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-14 Thread Dave Chinner
On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > This lets userland get the filesystem freezing status, aka whether the > filesystem is frozen or not. This is so that an application can know if > it should freeze the filesystem or if it isn't necessary when taking a > snapshot.

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-14 Thread Mateusz Guzik
On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > This lets userland get the filesystem freezing status, aka whether the > filesystem is frozen or not. This is so that an application can know if > it should freeze the filesystem or if it isn't necessary when taking a > snapshot.

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-14 Thread Mateusz Guzik
On Thu, Apr 14, 2016 at 09:57:07AM +0200, Florian Margaine wrote: > This lets userland get the filesystem freezing status, aka whether the > filesystem is frozen or not. This is so that an application can know if > it should freeze the filesystem or if it isn't necessary when taking a > snapshot.

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-14 Thread Florian Margaine
23:25:28 +0100 Subject: [PATCH] fs: add the FIGETFROZEN ioctl call This lets userland get the filesystem freezing status, aka whether the filesystem is frozen or not. This is so that an application can know if it should freeze the filesystem or if it isn't necessary when taking a snapshot

Re: [PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-14 Thread Florian Margaine
TCH] fs: add the FIGETFROZEN ioctl call This lets userland get the filesystem freezing status, aka whether the filesystem is frozen or not. This is so that an application can know if it should freeze the filesystem or if it isn't necessary when taking a snapshot. ---  fs/compat_ioctl.c   |  1 +

[PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-14 Thread Florian Margaine
This lets userland get the filesystem freezing status, aka whether the filesystem is frozen or not. This is so that an application can know if it should freeze the filesystem or if it isn't necessary when taking a snapshot. --- fs/compat_ioctl.c | 1 + fs/ioctl.c | 13

[PATCH] fs: add the FIGETFROZEN ioctl call

2016-04-14 Thread Florian Margaine
This lets userland get the filesystem freezing status, aka whether the filesystem is frozen or not. This is so that an application can know if it should freeze the filesystem or if it isn't necessary when taking a snapshot. --- fs/compat_ioctl.c | 1 + fs/ioctl.c | 13