[PATCH 1/2 linux-next] fs/affs: bugfix: enable writes on OFS disks

2017-04-24 Thread Fabian Frederick
found here: https://bugzilla.kernel.org/show_bug.cgi?id=114961 Signed-off-by: Fabian Frederick --- fs/affs/file.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/fs/affs/file.c b/fs/affs/file.c index 0deec9c..e5c5de6 100644 --- a/fs/affs/file.c +++ b/fs/affs/file.c

[PATCH 1/2 linux-next] udf: use octal for permissions

2017-04-23 Thread Fabian Frederick
According to commit f90774e1fd27 ("checkpatch: look for symbolic permissions and suggest octal instead") Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/udf/inode.c | 14 +++--- fs/udf/namei.c | 2 +- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git

[PATCH 2/2 linux-next] udf: use kmap_atomic for memcpy copying

2017-04-23 Thread Fabian Frederick
Use temporary mapping for memory copying operations. To avoid any sleeping problem, mark_inode_dirty(inode) was moved after kunmap() in udf_adinicb_readpage() down_write(>i_data_sem) set before kmap_atomic() in udf_expand_file_adinicb() Signed-off-by: Fabian Frederick <f...@sky

[PATCH 1/2 linux-next] udf: use octal for permissions

2017-04-23 Thread Fabian Frederick
According to commit f90774e1fd27 ("checkpatch: look for symbolic permissions and suggest octal instead") Signed-off-by: Fabian Frederick --- fs/udf/inode.c | 14 +++--- fs/udf/namei.c | 2 +- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/fs/udf/inode.c b/fs/u

[PATCH 2/2 linux-next] udf: use kmap_atomic for memcpy copying

2017-04-23 Thread Fabian Frederick
Use temporary mapping for memory copying operations. To avoid any sleeping problem, mark_inode_dirty(inode) was moved after kunmap() in udf_adinicb_readpage() down_write(>i_data_sem) set before kmap_atomic() in udf_expand_file_adinicb() Signed-off-by: Fabian Frederick --- This is untes

[PATCH 1/1 linux-next] fs/affs: remove node generation check

2017-04-15 Thread Fabian Frederick
node generation has to be stored on disk. AFAICS we won't be able to manage it on AFFS. This patch removes relevant check in affs_nfs_get_inode() Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/affs/namei.c | 5 - 1 file changed, 5 deletions(-) diff --git a/fs/affs/namei.c b/f

[PATCH 1/1 linux-next] fs/affs: remove node generation check

2017-04-15 Thread Fabian Frederick
node generation has to be stored on disk. AFAICS we won't be able to manage it on AFFS. This patch removes relevant check in affs_nfs_get_inode() Signed-off-by: Fabian Frederick --- fs/affs/namei.c | 5 - 1 file changed, 5 deletions(-) diff --git a/fs/affs/namei.c b/fs/affs/namei.c index

[PATCH 1/1 linux-next] fs/affs: bugfix: make symbolic links work again

2017-04-15 Thread Fabian Frederick
de size when reading symbolic link from disk or creating a new one. Result was zero allocation in pagecache. ln -s file symlink ls -lrt file symlink -> This patch adds inode isize information on inode get and symbolic link addition. Signed-off-by: Fabian Frederick <f...@skynet.be> --- f

[PATCH 1/1 linux-next] fs/affs: bugfix: make symbolic links work again

2017-04-15 Thread Fabian Frederick
de size when reading symbolic link from disk or creating a new one. Result was zero allocation in pagecache. ln -s file symlink ls -lrt file symlink -> This patch adds inode isize information on inode get and symbolic link addition. Signed-off-by: Fabian Frederick --- fs/affs/inode.c | 1 + f

[PATCH 1/1 linux-next] fs/affs: import amigaffs.h

2017-04-15 Thread Fabian Frederick
Have that file in global include/linux is not needed. Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/affs/affs.h| 2 +- {include/linux => fs/affs}/amigaffs.h | 0 2 files changed, 1 insertion(+), 1 deletion(-) rename {include/linux => fs/affs}/ami

[PATCH 1/1 linux-next] fs/affs: import amigaffs.h

2017-04-15 Thread Fabian Frederick
Have that file in global include/linux is not needed. Signed-off-by: Fabian Frederick --- fs/affs/affs.h| 2 +- {include/linux => fs/affs}/amigaffs.h | 0 2 files changed, 1 insertion(+), 1 deletion(-) rename {include/linux => fs/affs}/amigaffs.h (100%) diff --gi

