Re: [PATCH 1/1 linux-next] ext4: add compatibility flag check

2016-11-24 Thread Fabian Frederick
> On 25 November 2016 at 06:26 Theodore Ts'o wrote: > > > On Thu, Nov 24, 2016 at 08:47:41PM +0100, Fabian Frederick wrote: > > data=journal mount option should disable O_DIRECT access > > (See Documentation/filesystems/ext4.txt) but open operations > > using O_CRE

Re: [PATCH 1/1 linux-next] ext4: add compatibility flag check

2016-11-24 Thread Fabian Frederick
> On 24 November 2016 at 21:39 Al Viro <v...@zeniv.linux.org.uk> wrote: > > > On Thu, Nov 24, 2016 at 08:47:41PM +0100, Fabian Frederick wrote: > > data=journal mount option should disable O_DIRECT access > > (See Documentation/filesystems/ext4.txt) but open opera

Re: [PATCH 1/1 linux-next] ext4: add compatibility flag check

2016-11-24 Thread Fabian Frederick
> On 24 November 2016 at 21:39 Al Viro wrote: > > > On Thu, Nov 24, 2016 at 08:47:41PM +0100, Fabian Frederick wrote: > > data=journal mount option should disable O_DIRECT access > > (See Documentation/filesystems/ext4.txt) but open operations > > using O_CRE

[PATCH 1/1 linux-next] ext4: add compatibility flag check

2016-11-24 Thread Fabian Frederick
in such a case. Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/ext4/super.c| 10 ++ fs/namei.c | 12 include/linux/fs.h | 1 + 3 files changed, 23 insertions(+) diff --git a/fs/ext4/super.c b/fs/ext4/super.c index 72b459d..f2cd8e3 100644 --- a/fs/ext4/s

[PATCH 1/1 linux-next] ext4: add compatibility flag check

2016-11-24 Thread Fabian Frederick
in such a case. Signed-off-by: Fabian Frederick --- fs/ext4/super.c| 10 ++ fs/namei.c | 12 include/linux/fs.h | 1 + 3 files changed, 23 insertions(+) diff --git a/fs/ext4/super.c b/fs/ext4/super.c index 72b459d..f2cd8e3 100644 --- a/fs/ext4/super.c +++ b/fs/ext4

[PATCH 1/1 linux-next] vfs: fix comment typos

