Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-07-10 Thread NeilBrown
On Mon, Jul 10 2017, Steve Dickson wrote: > Hey Neil, > > On 07/04/2017 06:20 PM, NeilBrown wrote: >> On Tue, May 30 2017, NeilBrown wrote: >> >>> Systemd does not, and will not, support "bg" correctly. >>> It has other, better, ways to handle "background" mounting. >> >> For those who aren't cl

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-07-10 Thread Steve Dickson
Hey Neil, On 07/04/2017 06:20 PM, NeilBrown wrote: > On Tue, May 30 2017, NeilBrown wrote: > >> Systemd does not, and will not, support "bg" correctly. >> It has other, better, ways to handle "background" mounting. > > For those who aren't closely watching systemd development, a > patch was rece

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-07-04 Thread NeilBrown
On Tue, May 30 2017, NeilBrown wrote: > Systemd does not, and will not, support "bg" correctly. > It has other, better, ways to handle "background" mounting. For those who aren't closely watching systemd development, a patch was recently accepted which causes systemd to work correctly with NFS bg

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-10 Thread Andrei Borzenkov
07.06.2017 00:49, NeilBrown пишет: > On Tue, Jun 06 2017, Steve Dickson wrote: > >> Hello, >> >> On 05/29/2017 06:19 PM, NeilBrown wrote: >>> >>> Systemd does not, and will not, support "bg" correctly. >>> It has other, better, ways to handle "background" mounting. >> The only problem with this is

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-08 Thread NeilBrown
On Thu, Jun 08 2017, J. Bruce Fields wrote: > On Thu, Jun 08, 2017 at 03:16:52PM +1000, NeilBrown wrote: >> So I think I've found a solution for systemd to handle "bg" nfs mounts >> correctly. I'll submit some pull requests for consideration. > > Out of curiosity, after that change is there stil

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-08 Thread Steve Dickson
On 06/07/2017 12:08 PM, J. Bruce Fields wrote: > On Wed, Jun 07, 2017 at 06:04:12AM -0400, Steve Dickson wrote: >> # ps ax | grep mount >> 980 ?Ss 0:00 /sbin/mount.nfs nfssrv:/home/tmp /mnt/tmp -o rw,bg > > Right, but I think we also need to see a "systemctl status > remote-fs.target

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-08 Thread Steve Dickson
On 06/08/2017 01:16 AM, NeilBrown wrote: > On Wed, Jun 07 2017, Steve Dickson wrote: > >> On 06/07/2017 08:02 AM, Lennart Poettering wrote: >>> On Wed, 07.06.17 06:08, Steve Dickson (ste...@redhat.com) wrote: >>> On 06/06/2017 05:49 PM, NeilBrown wrote: > On Tue, Jun 06 2017,

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-07 Thread Mantas Mikulėnas
On Thu, Jun 8, 2017 at 8:16 AM, NeilBrown wrote: > it appears that the fg mount attempt failed quickly (ECONNREFUSED), so it > background the process immediately. Systemd sees that as success > (despite the fact that the filesystem isn't actually mounted) and > doesn't clean up the background pr

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-07 Thread NeilBrown
On Wed, Jun 07 2017, Steve Dickson wrote: > On 06/07/2017 08:02 AM, Lennart Poettering wrote: >> On Wed, 07.06.17 06:08, Steve Dickson (ste...@redhat.com) wrote: >> >>> >>> >>> On 06/06/2017 05:49 PM, NeilBrown wrote: On Tue, Jun 06 2017, Steve Dickson wrote: > Hello, > > On

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-07 Thread Steve Dickson
On 06/07/2017 08:02 AM, Lennart Poettering wrote: > On Wed, 07.06.17 06:08, Steve Dickson (ste...@redhat.com) wrote: > >> >> >> On 06/06/2017 05:49 PM, NeilBrown wrote: >>> On Tue, Jun 06 2017, Steve Dickson wrote: >>> Hello, On 05/29/2017 06:19 PM, NeilBrown wrote: > > Sy

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-07 Thread Lennart Poettering
On Wed, 07.06.17 06:08, Steve Dickson (ste...@redhat.com) wrote: > > > On 06/06/2017 05:49 PM, NeilBrown wrote: > > On Tue, Jun 06 2017, Steve Dickson wrote: > > > >> Hello, > >> > >> On 05/29/2017 06:19 PM, NeilBrown wrote: > >>> > >>> Systemd does not, and will not, support "bg" correctly. >

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-07 Thread Steve Dickson
On 06/06/2017 05:49 PM, NeilBrown wrote: > On Tue, Jun 06 2017, Steve Dickson wrote: > >> Hello, >> >> On 05/29/2017 06:19 PM, NeilBrown wrote: >>> >>> Systemd does not, and will not, support "bg" correctly. >>> It has other, better, ways to handle "background" mounting. >> The only problem with

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-07 Thread Steve Dickson
On 06/07/2017 04:12 AM, Lennart Poettering wrote: > On Tue, 06.06.17 14:07, Steve Dickson (ste...@redhat.com) wrote: > >> Hello, >> >> On 05/29/2017 06:19 PM, NeilBrown wrote: >>> >>> Systemd does not, and will not, support "bg" correctly. >>> It has other, better, ways to handle "background" mo

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-07 Thread Steve Dickson
On 06/07/2017 04:13 AM, Lennart Poettering wrote: > On Tue, 06.06.17 21:57, Michael Biebl (mbi...@gmail.com) wrote: > >> 2017-06-06 20:07 GMT+02:00 Steve Dickson : >>> Finally, the whole idea of systemd randomly/silently >>> strip off mount options is crazy... IMHO... >> >> Personally, I would p

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-07 Thread Lennart Poettering
On Tue, 06.06.17 21:57, Michael Biebl (mbi...@gmail.com) wrote: > 2017-06-06 20:07 GMT+02:00 Steve Dickson : > > Finally, the whole idea of systemd randomly/silently > > strip off mount options is crazy... IMHO... > > Personally, I would prefer if systemd simply logged a warning/error > message b

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-07 Thread Lennart Poettering
On Tue, 06.06.17 14:07, Steve Dickson (ste...@redhat.com) wrote: > Hello, > > On 05/29/2017 06:19 PM, NeilBrown wrote: > > > > Systemd does not, and will not, support "bg" correctly. > > It has other, better, ways to handle "background" mounting. > The only problem with this is bg mounts still w

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-06 Thread NeilBrown
On Tue, Jun 06 2017, Steve Dickson wrote: > Hello, > > On 05/29/2017 06:19 PM, NeilBrown wrote: >> >> Systemd does not, and will not, support "bg" correctly. >> It has other, better, ways to handle "background" mounting. > The only problem with this is bg mounts still work at least > up to 4.11 k

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-06 Thread Michael Biebl
2017-06-06 20:07 GMT+02:00 Steve Dickson : > Finally, the whole idea of systemd randomly/silently > strip off mount options is crazy... IMHO... Personally, I would prefer if systemd simply logged a warning/error message but would *not* strip the bg option. -- Why is it that all of the instrumen

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-06-06 Thread Steve Dickson
Hello, On 05/29/2017 06:19 PM, NeilBrown wrote: > > Systemd does not, and will not, support "bg" correctly. > It has other, better, ways to handle "background" mounting. The only problem with this is bg mounts still work at least up to 4.11 kernel... It appears there is a problem with a 4.12 ke

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-05-30 Thread Lennart Poettering
On Tue, 30.05.17 11:15, Michael Biebl (mbi...@gmail.com) wrote: > 2017-05-30 10:55 GMT+02:00 NeilBrown : > > On Tue, May 30 2017, Michael Biebl wrote: > > > >> 2017-05-30 0:19 GMT+02:00 NeilBrown : > >> > >>> +.B bg > >>> +option is not supported, and may be stripped from the option list. > >> > >

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-05-30 Thread Lennart Poettering
On Tue, 30.05.17 08:19, NeilBrown (ne...@suse.com) wrote: > > Systemd does not, and will not, support "bg" correctly. > It has other, better, ways to handle "background" mounting. > > Explain this. > > See also https://github.com/systemd/systemd/issues/6046 > > Signed-off-by: NeilBrown > ---

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-05-30 Thread Michael Biebl
2017-05-30 10:55 GMT+02:00 NeilBrown : > On Tue, May 30 2017, Michael Biebl wrote: > >> 2017-05-30 0:19 GMT+02:00 NeilBrown : >> >>> +.B bg >>> +option is not supported, and may be stripped from the option list. >> >> Either systemd is updated to actually strip the bg option or not. The >> document

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-05-30 Thread NeilBrown
On Tue, May 30 2017, Michael Biebl wrote: > 2017-05-30 0:19 GMT+02:00 NeilBrown : > >> +.B bg >> +option is not supported, and may be stripped from the option list. > > Either systemd is updated to actually strip the bg option or not. The > documentation should reflect that. > I don't think we sho

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-05-30 Thread Michael Biebl
2017-05-30 0:19 GMT+02:00 NeilBrown : > +.B bg > +option is not supported, and may be stripped from the option list. Either systemd is updated to actually strip the bg option or not. The documentation should reflect that. I don't think we should be vague about this, as it would only be confusing.

Re: [systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-05-29 Thread Niels de Vos
On Tue, May 30, 2017 at 08:19:16AM +1000, NeilBrown wrote: > > Systemd does not, and will not, support "bg" correctly. > It has other, better, ways to handle "background" mounting. > > Explain this. > > See also https://github.com/systemd/systemd/issues/6046 > > Signed-off-by: NeilBrown > ---

[systemd-devel] [PATCH] nfs.man: document incompatibility between "bg" option and systemd.

2017-05-29 Thread NeilBrown
Systemd does not, and will not, support "bg" correctly. It has other, better, ways to handle "background" mounting. Explain this. See also https://github.com/systemd/systemd/issues/6046 Signed-off-by: NeilBrown --- utils/mount/nfs.man | 18 +- 1 file changed, 17 insertions(+),