[PATCH] VSOCK: Don't dec ack backlog twice for rejected connections

2016-09-27 Thread Jorgen Hansen
If a pending socket is marked as rejected, we will decrease the sk_ack_backlog twice. So don't decrement it for rejected sockets in vsock_pending_work(). Testing of the rejected socket path was done through code modifications. Reported-by: Stefan Hajnoczi Signed-off-by: Jorgen Hansen Reviewed-b

Re: [PATCH] ARM: dts: imx6: Add support for Toradex Colibri iMX6 module

2016-09-27 Thread Marcel Ziswiler
On Wed, 2016-09-21 at 16:54 +0530, Sanchayan Maity wrote: > Add support for Toradex Colibri iMX6 module. > > Signed-off-by: Sanchayan Maity Acked-by: Marcel Ziswiler > --- >  arch/arm/boot/dts/Makefile   |   1 + >  arch/arm/boot/dts/imx6dl-colibri-eval-v3.dts | 253 >  

Re: [PATCH v2 1/3] ARM: dts: imx6qdl-apalis: Do not rely on DDC I2C bus bitbang for HDMI

2016-09-27 Thread Marcel Ziswiler
On Mon, 2016-09-19 at 10:41 +0530, Sanchayan Maity wrote: > Remove the use of DDC I2C bus bitbang to support reading of EDID > and rely on support from internal HDMI I2C master controller instead. > As a result remove the device tree property ddc-i2c-bus. > > Signed-off-by: Sanchayan Maity Acked

[PATCH v3 2/2] drm/panel: simple: add support for Sharp LQ150X1LG11 panels

2016-09-27 Thread Peter Rosin
From: Gustaf Lindström The Sharp 15" LQ150X1LG11 panel is an XGA TFT LCD panel. The simple-panel driver is used to get support for essential functionality of the panel. Signed-off-by: Gustaf Lindström Signed-off-by: Peter Rosin --- drivers/gpu/drm/panel/panel-simple.c | 27 ++

[PATCH] ipmi: fix crash on reading version from proc after unregisted bmc