[PATCH 1/1 linux-next] fs/affs: free affs_sb_info in put_super()

2017-03-30 Thread Fabian Frederick
kill_block_super() calls generic_shutdown_super() calling FS put_user() where we generally free superblock specific information. This removes unneeded affs_kill_sb() sbi is automatically reserved in fill_super() so there is no need to test it. Signed-off-by: Fabian Frederick <f...@skynet

[PATCH 1/1 linux-next] fs/affs: free affs_sb_info in put_super()

2017-03-30 Thread Fabian Frederick
kill_block_super() calls generic_shutdown_super() calling FS put_user() where we generally free superblock specific information. This removes unneeded affs_kill_sb() sbi is automatically reserved in fill_super() so there is no need to test it. Signed-off-by: Fabian Frederick --- fs/affs

[PATCH 1/1 linux-next] befs: make export work with cold dcache

2017-03-28 Thread Fabian Frederick
based on commit b3b42c0deaa1 ("fs/affs: make export work with cold dcache") This adds get_parent function so that nfs client can still work after cache drop (Tested on NFS v4 with echo 3 > /proc/sys/vm/drop_caches) Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/b

[PATCH 1/1 linux-next] befs: make export work with cold dcache

2017-03-28 Thread Fabian Frederick
based on commit b3b42c0deaa1 ("fs/affs: make export work with cold dcache") This adds get_parent function so that nfs client can still work after cache drop (Tested on NFS v4 with echo 3 > /proc/sys/vm/drop_caches) Signed-off-by: Fabian Frederick --- fs/befs/linuxvfs.c | 15 +

Re: [PATCH 01/14] locking/atomic: import atomic_dec_not_zero()

2017-02-01 Thread Fabian Frederick
> On 01 February 2017 at 10:25 Peter Zijlstra <pet...@infradead.org> wrote: > > > On Tue, Jan 31, 2017 at 09:55:08PM +0100, Fabian Frederick wrote: > > Once again it's just about readability: > > I feel APIs should be about common use-cases, not about sporadic wei

Re: [PATCH 01/14] locking/atomic: import atomic_dec_not_zero()

2017-02-01 Thread Fabian Frederick
> On 01 February 2017 at 10:25 Peter Zijlstra wrote: > > > On Tue, Jan 31, 2017 at 09:55:08PM +0100, Fabian Frederick wrote: > > Once again it's just about readability: > > I feel APIs should be about common use-cases, not about sporadic weird cases. > > > &q

Re: [PATCH 01/14] locking/atomic: import atomic_dec_not_zero()

2017-01-31 Thread Fabian Frederick
> On 31 January 2017 at 20:17 Peter Zijlstra <pet...@infradead.org> wrote: > > > On Tue, Jan 31, 2017 at 06:41:28PM +0100, Fabian Frederick wrote: > > > > > > > On 31 January 2017 at 11:41 Peter Zijlstra <pet...@infradead.org> wrote: > > >

Re: [PATCH 01/14] locking/atomic: import atomic_dec_not_zero()

2017-01-31 Thread Fabian Frederick
> On 31 January 2017 at 20:17 Peter Zijlstra wrote: > > > On Tue, Jan 31, 2017 at 06:41:28PM +0100, Fabian Frederick wrote: > > > > > > > On 31 January 2017 at 11:41 Peter Zijlstra wrote: > > > > > > > > > On Mon, Jan 30, 2017 at 07:

Re: [PATCH 01/14] locking/atomic: import atomic_dec_not_zero()

2017-01-31 Thread Fabian Frederick
> On 31 January 2017 at 11:41 Peter Zijlstra <pet...@infradead.org> wrote: > > > On Mon, Jan 30, 2017 at 07:39:38PM +0100, Fabian Frederick wrote: > > complementary definition to atomic_inc_not_zero() featured in > > lib/fault-inject.c > > Why? Ma

Re: [PATCH 01/14] locking/atomic: import atomic_dec_not_zero()

2017-01-31 Thread Fabian Frederick
> On 31 January 2017 at 11:41 Peter Zijlstra wrote: > > > On Mon, Jan 30, 2017 at 07:39:38PM +0100, Fabian Frederick wrote: > > complementary definition to atomic_inc_not_zero() featured in > > lib/fault-inject.c > > Why? Maybe this commit message should be

