Re: [PATCH v4] mmc: dw_mmc-k3: add sd support for hi3660

2017-06-30 Thread Guodong Xu
Hi, Li Wei I have some minor comments to your patch. > On Tue, Jun 13, 2017 at 3:25 PM, liwei wrote: First of all, please modify author part. It is recommended to use "Li Wei", instead of what you are using now "liwei". If you are using git format-patch, send-email, please modify your ~/.gitcon

Re: [PATCH] mm, vmscan: do not loop on too_many_isolated for ever

2017-06-30 Thread Michal Hocko
On Fri 30-06-17 09:14:22, Tetsuo Handa wrote: [...] > Ping? Ping? When are we going to apply this patch or watchdog patch? > This problem occurs with not so insane stress like shown below. > I can't test almost OOM situation because test likely falls into either > printk() v.s. oom_lock lockup prob

Re: [RFC][PATCHv3 2/5] printk: introduce printing kernel thread

2017-06-30 Thread Steven Rostedt
On Fri, 30 Jun 2017 21:42:24 +0900 Sergey Senozhatsky wrote: > I apply a very simple litmus test. if the answer to the question > "so we leave console_unlock() and there are pending messages, > who and when is going to flush the remaining messages?" is > "something sometime in the future" then it

Re: [PATCH v2] regmap: irq: allow different device for irq chip and regmap

2017-06-30 Thread Michael Grzeschik
On Fri, Jun 23, 2017 at 01:00:43PM +0100, Mark Brown wrote: > On Thu, Jun 22, 2017 at 01:03:20PM +0200, Michael Grzeschik wrote: > > > If the irq chip device is using the regmap of its parent device or > > a syscon regmap that doesn't have an associated device at all, > > allow the driver to provi

[PATCH] ARM: dts: imx6q-bx50v3: disable SD card

2017-06-30 Thread Romain Perier
From: Ian Ray Default boot medium must be internal eMMC, we don't want the SDIO to be used for these devices. This commit disables the SD card interface for GE Healthcare Bx50v3 devices. Signed-off-by: Ian Ray Signed-off-by: Romain Perier --- arch/arm/boot/dts/imx6q-bx50v3.dtsi | 4 1 fi

Re: [RFC][PATCHv3 2/5] printk: introduce printing kernel thread

2017-06-30 Thread Sergey Senozhatsky
On (06/30/17 15:16), Petr Mladek wrote: > Anyway, the handshake during offloading might be pretty > problematic. To be honest, I do not have much experience > with it. I have shared some my fears in the other mail[1]. > Jan Kara spent a lot of time on this and probably could > say more. > > Maybe,

Re: [PATCH NET V5 2/2] net: hns: Use phy_driver to setup Phy loopback

2017-06-30 Thread Andrew Lunn
> Any idea? Maybe consider what the self test is good for. My guess is, self test was added when a network interface card was a full size VME card, and had a couple of hundred components or more. They did break during there life, due to heat, mechanism stresses, causing parts to pop off the PCB,

Re: [PATCH 2/2] drm/atomic: Wait indefinitely and interruptibly for hw_done.

2017-06-30 Thread Daniel Vetter
On Wed, Jun 28, 2017 at 03:28:12PM +0200, Maarten Lankhorst wrote: > Without waiting for hw_done, previous atomic updates may dereference > the wrong state and cause a lot of confusion. The real fix is fixing > all obj->state to use the accessor macros, but for now wait > indefinitely and interrupt

Re: [PATCH] fs: GFS2: constify attribute_group structures.

2017-06-30 Thread Bob Peterson
- Original Message - | attribute_groups are not supposed to change at runtime. All functions | working with attribute_groups provided by work with const | attribute_group. So mark the non-const structs as const. | | File size before: |text data bss dec hex filenam

[PATCH] ARM: dts: imx: Correct B850v3 clock assignment

2017-06-30 Thread Romain Perier
From: Martyn Welch The IPU that drives HDMI must have its pre_sel set to pll2_pfd_396m to avoid stepping on the LVDS output's toes, as the PLL can't be clocked to the pixel clock and to the LVDS serial clock (3.5*pixel clock) at the same time. As we are using ipu1_di0 and ipu2_di0, ensure both a

Re: [PATCH] irqchip: or1k-pic: Fix interrupt system.

2017-06-30 Thread Stafford Horne
On Fri, Jun 30, 2017 at 02:09:32PM +0100, Marc Zyngier wrote: > On 30/06/17 14:04, Stafford Horne wrote: > > +CC Kernel List > > > > On Fri, Jun 30, 2017 at 08:33:56AM -0300, Pedro H. Penna wrote: > >> Usually, hardware implicitly acknowledges interruts when > >> reading them. However, if this is

Re: linux-next: build warning after merge of the pinctrl tree

2017-06-30 Thread Linus Walleij
On Fri, Jun 30, 2017 at 8:59 AM, Geert Uytterhoeven wrote: > On Fri, Jun 30, 2017 at 6:56 AM, Stephen Rothwell > wrote: >> After merging the pinctrl tree, today's linux-next build (x86_64 >> allmodconfig) produced this warning: >> >> drivers/pinctrl/pinctrl-rza1.c: In function 'rza1_pinctrl_prob

[PATCH][scsi-next] scsi: qla2xxx: fix a bunch of typos and spelling mistakes

2017-06-30 Thread Colin King
From: Colin Ian King Fix the following typos/spelling mistakes: "attribure" -> "attribute" "suppored" -> "supported" "Symobilic" -> "Symbolic" "iteself" -> "itself" "reqeust" -> "request" "nvme_wait_on_comand" -> "nvme_wait_on_command" "bount" -> "bound" "captrue_mask" -> "capture_mask" "tempela

