Re: [PATCH] mm, hugetlbfs: include mm.h for vm_operations_struct

2017-12-11 Thread Stephen Rothwell
Hi Arnd, On Mon, 11 Dec 2017 17:21:27 +0100 Arnd Bergmann wrote: > > The infiniband umem code causes a build failure in some configurations: > > In file included from drivers/infiniband/core/umem_odp.c:41:0: > include/linux/hugetlb.h: In function 'vma_kernel_pagesize': > include/linux/hugetlb.h:

RE: [PATCH] arm: dts: uniphier: add efuse node for UniPhier 32bit SoC

2017-12-11 Thread Keiji Hayashibara
Hello Yamada-san, Sorry, I will be careful next time. I was checking with the command of "get_maintainer.pl -f arch/arm/boot/dts/". Thank you. Best Regards, Keiji Hayashibara > From: Masahiro Yamada [mailto:yamada.masah...@socionext.com] > Sent: Tuesday, December 12, 2017 1:06 AM > > 2017-12

Re: [PATCH V3 1/2] Drivers/PCI: Export pcie_has_flr() interface

2017-12-11 Thread Bjorn Helgaas
On Mon, Dec 11, 2017 at 06:29:29PM -0600, Govinda Tatti wrote: > > Thanks Bjorn for your review comments. Please see below for my comments. > > On 12/8/2017 2:24 PM, Bjorn Helgaas wrote: > >On Thu, Dec 07, 2017 at 05:21:44PM -0500, Govinda Tatti wrote: > >>This patch exports pcie_has_flr() and it

[BUG] fs/f2fs: a possible sleep-in-atomic bug in f2fs_trace_pid

2017-12-11 Thread Jia-Ju Bai
According to fs/f2fs/trace.c, the kernel module may sleep under a spinlock. The function call path is: f2fs_trace_pid (acquire the spinlock) f2fs_radix_tree_insert cond_resched --> may sleep I do not find a good way to fix it, so I only report. This possible bug is found by my static analys

linux-next: manual merge of the net-next tree with the net tree