Re: [PATCH 14/14] xfs: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
> On 31 January 2017 at 01:30 "Darrick J. Wong" <darrick.w...@oracle.com> wrote: > > > On Mon, Jan 30, 2017 at 07:48:44PM +0100, Fabian Frederick wrote: > > instead of atomic_add_unless(value, -1, 0) > > > > Signed-off-by: Fabian Frederick <

Re: [PATCH 14/14] xfs: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
> On 31 January 2017 at 01:30 "Darrick J. Wong" wrote: > > > On Mon, Jan 30, 2017 at 07:48:44PM +0100, Fabian Frederick wrote: > > instead of atomic_add_unless(value, -1, 0) > > > > Signed-off-by: Fabian Frederick > > --- > >  fs/xfs/xfs

[PATCH 04/14] m5mols: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick <f...@skynet.be> --- drivers/media/i2c/m5mols/m5mols_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/m5mols/m5mols_core.c b/drivers/media/i2c/m5mols/m5mols_core.c

[PATCH 04/14] m5mols: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick --- drivers/media/i2c/m5mols/m5mols_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/m5mols/m5mols_core.c b/drivers/media/i2c/m5mols/m5mols_core.c index acb804b..3aab2ca

[PATCH 05/14] omap3isp: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick <f...@skynet.be> --- drivers/media/platform/omap3isp/ispstat.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/omap3isp/ispstat.c b/drivers/media/platform/omap3isp/isp

[PATCH 05/14] omap3isp: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick --- drivers/media/platform/omap3isp/ispstat.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/omap3isp/ispstat.c b/drivers/media/platform/omap3isp/ispstat.c index 47cbc7e

[PATCH 09/14] kdb: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick <f...@skynet.be> --- kernel/debug/kdb/kdb_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/debug/kdb/kdb_main.c b/kernel/debug/kdb/kdb_main.c index ca18391..095e6d8 100644 --- a/

[PATCH 08/14] ipmi: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick <f...@skynet.be> --- drivers/char/ipmi/ipmi_msghandler.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/char/ipmi/ipmi_msghandler.c b/drivers/char/ipmi/ipmi_msghandler.c index 9

[PATCH 09/14] kdb: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick --- kernel/debug/kdb/kdb_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/debug/kdb/kdb_main.c b/kernel/debug/kdb/kdb_main.c index ca18391..095e6d8 100644 --- a/kernel/debug/kdb

[PATCH 08/14] ipmi: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick --- drivers/char/ipmi/ipmi_msghandler.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/char/ipmi/ipmi_msghandler.c b/drivers/char/ipmi/ipmi_msghandler.c index 9f69995..961d677 100644

[PATCH 11/14] PM: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick <f...@skynet.be> --- kernel/power/user.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/power/user.c b/kernel/power/user.c index 22df9f7..c478572 100644 --- a/kernel/power/user.c +++ b/

[PATCH 11/14] PM: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick --- kernel/power/user.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/power/user.c b/kernel/power/user.c index 22df9f7..c478572 100644 --- a/kernel/power/user.c +++ b/kernel/power/user.c

[PATCH 10/14] PM / Hibernate: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick <f...@skynet.be> --- kernel/power/hibernate.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/power/hibernate.c b/kernel/power/hibernate.c index b26dbc4..cfa7930 100644 --- a/kernel

[PATCH 10/14] PM / Hibernate: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick --- kernel/power/hibernate.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/power/hibernate.c b/kernel/power/hibernate.c index b26dbc4..cfa7930 100644 --- a/kernel/power/hibernate.c +++ b

[PATCH 07/14] PM / RUNTIME: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick <f...@skynet.be> --- drivers/base/power/runtime.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/base/power/runtime.c b/drivers/base/power/runtime.c index 872eac4..8ed435f

[PATCH 06/14] s390/qeth: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick <f...@skynet.be> --- drivers/s390/net/qeth_core_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/s390/net/qeth_core_main.c b/drivers/s390/net/qeth_core_main.c index 315d8a2..9

[PATCH 06/14] s390/qeth: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick --- drivers/s390/net/qeth_core_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/s390/net/qeth_core_main.c b/drivers/s390/net/qeth_core_main.c index 315d8a2..9595b90 100644 --- a/drivers

[PATCH 07/14] PM / RUNTIME: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick --- drivers/base/power/runtime.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/base/power/runtime.c b/drivers/base/power/runtime.c index 872eac4..8ed435f 100644 --- a/drivers/base/power

[PATCH 14/14] xfs: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/xfs/xfs_buf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/xfs/xfs_buf.c b/fs/xfs/xfs_buf.c index ac3b4db..51b2167 100644 --- a/fs/xfs/xfs_buf.c +++ b/fs/xfs/xfs