Re: [patch 0/5] genirq: Distangle irq_request/release_resources() from irq_desc->lock

2017-06-30 Thread Marc Zyngier
On 29/06/17 22:33, Thomas Gleixner wrote: > The irq_request/release_resources() callbacks are a problem on RT as they > are called under irq_desc->lock with interrupts disabled. > > Aside of that calling them under irq_desc->lock is conceptually wrong and > has a (hard to trigger and probably theo

Re: [PATCH] pinctrl: rza1: make structures rza1_gpiochip_template and rza1_pinmux_ops static

2017-06-30 Thread Linus Walleij
On Fri, Jun 30, 2017 at 12:26 PM, Colin King wrote: > From: Colin Ian King > > structures rza1_gpiochip_template and rza1_pinmux_ops do not need to be > in global scope, so make them static. > > Cleans up sparse warnings: > symbol 'rza1_gpiochip_template' was not declared. Should it be static? >

Re: [PATCH] pinctrl: rza1: fix incorrect failure check on platform_get_resources

2017-06-30 Thread Linus Walleij
On Fri, Jun 30, 2017 at 12:33 PM, Colin King wrote: > From: Colin Ian King > > The return res from platform_get_resources should be checked for > a failure rather than ret. Fixes warning: > > 'ret' may be used uninitialized in this function [-Wmaybe-uninitialized] > > Fixes: 5a49b644b307 ("pinc

Re: [PATCH v2 1/2] livepatch: introduce shadow variable API

2017-06-30 Thread kbuild test robot
Hi Joe, [auto build test WARNING on jikos-livepatching/for-next] [also build test WARNING on v4.12-rc7 next-20170630] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Joe-Lawrence/livepatch

Re: [patch 1/5] pinctrl: samsung: Remove bogus irq_[un]mask from resource management

2017-06-30 Thread Linus Walleij
On Thu, Jun 29, 2017 at 11:33 PM, Thomas Gleixner wrote: > The irq chip callbacks irq_request/release_resources() have absolutely no > business with masking and unmasking the irq. > > The core code unmasks the interrupt after complete setup and masks it > before invoking irq_release_resources().

[PATCH] scsi: snic: fix a couple of spelling mistakes/typos

2017-06-30 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistakes/typos: "Allodating" -> "Allocating" "incative" -> "inactive" Signed-off-by: Colin Ian King --- drivers/scsi/snic/snic_isr.c | 4 ++-- drivers/scsi/snic/snic_scsi.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dri

NAK: [PATCH] scsi: snic: fix a couple of spelling mistakes/typos

2017-06-30 Thread Colin Ian King
Incorrect commit message, I'll resend. On 30/06/17 14:54, Colin King wrote: > From: Colin Ian King > > Trivial fix to spelling mistakes/typos: > > "Allodating" -> "Allocating" > "incative" -> "inactive" > > Signed-off-by: Colin Ian King > --- > drivers/scsi/snic/snic_isr.c | 4 ++-- > drive

Re: [Intel-gfx] [PATCH 1/2] drm/atomic: Change drm_atomic_helper_swap_state to return an error.

2017-06-30 Thread Daniel Vetter
On Wed, Jun 28, 2017 at 03:28:11PM +0200, Maarten Lankhorst wrote: > We want to change swap_state to wait indefinitely, but to do this > swap_state should wait interruptibly. This requires propagating > the error to each driver. All drivers have changes to deal with the > clean up. In order to allo

Re: [PATCH v2 6/8] iio: trigger: Add STM32 LPTimer trigger driver

2017-06-30 Thread Jonathan Cameron
On Mon, 26 Jun 2017 18:41:36 +0200 Fabrice Gasnier wrote: > On 06/24/2017 10:13 PM, Jonathan Cameron wrote: > > On Wed, 21 Jun 2017 16:30:13 +0200 > > Fabrice Gasnier wrote: > > > >> Add support for LPTIMx_OUT triggers that can be found on some STM32 > >> devices. These triggers can be used t

[PATCH] sched/pelt: fix false running accounting in PELT

2017-06-30 Thread Vincent Guittot
The running state is a subset of runnable state which means that running can't be set if runnable (weight) is cleared. There are corner cases where the current sched_entity has been already dequeued but cfs_rq->curr has not been updated yet and still points to the dequeued sched_entity. If ___updat

[PATCH][V2] scsi: snic: fix a couple of spelling mistakes/typos

2017-06-30 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistakes/typos: "requrest_irq" -> "request_irq" "Firmwqre" -> "Firmware" Signed-off-by: Colin Ian King --- drivers/scsi/snic/snic_isr.c | 4 ++-- drivers/scsi/snic/snic_scsi.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/

Re: [tip:sched/core] sched/cputime: Refactor the cputime_adjust() code

2017-06-30 Thread Rik van Riel
On Fri, 2017-06-30 at 06:10 -0700, tip-bot for Gustavo A. R. Silva wrote: > +++ b/kernel/sched/cputime.c > @@ -615,19 +615,13 @@ static void cputime_adjust(struct task_cputime > *curr, >    * userspace. Once a task gets some ticks, the monotonicy > code at >    * 'update' will ensure thing

RE: [PATCH] perf tools: set no branch type for dummy event in PT

2017-06-30 Thread Liang, Kan
> On Thu, Jun 29, 2017 at 03:50:29PM +, Liang, Kan wrote: > > > > > > > > > > On Thu, Jun 29, 2017 at 03:31:45PM +, Liang, Kan wrote: > > > > > > SNIP > > > > > > > > > static int intel_pt_recording_options(struct auxtrace_record *itr, > > > > > > struct

Re: [PATCH] perf tools: Add ARM Statistical Profiling Extensions (SPE) support

2017-06-30 Thread Mark Rutland
Hi Kim, On Wed, Jun 28, 2017 at 08:43:10PM -0500, Kim Phillips wrote: > 'perf record' and 'perf report --dump-raw-trace' supported in this > release. > > Example usage: > > taskset -c 2 ./perf record -C 2 -c 1024 -e arm_spe_0/ts_enable=1,pa_enable=1/ > \ > dd if=/dev/zero of=/dev/

[PATCH v2] modpost: abort if module name is too long

2017-06-30 Thread Wanlong Gao
Module name has a limited length, but currently the build system allows the build finishing even if the module name is too long. CC /root/kprobe_example/abcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyz.mod.o /root/kprobe_example/abcdefghijklmnopqrstuvwxyzabc

[PATCH] objtool: silence warnings for functions which use iret

2017-06-30 Thread Josh Poimboeuf
On Fri, Jun 30, 2017 at 10:32:03AM +0200, Ingo Molnar wrote: > > * Josh Poimboeuf wrote: > > > This is a major rewrite of objtool. Instead of only tracking frame > > pointer changes, it now tracks all stack-related operations, including > > all register saves/restores. > > > > In addition to m

Re: [RFC][PATCHv3 2/5] printk: introduce printing kernel thread

2017-06-30 Thread Sergey Senozhatsky
On (06/30/17 15:16), Petr Mladek wrote: [..] > > I wonder if I could add some counter into task_struct. > It might be configurable. I am not sure if people would > want this enabled on production systems where the level > of messages should be lower anyway. I don't think it really matters which on

Re: [PATCH] iio: Add LTC2471/LTC2473 driver

2017-06-30 Thread Lars-Peter Clausen
On 06/30/2017 09:21 AM, Mike Looijmans wrote: > The LTC2741 and LTC2473 are single voltage ADC chips. The LTC2473 > is similar to the LTC2471 but outputs a signed differential value. > > Datasheet: > http://cds.linear.com/docs/en/datasheet/24713fb.pdf > > Signed-off-by: Mike Looijmans Looks g

Re: [PATCH v5] drm/sun4i: hdmi: Implement I2C adapter for A10s DDC bus

2017-06-30 Thread Jonathan Liu
Hi Chen-Yu and Maxime, On 30 June 2017 at 13:16, Chen-Yu Tsai wrote: > On Fri, Jun 30, 2017 at 6:22 AM, Jonathan Liu wrote: >> Hi Maxime, >> >> On 30 June 2017 at 01:56, Maxime Ripard >> wrote: >>> On Wed, Jun 28, 2017 at 08:39:33PM +1000, Jonathan Liu wrote: >> + u32 int_status;

[PATCH v4 1/8] [media] s5p-jpeg: Call jpeg_bound_align_image after qbuf

2017-06-30 Thread Thierry Escande
From: Tony K Nadackal When queuing an OUTPUT buffer for decoder, s5p_jpeg_parse_hdr() function parses the input jpeg file and takes the width and height parameters from its header. These new width/height values will be used for the calculation of stride. HX_JPEG Hardware needs the width and heigh

[PATCH v4 7/8] [media] s5p-jpeg: Add support for resolution change event

2017-06-30 Thread Thierry Escande
From: henryhsu This patch adds support for resolution change event to notify clients so they can prepare correct output buffer. When resolution change happened, G_FMT for CAPTURE should return old resolution and format before CAPTURE queues streamoff. This event is used in the Chromium browser p

[PATCH v4 2/8] [media] s5p-jpeg: Correct WARN_ON statement for checking subsampling

2017-06-30 Thread Thierry Escande
From: Tony K Nadackal Corrects the WARN_ON statement for subsampling based on the JPEG Hardware version. Signed-off-by: Tony K Nadackal Signed-off-by: Thierry Escande Acked-by: Andrzej Pietrasiewicz Acked-by: Jacek Anaszewski --- drivers/media/platform/s5p-jpeg/jpeg-core.c | 6 -- 1 fil

[PATCH v4 6/8] [media] s5p-jpeg: Decode 4:1:1 chroma subsampling format

2017-06-30 Thread Thierry Escande
From: Tony K Nadackal This patch adds support for decoding 4:1:1 chroma subsampling in the jpeg header parsing function. Signed-off-by: Tony K Nadackal Signed-off-by: Thierry Escande Acked-by: Andrzej Pietrasiewicz Acked-by: Jacek Anaszewski --- drivers/media/platform/s5p-jpeg/jpeg-core.c |

[PATCH v4 5/8] [media] s5p-jpeg: Split s5p_jpeg_parse_hdr()

2017-06-30 Thread Thierry Escande
This patch moves the subsampling value decoding read from the jpeg header into its own function. This new function is called s5p_jpeg_subsampling_decode() and returns true if it successfully decodes the subsampling value, false otherwise. Signed-off-by: Thierry Escande Acked-by: Andrzej Pietrasie

[PATCH v4 8/8] [media] s5p-jpeg: Add stream error handling for Exynos5420

2017-06-30 Thread Thierry Escande
From: henryhsu On Exynos5420, the STREAM_STAT bit raised on the JPGINTST register means there is a syntax error or an unrecoverable error on compressed file when ERR_INT_EN is set to 1. Fix this case and report BUF_STATE_ERROR to videobuf2. Signed-off-by: Henry-Ruey Hsu Signed-off-by: Thierry

[PATCH v4 4/8] [media] s5p-jpeg: Don't use temporary structure in s5p_jpeg_buf_queue

2017-06-30 Thread Thierry Escande
If s5p_jpeg_parse_hdr() fails to parse the JPEG header, the passed s5p_jpeg_q_data structure is not modified so there is no need to use a temporary structure and the field-by-field copy can be avoided. Signed-off-by: Thierry Escande Acked-by: Andrzej Pietrasiewicz Acked-by: Jacek Anaszewski ---

[PATCH v4 0/8] [media] s5p-jpeg: Various fixes and improvements

2017-06-30 Thread Thierry Escande
Hi, This series contains various fixes and improvements for the Samsung s5p-jpeg driver. Most of these patches come from the Chromium v3.8 kernel tree. In this v4: - Correct a typo in patch #04 commit message (Thanks Jacek) - Add Acked-by from Andrzej and Jacek for the whole series v3: - R

[PATCH v4 3/8] [media] s5p-jpeg: Handle parsing error in s5p_jpeg_parse_hdr()

2017-06-30 Thread Thierry Escande
This patch modifies the s5p_jpeg_parse_hdr() function so it only modifies the passed s5p_jpeg_q_data structure if the jpeg header parsing is successful. Signed-off-by: Thierry Escande Acked-by: Andrzej Pietrasiewicz Acked-by: Jacek Anaszewski --- drivers/media/platform/s5p-jpeg/jpeg-core.c | 3

[PATCH V2 2/2] perf tools: always set no branch for dummy event in PT

2017-06-30 Thread kan . liang
From: Kan Liang An earlier kernel patch allowed enabling PT and LBR at the same time on Goldmont. commit ccbebba4c6bf ("perf/x86/intel/pt: Bypass PT vs. LBR exclusivity if the core supports it") However, users still cannot use Intel PT and LBRs simultaneously. $ sudo perf record -e cycles,inte

[PATCH V2 1/2] perf tools: set no_aux_samples for the tracking event in PT

2017-06-30 Thread kan . liang
From: Kan Liang The reason of introducing the tracking event (a dummy software event) is to collect side-band information. Additional sampling is wasteful. no_aux_samples should be set for tracking event. Signed-off-by: Kan Liang --- Changes since V1 - new patch to set no_aux_samples for the t

Re: [PATCH v3] of: document /sys/firmware/fdt

2017-06-30 Thread Rob Herring
On Wed, Jun 28, 2017 at 5:21 PM, wrote: > From: Frank Rowand > > Add ABI documentation for /sys/firmware/fdt > > Update contact email for /sys/firmware/devicetree/* and add mail list > > Signed-off-by: Frank Rowand > --- Applied. Rob

Re: [PATCH 0/7] selftests/ftrace: Some improvements of ftracetest

2017-06-30 Thread Shuah Khan
On 06/14/2017 08:17 AM, Shuah Khan wrote: > Hi Masami, > > On 06/13/2017 10:53 PM, Masami Hiramatsu wrote: >> On Wed, 14 Jun 2017 11:12:17 +0900 >> Masami Hiramatsu wrote: >> >>> Hi Shuah and Steve, >>> >>> Sorry for replying so late... >>> >>> On Thu, 8 Jun 2017 17:51:44 -0400 >>> Steven Rostedt

"mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-06-30 Thread Michal Hocko
fe53ca54270a ("mm: use early_pfn_to_nid in page_ext_init") seem to silently depend on CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID resp. CONFIG_HAVE_MEMBLOCK_NODE_MAP. early_pfn_to_nid is returning zero with !defined(CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID) && !defined(CONFIG_HAVE_MEMBLOCK_NODE_MAP) I am not sure

Re: Question regarding MAX_ARG_STRLEN with execve()

2017-06-30 Thread Michal Hocko
On Fri 30-06-17 11:59:37, Anshuman Khandual wrote: > Hello, > > execve() system call should support argument length of > MAX_ARG_STRLEN (PAGE_SIZE * 32). On 64K page size systems, we > are not able to pass 32 * PAGE_SIZE arguments into the execve() > system call because of the following reasons. >

Re: [PATCH 4/6] dt-bindings: serial: Document RDA Micro UART

2017-06-30 Thread Rob Herring
On Fri, Jun 30, 2017 at 6:12 AM, Andreas Färber wrote: > Am 29.06.2017 um 22:10 schrieb Rob Herring: >> On Tue, Jun 27, 2017 at 02:55:18AM +0200, Andreas Färber wrote: >>> Add an initial binding for the RDA8810PL UART. >>> >>> Signed-off-by: Andreas Färber >>> --- >>> Documentation/devicetree/bi

Re: wake_wide mechanism clarification

2017-06-30 Thread Josef Bacik
On Thu, Jun 29, 2017 at 08:04:59PM -0700, Joel Fernandes wrote: > Hi Josef, > > Thanks a lot for your reply, :-) > > On Thu, Jun 29, 2017 at 5:49 PM, Josef Bacik wrote: > > Because we are trying to detect the case that the master is waking many > > different processes, and the 'slave' processes

Re: [RFC][PATCHv3 2/5] printk: introduce printing kernel thread

2017-06-30 Thread Sergey Senozhatsky
On (06/30/17 09:33), Steven Rostedt wrote: [..] > > "something sometime in the future" is equal to "no one". > > > > we must stay and continue printing. because it gives the right > > answer - "current process and right now. until someone else > > (+printk_kthread) takes over". > > Would it be ac

Re: [tip:x86/platform] x86/PCI/mmcfg: Switch to ECAM config mode if possible

2017-06-30 Thread Thomas Gleixner
On Thu, 29 Jun 2017, Andi Kleen wrote: > > Would this patch actually void the commit: > > Yes it does. My origina; patches were opt-in for this reason, but Thomas > doesn't > believe in historical or other people's experience. I did not realize that particular wreckage. > But MCFG problems were

[PATCHv4 4/4] hwmon: da9052: add support for TSI channel

2017-06-30 Thread Sebastian Reichel
TSI channel has a 4 channel mux connected to it and is normally used for touchscreen support. The hardware may alternatively use it as general purpose adc. Signed-off-by: Sebastian Reichel --- drivers/hwmon/da9052-hwmon.c | 249 +- drivers/mfd/da9052-core

[PATCHv4 2/4] hwmon: da9052: replace S_IRUGO with 0444

2017-06-30 Thread Sebastian Reichel
Fix checkpatch warnings about S_IRUGO being less readable than providing the permissions octal as '0444'. Signed-off-by: Sebastian Reichel --- drivers/hwmon/da9052-hwmon.c | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/drivers/hwmon/da9

[PATCHv4 3/4] dt-bindings: mfd: da9052: support TSI as ADC

2017-06-30 Thread Sebastian Reichel
DA9052 allows using the touchscreen input pins as general purpose analogue input pin by wiring analogue inputs to X+, X-, Y+ and Y- and providing a reference voltage at TSIREF pin. Signed-off-by: Sebastian Reichel --- Documentation/devicetree/bindings/mfd/da9052-i2c.txt | 8 1 file chan

[PATCHv4 1/4] mfd: da9052: fix manual ADC read after timed out read

2017-06-30 Thread Sebastian Reichel
It is possible that under heavy system load, the counter in the completion struct, used for waiting for end of AD conversion, gets incremented twice. To make sure the driver recovers from this situation, the completion struct should be reinitialized. Signed-off-by: Sebastian Reichel --- drivers/

[PATCHv4 0/4] DA9052 hardware monitoring improvements

2017-06-30 Thread Sebastian Reichel
Hi, GE Healthcare's PPD [0] uses DA9053's touchscreen pins for hardware monitoring purposes. This adds support for the feature and fixes a bug, which came up during stress-testing of the driver. [0] https://patchwork.kernel.org/patch/9809681/ Changes since PATCHv3: - use correct index in da9052

[PATCH] random: reorder READ_ONCE() in get_random_uXX

2017-06-30 Thread Sebastian Andrzej Siewior
Since commit 4a072c71f49b ("random: silence compiler warnings and fix race") there is a READ_ONCE(). I doubt that this required here but I don't have the time to argue with Jason. Instead I push the access further down so we don't read crng_init if we can leave the function after arch_get_random_XX

Re: [PATCH v2 04/20] gcc-plugins: Add the randstruct plugin

2017-06-30 Thread Kees Cook
On Fri, Jun 30, 2017 at 1:27 AM, Arnd Bergmann wrote: > On Fri, Jun 30, 2017 at 9:55 AM, Ard Biesheuvel > wrote: >> On 30 June 2017 at 07:35, Arnd Bergmann wrote: >>> On Fri, Jun 30, 2017 at 12:53 AM, Kees Cook wrote: The first obviously won't fly. The second just bypasses the problem

Re: [tip:sched/core] sched/cputime: Refactor the cputime_adjust() code

2017-06-30 Thread Frans Klaver
On Fri, Jun 30, 2017 at 4:00 PM, Rik van Riel wrote: > On Fri, 2017-06-30 at 06:10 -0700, tip-bot for Gustavo A. R. Silva > wrote: > >> +++ b/kernel/sched/cputime.c >> @@ -615,19 +615,13 @@ static void cputime_adjust(struct task_cputime >> *curr, >>* userspace. Once a task gets some ticks,

[PATCH V2] dmaengine: qcom_hidma: correct API violation for submit

2017-06-30 Thread Sinan Kaya
Current code is violating the DMA Engine API by putting the submitted requests directly into the HW queue. This causes queued transactions to be started by another thread as soon as the first one finishes. The DMA Engine document clearly states this. "dmaengine_submit() will not start the DMA ope

Re: [RFC PATCH v3 2/6] dt-bindings: spi/core: add wakeup-source optional property

2017-06-30 Thread Rob Herring
On Mon, Jun 26, 2017 at 9:04 PM, jeffy wrote: > Hi Rob, > > On 06/27/2017 12:40 AM, Dmitry Torokhov wrote: >> >> On Mon, Jun 26, 2017 at 11:00:11AM -0500, Rob Herring wrote: >>> >>> On Wed, Jun 21, 2017 at 06:01:49PM +0800, Jeffy Chen wrote: Update document devicetree bindings to support

Re: [RFC][PATCHv3 2/5] printk: introduce printing kernel thread

2017-06-30 Thread Steven Rostedt
On Fri, 30 Jun 2017 23:28:51 +0900 Sergey Senozhatsky wrote: > I guess the question was, a knob that would determine what happens after > current wakes up printk_kthread -- does it stay in console_unlock() and > wait for new console_sem owner, printing the messages in the meantime, > or goes all

Re: [PATCH] selftests: ftrace: Use md5sum to take less time of checking logs

2017-06-30 Thread Steven Rostedt
On Thu, 29 Jun 2017 14:21:41 -0600 Shuah Khan wrote: \ > Hi Steve/Masami, > > I tested this patch on my test system running 4.12-rc7 and didn't see any > problems. I am going to leave this in the linux-kselftest next for 4.13-rc1. > Please let me if you think I should drop it. I played a little

[GIT PULL] Additional irqchip fixes for 4.13

2017-06-30 Thread Marc Zyngier
Hi Thomas, The following fixes have appeared in my Inbox today, and rather than holding them until -rc1, I though I might send them your way anyway. Nothing spectacular, just the usual fixes for brown paper bag type bugs... Thanks, M. irqchip/gic-v3-its: Don't assume GICv3 hardware su

Re: [PATCH] mm: Fix overflow check in expand_upwards()

2017-06-30 Thread Jörn Engel
On Fri, Jun 30, 2017 at 08:57:27AM +0200, Helge Deller wrote: > On 30.06.2017 01:02, Jörn Engel wrote: > > I believe the overflow check was correct, then got subtly broken by > > commit bd726c90b6b8 > > Author: Helge Deller > > Date: Mon Jun 19 17:34:05 2017 +0200 > > > > Al

Re: [PATCH V2 25/37] perf script: Add synthesized Intel PT power and ptwrite events

2017-06-30 Thread Arnaldo Carvalho de Melo
Em Fri, Jun 30, 2017 at 11:51:22AM +0300, Adrian Hunter escreveu: > On 06/30/2017 05:08 AM, Arnaldo Carvalho de Melo wrote: > > Em Thu, Jun 29, 2017 at 05:13:54PM -0300, Arnaldo Carvalho de Melo escreveu: > > Ok, passes all tests (gcc, clang where available) on: > > 41 ubuntu:16.10: Ok > > 42

Re: Applied "spi: imx: dynamic burst length adjust for PIO mode" to the spi tree

2017-06-30 Thread Jiada Wang
Hello Sascha On 06/30/2017 05:14 AM, Sascha Hauer wrote: Hi Mark, On Fri, Jun 30, 2017 at 01:00:22PM +0100, Mark Brown wrote: The patch spi: imx: dynamic burst length adjust for PIO mode has been applied to the spi tree at git://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.gi

Re: [PATCH 1/2] scsi: lpfc: spin_lock_irq() is not nestable

2017-06-30 Thread James Smart
On 6/30/2017 1:02 AM, Dan Carpenter wrote: We're calling spin_lock_irq() multiple times, the problem is that on the first spin_unlock_irq() then we will re-enable IRQs and we don't want that. Fixes: 966bb5b71196 ("scsi: lpfc: Break up IO ctx list into a separate get and put list") Signed-off-by

Re: [patch 1/5] pinctrl: samsung: Remove bogus irq_[un]mask from resource management

2017-06-30 Thread Krzysztof Kozlowski
On Fri, Jun 30, 2017 at 03:53:18PM +0200, Linus Walleij wrote: > On Thu, Jun 29, 2017 at 11:33 PM, Thomas Gleixner wrote: > > > The irq chip callbacks irq_request/release_resources() have absolutely no > > business with masking and unmasking the irq. > > > > The core code unmasks the interrupt af

Re: [alsa-devel] [PATCH v2 12/12] ASoC: Fix use-after-free at card unregistration

2017-06-30 Thread Robert Jarzmik
Mark Brown writes: > On Thu, Jun 29, 2017 at 12:03:58AM +0200, Robert Jarzmik wrote: >> Mark Brown writes: > >> > Based on the missing first patch I was expecting the series to get >> > reposted? > >> The first patch is only moving around a .h file. I was expecting a review >> before >> reposti

Re: [PATCH] crypto: ccp-platform: print error message on platform_get_irq failure

2017-06-30 Thread Gary R Hook
On 06/30/2017 12:59 AM, Gustavo A. R. Silva wrote: Print error message on platform_get_irq failure before return. Signed-off-by: Gustavo A. R. Silva --- drivers/crypto/ccp/ccp-platform.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/crypto/ccp/ccp-platform.c b/

[PATCH 1/2] mailbox: reset txdone_method TXDONE_BY_POLL if client knows_txdone

2017-06-30 Thread Sudeep Holla
Currently the mailbox framework sets txdone_method to TXDONE_BY_POLL if the controller sets txdone_by_poll. However some clients can have a mechanism to do TXDONE_BY_ACK which they can specify by knows_txdone. However, we endup setting both TXDONE_BY_POLL and TXDONE_BY_ACK in that case. In such sce

[PATCH 2/2] mailbox: mailbox-test: don't rely on rx_buffer content to signal data ready

2017-06-30 Thread Sudeep Holla
Currently we rely on the first byte of the Rx buffer to check if there's any data available to be read. If the first byte of the received buffer is zero (i.e. null character), then we fail to signal that data is available even when it's available. Instead introduce a boolean variable to track the

Re: [PATCH v2] netfilter: nfnetlink: Improve input length sanitization in nfnetlink_rcv

2017-06-30 Thread Mateusz Jurczyk
On Thu, Jun 29, 2017 at 6:22 PM, Pablo Neira Ayuso wrote: > On Tue, Jun 27, 2017 at 07:05:27PM +0200, Pablo Neira Ayuso wrote: >> On Tue, Jun 27, 2017 at 05:58:25PM +0200, Pablo Neira Ayuso wrote: >> > On Wed, Jun 07, 2017 at 03:50:38PM +0200, Mateusz Jurczyk wrote: >> > > Verify that the length o

Re: [PATCH] random: reorder READ_ONCE() in get_random_uXX

2017-06-30 Thread Jason A. Donenfeld
Seems reasonable to me. Signed-off-by: Jason A. Donenfeld

Re: [PATCH RFC 02/26] task_work: Replace spin_unlock_wait() with lock/unlock pair

2017-06-30 Thread Oleg Nesterov
On 06/30, Paul E. McKenney wrote: > > > > + raw_spin_lock_irq(&task->pi_lock); > > > + raw_spin_unlock_irq(&task->pi_lock); > > I agree that the spin_unlock_wait() implementations would avoid the > deadlock with an acquisition from an interrupt handler, while also > avoiding the nee

Re: [PATCH 0/6] staging: iio: tsl2x7x: staging cleanups

2017-06-30 Thread Brian Masney
On Thu, Jun 29, 2017 at 01:03:46PM -0400, Brian Masney wrote: > This begins my work to clean this driver up and eventually move it out > of staging. Driver changes were tested using a TSL2771 hooked up to a > Raspberry Pi 2. > > Thanks to Jon Brenner at AMS/TAOS for loaning me some hardware sample

Re: [PATCH v2 04/20] gcc-plugins: Add the randstruct plugin

2017-06-30 Thread Arnd Bergmann
On Fri, Jun 30, 2017 at 4:41 PM, Kees Cook wrote: > On Fri, Jun 30, 2017 at 1:27 AM, Arnd Bergmann wrote: >> On Fri, Jun 30, 2017 at 9:55 AM, Ard Biesheuvel >> wrote: >>> On 30 June 2017 at 07:35, Arnd Bergmann wrote: On Fri, Jun 30, 2017 at 12:53 AM, Kees Cook wrote: > The first obvi

Re: [PATCH v2] ACPI: surface3_power: MSHW0011 rev-eng implementation

2017-06-30 Thread Benjamin Tissoires
On Jun 29 2017 or thereabouts, Rafael J. Wysocki wrote: > On Thu, Jun 29, 2017 at 4:22 PM, Andy Shevchenko > wrote: > > +Cc: Hans (he might give some advice regarding to the below) > > > > On Thu, Jun 29, 2017 at 3:10 PM, Benjamin Tissoires > > wrote: > >> MSHW0011 replaces the battery firmware b

Re: Where to update regulator register with initial voltage set by HW

2017-06-30 Thread Waldemar Rymarkiewicz
On 30 June 2017 at 14:14, Mark Brown wrote: > On Fri, Jun 30, 2017 at 01:37:32PM +0200, Waldemar Rymarkiewicz wrote: > >> I am not sure if I understand. Do you mean, a regulator should >> determine a voltage and update a register with a right selector when >> system boots, so the regulator framewo

Re: [PATCH v2] ACPI: surface3_power: MSHW0011 rev-eng implementation

2017-06-30 Thread Benjamin Tissoires
On Jun 30 2017 or thereabouts, Hans de Goede wrote: > Hi, > > On 29-06-17 16:22, Andy Shevchenko wrote: > > +Cc: Hans (he might give some advice regarding to the below) > > Thank you for the Cc, so here we have the opposite situation as > with the devices with the AXP288 PMIC and the Cherry Trail

fscrypt request_module() deadlock

2017-06-30 Thread Richard Weinberger
Hi! David and I faced a deadlock with switch_root when fscrypt was in use. When /sbin/modprobe is encrypted using fscrypt and no other kernel component requested an AES cipher before, first access to an encrypted file will trigger the module_request() function, which will execute usermode helper

[PATCH] ARM: fix randomized task_struct

2017-06-30 Thread Arnd Bergmann
With the new task struct randomization, we can run into a build failure for certain random seeds: arch/arm/kernel/entry-armv.S: Assembler messages: arch/arm/kernel/entry-armv.S:803: Error: bad immediate value for offset (4096) Only two constants in asm-offset.h are affected, and I'm changing both

[REGRESSION 4.12] i915 Oops at intel_fbdev_invalidate()

2017-06-30 Thread Takashi Iwai
Hi, I hit an Oops with the latest Linus tree (4.12-rc7+) on a HSW machine like the following at boot: BUG: unable to handle kernel NULL pointer dereference at 00b0 IP: intel_fbdev_invalidate.isra.3+0xc/0x40 [i915] Oops: [#1] PREEMPT SMP CPU: 2 PID: 833 Comm: X Not tainted 4.1

[PATCH] ARM: decompressor: add a memcmp() declaration

2017-06-30 Thread Arnd Bergmann
A reference to memcmp() was added to bitmap.h, which causes another build warning in the decompressor due to the incompatibility with the regular kernel string functions: In file included from /git/arm-soc/include/linux/nodemask.h:94:0, from /git/arm-soc/include/linux/mmzone.h:16,

Re: [REGRESSION 4.12] i915 Oops at intel_fbdev_invalidate()

2017-06-30 Thread Chris Wilson
Quoting Takashi Iwai (2017-06-30 16:38:46) > Hi, > > I hit an Oops with the latest Linus tree (4.12-rc7+) on a HSW machine > like the following at boot: > > BUG: unable to handle kernel NULL pointer dereference at 00b0 > IP: intel_fbdev_invalidate.isra.3+0xc/0x40 [i915] > Oops: 000

Re: [PATCH v2] ACPI: surface3_power: MSHW0011 rev-eng implementation

2017-06-30 Thread Hans de Goede
HI, On 30-06-17 17:24, Benjamin Tissoires wrote: On Jun 29 2017 or thereabouts, Rafael J. Wysocki wrote: On Thu, Jun 29, 2017 at 4:22 PM, Andy Shevchenko wrote: +Cc: Hans (he might give some advice regarding to the below) On Thu, Jun 29, 2017 at 3:10 PM, Benjamin Tissoires wrote: MSHW0011

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-06-30 Thread Michal Hocko
On Fri 30-06-17 16:18:47, Michal Hocko wrote: > fe53ca54270a ("mm: use early_pfn_to_nid in page_ext_init") seem > to silently depend on CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID resp. > CONFIG_HAVE_MEMBLOCK_NODE_MAP. early_pfn_to_nid is returning zero with > !defined(CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID) &&

Re: "mm: use early_pfn_to_nid in page_ext_init" broken on some configurations?

2017-06-30 Thread Michal Hocko
On Fri 30-06-17 17:42:24, Michal Hocko wrote: [...] > diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h > index 16532fa0bb64..894697c1e6f5 100644 > --- a/include/linux/mmzone.h > +++ b/include/linux/mmzone.h > @@ -1055,6 +1055,7 @@ static inline struct zoneref > *first_zones_zonelist(st

Re: [PATCH v2] ACPI: surface3_power: MSHW0011 rev-eng implementation

2017-06-30 Thread Hans de Goede
Hi, On 30-06-17 17:26, Benjamin Tissoires wrote: On Jun 30 2017 or thereabouts, Hans de Goede wrote: +static int mshw0011_i2c_resource_lookup(struct mshw0011_data *cdata, + unsigned int index) +{ + struct i2c_client *client = cdata->adp1; +

Re: [PATCH v2 6/8] x86/entry: add unwind hint annotations

2017-06-30 Thread Andy Lutomirski
On Fri, Jun 30, 2017 at 6:11 AM, Josh Poimboeuf wrote: > On Thu, Jun 29, 2017 at 10:41:44PM -0700, Andy Lutomirski wrote: >> On Thu, Jun 29, 2017 at 10:05 PM, Andy Lutomirski wrote: >> > Hmm. There's another option that might be considerably nicer, though: >> > put the IRQ stack at a known (at l

[PATCH] [net-next] net/mlx5: include wq.o in non-ethernet build for FPGA

2017-06-30 Thread Arnd Bergmann
Both the ethernet and FPGA portions of MLX5 now require the wq functions, and we get a link error when CONFIG_MLX5_CORE_EN is disabled: drivers/net/ethernet/mellanox/mlx5/core/fpga/conn.o: In function `mlx5_fpga_conn_create_cq': conn.c:(.text+0x10b3): undefined reference to `mlx5_cqwq_create' con

Re: [PATCH/RFC 1/4] perf annotate: Add --source-only option

2017-06-30 Thread Taeung Song
Thank you for detailed review. On 06/30/2017 12:18 AM, Namhyung Kim wrote: Resend with a proper mail address.. On Fri, Jun 30, 2017 at 12:14 AM, Namhyung Kim wrote: Hi Taeung, Thanks for doing this work! On Wed, Jun 28, 2017 at 12:14 PM, Taeung Song wrote: The option can show the result o

Re: Early loading of microcode updates with all firmware

2017-06-30 Thread Paul Menzel
Dear Borislav, On 06/30/17 13:37, Borislav Petkov wrote: On Fri, Jun 30, 2017 at 12:44:43PM +0200, Paul Menzel wrote: But, the microcode is not updated. For example, I have to manually run the command below. Yes, you need something in userspace to trigger that reload. Reading the document

[PATCH] platform/x86: sony-laptop: revert 'const' on keymap

2017-06-30 Thread Arnd Bergmann
Marking sony_laptop_input_keycode_map 'const' sounds like a good idea in principle, but unfortunately causes a compiler warning: drivers/platform/x86/sony-laptop.c: In function 'sony_laptop_setup_input': drivers/platform/x86/sony-laptop.c:494:19: error: assignment discards 'const' qualifier from

Re: [tip:sched/core] sched/cputime: Refactor the cputime_adjust() code

2017-06-30 Thread Frederic Weisbecker
On Fri, Jun 30, 2017 at 04:41:50PM +0200, Frans Klaver wrote: > On Fri, Jun 30, 2017 at 4:00 PM, Rik van Riel wrote: > > On Fri, 2017-06-30 at 06:10 -0700, tip-bot for Gustavo A. R. Silva > > wrote: > > > >> +++ b/kernel/sched/cputime.c > >> @@ -615,19 +615,13 @@ static void cputime_adjust(struct

[PATCH] usb: typec: include linux/device.h in ucsi.h

2017-06-30 Thread Arnd Bergmann
The new driver causes a build failure in some configurations: In file included from /git/arm-soc/drivers/usb/typec/ucsi/trace.h:9:0, from /git/arm-soc/drivers/usb/typec/ucsi/trace.c:2: drivers/usb/typec/ucsi/ucsi.h:331:39: error: 'struct device' declared inside parameter list wil

[PATCH] kmod: add dependencies for test module

2017-06-30 Thread Arnd Bergmann
When we try to 'select' drivers whose dependencies are not met, we get a kconfig warning and undefined behavior as further 'select' statements in those drivers can not be honored: warning: (TEST_KMOD) selects TUN which has unmet direct dependencies (NETDEVICES && NET_CORE && INET) warning: (TEST_

Re: [PATCH] ARM: fix randomized task_struct

2017-06-30 Thread Ard Biesheuvel
> On 30 Jun 2017, at 15:34, Arnd Bergmann wrote: > > With the new task struct randomization, we can run into a build > failure for certain random seeds: > > arch/arm/kernel/entry-armv.S: Assembler messages: > arch/arm/kernel/entry-armv.S:803: Error: bad immediate value for offset (4096) > > O

Re: [PATCHv3] ARM: dts: imx53: Add GE Healthcare PPD

2017-06-30 Thread Fabio Estevam
Hi Sebastian, On Fri, Jun 30, 2017 at 6:54 AM, Sebastian Reichel wrote: > diff --git a/arch/arm/boot/dts/imx53-ppd.dts b/arch/arm/boot/dts/imx53-ppd.dts > new file mode 100644 > index ..56c24765b6f9 > --- /dev/null > +++ b/arch/arm/boot/dts/imx53-ppd.dts > @@ -0,0 +1,1022 @@ > +/* >

<    1   2   3   4   5   6   7   8   >