2017-12-11 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the net-next tree got a conflict in: drivers/net/phy/meson-gxl.c between commit: f1e2400a80ff ("net: phy: meson-gxl: detect LPA corruption") from the net tree and commit: 80274abafc60 ("net: phy: remove generic settings for callbacks config_aneg and

Re: [PATCH v5 2/9] ACPI/PPTT: Add Processor Properties Topology Table parsing

2017-12-11 Thread Rafael J. Wysocki
On Friday, December 1, 2017 11:23:23 PM CET Jeremy Linton wrote: > ACPI 6.2 adds a new table, which describes how processing units > are related to each other in tree like fashion. Caches are > also sprinkled throughout the tree and describe the properties > of the caches in relation to other cache

[PATCH v7 2/3] dt-bindings: display: rockchip: update DSI controller

2017-12-11 Thread Nickey Yang
This patch update describe panel/port links, including unit addresses in documentation of device tree bindings for the rockchip DSI controller based on the Synopsys DesignWare MIPI DSI host controller. Signed-off-by: Nickey Yang Reviewed-by: Brian Norris --- .../display/rockchip/dw_mipi_dsi_roc

[PATCH v7 0/3] Update ROCKCHIP DSI driver that uses dw-mipi-dsi bridge

2017-12-11 Thread Nickey Yang
We now have a generic dw-mipi-dsi bridge driver.So we send this patchs to moving rockchip dw-mipi-dsi driver to that in order to add new features(dual mipi support). Update ROCKCHIP DSI controller driver that uses the Synopsys DesignWare MIPI DSI host controller bridge. ChangeLog: v2: add err_p

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

2017-12-11 Thread Nickey Yang
From: Brian Norris Bridge drivers/helpers shouldn't be clobbering the drvdata, since a parent driver might need to own this. Instead, let's return our 'dw_mipi_dsi' object and have callers pass that back to us for removal. Signed-off-by: Brian Norris Signed-off-by: Nickey Yang Link:https://pat

Re: [PATCH v5 4/9] drivers: base cacheinfo: Add support for ACPI based firmware tables

2017-12-11 Thread Rafael J. Wysocki
On Friday, December 1, 2017 11:23:25 PM CET Jeremy Linton wrote: > Add a entry to to struct cacheinfo to maintain a reference to the PPTT > node which can be used to match identical caches across cores. Also > stub out cache_setup_acpi() so that individual architectures can > enable ACPI topology p

[PATCH v7 3/3] drm/rockchip: Add ROCKCHIP DW MIPI DSI controller driver

2017-12-11 Thread Nickey Yang
Add the ROCKCHIP DSI controller driver that uses the Synopsys DesignWare MIPI DSI host controller bridge. Signed-off-by: Nickey Yang Signed-off-by: Brian Norris Reviewed-by: Brian Norris Reviewed-by: Sean Paul --- changes: v2: add err_pllref, remove unnecessary encoder.enable & disable

Re: [PATCH v5 6/9] ACPI/PPTT: Add topology parsing code

2017-12-11 Thread Rafael J. Wysocki
On Friday, December 1, 2017 11:23:27 PM CET Jeremy Linton wrote: > The PPTT can be used to determine the groupings of CPU's at > given levels in the system. Lets add a few routines to the PPTT > parsing code to return a unique id for each unique level in the > processor hierarchy. This can then be

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

2017-12-11 Thread Huang, Ying
Hi, Pual, "Paul E. McKenney" writes: > On Mon, Dec 11, 2017 at 01:30:03PM +0800, Huang, Ying wrote: >> Andrew Morton writes: >> >> > On Fri, 08 Dec 2017 16:41:38 +0800 "Huang\, Ying" >> > wrote: >> > >> >> > Why do we need srcu here? Is it enough with rcu like below? >> >> > >> >> > It might

Re: [PATCH v1 4/4] arm64: dts: mediatek: add mt2712 cpufreq related device nodes

2017-12-11 Thread Rafael J. Wysocki
On Monday, December 11, 2017 8:57:19 AM CET Viresh Kumar wrote: > On 08-12-17, 14:07, Andrew-sh Cheng wrote: > > Add opp v2 information, > > and also add clocks, regulators and opp information into cpu nodes > > > > Signed-off-by: Andrew-sh Cheng > > --- > > arch/arm64/boot/dts/mediatek/mt2712-e

Re: [PATCH] cgroup: limit max copied char length for cgroup file name

2017-12-11 Thread Ma Shimiao
On 12/11/2017 11:05 PM, Tejun Heo wrote: > On Sat, Dec 09, 2017 at 01:47:58PM +0800, Ma Shimiao wrote: >> the result of cgroup_file_name will be used by kernfs_remove_name, >> and then by kernfs_remove_by_name_ns(). >> If reached the max length, may have problem printed by WARN() in >> kernfs_remov

[BUG] sound/soc/samsung: a possible sleep-in-atomic bugs in s3c_pcm_hw_params

2017-12-11 Thread Jia-Ju Bai
According to sound/soc/samsung/pcm.c, the kernel module may sleep under a spinlock. The function call path is: s3c_pcm_hw_params (acquire the spinlock) clk_get_rate clk_core_get_rate clk_prepare_lock mutex_lock --> may sleep I do not find a good way to fix it, so I only repor

Re: [PATCH 4.4 13/16] ocfs2: should wait dio before inode lock in ocfs2_setattr()

2017-12-11 Thread alex chen
On 2017/12/8 18:04, Changwei Ge wrote: > On 2017/12/8 14:21, alex chen wrote: >> >> >> On 2017/12/8 13:36, Ben Hutchings wrote: >>> On Fri, 2017-12-08 at 12:03 +0800, alex chen wrote: On 2017/12/8 10:26, Ben Hutchings wrote: > On Fri, 2017-12-08 at 08:39 +0800, alex chen wrote:

[PATCH v2] igb: Free IRQs when device is hotplugged

2017-12-11 Thread Lyude Paul
Recently I got a Caldigit TS3 Thunderbolt 3 dock, and noticed that upon hotplugging my kernel would immediately crash due to igb: [ 680.825801] kernel BUG at drivers/pci/msi.c:352! [ 680.828388] invalid opcode: [#1] SMP [ 680.829194] Modules linked in: igb(O) thunderbolt i2c_algo_bit joyde

[BUG] drivers/tty/hvc: a possible sleep-in-atomic bug in xen_pv_console_init

2017-12-11 Thread Jia-Ju Bai
According to drivers/tty/hvc/hvc_xen.c, the kernel module may sleep under a spinlock. The function call path is: xen_pv_console_init (acquire the spinlock) xencons_info_pv_init gfn_to_virt (gfn_to_pfn) --> may sleep I do not find a good way to fix it, so I only report. This possible bug is

Re: [PATCH v6 3/3] drm/rockchip: Add ROCKCHIP DW MIPI DSI controller driver

2017-12-11 Thread Nickey Yang
Hi Brian, On 2017年12月07日 05:52, Brian Norris wrote: Hi Nickey, others, I just want to highlight a thing or two here. Otherwise, my 'Reviewed-by' still basically stands (FWIW). On Wed, Dec 06, 2017 at 05:08:21PM +0800, Nickey Yang wrote: Add the ROCKCHIP DSI controller driver that uses the Sy

[PATCH] cgroup: avoid cpoied names in cgroup longer than max

2017-12-11 Thread Ma Shimiao
cgroup root name and file name have max length limit, we should avoid copying longer name than that to the name. Signed-off-by: Ma Shimiao --- kernel/cgroup/cgroup.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/kernel/cgroup/cgroup.c b/kernel/cgroup/cgroup.c index 0b

Re: [RESEND PATCH] arm64: v8.4: Support for new floating point multiplication variant

2017-12-11 Thread gengdongjiu
On 2017/12/11 21:29, Dave Martin wrote: >> Thanks for the point out. >> In fact, this feature only adds two instructions: >> FP16 * FP16 + FP32 >> FP16 * FP16 - FP32 >> >> The spec call this bit to ID_AA64ISAR0_EL1.FHM, I do not know why it >> will call "FHM", I think call it "FMLXL" may be bette

[BUG] drivers/scsi/advansys: three possible sleep-in-atomic bugs in advansys_interrupt

2017-12-11 Thread Jia-Ju Bai
According to drivers/scsi/advansys.c, the kernel module may sleep in the interrupt handler. The function call paths are: advansys_interrupt (interrupt handler) AdvISR adv_async_callback AdvResetChipAndSB AdvInitAsc38C1600Driver request_firmware --> may sleep

Re: [PATCH v3 4/4] 44x/fsp2: add irq error handlers

2017-12-11 Thread Alistair Popple
On Friday, 8 December 2017 6:41:02 PM AEDT Ivan Mikhaylov wrote: > add irq error handlers for cmu, plb, opb, mcue, conf > with debug information output in case of problems. > > Signed-off-by: Ivan Mikhaylov Acked-by: Alistair Popple > --- > arch/powerpc/platforms/44x/fsp2.c | 198 >

[PATCH] fs/ceph: delete unused ceph_file_layout_is_valid function

2017-12-11 Thread Yaowei Bai
There's no users of ceph_file_layout_is_valid, remove it. Signed-off-by: Yaowei Bai --- include/linux/ceph/ceph_fs.h | 1 - net/ceph/ceph_fs.c | 23 --- 2 files changed, 24 deletions(-) diff --git a/include/linux/ceph/ceph_fs.h b/include/linux/ceph/ceph_fs.h index

[BUG] drivers/scsi/dpt_i2o: a possible sleep-in-atomic bug in adpt_isr

2017-12-11 Thread Jia-Ju Bai
According to drivers/scsi/dpt_i2o.c, the kernel module may sleep in the interrupt handler. The function call path is: adpt_isr (interrupt handler) adpt_send_nop schedule_timeout_uninterruptible --> may sleep A possible fixing is to replace "schedule_timeout_uninterruptible" with "mdelay".

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

2017-12-11 Thread Stephen Rothwell
Hi Johannes, After merging the mac80211-next tree, today's linux-next build (x86_64 allmodconfig) failed like this: drivers/net/wireless/mediatek/mt76/mt76x2_main.c:539:19: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types] .wake_tx_queue = mt76_wake_tx_q

Re: [PATCH v3 17/33] nds32: VDSO support

2017-12-11 Thread Vincent Chen
2017-12-08 20:14 GMT+08:00 Mark Rutland : > On Fri, Dec 08, 2017 at 07:54:42PM +0800, Greentime Hu wrote: >> 2017-12-08 18:21 GMT+08:00 Mark Rutland : >> > On Fri, Dec 08, 2017 at 05:12:00PM +0800, Greentime Hu wrote: >> >> +static int grab_timer_node_info(void) >> >> +{ >> >> + struct device_n

Re: [BUG] fs/f2fs: a possible sleep-in-atomic bug in f2fs_trace_pid

2017-12-11 Thread Chao Yu
Hi Jia-Ju, Thanks for your report. :) On 2017/12/12 8:59, Jia-Ju Bai wrote: > According to fs/f2fs/trace.c, the kernel module may sleep under a spinlock. > The function call path is: > f2fs_trace_pid (acquire the spinlock) >f2fs_radix_tree_insert > cond_resched --> may sleep > > I do no

Re: [PATCH 1/2] mm: NUMA stats code cleanup and enhancement

2017-12-11 Thread kemi
On 2017年12月08日 16:47, Michal Hocko wrote: > On Fri 08-12-17 16:38:46, kemi wrote: >> >> >> On 2017年11月30日 17:45, Michal Hocko wrote: >>> On Thu 30-11-17 17:32:08, kemi wrote: >> >> After thinking about how to optimize our per-node stats more gracefully, >> we may add u64 vm_numa_stat_diff[] in s

Re: [RESEND PATCH] arm64: v8.4: Support for new floating point multiplication variant

2017-12-11 Thread gengdongjiu
On 2017/12/12 2:58, Suzuki K Poulose wrote: > Hi gengdongjiu > > Sorry for the late response. I have a similar patch to add the support for > "FHM", which I was about to post it this week. Suzuki, you are welcome. May be you can not post again to avoid the duplicate review, thanks! > > On 11/12

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

2017-12-11 Thread Stephen Rothwell
Hi Lee, On Wed, 6 Dec 2017 08:58:03 + Lee Jones wrote: > > On Wed, 06 Dec 2017, Stephen Rothwell wrote: > > > On Mon, 4 Dec 2017 09:39:53 + Lee Jones wrote: > > > > > > > I added the following fix patch: > > > > > > > > From: Stephen Rothwell > > > > Date: Mon, 4 Dec 2017 14:10:34

Re: [f2fs-dev] [PATCH 1/2] f2fs: pass down write hints to block layer for bufferd write

2017-12-11 Thread Hyunchul Lee
Hi Chao, On 12/11/2017 10:15 PM, Chao Yu wrote: > Hi Hyunchul, > > On 2017/12/1 16:28, Hyunchul Lee wrote: >> Hi Chao, >> >> On 11/30/2017 04:06 PM, Chao Yu wrote: >>> Hi Hyunchul, >>> >>> On 2017/11/28 8:23, Hyunchul Lee wrote: From: Hyunchul Lee This implements which hint is pas

[BUG] drivers/scsi/dpt_i2o: a possible sleep-in-atomic bug in adpt_i2o_post_this

2017-12-11 Thread Jia-Ju Bai
According to drivers/scsi/dpt_i2o.c, the kernel module may sleep under a spinlock. The function call paths are: adpt_abort (acquire the spinlock) adpt_i2o_post_wait adpt_i2o_post_this schedule_timeout_uninterruptible--> may sleep adpt_device_reset (acquire the spinlock) adpt_i2o_po

[BUG] drivers/scsi/dpt_i2o: a possible sleep-in-atomic bug in adpt_i2o_status_get

2017-12-11 Thread Jia-Ju Bai
According to drivers/scsi/dpt_i2o.c, the kernel module may sleep under a spinlock. The function call paths are: adpt_reset (acquire the spinlock) __adpt_reset adpt_hba_reset adpt_i2o_activate_hba adpt_i2o_status_get dma_alloc_coherent(GFP_KERNEL) --> may sleep

Re: [PATCH net-next] libbpf: add function to setup XDP

2017-12-11 Thread Daniel Borkmann
On 12/10/2017 10:07 PM, David Ahern wrote: > On 12/10/17 1:34 PM, Eric Leblond wrote: >>> Would it be possible to print out or preferably return to the caller >>> the ext ack error message? A couple of drivers are using it for XDP >>> mis-configuration reporting instead of printks. We should enco

Re: [PATCH net-next v5 2/2] net: ethernet: socionext: add AVE ethernet driver

2017-12-11 Thread Masami Hiramatsu
Hi Russell, 2017-12-11 22:46 GMT+09:00 Russell King - ARM Linux : > On Mon, Dec 11, 2017 at 10:34:17PM +0900, Masami Hiramatsu wrote: >> IMHO, even if we use SPDX license identifier, I recommend to use >> C-style comments as many other files do, since it is C code. >> If SPDX identifier requires C

Dear Friend,

2017-12-11 Thread Mr Akpala Maiky
Dear Friend, I am the head of Accounts and Audit Department of Bank of Africa, Ouagadougou . I decided to contact you after a careful thought that you may be capable of handling this business transaction which I explained below; In my department, I discovered an abandoned sum of $13.5m US dollars

Re: [PATCH][next] scsi: arcmsr: remove redundant check for secs < 0

2017-12-11 Thread Martin K. Petersen
Colin, > The check for secs being less than zero is redundant for two reasons. > Firstly, secs is unsigned so the check is always going to be false. > Secondly, if secs was signed the proceeding calculation of secs is > never going to be negative. Hence we can remove this redundant check > and d

Re: [PATCH] scsi: bfa: convert to strlcpy/strlcat

2017-12-11 Thread Martin K. Petersen
Arnd, > The bfa driver has a number of real issues with string termination > that gcc-8 now points out: Applied to 4.16/scsi-queue. Thank you! -- Martin K. Petersen Oracle Linux Engineering

Re: [PATCH 13/13] arc: do not use __print_symbol()

2017-12-11 Thread Sergey Senozhatsky
On (12/11/17 08:28), Vineet Gupta wrote: > On 12/11/2017 04:53 AM, Sergey Senozhatsky wrote: > > __print_symbol() uses extra stack space to sprintf() symbol > > information and then to feed that buffer to printk() > > > >char buffer[KSYM_SYMBOL_LEN]; > > > >sprint_symbol(buffer, address);

Re: [PATCH 08/13] x86: do not use print_symbol()

2017-12-11 Thread Sergey Senozhatsky
On (12/11/17 18:45), Borislav Petkov wrote: > For the mce.c bit above: > > Acked-by: Borislav Petkov thanks. -ss

Re: [PATCH 1/1] scsi: fnic: add a space after %p in printf format

2017-12-11 Thread Martin K. Petersen
Nicolas, > fnic_fcpio_icmnd_cmpl_handler() displays the value of sc with: > As the literal strings get merged, the function uses %ps instead of the > intended raw %p format. Fix this by inserting a space. Applied to 4.16/scsi-queue. Thank you! -- Martin K. Petersen Oracle Linux Engineeri

Re: [f2fs-dev] [PATCH 1/2] f2fs: pass down write hints to block layer for bufferd write

2017-12-11 Thread Chao Yu
Hi Hyunchul, On 2017/12/12 10:15, Hyunchul Lee wrote: > Hi Chao, > > On 12/11/2017 10:15 PM, Chao Yu wrote: >> Hi Hyunchul, >> >> On 2017/12/1 16:28, Hyunchul Lee wrote: >>> Hi Chao, >>> >>> On 11/30/2017 04:06 PM, Chao Yu wrote: Hi Hyunchul, On 2017/11/28 8:23, Hyunchul Lee wrote:

Re: [PATCH 00/13] replace print_symbol() with printk()-s

2017-12-11 Thread Sergey Senozhatsky
On (12/11/17 08:26), Joe Perches wrote: > On Mon, 2017-12-11 at 21:50 +0900, Sergey Senozhatsky wrote: > > print_symbol > > Yay. > > Just about exactly 5 years earlier... > http://lists.infradead.org/pipermail/linux-arm-kernel/2012-December/137121.html indeed :) hopefully it won't take us anoth

[PATCH] NCR5380: Fix a possible sleep-in-atomic bug in NCR5380_poll_politely2

2017-12-11 Thread Jia-Ju Bai
From: Jia-Ju Bai The kernel module may sleep under a spinlock. The function call paths are: NCR5380_select (acquire the spinlock) NCR5380_reselect NCR5380_poll_politely NCR5380_poll_politely2 schedule_timeout_uninterruptible --> may sleep NCR5380_abort (acquire the spinlock)

Re: [PATCH] MAINTIANERS: change FCoE list to linux-scsi

2017-12-11 Thread Martin K. Petersen
Johannes, > fcoe-de...@open-fcoe.org is defunct and all patches are routed via the > SCSI tree anyways. Applied to 4.15/scsi-fixes. Thank yo! -- Martin K. Petersen Oracle Linux Engineering

Re: [PATCH 11/13] irq debug: do not use print_symbol()

2017-12-11 Thread Sergey Senozhatsky
On (12/11/17 12:55), David Laight wrote: > > kernel/irq/debug.h | 8 +++- > > 1 file changed, 3 insertions(+), 5 deletions(-) > > > > diff --git a/kernel/irq/debug.h b/kernel/irq/debug.h > > index 17f05ef8f575..5766e15c1160 100644 > > --- a/kernel/irq/debug.h > > +++ b/kernel/irq/debug.h > ..

Re: [PATCH 7/9] workqueue: remove unneeded kallsyms include

2017-12-11 Thread Sergey Senozhatsky
On (12/11/17 07:16), Tejun Heo wrote: > On Fri, Dec 08, 2017 at 11:56:14AM +0900, Sergey Senozhatsky wrote: > > The filw was converted from print_symbol() to %pf some time > > ago (044c782ce3a901fb "workqueue: fix checkpatch issues"). > > kallsyms does not seem to be needed anymore. > > > > Signed

Re: [PATCH 18/45] drivers: scsi: qla2xxx: remove duplicate includes

2017-12-11 Thread Martin K. Petersen
Pravin, > These duplicate includes have been found with scripts/checkincludes.pl but > they have been removed manually to avoid removing false positives. Applied to 4.16/scsi-queue. Thank you! -- Martin K. Petersen Oracle Linux Engineering

Re: [PATCH v2 2/2] drm/tinydrm: add driver for ST7735R panels

2017-12-11 Thread David Lechner
On 12/11/2017 03:18 PM, Noralf Trønnes wrote: Den 10.12.2017 23.10, skrev David Lechner: This adds a new driver for Sitronix ST7735R display panels. This has been tested using an Adafruit 1.8" TFT. Signed-off-by: David Lechner --- +static void st7735r_pipe_enable(struct drm_simple_display

[PATCH 0/8] make some functions return bool

2017-12-11 Thread Yaowei Bai
This patchset makes some *_is_* like functions return bool because these functions only use true or false as their return values. No functional changes. Yaowei Bai (8): mm/memblock: memblock_is_map/region_memory can be boolean mm/mmzone: mm/lru: is_file/active_lru can be boolean lib/lockref

[PATCH 3/8] lib/lockref: __lockref_is_dead can be boolean

2017-12-11 Thread Yaowei Bai
This patch makes __lockref_is_dead return bool due to this function only using either true or false as its return value. No functional change. Signed-off-by: Yaowei Bai --- include/linux/lockref.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/lockref.h b/incl

[PATCH 2/8] mm/mmzone: mm/lru: is_file/active_lru can be boolean

2017-12-11 Thread Yaowei Bai
This patch makes is_file/active_lru return bool due to these two functions only using either true or false as its return value. No functional change. Signed-off-by: Yaowei Bai --- include/linux/mmzone.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/mmzone

[PATCH 7/8] kernel/mutex: mutex_is_locked can be boolean

2017-12-11 Thread Yaowei Bai
This patch makes mutex_is_locked return bool due to this particular function only using either one or zero as its return value. No functional change. Signed-off-by: Yaowei Bai --- include/linux/mutex.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/mutex.h

Re: [PATCH v1 2/2] usb: chipidea: tegra: Select Tegra's PHY in Kconfig

2017-12-11 Thread Peter Chen
On Mon, Dec 11, 2017 at 04:09:44PM +0300, Dmitry Osipenko wrote: > On 11.12.2017 13:04, Thierry Reding wrote: > > On Mon, Dec 11, 2017 at 02:10:00AM +0300, Dmitry Osipenko wrote: > >> UDC driver won't probe without Tegra's PHY, hence select it in the > >> Kconfig. > >> > >> Signed-off-by: Dmitry Os

[PATCH 8/8] crash_dump: is_kdump_kernel can be boolean

2017-12-11 Thread Yaowei Bai
This patch makes is_kdump_kernel return bool due to this particular function only using either one or zero as its return value. No functional change. Signed-off-by: Yaowei Bai --- include/linux/crash_dump.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/include/l

[PATCH 4/8] kernel/cpuset: current_cpuset_is_being_rebound can be boolean

2017-12-11 Thread Yaowei Bai
This patch makes current_cpuset_is_being_rebound return bool due to this particular function only using either one or zero as its return value. No functional change. Signed-off-by: Yaowei Bai --- include/linux/cpuset.h | 6 +++--- kernel/cgroup/cpuset.c | 4 ++-- 2 files changed, 5 insertions(+

[PATCH 1/8] mm/memblock: memblock_is_map/region_memory can be boolean

2017-12-11 Thread Yaowei Bai
This patch makes memblock_is_map/region_memory return bool due to these two functions only using either true or false as its return value. No functional change. Signed-off-by: Yaowei Bai --- include/linux/memblock.h | 4 ++-- mm/memblock.c| 6 +++--- 2 files changed, 5 insertions(+)

[PATCH 6/8] kernel/module: module_is_live can be boolean

2017-12-11 Thread Yaowei Bai
This patch makes module_is_live return bool due to this particular function only using either one or zero as its return value. No functional change. Signed-off-by: Yaowei Bai --- include/linux/module.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/module.h b/

[PATCH 5/8] kernel/resource: iomem_is_exclusive can be boolean

2017-12-11 Thread Yaowei Bai
This patch makes iomem_is_exclusive return bool due to this particular function only using either one or zero as its return value. No functional change. Signed-off-by: Yaowei Bai --- include/linux/ioport.h | 2 +- kernel/resource.c | 10 +- 2 files changed, 6 insertions(+), 6 dele

[BUG] drivers/scsi/ipr: two possible sleep-in-atomic bugs

2017-12-11 Thread Jia-Ju Bai
According to drivers/scsi/ipr.c, the kernel module may sleep under a spinlock. The function call paths are: ipr_shutdown (acquire the spinlock) irq_poll_disable msleep --> may sleep ipr_ata_post_internal (acquire the spinlock) ipr_device_reset ipr_send_blocking_cmd wait_for_com

Re: [PATCH v2] perf test shell: Fix check open filename arg using 'perf trace

2017-12-11 Thread Michael Petlan
On Mon, 11 Dec 2017, Arnaldo Carvalho de Melo wrote: > It is not working here: Hmm, I think I got it. The following construction: evts=`perf list syscalls:sys_enter_open* |& egrep 'open(at)? ' | sed -r 's/.*sys_enter_(.*) +\[.*/-e \1/'` ... expands to: "-e open -e openat" ... which is a form

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

2017-12-11 Thread Smitha T Murthy
On Sat, 2017-12-09 at 20:48 +0200, Stanimir Varbanov wrote: > Hi Smitha, > > Thanks for the patches! > > On 12/08/2017 11:08 AM, Smitha T Murthy wrote: > > Add v4l2 controls for HEVC encoder > > > > Signed-off-by: Smitha T Murthy > > Reviewed-by: Andrzej Hajda > > --- > > drivers/media/v4l2-c

Re: [PATCH] USB: chipidea: msm: fix ulpi-node lookup

2017-12-11 Thread Peter Chen
On Mon, Nov 13, 2017 at 11:12:58AM +0100, Johan Hovold wrote: > Fix child-node lookup during probe, which ended up searching the whole > device tree depth-first starting at the parent rather than just matching > on its children. > > Note that the original premature free of the parent node has alre

Re: [PATCH 00/13] replace print_symbol() with printk()-s

2017-12-11 Thread Joe Perches
On Tue, 2017-12-12 at 11:47 +0900, Sergey Senozhatsky wrote: > On (12/11/17 08:26), Joe Perches wrote: > > On Mon, 2017-12-11 at 21:50 +0900, Sergey Senozhatsky wrote: > > > print_symbol > > Yay. > > Just about exactly 5 years earlier... > > http://lists.infradead.org/pipermail/linux-arm-kernel/201

[BUG] drivers/scsi/wd719x: a possible sleep-in-atomic bug in wd719x_host_reset

2017-12-11 Thread Jia-Ju Bai
According to drivers/scsi/wd719x.c, the kernel module may sleep under a spinlock. The function call path is: wd719x_host_reset (acquire the spinlock) wd719x_chip_init request_firmware --> may sleep I do not find a good way to fix it, so I only report. This possible bug is found by my stati

Re: a racy access flag clearing warning when calling mmap system call

2017-12-11 Thread chenjiankang
> On Fri, Dec 08, 2017 at 11:19:52AM +0800, chenjiankang wrote: >> 在 2017/12/7 21:23, Will Deacon 写道: >>> diff --git a/arch/arm64/include/asm/pgtable.h >>> b/arch/arm64/include/asm/pgtable.h >>> index 149d05fb9421..8fe103b1e101 100644 >>> --- a/arch/arm64/include/asm/pgtable.h >>> +++ b/arch/arm6

[BUG] drivers/usb/host/isp116x-hcd: a possible sleep-in-atomic bug in isp116x_start

2017-12-11 Thread Jia-Ju Bai
According to drivers/usb/host/isp116x-hcd.c, the kernel module may sleep under a spinlock. The function call path is: isp116x_start (acquire the spinlock) device_init_wakeup device_wakeup_enable wakeup_source_register wakeup_source_create kmalloc(GFP_KERNEL) --> may

Setting large MTU size on slave interfaces may stall the whole system

2017-12-11 Thread Qing Huang
(resend this email in text format) Hi, We found an issue with the bonding driver when testing Mellanox devices. The following test commands will stall the whole system sometimes, with serial console flooded with log messages from the bond_miimon_inspect() function. Setting mtu size to be 150

Re: [RESEND PATCH V2] arm64: fault: avoid send SIGBUS two times

2017-12-11 Thread Xie XiuQi
Hi Dongjiu, On 2017/12/12 0:05, Dongjiu Geng wrote: > do_sea() calls arm64_notify_die() which will always signal > user-space. It also returns whether APEI claimed the external > abort as a RAS notification. If it returns failure do_mem_abort() > will signal user-space too. > > do_mem_abort() wan

[PATCH] fix memory leak in ramoops_init

2017-12-11 Thread nixiaoming
1, memory leak in ramoops_register_dummy. dummy_data = kzalloc(sizeof(*dummy_data), GFP_KERNEL); but no free when platform_device_register_data return fail 2, if kzalloc(sizeof(*dummy_data), GFP_KERNEL) return NULL, but platform_driver_register(&ramoops_driver) return 0 kfree(NULL) in

[BUG] 3com/3c59x: two possible sleep-in-atomic bugs

2017-12-11 Thread Jia-Ju Bai
According to drivers/net/ethernet/3com/3c59x.c, the kernel module may sleep in the interrupt handler. The function call paths are: boomerang_interrupt (interrupt handler) vortex_error vortex_up pci_set_power_state --> may sleep pci_enable_device --> may sleep vortex_interrupt (

Re: [PATCH RESEND] KVM: X86: Fix load bad host fpu state

2017-12-11 Thread Peter Xu
On Tue, Dec 12, 2017 at 05:51:26AM +0800, Wanpeng Li wrote: > 2017-12-12 4:48 GMT+08:00 David Hildenbrand : > > On 10.12.2017 22:44, Wanpeng Li wrote: > >> From: Wanpeng Li > >> > >> [ cut here ] > >> Bad FPU state detected at kvm_put_guest_fpu+0xd8/0x2d0 [kvm], > >> rein

linux-next: build warning after merge of the char-misc tree

2017-12-11 Thread Stephen Rothwell
Hi all, After merging the char-misc tree, today's linux-next build (x86_64_allmodconfig) produced this warning: drivers/misc/mic/vop/vop_vringh.c: In function 'vop_ioctl': drivers/misc/mic/vop/vop_vringh.c:1001:1: warning: label 'done' defined but not used [-Wunused-label] done: ^ Introduced

Re: [PATCH] NCR5380: Fix a possible sleep-in-atomic bug in NCR5380_poll_politely2

2017-12-11 Thread Finn Thain
On Tue, 12 Dec 2017, Jia-Ju Bai wrote: > From: Jia-Ju Bai > > The kernel module may sleep under a spinlock. The spinlock is always taken in irq mode, and the schedule_timeout_uninterruptible() is conditional on !irqs_disabled(). > The function call paths are: > NCR5380_select (acquire the spi

Re: [PATCH] NCR5380: Fix a possible sleep-in-atomic bug in NCR5380_poll_politely2

2017-12-11 Thread Jia-Ju Bai
Thanks for your reply :) On 2017/12/12 11:38, Finn Thain wrote: On Tue, 12 Dec 2017, Jia-Ju Bai wrote: From: Jia-Ju Bai The kernel module may sleep under a spinlock. The spinlock is always taken in irq mode, and the schedule_timeout_uninterruptible() is conditional on !irqs_disabled(). I

[BUG] drivers/gpio: some possible sleep-in-atomic bugs

2017-12-11 Thread Jia-Ju Bai
According to drivers/gpio/gpio-grgpio.c, the kernel module may sleep under a spinlock. The function call paths are: grgpio_remove (acquire the spinlock) gpiochip_remove gpiochip_sysfs_unregister device_unregister --> may sleep mutex_lock --> may sleep grgpio_remove (acquire the

Re: [RFC][PATCH] new byteorder primitives - ..._{replace,get}_bits()

2017-12-11 Thread Jakub Kicinski
On Mon, 11 Dec 2017 15:54:22 +, Al Viro wrote: > Essentially, it gives helpers for work with bitfields in fixed-endian. > Suppose we have e.g. a little-endian 32bit value with fixed layout; > expressing that as a bitfield would go like > struct foo { > unsigned foo:4;

[PATCH] ARM: dts: omap3-evm: Fix missing NAND partition information

2017-12-11 Thread Derald D. Woods
The partition information was omitted during the conversion to OMAP3430 specific data. The data added by this commit is consistent with current U-Boot default definitions. Fixes: 62fe1d337461 ("ARM: dts: omap3-evm: Add OMAP3530 specific device tree processor data") Signed-off-by: Derald D. Woods

RE: Re: ftrace/module: Move ftrace_release_mod() to ddebug_cleanup label

2017-12-11 Thread AMIT SAHRAWAT
Hi Steven, >> ftrace_module_init happen after dynamic_debug_setup, it is desired that >> cleanup should be called after this label however in current implementation >> it is called in free module label,ie:even though ftrace in not initialized, >> from so many fail case ftrace_release_mod() will be

Re: [RESEND PATCH V2] arm64: fault: avoid send SIGBUS two times

2017-12-11 Thread gengdongjiu
On 2017/12/12 11:31, Xie XiuQi wrote: >> +return 0; > It looks good to me. do_sea() has done all necessary action for SEA, so it > should always return 0, > no matter ghes_notify_sea() return true or false. yes, it is. > > Reviewed-by: Xie XiuQi Thanks XiuQi's review and comments. > >>

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

2017-12-11 Thread Katsuhiro Suzuki
Hello Vinod, Mark, > -Original Message- > From: Vinod Koul [mailto:vinod.k...@intel.com] > Sent: Tuesday, December 12, 2017 2:49 AM > To: Mark Brown > Cc: Suzuki, Katsuhiro/鈴木 勝博 ; > devicet...@vger.kernel.org; alsa-de...@alsa-project.org; Masami Hiramatsu > ; Yamada, Masahiro/山田 真弘 > ; l

RE: [RESEND PATCH 1/2 v2] staging: fsl-mc: Allocate IRQ's before scanning DPRC objects

2017-12-11 Thread Bharat Bhushan
> -Original Message- > From: Nipun Gupta [mailto:nipun.gu...@nxp.com] > Sent: Monday, December 11, 2017 9:16 PM > To: Laurentiu Tudor ; stuyo...@gmail.com; > Bharat Bhushan ; gre...@linuxfoundation.org; > cakt...@gmail.com; bretth...@gmail.com; a...@arndb.de > Cc: linux-kernel@vger.kernel

RE: [PATCH v2 1/5] dt-bindings: rtc: add bindings for i.MX53 SRTC

2017-12-11 Thread Patrick Brünn
>From: Fabio Estevam [mailto:feste...@gmail.com] >Sent: Dienstag, 12. Dezember 2017 00:08 >Hi Patrick, > Hi Fabio, >On Mon, Dec 11, 2017 at 5:08 AM, Patrick Brünn >wrote: > >>>rtc@... >>> >> The rtc for which this series adds support is embedded within a function >block called >> "Secure Real Time

linux-next: Tree for Dec 12

2017-12-11 Thread Stephen Rothwell
Hi all, Changes since 20171211: The clk tree still had its build failures so I used the version from next-20171208. The net-next tree gained a conflict against the net tree. The mac80211-next tree gained a build failure due to an interaction with the wireless-drivers-next tree for which I

[PATCH] KVM/Emulate: Mask linear address with actual address width in order to avoid conflict with UNMAPPED_GVA

2017-12-11 Thread Lan Tianyu
Reported by syzkaller: WARNING: CPU: 0 PID: 27962 at arch/x86/kvm/emulate.c:5631 x86_emulate_insn+0x557/0x15f0 [kvm] Modules linked in: kvm_intel kvm [last unloaded: kvm] CPU: 0 PID: 27962 Comm: syz-executor Tainted: GB W 4.15.0-rc2-next-20171208+ #32 Hardware na

Re: [PATCH v4] printk: Add console owner and waiter logic to load balance console writes

2017-12-11 Thread Sergey Senozhatsky
Hello, On (12/08/17 15:00), Petr Mladek wrote: [..] > > However, now that cross-release was introduces, lockdep can be applied > > to semaphore operations. Actually, I have a plan to do that. I think it > > would be better to make semaphore tracked with lockdep and remove all > > these manual acqu

Re: [PATCH RESEND] KVM: X86: Fix load bad host fpu state

2017-12-11 Thread Wanpeng Li
2017-12-12 11:36 GMT+08:00 Peter Xu : > On Tue, Dec 12, 2017 at 05:51:26AM +0800, Wanpeng Li wrote: >> 2017-12-12 4:48 GMT+08:00 David Hildenbrand : >> > On 10.12.2017 22:44, Wanpeng Li wrote: >> >> From: Wanpeng Li >> >> >> >> [ cut here ] >> >> Bad FPU state detected at

Re: [PATCH 0/8] make some functions return bool

2017-12-11 Thread David Rientjes
On Mon, 11 Dec 2017, Yaowei Bai wrote: > This patchset makes some *_is_* like functions return bool because > these functions only use true or false as their return values. > > No functional changes. > I think the concern about this type of patchset in the past is that it is unnecessary churn

[PATCH] KVM: X86: Fix stack-out-of-bounds read in write_mmio

2017-12-11 Thread Wanpeng Li
From: Wanpeng Li Reported by syzkaller: BUG: KASAN: stack-out-of-bounds in write_mmio+0x11e/0x270 [kvm] Read of size 8 at addr 8803259df7f8 by task syz-executor/32298 CPU: 6 PID: 32298 Comm: syz-executor Tainted: G OE4.15.0-rc2+ #18 Hardware name: LENOVO ThinkCentre

[PATCH] arm64: allwinner: a64: a64-olinuxino: add usb otg

2017-12-11 Thread Jagan Teki
Add usb otg support for a64-olinuxino board, - USB0-ID connected with PH9 - USB0-VBUSDET connected with PH6 - USB-DRVVBUS controlled by N_VBUSEN pin from PMIC Signed-off-by: Jagan Teki --- .../boot/dts/allwinner/sun50i-a64-olinuxino.dts| 26 ++ 1 file changed, 26 insertio

Re: [PATCH] writeback: synchronize sync(2) against cgroup writeback membership switches

2017-12-11 Thread xuejiufei
Hi Tejun, On 2017/12/12 上午3:50, Tejun Heo wrote: > sync_inodes_sb() can race against cgwb (cgroup writeback) membership > switches and fail to writeback some inodes. For example, if an inode > switches to another wb while sync_inodes_sb() is in progress, the new > wb might not be visible to bdi_s

[PATCH] crypto: AF_ALG - limit mask and type

2017-12-11 Thread Stephan Müller
Hi Herbert, you see the reported problem by simply using sa.salg_mask = 0x; Note, I am not fully sure about whether CRYPTO_AF_ALG_ALLOWED_MASK and CRYPTO_AF_ALG_ALLOWED_TYPE have the correct value. But I think that all that user space should reach is potentially the ASYNC flag and the ci

Re: [PATCH] media: v4l: xilinx: Use SPDX-License-Identifier

2017-12-11 Thread Dhaval Shah
Hi Laurent Pinchart, Thanks a lot for the review. On Mon, Dec 11, 2017 at 7:17 PM, Laurent Pinchart wrote: > > Hi Dhaval, > > Thank you for the patch. > > On Friday, 8 December 2017 14:35:37 EET Dhaval Shah wrote: > > SPDX-License-Identifier is used for the Xilinx Video IP and > > related driver

[PATCH v5 3/3] dt-bindings: timer: Add andestech atcpit100 timer binding doc

2017-12-11 Thread Rick Chen
Add a document to describe Andestech atcpit100 timer and binding information. Signed-off-by: Rick Chen Signed-off-by: Greentime Hu Acked-by: Rob Herring --- .../bindings/timer/andestech,atcpit100-timer.txt | 33 ++ 1 file changed, 33 insertions(+) create mode 100644 Doc

[PATCH v5 0/3] Add andestech atcpit100 timer

2017-12-11 Thread Rick Chen
Changelog v5: - Patch 1/3: Changes - Patch 2/3: New - Patch 3/3: Changes [Patch 1/3] clocksource/drivers/atcpit100: Add andestech atcpit100 timer 1 No need to split out the Makefile patch from the actual driver. Suggested by Arnd Bergmann 2 Add of_clk.name = "PCLK" to be explicit on

[PATCH] f2fs: fix potential hangtask in f2fs_trace_pid

2017-12-11 Thread Chao Yu
As Jia-Ju Bai reported: "According to fs/f2fs/trace.c, the kernel module may sleep under a spinlock. The function call path is: f2fs_trace_pid (acquire the spinlock) f2fs_radix_tree_insert cond_resched --> may sleep I do not find a good way to fix it, so I only report. This possible bug i

[PATCH v5 2/3] clocksource/drivers/atcpit100: VDSO support

2017-12-11 Thread Rick Chen
VDSO needs real-time cycle count to ensure the time accuracy. Unlike others, nds32 architecture does not define clock source, hence VDSO needs atcpit100 offering real-time cycle count to derive the correct time. Signed-off-by: Vincent Chen Signed-off-by: Rick Chen Signed-off-by: Greentime Hu --

[PATCH v5 1/3] clocksource/drivers/atcpit100: Add andestech atcpit100 timer

2017-12-11 Thread Rick Chen
ATCPIT100 is often used on the Andes architecture, This timer provide 4 PIT channels. Each PIT channel is a multi-function timer, can be configured as 32,16,8 bit timers or PWM as well. For system timer it will set channel 1 32-bit timer0 as clock source and count downwards until underflow and res

<    1   2   3   4   5   6   7   8   9   >