[PATCH 13/14] ext4: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/ext4/ext4.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h index 164a2e0..3184c25 100644 --- a/fs/ext4/ext4.h +++ b/fs/ext4/ext4.h @@ -

[PATCH 14/14] xfs: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick --- fs/xfs/xfs_buf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/xfs/xfs_buf.c b/fs/xfs/xfs_buf.c index ac3b4db..51b2167 100644 --- a/fs/xfs/xfs_buf.c +++ b/fs/xfs/xfs_buf.c @@ -1684,7 +1684,7

[PATCH 13/14] ext4: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick --- fs/ext4/ext4.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h index 164a2e0..3184c25 100644 --- a/fs/ext4/ext4.h +++ b/fs/ext4/ext4.h @@ -2916,7 +2916,7 @@ static

[PATCH 12/14] s390/topology: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick <f...@skynet.be> --- arch/s390/kernel/topology.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/s390/kernel/topology.c b/arch/s390/kernel/topology.c index 7b2b19b..820ef64 100644 --- a/arc

[PATCH 12/14] s390/topology: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick --- arch/s390/kernel/topology.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/s390/kernel/topology.c b/arch/s390/kernel/topology.c index 7b2b19b..820ef64 100644 --- a/arch/s390/kernel

[PATCH 03/14] drm/omap: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick <f...@skynet.be> --- drivers/gpu/drm/omapdrm/omap_dmm_tiler.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c b/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c

[PATCH 03/14] drm/omap: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick --- drivers/gpu/drm/omapdrm/omap_dmm_tiler.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c b/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c index 3cab066..abd0b15

[PATCH 02/14] drm/exynos: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick <f...@skynet.be> --- drivers/gpu/drm/exynos/exynos_drm_fimd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c b/drivers/gpu/drm/exynos/exynos_drm_fimd.c

[PATCH 02/14] drm/exynos: use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
instead of atomic_add_unless(value, -1, 0) Signed-off-by: Fabian Frederick --- drivers/gpu/drm/exynos/exynos_drm_fimd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c b/drivers/gpu/drm/exynos/exynos_drm_fimd.c index 745cfbd..0e191b6

[PATCH 01/14] locking/atomic: import atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
complementary definition to atomic_inc_not_zero() featured in lib/fault-inject.c Signed-off-by: Fabian Frederick <f...@skynet.be> --- include/linux/atomic.h | 2 ++ lib/fault-inject.c | 2 -- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/atomic.h b/i

[PATCH 00/14] use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
complementary definition to atomic_inc_not_zero() featured in lib/fault-inject.c. This small patchset moves it to include/linux/atomic.h using it instead of atomic_add_unless(value, -1, 0) s390 patches were not compile-tested. Fabian Frederick (14): locking/atomic: import atomic_dec_not_zero

[PATCH 01/14] locking/atomic: import atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
complementary definition to atomic_inc_not_zero() featured in lib/fault-inject.c Signed-off-by: Fabian Frederick --- include/linux/atomic.h | 2 ++ lib/fault-inject.c | 2 -- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/atomic.h b/include/linux/atomic.h index

[PATCH 00/14] use atomic_dec_not_zero()

2017-01-30 Thread Fabian Frederick
complementary definition to atomic_inc_not_zero() featured in lib/fault-inject.c. This small patchset moves it to include/linux/atomic.h using it instead of atomic_add_unless(value, -1, 0) s390 patches were not compile-tested. Fabian Frederick (14): locking/atomic: import atomic_dec_not_zero

[PATCH 2/3 linux-next] udf: simplify udf_ioctl()

2017-01-24 Thread Fabian Frederick
"out" label was only returning error code. Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/udf/file.c | 39 +++ 1 file changed, 15 insertions(+), 24 deletions(-) diff --git a/fs/udf/file.c b/fs/udf/file.c index d44b3cb..00931fa 100

[PATCH 2/3 linux-next] udf: simplify udf_ioctl()

2017-01-24 Thread Fabian Frederick
"out" label was only returning error code. Signed-off-by: Fabian Frederick --- fs/udf/file.c | 39 +++ 1 file changed, 15 insertions(+), 24 deletions(-) diff --git a/fs/udf/file.c b/fs/udf/file.c index d44b3cb..00931fa 100644 --- a/fs/udf/file.c ++

[PATCH 1/3 linux-next] udf: fix ioctl errors

