Re: [PATCH v6 0/5] clk: Add Aspeed clock driver

2017-12-05 Thread Joel Stanley
Hello clk maintainers, Has anyone had a chance to review these patches? On Tue, Nov 28, 2017 at 5:49 PM, Joel Stanley wrote: > This driver supports the ast2500, ast2400 (and derivative) BMC SoCs from > Aspeed. > > This is v6. See patches for detailed changelogs. > > v6: Added reviewed-bys > v5:

Re: [PATCH] [linux][master][v1] misc: Add Xilinx ZYNQMP VCU logicoreIP init driver

2017-12-05 Thread 'Greg KH'
On Wed, Dec 06, 2017 at 06:01:37AM +, Dhaval Rajeshbhai Shah wrote: > Hi Greg k-h, > > Thanks a lot for the review. > > Replies inline. As they should be, perhaps you need a better email client :) > > > +config XILINX_VCU > > + tristate "Xilinx VCU Init" > > + default n > > Th

Re: [PATCH V2] clk: fix a panic error caused by accessing NULL pointer

2017-12-05 Thread Chunyan Zhang
On 6 December 2017 at 07:28, Stephen Boyd wrote: > On 11/21, Chunyan Zhang wrote: >> From: Cai Li >> >> In some cases the clock parent would be set NULL when doing re-parent, >> it will cause a NULL pointer accessing if clk_set trace event is >> enabled. >> >> This patch sets the parent as "none"

Re: Multiple oom_reaper BUGs: unmap_page_range racing with exit_mmap

2017-12-05 Thread David Rientjes
On Wed, 6 Dec 2017, Tetsuo Handa wrote: > > > One way to solve the issue is to have two mm flags: one to indicate the > > > mm > > > is entering unmap_vmas(): set the flag, do down_write(&mm->mmap_sem); > > > up_write(&mm->mmap_sem), then unmap_vmas(). The oom reaper needs this > > > flag cle

Re: [PATCH 4.4 02/96] s390/runtime instrumention: fix possible memory corruption

2017-12-05 Thread Greg Kroah-Hartman
On Tue, Dec 05, 2017 at 07:15:34PM +0100, Heiko Carstens wrote: > On Tue, Dec 05, 2017 at 06:08:47PM +0100, Greg Kroah-Hartman wrote: > > On Tue, Dec 05, 2017 at 05:02:32PM +, Ben Hutchings wrote: > > > On Tue, 2017-11-28 at 11:22 +0100, Greg Kroah-Hartman wrote: > > > > 4.4-stable review patch

Re: [PATCH] clk: fix a panic error caused by accessing NULL pointer

2017-12-05 Thread Chunyan Zhang
On 6 December 2017 at 07:28, Stephen Boyd wrote: > On 11/21, Chunyan Zhang wrote: >> On 21 November 2017 at 16:57, Chunyan Zhang wrote: >> > On 21 November 2017 at 03:12, Stephen Boyd wrote: >> >> On 11/20, Chunyan Zhang wrote: >> >>> From: Cai Li >> >>> >> >>> In some cases the clock parent wo

Re: [PATCH 0/2] mm: introduce MAP_FIXED_SAFE

2017-12-05 Thread Florian Weimer
On 12/06/2017 08:33 AM, John Hubbard wrote: In that case, maybe: MAP_EXACT ? ...because that's the characteristic behavior. Is that true? mmap still silently rounding up the length to the page size, I assume, so even that name is misleading. Thanks, Florian

Re: [PATCH] doc: convert printk-formats.txt to rst

2017-12-05 Thread Joe Perches
On Wed, 2017-12-06 at 08:11 +0100, Markus Heiser wrote: > > Am 06.12.2017 um 02:45 schrieb Tobin C. Harding : > > Documentation/printk-formats.txt is a candidate for conversion to > > ReStructuredText format. Some effort has already been made to do this > > conversion even thought the suffix is cur

Re: [PATCH 0/2] mm: introduce MAP_FIXED_SAFE

2017-12-05 Thread John Hubbard
On 12/05/2017 11:03 PM, Matthew Wilcox wrote: > On Tue, Dec 05, 2017 at 08:54:35PM -0800, Matthew Wilcox wrote: >> On Wed, Dec 06, 2017 at 03:51:44PM +1100, Michael Ellerman wrote: >>> Cyril Hrubis writes: >>> Hi! >> MAP_FIXED_UNIQUE >> MAP_FIXED_ONCE >> MAP_FIXED_FRESH >

Re: [PATCH 0/2] mm: introduce MAP_FIXED_SAFE

2017-12-05 Thread Rasmus Villemoes
On 2017-12-06 05:50, Michael Ellerman wrote: > Michal Hocko writes: > >> On Wed 29-11-17 14:25:36, Kees Cook wrote: >> It is safe in a sense it doesn't perform any address space dangerous >> operations. mmap is _inherently_ about the address space so the context >> should be kind of clear. > > S

[PATCH v6 3/6] kernel/reboot.c: export pm_power_off_prepare

2017-12-05 Thread Oleksij Rempel
Export pm_power_off_prepare. It is needed to implement power off on Freescale/NXP iMX6 based boards with external power management integrated circuit (PMIC). Signed-off-by: Oleksij Rempel --- kernel/reboot.c | 1 + 1 file changed, 1 insertion(+) diff --git a/kernel/reboot.c b/kernel/reboot.c in

[PATCH v6 2/6] ARM: imx6: register pm_power_off handler if "fsl,pmic-stby-poweroff" is set

