Bug#669314: NFS: kernel forces trailing slash for export in /proc/self/mounts

2012-10-18 Thread Chris Hiestand
I've applied the patch to Linus' master HEAD (43c422eda99b894f18d1cca17bcd2401efaf7bd0, at the time) and the patch seems to work fine. /proc/self/mounts correctly reflects whether or not the user specified to use a trailing slash - and nothing is obviously broken. If the patch looks fine to

Bug#669314: NFS: kernel forces trailing slash for export in /proc/self/mounts

2012-10-02 Thread Chris Hiestand
I haven't had a chance to test it yet thanks. I'll be too busy until at least mid-next week, but I will test it then if nobody beats me to it. -Chris On Sep 30, 2012, at 2:23 PM, Jonathan Nieder jrnie...@gmail.com wrote: Thanks for looking it over. Did you get a chance to test Ben's patch?

Bug#669314: NFS: kernel forces trailing slash for export in /proc/self/mounts

2012-09-30 Thread Jonathan Nieder
Hi Chris, Chris Hiestand wrote: On Sep 16, 2012, at 7:00 AM, Ben Hutchings wrote: This was my first thought - but what if userland provides a device name with a slash on the end? I think we have to report it back with the slash in that case. [...] As a point of comparison, matching the

Bug#669314: NFS: kernel forces trailing slash for export in /proc/self/mounts

2012-09-16 Thread Chris Hiestand
I had a couple friends over today and we made a trivial patch to remove trailing slashes. We do not know C and have never created a patch for the kernel before, so there is undoubtedly a better way to do it. However we hope this helps in your efforts.

Bug#669314: NFS: kernel forces trailing slash for export in /proc/self/mounts

2012-09-16 Thread Ben Hutchings
On Sun, 2012-09-16 at 03:24 -0700, Chris Hiestand wrote: I had a couple friends over today and we made a trivial patch to remove trailing slashes. We do not know C and have never created a patch for the kernel before, so there is undoubtedly a better way to do it. However we hope this helps in

Bug#669314: NFS: kernel forces trailing slash for export in /proc/self/mounts

2012-09-16 Thread Chris Hiestand
On Sep 16, 2012, at 7:00 AM, Ben Hutchings wrote: This was my first thought - but what if userland provides a device name with a slash on the end? I think we have to report it back with the slash in that case. That is a great point. We could not find any standard as to whether or not

Bug#669314: NFS: kernel forces trailing slash for export in /proc/self/mounts

2012-09-15 Thread Ben Hutchings
On Tue, 2012-06-19 at 12:43 -0700, Chris Hiestand wrote: Hi Alexander Viro et al, This is an escalation of Debian Bug #669314 http://bugs.debian.org/669314, which I will re-elaborate in this email for your convenience. You committed a change to the way the linux kernel reports NFS mounts

Bug#669314: NFS: kernel forces trailing slash for export in /proc/self/mounts

2012-09-14 Thread Jonathan Nieder
Hi Chris, Chris Hiestand wrote: [Subject: No Reply From Alexander Viro] Please keep in mind that these appear as emails in a crowded inbox, so the subject ine can be a good place to put valuable context. Just an update, I emailed Alexander Viro twice and I haven't heard back from Alexander

Bug#669314: NFS: kernel forces trailing slash for export in /proc/self/mounts

2012-06-19 Thread Chris Hiestand
Hi Alexander Viro et al, This is an escalation of Debian Bug #669314 http://bugs.debian.org/669314, which I will re-elaborate in this email for your convenience. You committed a change to the way the linux kernel reports NFS mounts - now with a trailing slash for the remote export (among other