2016-11-24 Thread Fabian Frederick
Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/namei.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/namei.c b/fs/namei.c index 1669c93d..ca829b4 100644 --- a/fs/namei.c +++ b/fs/namei.c @@ -3119,12 +3119,12 @@ static int lookup_open(struct nameida

[PATCH 1/1 linux-next] vfs: fix comment typos

2016-11-24 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- fs/namei.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/namei.c b/fs/namei.c index 1669c93d..ca829b4 100644 --- a/fs/namei.c +++ b/fs/namei.c @@ -3119,12 +3119,12 @@ static int lookup_open(struct nameidata *nd, struct path *path

[PATCH V2 linux-next] ALSA: pci: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick <f...@skynet.be> --- V2: Convert everything in PCI branch in one patch (suggested by Takashi Iwai) sound/pci/als4000.c | 2 +- sound/pci/au88x0/au88x0_game.c | 2 +- sound/pci/azt3328.c | 2 +- sound/pci/cmipci.c

[PATCH V2 linux-next] ALSA: pci: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- V2: Convert everything in PCI branch in one patch (suggested by Takashi Iwai) sound/pci/als4000.c | 2 +- sound/pci/au88x0/au88x0_game.c | 2 +- sound/pci/azt3328.c | 2 +- sound/pci/cmipci.c | 2 +- sound/pci/cs4281

[PATCH 1/1 linux-next] usb: isp1301-omap: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick <f...@skynet.be> --- drivers/usb/phy/phy-isp1301-omap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/phy/phy-isp1301-omap.c b/drivers/usb/phy/phy-isp1301-omap.c index 8d111ec..042c5a8 100644 --- a/drivers/usb/phy/phy-i

[PATCH 1/1 linux-next] usb: isp1301-omap: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- drivers/usb/phy/phy-isp1301-omap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/phy/phy-isp1301-omap.c b/drivers/usb/phy/phy-isp1301-omap.c index 8d111ec..042c5a8 100644 --- a/drivers/usb/phy/phy-isp1301-omap.c +++ b/drivers

[PATCH 1/1 linux-next] drm/nouveau: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick <f...@skynet.be> --- drivers/gpu/drm/nouveau/nouveau_hwmon.c | 6 +++--- drivers/gpu/drm/nouveau/nvkm/subdev/pci/agp.h | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_hwmon.c b/drivers/gpu/drm/n

[PATCH 1/1 linux-next] drm/nouveau: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- drivers/gpu/drm/nouveau/nouveau_hwmon.c | 6 +++--- drivers/gpu/drm/nouveau/nvkm/subdev/pci/agp.h | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_hwmon.c b/drivers/gpu/drm/nouveau/nouveau_hwmon.c

[RFC 1/1 linux-next] ALSA: pci: declare SUPPORT_JOYSTICK in gameport.h

2016-11-12 Thread Fabian Frederick
Currently, all gameport callsites declare SUPPORT_JOYSTICK under opencoded IS_REACHABLE(). This patch moves definition in gameport.h and removes opencoding. Signed-off-by: Fabian Frederick <f...@skynet.be> --- This is untested include/linux/gameport.h| 3 ++- sound/pci/als4000.c

[RFC 1/1 linux-next] ALSA: pci: declare SUPPORT_JOYSTICK in gameport.h

2016-11-12 Thread Fabian Frederick
Currently, all gameport callsites declare SUPPORT_JOYSTICK under opencoded IS_REACHABLE(). This patch moves definition in gameport.h and removes opencoding. Signed-off-by: Fabian Frederick --- This is untested include/linux/gameport.h| 3 ++- sound/pci/als4000.c | 4 sound/pci

[PATCH 1/1 linux-next] ALSA: pci: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick <f...@skynet.be> --- sound/pci/au88x0/au88x0_game.c | 2 +- sound/pci/azt3328.c | 2 +- sound/pci/cs4281.c | 2 +- sound/pci/cs46xx/cs46xx_lib.c| 2 +- sound/pci/emu10k1/emu10k1.c | 2 +- sound/pci/trident/trident_main

[PATCH 1/1 linux-next] ALSA: pci: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- sound/pci/au88x0/au88x0_game.c | 2 +- sound/pci/azt3328.c | 2 +- sound/pci/cs4281.c | 2 +- sound/pci/cs46xx/cs46xx_lib.c| 2 +- sound/pci/emu10k1/emu10k1.c | 2 +- sound/pci/trident/trident_main.c | 2 +- 6 files

[PATCH 1/1 linux-next] ALSA: oss: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick <f...@skynet.be> --- sound/core/oss/pcm_oss.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/core/oss/pcm_oss.c b/sound/core/oss/pcm_oss.c index ebc9fdf..698a014 100644 --- a/sound/core/oss/pcm_oss.c +++ b/sound/core/oss/pcm

[PATCH 1/1 linux-next] ALSA: oss: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- sound/core/oss/pcm_oss.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/core/oss/pcm_oss.c b/sound/core/oss/pcm_oss.c index ebc9fdf..698a014 100644 --- a/sound/core/oss/pcm_oss.c +++ b/sound/core/oss/pcm_oss.c @@ -2501,7 +2501,7

[PATCH 1/1 linux-next] ALSA: rawmidi: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick <f...@skynet.be> --- sound/core/rawmidi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/core/rawmidi.c b/sound/core/rawmidi.c index b450a27..2096bb0 100644 --- a/sound/core/rawmidi.c +++ b/sound/core/rawmidi.c @@ -1610,7 +

[PATCH 1/1 linux-next] ALSA: opl4: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick <f...@skynet.be> --- sound/drivers/opl4/opl4_lib.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/drivers/opl4/opl4_lib.c b/sound/drivers/opl4/opl4_lib.c index 89c7aa0..240656e 100644 --- a/sound/drivers/opl4/opl4_lib.c +++ b

[PATCH 1/1 linux-next] ALSA: rawmidi: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- sound/core/rawmidi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/core/rawmidi.c b/sound/core/rawmidi.c index b450a27..2096bb0 100644 --- a/sound/core/rawmidi.c +++ b/sound/core/rawmidi.c @@ -1610,7 +1610,7 @@ static int

[PATCH 1/1 linux-next] ALSA: opl4: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- sound/drivers/opl4/opl4_lib.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/drivers/opl4/opl4_lib.c b/sound/drivers/opl4/opl4_lib.c index 89c7aa0..240656e 100644 --- a/sound/drivers/opl4/opl4_lib.c +++ b/sound/drivers/opl4

[PATCH 1/1 linux-next] ALSA: opl3: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick <f...@skynet.be> --- sound/drivers/opl3/opl3_lib.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/drivers/opl3/opl3_lib.c b/sound/drivers/opl3/opl3_lib.c index 369cef2..cd9e9f3 100644 --- a/sound/drivers/opl3/opl3_lib.c +++ b

[PATCH 1/1 linux-next] ALSA: opl3: don't opencode IS_REACHABLE()

2016-11-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- sound/drivers/opl3/opl3_lib.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/drivers/opl3/opl3_lib.c b/sound/drivers/opl3/opl3_lib.c index 369cef2..cd9e9f3 100644 --- a/sound/drivers/opl3/opl3_lib.c +++ b/sound/drivers/opl3

[PATCH V2 linux-next] ext4: fix block_validity definition

2016-11-06 Thread Fabian Frederick
Fix ext4 documentation according to commit 45f1a9c3f63d ("ext4: enable block_validity by default") Also fix some typos. Reviewed-by: Darrick J. Wong <darrick.w...@oracle.com> Signed-off-by: Fabian Frederick <f...@skynet.be> --- V2: Replace beginning of the definition

[PATCH V2 linux-next] ext4: fix block_validity definition

2016-11-06 Thread Fabian Frederick
Fix ext4 documentation according to commit 45f1a9c3f63d ("ext4: enable block_validity by default") Also fix some typos. Reviewed-by: Darrick J. Wong Signed-off-by: Fabian Frederick --- V2: Replace beginning of the definition (suggested by Darrick). Documentation/filesystems/ex

[PATCH 1/1 linux-next] ext4: fix block_validity definition

2016-10-23 Thread Fabian Frederick
Fix ext4 documentation according to commit 45f1a9c3f63d ("ext4: enable block_validity by default") Also fix some typos. Signed-off-by: Fabian Frederick <f...@skynet.be> --- Documentation/filesystems/ext4.txt | 7 +++ 1 file changed, 3 insertions(+), 4 deletion

[PATCH 1/1 linux-next] ext4: fix block_validity definition

2016-10-23 Thread Fabian Frederick
Fix ext4 documentation according to commit 45f1a9c3f63d ("ext4: enable block_validity by default") Also fix some typos. Signed-off-by: Fabian Frederick --- Documentation/filesystems/ext4.txt | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/Documentation/f

[PATCH V2 linux-next] ext4: make online resizing conditional

2016-10-21 Thread Fabian Frederick
bled: "Operation not supported While checking for on-line resizing support" Signed-off-by: Fabian Frederick <f...@skynet.be> --- V2: -Fix checkpatch warnings on identifier names like: WARNING: function definition argument 'struct super_block *' should also have an identifier name #130: FILE:

[PATCH V2 linux-next] ext4: make online resizing conditional

2016-10-21 Thread Fabian Frederick
bled: "Operation not supported While checking for on-line resizing support" Signed-off-by: Fabian Frederick --- V2: -Fix checkpatch warnings on identifier names like: WARNING: function definition argument 'struct super_block *' should also have an identifier name #130: FILE: fs/ext4/resize.h:8:

[PATCH 1/1 linux-next] ext4: make online resizing conditional

2016-10-20 Thread Fabian Frederick
bled: "Operation not supported While checking for on-line resizing support" Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/ext4/Kconfig | 7 +++ fs/ext4/Makefile | 3 ++- fs/ext4/ext4.h | 12 fs/ext4/ioctl.c | 1 + fs/ext4/resize.

[PATCH 1/1 linux-next] ext4: make online resizing conditional

2016-10-20 Thread Fabian Frederick
bled: "Operation not supported While checking for on-line resizing support" Signed-off-by: Fabian Frederick --- fs/ext4/Kconfig | 7 +++ fs/ext4/Makefile | 3 ++- fs/ext4/ext4.h | 12 fs/ext4/ioctl.c | 1 + fs/ext4/resize.c | 2 +- fs/ext4/resize.h | 42 +++

Re: [PATCH 1/1 linux-next] netfilter: conntrack: fix kmemleak false positive

2016-09-23 Thread Fabian Frederick
> On 22 September 2016 at 23:56 Florian Westphal <f...@strlen.de> wrote: > > > Fabian Frederick <f...@skynet.be> wrote: > > Hello Florian, > > > >         First problem is solved: table gets cleared 3 minutes earlier > > but I still have kmemleak

Re: [PATCH 1/1 linux-next] netfilter: conntrack: fix kmemleak false positive

2016-09-23 Thread Fabian Frederick
> On 22 September 2016 at 23:56 Florian Westphal wrote: > > > Fabian Frederick wrote: > > Hello Florian, > > > >         First problem is solved: table gets cleared 3 minutes earlier > > but I still have kmemleak before running the following: > >

Re: [PATCH 1/1 linux-next] netfilter: conntrack: fix kmemleak false positive

2016-09-22 Thread Fabian Frederick
> On 21 September 2016 at 23:02 Florian Westphal <f...@strlen.de> wrote: > > > Fabian Frederick <f...@skynet.be> wrote: > > Since commit f330a7fdbe16 > > ("netfilter: conntrack: get rid of conntrack timer") > > > > closed connectio

Re: [PATCH 1/1 linux-next] netfilter: conntrack: fix kmemleak false positive

2016-09-22 Thread Fabian Frederick
> On 21 September 2016 at 23:02 Florian Westphal wrote: > > > Fabian Frederick wrote: > > Since commit f330a7fdbe16 > > ("netfilter: conntrack: get rid of conntrack timer") > > > > closed connections remain longer in /proc/net/nf_conntrack &

[PATCH 1/1 linux-next] netfilter: conntrack: fix kmemleak false positive

2016-09-21 Thread Fabian Frederick
ctures being cleared in gc_worker() later on we can't talk about unreferenced object so this patch uses kmemleak_not_leak() to prevent those warnings. Signed-off-by: Fabian Frederick <f...@skynet.be> --- net/netfilter/nf_conntrack_core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/net/netfil

[PATCH 1/1 linux-next] netfilter: conntrack: fix kmemleak false positive

2016-09-21 Thread Fabian Frederick
ctures being cleared in gc_worker() later on we can't talk about unreferenced object so this patch uses kmemleak_not_leak() to prevent those warnings. Signed-off-by: Fabian Frederick --- net/netfilter/nf_conntrack_core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/net/netfilter/nf_conntrack_cor

[PATCH 2/6 linux-next] ext4: remove unneeded test in ext4_alloc_file_blocks()

2016-08-24 Thread Fabian Frederick
ext4_alloc_file_blocks() is called from ext4_zero_range() and ext4_fallocate() both already testing EXT4_INODE_EXTENTS We can call ext_depth(inode) unconditionnally. Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/ext4/extents.c | 8 +--- 1 file changed, 1 insertion(+), 7 del

[PATCH 2/6 linux-next] ext4: remove unneeded test in ext4_alloc_file_blocks()

2016-08-24 Thread Fabian Frederick
ext4_alloc_file_blocks() is called from ext4_zero_range() and ext4_fallocate() both already testing EXT4_INODE_EXTENTS We can call ext_depth(inode) unconditionnally. Signed-off-by: Fabian Frederick --- fs/ext4/extents.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git

[PATCH 0/6 linux-next] ext4: fix extent leaking and clean-up

2016-08-24 Thread Fabian Frederick
Last patch of this small patchset fixes an extent path memory leak. The rest is some clean-up. Fabian Frederick (6): ext4: avoid EXT4_INODE_EXTENTS double checking ext4: remove unneeded test in ext4_alloc_file_blocks() ext4: create EXT4_MAX_BLOCKS() macro ext4: use bool for check

[PATCH 0/6 linux-next] ext4: fix extent leaking and clean-up

2016-08-24 Thread Fabian Frederick
Last patch of this small patchset fixes an extent path memory leak. The rest is some clean-up. Fabian Frederick (6): ext4: avoid EXT4_INODE_EXTENTS double checking ext4: remove unneeded test in ext4_alloc_file_blocks() ext4: create EXT4_MAX_BLOCKS() macro ext4: use bool for check

[PATCH 1/6 linux-next] ext4: avoid EXT4_INODE_EXTENTS double checking

2016-08-24 Thread Fabian Frederick
ext4_collapse_range() and ext4_insert_range() already checked inode flag at the beginning of function. Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/ext4/extents.c | 11 --- 1 file changed, 11 deletions(-) diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c index d

[PATCH 1/6 linux-next] ext4: avoid EXT4_INODE_EXTENTS double checking

2016-08-24 Thread Fabian Frederick
ext4_collapse_range() and ext4_insert_range() already checked inode flag at the beginning of function. Signed-off-by: Fabian Frederick --- fs/ext4/extents.c | 11 --- 1 file changed, 11 deletions(-) diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c index d7ccb7f..5d9f99a 100644

[PATCH 4/6 linux-next] ext4: use bool for check in ext4_ext_space_()

2016-08-24 Thread Fabian Frederick
check is used in 0/1 context. Also use unsigned int instead of unsigned (checkpatch warning) Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/ext4/extents.c | 33 ++--- 1 file changed, 18 insertions(+), 15 deletions(-) diff --git a/fs/ext4/extents.c b/f

[PATCH 5/6 linux-next] ext4: remove unused definition

2016-08-24 Thread Fabian Frederick
MAX_32_NUM isn't used in ext4 Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/ext4/ioctl.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/fs/ext4/ioctl.c b/fs/ext4/ioctl.c index 10686fd..5a708c87 100644 --- a/fs/ext4/ioctl.c +++ b/fs/ext4/ioctl.c @@ -19,8 +19,6 @@ #i

[PATCH 4/6 linux-next] ext4: use bool for check in ext4_ext_space_()

2016-08-24 Thread Fabian Frederick
check is used in 0/1 context. Also use unsigned int instead of unsigned (checkpatch warning) Signed-off-by: Fabian Frederick --- fs/ext4/extents.c | 33 ++--- 1 file changed, 18 insertions(+), 15 deletions(-) diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c index

[PATCH 5/6 linux-next] ext4: remove unused definition

2016-08-24 Thread Fabian Frederick
MAX_32_NUM isn't used in ext4 Signed-off-by: Fabian Frederick --- fs/ext4/ioctl.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/fs/ext4/ioctl.c b/fs/ext4/ioctl.c index 10686fd..5a708c87 100644 --- a/fs/ext4/ioctl.c +++ b/fs/ext4/ioctl.c @@ -19,8 +19,6 @@ #include "ext4_jbd2.h"

[PATCH 6/6 linux-next] ext4: fix memory leak in ext4_insert_range()

2016-08-24 Thread Fabian Frederick
+0x3f/0x60 [] entry_SYSCALL_64_fastpath+0x13/0x8f [] 0x Problem seems mitigated by dropping refs and freeing path when there's no path[depth].p_ext Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/ext4/extents.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/f

[PATCH 3/6 linux-next] ext4: create EXT4_MAX_BLOCKS() macro

2016-08-24 Thread Fabian Frederick
Create a macro to calculate length + offset -> maximum blocks This adds more readability. Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/ext4/ext4.h| 3 +++ fs/ext4/extents.c | 15 +++ fs/ext4/file.c| 3 +-- 3 files changed, 7 insertions(+), 14 deletions

[PATCH 6/6 linux-next] ext4: fix memory leak in ext4_insert_range()

2016-08-24 Thread Fabian Frederick
+0x3f/0x60 [] entry_SYSCALL_64_fastpath+0x13/0x8f [] 0x Problem seems mitigated by dropping refs and freeing path when there's no path[depth].p_ext Signed-off-by: Fabian Frederick --- fs/ext4/extents.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/fs/ext4/extents.c

[PATCH 3/6 linux-next] ext4: create EXT4_MAX_BLOCKS() macro

2016-08-24 Thread Fabian Frederick
Create a macro to calculate length + offset -> maximum blocks This adds more readability. Signed-off-by: Fabian Frederick --- fs/ext4/ext4.h| 3 +++ fs/ext4/extents.c | 15 +++ fs/ext4/file.c| 3 +-- 3 files changed, 7 insertions(+), 14 deletions(-) diff --git a/fs/e

[PATCH V2 linux-next] iommu/amd: remove AMD_IOMMU_STATS

2016-08-22 Thread Fabian Frederick
Commit e85e8f69cedb ("iommu/amd: Remove statistics code") removed that configuration. Also remove function definition (suggested by Joerg Roedel) Signed-off-by: Fabian Frederick <f...@skynet.be> --- V2: Remove function definition. arch/x86/configs/x86_64_defconfig | 1

[PATCH V2 linux-next] iommu/amd: remove AMD_IOMMU_STATS

2016-08-22 Thread Fabian Frederick
Commit e85e8f69cedb ("iommu/amd: Remove statistics code") removed that configuration. Also remove function definition (suggested by Joerg Roedel) Signed-off-by: Fabian Frederick --- V2: Remove function definition. arch/x86/configs/x86_64_defconfig | 1 - drivers/iommu/amd_iommu_pro

[PATCH V2 linux-next] hwrng: update Freescale i.MX RNGA Random Number Generator

2016-08-16 Thread Fabian Frederick
/hw_random Previously, HW_RANDOM_MXC_RNGA required ARCH_HAS_RNGA which was based on IMX_HAVE_PLATFORM_MXC_RNGA && ARCH_MXC. IMX_HAVE_PLATFORM_MXC_RNGA was based on SOC_IMX31. Fixes: c9ee94965dce ("ARM: imx: deconstruct mxc_rnga initialization") Signed-off-by: Fabian Frederick <f

[PATCH V2 linux-next] hwrng: update Freescale i.MX RNGA Random Number Generator

2016-08-16 Thread Fabian Frederick
/hw_random Previously, HW_RANDOM_MXC_RNGA required ARCH_HAS_RNGA which was based on IMX_HAVE_PLATFORM_MXC_RNGA && ARCH_MXC. IMX_HAVE_PLATFORM_MXC_RNGA was based on SOC_IMX31. Fixes: c9ee94965dce ("ARM: imx: deconstruct mxc_rnga initialization") Signed-off-by: Fabian Freder

[PATCH 1/1 linux-next RESEND] ARM: multi_v7_defconfig: update XILINX_VDMA

2016-08-15 Thread Fabian Frederick
Commit fde57a7c4474 ("dmaengine: xilinx: Rename driver and config") renamed config XILINX_VDMA to config XILINX_DMA Update defconfig accordingly. Signed-off-by: Fabian Frederick <f...@skynet.be> --- Sending to arm-soc (suggested by Shawn Guo) arch/arm/configs/multi_v7_defconfi

[PATCH 1/1 linux-next RESEND] ARM: multi_v7_defconfig: update XILINX_VDMA

2016-08-15 Thread Fabian Frederick
Commit fde57a7c4474 ("dmaengine: xilinx: Rename driver and config") renamed config XILINX_VDMA to config XILINX_DMA Update defconfig accordingly. Signed-off-by: Fabian Frederick --- Sending to arm-soc (suggested by Shawn Guo) arch/arm/configs/multi_v7_defconfig | 2 +- 1 file

Re: [PATCH 1/1 linux-next] hwrng: update Freescale i.MX RNGA Random Number Generator

2016-08-15 Thread Fabian Frederick
> On 15 August 2016 at 17:05 Arnd Bergmann <a...@arndb.de> wrote: > > > On Friday 12 August 2016, Fabian Frederick wrote: > > We can directly depend on SOC_IMX31 since > > commit c9ee94965dce > > ("ARM: imx: deconstruct mxc_rnga initialization&quo

Re: [PATCH 1/1 linux-next] hwrng: update Freescale i.MX RNGA Random Number Generator

2016-08-15 Thread Fabian Frederick
> On 15 August 2016 at 17:05 Arnd Bergmann wrote: > > > On Friday 12 August 2016, Fabian Frederick wrote: > > We can directly depend on SOC_IMX31 since > > commit c9ee94965dce > > ("ARM: imx: deconstruct mxc_rnga initialization") > > > &

[PATCH 1/1 linux-next] xtensa: remove CONFIG_BLK_CPQ_DA

2016-08-12 Thread Fabian Frederick
Commit d436641439e0 ("cpqarray: remove it from the kernel") removed BLK_CPQ_DA Signed-off-by: Fabian Frederick <f...@skynet.be> --- arch/xtensa/configs/common_defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/xtensa/configs/common_defconfig b/arch/xtensa/configs/

[PATCH 1/1 linux-next] xtensa: remove CONFIG_BLK_CPQ_DA

2016-08-12 Thread Fabian Frederick
Commit d436641439e0 ("cpqarray: remove it from the kernel") removed BLK_CPQ_DA Signed-off-by: Fabian Frederick --- arch/xtensa/configs/common_defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/xtensa/configs/common_defconfig b/arch/xtensa/configs/common_defconfig ind

[PATCH 1/1 linux-next] mips: pistachio_defconfig: remove ANDROID_TIMED_OUTPUT

2016-08-12 Thread Fabian Frederick
According to commit 9f6b68774f29 ("android: remove timed output/gpio driver") Signed-off-by: Fabian Frederick <f...@skynet.be> --- arch/mips/configs/pistachio_defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/mips/configs/pistachio_defconfig b/a

[PATCH 1/1 linux-next] mips: pistachio_defconfig: remove ANDROID_TIMED_OUTPUT

2016-08-12 Thread Fabian Frederick
According to commit 9f6b68774f29 ("android: remove timed output/gpio driver") Signed-off-by: Fabian Frederick --- arch/mips/configs/pistachio_defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/mips/configs/pistachio_defconfig b/arch/mips/configs/pistachio_defconfig ind

[PATCH 1/1 linux-next] iommu/amd: remove AMD_IOMMU_STATS

2016-08-12 Thread Fabian Frederick
Commit e85e8f69cedb ("iommu/amd: Remove statistics code") removed that config. Signed-off-by: Fabian Frederick <f...@skynet.be> --- arch/x86/configs/x86_64_defconfig | 1 - drivers/iommu/amd_iommu_proto.h | 4 2 files changed, 5 deletions(-) diff --git a/arch/x86/configs/

[PATCH 1/1 linux-next] iommu/amd: remove AMD_IOMMU_STATS

2016-08-12 Thread Fabian Frederick
Commit e85e8f69cedb ("iommu/amd: Remove statistics code") removed that config. Signed-off-by: Fabian Frederick --- arch/x86/configs/x86_64_defconfig | 1 - drivers/iommu/amd_iommu_proto.h | 4 2 files changed, 5 deletions(-) diff --git a/arch/x86/configs/x86_64_defconfig

[PATCH 1/1 linux-next] Documentation/features/core: update HAVE_BPF_JIT

2016-08-12 Thread Fabian Frederick
Update documentation according to commit 606b5908 ("bpf: split HAVE_BPF_JIT into cBPF and eBPF variant") Signed-off-by: Fabian Frederick <f...@skynet.be> --- Documentation/features/core/BPF-JIT/arch-support.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 1/1 linux-next] Documentation/features/core: update HAVE_BPF_JIT

2016-08-12 Thread Fabian Frederick
Update documentation according to commit 606b5908 ("bpf: split HAVE_BPF_JIT into cBPF and eBPF variant") Signed-off-by: Fabian Frederick --- Documentation/features/core/BPF-JIT/arch-support.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/fea

[PATCH 1/1 linux-next] ARM: imx: remove platform-mxc_rnga

2016-08-12 Thread Fabian Frederick
According to commit c9ee94965dce ("ARM: imx: deconstruct mxc_rnga initialization") Signed-off-by: Fabian Frederick <f...@skynet.be> --- arch/arm/mach-imx/devices/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-imx/devices/Makefile b/arch/arm/mach-imx/

[PATCH 1/1 linux-next] ARM: imx: remove platform-mxc_rnga

2016-08-12 Thread Fabian Frederick
According to commit c9ee94965dce ("ARM: imx: deconstruct mxc_rnga initialization") Signed-off-by: Fabian Frederick --- arch/arm/mach-imx/devices/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-imx/devices/Makefile b/arch/arm/mach-imx/devices/Makefile ind

[PATCH 1/1 linux-next] hwrng: update Freescale i.MX RNGA Random Number Generator

2016-08-12 Thread Fabian Frederick
We can directly depend on SOC_IMX31 since commit c9ee94965dce ("ARM: imx: deconstruct mxc_rnga initialization") Signed-off-by: Fabian Frederick <f...@skynet.be> --- drivers/char/hw_random/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/char/hw_

[PATCH 1/1 linux-next] hwrng: update Freescale i.MX RNGA Random Number Generator

2016-08-12 Thread Fabian Frederick
We can directly depend on SOC_IMX31 since commit c9ee94965dce ("ARM: imx: deconstruct mxc_rnga initialization") Signed-off-by: Fabian Frederick --- drivers/char/hw_random/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/char/hw_random/Kconfig b/dr

[RFC linux-next] ARM: clps711x: update defconfig

2016-08-12 Thread Fabian Frederick
sync defconfig to commit 4a56f46a7dc6 ("ARM: clps711x: Remove boards support") or maybe we could just remove it ? Signed-off-by: Fabian Frederick <f...@skynet.be> --- arch/arm/configs/clps711x_defconfig | 5 - 1 file changed, 5 deletions(-) diff --git a/arch/arm/configs/cl

[RFC linux-next] ARM: clps711x: update defconfig

2016-08-12 Thread Fabian Frederick
sync defconfig to commit 4a56f46a7dc6 ("ARM: clps711x: Remove boards support") or maybe we could just remove it ? Signed-off-by: Fabian Frederick --- arch/arm/configs/clps711x_defconfig | 5 - 1 file changed, 5 deletions(-) diff --git a/arch/arm/configs/clps711x_defconfig

[PATCH 1/1 linux-next] CRIS: defconfig: remove MTDRAM_ABS_POS

2016-08-12 Thread Fabian Frederick
According to commit ef158bdf8374 ("mtd: Remove unused symbol CONFIG_MTDRAM_ABS_POS") Signed-off-by: Fabian Frederick <f...@skynet.be> --- arch/cris/configs/artpec_3_defconfig | 1 - arch/cris/configs/etrax-100lx_v2_defconfig | 1 - arch/cris/configs/etraxfs_defconfig

[PATCH 1/1 linux-next] CRIS: defconfig: remove MTDRAM_ABS_POS

2016-08-12 Thread Fabian Frederick
According to commit ef158bdf8374 ("mtd: Remove unused symbol CONFIG_MTDRAM_ABS_POS") Signed-off-by: Fabian Frederick --- arch/cris/configs/artpec_3_defconfig | 1 - arch/cris/configs/etrax-100lx_v2_defconfig | 1 - arch/cris/configs/etraxfs_defconfig| 1 - 3 files

[PATCH 1/1 linux-next] selftests/zram: replace ZRAM_LZ4_COMPRESS

2016-08-12 Thread Fabian Frederick
Since commit ce1ed9f98e88 ("zram: delete custom lzo/lz4") we need CONFIG_CRYPTO_LZ4=y instead of CONFIG_ZRAM_LZ4_COMPRESS Signed-off-by: Fabian Frederick <f...@skynet.be> --- tools/testing/selftests/zram/README | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 1/1 linux-next] selftests/zram: replace ZRAM_LZ4_COMPRESS

2016-08-12 Thread Fabian Frederick
Since commit ce1ed9f98e88 ("zram: delete custom lzo/lz4") we need CONFIG_CRYPTO_LZ4=y instead of CONFIG_ZRAM_LZ4_COMPRESS Signed-off-by: Fabian Frederick --- tools/testing/selftests/zram/README | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/testing/self

[PATCH 1/1 linux-next] ARM: multi_v7_defconfig: update XILINX_VDMA

2016-08-11 Thread Fabian Frederick
Commit fde57a7c4474 ("dmaengine: xilinx: Rename driver and config") renamed config XILINX_VDMA to config XILINX_DMA Update defconfig accordingly. Signed-off-by: Fabian Frederick <f...@skynet.be> --- arch/arm/configs/multi_v7_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 d

[PATCH 1/1 linux-next] ARM: multi_v7_defconfig: update XILINX_VDMA

2016-08-11 Thread Fabian Frederick
Commit fde57a7c4474 ("dmaengine: xilinx: Rename driver and config") renamed config XILINX_VDMA to config XILINX_DMA Update defconfig accordingly. Signed-off-by: Fabian Frederick --- arch/arm/configs/multi_v7_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 1/1 linux-next] net: hns: fix typo in g_gmac_stats_string[]

