Re: [PATCH v3 1/4] crypto: exynos - Support Exynos5250+ SoCs

2017-12-13 Thread Krzysztof Kozlowski
On Tue, Dec 12, 2017 at 5:36 PM, Łukasz Stelmach wrote: > Add support for PRNG in Exynos5250+ SoCs. > > Signed-off-by: Łukasz Stelmach > --- > .../bindings/crypto/samsung,exynos-rng4.txt| 4 +++- > drivers/crypto/exynos-rng.c| 27 > -- > 2 fi

Re: [PATCH v3 2/4] crypto: exynos - Improve performance of PRNG

2017-12-13 Thread Krzysztof Kozlowski
On Tue, Dec 12, 2017 at 5:36 PM, Łukasz Stelmach wrote: > Use memcpy_fromio() instead of custom exynos_rng_copy_random() function > to retrieve generated numbers from the registers of PRNG. If I recall correctly, you mentioned before that it improves the performance. If so it would be nice to men

Re: [RFC PATCH 1/2] dt: bindings: as3645a: Update dt node example with standard

2017-12-13 Thread Laurent Pinchart
Hi Dan, Thank you for the patch. On Tuesday, 12 December 2017 23:50:23 EET Dan Murphy wrote: > Update the DT binding to remove the device name from > the DT parent node as well as removing the device > name from the label. The LED label will be generated > based off the id name stored in the loc

Re: [PATCH v3 3/4] crypto: exynos - Reseed PRNG after generating 2^16 random bytes

2017-12-13 Thread Krzysztof Kozlowski
On Tue, Dec 12, 2017 at 5:36 PM, Łukasz Stelmach wrote: > Reseed PRNG after reading 65 kB of randomness. Although this may reduce > performance, in most cases the loss is not noticeable. Also the time > based threshold for reseeding is changed to one second. Reseeding is > performed whenever eithe

[PATCH 0/3] platform/x86: dell-laptop: All the warnings are resolved.

2017-12-13 Thread Dhaval Shah
There are total three types of warnings are resolved. First patch : Block comments use * on subsequent lines Second patch : Missing a blank line after declarations third patch : braces {} are not necessary for single statement blocks. Dhaval Shah (3): platform/x86: dell-laptop: Block comments

Re: [PATCH v2] spi: s3c64xx: add SPDX identifier

2017-12-13 Thread Krzysztof Kozlowski
On Wed, Dec 13, 2017 at 9:05 AM, Andi Shyti wrote: > Replace the original license statement with the SPDX identifier. > > Signed-off-by: Andi Shyti > --- > Hi, > > this second version unlike v1, where I tried to move the license > from GPLv2+ to GPLv2, preserves the original licese while adding >

[PATCH 3/3] platform/x86: dell-laptop: braces {} are not necessary for single statement blocks

2017-12-13 Thread Dhaval Shah
Resolved braces {} are not necessary for single statement blocks checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/platform/x86/dell-laptop.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/platform/x86/dell-laptop.c b/drivers/p

Re: [PATCH AUTOSEL for 4.14 19/60] Bluetooth: avoid silent hci_bcm ACPI PM regression

2017-12-13 Thread Johan Hovold
On Wed, Dec 13, 2017 at 01:55:14AM +, alexander.le...@verizon.com wrote: > From: Johan Hovold > > [ Upstream commit 4294625e029028854596865be401b9c5c1f906ef ] > > The hci_bcm platform-device hack which was used to implement > power management for ACPI devices is being replaced by a > serial-

[PATCH 2/3] platform/x86: dell-laptop: Missing a blank line after declarations

2017-12-13 Thread Dhaval Shah
Resolved Missing a blank line after declarations checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/platform/x86/dell-laptop.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/platform/x86/dell-laptop.c b/drivers/platform/x86/dell-laptop.c inde

Re: [PATCH v2 05/13] pinctrl: Add Microsemi Ocelot SoC driver

2017-12-13 Thread Linus Walleij
On Fri, Dec 8, 2017 at 4:46 PM, Alexandre Belloni wrote: > The Microsemi Ocelot SoC has a few pins that can be used as GPIOs or take > multiple other functions. Add a driver for the pinmuxing and the GPIOs. > > There is currently no support for interrupts. > > Cc: Linus Walleij > Cc: linux-g...@

[PATCH 1/3] platform/x86: dell-laptop: Block comments use * on subsequent lines

2017-12-13 Thread Dhaval Shah
Resolved Block comments use * on subsequent lines checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/platform/x86/dell-laptop.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/platform/x86/dell-laptop.c b/drivers/platform/x86/del

Re: [PATCH] dt-bindings: pinctrl: stm32: fix copyright and adopt SPDX identifier

2017-12-13 Thread Linus Walleij
On Fri, Dec 8, 2017 at 4:53 PM, Alexandre Torgue wrote: > Add missing copyright and add SPDX identifier. > > Signed-off-by: Alexandre Torgue Patch applied. Yours, Linus Walleij

[PATCH] platform/x86: dell-wmi-aio: Block comments use * on subsequent lines

2017-12-13 Thread Dhaval Shah
Resolved Block comments use * on subsequent lines checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/platform/x86/dell-wmi-aio.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/platform/x86/dell-wmi-aio.c b/drivers/platform/x

Re: [PATCH v3 09/33] nds32: Cache and TLB routines

2017-12-13 Thread Guo Ren
On Wed, Dec 13, 2017 at 01:45:02PM +0800, Greentime Hu wrote: > I think it should be fine if an interruption between mtsr_dsb and > tlbop_rwr because this is a optimization by sw. Fine? When there is an unexpected vaddr in SR_TLB_VPN, tlbop_rwr(*pte) will break that vaddr's pfn in the CPU tlb-bu