2017-01-24 Thread Fabian Frederick
by Jan Kara, if arg is NULL with a correct ioctl, we return -VM_FAULT_SIGBUS to report error. Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/udf/file.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/fs/udf/file.c b/fs/udf/file.c index dbcb3a4a..d44b3cb

[PATCH 1/3 linux-next] udf: fix ioctl errors

2017-01-24 Thread Fabian Frederick
by Jan Kara, if arg is NULL with a correct ioctl, we return -VM_FAULT_SIGBUS to report error. Signed-off-by: Fabian Frederick --- fs/udf/file.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/fs/udf/file.c b/fs/udf/file.c index dbcb3a4a..d44b3cb 100644 --- a/fs/udf

[PATCH 3/3 linux-next] udf: remove else after return in udf_ioctl()

2017-01-24 Thread Fabian Frederick
else after return is not needed. Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/udf/file.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/fs/udf/file.c b/fs/udf/file.c index 00931fa..a1fec1b 100644 --- a/fs/udf/file.c +++ b/fs/udf/file.c @@ -194,8 +194,7 @

[PATCH 3/3 linux-next] udf: remove else after return in udf_ioctl()

2017-01-24 Thread Fabian Frederick
else after return is not needed. Signed-off-by: Fabian Frederick --- fs/udf/file.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/fs/udf/file.c b/fs/udf/file.c index 00931fa..a1fec1b 100644 --- a/fs/udf/file.c +++ b/fs/udf/file.c @@ -194,8 +194,7 @@ long udf_ioctl(struct

Re: [PATCH 1/1 linux-next] jfs: atomically read inode size

2017-01-23 Thread Fabian Frederick
> On 23 January 2017 at 19:43 Dave Kleikamp <dave.kleik...@oracle.com> wrote: > > > On 01/23/2017 11:50 AM, Fabian Frederick wrote: > > See i_size_read() comments in include/linux/fs.h > > Is this fixing a real problem? Can the bd_inode size change whil

Re: [PATCH 1/1 linux-next] jfs: atomically read inode size

2017-01-23 Thread Fabian Frederick
> On 23 January 2017 at 19:43 Dave Kleikamp wrote: > > > On 01/23/2017 11:50 AM, Fabian Frederick wrote: > > See i_size_read() comments in include/linux/fs.h > > Is this fixing a real problem? Can the bd_inode size change while we're > mounting or resizing the filesys

[PATCH 1/1 linux-next] ext4: atomically read inode size

2017-01-23 Thread Fabian Frederick
See i_size_read() comments in include/linux/fs.h Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/ext4/super.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/fs/ext4/super.c b/fs/ext4/super.c index 9d15a62..3746f77 100644 --- a/fs/ext4/super.c +++ b/fs/ext4/s

[PATCH 1/1 linux-next] ext4: atomically read inode size

2017-01-23 Thread Fabian Frederick
See i_size_read() comments in include/linux/fs.h Signed-off-by: Fabian Frederick --- fs/ext4/super.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/fs/ext4/super.c b/fs/ext4/super.c index 9d15a62..3746f77 100644 --- a/fs/ext4/super.c +++ b/fs/ext4/super.c @@ -3798,7

[PATCH 1/1 linux-next] hfsplus: atomically read inode size

2017-01-23 Thread Fabian Frederick
See i_size_read() comments in include/linux/fs.h Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/hfsplus/wrapper.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/hfsplus/wrapper.c b/fs/hfsplus/wrapper.c index ebb85e5..e254fa0 100644 --- a/fs/hfsplus/wra

[PATCH 1/1 linux-next] hfsplus: atomically read inode size

2017-01-23 Thread Fabian Frederick
See i_size_read() comments in include/linux/fs.h Signed-off-by: Fabian Frederick --- fs/hfsplus/wrapper.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/hfsplus/wrapper.c b/fs/hfsplus/wrapper.c index ebb85e5..e254fa0 100644 --- a/fs/hfsplus/wrapper.c +++ b/fs/hfsplus

[PATCH 1/1 linux-next] hfs: atomically read inode size

2017-01-23 Thread Fabian Frederick
See i_size_read() comments in include/linux/fs.h Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/hfs/mdb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/hfs/mdb.c b/fs/hfs/mdb.c index a3ec3ae..482081b 100644 --- a/fs/hfs/mdb.c +++ b/fs/hfs/mdb.c @@ -38,7

[PATCH 1/1 linux-next] hfs: atomically read inode size

2017-01-23 Thread Fabian Frederick
See i_size_read() comments in include/linux/fs.h Signed-off-by: Fabian Frederick --- fs/hfs/mdb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/hfs/mdb.c b/fs/hfs/mdb.c index a3ec3ae..482081b 100644 --- a/fs/hfs/mdb.c +++ b/fs/hfs/mdb.c @@ -38,7 +38,7 @@ static int

[PATCH 1/1 linux-next] jfs: atomically read inode size

2017-01-23 Thread Fabian Frederick
See i_size_read() comments in include/linux/fs.h Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/jfs/resize.c | 4 ++-- fs/jfs/super.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/fs/jfs/resize.c b/fs/jfs/resize.c index bd9b641..7ddcb44 100644 --- a/

[PATCH 1/1 linux-next] jfs: atomically read inode size

2017-01-23 Thread Fabian Frederick
See i_size_read() comments in include/linux/fs.h Signed-off-by: Fabian Frederick --- fs/jfs/resize.c | 4 ++-- fs/jfs/super.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/fs/jfs/resize.c b/fs/jfs/resize.c index bd9b641..7ddcb44 100644 --- a/fs/jfs/resize.c +++ b/fs

[PATCH 1/1 linux-next] fs/reiserfs: atomically read inode size

2017-01-23 Thread Fabian Frederick
See i_size_read() comments in include/linux/fs.h 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 e314cb3..feabcde 100644 --- a/fs/reiserfs/super.c ++

[PATCH 1/1 linux-next] fs/reiserfs: atomically read inode size

2017-01-23 Thread Fabian Frederick
See i_size_read() comments in include/linux/fs.h Signed-off-by: Fabian Frederick --- 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 e314cb3..feabcde 100644 --- a/fs/reiserfs/super.c +++ b/fs/reiserfs

[PATCH 1/1 linux-next] udf: return inappropriate ioctl instead of argument error

2017-01-19 Thread Fabian Frederick
on ..." If arg is NULL, we return specific operation error anyway. Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/udf/file.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/fs/udf/file.c b/fs/udf/file.c index dbcb3a4a..964620b 100644 --- a/fs/udf/

[PATCH 1/1 linux-next] udf: return inappropriate ioctl instead of argument error

2017-01-19 Thread Fabian Frederick
on ..." If arg is NULL, we return specific operation error anyway. Signed-off-by: Fabian Frederick --- fs/udf/file.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/fs/udf/file.c b/fs/udf/file.c index dbcb3a4a..964620b 100644 --- a/fs/udf/file.c +++ b/fs/udf/file

Re: [PATCH V2 0/7 linux-next] make FS exportable plus some clean-up

2017-01-18 Thread Fabian Frederick
> On 09 January 2017 at 20:12 Fabian Frederick <f...@skynet.be> wrote: > > > This small patchset makes AFFS work with NFS for standard operations. > V2 adds a new patch suggested by Alexander Viro in order to work with cold > dcache. > > Fabian Frederick (7):

Re: [PATCH V2 0/7 linux-next] make FS exportable plus some clean-up

2017-01-18 Thread Fabian Frederick
> On 09 January 2017 at 20:12 Fabian Frederick wrote: > > > This small patchset makes AFFS work with NFS for standard operations. > V2 adds a new patch suggested by Alexander Viro in order to work with cold > dcache. > > Fabian Frederick (7): >   fs/affs: remove r

[RFC 1/1 linux-next] udf: allow implicit blocksize specification during mount

2017-01-18 Thread Fabian Frederick
other values but maybe VRS is not mandatory there ? Tested with 512, 1024, 2048 and 4096 blocksize Reported-by: Jan Kara <j...@suse.com> Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/udf/super.c | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) di