2016-09-27 Thread Xie XiuQi
I meet a crash, which could be reproduce: 1) while true; do cat /proc/ipmi/0/version; done 2) modprobe -rv ipmi_si ipmi_msghandler ipmi_devintf [82761.021137] IPMI BT: req2rsp=5 secs retries=2 [82761.034524] ipmi device interface [82761.18] ipmi_si ipmi_si.0: Found new BMC (man_id: 0x0007db, p

Re: [PATCH 5/5] ISDN-Gigaset: Enclose two expressions for the sizeof operator by parentheses

2016-09-27 Thread Dan Carpenter
On Mon, Sep 26, 2016 at 08:38:14PM +0300, Sergei Shtylyov wrote: > >@@ -53,7 +53,7 @@ void gigaset_dbg_buffer(enum debuglevel level, const > >unsigned char *msg, > > { > > unsigned char outbuf[80]; > > unsigned char c; > >-size_t space = sizeof outbuf - 1; > >+size_t space = sizeof

Re: [PATCH 4/5] ISDN-Gigaset: Release memory in gigaset_initcs() after an allocation failure

2016-09-27 Thread Dan Carpenter
This patch creates new bugs. I have a policy of not telling Markus where the bug is, because otherwise he'll just resend the patch and I have told him many times to stop sending these cleanup patches that just introduce bugs and waste maintainer time. regards, dan carpenter

Re: [PATCH V2 2/2] fs/super.c: don't fool lockdep in freeze_super() and thaw_super() paths

2016-09-27 Thread Dave Chinner
On Tue, Sep 27, 2016 at 08:51:35AM +0200, Jan Kara wrote: > On Mon 26-09-16 18:55:25, Oleg Nesterov wrote: > > On 09/26, Jan Kara wrote: > > > > > > On Mon 26-09-16 18:08:06, Oleg Nesterov wrote: > > > > +/* > > > > + * Tell lockdep we are holding these locks before we call > > > > ->unfreeze_fs(s

Re: [PATCH] staging: skein: Remove extra blank lines

2016-09-27 Thread Dan Carpenter
On Mon, Sep 26, 2016 at 11:56:27AM -0300, Martin wrote: > Fix coding style issue "multiple blank lines" > detected by checkpatch.pl in threefish_block.c file. > > Signed-off-by: Martin This Signed-off-by are like a legal document. Use first and last name. Also fix your from header as well. re

Re: [PATCH 5/5] ISDN-Gigaset: Enclose two expressions for the sizeof operator by parentheses

2016-09-27 Thread Dan Carpenter
On Tue, Sep 27, 2016 at 10:08:37AM +0300, Dan Carpenter wrote: > On Mon, Sep 26, 2016 at 08:38:14PM +0300, Sergei Shtylyov wrote: > > >@@ -53,7 +53,7 @@ void gigaset_dbg_buffer(enum debuglevel level, const > > >unsigned char *msg, > > > { > > > unsigned char outbuf[80]; > > > unsigned char c;

Re: ISDN-Gigaset: Release memory in gigaset_initcs() after an allocation failure

2016-09-27 Thread SF Markus Elfring
> This patch creates new bugs. Thanks for your information. > I have a policy of not telling Markus where the bug is, I find this kind of response strange. > because otherwise he'll just resend the patch This can also happen when the other contributors request it. > and I have told him man

Re: [PATCH 4/5] ISDN-Gigaset: Release memory in gigaset_initcs() after an allocation failure

2016-09-27 Thread Dan Carpenter
Ah well... Someone else discovered the double free bug first and gave it away. Reassuring, I guess. regards, dan carpenter

Re: [PATCH net v2] L2TP:Adjust intf MTU,factor underlay L3,overlay L2

2016-09-27 Thread David Miller
From: "R. Parameswaran" Date: Thu, 22 Sep 2016 13:52:43 -0700 (PDT) > From ed585bdd6d3d2b3dec58d414f514cd764d89159d Mon Sep 17 00:00:00 2001 > From: "R. Parameswaran" > Date: Thu, 22 Sep 2016 13:19:25 -0700 > Subject: [PATCH] L2TP:Adjust intf MTU,factor underlay L3,overlay L2 > > Take into acco

Re: [PATCH] kcov: properly check if we are in an interrupt

2016-09-27 Thread Peter Zijlstra
On Tue, Sep 27, 2016 at 08:21:32AM +0200, Dmitry Vyukov wrote: > > I suspect there is a bunch of places that use in_interrupt(), but mean > the same as KCOV wants -- am I in interrupt? and not am I in interrupt > context or in normal task context but inside local_bh_disable(). For > example, why d

Re: [PATCH 06/11] pipe: no need to confirm page cache buf

2016-09-27 Thread Miklos Szeredi
On Tue, Sep 27, 2016 at 5:40 AM, Al Viro wrote: > On Wed, Sep 14, 2016 at 10:37:11AM +0200, Miklos Szeredi wrote: > >> Things could happen to that page that make it not uptodate while sitting in >> the pipe, but it's questionable whether we should care about that. >> Checking for being uptodate in

[PATCHv4 3/4] staging:android:ion: Fix blank line issues

2016-09-27 Thread Yannis Damigos
This patch fixes the following blank line issues: CHECK: Please don't use multiple blank lines CHECK: Please use a blank line after function/struct/union/enum declarations +} +RESERVEDMEM_OF_DECLARE(ion, "ion-region", rmem_ion_setup); Signed-off-by: Yannis Damigos --- drivers/staging/android/i

[PATCHv4 1/4] staging:android:ion: Fix comparison with NULL

2016-09-27 Thread Yannis Damigos
This patch fixes the following comparison with NULL issues: CHECK: Comparison to NULL could be written "compatible[i].name" + for (i = 0; compatible[i].name != NULL; i++) { CHECK: Comparison to NULL could be written "!compatible[i].name" + if (compatible[i].name == NULL) Signed-off-b

[PATCHv4 4/4] staging:android:ion: Fix whitespace issue

2016-09-27 Thread Yannis Damigos
This patch fixes the following whitespace issue: CHECK: spaces preferred around that '*' (ctx:VxV) FILE: drivers/staging/android/ion/ion_of.c:91: +sizeof(struct ion_platform_heap)*num_heaps, Signed-off-by: Yannis Damigos --- drivers/staging/android/ion/ion_of.c | 2 +

Re: solo6010 modprobe lockup since e1ceb25a (v4.3 regression)

2016-09-27 Thread Andrey Utkin
On Tue, Sep 27, 2016 at 07:27:53AM +0200, Krzysztof Hałasa wrote: > Andrey Utkin writes: > > >> Does (only) adding the > >> > >>pci_read_config_word(solo_dev->pdev, PCI_STATUS, &val); > >> > >> in solo_reg_write() help? > > > > Yes. > > I have posted a patch with this change few days ago, I

[PATCHv4 2/4] staging:android:ion: Fix alignment issues

2016-09-27 Thread Yannis Damigos
This patch fixes the following whitespace issues: CHECK: Alignment should match open parenthesis + pr_info("%s: id %d type %d name %s align %lx\n", __func__, + heap->id, heap->type, heap->name, heap->align); CHECK: Alignment should match open parenthesis +struct ion_pl

Re: linux-next: manual merge of the vfs-jk tree with the fuse tree

2016-09-27 Thread Jan Kara
Hi Stephen, On Tue 27-09-16 10:12:04, Stephen Rothwell wrote: > Today's linux-next merge of the vfs-jk tree got a conflict in: > > fs/fuse/dir.c > > between commits: > > 3b3fb74935ce ("fuse: invalidate dir dentry after chmod") > 5f69e628df75 ("fuse: fix killing s[ug]id in setattr") > > f

Re: [PATCH 0/2] NFS: Use complete() instead complete_all()

2016-09-27 Thread Daniel Wagner
Hi Anna, On 09/26/2016 03:31 PM, Anna Schumaker wrote: > No, there are a handful that we know don't pass. When I run on NFS v4.1, I > get: > > Failures: generic/035 generic/071 generic/128 generic/294 > generic/306 generic/309 generic/310 generic/313 generic/314 > generic/315 generic/316 generi

[PATCH V2 0/6] ARM64: Uprobe support added

2016-09-27 Thread Pratyush Anand
is_tarp_insn() * Changes for rebase conflict resolution V1 was here: https://lkml.org/lkml/2016/8/2/29 Patches have been rebased on next-20160927, so that there would be no conflicts with other arm64/for-next/core patches. Patches have been tested for following: 1. Step-able instructions, like sub

Re: [PATCH -v2 7/9] sched,tracing: Update trace_sched_pi_setprio()

2016-09-27 Thread Peter Zijlstra
On Mon, Sep 26, 2016 at 01:04:30PM -0400, Steven Rostedt wrote: > On Mon, 26 Sep 2016 14:32:20 +0200 > Peter Zijlstra wrote: > > > Pass the PI donor task, instead of a numerical priority. > > > > Numerical priorities are not sufficient to describe state ever since > > SCHED_DEADLINE. > > > > An

Re: [PATCHv2 2/3] tty/serial: at91: fix hardware handshake with GPIOs

2016-09-27 Thread Richard Genoud
2016-09-22 15:20 GMT+02:00 Uwe Kleine-König : > On Mon, Sep 12, 2016 at 11:47:32AM +0200, Richard Genoud wrote: >> Commit 1cf6e8fc8341 ("tty/serial: at91: fix RTS line management when >> hardware handshake is enabled") broke the hardware handshake when GPIOs >> where used. > > s/where/were/ ok. >>

[PATCH V2 4/6] arm64: Handle TRAP_BRKPT for user mode as well

2016-09-27 Thread Pratyush Anand
uprobe is registered at break_hook with a unique ESR code. So, when a TRAP_BRKPT occurs, call_break_hook checks if it was for uprobe. If not, then send a SIGTRAP to user. Signed-off-by: Pratyush Anand --- arch/arm64/kernel/debug-monitors.c | 18 +++--- 1 file changed, 11 insertions(+

[PATCH V2 5/6] arm64: introduce mm context flag to keep 32 bit task information

2016-09-27 Thread Pratyush Anand
We need to decide in some cases like uprobe instruction analysis that whether the current mm context belongs to a 32 bit task or 64 bit. This patch has introduced an unsigned flag variable in mm_context_t. Currently, we set and clear TIF_32BIT depending on the condition that whether an elf binary

[PATCH V2 2/6] arm64: kgdb_step_brk_fn: ignore other's exception

2016-09-27 Thread Pratyush Anand
ARM64 step exception does not have any syndrome information. So, it is responsibility of exception handler to take care that they handle it only if exception was raised for them. Since kgdb_step_brk_fn() always returns 0, therefore we might have problem when we will have other step handler registe

[PATCH V2 1/6] arm64: kprobe: protect/rename few definitions to be reused by uprobe

2016-09-27 Thread Pratyush Anand
decode-insn code has to be reused by arm64 uprobe implementation as well. Therefore, this patch protects some portion of kprobe code and renames few other, so that decode-insn functionality can be reused by uprobe even when CONFIG_KPROBES is not defined. kprobe_opcode_t and struct arch_specific_in

Re: [PATCH 2/2] bpf samples: update tracex5 sample to use __seccomp_filter

2016-09-27 Thread David Miller
From: "Naveen N. Rao" Date: Sat, 24 Sep 2016 02:10:05 +0530 > seccomp_phase1() does not exist anymore. Instead, update sample to use > __seccomp_filter(). While at it, set max locked memory to unlimited. > > Signed-off-by: Naveen N. Rao Also applied to net-next, thanks.

Re: [PATCH v3] net: ip, diag -- Add diag interface for raw sockets

2016-09-27 Thread Cyrill Gorcunov
On Mon, Sep 26, 2016 at 07:54:37PM -0600, David Ahern wrote: > On 9/26/16 4:38 PM, Cyrill Gorcunov wrote: > > Something like > > > > Index: linux-ml.git/include/uapi/linux/inet_diag.h > > === > > --- linux-ml.git.orig/include/uapi/lin

Re: [PATCH 1/2] bpf samples: fix compiler errors with sockex2 and sockex3

2016-09-27 Thread David Miller
From: "Naveen N. Rao" Date: Sat, 24 Sep 2016 02:10:04 +0530 > These samples fail to compile as 'struct flow_keys' conflicts with > definition in net/flow_dissector.h. Fix the same by renaming the > structure used in the sample. > > Signed-off-by: Naveen N. Rao Applied to net-next.

[PATCH V2 6/6] arm64: Add uprobe support

2016-09-27 Thread Pratyush Anand
This patch adds support for uprobe on ARM64 architecture. Unit tests for following have been done so far and they have been found working 1. Step-able instructions, like sub, ldr, add etc. 2. Simulation-able like ret, cbnz, cbz etc. 3. uretprobe 4. Reject-able instructions like sev

[PATCH V2 3/6] arm64: Handle TRAP_TRACE for user mode as well

2016-09-27 Thread Pratyush Anand
uprobe registers a handler at step_hook. So, single_step_handler now checks for user mode as well if there is a valid hook. Signed-off-by: Pratyush Anand --- arch/arm64/kernel/debug-monitors.c | 22 -- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/arch/arm64

Re: [PATCH] kcov: properly check if we are in an interrupt

2016-09-27 Thread Dmitry Vyukov
On Tue, Sep 27, 2016 at 9:34 AM, Peter Zijlstra wrote: > On Tue, Sep 27, 2016 at 08:21:32AM +0200, Dmitry Vyukov wrote: >> >> I suspect there is a bunch of places that use in_interrupt(), but mean >> the same as KCOV wants -- am I in interrupt? and not am I in interrupt >> context or in normal tas

Re: [PATCH v3] bpf: Set register type according to is_valid_access()

2016-09-27 Thread David Miller
From: Mickaël Salaün Date: Sat, 24 Sep 2016 20:01:50 +0200 > This prevent future potential pointer leaks when an unprivileged eBPF > program will read a pointer value from its context. Even if > is_valid_access() returns a pointer type, the eBPF verifier replace it > with UNKNOWN_VALUE. The regis

[PATCH] MAINTAINERS: update fbdev entries

2016-09-27 Thread Tomi Valkeinen
Remove Jean-Christophe from the maintainers, and remove links to old unmaintained web pages and git trees. Signed-off-by: Tomi Valkeinen --- Jean-Christophe hasn't done any fbdev work for years, hasn't responded to any emails, and now emails to him bounce back. MAINTAINERS | 3 --- 1 file chan

[PATCH v3 0/2] drm/panel: simple: add support for Sharp LQ150X1LG11 panels

2016-09-27 Thread Peter Rosin
Hi! I'm back with a new attempt at getting this panel upstream. This time with the bindings in a separate commit, as I hear that is preferred. I think I covered all the pins on the panel... Cheers, Peter Gustaf Lindström (1): drm/panel: simple: add support for Sharp LQ150X1LG11 panels Peter

Re: [PATCH 15/19] reset: sti: Remove STiH415/6 reset support

2016-09-27 Thread Philipp Zabel
Hi Peter, Am Mittwoch, den 14.09.2016, 14:27 +0100 schrieb Peter Griffin: > Support for STiH415/6 SoCs is being removed from the > kernel because the platforms are obsolete. This patch removes > the reset drivers for these SoC's. > > Signed-off-by: Peter Griffin > Cc: > --- > arch/arm/mach-sti

Re: [PATCH 2/3] regulator: max77693: Also manipulate the fast charge current

2016-09-27 Thread Krzysztof Kozlowski
On Tue, Sep 27, 2016 at 01:31:09AM +0200, Wolfgang Wiedmeyer wrote: > For MAX77693, the fast charge current also needs to be manipulated for > proper charging. The fast charge current is only set in the case of > the MAX77693 type, as the MAX77843 properly manipulates the fast > charge current. Ar

Re: [PATCH v2] fs/select: add vmalloc fallback for select(2)

2016-09-27 Thread Vlastimil Babka
On 09/27/2016 02:01 AM, Andrew Morton wrote: On Thu, 22 Sep 2016 18:43:59 +0200 Vlastimil Babka wrote: The select(2) syscall performs a kmalloc(size, GFP_KERNEL) where size grows with the number of fds passed. We had a customer report page allocation failures of order-4 for this allocation. Th

Re: [PATCH 1/3] mfd: max77693: Add defines for charger current control

2016-09-27 Thread Krzysztof Kozlowski
On Tue, Sep 27, 2016 at 01:31:08AM +0200, Wolfgang Wiedmeyer wrote: > This prepares for an updated regulator and charger driver. The defines > are needed to set the maximum input current and the fast charge > current. > > Signed-off-by: Wolfgang Wiedmeyer This shouldn't be separate patch, becaus

Re: [PATCH] video: fbdev: i810: add in missing white space in error message text

2016-09-27 Thread Tomi Valkeinen
On 12/09/16 15:14, Colin King wrote: > From: Colin Ian King > > A printk message spans two lines and the literal string is missing > a white space between words. Add the white space. > > Signed-off-by: Colin Ian King > --- > drivers/video/fbdev/i810/i810_main.c | 2 +- > 1 file changed, 1 in

Re: [PATCH] video: fbdev: add missing \n at end of printk error message

2016-09-27 Thread Tomi Valkeinen
On 12/09/16 13:24, Colin King wrote: > From: Colin Ian King > > The message is missing a \n, add it. > > Signed-off-by: Colin Ian King > --- > drivers/video/fbdev/s1d13xxxfb.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/video/fbdev/s1d13xxxfb.c > b/driver

Re: [PATCH v3 2/2] ARM: exynos_defconfig: Remove old non-working MIPI driver

2016-09-27 Thread Tomi Valkeinen
On 16/09/16 11:32, Krzysztof Kozlowski wrote: > The Exynos MIPI driver does not work anymore (it is board file only) so > it is removed. Remove also config options. > > Cc: Inki Dae > Cc: Donghwa Lee > Cc: Kyungmin Park > Cc: Tomi Valkeinen > Signed-off-by: Krzysztof Kozlowski > > --- > >

Re: [PATCH v2] fs/select: add vmalloc fallback for select(2)

2016-09-27 Thread Vlastimil Babka
On 09/27/2016 03:38 AM, Eric Dumazet wrote: On Mon, 2016-09-26 at 17:01 -0700, Andrew Morton wrote: I don't share Eric's concerns about performance here. If the vmalloc() is called, we're about to write to that quite large amount of memory which we just allocated, and the vmalloc() overhead wi

Re: [PATCH 3/3] power_supply: max77693: Listen for cable events and enable charging

2016-09-27 Thread Krzysztof Kozlowski
On Tue, Sep 27, 2016 at 01:31:10AM +0200, Wolfgang Wiedmeyer wrote: > This patch adds a listener for extcon cable events and enables > charging if an USB cable is connected. It recognizes SDP and DCP cable > types and treats them the same (same input current and fast charge > current). The maximum

Re: [PATCH 04/26] matroxfb: constify local structures

2016-09-27 Thread Tomi Valkeinen
On 11/09/16 16:05, Julia Lawall wrote: > For structure types defined in the same file or local header files, find > top-level static structure declarations that have the following > properties: > 1. Never reassigned. > 2. Address never taken > 3. Not passed to a top-level macro call > 4. No pointer

Re: [PATCH RESEND 2/2] mfd: twl6040: Register child device for twl6040-pdmclk

2016-09-27 Thread Peter Ujfalusi
| 2 +- >> 3 files changed, 7 insertions(+), 1 deletion(-) > > Applied, thanks. Lee, for some reason this patch is not found in linux-next (20160927). I can see patch 1, but this one is missing. -- Péter >> diff --git a/Documentation/devicetree/bindings/mfd/twl60

Re: [PATCH] video: fbdev: constify fb_fix_screeninfo and fb_var_screeninfo structures

2016-09-27 Thread Tomi Valkeinen
On 11/09/16 18:17, Julia Lawall wrote: > These structures are only used to copy into other structures, so declare > them as const. > > The semantic patch that makes this change in the fb_fix_screeninfo case is > as follows (http://coccinelle.lip6.fr/). The fb_var_screeninfo case is > analogous. >

Re: [PATCH] simplefb: Disable and release clocks and regulators in destroy callback

2016-09-27 Thread Tomi Valkeinen
On 07/09/16 12:09, Chen-Yu Tsai wrote: > simplefb gets unregister when a proper framebuffer driver comes in and > kicks it out. However the claimed clocks and regulators stay enabled > as they are only released in the platform device remove function, which > in theory would never get called. > > M

[PATCH] soc/fsl: dpaa_sys.h: fix ARM32 typo

2016-09-27 Thread Valentin Rothberg
The Kconfig symbol for 32bit ARM is 'ARM', not 'ARM32'. Signed-off-by: Valentin Rothberg --- drivers/soc/fsl/qbman/dpaa_sys.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/soc/fsl/qbman/dpaa_sys.h b/drivers/soc/fsl/qbman/dpaa_sys.h index b63fd72295c6..608b1f608e13 1

Re: [PATCH v2] sysfs print name of undiscoverable attribute group

2016-09-27 Thread Johannes Thumshirn
On Mon, Sep 19, 2016 at 10:14:54AM +0200, Johannes Thumshirn wrote: > Print the name of an undiscoverable attribute group and not the > pointer's address. > > Signed-off-by: Johannes Thumshirn > --- > fs/sysfs/group.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/

[PATCH] ASoC: tlv320aic31xx: do not declare support for mono DAI

2016-09-27 Thread Nikita Yushchenko
This hardware supports only 2-channel DAI, even mono ADC digital output has two channels with the same data. Having min_channels=1 results in broken playback of mono files in setups where CPU DAI supports mono. Signed-off-by: Nikita Yushchenko --- sound/soc/codecs/tlv320aic31xx.c | 6 +++--- 1

Re: [PATCH 0/2] MCB patches for v4.9

2016-09-27 Thread Johannes Thumshirn
On Wed, Sep 14, 2016 at 12:05:22PM +0200, Johannes Thumshirn wrote: > Hi Greg, > > Here are the additional MCB patches for v4.9. > > They're around enabling of DMA for devices on an MCBus, are tested and > apply cleanly against your char-misc-next tree. > > Thanks, > Johannes > > Michael

Re: [PATCH 0/2] NFS: Use complete() instead complete_all()

2016-09-27 Thread Daniel Wagner
On 09/27/2016 09:42 AM, Daniel Wagner wrote: > Failures: generic/035 generic/087 generic/088 generic/089 generic/126 > generic/184 generic/285 generic/347 generic/350 > Failed 9 of 62 tests > > Hmm, not sure if this good or bad. Does it help? FWIW, running the same tests on non patched kernel gi

Re: [PATCH 1/2] fbdev: ssd1307fb: constify the device_info pointer

2016-09-27 Thread Tomi Valkeinen
On 16/08/16 12:27, LABBE Corentin wrote: > of_match_device return const data, so instead of casting its return value > this patch constify the device_info pointer. > > Signed-off-by: LABBE Corentin > --- > drivers/video/fbdev/ssd1307fb.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions

Re: [PATCH] fs/select: add vmalloc fallback for select(2)

2016-09-27 Thread Vlastimil Babka
On 09/23/2016 06:47 PM, Jason Baron wrote: Hi, On 09/23/2016 03:24 AM, Nicholas Piggin wrote: On Fri, 23 Sep 2016 14:42:53 +0800 "Hillf Danton" wrote: The select(2) syscall performs a kmalloc(size, GFP_KERNEL) where size grows with the number of fds passed. We had a customer report page all

Re: [PATCH] matroxfb: fix size of memcpy

2016-09-27 Thread Tomi Valkeinen
On 25/08/16 20:44, Sudip Mukherjee wrote: > hw->DACreg has a size of 80 bytes and MGADACbpp32 has 21. So when > memcpy copies MGADACbpp32 to hw->DACreg it copies 80 bytes but > only 21 bytes are valid. > > Signed-off-by: Sudip Mukherjee > --- > drivers/video/fbdev/matrox/matroxfb_Ti3026.c | 2 +-

Re: [PATCH v2 4/4] PM / AVS: rockchip-cpu-avs: add driver handling Rockchip cpu avs

2016-09-27 Thread Finley Xiao
在 2016/9/26 16:05, Heiko Stuebner 写道: Am Montag, 26. September 2016, 09:25:11 CEST schrieb Viresh Kumar: On 12-09-16, 14:55, Stephen Boyd wrote: On 08/29, Viresh Kumar wrote: On 18-08-16, 16:52, Finlye Xiao wrote: +static int rockchip_adjust_opp_table(struct device *cpu_dev, +

[PATCH v3] fs/select: add vmalloc fallback for select(2)

2016-09-27 Thread Vlastimil Babka
The select(2) syscall performs a kmalloc(size, GFP_KERNEL) where size grows with the number of fds passed. We had a customer report page allocation failures of order-4 for this allocation. This is a costly order, so it might easily fail, as the VM expects such allocation to have a lower-order fallb

Re: [PATCH] video: smscufx: remove unused variable

2016-09-27 Thread Tomi Valkeinen
On 25/05/16 15:19, Sudip Mukherjee wrote: > The variable was only assigned with the pointer but was never used. > We can safely remove it. > > Signed-off-by: Sudip Mukherjee > --- > drivers/video/fbdev/smscufx.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/video/fbdev/smscu

Re: Regression in 4.8 - CPU speed set very low

2016-09-27 Thread Larry Finger
On 09/26/2016 10:12 PM, Doug Smythies wrote: On 2016.09.26 18:31 Srinivas Pandruvada wrote: On Mon, 2016-09-26 at 19:48 -0500, Larry Finger wrote: On 09/26/2016 07:21 PM, Rafael J. Wysocki wrote: On Tue, Sep 27, 2016 at 1:53 AM, Larry Finger wrote: But for both we need a reproducer anyway. I

Re: ISDN-Gigaset: Release memory in gigaset_initcs() after an allocation failure

2016-09-27 Thread Paul Bolle
On Tue, 2016-09-27 at 07:20 +0200, SF Markus Elfring wrote: > Will it matter here if the function "kfree" will be called for the > data structure members "bcs" and "inbuf" after a later function call > failed within the implementation of "gigaset_initcs"? My translation of this question is: could

Re: [Update][RFC/RFT][PATCH v3 2/5] driver core: Functional dependencies tracking support

2016-09-27 Thread Lukas Wunner
On Fri, Sep 16, 2016 at 02:33:55PM +0200, Rafael J. Wysocki wrote: > +void device_links_unbind_consumers(struct device *dev) > +{ > + struct device_link *link; > + int idx; > + > + start: > + idx = device_links_read_lock(); > + > + list_for_each_entry_rcu(link, &dev->links_to_consum

Re: [bug] crypto/vmx/p8_ghash memory corruption in 4.8-rc7

2016-09-27 Thread Jan Stancek
- Original Message - > From: "Herbert Xu" > To: "Marcelo Cerri" > Cc: "Jan Stancek" , "rui y wang" , > mhce...@linux.vnet.ibm.com, > leosi...@linux.vnet.ibm.com, pfsmor...@linux.vnet.ibm.com, > linux-cry...@vger.kernel.org, > linuxppc-...@lists.ozlabs.org, linux-kernel@vger.kernel.o

Re: [PATCH] padata: remove recently added module usage from bool code

2016-09-27 Thread Sebastian Andrzej Siewior
On 2016-09-26 21:18:21 [-0400], Paul Gortmaker wrote: > > ...and so it currently is not being built as a module by anyone. that is correct. Acked-by: Sebastian Andrzej Siewior Sebastian

Re: [PATCH] brcmfmac: implement more accurate skb tracking

2016-09-27 Thread Arend Van Spriel
On 26-9-2016 16:59, Dan Williams wrote: > On Mon, 2016-09-26 at 14:13 +0200, Rafał Miłecki wrote: >> On 26 September 2016 at 13:46, Arend Van Spriel >> wrote: >>> >>> On 26-9-2016 12:23, Rafał Miłecki wrote: From: Rafał Miłecki We need to track 802.1x packets to know if there

[GIT PULL] tpmdd reverts for Linux 4.9

2016-09-27 Thread Jarkko Sakkinen
' of git://git.infradead.org/users/pcmoore/selinux into next (2016-09-21 11:54:19 +1000) are available in the git repository at: git://git.infradead.org/users/jjs/linux-tpmdd.git tags/tpmdd-next-20160927 for you to fetch changes up to 324152502b0ef2f65a3fe039b7dcd27b223d14ac: Revert "tpm/tpm_

[PATCH 4.9] brcmfmac: use correct skb freeing helper when deleting flowring

2016-09-27 Thread Rafał Miłecki
From: Rafał Miłecki Flowrings contain skbs waiting for transmission that were passed to us by netif. It means we checked every one of them looking for 802.1x Ethernet type. When deleting flowring we have to use freeing function that will check for 802.1x type as well. Freeing skbs without a prop

[PATCH] greybus: manifest: style fix missing space before '('

2016-09-27 Thread Quentin Lambert
Checkpatch printed a style ERROR concerning a missing space befire '('. This patch fix this issue. Signed-off-by: Quentin Lambert --- drivers/staging/greybus/manifest.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/staging/greybus/manifest.c +++ b/drivers/staging/greybu

Re: [GIT PULL] tpmdd reverts for Linux 4.9

2016-09-27 Thread James Morris
/Jarkko > > The following changes since commit 8a17ef9d8543744a0c6a6f5fdc65b5770e7eaa64: > > Merge branch 'stable-4.9' of git://git.infradead.org/users/pcmoore/selinux > into next (2016-09-21 11:54:19 +1000) > > are available in the git repository at: > > git://gi

Re: [PATCH] brcmfmac: implement more accurate skb tracking

2016-09-27 Thread Arend Van Spriel
On 26-9-2016 14:38, Rafał Miłecki wrote: > On 26 September 2016 at 14:13, Rafał Miłecki wrote: >> On 26 September 2016 at 13:46, Arend Van Spriel >> wrote: >>> On 26-9-2016 12:23, Rafał Miłecki wrote: From: Rafał Miłecki We need to track 802.1x packets to know if there are any pen

Re: [PATCH v3 2/4] tmp/tpm_crb: fix Intel PTT hw bug during idle state

2016-09-27 Thread Jarkko Sakkinen
On Thu, Sep 15, 2016 at 09:23:29AM +0300, Jarkko Sakkinen wrote: > On Mon, Sep 12, 2016 at 04:04:19PM +0300, Tomas Winkler wrote: > > There is a HW bug in Skylake, and Broxton PCH Intel PTT device, where > > most of the registers in the control area except START, REQUEST, CANCEL, > > and LOC_CTRL l

Re: [PATCH] greybus: manifest: style fix missing space before '('

2016-09-27 Thread Viresh Kumar
On Tue, Sep 27, 2016 at 2:50 PM, Quentin Lambert wrote: > Checkpatch printed a style ERROR concerning a missing space befire '('. > This patch fix this issue. > > Signed-off-by: Quentin Lambert > --- > drivers/staging/greybus/manifest.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [PATCH] greybus: audio: fix uninitialized variable errors found by cppcheck

2016-09-27 Thread Viresh Kumar
On Sat, Sep 24, 2016 at 11:06 PM, Vaibhav Agarwal wrote: > Currently, if info is null, the dev_err message is dereferencing an > uninitialized module pointer. Instead, it should use codec->dev pointer > in dev_err call and better align with other err msg in this function. > > Also, ret variable m

Re: [PATCH] drivers/staging/greybus: gpio.c - Fixed a checkpatch generated warning

2016-09-27 Thread Viresh Kumar
On Mon, Sep 26, 2016 at 2:05 AM, Chase Metzger wrote: > Removed braces for single line if statement. > > Signed-off-by: Chase Metzger > --- > drivers/staging/greybus/gpio.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/staging/greybus/gpio.c b/drivers/staging

Re: [PATCH] greybus: manifest: style fix missing space before '('

2016-09-27 Thread Quentin Lambert
On 27/09/2016 11:31, Viresh Kumar wrote: On Tue, Sep 27, 2016 at 2:50 PM, Quentin Lambert wrote: Checkpatch printed a style ERROR concerning a missing space befire '('. This patch fix this issue. Signed-off-by: Quentin Lambert --- drivers/staging/greybus/manifest.c |2 +- 1 file chan

Re: ISDN-Gigaset: Release memory in gigaset_initcs() after an allocation failure

2016-09-27 Thread SF Markus Elfring
>> Will it matter here if the function "kfree" will be called for the >> data structure members "bcs" and "inbuf" after a later function call >> failed within the implementation of "gigaset_initcs"? > > My translation of this question is: could you please hold my hand while > I read the code of a

Re: [PATCH] greybus: manifest: style fix missing space before '('

2016-09-27 Thread Viresh Kumar
On 27 September 2016 at 15:03, Quentin Lambert wrote: > > > On 27/09/2016 11:31, Viresh Kumar wrote: >> >> On Tue, Sep 27, 2016 at 2:50 PM, Quentin Lambert >> wrote: >>> >>> Checkpatch printed a style ERROR concerning a missing space befire '('. >>> This patch fix this issue. >>> >>> Signed-off-b

Re: [PATCH 00/17] clean up readlinks

2016-09-27 Thread Miklos Szeredi
On Tue, Sep 27, 2016 at 5:10 AM, Al Viro wrote: > On Mon, Sep 12, 2016 at 09:29:02PM +0200, Miklos Szeredi wrote: >> The first patch is actually a bug fix, but I put it into this bunch for >> simplicity... >> >> The rest are really cleanups as well as minor bugfixes that are byproducts >> of the c

[PATCH v2] greybus: manifest: style fix missing space before '('

2016-09-27 Thread Quentin Lambert
Checkpatch printed a style ERROR concerning a missing space before '('. This patch fixes this issue. Signed-off-by: Quentin Lambert Acked-by: Viresh Kumar --- v2: fixes typos in commit message drivers/staging/greybus/manifest.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/

BUG: scheduling while atomic in f_fs when gadget remove driver

2016-09-27 Thread Chen Yu
Hi All, I'm working on Hikey board based around the HiSilicon Kirin 620, with linaro kernel version 4.8.rc1 and I get below BUG error while extracting USB cable from PC. The funtion using f_fs is adb and usb_gadget_unregister_driver will be called after extracting USB cable from PC. [ 89.456

Re: [f2fs-dev] [PATCH v2] f2fs: introduce get_checkpoint_version for cleanup

2016-09-27 Thread Chao Yu
On 2016/9/27 10:05, Tiezhu Yang wrote: > There exists almost same codes when get the value of pre_version > and cur_version in function validate_checkpoint, this patch adds > get_checkpoint_version to clean up redundant codes. > > Signed-off-by: Tiezhu Yang > --- > fs/f2fs/checkpoint.c | 63 > +

Re: [PATCH] greybus: manifest: style fix missing space before '('

2016-09-27 Thread Dan Carpenter
On Tue, Sep 27, 2016 at 11:33:49AM +0200, Quentin Lambert wrote: > > > On 27/09/2016 11:31, Viresh Kumar wrote: > >On Tue, Sep 27, 2016 at 2:50 PM, Quentin Lambert > > wrote: > >>Checkpatch printed a style ERROR concerning a missing space befire '('. > >>This patch fix this issue. > >> > >>Signed

Re: ISDN-Gigaset: Release memory in gigaset_initcs() after an allocation failure

2016-09-27 Thread Paul Bolle
You're in Eliza mode again. (Hat tip to Björn Mork, https://lkml.org/lkml/2016/1/4/259). Paul Bolle

RE: [PATCH v3 2/4] tmp/tpm_crb: fix Intel PTT hw bug during idle state

2016-09-27 Thread Winkler, Tomas
> > On Thu, Sep 15, 2016 at 09:23:29AM +0300, Jarkko Sakkinen wrote: > > On Mon, Sep 12, 2016 at 04:04:19PM +0300, Tomas Winkler wrote: > > > There is a HW bug in Skylake, and Broxton PCH Intel PTT device, > > > where most of the registers in the control area except START, > > > REQUEST, CANCEL, a

Re: BUG: scheduling while atomic in f_fs when gadget remove driver

2016-09-27 Thread Felipe Balbi
Hi, Chen Yu writes: > Hi All, > > I'm working on Hikey board based around the HiSilicon Kirin 620, with > linaro kernel version 4.8.rc1 and I get below BUG error while > extracting USB cable from PC. which peripheral controller does this one have? Is it dwc3? I'm very interested in knowing abo

[PATCH] MIPS: loongson32: Remove several RTC-related macros

2016-09-27 Thread Yang Ling
Add regs-rtc.h to replace the macros of redundancy. Signed-off-by: Yang Ling --- arch/mips/include/asm/mach-loongson32/regs-rtc.h | 23 +++ arch/mips/loongson32/common/platform.c | 23 ++- 2 files changed, 33 insertions(+), 13 deletions(-) creat

Re: linux-next: Tree for Sep 27

2016-09-27 Thread Sergey Senozhatsky
Hello, On (09/27/16 16:40), Stephen Rothwell wrote: > > Changes since 20160923: > seems that commit e3b37f11e6e4e6b6 ("netfilter: replace list_head with single linked list") breaks the build on !CONFIG_NETFILTER_INGRESS systems accessing ->nf_hooks_ingress static void nf_set_hooks_head(struct

Re: [PATCH 4.9] brcmfmac: use correct skb freeing helper when deleting flowring

2016-09-27 Thread Arend Van Spriel
On 27-9-2016 11:14, Rafał Miłecki wrote: > From: Rafał Miłecki > > Flowrings contain skbs waiting for transmission that were passed to us > by netif. It means we checked every one of them looking for 802.1x > Ethernet type. When deleting flowring we have to use freeing function > that will check

Re: [PATCH] drm/imx: hide an unused label

2016-09-27 Thread Philipp Zabel
Am Donnerstag, den 22.09.2016, 11:50 +0200 schrieb Arnd Bergmann: > The imx_drm_bind function causes a warning in linux-next when > CONFIG_DRM_FBDEV_EMULATION is not set: > > drivers/gpu/drm/imx/imx-drm-core.c: In function 'imx_drm_bind': > drivers/gpu/drm/imx/imx-drm-core.c:441:1: error: label 'e

Re: [PATCH -next] gpu: ipu-v3: Use ERR_CAST instead of ERR_PTR(PTR_ERR())

2016-09-27 Thread Philipp Zabel
Am Mittwoch, den 21.09.2016, 15:12 + schrieb Wei Yongjun: > From: Wei Yongjun > > Use ERR_CAST inlined function instead of ERR_PTR(PTR_ERR(...)). > > Generated by: scripts/coccinelle/api/err_cast.cocci > > Signed-off-by: Wei Yongjun Applied, thank you. regards Philipp

Re: qemu:metag image runtime failure in -next due to 'kthread: allow to cancel kthread work'

2016-09-27 Thread Petr Mladek
On Mon 2016-09-19 08:45:09, Guenter Roeck wrote: > On Mon, Sep 19, 2016 at 03:55:29PM +0100, James Hogan wrote: > > On Sat, Sep 17, 2016 at 12:32:49AM +0100, James Hogan wrote: > > > Here this version of QEMU puts the args at where it thinks the end of > > > the loaded image is, which is based on t

[PATCH] brcmfmac: replace WARNING on timeout with a simple error message

2016-09-27 Thread Rafał Miłecki
From: Rafał Miłecki Even with timeout increased to 950 ms we get WARNINGs from time to time. It mostly happens on A-MPDU stalls (e.g. when station goes out of range). It may take up to 5-10 secods for the firmware to recover and for that time it doesn't process packets. It's still useful to have

BUG Re: mm: vma_merge: fix vm_page_prot SMP race condition against rmap_walk

2016-09-27 Thread Shaun Tancheff
git bisect points at commit c9634dcf00c9c93b ("mm: vma_merge: fix vm_page_prot SMP race condition against rmap_walk") Last lines to console are [transcribed]: vma 8c3d989a7c78 start 7fe02ed4c000 end 7fe02ed52000 next 8c3d96de0c38 prev 8c3d989a6e40 mm 8c3d071cbac0 prot 800

Re: [PATCH] tools/testing/selftests/powerpc: Add Anton's null_syscall benchmark to the selftests

2016-09-27 Thread Michael Ellerman
Rui Teng writes: > From: Anton Blanchard > > Pull in a version of Anton's null_syscall benchmark: > http://ozlabs.org/~anton/junkcode/null_syscall.c > Into tools/testing/selftests/powerpc/benchmarks. > > Suggested-by: Michael Ellerman > Signed-off-by: Anton Blanchard > Signed-off-by: Rui Teng

Re: qemu:metag image runtime failure in -next due to 'kthread: allow to cancel kthread work'

2016-09-27 Thread James Hogan
On 27 September 2016 11:12:36 BST, Petr Mladek wrote: >On Mon 2016-09-19 08:45:09, Guenter Roeck wrote: >> On Mon, Sep 19, 2016 at 03:55:29PM +0100, James Hogan wrote: >> > On Sat, Sep 17, 2016 at 12:32:49AM +0100, James Hogan wrote: >> > > Here this version of QEMU puts the args at where it think

Re: Warning from free_init_pages with large initrd

2016-09-27 Thread Sitsofe Wheeler
(See http://www.gossamer-threads.com/lists/linux/kernel/2534175 for the history of this thread ) On 26 September 2016 at 20:00, Randy Dunlap wrote: > > but the warning in free_init_pages() is about alignment, not size... > Maybe the concatenation is bad? What would l have to pull apart to be abl

Re: [PATCH v3] fs/select: add vmalloc fallback for select(2)

2016-09-27 Thread Michal Hocko
On Tue 27-09-16 10:45:36, Vlastimil Babka wrote: > The select(2) syscall performs a kmalloc(size, GFP_KERNEL) where size grows > with the number of fds passed. We had a customer report page allocation > failures of order-4 for this allocation. This is a costly order, so it might > easily fail, as t

  1   2   3   4   5   6   7   >