2017-12-05 Thread Oleksij Rempel
One of the Freescale recommended sequences for power off with external PMIC is the following: ... 3. SoC is programming PMIC for power off when standby is asserted. 4. In CCM STOP mode, Standby is asserted, PMIC gates SoC supplies. See: http://www.nxp.com/assets/documents/data/en/reference-manua

[PATCH RESEND v6 0/6] provide power off support for iMX6 with external PMIC

2017-12-05 Thread Oleksij Rempel
2017.12.06: Adding Linus. Probably there is no maintainer for this patch set. No changes are made, tested on v4.15-rc1. 2017.10.27: Last version of this patch set was send at 20 Jun 2017, this is a rebase against kernel v4.14-rc6. Probably this set got lost. If I forgot to address some comments,

[PATCH v6 1/6] ARM: imx6q: provide documentation for new fsl,pmic-stby-poweroff property

2017-12-05 Thread Oleksij Rempel
Signed-off-by: Oleksij Rempel Acked-by: Rob Herring --- Documentation/devicetree/bindings/clock/imx6q-clock.txt | 8 1 file changed, 8 insertions(+) diff --git a/Documentation/devicetree/bindings/clock/imx6q-clock.txt b/Documentation/devicetree/bindings/clock/imx6q-clock.txt index aa0

[PATCH v6 5/6] regulator: pfuze100-regulator: provide pm_power_off_prepare handler

2017-12-05 Thread Oleksij Rempel
On some boards the SoC can use one pin "PMIC_STBY_REQ" to notify th PMIC about state changes. In this case internal state of PMIC must be preconfigured for upcomming state change. It works fine with the current regulator framework, except with the power-off case. This patch is providing an optiona

[PATCH v6 4/6] regulator: pfuze100: add fsl,pmic-stby-poweroff property

2017-12-05 Thread Oleksij Rempel
Document the new optional "fsl,pmic-stby-poweroff" property. Signed-off-by: Oleksij Rempel Acked-by: Rob Herring --- Documentation/devicetree/bindings/regulator/pfuze100.txt | 7 +++ 1 file changed, 7 insertions(+) diff --git a/Documentation/devicetree/bindings/regulator/pfuze100.txt b/Do

[PATCH v6 6/6] ARM: dts: imx6: RIoTboard provide standby on power off option

2017-12-05 Thread Oleksij Rempel
This board, as well as some other boards with i.MX6 and a PMIC, uses a "PMIC_STBY_REQ" line to notify the PMIC about a state change. The PMIC is programmed for a specific state change before triggering the line. In this case, PMIC_STBY_REQ can be used for stand by, sleep and power off modes. Signe

Re: [PATCH] doc: convert printk-formats.txt to rst

2017-12-05 Thread Markus Heiser
> Am 06.12.2017 um 02:45 schrieb Tobin C. Harding : > > Documentation/printk-formats.txt is a candidate for conversion to > ReStructuredText format. Some effort has already been made to do this > conversion even thought the suffix is currently .txt > > Changes required to complete conversion >

Re: [PATCH v4 1/3] drm/bridge/synopsys: dsi: stop clobbering drvdata

2017-12-05 Thread Nickey Yang
Hi Brian, On 2017年12月06日 02:56, Brian Norris wrote: Hi Nickey, On Tue, Dec 05, 2017 at 05:14:11PM +0800, Nickey Yang wrote: On 2017年12月01日 18:07, Philippe CORNU wrote: On 12/01/2017 10:11 AM, Nickey Yang wrote: On 2017年12月01日 16:32, Philippe CORNU wrote: I am sorry to say that but you can

[PATCH] sched/wait: fix add_wait_queue() behavior change

2017-12-05 Thread Omar Sandoval
From: Omar Sandoval Commit 50816c48997a ("sched/wait: Standardize internal naming of wait-queue entries") changed the behavior of add_wait_queue() from inserting the wait entry at the head of the wait queue to the tail of the wait queue. That commit was a cleanup and didn't mention any functional

Re: rsi_91x: Failed to read status register on failed authentication

2017-12-05 Thread Amitkumar Karwar
On Tue, Dec 5, 2017 at 9:41 PM, Alexey Brodkin wrote: > Hi Amit, > > I'm seeing quite a strange behavior of RedPine module. > It connects perfectly fine to one of access points but fails > to connect to another. > > Moreover after that failure RSI driver starts to flood me with > messages saying:

RE: [Bug fix] octeon-i2c driver updates