[RFC 1/1 linux-next] udf: allow implicit blocksize specification during mount

2017-01-18 Thread Fabian Frederick
other values but maybe VRS is not mandatory there ? Tested with 512, 1024, 2048 and 4096 blocksize Reported-by: Jan Kara Signed-off-by: Fabian Frederick --- fs/udf/super.c | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/fs/udf/super.c b/fs/udf/super.c

[PATCH 1/1 linux-next] Documentation/filesystems/proc.txt: add VmPin

2017-01-14 Thread Fabian Frederick
Commit bc3e53f682d9 ("mm: distinguish between mlocked and pinned pages") added VmPin in /proc//status. Report that in Documentation/filesystems/proc.txt Also move Umask after Name to keep correct order. Signed-off-by: Fabian Frederick <f...@skynet.be> --- Documentation/filesyst

[PATCH 1/1 linux-next] Documentation/filesystems/proc.txt: add VmPin

2017-01-14 Thread Fabian Frederick
Commit bc3e53f682d9 ("mm: distinguish between mlocked and pinned pages") added VmPin in /proc//status. Report that in Documentation/filesystems/proc.txt Also move Umask after Name to keep correct order. Signed-off-by: Fabian Frederick --- Documentation/filesystems/proc.txt | 5 +++

[PATCH V2 6/7 linux-next] fs/affs/namei.c: forward declarations clean-up