[PATCH] cpufreq: powernv: Add support of frequency domain

2017-12-13 Thread Abhishek Goel
Frequency-domain indicates group of CPUs that would share same frequency. It is detected using device-tree node "frequency-domain-indicator". frequency-domain-indicator is a bitmask which will have different value depending upon the generation of the processor. CPUs of the same chip for which the

[PATCH] platform/x86: dell-smbios: Block comments use * on subsequent lines

2017-12-13 Thread Dhaval Shah
Resolved Block comments use * on subsequent lines checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/platform/x86/dell-smbios.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/platform/x86/dell-smbios.c b/drivers/platform/x86/del

Re: mailbox: ti-msgmgr: Switch to SPDX Licensing

2017-12-13 Thread Lokesh Vutla
On Saturday 02 December 2017 03:52 PM, Nishanth Menon wrote: > Switch to SPDX licensing and drop the GPL text which comes redundant. > > Signed-off-by: Nishanth Menon Reviewed-by: Lokesh Vutla Thanks and regards, Lokesh

Re: [PATCH v4] Bluetooth: hci_ll: Add optional nvmem BD address source

2017-12-13 Thread Marcel Holtmann
Hi David, > This adds an optional nvmem consumer to get a BD address from an external > source. The BD address is then set in the Bluetooth chip after the > firmware has been loaded. > > This has been tested working with a TI CC2560A chip (in a LEGO MINDSTORMS > EV3). > > Signed-off-by: David Le

Re: [PATCH] pty: cancel pty slave port buf's work in tty_release

2017-12-13 Thread Greg KH
On Wed, Dec 13, 2017 at 09:10:48AM +0400, kpark3...@gmail.com wrote: > From: Sahara I need a "Full" name here, I doubt you sign legal documents with just the single name, right? > > In case that CONFIG_SLUB_DEBUG is on and pty is used, races between > release_one_tty and flush_to_ldisc work thr

[PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines

2017-12-13 Thread Dhaval Shah
Resolved Block comments use * on subsequent lines checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/platform/x86/dell-rbtn.c | 26 +- drivers/platform/x86/dell-rbtn.h | 26 +- 2 files changed, 26 insertions(+),

[PATCH] platform/x86: dell-smo8800: Possible unnecessary 'out of memory' message

2017-12-13 Thread Dhaval Shah
Removed Possible unnecessary 'out of memory' message checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/platform/x86/dell-smo8800.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/platform/x86/dell-smo8800.c b/drivers/platform/x

Re: [PATCH v5 0/4] ARM: ep93xx: ts72xx: Add support for BK3 board

2017-12-13 Thread Alexander Sverdlin
Hello Linus! On Wed Dec 13 08:34:22 2017 Linus Walleij wrote: > On Tue, Dec 12, 2017 at 12:36 AM, Lukasz Majewski wrote: > Out of curiosity: Liebherr is obviously doing heavy-duty industrial > control systems. Likewise Hartley is doing similar business over > at Vision Engravings. > > Is the si

Re: [PATCH 1/2] PCI: rcar: Fix use-after-free in probe error path

2017-12-13 Thread Simon Horman
On Tue, Dec 12, 2017 at 01:29:42PM -0600, Bjorn Helgaas wrote: > On Thu, Dec 07, 2017 at 11:15:19AM +0100, Geert Uytterhoeven wrote: > > If CONFIG_DEBUG_SLAB=y, and no PCIe card is inserted, the kernel crashes > > during probe on r8a7791/koelsch: > > > > rcar-pcie fe00.pcie: PCIe link down

Re: [PATCH] platform/x86: dell-smo8800: Possible unnecessary 'out of memory' message

2017-12-13 Thread Pali Rohár
On Wednesday 13 December 2017 13:53:54 Dhaval Shah wrote: > Removed Possible unnecessary 'out of memory' message checkpatch warnings. > Issue found by checkpatch. > > Signed-off-by: Dhaval Shah > --- > drivers/platform/x86/dell-smo8800.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-)

Re: [PATCH v3 09/33] nds32: Cache and TLB routines

2017-12-13 Thread Greentime Hu
2017-12-13 16:19 GMT+08:00 Guo Ren : > On Wed, Dec 13, 2017 at 01:45:02PM +0800, Greentime Hu wrote: > >> I think it should be fine if an interruption between mtsr_dsb and >> tlbop_rwr because this is a optimization by sw. > > Fine? When there is an unexpected vaddr in SR_TLB_VPN, tlbop_rwr(*pte) w

Re: [PATCH v2] rtc: Add tracepoints for RTC system

2017-12-13 Thread Arnd Bergmann
On Wed, Dec 13, 2017 at 6:47 AM, Baolin Wang wrote: > >>> diff --git a/include/trace/events/rtc.h b/include/trace/events/rtc.h >>> new file mode 100644 >>> index 000..b5a4add >>> --- /dev/null >>> +++ b/include/trace/events/rtc.h >>> + >> >> Also, I'm a bit concerned about having a struct rtc_

Re: [PATCH] s390/decompressor: add fortify_panic as x86 has.

2017-12-13 Thread Heiko Carstens
On Mon, Dec 11, 2017 at 02:09:04PM +0100, Michal Suchánek wrote: > Hello, > > On Thu, 7 Dec 2017 13:55:07 +0100 > Martin Schwidefsky wrote: > > > On Thu, 7 Dec 2017 11:37:27 +0100 > > Michal Suchanek wrote: > > > > > Fix following error: > > > > > > LD arch/s390/boot/compressed/vmlinu

TSC x86 fixes for LTS kernel 4.9.x

