Ian Kent ra...@themaw.net wrote:
Is this something others need?
Not as far as I know... I think autofs is the only one doing out-of-kernel
automounting.
That doesn't mean it shouldn't be provided, though...
Again, the exists vs not yet exists case for paths within indirect
autofs mounts.
These are filesystem-independent mount options that get passed to
mount.cifs too. Handle them appropriately by enabling and disabling
MS_MANDLOCK and not handing them off to the kernel.
Also, don't set MS_MANDLOCK by default. There's no reason to ask the
kernel to enforce mandatory locking by
On Tue, 2010-07-27 at 09:48 +0100, David Howells wrote:
Ian Kent ra...@themaw.net wrote:
Is this something others need?
Not as far as I know... I think autofs is the only one doing out-of-kernel
automounting.
That doesn't mean it shouldn't be provided, though...
Again, the exists
On 07/26/2010 11:55 PM, Jeff Layton wrote:
I'm not sure why this was merged with this flag hardcoded on, but it
seems quite dangerous. Turn it off.
Mea culpa.. I forgot to remove this during the second post.
Also, mount.cifs hands unrecognized options off to the kernel so there
should be no
David Howells dhowe...@redhat.com wrote:
Also, not all the flags are per-filesystem. The following are:
FS_SPECIAL_FL /* Special file as found in procfs/sysfs */
FS_REMOTE_FL/* File is remote */
Actually, that last is not true; FS_REMOTE_FL is per-file,
Document the newly added local caching feature using FS-Cache. This patch
could be queued and considered once the local caching patches gets merged
upstream.
Signed-off-by: Suresh Jayaraman sjayara...@suse.de
---
doc/mount.cifs.8.xml | 17 +
1 files changed, 17 insertions(+), 0
Add 'fsc' mount option to the 'Less commonly used options' section of
mount.cifs usage help text. As with the previous patch, this one too could be
queued and considered once the local caching patches gets merged upstream.
Signed-off-by: Suresh Jayaraman sjayara...@suse.de
---
mount.cifs.c |
On Tue, 27 Jul 2010 19:20:00 +0530
Suresh Jayaraman sjayara...@suse.de wrote:
Document the newly added local caching feature using FS-Cache. This patch
could be queued and considered once the local caching patches gets merged
upstream.
Signed-off-by: Suresh Jayaraman sjayara...@suse.de
---
Yes - I agree with Jeff, need to add the mention of the config option
and perhaps how to tell if fsc feature is available in kernel.
On Tue, Jul 27, 2010 at 9:33 AM, Jeff Layton jlay...@redhat.com wrote:
On Tue, 27 Jul 2010 19:20:00 +0530
Suresh Jayaraman sjayara...@suse.de wrote:
Document
On Tue, 27 Jul 2010 10:13:14 -0500
Steve French smfre...@gmail.com wrote:
Yes - I agree with Jeff, need to add the mention of the config option
and perhaps how to tell if fsc feature is available in kernel.
How would one tell? :)
On Tue, Jul 27, 2010 at 9:33 AM, Jeff Layton
scripts/extract-ikconfig could be run against the image (or look at
/proc/config.gz) - but unless there is an easier way to view kernel
config options on particular distros via distro specific tools -
probably easier just to note that CONFIG_FSCACHE must be selected in
the kernel build (since it
On Tue, 27 Jul 2010 10:21:51 -0500
Steve French smfre...@gmail.com wrote:
scripts/extract-ikconfig could be run against the image (or look at
/proc/config.gz) - but unless there is an easier way to view kernel
config options on particular distros via distro specific tools -
probably easier
On 07/27/2010 08:51 PM, Steve French wrote:
scripts/extract-ikconfig could be run against the image (or look at
/proc/config.gz) - but unless there is an easier way to view kernel
config options on particular distros via distro specific tools -
probably easier just to note that CONFIG_FSCACHE
On Tue, Jul 27, 2010 at 11:33 AM, Suresh Jayaraman sjayara...@suse.de wrote:
On 07/27/2010 08:51 PM, Steve French wrote:
scripts/extract-ikconfig could be run against the image (or look at
/proc/config.gz) - but unless there is an easier way to view kernel
config options on particular distros
On Tue, 27 Jul 2010 19:20:00 +0530
Suresh Jayaraman sjayara...@suse.de wrote:
Document the newly added local caching feature using FS-Cache. This patch
could be queued and considered once the local caching patches gets merged
upstream.
Signed-off-by: Suresh Jayaraman sjayara...@suse.de
---
On Mon, 26 Jul 2010 13:05:28 -0400
Jeff Layton jlay...@samba.org wrote:
Even though all known kernels send the uid= parm to userspace,
cifs.upcall doesn't technically require it. It should though. If one
wasn't sent for some reason, then the setuid wouldn't occur. Error out
if there is no
On Tue, 27 Jul 2010 19:26:46 +0530
Suresh Jayaraman sjayara...@suse.de wrote:
Add 'fsc' mount option to the 'Less commonly used options' section of
mount.cifs usage help text. As with the previous patch, this one too could be
queued and considered once the local caching patches gets merged
On Mon, 26 Jul 2010 13:05:29 -0400
Jeff Layton jlay...@samba.org wrote:
These are filesystem-independent mount options that get passed to
mount.cifs too. Handle them appropriately by enabling and disabling
MS_MANDLOCK and not handing them off to the kernel.
Signed-off-by: Jeff Layton
Changes since last post:
- added the information about the kernel CONFIG option
- also added the information that caching is currently enabled for
files opened as read-only
Document the newly added local caching feature using FS-Cache. This patch
could be queued and considered
On Tue, 27 Jul 2010 22:18:10 +0530
Suresh Jayaraman sjayara...@suse.de wrote:
Changes since last post:
- added the information about the kernel CONFIG option
- also added the information that caching is currently enabled for
files opened as read-only
Document the newly added
This patchset adds the -Wextra flag to CFLAGS and fixes up some of the
warnings that that generates. There are still some others that I'm
not sure how best to fix yet.
Jeff Layton (4):
automake: add -Wextra to CFLAGS
mtab: add __attribute__((unused)) to unused variables
cifs.upcall: swap
...for extra warning goodness.
Signed-off-by: Jeff Layton jlay...@samba.org
---
Makefile.am |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/Makefile.am b/Makefile.am
index 1cf7d23..c53c9ec 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -1,4 +1,4 @@
-AM_CFLAGS = -Wall
...to silence -Wextra warnings.
Signed-off-by: Jeff Layton jlay...@samba.org
---
mtab.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/mtab.c b/mtab.c
index 70789bc..de1aabd 100644
--- a/mtab.c
+++ b/mtab.c
@@ -51,12 +51,12 @@ static int lockfile_fd = -1;
static int
To silence these warnings:
data_blob.c: In function ‘data_blob_hex_string_lower’:
data_blob.c:155:16: warning: comparison between signed and unsigned integer
expressions
data_blob.c: In function ‘data_blob_hex_string_upper’:
data_blob.c:172:16: warning: comparison between signed and unsigned
gcc says:
cifs.upcall.c: In function ‘cifs_krb5_get_req’:
cifs.upcall.c:261:2: warning: missing initializer
cifs.upcall.c:261:2: warning: (near initialization for ‘in_creds.client’)
cifs.upcall.c: In function ‘main’:
cifs.upcall.c:622:9: warning: missing initializer
cifs.upcall.c:622:9: warning:
On Tue, 27 Jul 2010 11:38:20 -0500 Steve French wrote:
On Tue, Jul 27, 2010 at 11:33 AM, Suresh Jayaraman sjayara...@suse.de wrote:
On 07/27/2010 08:51 PM, Steve French wrote:
scripts/extract-ikconfig could be run against the image (or look at
/proc/config.gz) - but unless there is an
On Thu, 22 Jul 2010 10:24:17 -0700
Linus Torvalds torva...@linux-foundation.org wrote:
Of people can just use xattrs and do it all entirely in user space. I
assume that's what samba does now, even outside of birthtime.
Much as I despise xattrs, this would definitely be my preference.
ctime
27 matches
Mail list logo