2017-01-09 Thread Fabian Frederick
move dentry_operations structures and remove forward declarations Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/affs/namei.c | 30 ++ 1 file changed, 10 insertions(+), 20 deletions(-) diff --git a/fs/affs/namei.c b/fs/affs/namei.c index 906ff5b..f

[PATCH V2 6/7 linux-next] fs/affs/namei.c: forward declarations clean-up

2017-01-09 Thread Fabian Frederick
move dentry_operations structures and remove forward declarations Signed-off-by: Fabian Frederick --- fs/affs/namei.c | 30 ++ 1 file changed, 10 insertions(+), 20 deletions(-) diff --git a/fs/affs/namei.c b/fs/affs/namei.c index 906ff5b..fb88446 100644 --- a/fs

[PATCH V2 5/7 linux-next] fs/affs: add prefix to some functions

2017-01-09 Thread Fabian Frederick
secs_to_datestamp(time64_t secs, struct affs_date *ds); prot_to_mode(u32 prot); mode_to_prot(struct inode *inode); were declared without affs_ prefix Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/affs/affs.h | 6 +++--- fs/affs/amigaffs.c | 6 +++--- fs/affs/inode.c

[PATCH V2 5/7 linux-next] fs/affs: add prefix to some functions

2017-01-09 Thread Fabian Frederick
secs_to_datestamp(time64_t secs, struct affs_date *ds); prot_to_mode(u32 prot); mode_to_prot(struct inode *inode); were declared without affs_ prefix Signed-off-by: Fabian Frederick --- fs/affs/affs.h | 6 +++--- fs/affs/amigaffs.c | 6 +++--- fs/affs/inode.c| 9 + fs/affs

[PATCH V2 2/7 linux-next] fs/affs: add validation block function

2017-01-09 Thread Fabian Frederick
avoid repeating 4 times the same calculation. Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/affs/affs.h | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/fs/affs/affs.h b/fs/affs/affs.h index 899256b..efe6839 100644 --- a/fs/affs/affs.h +++ b/f

[PATCH V2 2/7 linux-next] fs/affs: add validation block function

2017-01-09 Thread Fabian Frederick
avoid repeating 4 times the same calculation. Signed-off-by: Fabian Frederick --- fs/affs/affs.h | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/fs/affs/affs.h b/fs/affs/affs.h index 899256b..efe6839 100644 --- a/fs/affs/affs.h +++ b/fs/affs/affs.h @@ -212,6

[PATCH V2 7/7 linux-next] fs/affs: make export work with cold dcache

2017-01-09 Thread Fabian Frederick
This adds get_parent function so that nfs client can still work after cache drop (Tested on NFS v4 with echo 3 > /proc/sys/vm/drop_caches) Suggested-by: Alexander Viro <v...@zeniv.linux.org.uk> Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/affs/namei.c | 19

[PATCH V2 7/7 linux-next] fs/affs: make export work with cold dcache

2017-01-09 Thread Fabian Frederick
This adds get_parent function so that nfs client can still work after cache drop (Tested on NFS v4 with echo 3 > /proc/sys/vm/drop_caches) Suggested-by: Alexander Viro Signed-off-by: Fabian Frederick --- fs/affs/namei.c | 19 +++ 1 file changed, 19 insertions(+) diff --

[PATCH V2 1/7 linux-next] fs/affs: remove reference to affs_parent_ino()

2017-01-09 Thread Fabian Frederick
That function was removed a long time ago. Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/affs/affs.h | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/affs/affs.h b/fs/affs/affs.h index 2f08877..899256b 100644 --- a/fs/affs/affs.h +++ b/fs/affs/affs.h @@ -178,7 +178,6 @@ exte

[PATCH V2 1/7 linux-next] fs/affs: remove reference to affs_parent_ino()

