Re: [PATCH 1/6] usb: xhci: add xhci_log_cmd trace events

2017-02-14 Thread Felipe Balbi
Hi, Lu Baolu writes: > diff --git a/drivers/usb/host/xhci-trace.h b/drivers/usb/host/xhci-trace.h > index 1ac2cdf..c31eeaf 100644 > --- a/drivers/usb/host/xhci-trace.h > +++ b/drivers/usb/host/xhci-trace.h > @@ -285,6 +285,96 @@ DEFINE_EVENT(xhci_log_urb, xhci_urb_dequeue, > TP_ARGS(urb) >

[PATCH] staging: rtl8192u: Fix warnings about endianness

2017-02-14 Thread maomao xu
drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast to restricted __le16 drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast to restricted __le16 drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning: cast to restricted

[PATCH] PCI-DW: ARM64: check the result of hos_init

2017-02-14 Thread songxiaowei
Enumeration should be prevented when Host initialization fails. Only one device link with Host escpecially, if host initialization fails or establishling link-up is not successful, there may no need for the Enumeration. Signed-off-by: songxiaowei --- drivers/pci/host/pcie-designware.c | 7

Re: [PATCH] rt2500usb: don't mark register accesses as inline

2017-02-14 Thread Arnd Bergmann
On Wed, Feb 15, 2017 at 6:49 AM, Kalle Valo wrote: > Arnd Bergmann writes: > >> When CONFIG_KASAN is set, we get a rather large stack here: >> >> drivers/net/wireless/ralink/rt2x00/rt2500usb.c: In function >> 'rt2500usb_set_device_state': >> drivers/net/wireless/ralink/rt2x00/rt2500usb.c:1074:1:

Re: [PATCH 3/2] sched/deadline: Use deadline instead of period when calculating overflow

2017-02-14 Thread Luca Abeni
Hi Steven, On Tue, 14 Feb 2017 19:14:17 -0500 Steven Rostedt wrote: [...] > > I am not sure about the correct fix (wouldn't > > "runtime / (deadline - t) > dl_runtime / dl_deadline" allow the > > task to use a fraction of CPU time equal to dl_runtime / > > dl_deadline?) > > > > The current code

Re: [usb-storage] usb: storage: suspicious code

2017-02-14 Thread Gustavo A. R. Silva
Hi Oliver, Quoting Oliver Neukum : Am Dienstag, den 14.02.2017, 23:06 -0600 schrieb Gustavo A. R. Silva: Hi, waitcount = 0; do { result = jumpshot_get_status(us); if (result != USB_STOR_TRANSPORT_GOOD) { // I have not experimented to find the smallest value.  

Re: [PATCH 0/8] net: stmmac: misc patchs

2017-02-14 Thread Corentin Labbe
On Wed, Feb 15, 2017 at 07:34:41AM +0100, Giuseppe CAVALLARO wrote: > Hello Corentin > > On 2/14/2017 8:54 PM, Corentin Labbe wrote: > > Hello > > > > This is a follow up of my previous stmmac serie which address some comment > > done in v2. > > I wonder if you can resend all the patches as V3 >

Re: [PATCH v7 0/5] Update LZ4 compressor module

2017-02-14 Thread Minchan Kim
Hi Sven, On Mon, Feb 13, 2017 at 01:08:41PM +0100, Sven Schmidt wrote: > On Mon, Feb 13, 2017 at 09:03:24AM +0900, Minchan Kim wrote: > > Hi Sven, > > > > On Sun, Feb 12, 2017 at 12:16:17PM +0100, Sven Schmidt wrote: > > > > > > > > > > > > On 02/10/2017 01:13 AM, Minchan Kim wrote: > > > > He

Re: [PATCH v4 3/4] dmaengine: Add Broadcom SBA RAID driver

2017-02-14 Thread Dan Williams
On Tue, Feb 14, 2017 at 11:03 PM, Anup Patel wrote: > On Wed, Feb 15, 2017 at 12:13 PM, Dan Williams > wrote: >> On Tue, Feb 14, 2017 at 10:25 PM, Anup Patel wrote: >>> On Tue, Feb 14, 2017 at 10:04 PM, Dan Williams >>> wrote: On Mon, Feb 13, 2017 at 10:51 PM, Anup Patel wrote: >>

[PATCH 3/3] crypto: cavium/cptvf: switch to pci_alloc_irq_vectors

2017-02-14 Thread Christoph Hellwig
pci_enable_msix has been long deprecated, but this driver adds a new instance. Convert it to pci_alloc_irq_vectors and greatly simplify the code, and make sure the prope code properly unwinds. Signed-off-by: Christoph Hellwig --- drivers/crypto/cavium/cpt/cptvf.h | 3 - drivers/crypto/ca

[PATCH 2/3] crypto: cavium/cptpf: switch to pci_alloc_irq_vectors

2017-02-14 Thread Christoph Hellwig
pci_enable_msix has been long deprecated, but this driver adds a new instance. Convert it to pci_alloc_irq_vectors and greatly simplify the code. Signed-off-by: Christoph Hellwig --- drivers/crypto/cavium/cpt/cptpf.h | 5 --- drivers/crypto/cavium/cpt/cptpf_main.c | 58 ++-

[PATCH 1/3] crypto: cavium: remove dead MSI-X related define

2017-02-14 Thread Christoph Hellwig
Signed-off-by: Christoph Hellwig --- drivers/crypto/cavium/cpt/cpt_common.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/crypto/cavium/cpt/cpt_common.h b/drivers/crypto/cavium/cpt/cpt_common.h index ede612f306d3..225078d03773 100644 --- a/drivers/crypto/cavium/cpt/cpt_common.h ++

crypto/cavium MSI-X fixups

2017-02-14 Thread Christoph Hellwig
Hi George, your commit "crypto: cavium - Add Support for Octeon-tx CPT Engine" add a new caller to pci_enable_msix. This API has long been deprecated so this series switches it to use pci_alloc_irq_vectors instead. Can you please test it and make sure it goes in before the end of the merge windo

Re: [RFC] mm/zsmalloc: remove redundant SetPagePrivate2 in create_page_chain

2017-02-14 Thread Minchan Kim
On Tue, Feb 14, 2017 at 08:48:29PM +0800, Yisheng Xie wrote: > We had used page->lru to link the component pages (except the first > page) of a zspage, and used INIT_LIST_HEAD(&page->lru) to init it. > Therefore, to get the last page's next page, which is NULL, we had to > use page flag PG_Private_

Re: [PATCH v9 8/8] perf/amd/iommu: Enable support for multiple IOMMUs

2017-02-14 Thread Suravee Suthikulpanit
Boris, On 2/9/17 02:33, Borislav Petkov wrote: On Tue, Feb 07, 2017 at 02:40:36AM -0600, Suravee Suthikulpanit wrote: From: Suravee Suthikulpanit [..] + perf_iommu->max_banks= amd_iommu_pc_get_max_banks(idx); + perf_iommu->max_counters = amd_iommu_pc_get_max_counters(idx);

Re: [PATCH 1/3] powerpc: kprobes: add support for KPROBES_ON_FTRACE

2017-02-14 Thread Masami Hiramatsu
Hi Naveen, On Wed, 15 Feb 2017 00:28:34 +0530 "Naveen N. Rao" wrote: > diff --git a/arch/powerpc/kernel/optprobes.c b/arch/powerpc/kernel/optprobes.c > index e51a045f3d3b..a8f414a0b141 100644 > --- a/arch/powerpc/kernel/optprobes.c > +++ b/arch/powerpc/kernel/optprobes.c > @@ -70,6 +70,9 @@ stat

Re: [PATCH] mm: free reserved area's memmap if possiable

2017-02-14 Thread Ard Biesheuvel
On 15 February 2017 at 01:44, zhouxianrong wrote: > > > On 2017/2/14 17:03, Ard Biesheuvel wrote: >> >> On 14 February 2017 at 06:53, wrote: >>> >>> From: zhouxianrong >>> >>> just like freeing no-map area's memmap (gaps of memblock.memory) >>> we could free reserved area's memmap (areas of mem

Re: [PATCH v4 3/4] dmaengine: Add Broadcom SBA RAID driver

2017-02-14 Thread Anup Patel
On Wed, Feb 15, 2017 at 12:13 PM, Dan Williams wrote: > On Tue, Feb 14, 2017 at 10:25 PM, Anup Patel wrote: >> On Tue, Feb 14, 2017 at 10:04 PM, Dan Williams >> wrote: >>> On Mon, Feb 13, 2017 at 10:51 PM, Anup Patel >>> wrote: The Broadcom stream buffer accelerator (SBA) provides offloa

Re: [usb-storage] usb: storage: suspicious code

2017-02-14 Thread Oliver Neukum
Am Dienstag, den 14.02.2017, 23:06 -0600 schrieb Gustavo A. R. Silva: Hi, > waitcount = 0; > do { > result = jumpshot_get_status(us); > if (result != USB_STOR_TRANSPORT_GOOD) { > // I have not experimented to find the smallest > value. > // >   

[PATCH] staging: rtl8192u: Fix sparse warnings about endianness

2017-02-14 Thread maomao xu
drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c:175:36: warning: incorrect type in assignment (different base types) drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c:175:36:expected unsigned short [unsigned] [short] [usertype] drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c:175:36:g

[RESEND PATCH linux] aspeed: pinctrl: Allow disabling Port D and Port E loopback mode

2017-02-14 Thread Rick Altherr
Port D and port E GPIO loopback modes are commonly enabled via hardware straps for use with front-panel buttons. When the BMC is powered off or fails to boot, the front-panel buttons are directly connected to the host chipset via the loopback to allow direct power-on and reset control. Once the BM

Re: [PATCH] uapi: add missing install of dma-buf.h

2017-02-14 Thread Sumit Semwal
Hi Denys, On 15 February 2017 at 01:30, Denys Dmytriyenko wrote: > From: Denys Dmytriyenko > > As part of c11e391da2a8fe973c3c2398452000bed505851e "dma-buf: Add ioctls to > allow userspace to flush" a new uapi header file dma-buf.h was added, but an > entry was not added on Kbuild to install it.

Re: [PATCH v4 3/4] dmaengine: Add Broadcom SBA RAID driver

2017-02-14 Thread Dan Williams
On Tue, Feb 14, 2017 at 10:25 PM, Anup Patel wrote: > On Tue, Feb 14, 2017 at 10:04 PM, Dan Williams > wrote: >> On Mon, Feb 13, 2017 at 10:51 PM, Anup Patel wrote: >>> The Broadcom stream buffer accelerator (SBA) provides offloading >>> capabilities for RAID operations. This SBA offload engine

Re: [PATCH 0/8] net: stmmac: misc patchs

2017-02-14 Thread Giuseppe CAVALLARO
Hello Corentin On 2/14/2017 8:54 PM, Corentin Labbe wrote: Hello This is a follow up of my previous stmmac serie which address some comment done in v2. I wonder if you can resend all the patches as V3 please add the Acked-by and Reviewed-by. This will help on final review and integration. Th

Re: [PATCH V8 05/10] acpi: apei: handle SEA notification type for ARMv8

2017-02-14 Thread Zhengqiang
Hi Tyler: On 2017/2/2 1:16, Tyler Baicar wrote: > ARM APEI extension proposal added SEA (Synchronous External Abort) > notification type for ARMv8. > Add a new GHES error source handling function for SEA. If an error > source's notification type is SEA, then this function can be registered > into

Re: net/xfrm: stack out-of-bounds in xfrm_flowi_sport

2017-02-14 Thread Steffen Klassert
On Tue, Feb 14, 2017 at 10:16:44AM +0100, Dmitry Vyukov wrote: > > I've run the repro with you patch and don't see the bug any more: > > Tested-by: Dmitry Vyukov I've applied this to the ipsec tree now. Thanks for testing!

Re: [PATCH v4 3/4] dmaengine: Add Broadcom SBA RAID driver

2017-02-14 Thread Anup Patel
On Tue, Feb 14, 2017 at 10:04 PM, Dan Williams wrote: > On Mon, Feb 13, 2017 at 10:51 PM, Anup Patel wrote: >> The Broadcom stream buffer accelerator (SBA) provides offloading >> capabilities for RAID operations. This SBA offload engine is >> accessible via Broadcom SoC specific ring manager. >>

RE: [PATCH] scsi: megaraid_sas: handle dma_addr_t right on 32-bit

2017-02-14 Thread Sumit Saxena
>-Original Message- >From: Arnd Bergmann [mailto:a...@arndb.de] >Sent: Wednesday, February 15, 2017 2:52 AM >To: James E.J. Bottomley; Martin K. Petersen >Cc: Arnd Bergmann; Kashyap Desai; Sumit Saxena; Shivasharan S; Tomas Henzl; >Hannes Reinecke; Sasikumar Chandrasekaran; >megaraidlinux..

[PATCH linux] aspeed: pinctrl: Allow disabling Port D and Port E loopback mode

2017-02-14 Thread Rick Altherr
Port D and port E GPIO loopback modes are commonly enabled via hardware straps for use with front-panel buttons. When the BMC is powered off or fails to boot, the front-panel buttons are directly connected to the host chipset via the loopback to allow direct power-on and reset control. Once the BM

[PATCH 2/2] clk: qoriq: Separate root input clock for core PLLs on ls1012a

2017-02-14 Thread yuantian.tang
From: Tang Yuantian ls1012a has separate input root clocks for core PLLs versus the platform PLL, with the latter described as sysclk in the hw docs. If a second input clock, named "coreclk", is present, this clock will be used for the core PLLs. Signed-off-by: Scott Wood Signed-off-by: Tang Yu

[PATCH 1/2] dt-bindings: qoriq-clock: Add coreclk

2017-02-14 Thread yuantian.tang
From: Tang Yuantian ls1012a has separate input root clocks for core PLLs versus the platform PLL, with the latter described as sysclk in the hw docs. Update the qoriq-clock binding to allow a second input clock, named "coreclk". If present, this clock will be used for the core PLLs. Signed-off-

Looking more details and reasons for using orig_add_limit.

2017-02-14 Thread Sodagudi Prasad
Hi All, Would like to understand the reasons behind using the orig_add_limit variable in the following code. Can you please share more details ? "arch/arm64/mm/fault.c" static int __kprobes do_page_fault(unsigned long addr, unsigned int esr, struct pt_regs

Re: [PATCH] rt2500usb: don't mark register accesses as inline

2017-02-14 Thread Kalle Valo
Arnd Bergmann writes: > When CONFIG_KASAN is set, we get a rather large stack here: > > drivers/net/wireless/ralink/rt2x00/rt2500usb.c: In function > 'rt2500usb_set_device_state': > drivers/net/wireless/ralink/rt2x00/rt2500usb.c:1074:1: error: the frame size > of 3032 bytes is larger than 100 b

linux-next: Tree for Feb 15

2017-02-14 Thread Stephen Rothwell
Hi all, Changes since 20170214: The net tree still had its build failure for which I applied a fix patch. The f2fs tree gained a conflict against the fscrypt tree. The rdma tree gained a build failure due to an interaction with the net-next tree for which I applied a merge fix patch. The kvm

Re: [PATCH V2] vhost: introduce O(1) vq metadata cache

2017-02-14 Thread Jason Wang
On 2016年12月14日 17:53, Jason Wang wrote: When device IOTLB is enabled, all address translations were stored in interval tree. O(lgN) searching time could be slow for virtqueue metadata (avail, used and descriptors) since they were accessed much often than other addresses. So this patch introduce

Re: [PATCH 1/1] crypto: brcm - Avoid double free in ahash_finup()

2017-02-14 Thread Herbert Xu
On Tue, Feb 14, 2017 at 12:45:52PM -0500, Rob Rice wrote: > In Broadcom SPU driver, in case where incremental hash > is done in software in ahash_finup(), tmpbuf was freed > twice. > > Reported-by: Dan Carpenter > Signed-off-by: Rob Rice Patch applied. Thanks. -- Email: Herbert Xu Home Page:

Re: [PATCH] crypto: cavium: fix Kconfig dependencies

2017-02-14 Thread Herbert Xu
On Tue, Feb 14, 2017 at 06:07:31PM +0100, Arnd Bergmann wrote: > The driver fails to build if MSI support is disabled: > > In file included from > /git/arm-soc/drivers/crypto/cavium/cpt/cptpf_main.c:18:0: > drivers/crypto/cavium/cpt/cptpf.h:57:20: error: array type has incomplete > element type

Re: [PATCH] drivers: crypto: cpt: cpt_bind_vq_to_grp could return an error code

2017-02-14 Thread Herbert Xu
On Tue, Feb 14, 2017 at 09:23:17AM +, George Cherian wrote: > cpt_bind_vq_to_grp() could return an error code. However, it currently > returns a u8. This produce the static checker warning. > > drivers/crypto/cavium/cpt/cptpf_mbox.c:70 cpt_bind_vq_to_grp() warn: > signedness bug returning '(-

Re: [PATCH next 0/2] crypto: atmel-sha: fix error management

2017-02-14 Thread Herbert Xu
On Thu, Feb 09, 2017 at 05:51:19PM +0100, Cyrille Pitchen wrote: > Hi all, > > this series is based on next-20170209. All applied. Thanks. -- Email: Herbert Xu Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

Re: [PATCH v8 1/2] soc: qcom: Add SoC info driver

2017-02-14 Thread Imran Khan
On 2/15/2017 5:54 AM, Bjorn Andersson wrote: > On Tue 10 Jan 07:48 PST 2017, Imran Khan wrote: > >> diff --git a/drivers/soc/qcom/socinfo.c b/drivers/soc/qcom/socinfo.c >> new file mode 100644 >> index 000..40c180d >> --- /dev/null >> +++ b/drivers/soc/qcom/socinfo.c >> @@ -0,0 +1,516 @@ >> +/

[PATCH v7 2/7] drm/rockchip/dsi: dw-mipi: support RK3399 mipi dsi

2017-02-14 Thread Chris Zhong
The vopb/vopl switch register of RK3399 mipi is different from RK3288, the default setting for mipi dsi mode is different too, so add a of_device_id structure to distinguish them, and make sure set the correct mode before mipi phy init. Signed-off-by: Chris Zhong Signed-off-by: Mark Yao Reviewed

Re: [PATCH V3 2/4] phy: phy-exynos-pcie: Add support for Exynos PCIe phy

2017-02-14 Thread Vivek Gautam
On 02/13/2017 01:56 PM, Jaehoon Chung wrote: This patch adds support for Generic PHY framework about Exynos SoCs. Current Exynos PCIe driver doesn't use the PHY framework. It's making a difficult to upstream the other Exynos variants because of different PHY registers. Move the codes relevant t

Re: [PATCH] IB/hfi1: use size_t for passing array length

2017-02-14 Thread Leon Romanovsky
On Tue, Feb 14, 2017 at 10:23:07PM +0100, Arnd Bergmann wrote: > gcc-7 produces a mysterious warning about the size argument being potentially > out > of range: > > drivers/infiniband/hw/hfi1/verbs.c: In function 'init_cntr_names': > drivers/infiniband/hw/hfi1/verbs.c:1644:2: error: 'memcpy': spec

Re: [PATCH] sched/deadline: Remove unnecessary condition in push_dl_task()

2017-02-14 Thread Byungchul Park
On Wed, Feb 15, 2017 at 02:11:51PM +0900, Byungchul Park wrote: > Once pick_next_pushable_dl_task(rq) return a task, it guarantees that > the task's cpu is rq->cpu, so task_cpu(next_task) is always rq->cpu if > task == next_task. Remove a redundant condition and make code simpler. Furthermore, unn

[PATCH] ext4: delayed inode insertion into transaction at write()

2017-02-14 Thread Seongbae Son
If a write request writes a data which is larger than 4 KB, the inode of the file is updated by the write system call, and also by kworker thread. This duplicated inode update causes malfunction of the EXT4 journaling behavior in ordered mode. For example, if 6KB write(fileA) on 18KB file is per

Re: [PATCH] prctl: implement PR_GET_ENDIAN for all architectures

2017-02-14 Thread Michael Ellerman
"Maciej W. Rozycki" writes: > On Sun, 5 Feb 2017, Matt Turner wrote: > >> > I don't think it is other than for PPC. If you're not variable endian >> > (which is only PPC to date), then you should know a priori what endian >> > you are from the #defines in userspace. >> >> MIPS as well, but it s

[PATCH] PCI/PME: Restore pcie_pme_driver.remove

2017-02-14 Thread Yinghai Lu
Found 4.9 and later, removing pci device for pcie port via /sys failed: [ cut here ] kernel BUG at drivers/pci/msi.c:370! invalid opcode: [#1] SMP Modules linked in: CPU: 1 PID: 14509 Comm: sh Tainted: GW 4.8.0-rc1-yh-00012-gd29438d RIP: 0010:[] free_msi_irqs+0x65

[RFC PATCH v2 2/3] crypto: zip - Wire-up Compression / decompression HW offload

2017-02-14 Thread Mahipal Challa
This contains changes for adding compression/decompression h/w offload functionality for both DEFLATE and LZS. Signed-off-by: Mahipal Challa Signed-off-by: Jan Glauber --- drivers/crypto/cavium/zip/Makefile | 5 +- drivers/crypto/cavium/zip/zip_crypto.c | 314 +++

[RFC PATCH v2 3/3] crypto: zip - Add Compression/Decompression statistics

2017-02-14 Thread Mahipal Challa
Add statistics for compression/decompression hardware offload under debugfs. Signed-off-by: Mahipal Challa Signed-off-by: Jan Glauber --- drivers/crypto/cavium/zip/zip_deflate.c | 10 ++ drivers/crypto/cavium/zip/zip_inflate.c | 12 ++ drivers/crypto/cavium/zip/zip_main.c| 234 +++

[RFC PATCH v2 0/3] Cavium ThunderX ZIP driver

2017-02-14 Thread Mahipal Challa
Hi Herbert, This series adds support for hardware accelerated compression & decompression as found on ThunderX (arm64) SOCs. As per your suggestion, we've switched to the new crypto acomp/scomp interface. To test the ZIP driver, we modified the kernel's ZSWAP to use acomp API's. Performance num

[RFC PATCH v2 1/3] crypto: zip - Add ThunderX ZIP driver core

2017-02-14 Thread Mahipal Challa
Add a driver for the ZIP engine found on Cavium ThunderX SOCs. The ZIP engine supports hardware accelerated compression and decompression. It includes 2 independent ZIP cores and supports: - DEFLATE compression and decompression (RFC 1951) - LZS compression and decompression (RFC 2395 and ANSI X3.

Re: [v4,1/2] arch/powerpc: Implement Optprobes

2017-02-14 Thread Anju T Sudhakar
Thank You Michael. :) On Tuesday 14 February 2017 06:10 PM, Michael Ellerman wrote: On Wed, 2017-02-08 at 09:50:51 UTC, Anju T wrote: Current infrastructure of kprobe uses the unconditional trap instruction to probe a running kernel. Optprobe allows kprobe to replace the trap with a branch in

[PATCH] sched/deadline: Remove unnecessary condition in push_dl_task()

2017-02-14 Thread Byungchul Park
Once pick_next_pushable_dl_task(rq) return a task, it guarantees that the task's cpu is rq->cpu, so task_cpu(next_task) is always rq->cpu if task == next_task. Remove a redundant condition and make code simpler. Signed-off-by: Byungchul Park --- kernel/sched/deadline.c | 2 +- 1 file changed, 1

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

2017-02-14 Thread Selvin Xavier
On Wed, Feb 15, 2017 at 6:35 AM, Doug Ledford wrote: >> From: Stephen Rothwell >> Date: Wed, 15 Feb 2017 11:23:25 +1100 >> Subject: [PATCH] RDMA/bnxt_re: fix for "bnxt_en: Update to firmware >> interface spec 1.7.0." >> >> Signed-off-by: Stephen Rothwell > > Thanks Stephen. I had been forewarn

usb: storage: suspicious code

2017-02-14 Thread Gustavo A. R. Silva
Hello, I ran into the following piece of code at drivers/usb/storage/jumpshot.c:305 (linux-next), and it seems a little bit suspicious: // read the result. apparently the bulk write can complete // before the jumpshot drive is finished writing. so we loop // here until we get a good return cod

Re: Regression in next with use printk_safe buffers in printk

2017-02-14 Thread Sergey Senozhatsky
On (02/14/17 19:29), Peter Zijlstra wrote: > On Wed, Feb 15, 2017 at 01:56:45AM +0900, Sergey Senozhatsky wrote: > > that crossed my mind, but I kinda assumed that we do printk() from > > under tk_core using sched fair, and rt_runtime_lock is from sched rt. > > That's all true; lockdep doesn't car

Re: Regression in next with use printk_safe buffers in printk

2017-02-14 Thread Sergey Senozhatsky
On (02/14/17 09:03), Tony Lindgren wrote: [..] > > Do not call printk() from tk_debug_account_sleep_time(), because > > tk_debug_account_sleep_time() is called under tk_core seq lock. > > It's not safe to call printk() under tk_core, because console_sem > > invokes scheduled (via wake_up_process()-

[PATCH] time/timekeeping_debug: use deferred printk() version

2017-02-14 Thread Sergey Senozhatsky
We cannot do printk() from tk_debug_account_sleep_time(), because tk_debug_account_sleep_time() is called under tk_core seq lock. The reason why printk() is unsafe there is that console_sem may invoke scheduler (up()->wake_up_process()->activate_task()), which, in turn, can return back to timekeepi

Re: [PATCH 1/3] powerpc: kprobes: add support for KPROBES_ON_FTRACE

2017-02-14 Thread Ananth N Mavinakayanahalli
On Wed, Feb 15, 2017 at 12:28:34AM +0530, Naveen N. Rao wrote: > Allow kprobes to be placed on ftrace _mcount() call sites. This > optimization avoids the use of a trap, by riding on ftrace > infrastructure. > > This depends on HAVE_DYNAMIC_FTRACE_WITH_REGS which depends on > MPROFILE_KERNEL, whic

Re: [PATCH V2 3/3] mm: Enable Buddy allocation isolation for CDM nodes

2017-02-14 Thread Anshuman Khandual
On 02/14/2017 03:44 PM, Anshuman Khandual wrote: > On 02/14/2017 01:58 PM, Vlastimil Babka wrote: >> On 02/10/2017 11:06 AM, Anshuman Khandual wrote: >>> This implements allocation isolation for CDM nodes in buddy allocator by >>> discarding CDM memory zones all the time except in the cases where t

[PATCH v7 0/7] Rockchip dw-mipi-dsi driver

2017-02-14 Thread Chris Zhong
Hi all This version does not change the existing v6 patches, just to add the "bandwidth fix" patch back, since we really need it. This patch serial is for RK3399 MIPI DSI. The MIPI DSI controller of RK3399 is almost the same as RK3288, except a little bit of difference in phy clock controlling an

[PATCH v7 1/7] dt-bindings: add rk3399 support for dw-mipi-rockchip

2017-02-14 Thread Chris Zhong
The dw-mipi-dsi of rk3399 is almost the same as rk3288, the rk3399 has additional phy config clock. Signed-off-by: Chris Zhong Acked-by: Rob Herring --- Changes in v6: None Changes in v5: None Changes in v4: None Changes in v3: None .../devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchi

[PATCH] Staging: wlan-ng: Fixed the code style issue:

2017-02-14 Thread Bo YU
WARNING: Block comments should align the * on each line Signed-off-by: Bo YU --- drivers/staging/wlan-ng/p80211metastruct.h | 88 ++-- 1 file changed, 44 insertions(+), 44 deletions(-) diff --git a/drivers/staging/wlan-ng/p80211metastruct.h b/drivers/staging/wlan-ng/

Re: [PATCH v2 11/11] drm/rockchip/dsi: fix insufficient bandwidth of some panel

2017-02-14 Thread Chris Zhong
Hi John On 01/17/2017 06:54 PM, John Keeping wrote: On Tue, 17 Jan 2017 17:31:53 +0800, Chris Zhong wrote: On 01/16/2017 08:44 PM, John Keeping wrote: On Mon, 16 Jan 2017 18:08:31 +0800, Chris Zhong wrote: Set the lanes bps to 1 / 0.9 times of pclk, the margin is not enough for some panel

[PATCH] usb: class: remove logically dead code

2017-02-14 Thread Gustavo A. R. Silva
Remove logically dead code. 'cntr' is always equal to zero when the following line of code is executed: rv = cntr ? cntr : -EAGAIN; Addresses-Coverity-ID: 113227 Signed-off-by: Gustavo A. R. Silva --- drivers/usb/class/cdc-wdm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a

Re: [PATCH] cpufreq: cppc: add ACPI_PROCESSOR dependency

2017-02-14 Thread Viresh Kumar
On 14-02-17, 22:19, Arnd Bergmann wrote: > Without the Kconfig dependency, we can get this warning: > > warning: ACPI_CPPC_CPUFREQ selects ACPI_CPPC_LIB which has unmet direct > dependencies (ACPI && ACPI_PROCESSOR) > > Fixes: 5477fb3bd1e8 ("ACPI / CPPC: Add a CPUFreq driver for use with CPPC")

Re: [PATCH 2/2] iommu: add warning when sharing groups

2017-02-14 Thread okaya
On 2017-02-14 18:51, Alex Williamson wrote: On Tue, 14 Feb 2017 16:25:22 -0500 Sinan Kaya wrote: The ACS requirement has been obscured in the current code and is only known by certain individuals who happen to read the code. Print out a warning with ACS path failure when ACS requirement is not

Re: crypto: NULL deref in sha512_mb_mgr_get_comp_job_avx2

2017-02-14 Thread Herbert Xu
On Mon, Feb 13, 2017 at 09:20:48AM -0800, Tim Chen wrote: > > Megha is now able to create a test set up that produce > similar problem reported by Dmitry.  This patch did not > completely fix it.  So maybe you can hold off on merging > this patch to the mainline till we can develop a more > comple

[PATCH] KVM: Prevent double-free on HPT resize commit path

2017-02-14 Thread David Gibson
resize_hpt_release(), called once the HPT resize of a KVM guest is completed (successfully or unsuccessfully) free()s the state structure for the resize. It is currently not safe to call with a NULL pointer. However, one of the error paths in kvm_vm_ioctl_resize_hpt_commit() can invoke it with a

Re: [PATCH v3 23/24] drm/rockchip: dw-mipi-dsi: add reset control

2017-02-14 Thread Chris Zhong
Hi John On 01/29/2017 09:24 PM, John Keeping wrote: In order to fully reset the state of the MIPI controller we must assert this reset. This is slightly more complicated than it could be in order to maintain compatibility with device trees that do not specify the reset property. Signed-off-by:

Re: [PATCH] staging: xgifb: function definition argument should also have an identifier name

2017-02-14 Thread Joe Perches
On Wed, 2017-02-15 at 09:01 +0530, Arushi Singhal wrote: > Hi Joe > Sorry but I am unable to find the identifier of "unsigned long" .Shall you > please help me in how to find the identifier. > Thanks > Arushi Singhal $ git grep -w XGIRegInit drivers/staging/xgifb/XGI_main_26.c:void XGIRegInit(stru

Re: [PATCH v2 2/3] staging: fbtft: Replace magic number with constant

2017-02-14 Thread Joe Perches
On Wed, 2017-02-15 at 14:27 +1100, Tobin C. Harding wrote: > Current call to strncmp() uses a magic number. There is a compile > time constant defined for this buffer, included and used already at > other sites in the file. > > Remove magic number. Replace with pre-existing compile time constant.

Re: [PATCH v2 6/9] staging: rtl8192u: Fixing coding style issues

2017-02-14 Thread Joe Perches
On Wed, 2017-02-15 at 08:52 +0530, SIMRAN SINGHAL wrote: > Yes, I totally agree with joe. > > But as there is no coding style issue in this patch. Style is more than just checkpatch conformity. > So, do I have to resend the complete patch series again? I'm not applying any of these so that's en

Re: [PATCH] net: ethernet: ti: cpsw: use var instead of func for usage count

2017-02-14 Thread David Miller
From: Ivan Khoronzhuk Date: Tue, 14 Feb 2017 16:02:36 +0200 > The usage count function is based on ndev_running flag that is > updated before calling ndo_open/close, but if ndo is called in > another place, as with suspend/resume, the counter is not changed, > that breaks sus/resume. For common r

[PATCH v2 0/3] staging: fbtft: Fix buffer overflow vulnerability

2017-02-14 Thread Tobin C. Harding
Module copies a user supplied string (module parameter) into a buffer using strncpy() and does not check that the buffer is null terminated. Replace call to strncpy() with call to strlcpy() ensuring that the buffer is null terminated. Replace magic number with pre-existing compile time constant.

[PATCH v2 3/3] staging: fbtft: Add check on strlcpy() return value

2017-02-14 Thread Tobin C. Harding
Return value of strlcpy() is not checked. Name string is silently truncated if longer that SPI_NAME_SIZE, whilst not detrimental to the program logic it would be nice to notify the user. Module is currently quite verbose, adding extra pr_warn() calls will not overly impact this verbosity. Check re

[PATCH] block: do not allow updates through sysfs until registration completes

2017-02-14 Thread Tahsin Erdogan
When a new disk shows up, sysfs queue directory is created before elevator is registered. This allows a user to attempt a scheduler switch even though the initial registration hasn't completed yet. In one scenario, blk_register_queue() calls elv_register_queue() and right before cfq_registered_que

[PATCH v2 2/3] staging: fbtft: Replace magic number with constant

2017-02-14 Thread Tobin C. Harding
Current call to strncmp() uses a magic number. There is a compile time constant defined for this buffer, included and used already at other sites in the file. Remove magic number. Replace with pre-existing compile time constant. Signed-off-by: Tobin C. Harding --- drivers/staging/fbtft/fbtft_de

[PATCH v2 1/3] staging: fbtft: Fix buffer overflow vulnerability

2017-02-14 Thread Tobin C. Harding
Module copies a user supplied string (module parameter) into a buffer using strncpy() and does not check that the buffer is null terminated. Replace call to strncpy() with call to strlcpy() ensuring that the buffer is null terminated. Signed-off-by: Tobin C. Harding --- drivers/staging/fbtft/fb

Re: [PATCH v2 6/9] staging: rtl8192u: Fixing coding style issues

2017-02-14 Thread SIMRAN SINGHAL
Yes, I totally agree with joe. But as there is no coding style issue in this patch. So, do I have to resend the complete patch series again? On Wed, Feb 15, 2017 at 1:53 AM, Joe Perches wrote: > On Wed, 2017-02-15 at 01:44 +0530, simran singhal wrote: > [] >> diff --git a/drivers/staging/rtl819

Re: [PATCH v2] uapi: fix linux/if_pppol2tp.h userspace compilation errors

2017-02-14 Thread David Miller
From: "Dmitry V. Levin" Date: Wed, 15 Feb 2017 05:23:26 +0300 > Because of interface limitations, > provided by libc cannot be included after , therefore any > header that includes cannot be included after . > > Change uapi/linux/l2tp.h, the last uapi header that includes > , to include and

Re: [RFC] blkio_delay_total is calculated mistakenly

2017-02-14 Thread Weiwei Jia
On Tue, Feb 14, 2017 at 10:00 PM, Greg KH wrote: > On Tue, Feb 14, 2017 at 08:27:36PM -0500, Weiwei Jia wrote: >> Hi, >>Balbir Singh >> When I read Linux Kernel source codes about how "blkio_delay_total" >> (delay waiting for synchronous block I/O to complete for one task) is >> calculated in Lin

Re: [RFC] blkio_delay_total is calculated mistakenly

2017-02-14 Thread Greg KH
On Tue, Feb 14, 2017 at 08:27:36PM -0500, Weiwei Jia wrote: > Hi, > > When I read Linux Kernel source codes about how "blkio_delay_total" > (delay waiting for synchronous block I/O to complete for one task) is > calculated in Linux/kernel/delayacct.c file [1], I find that > "blkio_delay_total" add

[PATCH] staging: rtl8712: Fix sparse warnings about endianness

2017-02-14 Thread maomao xu
drivers/staging/rtl8712/rtl871x_xmit.c:350:44: warning: restricted __le32 degrades to integer Signed-off-by: maomao xu diff --git a/drivers/staging/rtl8712/rtl871x_xmit.c b/drivers/staging/rtl8712/rtl871x_xmit.c index 4ab82ba..1c2ac28 100644 --- a/drivers/staging/rtl8712/rtl871x_xmit.c +++ b/d

[PATCH] powerpc/xmon: Fix an unexpected xmon onoff state change

2017-02-14 Thread Pan Xinhui
Once xmon is triggered by sysrq-x, it is enabled always afterwards even if it is disabled during boot. This will cause a system reset interrut fail to dump. So keep xmon in its original state after exit. Signed-off-by: Pan Xinhui --- arch/powerpc/xmon/xmon.c | 5 - 1 file changed, 4 insertio

Re: [PATCH V6 1/3] uapi: sed-opal fix IOW for activate lsp to use correct struct

2017-02-14 Thread Jens Axboe
On 02/14/2017 05:29 PM, Scott Bauer wrote: > The IOC_OPAL_ACTIVATE_LSP took the wrong strcure which would > give us the wrong size when using _IOC_SIZE, switch it to the > right structure. Applied 1-3, thanks. -- Jens Axboe

[rcu:rcu/dev 46/46] arch/m32r/include/asm/cmpxchg.h:59: undefined reference to `__xchg_called_with_bad_pointer'

2017-02-14 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git rcu/dev head: f09c1acfaa4b198694a31ba3e454c7d17c4fa680 commit: f09c1acfaa4b198694a31ba3e454c7d17c4fa680 [46/46] rcu: Add consistency checks for uses of segmented callback lists config: m32r-usrv_defconfig (attached as

Re: [PATCH v2] tty: pl011: Work around QDF2400 E44 stuck BUSY bit

2017-02-14 Thread Timur Tabi
Christopher Covington wrote: The Qualcomm Datacenter Technologies QDF2400 family of SoCs contains a custom (non-PrimeCell) implementation of the SBSA UART. Occasionally the BUSY bit in the Flag Register gets stuck as 1, erratum 44 for both 2432v1 and 2400v1 SoCs. Checking that the Transmit FIFO E

linux-next: manual merge of the kvm tree with the powerpc tree

2017-02-14 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the kvm tree got a conflict in: arch/powerpc/kvm/book3s_hv_rm_xics.c between commit: ab9bad0ead9a ("powerpc/powernv: Remove separate entry for OPAL real mode calls") from the powerpc tree and commit: 21acd0e4df04 ("KVM: PPC: Book 3S: XICS: Don't lock

[PATCH 5/6] usb: xhci: fix link trb decoding

2017-02-14 Thread Lu Baolu
xhci_decode_trb() treats a link trb in the same way as that for an event trb. This patch fixes this by decoding the link trb according to the spec. Signed-off-by: Lu Baolu --- drivers/usb/host/xhci.h | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/drivers/usb/hos

[PATCH 1/6] usb: xhci: add xhci_log_cmd trace events

2017-02-14 Thread Lu Baolu
This patch creates a new event class called xhci_log_cmd, and defines the events used for tracing the life cycle of commands issued for various purposes. This info can be used, later, to print, in a human readable way, the life cycle of an xHCI command using the trace-cmd tool and the appropriate

[PATCH 3/6] usb: xhci: remove xhci_debug_trb()

2017-02-14 Thread Lu Baolu
Every XHCI TRB has already been traced by the trb trace events. It is unnecessary to put the same message in kernel log. This patch removes xhci_debug_trb(). Signed-off-by: Lu Baolu --- drivers/usb/host/xhci-dbg.c | 57 drivers/usb/host/xhci-ring.c |

[PATCH 2/6] usb: xhci: enhance xhci_log_ctx trace events

2017-02-14 Thread Lu Baolu
XHCI driver has defined xhci_log_ctx trace events to trace the change of an xhci input or output context. This patch extends the trace class of xhci_log_ctx to print out the contents of a context block in a human readable way. This patch also adds some other xhci_log_ctx based events where the xhc

[PATCH 4/6] usb: xhci: remove xhci_dbg_ctx()

2017-02-14 Thread Lu Baolu
XHCI context changes have already been traced by the trace events. It's unnecessary to put the same message in kernel log. This patch removes the use of xhci_dbg_ctx(). Signed-off-by: Lu Baolu --- drivers/usb/host/xhci-dbg.c | 143 drivers/usb/host/xh

[PATCH 6/6] usb: xhci: cleanup xhci_decode_trb() slightly

2017-02-14 Thread Lu Baolu
Replace 'TRB_FIELD_TO_TYPE(field3)' with 'type' to simplify code. Signed-off-by: Lu Baolu --- drivers/usb/host/xhci.h | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/drivers/usb/host/xhci.h b/drivers/usb/host/xhci.h index ff12c8a..b97fb7

[PATCH 0/6] usb: xhci: several patches for xhci trace

2017-02-14 Thread Lu Baolu
Hi Mathias, This patch set includes several patches for traces in xhci driver. One trace class is for command. Several trace events are defined to trace the life cycle of any xhci command. The other trace class is for context. Several trace events are defined to trace the change in input/output de

[PATCH] f2fs: introduce noinline_xattr mount option

2017-02-14 Thread Chao Yu
This patch introduces new mount option 'noinline_xattr', so we can disable inline xattr functionality which is already set as a default mount option. Signed-off-by: Chao Yu --- Documentation/filesystems/f2fs.txt | 1 + fs/f2fs/super.c| 11 +++ 2 files changed, 12 ins

Re: [PATCH v7 1/5] x86: add simple udelay calibration

2017-02-14 Thread Lu Baolu
Hi, On 02/14/2017 05:23 PM, Sergei Shtylyov wrote: > Hello! > > On 2/14/2017 5:27 AM, Lu Baolu wrote: > >> Add a simple udelay calibration in x86 architecture-specific >> boot-time initializations. This will get a workable estimate >> for loops_per_jiffy. Hence, udelay() could be used after this >

[PATCH v2 2/2] arm: dts: mt2701: Add i2c device node

2017-02-14 Thread Jun Gao
From: Jun Gao Add i2c device node for MT2701. Change-Id: Idd33ad9010c43809328814350f9c29284891d171 Signed-off-by: Jun Gao --- arch/arm/boot/dts/mt2701-evb.dts | 44 ++ arch/arm/boot/dts/mt2701.dtsi| 42 2 files ch

[PATCH v2 0/2] Add i2c DT binding and device node for Mediatek MT2701 Soc

2017-02-14 Thread Jun Gao
This patch series based on v4.10-rc2, include MT2701 i2c DT binding and device node. changes since v1: - Modify commit message Dependent on "Add clock and power domain DT nodes for Mediatek MT2701"[1]. [1] http://lists.infradead.org/pipermail/linux-mediatek/2016-December/007637.html Jun Gao (

  1   2   3   4   5   6   7   8   >