2016-08-10 Thread Fabian Frederick
s/gamc/gmac/ Signed-off-by: Fabian Frederick <f...@skynet.be> --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_gmac.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/hisilicon/hns/hns_dsaf_gmac.c b/drivers/net/ethernet/hisilicon/hns/hns_dsaf_gmac.c

[PATCH 1/1 linux-next] net: hns: fix typo in g_gmac_stats_string[]

2016-08-10 Thread Fabian Frederick
s/gamc/gmac/ Signed-off-by: Fabian Frederick --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_gmac.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/hisilicon/hns/hns_dsaf_gmac.c b/drivers/net/ethernet/hisilicon/hns/hns_dsaf_gmac.c index 1235c7f..1e1eb92

[PATCH 1/1 linux-next] sysv, ipc: fix security-layer leaking

2016-08-01 Thread Fabian Frederick
hr Bueso to only use ipc_rcu_free in case of security allocation failure in newary() Signed-off-by: Fabian Frederick <f...@skynet.be> --- This patch replaces "ipc/msg.c: fix memory leak in do_msgsnd()" Semaphore update is untested. ipc/msg.c | 2 +- ipc/sem.c | 12 ++-- 2 files c

[PATCH 1/1 linux-next] sysv, ipc: fix security-layer leaking

2016-08-01 Thread Fabian Frederick
hr Bueso to only use ipc_rcu_free in case of security allocation failure in newary() Signed-off-by: Fabian Frederick --- This patch replaces "ipc/msg.c: fix memory leak in do_msgsnd()" Semaphore update is untested. ipc/msg.c | 2 +- ipc/sem.c | 12 ++-- 2 files changed, 7 insertions(

Re: [PATCH 1/1 linux-next] ipc/msg.c: fix memory leak in do_msgsnd()

2016-08-01 Thread Fabian Frederick
> On 31 July 2016 at 16:36 Manfred Spraul <manf...@colorfullife.com> wrote: > > > Hi Fabian, > > On 07/29/2016 10:15 AM, Fabian Frederick wrote: > > Running LTP msgsnd06 with kmemleak gives the following: > > > > cat /sys/kernel/debug/kmemleak > >

Re: [PATCH 1/1 linux-next] ipc/msg.c: fix memory leak in do_msgsnd()

2016-08-01 Thread Fabian Frederick
> On 31 July 2016 at 16:36 Manfred Spraul wrote: > > > Hi Fabian, > > On 07/29/2016 10:15 AM, Fabian Frederick wrote: > > Running LTP msgsnd06 with kmemleak gives the following: > > > > cat /sys/kernel/debug/kmemleak > > > > unreferenced object 0x

[PATCH V2 linux-next] ipc/msg.c: fix memory leak in do_msgsnd()

2016-08-01 Thread Fabian Frederick
loc_trace+0xe1/0x180 [] selinux_msg_queue_alloc_security+0x3f/0xd0 [] security_msg_queue_alloc+0x2e/0x40 [] newque+0x4e/0x150 [] ipcget+0x159/0x1b0 [] SyS_msgget+0x39/0x40 [] entry_SYSCALL_64_fastpath+0x13/0x8f [] 0xffff Signed-off-by: Fabian Frederick <f...@skynet.be>

[PATCH V2 linux-next] ipc/msg.c: fix memory leak in do_msgsnd()

2016-08-01 Thread Fabian Frederick
loc_trace+0xe1/0x180 [] selinux_msg_queue_alloc_security+0x3f/0xd0 [] security_msg_queue_alloc+0x2e/0x40 [] newque+0x4e/0x150 [] ipcget+0x159/0x1b0 [] SyS_msgget+0x39/0x40 [] entry_SYSCALL_64_fastpath+0x13/0x8f [] 0xffff Signed-off-by: Fabian Frederick --- V2: Update description with or

[PATCH 1/1 linux-next] GFS2: remove inline on static in c file

2016-07-31 Thread Fabian Frederick
' +static void gfs2_update_stats(struct gfs2_lkstats *s, unsigned index, Inspired-by: "David S. Miller" <da...@davemloft.net> Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/gfs2/bmap.c | 15 --- fs/gfs2/dir.c | 4 ++-- fs/gfs2/glock.c

[PATCH 1/1 linux-next] GFS2: remove inline on static in c file

2016-07-31 Thread Fabian Frederick
' +static void gfs2_update_stats(struct gfs2_lkstats *s, unsigned index, Inspired-by: "David S. Miller" Signed-off-by: Fabian Frederick --- fs/gfs2/bmap.c | 15 --- fs/gfs2/dir.c | 4 ++-- fs/gfs2/glock.c| 8 fs/gfs2/lock_dlm.c | 6 +++--- fs/

[PATCH 1/1 linux-next] ipc/msg.c: fix memory leak in do_msgsnd()

2016-07-29 Thread Fabian Frederick
+0x13/0x8f [] 0x ipc_rcu_free() was given to ipc_rcu_putref() instead of msg_rcu_free() which does security cleaning. Signed-off-by: Fabian Frederick <f...@skynet.be> --- ipc/msg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ipc/msg.c b/ipc/msg.c index

[PATCH 1/1 linux-next] ipc/msg.c: fix memory leak in do_msgsnd()

2016-07-29 Thread Fabian Frederick
+0x13/0x8f [] 0x ipc_rcu_free() was given to ipc_rcu_putref() instead of msg_rcu_free() which does security cleaning. Signed-off-by: Fabian Frederick --- ipc/msg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ipc/msg.c b/ipc/msg.c index 59559a2..43e9631 100644

Re: [PATCH 1/1 linux-next] kbuild: add make force=1 for testing

2016-07-27 Thread Fabian Frederick
> On 26 July 2016 at 23:47 Michal Marek <mma...@suse.com> wrote: > > > Dne 26.7.2016 v 02:05 Andrew Morton napsal(a): > > On Sun, 24 Jul 2016 15:28:18 +0200 Fabian Frederick <f...@skynet.be> wrote: > >> This patch adds ability to do make force=1 for people

Re: [PATCH 1/1 linux-next] kbuild: add make force=1 for testing

2016-07-27 Thread Fabian Frederick
> On 26 July 2016 at 23:47 Michal Marek wrote: > > > Dne 26.7.2016 v 02:05 Andrew Morton napsal(a): > > On Sun, 24 Jul 2016 15:28:18 +0200 Fabian Frederick wrote: > >> This patch adds ability to do make force=1 for people > >> not interested in modules_i

[PATCH 1/1 linux-next] GFS2: use BIT() macro

2016-07-24 Thread Fabian Frederick
Replace 1 << value shift by more explicit BIT() macro Also fixes two bare unsigned definitions: WARNING: Prefer 'unsigned int' to bare use of 'unsigned' + unsigned hsize = BIT(ip->i_depth); Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/gfs2/aops.c |

[PATCH 1/1 linux-next] GFS2: use BIT() macro

2016-07-24 Thread Fabian Frederick
Replace 1 << value shift by more explicit BIT() macro Also fixes two bare unsigned definitions: WARNING: Prefer 'unsigned int' to bare use of 'unsigned' + unsigned hsize = BIT(ip->i_depth); Signed-off-by: Fabian Frederick --- fs/gfs2/aops.c | 2 +- fs/gf

[PATCH 1/1 linux-next] kbuild: add make force=1 for testing

2016-07-24 Thread Fabian Frederick
ries but volume name is not always writable. This patch adds ability to do make force=1 for people not interested in modules_install in this case but only testing. (Note that other options could go under ifndef force) Signed-off-by: Fabian Frederick <f...@skynet.be> --- Makefile | 5 - 1

[PATCH 1/1 linux-next] kbuild: add make force=1 for testing

2016-07-24 Thread Fabian Frederick
ries but volume name is not always writable. This patch adds ability to do make force=1 for people not interested in modules_install in this case but only testing. (Note that other options could go under ifndef force) Signed-off-by: Fabian Frederick --- Makefile | 5 - 1 file changed, 4 inser

[PATCH 8/8 net-next] irda: ma600-sir: use module_sirdev_driver() helper

2016-07-13 Thread Fabian Frederick
Avoid using generic init/empty functions. Signed-off-by: Fabian Frederick <f...@skynet.be> --- drivers/net/irda/ma600-sir.c | 17 ++--- 1 file changed, 2 insertions(+), 15 deletions(-) diff --git a/drivers/net/irda/ma600-sir.c b/drivers/net/irda/ma600-sir.c index a764817..a

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