Re: constification and cocci / kernel build test robot ?

2016-08-28 Thread Julia Lawall
On Sat, 27 Aug 2016, Joe Perches wrote: > On Sat, 2016-08-27 at 20:59 +0200, Julia Lawall wrote: > >  Make sure (of/i2c/platform)_device_id tables are NULL terminated > > Generated by: scripts/coccinelle/misc/of_table.cocci > > Along the same lines, I submitted a manually generated > patch to

Re: constification and cocci / kernel build test robot ?

2016-08-28 Thread Julia Lawall
On Sat, 27 Aug 2016, Joe Perches wrote: > On Sat, 2016-08-27 at 20:59 +0200, Julia Lawall wrote: > >  Make sure (of/i2c/platform)_device_id tables are NULL terminated > > Generated by: scripts/coccinelle/misc/of_table.cocci > > Along the same lines, I submitted a manually generated > patch to

psmouse_disconnect lockdep splat

2016-08-28 Thread Borislav Petkov
Hi, just saw this while booting rc3+tip/master. Thought I should report it in case it hasn't been caught and fixed yet. --- ... [6.257924] snd_hda_codec_realtek hdaudioC0D0: Internal Mic=0x12 [6.280232] iTCO_vendor_support: vendor-support=0 [6.282063] snd_hda_intel

psmouse_disconnect lockdep splat

2016-08-28 Thread Borislav Petkov
Hi, just saw this while booting rc3+tip/master. Thought I should report it in case it hasn't been caught and fixed yet. --- ... [6.257924] snd_hda_codec_realtek hdaudioC0D0: Internal Mic=0x12 [6.280232] iTCO_vendor_support: vendor-support=0 [6.282063] snd_hda_intel

Re: [PATCH 1/5] IA64-IRQ: Use kmalloc_array() in sn_irq_lh_init()

2016-08-28 Thread Julia Lawall
On Sat, 27 Aug 2016, Joe Perches wrote: > On Sat, 2016-08-27 at 09:02 +0200, SF Markus Elfring wrote: > > > If you _really wanted to clear up this code and make it more > > > robust/better, it'd probably be nicer to convert the > > > struct list_head **sn_irq_lh to a single struct list_head * >

Re: [PATCH 1/5] IA64-IRQ: Use kmalloc_array() in sn_irq_lh_init()

2016-08-28 Thread Julia Lawall
On Sat, 27 Aug 2016, Joe Perches wrote: > On Sat, 2016-08-27 at 09:02 +0200, SF Markus Elfring wrote: > > > If you _really wanted to clear up this code and make it more > > > robust/better, it'd probably be nicer to convert the > > > struct list_head **sn_irq_lh to a single struct list_head * >

Re: [PATCH 3/5] rxrpc: fix last_call processing

2016-08-28 Thread David Howells
This is fixed by: commit 2266ffdef5737fdfa96005204fc5606dbd559956 subject: rxrpc: Fix conn-based retransmit which is in net-next. David

Re: [PATCH 3/5] rxrpc: fix last_call processing

2016-08-28 Thread David Howells
This is fixed by: commit 2266ffdef5737fdfa96005204fc5606dbd559956 subject: rxrpc: Fix conn-based retransmit which is in net-next. David

Re: [PATCH v2] ARM: dts: sun8i: enable UART1 for iNet D978 Rev2 board