2017-12-13 Thread Dan Aloni
Hi all, I've tested the following changes, belonging to merge commit f7dd3b1734e, on top of 4.9.68 after a very easy backport from 4.10, and I think it may be worthwhile adding them to 4.9.x: x86/tsc: Limit the adjust value further x86/tsc: Annotate printouts as firmware bug x86

[PATCH 5/4] scsi: arcmsr: simplify arcmsr_request_device_map routine

2017-12-13 Thread Ching Huang
From: Ching Huang simplify arcmsr_request_device_map routine Signed-off-by: Ching Huang --- diff --git a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c index 95c9f08..4774559 100755 --- a/drivers/scsi/arcmsr/arcmsr_hba.c +++ b/drivers/scsi/arcmsr/arcmsr_hba.c @@ -122,9 +12

[PATCH] USB: serial: ark3116.c: Remove unused TIOCSSERIAL case from ioctl

2017-12-13 Thread Mikhail Zaytsev
The patch removes unused TIOCSSERIAL case from ioctl. TIOCGSERIAL case moves to the get_serial_info() function. Any magic numbers moves to #define directives. Signed-off-by: Mikhail Zaytsev --- drivers/usb/serial/ark3116.c | 54 ++-- 1 file changed, 32 i

Re: [PATCH 0/2] Bluetooth: Add device tree compatible for TI CC2560

2017-12-13 Thread Marcel Holtmann
Hi David, > This series updates the bindings TI WiLink 7/8 Bluetooth to add TI CC256x > chips as well. A compatible string is also added to the hci_ll driver for > TI CC2560. > > David Lechner (2): > dt-bindings: net: add TI CC2560 Bluetooth chip > Bluetooth: hci_ll: add "ti,cc2560" compatible

Lithium battery protection was Re: [PATCH v4.14] Add support for bq27521 battery monitor

2017-12-13 Thread Pavel Machek
Hi! > > >> This adds basic support for BQ27521 battery monitor, used in Nokia N9 > > >> and N950. In particular, battery voltage is important to be able to > > >> tell when the battery is almost empty. Emptying battery on N950 is > > >> pretty painful, as flasher needs to be used to recover phone

Re: [PATCH 1/1] dt-bindings: arm: document supported STM32 SoC family

2017-12-13 Thread Ludovic BARRE
hi Rob Today there was no bindings documentation for STM32 SoC already upstreamed. This patch adds initial list of STM32 existing in kernel. The next serie adds just new soc and one compatible on STM32 list. So, I think you could apply this patch. BR Ludo On 12/13/2017 12:17 AM, Rob Herring wro

[PATCH] drm: Update edid-derived drm_display_info fields at edid property set [v2]

2017-12-13 Thread Keith Packard
There are a set of values in the drm_display_info structure for each connector which hold information derived from EDID. These are computed in drm_add_display_info. Before this patch, that was only called in drm_add_edid_modes. This meant that they were only set when EDID was present and never rese

Re: [PATCH] KVM: introduce kvm_arch_vcpu_async_ioctl

2017-12-13 Thread Cornelia Huck
On Tue, 12 Dec 2017 18:31:55 +0100 Paolo Bonzini wrote: > After the vcpu_load/vcpu_put pushdown, the handling of asynchronous VCPU > ioctl is already much clearer in that it is obvious that they bypass > vcpu_load and vcpu_put. > > However, it is still not perfect in that the different state of

Re: [PATCH 4.14 000/164] 4.14.6-stable review

2017-12-13 Thread Daniel Vetter
On Wed, Dec 13, 2017 at 8:57 AM, Greg Kroah-Hartman wrote: > On Wed, Dec 13, 2017 at 07:48:43AM +0100, Marek Szyprowski wrote: >> Hi Shuah and Greg, >> >> On 2017-12-12 15:47, Shuah Khan wrote: >> > On 12/12/2017 05:43 AM, Greg Kroah-Hartman wrote: >> > > This is the start of the stable review cyc

Re: [PATCH v5 0/4] ARM: ep93xx: ts72xx: Add support for BK3 board

2017-12-13 Thread Lukasz Majewski
Hi Alexander, Linus > Hello Linus! > > On Wed Dec 13 08:34:22 2017 Linus Walleij > wrote: > > On Tue, Dec 12, 2017 at 12:36 AM, Lukasz Majewski > > wrote: Out of curiosity: Liebherr is obviously doing heavy-duty > > industrial control systems. Likewise Hartley is doing similar > > business over

Re: net/wireless/shipped-certs.c:2:1: error: expected expression at end of input

