[PATCH] befs : Add small_data support through xattr handlers

2013-12-02 Thread Fabian Frederick
in maintainer list. Signed off by Fabian Frederick diff --git a/fs/befs/befs.h b/fs/befs/befs.h index b266428..02eb7ae 100644 --- a/fs/befs/befs.h +++ b/fs/befs/befs.h @@ -57,6 +57,14 @@ typedef struct befs_sb_info { } befs_sb_info; +typedef struct befs_small_datamem { + u32 type; + char name

[PATCH] befs : Add small_data support through xattr handlers (v2)

2013-12-03 Thread Fabian Frederick
. Signed off by Fabian Frederick diff --git a/fs/befs/befs.h b/fs/befs/befs.h index b266428..02eb7ae 100644 --- a/fs/befs/befs.h +++ b/fs/befs/befs.h @@ -57,6 +57,14 @@ typedef struct befs_sb_info { } befs_sb_info; +typedef struct befs_small_datamem { + u32 type; + char name

[PATCH v2] BEFS: Logging clean-up

2014-03-12 Thread Fabian Frederick
-All printk(KERN_foo converted to pr_foo(). -Add pr_fmt and remove redundant prefixes. -Convert befs_() to va_format (based on patch by Joe Perches). -Remove non standard %Lu. -Use __func__ for all debugging. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/befs/Makefile | 2 +- fs

[PATCH 1/1] fs/ceph/inode.c: Remove get/set acl on symlinks

2014-03-15 Thread Fabian Frederick
Remove unsupported symlink operations. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/ceph/inode.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/fs/ceph/inode.c b/fs/ceph/inode.c index 32d519d..704d714 100644 --- a/fs/ceph/inode.c +++ b/fs/ceph/inode.c @@ -1616,8 +1616,6 @@ static

[PATCH 1/1] NTFS: Logging clean-up

2014-03-16 Thread Fabian Frederick
-All printk(KERN_foo converted to pr_foo(). -Add pr_fmt and remove redundant prefixes. -Convert spinlock/static array to va_format (inspired by Joe Perches help on previous logging patches). Signed-off-by: Fabian Frederick f...@skynet.be --- fs/ntfs/debug.c | 54

Re: [PATCH 1/1] NTFS: Logging clean-up

2014-03-16 Thread Fabian Frederick
you have test it builds? Andrew, can you please merge this via your patch series? Thanks! I think a few things need fixing first Yes, quite right - I had assume that had been tested! On 16 Mar 2014, at 12:27, Fabian Frederick f...@skynet.be wrote: -All printk(KERN_foo

[PATCH V2] NTFS: Logging clean-up

2014-03-16 Thread Fabian Frederick
is unchanged (still guarded by #if DEBUG)) -Improve !DEBUG to parse all arguments (Joe Perches). -Sparse pr_foo() conversions in super.c NTFS, NTFS-fs prefixes as well as 'warning' and 'error' were removed : pr_foo() automatically adds module name and error level is already specified. Signed-off-by: Fabian

[PATCH V3] NTFS: Logging clean-up

2014-03-17 Thread Fabian Frederick
is still guarded by #if DEBUG) -Improve !DEBUG to parse all arguments (Joe Perches). -Sparse pr_foo() conversions in super.c NTFS, NTFS-fs prefixes as well as 'warning' and 'error' were removed : pr_foo() automatically adds module name and error level is already specified. Signed-off-by: Fabian Frederick

[PATCH 1/1] affs: Add __init to init_inodecache ()

2014-02-21 Thread Fabian Frederick
init_inodecache is only called by __init init_affs_fs Signed-off-by: Fabian Frederick f...@skynet.be --- fs/affs/super.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/affs/super.c b/fs/affs/super.c index d098731..41a7557 100644 --- a/fs/affs/super.c +++ b/fs/affs/super.c

[PATCH 1/1] fs/ext4/inode.c: Add unlikely() to ext4_iget

2014-02-23 Thread Fabian Frederick
Isolate no memory and bad inode cases in ext4_iget. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/ext4/inode.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c index 6e39895..e11c30b9 100644 --- a/fs/ext4/inode.c +++ b/fs

[PATCH 1/1] fs/gfs2/quota.c: replace kmalloc - __vmalloc / memset 0

2014-02-26 Thread Fabian Frederick
Use kzalloc and __vmalloc __GFP_ZERO for clean sd_quota_bitmap allocation. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/gfs2/quota.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/fs/gfs2/quota.c b/fs/gfs2/quota.c index 8bec0e31..1ee4b88 100644 --- a/fs/gfs2

[PATCH 1/1] fs/befs/linuxvfs.c: add __init to befs_init_inodecache()

2014-02-26 Thread Fabian Frederick
init_inodecache is only called by __init init_befs_fs. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/befs/linuxvfs.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/fs/befs/linuxvfs.c b/fs/befs/linuxvfs.c index 845d2d6..ab4c61f 100644 --- a/fs/befs/linuxvfs.c +++ b

[PATCH 1/1] Documentation/filesystems/ntfs.txt: Remove changelog reference

2014-02-27 Thread Fabian Frederick
File was removed in the following commit : 7c821a179f91c3ad52588400ce52a7fb48b9868c (Remove fs/ntfs/ChangeLog) Signed-off-by: Fabian Frederick f...@skynet.be --- Documentation/filesystems/ntfs.txt | 2 -- 1 file changed, 2 deletions(-) diff --git a/Documentation/filesystems/ntfs.txt b

[PATCH 1/1] fs/affs/dir.c: unlock/brelse dir on failure + code clean-up

2014-02-28 Thread Fabian Frederick
in the commit above). Signed-off-by: Fabian Frederick f...@skynet.be --- fs/affs/dir.c | 28 ++-- 1 file changed, 18 insertions(+), 10 deletions(-) diff --git a/fs/affs/dir.c b/fs/affs/dir.c index f1eba8c..cbbda47 100644 --- a/fs/affs/dir.c +++ b/fs/affs/dir.c @@ -52,8

[PATCH 1/1] affs: AFFS_NO_TRUNCATE added to Kconfig

2014-03-03 Thread Fabian Frederick
Filename truncate process is defined in affs.h. This patch exports current define to Kconfig and removes AFFS_MAX_PREALLOC which is never used. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/affs/Kconfig| 12 fs/affs/affs.h | 9 - fs/affs/amigaffs.c | 2

Re: [PATCH 1/1] affs: AFFS_NO_TRUNCATE added to Kconfig

2014-03-03 Thread Fabian Frederick
On Mon, 3 Mar 2014 14:43:27 -0500 Dave Jones da...@redhat.com wrote: On Mon, Mar 03, 2014 at 06:22:18PM +0800, Fabian Frederick wrote: Filename truncate process is defined in affs.h. This patch exports current define to Kconfig and removes AFFS_MAX_PREALLOC which is never used

[PATCH 1/1] fs/quota/Kconfig: Update filesystems

2014-03-04 Thread Fabian Frederick
Update Kconfig with a complete list of supported filesystems. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/quota/Kconfig | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/fs/quota/Kconfig b/fs/quota/Kconfig index 880fd98..c51df1d 100644 --- a/fs/quota/Kconfig

[PATCH 1/1] fs: gfs2: global conversion to pr_foo()

2014-03-05 Thread Fabian Frederick
-All printk(KERN_foo converted to pr_foo(). -Messages updated to fit in 80 columns. -fs_macros converted as well. -fs_printk removed. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/gfs2/dir.c| 7 --- fs/gfs2/glock.c | 19 ++- fs/gfs2/lock_dlm.c | 6

Re: [PATCH 1/1] fs: gfs2: global conversion to pr_foo()

2014-03-06 Thread Fabian Frederick
On Thu, 06 Mar 2014 15:44:04 + Steven Whitehouse swhit...@redhat.com wrote: Hi, On Wed, 2014-03-05 at 22:06 +0800, Fabian Frederick wrote: -All printk(KERN_foo converted to pr_foo(). -Messages updated to fit in 80 columns. -fs_macros converted as well. -fs_printk removed

[PATCH v2 linux-next] mtd: block2mtd: check device size

2014-03-06 Thread Fabian Frederick
fixme applied : check device size is a multiple of erasesize. Signed-off-by: Fabian Frederick f...@skynet.be --- drivers/mtd/devices/block2mtd.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/mtd/devices/block2mtd.c b/drivers/mtd/devices/block2mtd.c index

Re: [PATCH 3/4] mtd: block2mtd: check device size

2014-03-06 Thread Fabian Frederick
On Thu, 6 Mar 2014 00:54:22 -0800 Brian Norris computersforpe...@gmail.com wrote: Hi, On Wed, Jan 29, 2014 at 08:51:05PM +0800, Fabian Frederick wrote: On Wed, 29 Jan 2014 07:04:25 -0300 Ezequiel Garcia ezequiel.gar...@free-electrons.com wrote: I saw you sent a v2 for one

Re: [PATCH 1/1] fs: gfs2: global conversion to pr_foo()

2014-03-06 Thread Fabian Frederick
On Thu, 06 Mar 2014 17:57:42 + Steven Whitehouse swhit...@redhat.com wrote: Hi, On Thu, 2014-03-06 at 17:24 +0800, Fabian Frederick wrote: On Thu, 06 Mar 2014 15:44:04 + Steven Whitehouse swhit...@redhat.com wrote: Hi, On Wed, 2014-03-05 at 22:06 +0800, Fabian

[PATCH 1/1 linux-next] fs/ceph/super.c: Undefined field set to 0 + clean-up

2014-03-07 Thread Fabian Frederick
-statfs(2) /undefined -Don't recalculate f_frsize and f_bavail. -Remove ?? comment. -Use defined CEPH_BLOCK. -Directly work with 12 for 4Kb conversions (like in dir.c) Signed-off-by: Fabian Frederick f...@skynet.be --- fs/ceph/super.c | 14 +++--- 1 file changed, 7 insertions(+), 7

[PATCH v2] affs: mount option to avoid name truncates

2014-03-09 Thread Fabian Frederick
Filename truncate process is defined in affs.h This patch adds 'nofilenametruncate' mount option and removes AFFS_MAX_PREALLOC which is never used. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/affs/affs.h | 20 fs/affs/amigaffs.c | 23

[PATCH 1/1] fs/bfs/inode.c: add __init to init_inodecache()

2014-03-10 Thread Fabian Frederick
init_inodecache is only called by __init init_bfs_fs Signed-off-by: Fabian Frederick f...@skynet.be --- fs/bfs/inode.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/bfs/inode.c b/fs/bfs/inode.c index 8defc6b..0693cbf 100644 --- a/fs/bfs/inode.c +++ b/fs/bfs/inode.c

[PATCH 1/1] CODA: add __init to init_inodecache()

2014-03-10 Thread Fabian Frederick
init_inodecache is only called by __init init_coda Signed-off-by: Fabian Frederick f...@skynet.be --- fs/coda/coda_int.h | 2 +- fs/coda/inode.c| 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/coda/coda_int.h b/fs/coda/coda_int.h index b7143cf..381c993 100644

[PATCH 1/1] HFSPLUS: add __init to hfsplus_create_attr_tree_cache()

2014-03-10 Thread Fabian Frederick
hfsplus_create_attr_tree_cache is only called by __init init_hfsplus_fs Signed-off-by: Fabian Frederick f...@skynet.be --- fs/hfsplus/attributes.c | 2 +- fs/hfsplus/hfsplus_fs.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/hfsplus/attributes.c b/fs/hfsplus

[PATCH 1/1] fs/isofs/inode.c add __init to init_inodecache()

2014-03-10 Thread Fabian Frederick
init_inodecache is only called by __init init_iso9660_fs Signed-off-by: Fabian Frederick f...@skynet.be --- fs/isofs/inode.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/isofs/inode.c b/fs/isofs/inode.c index 4a9e10e..7df1914 100644 --- a/fs/isofs/inode.c +++ b/fs/isofs

[PATCH v3] AFFS: mount option to avoid filename truncates

2014-03-11 Thread Fabian Frederick
be enabled but needs module compilation. This patch adds 'nofilenametruncate' mount option so that user can easily activate that feature and avoid a lot of problems (eg overwrite files ...) Signed-off-by: Fabian Frederick f...@skynet.be --- Documentation/filesystems/affs.txt | 9 ++--- fs/affs/affs.h

[PATCH 1/1] BEFS: Global conversion to pr_foo()

2014-03-11 Thread Fabian Frederick
-All printk(KERN_foo converted to pr_foo(). -Add pr_fmt and remove redundant prefixes. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/befs/debug.c| 14 +++--- fs/befs/linuxvfs.c | 27 +-- 2 files changed, 20 insertions(+), 21 deletions(-) diff --git

[PATCH 1/1] sys_sysfs: Add CONFIG_SYSFS_SYSCALL

2014-02-13 Thread Fabian Frederick
sys_sysfs is an obsolete system call no longer supported by libc ... -This patch adds a default CONFIG_SYSFS_SYSCALL=y -Option can be turned off in expert mode. -cond_syscall added to kernel/sys_ni.c Signed-off-by: Fabian Frederick f...@skynet.be --- fs/filesystems.c | 2 ++ init/Kconfig

[PATCH 1/1] fs: xattr: Don't display attributes without read access

2014-02-16 Thread Fabian Frederick
Any user can display extented attribute names without read access. eg: attr -l filename This patch checks inode_permission in listxattr common function before executing vfs_listxattr. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/xattr.c | 4 1 file changed, 4 insertions(+) diff

Re: [PATCH 1/1] fs: xattr: Don't display attributes without read access

2014-02-18 Thread Fabian Frederick
On Tue, 18 Feb 2014 15:16:50 -0800 Andrew Morton a...@linux-foundation.org wrote: On Sun, 16 Feb 2014 20:31:01 +0800 Fabian Frederick f...@skynet.be wrote: Any user can display extented attribute names without read access. eg: attr -l filename This patch checks inode_permission

[PATCH 1/1] befs: Replace kmalloc/memset 0 by kzalloc

2014-02-19 Thread Fabian Frederick
Use kzalloc for clean fs_info allocation like other filesystems. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/befs/linuxvfs.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/fs/befs/linuxvfs.c b/fs/befs/linuxvfs.c index 845d2d6..e0002e0 100644 --- a/fs/befs

Re: [PATCH 1/1] sys_sysfs: Add CONFIG_SYSFS_SYSCALL

2014-02-19 Thread Fabian Frederick
On Thu, 13 Feb 2014 14:43:19 -0800 Andrew Morton a...@linux-foundation.org wrote: On Thu, 13 Feb 2014 17:31:41 -0500 Dave Jones da...@redhat.com wrote: On Thu, Feb 13, 2014 at 10:09:42PM +0800, Fabian Frederick wrote: sys_sysfs is an obsolete system call no longer supported by libc

Re: [PATCH 1/1] fs: xattr: Don't display attributes without read access

2014-02-19 Thread Fabian Frederick
On Tue, 18 Feb 2014 15:16:50 -0800 Andrew Morton a...@linux-foundation.org wrote: On Sun, 16 Feb 2014 20:31:01 +0800 Fabian Frederick f...@skynet.be wrote: Any user can display extented attribute names without read access. eg: attr -l filename This patch checks inode_permission

[PATCH 1/1] afs: afs_alloc_inode: use kmem_cache_zalloc

2014-02-20 Thread Fabian Frederick
afs_vnode is currently cleared with 2 memsets after allocation and 1 in constructor (afs_i_init_once). -This patch calls zalloc for explicit zero fill. -Fix some typos. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/afs/super.c | 12 +--- 1 file changed, 5

[PATCH v2] afs: afs_alloc_inode: use kmem_cache_zalloc

2014-02-20 Thread Fabian Frederick
This patch calls zalloc for explicit zero fill and removes all memset 0. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/afs/super.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/fs/afs/super.c b/fs/afs/super.c index c486155..aff91ee 100644 --- a/fs/afs/super.c

Re: Has slab ctor operation changed? -- was [PATCH 1/1] afs: afs_alloc_inode: use kmem_cache_zalloc

2014-02-20 Thread Fabian Frederick
On Thu, 20 Feb 2014 22:23:15 + David Howells dhowe...@redhat.com wrote: Fabian Frederick f...@skynet.be wrote: afs_vnode is currently cleared with 2 memsets after allocation and 1 in constructor (afs_i_init_once). -This patch calls zalloc for explicit zero fill. Ummm

[RFC 1/1] fs: Quota: Refresh Kconfig

2014-02-21 Thread Fabian Frederick
Update Kconfig with a complete list of supported filesystems. (Other local filesystems concerned by quota and/or journaled quotas ?) --- fs/quota/Kconfig | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/fs/quota/Kconfig b/fs/quota/Kconfig index 880fd98..7339003 100644

Re: [PATCH 1/1] kernel: panic: display reason at end + pr_emerg

2014-02-12 Thread Fabian Frederick
On Tue, 11 Feb 2014 15:34:45 -0800 Andrew Morton a...@linux-foundation.org wrote: On Sat, 8 Feb 2014 15:45:22 +0800 Fabian Frederick f...@skynet.be wrote: Currently, booting without initrd specified on 80x25 screen gives a call trace followed by atkbd : Spurious ACK. Original message

[RFC 1/2] Arch: Kconfig: Disable obsolete syscalls

2014-02-12 Thread Fabian Frederick
This patch adds a generic configuration flag to disable obsolete system calls (ie no longer supported by current c libraries). Signed-off-by: Fabian Frederick f...@skynet.be --- arch/Kconfig | 9 + 1 file changed, 9 insertions(+) diff --git a/arch/Kconfig b/arch/Kconfig index 80bbb8c

[RFC 2/2] Arch: Kconfig: Disable obsolete syscalls

2014-02-12 Thread Fabian Frederick
fs: sysfs syscall: Disable if DISABLE_OBSOLETE_SYSCALLS sysfs syscall is obsolete (cf man sysfs). Don't define fs_index, fs_name, fs_maxindex functions when DISABLE_OBSOLETE_SYSCALLS is defined. In that case, SYSCALL_DEFINE3(sysfs...) returns automatically -EINVAL. Signed-off-by: Fabian

Re: [RFC 2/2] Arch: Kconfig: Disable obsolete syscalls

2014-02-12 Thread Fabian Frederick
On Wed, 12 Feb 2014 13:13:14 -0800 Andrew Morton a...@linux-foundation.org wrote: On Wed, 12 Feb 2014 21:04:40 +0800 Fabian Frederick f...@skynet.be wrote: fs: sysfs syscall: Disable if DISABLE_OBSOLETE_SYSCALLS sysfs syscall is obsolete (cf man sysfs). Don't define fs_index

[PATCH 1/1] fs/freevfxs/vfxs_lookup.c: Update function comment

2014-03-18 Thread Fabian Frederick
nameidata was replaced by flags in 00cd8dd3bf95f2 (stop passing nameidata to -lookup()) Signed-off-by: Fabian Frederick f...@skynet.be --- fs/freevxfs/vxfs_lookup.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/freevxfs/vxfs_lookup.c b/fs/freevxfs/vxfs_lookup.c index

[PATCH v2] fs/freevxfs/vxfs_lookup.c: Update function comment

2014-03-18 Thread Fabian Frederick
nameidata was replaced by flags in 00cd8dd3bf95f2 (stop passing nameidata to -lookup()) Signed-off-by: Fabian Frederick f...@skynet.be --- fs/freevxfs/vxfs_lookup.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/freevxfs/vxfs_lookup.c b/fs/freevxfs/vxfs_lookup.c index

[PATCH 1/1] MINIX: specific inode data access standardization

2014-03-18 Thread Fabian Frederick
minix_i uppercase conversion (like majority of other filesystems). Signed-off-by: Fabian Frederick f...@skynet.be --- fs/minix/bitmap.c | 2 +- fs/minix/inode.c| 10 +- fs/minix/itree_v1.c | 2 +- fs/minix/itree_v2.c | 2 +- fs/minix/minix.h| 2 +- 5 files changed, 9

[PATCH 1/1] mm/filemap.c: ra_submit inline

2014-03-20 Thread Fabian Frederick
f9acc8c7b35a10 (readahead: sanify file_ra_state names) left ra_submit with a single function call. Add 'inline' to save some stack (suggested by Andrew Morton). Signed-off-by: Fabian Frederick f...@skynet.be --- mm/readahead.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 1/1] fs/reiserfs/super.c: add __init to init_inodecache

2014-03-21 Thread Fabian Frederick
init_inodecache is only called by __init init_reiserfs_fs. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/reiserfs/super.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/reiserfs/super.c b/fs/reiserfs/super.c index 2c80335..7035087 100644 --- a/fs/reiserfs/super.c

[RFC 1/1] fs/reiserfs/journal.c: Remove obsolete __GFP_NOFAIL

2014-03-21 Thread Fabian Frederick
Loop around congestion_wait on allocation failure/alloc_journal_list like already fixed in other FS. (Does it need returning -ENOMEM after some retries ?) Signed-off-by: Fabian Frederick f...@skynet.be --- fs/reiserfs/journal.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions

Re: [PATCH 1/1] mm/filemap.c: ra_submit inline

2014-03-21 Thread Fabian Frederick
On Thu, 20 Mar 2014 14:58:29 -0700 Andrew Morton a...@linux-foundation.org wrote: On Thu, 20 Mar 2014 22:53:25 +0100 Fabian Frederick f...@skynet.be wrote: f9acc8c7b35a10 (readahead: sanify file_ra_state names) left ra_submit with a single function call. Add 'inline' to save some

[PATCH 1/1 RESEND] fs/ceph/inode.c: Remove get/set acl on symlinks

2014-03-21 Thread Fabian Frederick
Remove unsupported symlink operations. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/ceph/inode.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/fs/ceph/inode.c b/fs/ceph/inode.c index 32d519d..704d714 100644 --- a/fs/ceph/inode.c +++ b/fs/ceph/inode.c @@ -1616,8 +1616,6 @@ static

[PATCH V3 RESEND] NTFS: Logging clean-up

2014-03-21 Thread Fabian Frederick
is still guarded by #if DEBUG) -Improve !DEBUG to parse all arguments (Joe Perches). -Sparse pr_foo() conversions in super.c NTFS, NTFS-fs prefixes as well as 'warning' and 'error' were removed : pr_foo() automatically adds module name and error level is already specified. Signed-off-by: Fabian Frederick

Re: [RFC 1/1] fs/reiserfs/journal.c: Remove obsolete __GFP_NOFAIL

2014-03-21 Thread Fabian Frederick
On Fri, 21 Mar 2014 13:00:55 -0700 Andrew Morton a...@linux-foundation.org wrote: On Fri, 21 Mar 2014 17:18:30 +0100 Fabian Frederick f...@skynet.be wrote: Loop around congestion_wait on allocation failure/alloc_journal_list like already fixed in other FS. ... --- a/fs/reiserfs

Re: [RFC 1/1] fs/reiserfs/journal.c: Remove obsolete __GFP_NOFAIL

2014-03-22 Thread Fabian Frederick
On Sat, 22 Mar 2014 10:55:24 -0700 Andrew Morton a...@linux-foundation.org wrote: On Sat, 22 Mar 2014 13:32:07 -0400 ty...@mit.edu wrote: On Sat, Mar 22, 2014 at 01:26:06PM -0400, ty...@mit.edu wrote: Well. Converting an existing retry-for-ever caller to GFP_NOFAIL is good. Adding

[PATCH 0/3] QNX6 logging clean-up

2014-06-05 Thread Fabian Frederick
This small patchset converts QNX6 to current logging functions. Fabian Frederick (3): FS/QNX6: Convert printk to pr_foo() FS/QNX6: use pr_fmt and __func__ in logging FS/QNX6: update debugging to current functions. fs/qnx6/Makefile| 1 + fs/qnx6/dir.c | 26 +++--- fs

[PATCH 3/3] FS/QNX6: update debugging to current functions.

2014-06-05 Thread Fabian Frederick
Perches j...@perches.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- fs/qnx6/Makefile | 1 + fs/qnx6/dir.c| 12 ++-- fs/qnx6/inode.c | 37 + fs/qnx6/namei.c | 6 +++--- fs/qnx6/qnx6.h | 6

[PATCH 2/3] FS/QNX6: use pr_fmt and __func__ in logging

2014-06-05 Thread Fabian Frederick
remove qnx6: and qnx6: from each logging instruction Cc: Joe Perches j...@perches.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- fs/qnx6/dir.c | 12 ++-- fs/qnx6/inode.c | 44 ++-- fs

[PATCH 1/3] FS/QNX6: Convert printk to pr_foo()

2014-06-05 Thread Fabian Frederick
use current logging functions. coalesce formats Cc: Joe Perches j...@perches.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- fs/qnx6/dir.c | 14 ++-- fs/qnx6/inode.c | 62

[PATCH 1/1] kernel/trace/ring_buffer.c: kernel-doc warning fixes

2014-06-05 Thread Fabian Frederick
Also fixes some typo. Cc: Steven Rostedt rost...@goodmis.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/trace/ring_buffer.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/kernel/trace/ring_buffer.c b

[PATCH 1/1] kernel/trace/blktrace.c: logging clean-up

2014-06-05 Thread Fabian Frederick
-Remove Warning: from logging (already in log level) -Convert pr_warning to standard pr_warn -Define pr_fmt(fmt) fmt to avoid any future default fmt definition Cc: Steven Rostedt rost...@goodmis.org Cc: Joe Perches j...@perches.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian

[PATCH 1/1] kernel/locking/rtmutex-tester.c: kernel-doc warning fix

2014-06-05 Thread Fabian Frederick
attr was added by commit 4a0b2b4dbe1335 (sysdev: Pass the attribute to the low level sysdev show/store function) and is unused here. Cc: Peter Zijlstra pet...@infradead.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/locking/rtmutex

[PATCH 1/1] kernel/locking/lockdep_proc.c: replace seq_printf and seq_puts

2014-06-05 Thread Fabian Frederick
replace seq_printf by seq_puts and seq_puts by seq_putc where possible Cc: Peter Zijlstra pet...@infradead.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/locking/lockdep_proc.c | 20 ++-- 1 file changed, 10 insertions

[PATCH 1/1] include/linux/memblock.h: add __init to memblock_set_bottom_up

2014-06-07 Thread Fabian Frederick
memblock_set_bottom_up is only called by __init cmdline_parse_movable_node and __init numa_init. Cc: Andrew Morton a...@linux-foundation.org Cc: Tang Chen tangc...@cn.fujitsu.com Signed-off-by: Fabian Frederick f...@skynet.be --- This is untested. include/linux/memblock.h | 4 ++-- 1 file

[PATCH 1/1] mm/zswap.c: add __init to zswap_entry_cache_destroy

2014-06-07 Thread Fabian Frederick
zswap_entry_cache_destroy is only called by __init init_zswap This patch also fixes function name zswap_entry_cache_ s/destory/destroy Cc: Seth Jennings sjenni...@variantweb.net Cc: linux...@kvack.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be

[PATCH 1/1] mm/slab.c: add __init to init_lock_keys

2014-06-07 Thread Fabian Frederick
init_lock_keys is only called by __init kmem_cache_init_late Cc: Christoph Lameter c...@linux-foundation.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- mm/slab.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mm/slab.c

[PATCH 1/1] kernel/trace/trace_events.c: logging clean-up

2014-06-07 Thread Fabian Frederick
-Convert pr_warning to standard pr_warn -Define pr_fmt(fmt) fmt to avoid any future default fmt definition Cc: Steven Rostedt rost...@goodmis.org Cc: Joe Perches j...@perches.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/trace

[PATCH 1/1] drivers/mmc/host/dw_mmc.c: use ARRAY_SIZE instead of sizeof/sizeof[0]

2014-06-07 Thread Fabian Frederick
Use kernel.h definition. Cc: Seungwon Jeon tgih@samsung.com Cc: Jaehoon Chung jh80.ch...@samsung.com Cc: linux-...@vger.kernel.org Signed-off-by: Fabian Frederick f...@skynet.be --- This is untested. drivers/mmc/host/dw_mmc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 1/1 linux-next] drivers/net/ethernet/mellanox/mlx4/main.c: use ARRAY_SIZE instead of sizeof/sizeof[0]

2014-06-07 Thread Fabian Frederick
Use kernel.h definition. Cc: David S. Miller da...@davemloft.net Cc: Amir Vadai am...@mellanox.com Signed-off-by: Fabian Frederick f...@skynet.be --- drivers/net/ethernet/mellanox/mlx4/main.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/drivers/net/ethernet

[PATCH 1/1] fs/proc/kcore.c: use PAGE_ALIGN instead of ALIGN(PAGE_SIZE)

2014-06-07 Thread Fabian Frederick
use mm.h definition Cc: Andrew Morton a...@linux-foundation.org Cc: Xishi Qiu qiuxi...@huawei.com Signed-off-by: Fabian Frederick f...@skynet.be --- fs/proc/kcore.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/proc/kcore.c b/fs/proc/kcore.c index 39e6ef3..6df8d07 100644

[PATCH 1/1] kernel/trace/ftrace.c: replace kmalloc/snprintf by kasprintf

2014-06-09 Thread Fabian Frederick
Cc: Steven Rostedt rost...@goodmis.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/trace/ftrace.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index 4a54a25..92a3316

[PATCH 1/1] fs/cachefiles/daemon.c: remove unnecessary tests on unsigned values

2014-06-09 Thread Fabian Frederick
size_t and unsigned long values are never 0 Cc: David Howells dhowe...@redhat.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- fs/cachefiles/daemon.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/fs/cachefiles

[PATCH 1/1] drivers/edac/edac_module.c: remove unnecessary test on unsigned value

2014-06-09 Thread Fabian Frederick
unsigned long value is never 0 Cc: Doug Thompson dougthomp...@xmission.com Cc: Borislav Petkov b...@alien8.de Signed-off-by: Fabian Frederick f...@skynet.be --- drivers/edac/edac_module.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/edac/edac_module.c b/drivers

[PATCH 1/1] KERNEL: add __attribute__ in c file

2014-05-07 Thread Fabian Frederick
: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- include/linux/module.h | 4 ++-- kernel/module.c| 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/include/linux/module.h b/include/linux/module.h index f520a76..05ab8f3 100644

[PATCH 1/1] kernel/profile.c: use static const char instead of static char

2014-05-07 Thread Fabian Frederick
schedstr, sleepstr and kvmstr are only used in strcmp strlen Cc: Andrew Morton a...@linux-foundation.org Cc: Paul Gortmaker paul.gortma...@windriver.com Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/profile.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[PATCH 1/1] kernel/res_counter.c: replace simple_strtoull by kstrtoull

2014-05-07 Thread Fabian Frederick
Cc: Andrew Morton a...@linux-foundation.org Cc: Michal Hocko mho...@suse.cz Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/res_counter.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/kernel/res_counter.c b/kernel/res_counter.c index 51dbac6..09c1a6a 100644

[PATCH 1/1] kernel/seccomp.c: kernel-doc warning fix

2014-05-07 Thread Fabian Frederick
+ fix small typo Cc: David S. Miller da...@davemloft.net Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/seccomp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/seccomp.c b/kernel/seccomp.c index b35c215

[PATCH 1/2] kernel/stop_machine.c: kernel-doc warning fix

2014-05-07 Thread Fabian Frederick
Cc: Peter Zijlstra pet...@infradead.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/stop_machine.c | 1 + 1 file changed, 1 insertion(+) diff --git a/kernel/stop_machine.c b/kernel/stop_machine.c index 01fbae5..695f0c6 100644

[PATCH 2/2] kernel/stop_machine.c: remove false assignment to static

2014-05-07 Thread Fabian Frederick
This patch also fixes function prototype over 80 characters Cc: Peter Zijlstra pet...@infradead.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/stop_machine.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/kernel

[PATCH V2] MM: make vmpressure_win dynamic

2014-05-07 Thread Fabian Frederick
-off-by: Fabian Frederick f...@skynet.be --- V2: Suggestions by Andrew Morton -Expose vmpressure_win -Remove spinlock include/linux/swap.h | 4 kernel/sysctl.c | 10 ++ mm/vmpressure.c | 4 +--- mm/vmstat.c | 6 ++ 4 files changed, 21 insertions(+), 3

Re: [PATCH 2/2] kernel/stop_machine.c: remove false assignment to static

2014-05-07 Thread Fabian Frederick
On Wed, 7 May 2014 23:04:11 +0200 Peter Zijlstra pet...@infradead.org wrote: On Wed, May 07, 2014 at 10:46:56PM +0200, Fabian Frederick wrote: This patch also fixes function prototype over 80 characters And does it wrong.. Also does it really matter to GCC that we init the bool? Surely

[PATCH V2] kernel/res_counter.c: replace simple_strtoull by kstrtoull

2014-05-08 Thread Fabian Frederick
Thanks to Andrew Morton for commenting first version. Cc: Andrew Morton a...@linux-foundation.org Cc: Michal Hocko mho...@suse.cz Signed-off-by: Fabian Frederick f...@skynet.be --- V2: Suggestions by Andrew Morton -use internal rc variable -simplify if condition kernel/res_counter.c | 4 ++-- 1

[PATCH 1/1] kernel/tracepoint.c: kernel-doc fixes

2014-05-08 Thread Fabian Frederick
Cc: Steven Rostedt rost...@goodmis.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/tracepoint.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/kernel/tracepoint.c b/kernel/tracepoint.c index ac5b23c..1b0a61c 100644 --- a/kernel

[PATCH V2] kernel/kprobes.c: code clean-up

2014-05-08 Thread Fabian Frederick
-printk to pr_foo() -Remove null static initialization -Simplify __get_valid_kprobe (by Joe Perches) Cc: Ananth N Mavinakayanahalli ana...@in.ibm.com Cc: Joe Perches j...@perches.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- V2: Simplify

[PATCH 1/1] kernel/hung_task.c: convert simple_strtoul to kstrtouint

2014-05-09 Thread Fabian Frederick
sysctl_hung_task_panic has been changed to unsigned int. use kstrtouint instead of obsolete simple_strtoul Inspired-By: Andrew Morton a...@linux-foundation.org Cc: Ingo Molnar mi...@kernel.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel

[PATCH 1/1] kernel/power/hibernate.c: convert simple_strtoul to kstrtoul

2014-05-09 Thread Fabian Frederick
Inspired-By: Andrew Morton a...@linux-foundation.org Cc: Rafael J. Wysocki r...@rjwysocki.net Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/power/hibernate.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/kernel

[PATCH 1/1] kernel/time/ntp.c: convert simple_strtol to kstrtol

2014-05-09 Thread Fabian Frederick
Inspired-By: Andrew Morton a...@linux-foundation.org Cc: John Stultz john.stu...@linaro.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/time/ntp.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/kernel/time/ntp.c b

[PATCH 1/1] kernel/user_namespace.c: kernel-doc/checkpatch fixes

2014-05-09 Thread Fabian Frederick
-uid-gid -split some function declarations -if/then/else warning Cc: Oleg Nesterov o...@redhat.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/user_namespace.c | 33 - 1 file changed, 20 insertions(+), 13

[PATCH 1/1] kernel/utsname_sysctl.c: replace obsolete __initcall by device_initcall

2014-05-09 Thread Fabian Frederick
Also fixes checkpatch warnings on proc_dostring function parameters Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/utsname_sysctl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/utsname_sysctl.c b/kernel

[PATCH 1/1] kernel/watchdog.c: convert printk/pr_warning to pr_foo()

2014-05-09 Thread Fabian Frederick
Replace some obsolete functions. Cc: Frederic Weisbecker fweis...@gmail.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/watchdog.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/kernel/watchdog.c b/kernel

[PATCH V2] kernel/time/ntp.c: convert simple_strtol to kstrtol

2014-05-09 Thread Fabian Frederick
Replace obsolete function Inspired-By: Andrew Morton a...@linux-foundation.org Cc: John Stultz john.stu...@linaro.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- V2 -Add patch description kernel/time/ntp.c | 5 - 1 file

[PATCH 1/1] kernel/reboot.c: convert simple_strtoul to kstrtoint

2014-05-09 Thread Fabian Frederick
Replace obsolete function. kstrtoint is used as reboot_cpu is an integer. Inspired-By: Andrew Morton a...@linux-foundation.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/reboot.c | 21 ++--- 1 file changed, 14 insertions

[PATCH V3] MM: make vmpressure_win dynamic

2014-05-10 Thread Fabian Frederick
Signed-off-by: Fabian Frederick f...@skynet.be --- V3: -Further description -sysctl handler / ulong -Cc list V2: Suggestions by Andrew Morton -Expose vmpressure_win -Remove spinlock include/linux/swap.h | 4 kernel/sysctl.c | 9 + mm/vmpressure.c | 4 +--- mm

Re: [PATCH 1/1] KERNEL: add __attribute__ in c file

2014-05-10 Thread Fabian Frederick
On Wed, 7 May 2014 15:25:57 -0700 Andrew Morton a...@linux-foundation.org wrote: On Wed, 7 May 2014 22:33:39 +0200 Fabian Frederick f...@skynet.be wrote: This patch fixes a sparse warning about noreturn attribute only featuring in module.h kernel/module.c:212:6: error: symbol

[PATCH V2 RESEND] ARCH/M68K/AMIGA: convert printk(foo to pr_foo()

2014-05-10 Thread Fabian Frederick
-no level printk converted to pr_warn/pr_info -fixed a small identation problem This is untested Cc: Geert Uytterhoeven ge...@linux-m68k.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- v2: use pr_cont after pr_ without newline (suggested

[PATCH 1/1 RESEND] arch/m68k/apollo/config.c: convert printk to pr_foo()

2014-05-10 Thread Fabian Frederick
no level printk converted to pr_info This is untested Cc: Geert Uytterhoeven ge...@linux-m68k.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- arch/m68k/apollo/config.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions

[PATCH 1/1 RESEND] arch/m68k/hp300/config.c: convert printk to pr_foo()

2014-05-10 Thread Fabian Frederick
This patch also fixes some checkpatch warnings This is untested Cc: Geert Uytterhoeven ge...@linux-m68k.org Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- arch/m68k/hp300/config.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions

[PATCH 1/1 RESEND] drivers/pnp/resource.c: remove positive test on unsigned values

2014-05-10 Thread Fabian Frederick
irq and dma are both resource_size_t (derived from phys_addr_t - unsigned) Cc: Rafael J. Wysocki rafael.j.wyso...@intel.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- drivers/pnp/resource.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[PATCH 1/1 RESEND] drivers/scsi/sd.c: remove positive test on unsigned value

2014-05-10 Thread Fabian Frederick
val is unsigned. Cc: James E.J. Bottomley jbottom...@parallels.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- drivers/scsi/sd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c index

[PATCH 1/1] FS/EXT4: use generic kvfree

2014-05-10 Thread Fabian Frederick
Remove ext4_kvfree and use generic one added in 39f1f78d53b9bcb (nick kvfree() from apparmor) Cc: Theodore Ts'o ty...@mit.edu Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- fs/ext4/ext4.h| 1 - fs/ext4/mballoc.c | 7 --- fs/ext4

  1   2   3   4   5   6   7   8   9   10   >