2016-08-28 Thread Rask Ingemann Lambertsen
On Sun, Aug 28, 2016 at 08:55:21AM +0800, Icenowy Zheng wrote: > + uart1_pins_cts_rts_a: uart1_cts_rts@0 { > + allwinner,pins = "PG8", "PG9"; > + allwinner,function = "uart1"; > +

Re: [PATCH v2] ARM: dts: sun8i: enable UART1 for iNet D978 Rev2 board

2016-08-28 Thread Rask Ingemann Lambertsen
On Sun, Aug 28, 2016 at 08:55:21AM +0800, Icenowy Zheng wrote: > + uart1_pins_cts_rts_a: uart1_cts_rts@0 { > + allwinner,pins = "PG8", "PG9"; > + allwinner,function = "uart1"; > +

undefined reference to `dib3000mc_pid_parse'

2016-08-28 Thread kbuild test robot
Hi Joe, It's probably a bug fix that unveils the link errors. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: af56ff27eba54fceee5f5643e79bf6531f2e1739 commit: cb984d101b30eb7478d32df56a0023e4603cba7f compiler-gcc: integrate the various

undefined reference to `dib3000mc_pid_parse'

2016-08-28 Thread kbuild test robot
Hi Joe, It's probably a bug fix that unveils the link errors. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: af56ff27eba54fceee5f5643e79bf6531f2e1739 commit: cb984d101b30eb7478d32df56a0023e4603cba7f compiler-gcc: integrate the various

Re: [PATCH][RFC v4] timekeeping: ignore the bogus sleep time if pm_trace is enabled

2016-08-28 Thread Chen Yu
On Sat, Aug 27, 2016 at 03:08:56PM +0800, Xunlei Pang wrote: > On 2016/08/18 at 18:43, Chen Yu wrote: > > Previously we encountered some memory overflow issues due to > > the bogus sleep time brought by inconsistent rtc, which is > > triggered when pm_trace is enabled, please refer to: > >

Re: [PATCH][RFC v4] timekeeping: ignore the bogus sleep time if pm_trace is enabled

2016-08-28 Thread Chen Yu
On Sat, Aug 27, 2016 at 03:08:56PM +0800, Xunlei Pang wrote: > On 2016/08/18 at 18:43, Chen Yu wrote: > > Previously we encountered some memory overflow issues due to > > the bogus sleep time brought by inconsistent rtc, which is > > triggered when pm_trace is enabled, please refer to: > >

Re: [RFC v2 09/10] landlock: Handle cgroups

2016-08-28 Thread Andy Lutomirski
On Aug 27, 2016 8:12 PM, "Alexei Starovoitov" wrote: > > On Sat, Aug 27, 2016 at 12:30:36AM -0700, Andy Lutomirski wrote: > > > cgroup is the common way to group multiple tasks. > > > Without cgroup only parent<->child relationship will be possible, > > > which will

Re: [RFC v2 09/10] landlock: Handle cgroups

2016-08-28 Thread Andy Lutomirski
On Aug 27, 2016 8:12 PM, "Alexei Starovoitov" wrote: > > On Sat, Aug 27, 2016 at 12:30:36AM -0700, Andy Lutomirski wrote: > > > cgroup is the common way to group multiple tasks. > > > Without cgroup only parent<->child relationship will be possible, > > > which will limit usability of such lsm to

Re: [RFC v2 09/10] landlock: Handle cgroups (performance)

2016-08-28 Thread Andy Lutomirski
On Aug 27, 2016 11:14 PM, "Mickaël Salaün" wrote: > > > On 27/08/2016 22:43, Alexei Starovoitov wrote: > > On Sat, Aug 27, 2016 at 09:35:14PM +0200, Mickaël Salaün wrote: > >> On 27/08/2016 20:06, Alexei Starovoitov wrote: > >>> On Sat, Aug 27, 2016 at 04:06:38PM +0200, Mickaël

Re: [RFC v2 09/10] landlock: Handle cgroups (performance)

2016-08-28 Thread Andy Lutomirski
On Aug 27, 2016 11:14 PM, "Mickaël Salaün" wrote: > > > On 27/08/2016 22:43, Alexei Starovoitov wrote: > > On Sat, Aug 27, 2016 at 09:35:14PM +0200, Mickaël Salaün wrote: > >> On 27/08/2016 20:06, Alexei Starovoitov wrote: > >>> On Sat, Aug 27, 2016 at 04:06:38PM +0200, Mickaël Salaün wrote: >

Re: [PATCH RFC UGLY] x86,mm,sched: make lazy TLB mode even lazier

2016-08-28 Thread Andy Lutomirski
On Aug 25, 2016 9:06 PM, "Rik van Riel" wrote: > > Subject: x86,mm,sched: make lazy TLB mode even lazier > > Lazy TLB mode can result in an idle CPU being woken up for a TLB > flush, when all it really needed to do was flush %cr3 before the > next context switch. > > This is

Re: [PATCH RFC UGLY] x86,mm,sched: make lazy TLB mode even lazier

2016-08-28 Thread Andy Lutomirski
On Aug 25, 2016 9:06 PM, "Rik van Riel" wrote: > > Subject: x86,mm,sched: make lazy TLB mode even lazier > > Lazy TLB mode can result in an idle CPU being woken up for a TLB > flush, when all it really needed to do was flush %cr3 before the > next context switch. > > This is mostly fine on bare

Re: [PATCH 3/3] scsi/ncr5380: Improve interrupt latency during PIO tranfers

2016-08-28 Thread Geert Uytterhoeven
Hi Finn, On Sat, Aug 27, 2016 at 4:30 AM, Finn Thain wrote: > Large PIO transfers are broken up into chunks to try to avoid disabling > local IRQs for long periods. But IRQs are still disabled for too long > and this causes SCC FIFO overruns during serial port

Re: [PATCH 3/3] scsi/ncr5380: Improve interrupt latency during PIO tranfers

2016-08-28 Thread Geert Uytterhoeven
Hi Finn, On Sat, Aug 27, 2016 at 4:30 AM, Finn Thain wrote: > Large PIO transfers are broken up into chunks to try to avoid disabling > local IRQs for long periods. But IRQs are still disabled for too long > and this causes SCC FIFO overruns during serial port transfers. This > patch fixes the

Re: [PATCH 2/3] scsi/ncr5380: Avoid a compiler warning

2016-08-28 Thread Geert Uytterhoeven
On Sat, Aug 27, 2016 at 4:30 AM, Finn Thain wrote: > With commit 3a0f64bfa907 ("mac_scsi: Fix pseudo DMA implementation") > some versions of gcc now warn: > > In file included from drivers/scsi/mac_scsi.c:335: > drivers/scsi/NCR5380.h:295: warning:

Re: [PATCH 2/3] scsi/ncr5380: Avoid a compiler warning

2016-08-28 Thread Geert Uytterhoeven
On Sat, Aug 27, 2016 at 4:30 AM, Finn Thain wrote: > With commit 3a0f64bfa907 ("mac_scsi: Fix pseudo DMA implementation") > some versions of gcc now warn: > > In file included from drivers/scsi/mac_scsi.c:335: > drivers/scsi/NCR5380.h:295: warning: `NCR5380_poll_politely' declared inline > after

Re: [PATCH 1/3] documentation/scsi: Remove nodisconnect parameter

2016-08-28 Thread Geert Uytterhoeven
On Sat, Aug 27, 2016 at 4:29 AM, Finn Thain wrote: > The driver that used the 'nodisconnect' parameter was removed in > commit 565bae6a4a8f ("[SCSI] 53c7xx: kill driver"). Related documentation > was cleaned up in commit f37a7238d379 ("[SCSI] 53c7xx: fix removal >

Re: [PATCH 1/3] documentation/scsi: Remove nodisconnect parameter

2016-08-28 Thread Geert Uytterhoeven
On Sat, Aug 27, 2016 at 4:29 AM, Finn Thain wrote: > The driver that used the 'nodisconnect' parameter was removed in > commit 565bae6a4a8f ("[SCSI] 53c7xx: kill driver"). Related documentation > was cleaned up in commit f37a7238d379 ("[SCSI] 53c7xx: fix removal > fallout"), except for the

[PATCH] scsi: ufs: add missing declaration in ufshcd.h

2016-08-28 Thread Baoyou Xie
We get 1 warning about global functions without a declaration in the scsi ufshcd driver when building with W=1: drivers/scsi/ufs/ufshcd.c:1991:5: warning: no previous prototype for 'ufshcd_query_descriptor_retry' [-Wmissing-prototypes] in fact, this function is implemented in ufshcd.c and

[PATCH] scsi: ufs: add missing declaration in ufshcd.h

2016-08-28 Thread Baoyou Xie
We get 1 warning about global functions without a declaration in the scsi ufshcd driver when building with W=1: drivers/scsi/ufs/ufshcd.c:1991:5: warning: no previous prototype for 'ufshcd_query_descriptor_retry' [-Wmissing-prototypes] in fact, this function is implemented in ufshcd.c and

Re: [Ksummit-discuss] checkkpatch (in)sanity ?

2016-08-28 Thread Alexey Dobriyan
On Sat, Aug 27, 2016 at 09:06:13PM -0400, Levin, Alexander via Ksummit-discuss wrote: > 3. This one is somewhat subjective: scripts/checkpatch.pl is a massive blob of > perl code that a fair amount of people don't know how to deal with. In 4.8 > it's > 6142 lines, making it the 124th largest

Re: [Ksummit-discuss] checkkpatch (in)sanity ?

2016-08-28 Thread Alexey Dobriyan
On Sat, Aug 27, 2016 at 09:06:13PM -0400, Levin, Alexander via Ksummit-discuss wrote: > 3. This one is somewhat subjective: scripts/checkpatch.pl is a massive blob of > perl code that a fair amount of people don't know how to deal with. In 4.8 > it's > 6142 lines, making it the 124th largest

Re: [PATCH 1/2] mm/nobootmem.c: make CONFIG_NO_BOOTMEM depend on CONFIG_HAVE_MEMBLOCK

2016-08-28 Thread kbuild test robot
Hi zijun_hu, [auto build test ERROR on mmotm/master] [also build test ERROR on v4.8-rc3 next-20160825] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenience) to record

Re: [PATCH 1/2] mm/nobootmem.c: make CONFIG_NO_BOOTMEM depend on CONFIG_HAVE_MEMBLOCK

2016-08-28 Thread kbuild test robot
Hi zijun_hu, [auto build test ERROR on mmotm/master] [also build test ERROR on v4.8-rc3 next-20160825] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenience) to record

Re: IA64-IRQ: Use kmalloc_array() in sn_irq_lh_init()

2016-08-28 Thread SF Markus Elfring
> While elimination of code duplication should be good, > what it means it you should avoid making changes that > are merely mechanical and strive to make changes that > improve code execution speed or reduce overall object > size while not impacting overall execution speed. Do other contributors

Re: IA64-IRQ: Use kmalloc_array() in sn_irq_lh_init()

2016-08-28 Thread SF Markus Elfring
> While elimination of code duplication should be good, > what it means it you should avoid making changes that > are merely mechanical and strive to make changes that > improve code execution speed or reduce overall object > size while not impacting overall execution speed. Do other contributors

[PATCH] fix:nouveau_display: mark symbols static where possible

2016-08-28 Thread Baoyou Xie
We get 1 warning about global functions without a declaration in the nouveau nouveau_display driver when building with W=1: drivers/gpu/drm/nouveau/nouveau_display.c:96:1: warning: no previous prototype for 'nouveau_display_scanoutpos_head' [-Wmissing-prototypes] In fact, this function is only

[PATCH] fix:nouveau_display: mark symbols static where possible

2016-08-28 Thread Baoyou Xie
We get 1 warning about global functions without a declaration in the nouveau nouveau_display driver when building with W=1: drivers/gpu/drm/nouveau/nouveau_display.c:96:1: warning: no previous prototype for 'nouveau_display_scanoutpos_head' [-Wmissing-prototypes] In fact, this function is only

Re: [PATCH 5/8] cris-cryptocop: Move an assignment for the variable "nooutpages" in cryptocop_ioctl_process()

2016-08-28 Thread SF Markus Elfring
>> +++ b/arch/cris/arch-v32/drivers/cryptocop.c >> @@ -2469,7 +2469,7 @@ static int cryptocop_ioctl_process(struct inode >> *inode, struct file *filp, unsig >> struct page **inpages = NULL; >> struct page **outpages = NULL; >> int

Re: [PATCH 5/8] cris-cryptocop: Move an assignment for the variable "nooutpages" in cryptocop_ioctl_process()

2016-08-28 Thread SF Markus Elfring
>> +++ b/arch/cris/arch-v32/drivers/cryptocop.c >> @@ -2469,7 +2469,7 @@ static int cryptocop_ioctl_process(struct inode >> *inode, struct file *filp, unsig >> struct page **inpages = NULL; >> struct page **outpages = NULL; >> int

Re: [PATCH 8/8] cris-cryptocop: Apply another recommendation from "checkpatch.pl"

2016-08-28 Thread SF Markus Elfring
>> @@ -2276,7 +2277,10 @@ static int cryptocop_job_setup(struct >> cryptocop_prio_job **pj, struct cryptocop_ >> (*pj)->iop->ctx_in.saved_data = operation->list_op.inlist; >> (*pj)->iop->ctx_in.saved_data_buf = >> operation->list_op.in_data_buf; >> } else { >> -

Re: [PATCH 8/8] cris-cryptocop: Apply another recommendation from "checkpatch.pl"

2016-08-28 Thread SF Markus Elfring
>> @@ -2276,7 +2277,10 @@ static int cryptocop_job_setup(struct >> cryptocop_prio_job **pj, struct cryptocop_ >> (*pj)->iop->ctx_in.saved_data = operation->list_op.inlist; >> (*pj)->iop->ctx_in.saved_data_buf = >> operation->list_op.in_data_buf; >> } else { >> -

[PATCH] fix:nv10_fence: mark symbols static where possible

2016-08-28 Thread Baoyou Xie
We get 1 warning about global functions without a declaration in the nouveau nv10_fence driver when building with W=1: drivers/gpu/drm/nouveau/nv10_fence.c:70:1: warning: no previous prototype for 'nv10_fence_context_new' [-Wmissing-prototypes] In fact, this function is only used in the file in

[PATCH] fix:nv10_fence: mark symbols static where possible

2016-08-28 Thread Baoyou Xie
We get 1 warning about global functions without a declaration in the nouveau nv10_fence driver when building with W=1: drivers/gpu/drm/nouveau/nv10_fence.c:70:1: warning: no previous prototype for 'nv10_fence_context_new' [-Wmissing-prototypes] In fact, this function is only used in the file in

Re: [PATCH] arch: all: include: asm: bitops: Use bool instead of int for all bit test functions

2016-08-28 Thread kbuild test robot
ons/20160828-134633 config: arm-at91_dt_defconfig (attached as .config) compiler: arm-linux-gnueabi-gcc (Debian 5.4.0-6) 5.4.0 20160609 reproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cr

Re: [PATCH] arch: all: include: asm: bitops: Use bool instead of int for all bit test functions

2016-08-28 Thread kbuild test robot
ons/20160828-134633 config: arm-at91_dt_defconfig (attached as .config) compiler: arm-linux-gnueabi-gcc (Debian 5.4.0-6) 5.4.0 20160609 reproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cr

Re: [PATCH] arch: all: include: asm: bitops: Use bool instead of int for all bit test functions

2016-08-28 Thread kbuild test robot
hat (public, well-known) commit your patch series was built on] [Check https://git-scm.com/docs/git-format-patch for more information] url: https://github.com/0day-ci/linux/commits/chengang-emindsoft-com-cn/arch-all-include-asm-bitops-Use-bool-instead-of-int-for-all-bit-test-functions/20160

Re: [PATCH] arch: all: include: asm: bitops: Use bool instead of int for all bit test functions

2016-08-28 Thread kbuild test robot
hat (public, well-known) commit your patch series was built on] [Check https://git-scm.com/docs/git-format-patch for more information] url: https://github.com/0day-ci/linux/commits/chengang-emindsoft-com-cn/arch-all-include-asm-bitops-Use-bool-instead-of-int-for-all-bit-test-functions/20160

[PATCH v2] fix:iio:common:st_sensors:st_sensors_trigger:mark symbols static where possible

2016-08-28 Thread Baoyou Xie
We get 2 warnings when biuld kernel with W=1: drivers/iio/common/st_sensors/st_sensors_trigger.c:69:13: warning: no previous prototype for 'st_sensors_irq_handler' [-Wmissing-prototypes] drivers/iio/common/st_sensors/st_sensors_trigger.c:85:13: warning: no previous prototype for

[PATCH v2] fix:iio:common:st_sensors:st_sensors_trigger:mark symbols static where possible

2016-08-28 Thread Baoyou Xie
We get 2 warnings when biuld kernel with W=1: drivers/iio/common/st_sensors/st_sensors_trigger.c:69:13: warning: no previous prototype for 'st_sensors_irq_handler' [-Wmissing-prototypes] drivers/iio/common/st_sensors/st_sensors_trigger.c:85:13: warning: no previous prototype for

Re: [PATCH v05 06/72] include/uapi/linux/hsi/cs-protocol.h: include linux/time.h

2016-08-28 Thread Mikko Rapeli
On Mon, Aug 22, 2016 at 10:52:20PM +0200, Sebastian Reichel wrote: > Hi, > > On Mon, Aug 22, 2016 at 08:32:23PM +0200, Mikko Rapeli wrote: > > Fixes userspace compilation errors due to missing timespec definition. > > > > Signed-off-by: Mikko Rapeli > > --- > >

Re: [PATCH v05 06/72] include/uapi/linux/hsi/cs-protocol.h: include linux/time.h

2016-08-28 Thread Mikko Rapeli
On Mon, Aug 22, 2016 at 10:52:20PM +0200, Sebastian Reichel wrote: > Hi, > > On Mon, Aug 22, 2016 at 08:32:23PM +0200, Mikko Rapeli wrote: > > Fixes userspace compilation errors due to missing timespec definition. > > > > Signed-off-by: Mikko Rapeli > > --- > >

Re: [PATCH] arch: all: include: asm: bitops: Use bool instead of int for all bit test functions

2016-08-28 Thread kbuild test robot
ons/20160828-134633 config: m68k-sun3_defconfig (attached as .config) compiler: m68k-linux-gcc (GCC) 4.9.0 reproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .con

Re: [PATCH] arch: all: include: asm: bitops: Use bool instead of int for all bit test functions

2016-08-28 Thread kbuild test robot
ons/20160828-134633 config: m68k-sun3_defconfig (attached as .config) compiler: m68k-linux-gcc (GCC) 4.9.0 reproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .con

[PATCH] memstick: r592: add sanity check in r592.c

2016-08-28 Thread Baoyou Xie
in function memstick_debug_get_tpc_name, it miss sanity check and will cause a panic. so this patch checks if the tpc parameter is valid, and if not so, it returns a dummy value to caller. Signed-off-by: Baoyou Xie --- drivers/memstick/host/r592.c | 3 +++ 1 file

[PATCH v2 3/5] checkkconfigsymbols.py: fix pylint and pep8 warnings

2016-08-28 Thread Valentin Rothberg
Fix pylint and pep8 warnings to have a consistent syntax and style. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 18 -- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/scripts/checkkconfigsymbols.py

[PATCH] memstick: r592: add sanity check in r592.c

2016-08-28 Thread Baoyou Xie
in function memstick_debug_get_tpc_name, it miss sanity check and will cause a panic. so this patch checks if the tpc parameter is valid, and if not so, it returns a dummy value to caller. Signed-off-by: Baoyou Xie --- drivers/memstick/host/r592.c | 3 +++ 1 file changed, 3 insertions(+) diff

[PATCH v2 3/5] checkkconfigsymbols.py: fix pylint and pep8 warnings

2016-08-28 Thread Valentin Rothberg
Fix pylint and pep8 warnings to have a consistent syntax and style. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 18 -- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/scripts/checkkconfigsymbols.py b/scripts/checkkconfigsymbols.py index

[PATCH v2 4/5] checkkconfigsymblos: consistent symbol terminology

2016-08-28 Thread Valentin Rothberg
'symbol' and 'feature' are used synonymously to refer to Kconfig symbols (configs, menus, etc.). Use the term 'symbol' to have a consistent terminology and to make the code more comprehensible. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py |

[PATCH v2 1/5] checkkconfigsymbols.py: port to Python 3

2016-08-28 Thread Valentin Rothberg
Python 2 is slowly dying, so port the script to Python 3. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 33 + 1 file changed, 17 insertions(+), 16 deletions(-) diff --git a/scripts/checkkconfigsymbols.py

[PATCH v2 4/5] checkkconfigsymblos: consistent symbol terminology

2016-08-28 Thread Valentin Rothberg
'symbol' and 'feature' are used synonymously to refer to Kconfig symbols (configs, menus, etc.). Use the term 'symbol' to have a consistent terminology and to make the code more comprehensible. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 134

[PATCH v2 1/5] checkkconfigsymbols.py: port to Python 3

2016-08-28 Thread Valentin Rothberg
Python 2 is slowly dying, so port the script to Python 3. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 33 + 1 file changed, 17 insertions(+), 16 deletions(-) diff --git a/scripts/checkkconfigsymbols.py b/scripts/checkkconfigsymbols.py

[PATCH v2 2/5] checkkconfigsymbols: use ArgumentParser

2016-08-28 Thread Valentin Rothberg
Replace the deprecated OptionParser with ArgumentParser, as recommended by pylint. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 142 - 1 file changed, 69 insertions(+), 73 deletions(-) diff --git

[PATCH v2 2/5] checkkconfigsymbols: use ArgumentParser

2016-08-28 Thread Valentin Rothberg
Replace the deprecated OptionParser with ArgumentParser, as recommended by pylint. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 142 - 1 file changed, 69 insertions(+), 73 deletions(-) diff --git a/scripts/checkkconfigsymbols.py

[PATCH v2 0/5] checkkconfigsymbols: several updates

2016-08-28 Thread Valentin Rothberg
Note that the patches base on [1], which has already been applied to Greg's char-misc tree [2] (testing branch). [1] https://lkml.org/lkml/2016/8/27/50 [2] git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git Changelog v2: - Break v1 into multiple changes (i.e., python3 port,

[PATCH v2 5/5] checkkconfigsymbols: use arglist instead of cmd string

2016-08-28 Thread Valentin Rothberg
Splitting a command string could lead to unintended arguments. Use an argument list in the execute() function instead. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 27 --- 1 file changed, 16 insertions(+), 11

[PATCH v2 0/5] checkkconfigsymbols: several updates

2016-08-28 Thread Valentin Rothberg
Note that the patches base on [1], which has already been applied to Greg's char-misc tree [2] (testing branch). [1] https://lkml.org/lkml/2016/8/27/50 [2] git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git Changelog v2: - Break v1 into multiple changes (i.e., python3 port,

[PATCH v2 5/5] checkkconfigsymbols: use arglist instead of cmd string

2016-08-28 Thread Valentin Rothberg
Splitting a command string could lead to unintended arguments. Use an argument list in the execute() function instead. Signed-off-by: Valentin Rothberg --- scripts/checkkconfigsymbols.py | 27 --- 1 file changed, 16 insertions(+), 11 deletions(-) diff --git

[PATCH v2] raid6: fix the input of raid6 algorithm

2016-08-28 Thread liuzhengyuan
From: ZhengYuan Liu To test and choose the best algorithm for raid6, disks number and disks data must be offered. These input depend on page size and gfmul table at current time. It would cause the disk number less than 4 when the page size is more than 64KB.This patch

[PATCH v2] raid6: fix the input of raid6 algorithm

2016-08-28 Thread liuzhengyuan
From: ZhengYuan Liu To test and choose the best algorithm for raid6, disks number and disks data must be offered. These input depend on page size and gfmul table at current time. It would cause the disk number less than 4 when the page size is more than 64KB.This patch would support arbitrarily

[PATCH v06] uapi dm-log-userspace.h: use __u32, __s32, __u64 and __s64 from linux/types.h

2016-08-28 Thread Mikko Rapeli
Fixes userspace compilation errors like: linux/dm-log-userspace.h:416:2: error: unknown type name ‘uint64_t’ Signed-off-by: Mikko Rapeli Cc: Bart Van Assche --- include/uapi/linux/dm-log-userspace.h | 53 ++- 1

[PATCH v06] uapi dm-log-userspace.h: use __u32, __s32, __u64 and __s64 from linux/types.h

2016-08-28 Thread Mikko Rapeli
Fixes userspace compilation errors like: linux/dm-log-userspace.h:416:2: error: unknown type name ‘uint64_t’ Signed-off-by: Mikko Rapeli Cc: Bart Van Assche --- include/uapi/linux/dm-log-userspace.h | 53 ++- 1 file changed, 27 insertions(+), 26 deletions(-)

[PATCH] tpm: fix a race condition in tpm2_unseal_trusted()

2016-08-28 Thread Jarkko Sakkinen
Unseal and load operations should be done as an atomic operation. This commit introduces unlocked tpm_transmit() so that tpm2_unseal_trusted() can do the locking by itself. v2: Introduced an unlocked unseal operation instead of changing locking strategy in order to make less intrusive bug fix

[PATCH] tpm: fix a race condition in tpm2_unseal_trusted()

2016-08-28 Thread Jarkko Sakkinen
Unseal and load operations should be done as an atomic operation. This commit introduces unlocked tpm_transmit() so that tpm2_unseal_trusted() can do the locking by itself. v2: Introduced an unlocked unseal operation instead of changing locking strategy in order to make less intrusive bug fix

Re: [PATCH v05 69/72] uapi rose.h: glibc netrose/rose.h header file compatibility fixes

2016-08-28 Thread Mikko Rapeli
On Fri, Aug 26, 2016 at 05:38:00PM +0200, walter harms wrote: > perhaps this not tested snipped would make sure that > you have included linux/rose.h ? > > #ifndef ROSE_KERNEL_H > #include > #endif > > #include Sorry, I did not quite get this. has conflicting definitions with glibc . The

Re: [PATCH v05 69/72] uapi rose.h: glibc netrose/rose.h header file compatibility fixes

2016-08-28 Thread Mikko Rapeli
On Fri, Aug 26, 2016 at 05:38:00PM +0200, walter harms wrote: > perhaps this not tested snipped would make sure that > you have included linux/rose.h ? > > #ifndef ROSE_KERNEL_H > #include > #endif > > #include Sorry, I did not quite get this. has conflicting definitions with glibc . The

[PATCH] memstick: r592: add missing declaration in r592.h

2016-08-28 Thread Baoyou Xie
We get 1 warning when build kernel with W=1: drivers/memstick/host/r592.c:50:13: warning: no previous prototype for 'memstick_debug_get_tpc_name' [-Wmissing-prototypes] in fact, function memstick_debug_get_tpc_name is implemented in r592.c and exported but need to be declared in header file. so

[PATCH] memstick: r592: add missing declaration in r592.h

2016-08-28 Thread Baoyou Xie
We get 1 warning when build kernel with W=1: drivers/memstick/host/r592.c:50:13: warning: no previous prototype for 'memstick_debug_get_tpc_name' [-Wmissing-prototypes] in fact, function memstick_debug_get_tpc_name is implemented in r592.c and exported but need to be declared in header file. so

Re: [PATCH] fscrypto: fix to null-terminate encrypted filename in fname_encrypt

2016-08-28 Thread Chao Yu
Hi Ted, On 2016/8/28 13:13, Theodore Ts'o wrote: > On Sun, Aug 28, 2016 at 09:13:28AM +0800, Chao Yu wrote: >> From: Chao Yu >> >> This patch fixes to add null character at the end of encrypted filename >> in fname_encrypt, in order to avoid incorrectly traversing random data

Re: [PATCH] fscrypto: fix to null-terminate encrypted filename in fname_encrypt

2016-08-28 Thread Chao Yu
Hi Ted, On 2016/8/28 13:13, Theodore Ts'o wrote: > On Sun, Aug 28, 2016 at 09:13:28AM +0800, Chao Yu wrote: >> From: Chao Yu >> >> This patch fixes to add null character at the end of encrypted filename >> in fname_encrypt, in order to avoid incorrectly traversing random data >> located after

Re: [PATCH 1/2] IB/hfi1: Fix a parameter of find_first_bit.

2016-08-28 Thread Leon Romanovsky
On Fri, Aug 26, 2016 at 03:34:48PM -0400, Doug Ledford wrote: > On 8/26/2016 3:29 PM, Leon Romanovsky wrote: > > On Fri, Aug 26, 2016 at 02:01:55PM -0400, Doug Ledford wrote: > >> On 8/26/2016 9:35 AM, Doug Ledford wrote: > >>> On 8/26/2016 12:49 AM, Christophe JAILLET wrote: > The 2nd

Re: [PATCH 1/2] IB/hfi1: Fix a parameter of find_first_bit.

2016-08-28 Thread Leon Romanovsky
On Fri, Aug 26, 2016 at 03:34:48PM -0400, Doug Ledford wrote: > On 8/26/2016 3:29 PM, Leon Romanovsky wrote: > > On Fri, Aug 26, 2016 at 02:01:55PM -0400, Doug Ledford wrote: > >> On 8/26/2016 9:35 AM, Doug Ledford wrote: > >>> On 8/26/2016 12:49 AM, Christophe JAILLET wrote: > The 2nd

<    1   2   3   4   5   6