2017-12-13 Thread Damian Tometzki
Hello Johannes, I got. the same Build error. VG Damian Tometzki > Am 05.12.2017 um 11:03 schrieb Johannes Berg : > >> On Tue, 2017-12-05 at 11:01 +0100, Paul Menzel wrote: >> >> >> ``` >> $ git describe >> v4.15-rc2-79-gfd6d2e506ce6 >> $ git log --oneline -1 >> fd6d2e506ce6 Merge tag 'd

Re: [PATCH -mm] mm, swap: Fix race between swapoff and some swap operations

2017-12-13 Thread Huang, Ying
Minchan Kim writes: > Hi Huang, > > Sorry for the late response. I'm in middle of long vacation. > > On Fri, Dec 08, 2017 at 08:32:16PM +0800, Huang, Ying wrote: >> Minchan Kim writes: >> >> > On Fri, Dec 08, 2017 at 04:41:38PM +0800, Huang, Ying wrote: >> >> Minchan Kim writes: >> >> >> >>

Re: [PATCH] hyperv: make HYPERV a menuconfig to ease disabling it all

2017-12-13 Thread Vincent Legoll
Hello, On Sun, Dec 10, 2017 at 6:50 AM, Stephen Hemminger wrote: > Will this break existing configs? I don't think so. Last time I did some similar changes, the kbuild test robot found some warnings on some configurations, I hope it will find problems (if any) for that series too (this one is no

Re: [PATCH v3 09/33] nds32: Cache and TLB routines

2017-12-13 Thread Guo Ren
On Wed, Dec 13, 2017 at 04:30:41PM +0800, Greentime Hu wrote: > 2017-12-13 16:19 GMT+08:00 Guo Ren : > > On Wed, Dec 13, 2017 at 01:45:02PM +0800, Greentime Hu wrote: > > > >> I think it should be fine if an interruption between mtsr_dsb and > >> tlbop_rwr because this is a optimization by sw. > >

Re: [PATCH 5/5] phy: renesas: rcar-gen3-usb2: add gpio handling

2017-12-13 Thread Geert Uytterhoeven
Hi Shimoda-san, On Wed, Dec 13, 2017 at 7:27 AM, Yoshihiro Shimoda wrote: > Some R-Car SoCs (e.g. R-Car D3) doesn't have dedicated pins of VBUS > and ID. So, they may be connected to gpio pins. To handle the gpio > pins, this patch adds the handling of VBUS and ID pins instead of > dedicated pins

[PATCH] pinctrl: axp209: dereference pointer after it's been set

2017-12-13 Thread Quentin Schulz
The number of GPIOs is gotten from a field within the structure referenced in the of_device.data but it was actually read before it was retrieved, thus it was dereferencing a null pointer. Set the number of GPIOs after retrieving of_device.data. Fixes: e1190083b89b ("pinctrl: axp209: add support

Re: [PATCH] KVM: arm/arm64: don't set vtimer->cnt_ctl in kvm_arch_timer_handler

2017-12-13 Thread Marc Zyngier
Hi Jia, On 13/12/17 07:00, Jia He wrote: > In our Armv8a server (qualcomm Amberwing, non VHE), after applying > Christoffer's timer optimizing patchset(Optimize arch timer register > handling), the guest is hang during kernel booting. > > The error root cause might be as follows: > 1. in kvm_arch

Re: [Patch v6 10/12] [media] v4l2: Add v4l2 control IDs for HEVC encoder

2017-12-13 Thread Smitha T Murthy
On Tue, 2017-12-12 at 10:46 +0100, Sylwester Nawrocki wrote: > On 12/12/2017 03:34 AM, Smitha T Murthy wrote: > >> s/Lay/Layer here and below > >> > > Ok I will change it. > > While it's fine to make such change for controls up to > V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L6_QP... > > >>> + case V4

[BUG] kaweth: a possible sleep-in-atomic bug in kaweth_start_xmit

2017-12-13 Thread Jia-Ju Bai
According to drivers/net/usb/kaweth.c, the driver may sleep under a spinlock. The function call path is: kaweth_start_xmit (acquire the spinlock) kaweth_async_set_rx_mode kaweth_control kaweth_internal_control_msg usb_start_wait_urb wait_event_timeout --> may sleep

Re: [Intel-gfx] GemniLake laptops goes power off directly after performing suspend

2017-12-13 Thread Chris Chiu
On Tue, Dec 12, 2017 at 9:32 PM, Imre Deak wrote: > On Fri, Dec 08, 2017 at 10:31:30AM +, Daniel Drake wrote: >> Hi, >> >> Adding intel-gfx list in case i915 developers can help. Updated summary >> below. >> >> On Thu, Dec 7, 2017 at 2:14 AM, Chris Chiu wrote: >> > On Wed, Dec 6, 2017 at 9:3

Re: [PATCH] ptp: make PTP a menuconfig to ease disabling it all

2017-12-13 Thread Vincent Legoll
Hello, On Mon, Dec 11, 2017 at 4:28 AM, Richard Cochran wrote: > NAK, you are introducing a new config option for no good reason. I don't think there are alternatives available in Kconfig to achieve the goal of easing the management of a .config with menuconfig. Do you have any idea about how t

Re: [PATCH 2/6] ARM: stm32: add initial support for STM32MP157

2017-12-13 Thread Ludovic BARRE
On 12/13/2017 12:24 AM, Rob Herring wrote: On Fri, Dec 08, 2017 at 03:11:13PM +0100, Ludovic Barre wrote: From: Ludovic Barre This patch adds initial support of STM32MP157 microprocessor (MPU) based on Arm Cortex-A7. Under new ARCH_STM32_MPU flag we select the needed Cortex-A infrastructure

Re: TSC x86 fixes for LTS kernel 4.9.x

2017-12-13 Thread Greg KH
On Wed, Dec 13, 2017 at 10:33:52AM +0200, Dan Aloni wrote: > Hi all, > > I've tested the following changes, belonging to merge commit f7dd3b1734e, > on top of 4.9.68 after a very easy backport from 4.10, and I think it > may be worthwhile adding them to 4.9.x: > > x86/tsc: Limit the adjust

Re: [PATCH v3 09/33] nds32: Cache and TLB routines

2017-12-13 Thread Greentime Hu
2017-12-13 16:53 GMT+08:00 Guo Ren : > On Wed, Dec 13, 2017 at 04:30:41PM +0800, Greentime Hu wrote: >> 2017-12-13 16:19 GMT+08:00 Guo Ren : >> > On Wed, Dec 13, 2017 at 01:45:02PM +0800, Greentime Hu wrote: >> > >> >> I think it should be fine if an interruption between mtsr_dsb and >> >> tlbop_rw

Re: [GIT PULL for v4.15-rc3] media fixes

2017-12-13 Thread Geert Uytterhoeven
Hi Mauro, On Mon, Dec 11, 2017 at 12:12 PM, Mauro Carvalho Chehab wrote: > Without this series, I was getting 809 lines of bogus warnings (see below), > with was preventing me to see new warnings on my incremental builds > while applying new patches at the media tree. $ linux-log-diff build.log{

Re: [PATCH] vfio: Simplify capability helper

2017-12-13 Thread Kirti Wankhede
On 12/13/2017 12:31 PM, Zhenyu Wang wrote: > On 2017.12.13 12:13:34 +1100, Alexey Kardashevskiy wrote: >> On 13/12/17 06:59, Alex Williamson wrote: >>> The vfio_info_add_capability() helper requires the caller to pass a >>> capability ID, which it then uses to fill in header fields, assuming >>>

Re: [PATCH v3 1/6] cpufreq: schedutil: reset sg_cpus's flags at IDLE enter

2017-12-13 Thread Viresh Kumar
On 12-12-17, 15:16, Patrick Bellasi wrote: > Since you are already changing some flags position, maybe we can have > a better organization by using lower flags for "general bits" and > higher ones for class specific, i.e. > > #define SCHED_CPUFREQ_CLEAR (1U << 0) > #define SCHED_CPUFREQ_IOWAIT

Re: [PATCH V3] PM: In kernel power management domain_pm created for async schedules

2017-12-13 Thread Vikas Bansal
  Sender : Rafael J. Wysocki  Date : 2017-12-06 19:48 (GMT+5:30)   > On Wednesday, December 6, 2017 3:12:38 PM CET gre...@linuxfoundation.org  > wrote: > > On Wed, Dec 06, 2017 at 12:07:14PM +, Vikas Bansal wrote: > > > Description: > >  > > Why is this here? > >  > > >  > > > If there is a d

Re: [PATCH] KVM: arm/arm64: don't set vtimer->cnt_ctl in kvm_arch_timer_handler

2017-12-13 Thread Auger Eric
Marc, On 13/12/17 09:56, Marc Zyngier wrote: > Hi Jia, > > On 13/12/17 07:00, Jia He wrote: >> In our Armv8a server (qualcomm Amberwing, non VHE), after applying >> Christoffer's timer optimizing patchset(Optimize arch timer register >> handling), the guest is hang during kernel booting. >> >> The

[PATCH] qedi: Fix a possible sleep-in-atomic bug in qedi_process_tmf_resp

2017-12-13 Thread Jia-Ju Bai
The driver may sleep under a spinlock. The function call path is: qedi_cpu_offline (acquire the spinlock) qedi_fp_process_cqes qedi_mtask_completion qedi_process_tmf_resp kzalloc(GFP_KERNEL) --> may sleep To fix it, GFP_KERNEL is replaced with GFP_ATOMIC. This bug is found by

Re: [PATCH v2] drm: rcar-du: calculate DPLLCR to be more small jitter

2017-12-13 Thread Laurent Pinchart
Hello Morimoto-san, Thank you for the patch. On Wednesday, 6 December 2017 08:05:38 EET Kuninori Morimoto wrote: > From: Kuninori Morimoto > > In general, PLL has VCO (= Voltage controlled oscillator), > one of the very important electronic feature called as "jitter" > is related to this VCO. >

Re: [PATCH 2/5] phy: renesas: rcar-gen3-usb2: unify OBINTEN handling

2017-12-13 Thread Sergei Shtylyov
Hello! On 12/13/2017 9:27 AM, Yoshihiro Shimoda wrote: This patch unifies the OBINTEN handling to clean-up the code. Signed-off-by: Yoshihiro Shimoda --- drivers/phy/renesas/phy-rcar-gen3-usb2.c | 23 +++ 1 file changed, 15 insertions(+), 8 deletions(-) diff --git a/dr

Re: [PATCH 6/6] crypto: tcrypt: add multibuf aead speed test

2017-12-13 Thread Gilad Ben-Yossef
Hi Herbert, On Mon, Dec 11, 2017 at 1:35 PM, Herbert Xu wrote: > On Thu, Nov 30, 2017 at 10:09:32AM +, Gilad Ben-Yossef wrote: >> The performance of some aead tfm providers is affected by >> the amount of parallelism possible with the processing. >> >> Introduce an async aead concurrent multi

Re: [PATCH] qedi: Fix a possible sleep-in-atomic bug in qedi_process_tmf_resp

2017-12-13 Thread Rangankar, Manish
On 13/12/17 2:41 PM, "Jia-Ju Bai" wrote: >The driver may sleep under a spinlock. >The function call path is: >qedi_cpu_offline (acquire the spinlock) > qedi_fp_process_cqes >qedi_mtask_completion > qedi_process_tmf_resp >kzalloc(GFP_KERNEL) --> may sleep > >To fix it, GFP_KERN

Re: [PATCH v5 10/15] soundwire: Add sysfs for SoundWire DisCo properties

2017-12-13 Thread Greg Kroah-Hartman
On Wed, Dec 06, 2017 at 09:17:06PM +0530, Vinod Koul wrote: > It helps to read the properties for understanding and debugging > systems, so add sysfs files for SoundWire DisCo properties. > > TODO: Add ABI files for sysfs Is this TODO done? > +/* > + * The sysfs for properties reflects the MIPI

Re: [PATCH v5 1/8] clocksource: dmtimer: Remove all the exports

2017-12-13 Thread Ladislav Michl
On Tue, Dec 12, 2017 at 10:21:50AM -0800, Tony Lindgren wrote: > * Ladislav Michl [171212 18:06]: > > I do not follow. Each general-purpose timer module has its own interrupt > > line, > > so claiming that irq directly using request_irq seems enough. Could you > > explain interrupt controller ide

Re: [PATCH v5 03/15] soundwire: Add Master registration

2017-12-13 Thread Greg Kroah-Hartman
On Wed, Dec 06, 2017 at 09:16:59PM +0530, Vinod Koul wrote: > A Master adds a SoundWire bus instance which scans the firmware > provided for device description. > > In this patch we scan ACPI namespaces and create SoundWire > Slave devices based on ACPI description > > Signed-off-by: Sanyog Kale

Re: [PATCH v5 02/15] soundwire: Add SoundWire bus type

2017-12-13 Thread Greg Kroah-Hartman
On Wed, Dec 06, 2017 at 09:16:58PM +0530, Vinod Koul wrote: > This adds the base SoundWire bus type, bus and driver registration. > along with changes to module device table for new SoundWire > device type. > > Signed-off-by: Sanyog Kale > Signed-off-by: Vinod Koul Looks good: Reviewed-by: Gre

Re: net/wireless/shipped-certs.c:2:1: error: expected expression at end of input

2017-12-13 Thread Paul Menzel
Dear Damian, On 12/13/17 08:53, Damian Tometzki wrote: I got. the same Build error. So first you didn’t have hexdump installed, and then after installing it you got it? Removing the generated file fixed the build? Kind regards, Paul

Re: [PATCH] KVM: arm/arm64: don't set vtimer->cnt_ctl in kvm_arch_timer_handler

2017-12-13 Thread Christoffer Dall
On Tue, Dec 12, 2017 at 11:00:07PM -0800, Jia He wrote: > In our Armv8a server (qualcomm Amberwing, non VHE), after applying > Christoffer's timer optimizing patchset(Optimize arch timer register > handling), the guest is hang during kernel booting. > > The error root cause might be as follows: >

Re: [PATCH v2] KVM: X86: Fix host dr6 miss restore

2017-12-13 Thread David Hildenbrand
On 13.12.2017 04:10, Wanpeng Li wrote: > From: Wanpeng Li > > Reported by syzkaller: > >WARNING: CPU: 0 PID: 12927 at arch/x86/kernel/traps.c:780 > do_debug+0x222/0x250 >CPU: 0 PID: 12927 Comm: syz-executor Tainted: G OE > 4.15.0-rc2+ #16 >RIP: 0010:do_debug+0x222/0x25

Re: [PATCH v5 00/15] soundwire: Add a new SoundWire subsystem

2017-12-13 Thread Greg Kroah-Hartman
On Wed, Dec 06, 2017 at 09:16:56PM +0530, Vinod Koul wrote: > This patch series adds a new SoundWire subsystem which implements a > new MIPI bus protocol 'SoundWire'. > > The SoundWire protocol is a robust, scalable, low complexity, low > power, low latency, two-pin (clock and data) multi-drop bus

Re: [PATCH] KVM: arm/arm64: don't set vtimer->cnt_ctl in kvm_arch_timer_handler

2017-12-13 Thread Christoffer Dall
On Wed, Dec 13, 2017 at 08:56:12AM +, Marc Zyngier wrote: > Hi Jia, > > On 13/12/17 07:00, Jia He wrote: > > In our Armv8a server (qualcomm Amberwing, non VHE), after applying > > Christoffer's timer optimizing patchset(Optimize arch timer register > > handling), the guest is hang during kerne

Re: [PATCH] pinctrl: axp209: dereference pointer after it's been set

2017-12-13 Thread Mylene JOSSERAND
Hello Quentin, Le Wed, 13 Dec 2017 09:55:03 +0100, Quentin Schulz a écrit : > The number of GPIOs is gotten from a field within the structure > referenced in the of_device.data but it was actually read before it was > retrieved, thus it was dereferencing a null pointer. > > Set the number of GP

Re: [PATCH v2] spi: s3c64xx: add SPDX identifier

2017-12-13 Thread Philippe Ombredanne
Andi, On Wed, Dec 13, 2017 at 9:05 AM, Andi Shyti wrote: > Replace the original license statement with the SPDX identifier. > > Signed-off-by: Andi Shyti > --- > Hi, > > this second version unlike v1, where I tried to move the license > from GPLv2+ to GPLv2, preserves the original licese while a

Re: [PATCH] vfio: Simplify capability helper

2017-12-13 Thread Auger Eric
Hi Alex, On 12/12/17 20:59, Alex Williamson wrote: > The vfio_info_add_capability() helper requires the caller to pass a > capability ID, which it then uses to fill in header fields, assuming > hard coded versions. This makes for an awkward and rigid interface. > The only thing we want this helpe

Re: [PATCH v2 05/13] pinctrl: Add Microsemi Ocelot SoC driver

2017-12-13 Thread Philippe Ombredanne
On Wed, Dec 13, 2017 at 9:15 AM, Linus Walleij wrote: >> +// SPDX-License-Identifier: (GPL-2.0 OR MIT) > > Wow never saw that before. OK I guess. That's the new thing. Less legalese boilerplate, and more code for the better IMHO. You can check the doc patches from Thomas for details [1] [1] htt

Re: [PATCH v10 00/13] Introduce framework for SLIMbus device driver

2017-12-13 Thread Greg Kroah-Hartman
On Mon, Dec 11, 2017 at 11:42:54PM +, srinivas.kandaga...@linaro.org wrote: > From: Srinivas Kandagatla > > SLIMbus (Serial Low Power Interchip Media Bus) is a specification > developed by MIPI (Mobile Industry Processor Interface) alliance. > SLIMbus is a 2-wire implementation, which is used

Re: [PATCH v10 08/13] regmap: add SLIMbus support

2017-12-13 Thread Greg Kroah-Hartman
On Mon, Dec 11, 2017 at 11:43:02PM +, srinivas.kandaga...@linaro.org wrote: > From: Srinivas Kandagatla > > This patch adds support to read/write SLIMbus value elements. > Currently it only supports byte read/write. Adding this support in > regmap would give codec drivers more flexibility whe

Re: [PATCH v3 4/4] regulator: core: Balance coupled regulators voltages

2017-12-13 Thread Maciej Purski
On 12/12/2017 12:54 PM, Mark Brown wrote: On Thu, Dec 07, 2017 at 10:46:15AM +0100, Maciej Purski wrote: @@ -2447,10 +2482,9 @@ static int _regulator_is_enabled(struct regulator_dev *rdev) return rdev->desc->ops->is_enabled(rdev); } -static int _regulator_list_voltage(struct reg

[PATCH v2 0/2] mm: introduce MAP_FIXED_SAFE

2017-12-13 Thread Michal Hocko
Hi, I am resending with some minor updates based on Michael's review and ask for inclusion. There haven't been any fundamental objections for the RFC [1] nor the previous version [2]. The biggest discussion revolved around the naming. There were many suggestions flowing around MAP_REQUIRED, MAP_E

[BUG]kernel softlockup due to sidtab_search_context run for long time because of too many sidtab context node

2017-12-13 Thread yangjihong
Hello, I am doing stressing testing on 3.10 kernel(centos 7.4), to constantly starting numbers of docker ontainers with selinux enabled, and after about 2 days, the kernel softlockup panic: [] sched_show_task+0xb8/0x120 [] show_lock_info+0x20f/0x3a0 [] watchdog_timer_fn+0x1da/0x2f0 [] ? w

[PATCH] bluecard: Fix a possible sleep-in-atomic bug in bluecard_write_wakeup

2017-12-13 Thread Jia-Ju Bai
The driver may sleep in the interrupt handler. The function call path is: bluecard_interrupt (interrupt handler) bluecard_write_wakeup schedule_timeout --> may sleep To fix it, schedule_timeout is replaced with mdelay. This bug is found by my static analysis tool(DSAC) and checked by my cod

[PATCH 2/2] fs, elf: drop MAP_FIXED usage from elf_map

2017-12-13 Thread Michal Hocko
From: Michal Hocko Both load_elf_interp and load_elf_binary rely on elf_map to map segments on a controlled address and they use MAP_FIXED to enforce that. This is however dangerous thing prone to silent data corruption which can be even exploitable. Let's take CVE-2017-1000253 as an example. At

[PATCH 1/2] mm: introduce MAP_FIXED_SAFE

2017-12-13 Thread Michal Hocko
From: Michal Hocko MAP_FIXED is used quite often to enforce mapping at the particular range. The main problem of this flag is, however, that it is inherently dangerous because it unmaps existing mappings covered by the requested range. This can cause silent memory corruptions. Some of them even w

Re: [BUG] kaweth: a possible sleep-in-atomic bug in kaweth_start_xmit

2017-12-13 Thread Oliver Neukum
Am Mittwoch, den 13.12.2017, 16:57 +0800 schrieb Jia-Ju Bai: > According to drivers/net/usb/kaweth.c, the driver may sleep under a > spinlock. > The function call path is: > kaweth_start_xmit (acquire the spinlock) >kaweth_async_set_rx_mode > kaweth_control >kaweth_internal_contro

RE: [Intel-gfx] [PATCH] drm/i915: Remove unused IRQ chip data of HDMI LPE audio

2017-12-13 Thread Chen, Augustine
> -Original Message- > From: Takashi Iwai [mailto:ti...@suse.de] > Sent: Tuesday, December 12, 2017 5:45 PM > To: Chen, Augustine > Cc: Ville Syrjälä ; Anand, Jerome > ; Thomas Gleixner ; intel- > g...@lists.freedesktop.org; alsa-de...@alsa-project.org; Bossart, Pierre-louis > ; Ingo Mol

Re: [PATCH] KVM: arm/arm64: don't set vtimer->cnt_ctl in kvm_arch_timer_handler

2017-12-13 Thread Marc Zyngier
On 13/12/17 09:08, Auger Eric wrote: > Marc, > On 13/12/17 09:56, Marc Zyngier wrote: >> Hi Jia, >> >> On 13/12/17 07:00, Jia He wrote: >>> In our Armv8a server (qualcomm Amberwing, non VHE), after applying >>> Christoffer's timer optimizing patchset(Optimize arch timer register >>> handling), the

Re: [PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines

2017-12-13 Thread Philippe Ombredanne
Dhaval, On Wed, Dec 13, 2017 at 9:23 AM, Dhaval Shah wrote: > Resolved Block comments use * on subsequent lines checkpatch warnings. > Issue found by checkpatch. > > Signed-off-by: Dhaval Shah > --- > drivers/platform/x86/dell-rbtn.c | 26 +- > drivers/platform/x86/dell-

[PATCH 2/3] regmap: don't create the debugfs entries if locking is disabled

2017-12-13 Thread Bartosz Golaszewski
User space can initiate concurrent access to regmap over debugfs and, if the locking is disabled, we can't protect it. Don't create the debugfs entries at all in this case. Suggested-by: Lars-Peter Clausen Signed-off-by: Bartosz Golaszewski --- drivers/base/regmap/internal.h | 2 ++ drive

[PATCH 1/3] regmap: rename regmap_lock_unlock_empty() to regmap_lock_unlock_none()

2017-12-13 Thread Bartosz Golaszewski
Minor naming convention tweak. Suggested-by: Andy Shevchenko Signed-off-by: Bartosz Golaszewski --- drivers/base/regmap/regmap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/base/regmap/regmap.c b/drivers/base/regmap/regmap.c index 1da2a9fc40b0..16a66ba84aef 1

[PATCH 0/3] regmap: follow-up for ("regmap: allow to disable all locking mechanisms")

2017-12-13 Thread Bartosz Golaszewski
Hi Mark, this series is a follow-up for my previous patch. The first commit renames the empty locking routine introduced in it - I agree with Andy that it just looks better. The second patch disables the debugfs entries when locking is disabled as suggested by Lars-Peter. The last patch is optio

[PATCH 3/3] regmap: duplicate the name string stored in regmap

2017-12-13 Thread Bartosz Golaszewski
Currently we just copy over the pointer passed to regmap_init() in the regmap config struct. To be on the safe side: duplicate the string so that if an unaware user passes an address to a stack-allocated buffer, we won't crash. Signed-off-by: Bartosz Golaszewski --- drivers/base/regmap/regmap.c

[PATCH] USB: serial: ark3116.c: Remove unused TIOCSSERIAL case from ioctl

2017-12-13 Thread Mikhail Zaytsev
The patch removes unused TIOCSSERIAL case from ioctl. TIOCGSERIAL case moves to the get_serial_info() function. Some magic numbers moves to #define directives. Signed-off-by: Mikhail Zaytsev --- drivers/usb/serial/ark3116.c | 54 ++-- 1 file changed, 32

Re: [PATCH v2] misc: mic: Use memdup_user() as a cleanup

2017-12-13 Thread Greg KH
On Tue, Dec 12, 2017 at 02:22:41PM +0100, Gomonovych, Vasyl wrote: > Hi, > Thanks. > Should I prepare fix patch only for missed label? I have no context here at all, sorry...

[PATCH 2/2] mmap.2: MAP_FIXED updated documentation

2017-12-13 Thread Michal Hocko
From: John Hubbard -- Expand the documentation to discuss the hazards in enough detail to allow avoiding them. -- Mention the upcoming MAP_FIXED_SAFE flag. -- Enhance the alignment requirement slightly. CC: Michael Ellerman CC: Jann Horn CC: Matthew Wilcox CC: Michal Hock

Re: [PATCH] KVM: introduce kvm_arch_vcpu_async_ioctl

2017-12-13 Thread Paolo Bonzini
On 12/12/2017 19:25, Christoffer Dall wrote: > Reviewed-by: Christoffer Dall > > (although it'd be great if we can introduce the dummy inline to get rid > of the #ifdef'ery as well). Well, it's ARM and x86 that are the culprit so it's up to us. :) I'll add an #ifdef HAVE_KVM_VCPU_ASYNC_IOCTL to

[PATCH 1/2] mmap.2: document new MAP_FIXED_SAFE flag

2017-12-13 Thread Michal Hocko
From: Michal Hocko 4.16+ kernels offer a new MAP_FIXED_SAFE flag which allows the caller to atomicaly probe for a given address range. [wording heavily updated by John Hubbard ] Signed-off-by: Michal Hocko --- man2/mmap.2 | 22 ++ 1 file changed, 22 insertions(+) diff --gi

Re: [PATCH] nvmem: uniphier: change access unit from 32bit to 8bit

2017-12-13 Thread Kunihiko Hayashi
Hello Srinivas, This patch was already reviewed and tested for all UniPhier SoCs: https://patchwork.kernel.org/patch/10069557/ Do you have any comments, or would you please pick it up? Thank you, On Wed, 22 Nov 2017 14:14:59 +0900 Kunihiko Hayashi wrote: > The efuse on UniPhier allows 8bit a

Re: [PATCH 3/3] regmap: duplicate the name string stored in regmap

2017-12-13 Thread Lars-Peter Clausen
On 12/13/2017 10:28 AM, Bartosz Golaszewski wrote: > Currently we just copy over the pointer passed to regmap_init() in > the regmap config struct. To be on the safe side: duplicate the string > so that if an unaware user passes an address to a stack-allocated > buffer, we won't crash. > > Signed-

Re: [PATCH] KVM: arm/arm64: don't set vtimer->cnt_ctl in kvm_arch_timer_handler

2017-12-13 Thread Christoffer Dall
On Wed, Dec 13, 2017 at 10:27 AM, Marc Zyngier wrote: > On 13/12/17 09:08, Auger Eric wrote: >> Marc, >> On 13/12/17 09:56, Marc Zyngier wrote: >>> Hi Jia, >>> >>> On 13/12/17 07:00, Jia He wrote: In our Armv8a server (qualcomm Amberwing, non VHE), after applying Christoffer's timer opti

Re: [patch 1/2] mm, mmu_notifier: annotate mmu notifiers with blockable invalidate callbacks

2017-12-13 Thread Christian König
Am 12.12.2017 um 22:28 schrieb David Rientjes: On Tue, 12 Dec 2017, Dimitri Sivanich wrote: --- a/drivers/misc/sgi-gru/grutlbpurge.c +++ b/drivers/misc/sgi-gru/grutlbpurge.c @@ -298,6 +298,7 @@ struct gru_mm_struct *gru_register_mmu_notifier(void) return ERR_PTR(-ENOMEM)

[PATCH v2] KVM: introduce kvm_arch_vcpu_async_ioctl

2017-12-13 Thread Paolo Bonzini
After the vcpu_load/vcpu_put pushdown, the handling of asynchronous VCPU ioctl is already much clearer in that it is obvious that they bypass vcpu_load and vcpu_put. However, it is still not perfect in that the different state of the VCPU mutex is still hidden in the caller. Separate those ioctls

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