2017-12-05 Thread Zhang, Sean C. (NSB - CN/Hangzhou)
Hi David, In fact, in current octeon_i2c_probe(), there has the bus reset unconditionally, It is reset by octeon_i2c_init_lowlevel() unconditionally. In my patch, I replace it by octeon_i2c_recovery(), which will be executed under three conditions. + /* +* Try to recover bus in thr

[PATCH] Staging: rtl8192u: Fix no spaces around '+'

2017-12-05 Thread Akash Kumar
Added spaces around '+'. Warning found using checkpatch.pl --- drivers/staging/rtl8192u/ieee80211/dot11d.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192u/ieee80211/dot11d.c b/drivers/staging/rtl8192u/ieee80211/dot11d.c index 64b13a5..ba284bf 100644 --

[PATCH] MIPS: Octeon: Fix logging messages with spurious periods after newlines

2017-12-05 Thread Joe Perches
Using a period after a newline causes bad output. Signed-off-by: Joe Perches --- arch/mips/cavium-octeon/octeon-irq.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/arch/mips/cavium-octeon/octeon-irq.c b/arch/mips/cavium-octeon/octeon-irq.c index 5b3a3f6a9ad3..b9

Re: [PATCH 0/2] mm: introduce MAP_FIXED_SAFE

2017-12-05 Thread Matthew Wilcox
On Tue, Dec 05, 2017 at 08:54:35PM -0800, Matthew Wilcox wrote: > On Wed, Dec 06, 2017 at 03:51:44PM +1100, Michael Ellerman wrote: > > Cyril Hrubis writes: > > > > > Hi! > > >> > MAP_FIXED_UNIQUE > > >> > MAP_FIXED_ONCE > > >> > MAP_FIXED_FRESH > > >> > > >> Well, I can open a poll for the best

Re: [PATCH] ARM: dts: introduce the sama5d2 ptc ek board

2017-12-05 Thread Ludovic Desroches
On Tue, Dec 05, 2017 at 07:01:41PM +0200, Baruch Siach wrote: > Hi Ludovic, > > On Tue, Dec 05, 2017 at 03:23:12PM +0100, Ludovic Desroches wrote: > > Add the official SAMA5D2 Peripheral Touch Controller Evaluation > > Kit board. > > > > Signed-off-by: Ludovic Desroches > > --- > > [...] > > >

[PATCH V2] ARM: dts: introduce the sama5d2 ptc ek board

2017-12-05 Thread Ludovic Desroches
Add the official SAMA5D2 Peripheral Touch Controller Evaluation Kit board. Signed-off-by: Ludovic Desroches --- Changes: - v2: - remove memory node - use SPDX-License-Identifier arch/arm/boot/dts/Makefile| 1 + arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts | 402 ++

Re: [PATCH] usb: dwc2: add optional usb ecc reset bit

2017-12-05 Thread John Youn
On 11/01/2017 08:35 AM, Dinh Nguyen wrote: > The dwc2 USB controller in Stratix10 has an additional ECC reset bit that > needs to get de-asserted in order for the controller to work properly. > > Signed-off-by: Dinh Nguyen > --- > drivers/usb/dwc2/core.h | 1 + > drivers/usb/dwc2/platform.c

A kernel warning will be triggered,during copy a big file into zstd compressed btrfs filesystem

2017-12-05 Thread guoxuenan
Hi all, I have found a kernel warning of btrfs ,during run my shell script in qemu qemu virtual machine. Linux mainline version Linux 4.15-rc2 (ae64f9bd) The script try to copy a big file(15G) into btrfs filesystem,when I login virtual machine,then run the script,A kernel warning will be

Re: [PATCH 4.14 00/95] 4.14.4-stable review

2017-12-05 Thread Greg Kroah-Hartman
On Wed, Dec 06, 2017 at 07:49:49AM +0100, Greg Kroah-Hartman wrote: > Don't fall down the trap of running code for the sake of running code > (i.e. like that web site that starts with a P) that doesn't actually > test anything that actually matters. I forgot to add: Especially when there are alre

Re: [PATCH 4.14 00/95] 4.14.4-stable review

2017-12-05 Thread Greg Kroah-Hartman
On Tue, Dec 05, 2017 at 03:45:07PM -0600, Tom Gall wrote: > > > > On Dec 5, 2017, at 12:24 AM, Greg Kroah-Hartman > > wrote: > > > > On Mon, Dec 04, 2017 at 03:12:45PM -0600, Tom Gall wrote: > >> > >> > >>> On Dec 4, 2017, at 9:59 AM, Greg Kroah-Hartman > >>> wrote: > >>> > >>> This is th

Re: [PATCH] bfa: fix access to bfad_im_port_s

2017-12-05 Thread James Bottomley
On Tue, 2017-11-28 at 16:26 +0100, Johannes Thumshirn wrote: > Commit 'cd21c605b2cf ("scsi: fc: provide fc_bsg_to_shost() helper")' > changed access to bfa's 'struct bfad_im_port_s' by using shost_priv() > instead of shost->hostdata[0]. > > This lead to crashes like in the following back-trace: >

[PATCH] usb: xhci: fix TDS for MTK xHCI1.1

2017-12-05 Thread Chunfeng Yun
For MTK's xHCI 1.0 or latter, TD size is the number of max packet sized packets remaining in the TD, not including this TRB (following spec). For MTK's xHCI 0.96 and older, TD size is the number of max packet sized packets remaining in the TD, including this TRB (not following spec). Signed-off-b

[PATCH] xen-netback: Fix logging message with spurious period after newline

2017-12-05 Thread Joe Perches
Using a period after a newline causes bad output. Signed-off-by: Joe Perches --- drivers/net/xen-netback/interface.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/xen-netback/interface.c b/drivers/net/xen-netback/interface.c index d6dff347f896..78ebe494fef0 100

Re: [PATCH] usb: core: Fix logging messages with spurious periods after newlines

2017-12-05 Thread Joe Perches
On Wed, 2017-12-06 at 07:27 +0100, Greg Kroah-Hartman wrote: > On Tue, Dec 05, 2017 at 10:22:05PM -0800, Joe Perches wrote: > > Using a period after a newline causes bad output. > > Nice catch, how did you find that? $ git grep '\\n\."'

Re: [PATCH] efi: move some sysfs files to be read-only by root

2017-12-05 Thread Greg Kroah-Hartman
On Wed, Dec 06, 2017 at 07:50:41AM +1100, Tobin C. Harding wrote: > On Tue, Dec 05, 2017 at 11:13:43AM +0100, Greg Kroah-Hartman wrote: > > Thanks to the scripts/leaking_addresses.pl script, it was found that > > some EFI values should not be readable by non-root users. > > > > So make them root-o

Re: [PATCH] driver-core: platform: Avoid to return IRQ 0 in platform_get_irq()

2017-12-05 Thread Arvind Yadav
Hi Dmitry, On Tuesday 05 December 2017 11:50 PM, Dmitry Torokhov wrote: On Tue, Dec 05, 2017 at 11:41:52PM +0530, Arvind Yadav wrote: Function platform_get_irq() can return 0. Which means NO_IRQ. So this change will not allow to return 0. This change is help to use platform_get_irq() without

Re: [PATCH v6 2/2] staging: ion: create one device entry per heap

2017-12-05 Thread Greg KH
On Tue, Dec 05, 2017 at 03:01:42PM -0800, Laura Abbott wrote: > On 12/02/2017 07:53 AM, Greg KH wrote: > > > This is one of the item in the TODO list before been able to unstage ION > > > which is my real need. > > Why does it matter where in the tree this code is? Don't go adding new > > things t

Re: [PATCH] usb: core: Fix logging messages with spurious periods after newlines

2017-12-05 Thread Greg Kroah-Hartman
On Tue, Dec 05, 2017 at 10:22:05PM -0800, Joe Perches wrote: > Using a period after a newline causes bad output. Nice catch, how did you find that? thanks, greg k-h

Re: In kernel power management domain_pm created for async schedules

2017-12-05 Thread gre...@linuxfoundation.org
On Wed, Dec 06, 2017 at 05:38:31AM +, Vikas Bansal wrote: > If there is a driver in system which starts creating async schedules just > after resume (Same as our case, in which we faced issue). Then > async_synchronize_full API in PM cores starts waiting for completion of async > schedules c

[PATCH] usb: core: Fix logging messages with spurious periods after newlines

2017-12-05 Thread Joe Perches
Using a period after a newline causes bad output. Miscellanea: o Coalesce formats too Signed-off-by: Joe Perches --- drivers/usb/core/driver.c | 8 drivers/usb/core/hub.c | 17 +++-- drivers/usb/core/message.c | 6 +++--- 3 files changed, 14 insertions(+), 17 deleti

Re: [PATCH] usb: dwc2: Fix TxFIFOn sizes and total TxFIFO size issues

2017-12-05 Thread John Youn
On 11/30/2017 12:16 AM, Minas Harutyunyan wrote: > In host mode reading from DPTXSIZn returning invalid value in > dwc2_check_param_tx_fifo_sizes function. > > In total TxFIFO size calculations unnecessarily reducing by ep_info. > hw->total_fifo_size can be fully allocated for FIFO's. > > Added num

drivers/nvme/target/fcloop.c:1080:9: warning: 'tport' may be used uninitialized in this function

2017-12-05 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 328b4ed93b69a6f2083d52f31a240a09e5de386a commit: fddc9923c6d41de9fe7b1f323a3cece53e046c88 nvme-fcloop: fix port deletes and callbacks date: 2 months ago config: i386-randconfig-h1-12061225 (attached as .co

Re: [PATCH v3 1/2] usb: dwc2: host: Don't retry NAKed transactions right away

2017-12-05 Thread John Youn
On 12/05/2017 08:18 AM, Stefan Wahren wrote: > Hi Felipe, > Hi John, > > > Am 30.10.2017 um 18:08 schrieb Douglas Anderson: >> On rk3288-veyron devices on Chrome OS it was found that plugging in an >> Arduino-based USB device could cause the system to lockup, especially >> if the CPU Frequency was

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

2017-12-05 Thread Kuninori Morimoto
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. In academic generalism, VCO should be maximum to be more small jitter. In high frequency clock, jitter will be large impact.

Re: [PATCH 5/8] ASoC: uniphier: add support for UniPhier AIO driver

2017-12-05 Thread Katsuhiro Suzuki
Hello, > -Original Message- > From: Mark Brown [mailto:broo...@kernel.org] > Sent: Tuesday, December 5, 2017 9:15 PM > To: Suzuki, Katsuhiro/鈴木 勝博 > Cc: alsa-de...@alsa-project.org; Rob Herring ; > devicet...@vger.kernel.org; Yamada, Masahiro/山田 真弘 > ; Masami Hiramatsu > ; Jassi Brar ; >

Re: Sending 802.1Q packets using AF_PACKET socket on filtered bridge forwards with wrong MAC addresses

2017-12-05 Thread Toshiaki Makita
Hi, (CC: Vlad) On 2017/11/30 7:01, Brandon Carpenter wrote: > I narrowed the search to a memmove() called from > skb_reorder_vlan_header() in net/core/skbuff.c. > >> memmove(skb->data - ETH_HLEN, skb->data - skb->mac_len - VLAN_HLEN, >>2 * ETH_ALEN); > > Calling skb_reset_mac_len() a

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

2017-12-05 Thread Kuninori Morimoto
Hi I noticed 1 typo, 1 bug on this patch. I will post v2 patch > 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. > In academic generalism, VCO should be maximum to

RE: [PATCH] [linux][master][v1] misc: Add Xilinx ZYNQMP VCU logicoreIP init driver

2017-12-05 Thread Dhaval Rajeshbhai Shah
Hi Greg k-h, Thanks a lot for the review. Replies inline. -Original Message- From: Greg KH [mailto:gre...@linuxfoundation.org] Sent: Tuesday, December 05, 2017 5:30 AM To: Dhaval Rajeshbhai Shah Cc: a...@arndb.de; linux-kernel@vger.kernel.org; michal.si...@xilinx.com; Hyun Kwon ; Dhav

[PATCH PTI] Fixup "x86/entry/64: Use a per-CPU trampoline stack for IDT entries"

2017-12-05 Thread Andy Lutomirski
Add to commit message: Using a trampoline stack would obnoxious for Xen PV because Xen PV enters entry_SYSCALL_64_after_hwframe on the stack indicated by sp0. This could be fixed, but I think it's nice to ensure the entry code can still work without a trampoline stack. So this patch doesn't use t

Re: [alsa-devel] [PATCH v4 06/15] soundwire: Add IO transfer

2017-12-05 Thread Vinod Koul
On Tue, Dec 05, 2017 at 08:48:03AM -0600, Pierre-Louis Bossart wrote: > On 12/5/17 7:43 AM, Pierre-Louis Bossart wrote: > >On 12/5/17 12:31 AM, Vinod Koul wrote: > >>On Sun, Dec 03, 2017 at 09:01:41PM -0600, Pierre-Louis Bossart wrote: > >>+static inline int do_transfer(struct sdw_bus *bus, st

Re: [PATCH v11 0/6] Add support for Qualcomm A53 CPU clock

2017-12-05 Thread Amit Kucheria
On Tue, Dec 5, 2017 at 9:16 PM, Georgi Djakov wrote: > This patchset adds support for the A53 CPU clock on MSM8916 platforms > and allows scaling of the CPU frequency on msm8916 based platforms. Though it currently needs some additional patches (that'll follow soon), FWIW, Tested-by: Amit Kucheri

Re: [PATCH v2 3/3] net: macb: change GFP_ATOMIC to GFP_KERNEL

2017-12-05 Thread Julia Lawall
On Tue, 5 Dec 2017, Julia Cartwright wrote: > Now that the rx_fs_lock is no longer held across allocation, it's safe > to use GFP_KERNEL for allocating new entries. > > This reverts commit 81da3bf6e3f88 ("net: macb: change GFP_KERNEL to > GFP_ATOMIC"). > > Cc: Julia Lawall > Signed-off-by: Juli

In kernel power management domain_pm created for async schedules

2017-12-05 Thread Vikas Bansal
If there is a driver in system which starts creating async schedules just after resume (Same as our case, in which we faced issue). Then async_synchronize_full API in PM cores starts waiting for completion of async schedules created by that driver (Even though those are in a domain). Because of

RE: [PATCH] [linux][master][v1] misc: Add Xilinx ZYNQMP VCU logicoreIP init driver

2017-12-05 Thread Dhaval Rajeshbhai Shah
Hi Arnd, Replies inline. -Original Message- From: arndbergm...@gmail.com [mailto:arndbergm...@gmail.com] On Behalf Of Arnd Bergmann Sent: Tuesday, December 05, 2017 5:32 AM To: Dhaval Rajeshbhai Shah Cc: gregkh ; Linux Kernel Mailing List ; Michal Simek ; Hyun Kwon Subject: Re: [PATC

Re: [PATCH v3 08/21] fpga: add Intel FPGA DFL PCIe device

2017-12-05 Thread Wu Hao
On Tue, Dec 05, 2017 at 11:00:22AM -0600, Alan Tull wrote: > On Mon, Dec 4, 2017 at 9:33 PM, Wu Hao wrote: > > On Mon, Dec 04, 2017 at 01:46:59PM -0600, Alan Tull wrote: > >> On Mon, Nov 27, 2017 at 9:15 PM, Wu Hao wrote: > >> > On Mon, Nov 27, 2017 at 10:28:04AM +, David Laight wrote: > >> >

Re: [PATCH v6 2/2] media: i2c: Add the ov7740 image sensor driver

2017-12-05 Thread Yang, Wenyou
Hi Sakari, On 2017/12/5 18:45, Sakari Ailus wrote: Hi Wenyou, On Mon, Dec 04, 2017 at 02:58:58PM +0800, Wenyou Yang wrote: The ov7740 (color) image sensor is a high performance VGA CMOS image snesor, which supports for output formats: RAW RGB and YUV and image sizes: VGA, and QVGA, CIF and an

Re: possible deadlock in generic_file_write_iter (2)

2017-12-05 Thread Byungchul Park
On Tue, Dec 05, 2017 at 10:41:50AM +0100, Jan Kara wrote: > > Hello Byungchul, > > On Tue 05-12-17 13:58:09, Byungchul Park wrote: > > On 12/4/2017 5:33 PM, Jan Kara wrote: > > >adding Peter and Byungchul to CC since the lockdep report just looks > > >strange and cross-release seems to be involve

[PATCH] trace-cmd:show: be picky on user argument 'cpu'

2017-12-05 Thread Federico Vaga
Before this patch the following command worked without errors: trace-cmd show -c b This command opens the CPU0 files. From `strace(1)`: openat(AT_FDCWD, "/sys/kernel/tracing/per_cpu/cpu0/trace", O_RDONLY) = 3 Signed-off-by: Federico Vaga --- trace-show.c | 9 - 1 file changed,

Re: [PATCH 1/2] mm: introduce MAP_FIXED_SAFE

2017-12-05 Thread Michael Ellerman
Hi Michal, Some comments below. Michal Hocko writes: > 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 >

Re: possible deadlock in generic_file_write_iter (2)

2017-12-05 Thread Byungchul Park
On Tue, Dec 05, 2017 at 10:19:07AM +0100, Dmitry Vyukov wrote: > On Tue, Dec 5, 2017 at 5:58 AM, Byungchul Park wrote: > > On 12/4/2017 5:33 PM, Jan Kara wrote: > >> > >> Hello, > >> > >> adding Peter and Byungchul to CC since the lockdep report just looks > >> strange and cross-release seems to b

Re: [PATCH 0/2] mm: introduce MAP_FIXED_SAFE

2017-12-05 Thread Matthew Wilcox
On Wed, Dec 06, 2017 at 03:51:44PM +1100, Michael Ellerman wrote: > Cyril Hrubis writes: > > > Hi! > >> > MAP_FIXED_UNIQUE > >> > MAP_FIXED_ONCE > >> > MAP_FIXED_FRESH > >> > >> Well, I can open a poll for the best name, but none of those you are > >> proposing sound much better to me. Yeah, nam

Re: [PATCH v4 72/73] xfs: Convert mru cache to XArray

2017-12-05 Thread Matthew Wilcox
On Tue, Dec 05, 2017 at 08:45:49PM -0800, Matthew Wilcox wrote: > The dquot code is just going to have to live with the fact that there's > additional locking going on that it doesn't need. I'm open to getting > rid of the irqsafety, but we can't give up the spinlock protection > without giving up

Re: [PATCH 0/2] mm: introduce MAP_FIXED_SAFE

2017-12-05 Thread Michael Ellerman
Cyril Hrubis writes: > Hi! >> > MAP_FIXED_UNIQUE >> > MAP_FIXED_ONCE >> > MAP_FIXED_FRESH >> >> Well, I can open a poll for the best name, but none of those you are >> proposing sound much better to me. Yeah, naming sucks... > > Given that MAP_FIXED replaces the previous mapping MAP_FIXED_NOREPL

Re: [PATCH 0/2] mm: introduce MAP_FIXED_SAFE

2017-12-05 Thread Michael Ellerman
Michal Hocko writes: > On Wed 29-11-17 14:25:36, Kees Cook wrote: >> On Wed, Nov 29, 2017 at 6:42 AM, Michal Hocko wrote: >> > The first patch introduced MAP_FIXED_SAFE which enforces the given >> > address but unlike MAP_FIXED it fails with ENOMEM if the given range >> > conflicts with an exist

Re: [PATCH v4 72/73] xfs: Convert mru cache to XArray

2017-12-05 Thread Matthew Wilcox
On Wed, Dec 06, 2017 at 02:14:56PM +1100, Dave Chinner wrote: > > The other conversions use the normal API instead of the advanced API, so > > all of this gets hidden away. For example, the inode cache does this: > > Ah, OK, that's not obvious from the code changes. :/ Yeah, it's a lot easier to

Re: [PATCH V11 4/5] vsprintf: add printk specifier %px

2017-12-05 Thread Linus Torvalds
On Tue, Dec 5, 2017 at 8:19 PM, Alexey Dobriyan wrote: > Because people who introduce this stupid %p hashing can't be bothered > to actually audit users: I don't see you having offered to audit the 12k+ cases, did you? So maybe it's easier to just change them all, and then the handful that break

Re: [PATCHv4 5/6] symbol lookup: introduce dereference_symbol_descriptor()

2017-12-05 Thread Sergey Senozhatsky
Hello, so we got a number of build-error reports [somehow I thought 0day has compile tested the patches already; well, I was wrong] basically on congifs that have no KALLSYMS. Petr, can we replace 0006 with the following patch? 8<--- --- --- From: Sergey Senozhatsky Subject: [PATCH] s

Re: [PATCH v4 3/3] powerpc/modules: Improve restore_r2() error message

2017-12-05 Thread Michael Ellerman
Kamalesh Babulal writes: > From: Josh Poimboeuf > > Print the function address associated with the restore_r2() error to > make it easier to debug the problem. > > Also clarify the wording a bit. > > Before: > > module_64: patch_foo: Expect noop after relocate, got 3c82 > > After: > > mo

Re: linux-next: build failure after merge of the printk tree

2017-12-05 Thread Sergey Senozhatsky
Hello, On (12/06/17 14:23), Stephen Rothwell wrote: > Hi Petr, > > After merging the printk tree, today's linux-next build (x86_64 > allnoconfig) failed like this: > > lib/vsprintf.o: In function `pointer': > vsprintf.c:(.text+0x2260): undefined reference to > `dereference_symbol_descriptor' >

Re: [PATCH V11 4/5] vsprintf: add printk specifier %px

2017-12-05 Thread Alexey Dobriyan
> more %p grepping [filtering out all `%ps %pf %pb' variants] gives > a huge number of print outs that potentially can be broken now Because people who introduce this stupid %p hashing can't be bothered to actually audit users: static int show_timer(struct seq_file *m, void *v) {

linux-next: Tree for Dec 6

2017-12-05 Thread Stephen Rothwell
Hi all, Changes since 20171205: Removed trees: tile (orphaned) drm-amd-dc (finished with) The printk tree gained a build failure for which I reverted a commit. The drm-misc tree gained a build failure due to an interaction with the drm-intel tree, so I applied a merge fix up patch

RE: [PATCH v6 2/3] dmaeninge: xilinx_dma: Fix bug in multiple frame stores scenario in vdma

2017-12-05 Thread Appana Durga Kedareswara Rao
Hi Mike Looijmans, Thanks for the review... Sorry for the long delay in the reply... Please find comments inline... >On 14-01-17 06:35, Kedareswara rao Appana wrote: >>  When VDMA is configured for more than one frame in the h/w. >>  For example h/w is configured for n number of

Re: [PATCH v3] scripts: leaking_addresses: add support for 32-bit kernel addresses

2017-12-05 Thread Tobin C. Harding
On Tue, Dec 05, 2017 at 11:56:44AM +0530, kaiwan.billimo...@gmail.com wrote: > Currently, leaking_addresses.pl only supports scanning 64 bit > architectures. This is due to how the regular expressions are formed. We > can do better than this. 32 architectures can be supported if we take > into cons

[PATCH 1/3] mmc: sdhci-pci-o2micro: Add hardware tuning for eMMC

2017-12-05 Thread LinuxPatchCommit
For O2micro/Bayhubtech SD Host DeviceID 8620, eMMC HS200 mode is working at 1.8v and it uses hardware tuning. The hardware tuning only needs to send one tuning command instead of multiple tuning commands with software tuning. Signed-off-by: ernest.zhang --- drivers/mmc/host/sdhci-pci-o2micro.c |

[PATCH v3 6/6] iommu/vt-d: Add debugfs support for Intel IOMMU Interrupt remapping

2017-12-05 Thread Sohil Mehta
Debugfs extension for Intel IOMMU to dump Interrupt remapping table entries for Interrupt remapping and Interrupt posting. The file /sys/kernel/debug/intel_iommu/ir_translation_struct provides detailed information, such as Index, Source Id, Destination Id, Vector and the raw values for entries wit

[PATCH v3 2/6] iommu/vt-d: Add Intel IOMMU debugfs to show context internals

2017-12-05 Thread Sohil Mehta
From: Gayatri Kammela IOMMU internals states such as root and context can be exported to the userspace. Example of such dump in Kabylake: root@OTC-KBLH-01:~# cat /sys/kernel/debug/intel_iommu/dmar_translation_struct IOMMU dmar2:Root Table Addr:4558a3000 Root tbl entries: Bus 0 L: 4558aa00

[PATCH v3 4/6] iommu/vt-d: Add debugfs extension to show register contents

2017-12-05 Thread Sohil Mehta
From: Gayatri Kammela Debugfs extension to dump all the register contents for each IOMMU device to the user space via debugfs. example: root@OTC-KBLH-01:~# cat /sys/kernel/debug/intel_iommu/iommu_regset DMAR: dmar1: reg_base_addr fed9 Name Offset Contents VER 0x00

[PATCH v3 0/6] Intel IOMMU debugfs support

2017-12-05 Thread Sohil Mehta
Hi all, This series aims to add debugfs support for Intel IOMMU. It exposes IOMMU registers, internal context and dumps individual table entries to help debug Intel IOMMUs. The first patch does the ground work for the following patches by creating a new Kconfig option - INTEL_IOMMU_DEBUG. It also

[PATCH v3 3/6] iommu/vt-d: Add Intel IOMMU debugfs to show extended context internals

2017-12-05 Thread Sohil Mehta
From: Gayatri Kammela Debugfs extension to dump internals such as extended context table entries for each IOMMU to the userspace. root@OTC-KBLH-01:~# cat /sys/kernel/debug/intel_iommu/dmar_translation_struct IOMMU dmar1: Extended Root Table Addr:4558a1800 Extended Root tbl entries: Bus 0 L: 455

[PATCH v3 5/6] iommu/vt-d: Add debugfs extension to show Pasid table contents

2017-12-05 Thread Sohil Mehta
From: Gayatri Kammela Debugfs extension to dump the internals such as pasid table entries for each IOMMU to the userspace. Example of such dump in Kabylake: root@OTC-KBLH-01:~# cat /sys/kernel/debug/intel_iommu/dmar_translation_struct IOMMU dmar0: Extended Root Table Addr:4310c4800 Extended Ro

[PATCH v3 1/6] iommu/vt-d: Add debugfs support for Intel IOMMU internals

2017-12-05 Thread Sohil Mehta
From: Gayatri Kammela Enable Intel IOMMU debug to export Intel IOMMU internals in debugfs Cc: Sohil Mehta Cc: Fenghua Yu Cc: Ashok Raj Signed-off-by: Jacob Pan Signed-off-by: Gayatri Kammela --- v3: No change v2: No change drivers/iommu/Kconfig | 10 ++ drivers/iommu/intel

[PATCH v2 3/3] checkpatch: warn for use of %px

2017-12-05 Thread Tobin C. Harding
Usage of the new %px specifier potentially leaks sensitive inforamtion. Printing kernel addresses exposes the kernel layout in memory, this is potentially exploitable. We have tools in the kernel to help us do the right thing. We can have checkpatch warn developers of potential dangers of using %px

[PATCH v2 2/3] checkpatch: add sub routine get_stat_here()

2017-12-05 Thread Tobin C. Harding
checkpatch currently contains duplicate code. We can define a sub routine and call that instead. This reduces code duplication and line count. Add subroutine get_stat_here() Signed-off-by: Tobin C. Harding --- scripts/checkpatch.pl | 54 +++ 1 fil

[PATCH v2 1/3] checkpatch: add sub routine get_stat_real()

2017-12-05 Thread Tobin C. Harding
checkpatch currently contains duplicate code. We can define a sub routine and call that instead. This reduces code duplication and line count. Add subroutine get_stat_real() Signed-off-by: Tobin C. Harding --- scripts/checkpatch.pl | 31 +++ 1 file changed, 15 insert

[PATCH v2 0/3] checkpatch: warn for use of %px

2017-12-05 Thread Tobin C. Harding
Patch 1 and 2 are cleanup patches. Each adds a new subroutine to remove duplicate code. Patch 3: Make checkpatch emit a warning for usage of specifier %px. The initial idea to emit the warning was given on LKML by Andrew Morton, v1 requested permission to use 'Suggested-by' tag. This version doe

[PATCH] usbip: vhci_hcd: print correct info in default case

2017-12-05 Thread Pei Zhang
In this switch-case logical, 2 cases will goes to default case. The default log should exclude the upper 2 cases. Signed-off-by: Pei Zhang --- drivers/usb/usbip/vhci_hcd.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/usb/usbip/vhci_hcd.c b/drivers/usb/usbip/vhci

[PATCH] usbip: vhci-hcd: return correct port ENABLE status

2017-12-05 Thread Pei Zhang
USB system will clear port's ENABLE feature for some USB devices when vdev is already assigned port address. This cause getPortStatus reports to system that this device is not enabled, client OS will failed to use this usb device. The failure devices include a SAMSUNG SSD storage, and a Logitech w

[PATCH] usbip: vhci-hcd: return correct port ENABLE status

2017-12-05 Thread Pei Zhang
USB system will clear port's ENABLE feature for some USB devices when vdev is already assigned port address. This cause getPortStatus reports to system that this device is not enabled, client OS will failed to use this usb device. The failure devices include a SAMSUNG SSD storage, and a Logitech w

Re: [RFC v3 PATCH 0/2] Introduce Security Version to EFI Stub

2017-12-05 Thread Gary Lin
On Tue, Dec 05, 2017 at 04:14:26PM -0500, Josh Boyer wrote: > On Tue, Dec 5, 2017 at 5:01 AM, Gary Lin wrote: > > The series of patches introduce Security Version to EFI stub. > > > > Security Version is a monotonically increasing number and designed to > > prevent the user from loading an insecur

linux-next: build failure after merge of the printk tree

2017-12-05 Thread Stephen Rothwell
Hi Petr, After merging the printk tree, today's linux-next build (x86_64 allnoconfig) failed like this: lib/vsprintf.o: In function `pointer': vsprintf.c:(.text+0x2260): undefined reference to `dereference_symbol_descriptor' Caused by commit b77929215209 ("symbol lookup: introduce dereferenc

Re: [PATCH] ptr_ring: add barriers

2017-12-05 Thread Jason Wang
On 2017年12月06日 10:53, Michael S. Tsirkin wrote: On Wed, Dec 06, 2017 at 10:31:39AM +0800, Jason Wang wrote: On 2017年12月06日 03:29, Michael S. Tsirkin wrote: Users of ptr_ring expect that it's safe to give the data structure a pointer and have it be available to consumers, but that actually re

Re: [PATCH] netlink: Add netns check on taps

2017-12-05 Thread Kevin Cernekee
On Tue, Dec 5, 2017 at 6:19 PM, David Ahern wrote: >> + if (!net_eq(dev_net(dev), sock_net(sk)) && >> + !net_eq(dev_net(dev), &init_net)) { > > Why is init_net special? Seems like snooping should be limited to the > namespace you are in. Depends how important it is to preserve the cur

Re: [PATCH v4 72/73] xfs: Convert mru cache to XArray

2017-12-05 Thread Dave Chinner
On Tue, Dec 05, 2017 at 06:02:08PM -0800, Matthew Wilcox wrote: > On Wed, Dec 06, 2017 at 12:36:48PM +1100, Dave Chinner wrote: > > > - if (radix_tree_preload(GFP_NOFS)) > > > - return -ENOMEM; > > > - > > > INIT_LIST_HEAD(&elem->list_node); > > > elem->key = key; > > > > > > - spin_l

[PATCH v4] mmap.2: MAP_FIXED updated documentation

2017-12-05 Thread john . hubbard
From: John Hubbard Previously, MAP_FIXED was "discouraged", due to portability issues with the fixed address. In fact, there are other, more serious issues. Also, alignment requirements were a bit vague. So: -- Expand the documentation to discuss the hazards in enough detail to allow

[PATCHv3 0/2] mtd: m25p80: restore the addressing mode when exiting

2017-12-05 Thread Zhiqiang Hou
From: Hou Zhiqiang Restore the status to be compatible with legacy devices. Take Freescale eSPI boot for example, it copies (in 3 Byte addressing mode) the RCW and bootloader images from SPI flash without firing a reset signal previously, so the reboot command will fail without reseting the addre

[PATCHv3 1/2] mtd: spi-nor: add an API to restore the status of SPI flash chip

2017-12-05 Thread Zhiqiang Hou
From: Hou Zhiqiang Add this API to restore the status of SPI flash chip to the default such as addressing mode, whenever detach the driver from device or reboot the system. Signed-off-by: Hou Zhiqiang --- V3: - no change. Documentation/mtd/spi-nor.txt | 3 +++ drivers/mtd/spi-nor/spi-nor.c

[PATCHv3 2/2] mtd: m25p80: restore the status of SPI flash when exiting

2017-12-05 Thread Zhiqiang Hou
From: Hou Zhiqiang Restore the status to be compatible with legacy devices. Take Freescale eSPI boot for example, it copies (in 3 Byte addressing mode) the RCW and bootloader images from SPI flash without firing a reset signal previously, so the reboot command will fail without reseting the addre

[PATCH] writeback: update comment in inode_io_list_move_locked

2017-12-05 Thread Wang Long
The @head can be wb->b_dirty_time, so update the comment. Signed-off-by: Wang Long --- fs/fs-writeback.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/fs-writeback.c b/fs/fs-writeback.c index 8d477cf..7cb1dd1 100644 --- a/fs/fs-writeback.c +++ b/fs/fs-writeback.c @@ -126

Re: [PATCH v2 2/3] hwrng: exynos - add Samsung Exynos True RNG driver

2017-12-05 Thread PrasannaKumar Muralidharan
Hi Lukasz, On 27 November 2017 at 15:28, Łukasz Stelmach wrote: > Add support for True Random Number Generator found in Samsung Exynos > 5250+ SoCs. > > Signed-off-by: Łukasz Stelmach > --- > MAINTAINERS | 7 + > drivers/char/hw_random/Kconfig | 12 ++ > driver

  1   2   3   4   5   6   7   8   9   10   >