2017-01-09 Thread Fabian Frederick
That function was removed a long time ago. Signed-off-by: Fabian Frederick --- fs/affs/affs.h | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/affs/affs.h b/fs/affs/affs.h index 2f08877..899256b 100644 --- a/fs/affs/affs.h +++ b/fs/affs/affs.h @@ -178,7 +178,6 @@ extern int affs_rename

[PATCH V2 0/7 linux-next] make FS exportable plus some clean-up

2017-01-09 Thread Fabian Frederick
This small patchset makes AFFS work with NFS for standard operations. V2 adds a new patch suggested by Alexander Viro in order to work with cold dcache. Fabian Frederick (7): fs/affs: remove reference to affs_parent_ino() fs/affs: add validation block function fs/affs: make affs exportable

[PATCH V2 0/7 linux-next] make FS exportable plus some clean-up

2017-01-09 Thread Fabian Frederick
This small patchset makes AFFS work with NFS for standard operations. V2 adds a new patch suggested by Alexander Viro in order to work with cold dcache. Fabian Frederick (7): fs/affs: remove reference to affs_parent_ino() fs/affs: add validation block function fs/affs: make affs exportable

[PATCH V2 4/7 linux-next] fs/affs: use octal for permissions

2017-01-09 Thread Fabian Frederick
According to commit f90774e1fd27 ("checkpatch: look for symbolic permissions and suggest octal instead") Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/affs/amigaffs.c | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff

[PATCH V2 3/7 linux-next] fs/affs: make affs exportable

2017-01-09 Thread Fabian Frederick
Add standard functions making AFFS work with NFS. Functions based on ext4 implementation. Tested on loop device. Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/affs/affs.h | 1 + fs/affs/namei.c | 40 fs/affs/super.c | 1 + 3 files c

[PATCH V2 4/7 linux-next] fs/affs: use octal for permissions

2017-01-09 Thread Fabian Frederick
According to commit f90774e1fd27 ("checkpatch: look for symbolic permissions and suggest octal instead") Signed-off-by: Fabian Frederick --- fs/affs/amigaffs.c | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/fs/affs/amigaffs.c

[PATCH V2 3/7 linux-next] fs/affs: make affs exportable

2017-01-09 Thread Fabian Frederick
Add standard functions making AFFS work with NFS. Functions based on ext4 implementation. Tested on loop device. Signed-off-by: Fabian Frederick --- fs/affs/affs.h | 1 + fs/affs/namei.c | 40 fs/affs/super.c | 1 + 3 files changed, 42 insertions

[PATCH 05/12 linux-next] udf: remove empty condition

2017-01-06 Thread Fabian Frederick
loc & 0x02 is empty since first git version in 2005 in udf_add_extendedattr() Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/udf/misc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/fs/udf/misc.c b/fs/udf/misc.c index 71d1c25..3949c4b 100644 --- a/fs/udf/misc.c ++

[PATCH 06/12 linux-next] udf: add udf_ajust_time

2017-01-06 Thread Fabian Frederick
Merge timespec affectations Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/udf/inode.c | 25 +++-- 1 file changed, 11 insertions(+), 14 deletions(-) diff --git a/fs/udf/inode.c b/fs/udf/inode.c index b5893f4..f563e97 100644 --- a/fs/udf/inode.c +++ b/fs/udf/i

[PATCH 05/12 linux-next] udf: remove empty condition

2017-01-06 Thread Fabian Frederick
loc & 0x02 is empty since first git version in 2005 in udf_add_extendedattr() Signed-off-by: Fabian Frederick --- fs/udf/misc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/fs/udf/misc.c b/fs/udf/misc.c index 71d1c25..3949c4b 100644 --- a/fs/udf/misc.c +++ b/fs/udf/misc.c @@ -1

[PATCH 06/12 linux-next] udf: add udf_ajust_time

2017-01-06 Thread Fabian Frederick
Merge timespec affectations Signed-off-by: Fabian Frederick --- fs/udf/inode.c | 25 +++-- 1 file changed, 11 insertions(+), 14 deletions(-) diff --git a/fs/udf/inode.c b/fs/udf/inode.c index b5893f4..f563e97 100644 --- a/fs/udf/inode.c +++ b/fs/udf/inode.c @@ -1612,6

[PATCH 09/12 linux-next] udf: merge module informations in super.c

2017-01-06 Thread Fabian Frederick
Move all module attributes at the end of one file like other FS. Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/udf/inode.c | 4 fs/udf/super.c | 9 ++--- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/fs/udf/inode.c b/fs/udf/inode.c index 2327f72..4

<    1   2   3   4   5   6   7   8   9   10   >