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

2017-12-12 Thread Greg Kroah-Hartman
On Wed, Dec 13, 2017 at 07:48:43AM +0100, Marek Szyprowski wrote: > Hi Shuah and Greg, > > On 2017-12-12 15:47, Shuah Khan wrote: > > On 12/12/2017 05:43 AM, Greg Kroah-Hartman wrote: > > > This is the start of the stable review cycle for the 4.14.6 release. > > > There are 164 patches in this

Re: [rtc-linux] Re: [PATCH v4 0/2] Add support for cros-ec-rtc driver.

2017-12-12 Thread Lee Jones
On Tue, 12 Dec 2017, Alexandre Belloni wrote: > On 12/12/2017 at 17:05:37 +0100, Alexandre Belloni wrote: > > On 12/12/2017 at 16:41:39 +0100, Enric Balletbo Serra wrote: > > > + Alessandro Zummo > > > + Alexandre Belloni > > > > > > 2017-11-10 22:55 GMT+01:00 Enric Balletbo i Serra > > >

Re: [PATCH] cgroup/cpuset: fix circular locking dependency

2017-12-12 Thread Prateek Sood
On 12/11/2017 08:50 PM, Tejun Heo wrote: > Hello, Peter. > > On Tue, Dec 05, 2017 at 12:01:17AM +0100, Peter Zijlstra wrote: >>> AFAICS, this should remove the circular dependency you originally >>> reported. I'll revert the two cpuset commits for now. >> >> So I liked his patches in that we

[PATCH 1/1] arm: sunxi: Add alternative pins for spi0

2017-12-12 Thread Stefan Mavrodiev
Allwinner A10/A13/A20 SoCs have pinmux for spi0 on port C. The patch adds these pins in the respective dts includes. Signed-off-by: Stefan Mavrodiev --- arch/arm/boot/dts/sun4i-a10.dtsi | 10 ++ arch/arm/boot/dts/sun5i.dtsi | 10 ++

Re: [BUG] skge: a possible sleep-in-atomic bug in skge_remove

2017-12-12 Thread Jia-Ju Bai
On 2017/12/13 13:18, Stephen Hemminger wrote: On Tue, 12 Dec 2017 20:57:01 -0500 (EST) David Miller wrote: From: Stephen Hemminger Date: Tue, 12 Dec 2017 10:22:40 -0800 On Tue, 12 Dec 2017 08:34:45 -0500 (EST) David Miller

Re: [PATCH v2 04/13] dt-bindings: pinctrl: Add bindings for Microsemi Ocelot

2017-12-12 Thread Linus Walleij
On Fri, Dec 8, 2017 at 4:46 PM, Alexandre Belloni wrote: > Add the documentation for the Microsemi Ocelot pinmuxing and gpio > controller. > > Cc: Linus Walleij > Cc: linux-g...@vger.kernel.org > Signed-off-by: Alexandre Belloni

Re: [BUG] scsi/qla2xxx: a possible sleep-in-atomic bug in qlt_get_tag

2017-12-12 Thread Jia-Ju Bai
On 2017/12/13 12:42, James Bottomley wrote: On Wed, 2017-12-13 at 11:18 +0800, Jia-Ju Bai wrote: The driver may sleep under a spinlock. The function call paths are: qlt_handle_abts_recv_work (acquire the spinlock) qlt_response_pkt_all_vps qlt_response_pkt

Re: general protection fault in page_mapping

2017-12-12 Thread Vlastimil Babka
On 12/12/2017 06:03 PM, syzbot wrote: > Hello, > > syzkaller hit the following crash on > 82bcf1def3b5f1251177ad47c44f7e17af039b4b > git://git.cmpxchg.org/linux-mmots.git/master > compiler: gcc (GCC) 7.1.1 20170620 > .config is attached > Raw console output is attached. > C reproducer is

Re: [PATCH v5 8/9] pinctrl: axp209: add support for AXP813 GPIOs

2017-12-12 Thread Linus Walleij
On Fri, Dec 8, 2017 at 2:41 PM, Quentin Schulz wrote: >> - pctl->desc = _data; >> + pctl->desc = (struct axp20x_pctrl_desc *)of_device_get_match_data(dev); >> pctl->regmap = axp20x->regmap; >> pctl->dev = >dev; >> > > I am using pctl->desc

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

2017-12-12 Thread Linus Walleij
On Tue, Dec 12, 2017 at 12:36 AM, Lukasz Majewski wrote: > This patch series adds support for Liebherr's BK3 board, being > a derivative of TS72XX design. All looks good. Acked-by: Linus Walleij Out of curiosity: Liebherr is obviously doing heavy-duty

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

2017-12-12 Thread Krzysztof Kozlowski
On Wed, Dec 13, 2017 at 2:48 AM, Andi Shyti wrote: > Hi Krzysztof, > >> > Here, this should be as a top line: >> > >> > // SPDX-License-Identifier: GPL-2.0+ >> > >> > >> > So I agree with the SPDX id but also pointing to the use of the C++ // >> > comment style as

[PATCH V8 3/7] device property: Introduce a common API to fetch device match data

2017-12-12 Thread Sinan Kaya
There is an OF/ACPI function to obtain the driver data. We want to hide OF/ACPI details from the device drivers and abstract following the device family of functions. Signed-off-by: Sinan Kaya Reviewed-by: Rob Herring Acked-by: Sakari Ailus

[PATCH V8 1/7] Documentation: DT: qcom_hidma: Bump HW revision for the bugfixed HW

2017-12-12 Thread Sinan Kaya
A new version of the HIDMA IP has been released with bug fixes. Bumping the hardware version to differentiate from others. Signed-off-by: Sinan Kaya Reviewed-by: Rob Herring --- Documentation/devicetree/bindings/dma/qcom_hidma_mgmt.txt | 4 ++-- 1 file

Re: [PATCH V2 2/3] perf/x86/intel/bm.c: Add Intel Branch Monitoring support

2017-12-12 Thread Peter Zijlstra
On Tue, Dec 12, 2017 at 03:08:00PM -0800, Megha Dey wrote: > > > > There's work on the way to allow multiple HW PMUs. You'll either have to > > wait for that or help in making that happen. What you do not do is > > silently hack around it. > > Could I get a pointer to the code implementing this?

[PATCH V8 5/7] ACPI: properties: Implement get_match_data() callback

2017-12-12 Thread Sinan Kaya
Now that we have a get_match_data() callback as part of the firmware node, implement the ACPI specific piece for it. Signed-off-by: Sinan Kaya Acked-by: Sakari Ailus --- drivers/acpi/property.c | 8 1 file changed, 8 insertions(+)

[PATCH V8 6/7] dmaengine: qcom_hidma: Add support for the new revision

2017-12-12 Thread Sinan Kaya
Add support for probing the newer HW and also organize MSI capable hardware into an array for maintenance reasons. Signed-off-by: Sinan Kaya --- drivers/dma/qcom/hidma.c | 34 +- 1 file changed, 13 insertions(+), 21 deletions(-) diff --git

[PATCH V8 7/7] dmaengine: qcom_hidma: Add identity register support

2017-12-12 Thread Sinan Kaya
The location for destination event channel register has been relocated from offset 0x28 to 0x40. Update the code accordingly. Signed-off-by: Sinan Kaya --- drivers/dma/qcom/hidma.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git

[PATCH V8 4/7] OF: properties: Implement get_match_data() callback

2017-12-12 Thread Sinan Kaya
Now that we have a get_match_data() callback as part of the firmware node, implement the OF specific piece for it. Signed-off-by: Sinan Kaya Reviewed-by: Rob Herring Acked-by: Sakari Ailus --- drivers/of/property.c | 8

[PATCH V8 2/7] ACPI / bus: Introduce acpi_get_match_data() function

2017-12-12 Thread Sinan Kaya
OF has of_device_get_match_data() function to extract driver specific data structure. Add a similar function for ACPI. Signed-off-by: Sinan Kaya Acked-by: Rafael J. Wysocki --- drivers/acpi/bus.c | 18 ++ include/linux/acpi.h

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

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

Re: [PATCH] iommu/vt-d: Fix shift overflow in qi_flush_dev_iotlb

2017-12-12 Thread Peter Xu
On Tue, Dec 12, 2017 at 03:43:08PM -0700, Alex Williamson wrote: [...] > diff --git a/drivers/iommu/dmar.c b/drivers/iommu/dmar.c > index 9a7ffd13c7f0..87888b102057 100644 > --- a/drivers/iommu/dmar.c > +++ b/drivers/iommu/dmar.c > @@ -1345,7 +1345,9 @@ void qi_flush_dev_iotlb(struct intel_iommu

Re: About the try to remove cross-release feature entirely by Ingo

2017-12-12 Thread Byungchul Park
On Wed, Dec 13, 2017 at 3:24 PM, Byungchul Park wrote: > Lockdep works, based on the following: > >(1) Classifying locks properly >(2) Checking relationship between the classes > > If (1) is not good or (2) is not good, then we > might get false positives. >

[PATCH v3] perf evsel: Enable ignore_missing_thread for pid option

2017-12-12 Thread Mengting Zhang
While monitoring a multithread process with pid option, perf sometimes may return sys_perf_event_open failure with 3(No such process) if any of the process's threads die before we open the event. However, we want perf continue monitoring the remaining threads and do not exit with error. Here, the

Re: [PATCH] vfio: Simplify capability helper

2017-12-12 Thread Zhenyu Wang
On 2017.12.13 12:13:34 +1100, Alexey Kardashevskiy wrote: > On 13/12/17 06:59, Alex Williamson wrote: > > The vfio_info_add_capability() helper requires the caller to pass a > > capability ID, which it then uses to fill in header fields, assuming > > hard coded versions. This makes for an awkward

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

2017-12-12 Thread Jia He
In our Armv8a server (qualcomm Amberwing, non VHE), after applying Christoffer's timer optimizing patchset(Optimize arch timer register handling), the guest is hang during kernel booting. The error root cause might be as follows: 1. in kvm_arch_timer_handler, it reset vtimer->cnt_ctl with current

[v3 PATCH 0/3] powernv-cpufreq: Multiple pstate related fixes.

2017-12-12 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" This is a third version of the patch to fix pstate related issues in the powernv-cpufreq driver. The previous versions can be found here: [v2]: https://lkml.org/lkml/2017/12/7/1562 [v1]: https://lkml.org/lkml/2017/11/29/1338 On POWERNV

[v3 PATCH 2/3] powernv-cpufreq: Fix pstate_to_idx() to handle non-continguous pstates

2017-12-12 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" The code in powernv-cpufreq, makes the following two assumptions which are not guaranteed by the device-tree bindings: 1) Pstate ids are continguous: This is used in pstate_to_idx() to obtain the reverse map from a pstate to it's

[v3 PATCH 1/3] powernv-cpufreq: Add helper to extract pstate from PMSR

2017-12-12 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" On POWERNV platform, the fields for pstates in the Power Management Status Register (PMSR) and the Power Management Control Register (PMCR) are 8-bits wide. On POWER8 the pstates are negatively numbered while on POWER9 they are positively

[v3 PATCH 3/3] powernv-cpufreq: Treat pstates as opaque 8-bit values

2017-12-12 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" On POWER8 and POWER9, the PMSR and the PMCR registers define pstates to be 8-bit wide values. The device-tree exports pstates as 32-bit wide values of which the lower byte is the actual pstate. The current implementation in the kernel treats

Re: [RFC PATCH 5/9] ethtool: implement GET_DRVINFO message

2017-12-12 Thread Jiri Pirko
Wed, Dec 13, 2017 at 12:54:39AM CET, mkube...@suse.cz wrote: >On Mon, Dec 11, 2017 at 05:16:01PM +0100, Jiri Pirko wrote: >> Mon, Dec 11, 2017 at 02:54:01PM CET, mkube...@suse.cz wrote: >> >+ >> >+ETHA_DRVINFO_DRIVER(string)driver name >> >+ETHA_DRVINFO_VERSION

Re: [PATCH] vfio: Simplify capability helper

2017-12-12 Thread Peter Xu
On Tue, Dec 12, 2017 at 12:59:39PM -0700, Alex Williamson wrote: > The vfio_info_add_capability() helper requires the caller to pass a > capability ID, which it then uses to fill in header fields, assuming > hard coded versions. This makes for an awkward and rigid interface. > The only thing we

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

2017-12-12 Thread Marek Szyprowski
Hi Shuah and Greg, On 2017-12-12 15:47, Shuah Khan wrote: On 12/12/2017 05:43 AM, Greg Kroah-Hartman wrote: This is the start of the stable review cycle for the 4.14.6 release. There are 164 patches in this series, all will be posted as a response to this one. If anyone has any issues with

[PATCH v2 01/11] ASoC: fsl_ssi: Rename fsl_ssi_private to fsl_ssi

2017-12-12 Thread Nicolin Chen
Shorten the private data structure to save some wrapped lines. Signed-off-by: Nicolin Chen --- sound/soc/fsl/fsl_ssi.c | 456 +++- 1 file changed, 220 insertions(+), 236 deletions(-) diff --git a/sound/soc/fsl/fsl_ssi.c

[PATCH v2 06/11] ASoC: fsl_ssi: Refine printk outputs

2017-12-12 Thread Nicolin Chen
This patches unifies the error message in the "failed to " format. It also reduces the length of one line and adds spaces to an operator. Signed-off-by: Nicolin Chen --- sound/soc/fsl/fsl_ssi.c | 16 1 file changed, 8 insertions(+), 8 deletions(-)

[PATCH v2 07/11] ASoC: fsl_ssi: Rename cpu_dai parameter to dai

2017-12-12 Thread Nicolin Chen
Shortens the variable name to save space, useful for dev_err outputs. Signed-off-by: Nicolin Chen --- sound/soc/fsl/fsl_ssi.c | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/sound/soc/fsl/fsl_ssi.c

[PATCH v2 05/11] ASoC: fsl_ssi: Refine indentations and wrappings

2017-12-12 Thread Nicolin Chen
This patch just simply unifies the coding style. Signed-off-by: Nicolin Chen --- Changelog v1->v2 * Added two missing indentation changes * Removed two extra blank lines. sound/soc/fsl/fsl_ssi.c | 239 +---

[PATCH v2 02/11] ASoC: fsl_ssi: Cache pdev->dev pointer

2017-12-12 Thread Nicolin Chen
There should be no trouble to understand dev = pdev->dev. This can save some space to have more print info or save some wrapped lines. Signed-off-by: Nicolin Chen --- sound/soc/fsl/fsl_ssi.c | 64 - 1 file changed, 31

[PATCH v2 09/11] ASoC: fsl_ssi: Replace fsl_ssi_rxtx_reg_val with fsl_ssi_regvals

2017-12-12 Thread Nicolin Chen
The name fsl_ssi_rxtx_reg_val is too long to read comfortably. So this patch shortens it by using an array (fsl_ssi_regvals, renamed from fsl_ssi_reg_val). To do that, it also introduces two macros (TX and RX) to replace the wrapper structure. This will also help further cleanups. Meanwhile, it

[PATCH v2 11/11] ASoC: fsl_ssi: Define ternary macros to simplify code

2017-12-12 Thread Nicolin Chen
Some regmap code looks redudant. So simplify it. Signed-off-by: Nicolin Chen --- sound/soc/fsl/fsl_ssi.c | 27 +++ sound/soc/fsl/fsl_ssi.h | 4 2 files changed, 15 insertions(+), 16 deletions(-) diff --git a/sound/soc/fsl/fsl_ssi.c

[PATCH v2 08/11] ASoC: fsl_ssi: Rename scr_val to scr

2017-12-12 Thread Nicolin Chen
Simplify the variable name. This reduces one over-80-character line. Signed-off-by: Nicolin Chen --- sound/soc/fsl/fsl_ssi.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/sound/soc/fsl/fsl_ssi.c b/sound/soc/fsl/fsl_ssi.c index

[PATCH v2 03/11] ASoC: fsl_ssi: Refine all comments

2017-12-12 Thread Nicolin Chen
This patch refines the comments by: 1) Removing all out-of-date comments 2) Removing all not-so-useful comments 3) Unifying the styles of all comments 4) Simplifying over-descriptive comments 5) Adding comments to improve code readablity 6) Moving all register related comments to fsl_ssi.h 7)

[PATCH v2 10/11] ASoC: fsl_ssi: Rename i2smode to i2s_net

2017-12-12 Thread Nicolin Chen
Since this i2smode also includes the setting of Network mode, it should have it in the name. This patch also adds its MASK define. Signed-off-by: Nicolin Chen --- sound/soc/fsl/fsl_ssi.c | 24 sound/soc/fsl/fsl_ssi.h | 1 + 2 files changed, 13

[PATCH v2 04/11] ASoC: fsl_ssi: Rename registers and fields macros

2017-12-12 Thread Nicolin Chen
This patch renames CCSR_SSI_xxx to REG_SSI_xxx and SSI_xxx_yyy style. It also slightly reduces the length of them to save some space. Signed-off-by: Nicolin Chen --- sound/soc/fsl/fsl_ssi.c | 374 +-- sound/soc/fsl/fsl_ssi.h

[PATCH v2 00/11] ASoC: fsl_ssi: Clean up - coding style level

2017-12-12 Thread Nicolin Chen
==Changelog== v1->v2 * Dropped one patch to remove "struct device" * Revised PATCH-03 "Refine all comments" * Revised PATCH-05 "Refine indentations and wrappings" * Rebased all other patches * Added PATCH-10 "Rename i2smode to i2s_net" * Added PATCH-11 "Define ternary macros to simplify

[PATCH 4/5] phy: renesas: rcar-gen3-usb2: add rcar_gen3_role_swap_ops

2017-12-12 Thread Yoshihiro Shimoda
This patch add rcar_gen3_role_swap_ops to support other feature (e.g. gpio handling) easily. Signed-off-by: Yoshihiro Shimoda --- drivers/phy/renesas/phy-rcar-gen3-usb2.c | 51 +++- 1 file changed, 43 insertions(+), 8 deletions(-)

[PATCH 0/5] phy: renesas: rcar-gen3-usb2: add gpio handling for R-Car D3

2017-12-12 Thread Yoshihiro Shimoda
This patch set is based on the latest linux-phy / next branch (commit id = 4fbd8d194f06c8a3fd2af1ce560ddb31f7ec8323). This new feature will be used by the renesas_usbhs driver on R-Car D3. Yoshihiro Shimoda (5): phy: renesas: rcar-gen3-usb2: call INIT_WORK() anyway phy: renesas:

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

2017-12-12 Thread Yoshihiro Shimoda
This patch unifies the OBINTEN handling to clean-up the code. Signed-off-by: Yoshihiro Shimoda --- drivers/phy/renesas/phy-rcar-gen3-usb2.c | 23 +++ 1 file changed, 15 insertions(+), 8 deletions(-) diff --git

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

2017-12-12 Thread Yoshihiro Shimoda
Some R-Car SoCs (e.g. R-Car D3) doesn't have dedicated pins of VBUS and ID. So, they may be connected to gpio pins. To handle the gpio pins, this patch adds the handling of VBUS and ID pins instead of dedicated pins. Signed-off-by: Yoshihiro Shimoda ---

[PATCH 1/5] phy: renesas: rcar-gen3-usb2: call INIT_WORK() anyway

2017-12-12 Thread Yoshihiro Shimoda
In the future, the work struct will be used by non-irq related code. So, this patch moves the INIT_WORK() timing. Signed-off-by: Yoshihiro Shimoda --- drivers/phy/renesas/phy-rcar-gen3-usb2.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH 3/5] phy: renesas: rcar-gen3-usb2: use prefix "has_otg_pins_" for dedicated pins handling

2017-12-12 Thread Yoshihiro Shimoda
To support gpio handling in the future, this patch clean-ups the code to use prefix "has_otg_pins_" functions. Signed-off-by: Yoshihiro Shimoda --- drivers/phy/renesas/phy-rcar-gen3-usb2.c | 130 --- 1 file changed, 85 insertions(+),

About the try to remove cross-release feature entirely by Ingo

2017-12-12 Thread Byungchul Park
Lockdep works, based on the following: (1) Classifying locks properly (2) Checking relationship between the classes If (1) is not good or (2) is not good, then we might get false positives. For (1), we don't have to classify locks 100% properly but need as enough as lockdep works. For

Re: [PATCH v5] ASoC: TSCS42xx: Add support for Tempo Semiconductor's TSCS42xx audio CODEC

2017-12-12 Thread Steven Eckhoff
On Tue, Dec 12, 2017 at 04:51:35PM -0600, Steven Eckhoff wrote: > > > > +static int tscs42xx_mute_stream(struct snd_soc_dai *dai, int mute, int > > > > stream) > > > > +{ > > > > + struct snd_soc_codec *codec = dai->codec; > > > > + int ret; > > > > + > > > > + if (mute) > > >

Re: [PATCH] openrisc: Fix conflicting types for _exext and _stext

2017-12-12 Thread Sergey Senozhatsky
On (12/13/17 14:12), Joel Stanley wrote: > The printk tree in linux-next has a patch "symbol lookup: introduce > dereference_symbol_descriptor()" that includes sections.h in kallsyms.h, > so arch/openrisc/kernel/traps.c gets a second extern definition for > _etext and _stext. > > Remove the local

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

2017-12-12 Thread Greentime Hu
Hi, Daniel: 2017-12-12 18:05 GMT+08:00 Daniel Lezcano : > On 12/12/2017 06:46, Rick Chen wrote: >> 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

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

2017-12-12 Thread Baolin Wang
Hi Alexandre, On 13 December 2017 at 06:16, Alexandre Belloni wrote: > Hi Baolin, > > On 16/11/2017 at 13:59:28 +0800, Baolin Wang wrote: >> @@ -779,6 +797,7 @@ static int rtc_timer_enqueue(struct rtc_device *rtc, >> struct rtc_timer *timer) >> } >>

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

2017-12-12 Thread Greentime Hu
2017-12-13 10:16 GMT+08:00 Guo Ren : > On Fri, Dec 08, 2017 at 05:11:52PM +0800, Greentime Hu wrote: >> From: Greentime Hu > [...] >> diff --git a/arch/nds32/mm/cacheflush.c b/arch/nds32/mm/cacheflush.c > [...] >> +#ifndef CONFIG_CPU_CACHE_ALIASING >>

linux-next: Tree for Dec 13

2017-12-12 Thread Stephen Rothwell
Hi all, Changes since 20171212: The clk tree still had its build failures so I used the version from next-20171208. The bpf-next tree gained a build failure so I used the version from next-20171212. Non-merge commits (relative to Linus' tree): 4015 4531 files changed, 137703 insertions

Re: [PATCH 2/6] blk-mq: replace timeout synchronization with a RCU and generation based scheme

2017-12-12 Thread jianchao.wang
Hi Tejun On 12/13/2017 03:01 AM, Tejun Heo wrote: > Currently, blk-mq timeout path synchronizes against the usual > issue/completion path using a complex scheme involving atomic > bitflags, REQ_ATOM_*, memory barriers and subtle memory coherence > rules. Unfortunatley, it contains quite a few

Re: [BUG] skge: a possible sleep-in-atomic bug in skge_remove

2017-12-12 Thread Stephen Hemminger
On Tue, 12 Dec 2017 20:57:01 -0500 (EST) David Miller wrote: > From: Stephen Hemminger > Date: Tue, 12 Dec 2017 10:22:40 -0800 > > > On Tue, 12 Dec 2017 08:34:45 -0500 (EST) > > David Miller wrote: > > > >> From: Jia-Ju

Re: KASAN: stack-out-of-bounds Read in xfrm_state_find (3)

2017-12-12 Thread Steffen Klassert
On Tue, Dec 12, 2017 at 01:00:31PM -0800, Eric Biggers wrote: > Hi Steffen, > > On Fri, Dec 01, 2017 at 08:27:43AM +0100, Steffen Klassert wrote: > > On Wed, Nov 22, 2017 at 08:05:00AM -0800, syzbot wrote: > > > syzkaller has found reproducer for the following crash on > > >

[PATCH 1/1] usb: doc: Update document for USB3 debug port usage

2017-12-12 Thread Lu Baolu
Update Documentation/driver-api/usb/usb3-debug-port.rst. This update includes the guide for using xHCI debug capability based TTY serial link. Cc: Mathias Nyman Cc: Greg Kroah-Hartman Signed-off-by: Lu Baolu

RE: [PATCH v2 1/2] Documentation: devicetree: Add DT bindings to xlnx_vcu driver

2017-12-12 Thread Dhaval Rajeshbhai Shah
Hi Rob, Thanks a lot for the review. > -Original Message- > From: Rob Herring [mailto:r...@kernel.org] > Sent: Tuesday, December 12, 2017 12:07 PM > To: Dhaval Rajeshbhai Shah > Cc: a...@arndb.de; gre...@linuxfoundation.org; > pombreda...@nexb.com;

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

2017-12-12 Thread kpark3469
From: Sahara In case that CONFIG_SLUB_DEBUG is on and pty is used, races between release_one_tty and flush_to_ldisc work threads may happen and lead to use-after-free condition on tty->link->port. Because SLUB_DEBUG is turned on, freed tty->link->port is filled with

Re: [PATCH v3 03/21] fpga: mgr: add status for fpga-manager

2017-12-12 Thread Wu Hao
On Tue, Dec 12, 2017 at 12:18:06PM -0600, Alan Tull wrote: > On Mon, Nov 27, 2017 at 12:42 AM, Wu Hao wrote: > > This patch adds status to fpga-manager data structure, to allow > > driver to store full/partial reconfiguration errors and other > > status information, and adds one

Re: [PATCH v2] dmaengine: fsl-edma: disable clks on all error paths

2017-12-12 Thread Vinod Koul
On Wed, Dec 06, 2017 at 02:50:23PM +0100, Andreas Platschek wrote: > Previously enabled clks are only disabled if clk_prepare_enable() fails. > However, there are other error paths were the previously enabled > clocks are not disabled. > > Found by Linux Driver Verification project

Re: [BUG] scsi/qla2xxx: a possible sleep-in-atomic bug in qlt_get_tag

2017-12-12 Thread James Bottomley
On Wed, 2017-12-13 at 11:18 +0800, Jia-Ju Bai wrote: > The driver may sleep under a spinlock. > The function call paths are: > qlt_handle_abts_recv_work (acquire the spinlock) >    qlt_response_pkt_all_vps >  qlt_response_pkt >    qlt_handle_cmd_for_atio >  qlt_get_tag >    

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

2017-12-12 Thread Wanpeng Li
From: Wanpeng Li [ cut here ] Bad FPU state detected at kvm_put_guest_fpu+0xd8/0x2d0 [kvm], reinitializing FPU registers. WARNING: CPU: 1 PID: 4594 at arch/x86/mm/extable.c:103 ex_handler_fprestore+0x88/0x90 CPU: 1 PID: 4594 Comm:

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

2017-12-12 Thread Mehta, Sohil
On Wed, 2017-12-13 at 10:28 +0800, Lu Baolu wrote: >  > > Would the recommendation be to use pr_warn instead of pr_err or > > should > > we entirely skip the message altogether? > Greg ever educated me about the use of debugfs_ functions in > this thread. > >

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

2017-12-12 Thread Wanpeng Li
2017-12-13 0:16 GMT+08:00 Paolo Bonzini : > On 12/12/2017 06:40, Wanpeng Li wrote: >> 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 :

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

2017-12-12 Thread Yaowei Bai
On Tue, Dec 12, 2017 at 07:47:09PM -0800, David Rientjes wrote: > On Tue, 12 Dec 2017, Randy Dunlap wrote: > > > Sure, but I didn't keep the patch emails. > > > > Acked-by: Randy Dunlap > > > > You may have noticed changing functions like is_file_lru() to bool when it

Re: [PATCH] clk: fix spin_lock/unlock imbalance on bad clk_enable() reentrancy

2017-12-12 Thread David Lechner
On 12/12/2017 05:43 PM, David Lechner wrote: If clk_enable() is called in reentrant way and spin_trylock_irqsave() is not working as expected, it is possible to get a negative enable_refcnt which results in a missed call to spin_unlock_irqrestore(). It works like this: 1. clk_enable() is

[PATCH v2 net-next 3/3] net: dsa: mediatek: update MAINTAINERS entry with MediaTek switch driver

2017-12-12 Thread sean.wang
From: Sean Wang I work for MediaTek and maintain SoC targeting to home gateway and also will keep extending and testing the function from MediaTek switch. Signed-off-by: Sean Wang Reviewed-by: Andrew Lunn --- MAINTAINERS | 7

[PATCH v2 net-next 2/3] net: dsa: mediatek: combine MediaTek tag with VLAN tag

2017-12-12 Thread sean.wang
From: Sean Wang In order to let MT7530 switch can recognize well those egress packets having both special tag and VLAN tag, the information about the special tag should be carried on the existing VLAN tag. On the other hand, it's unnecessary for extra handling for ingress

[PATCH v2 net-next 0/3] add VLAN support to DSA MT7530

2017-12-12 Thread sean.wang
From: Sean Wang Changes since v1: - fix up the typo - prefer ordering declarations longest to shortest - update that vlan_prepare callback should not change any state - use lower case letter for function naming The patchset extends DSA MT7530 to VLAN support through

[PATCH v2 net-next 1/3] net: dsa: mediatek: add VLAN support for MT7530

2017-12-12 Thread sean.wang
From: Sean Wang MT7530 can treat each port as either VLAN-unaware port or VLAN-aware port through the implementation of port matrix mode or port security mode on the ingress port, respectively. On one hand, Each port has been acting as the VLAN-unaware one whenever the

[PATCH] mm: save current->journal_info before calling fault/page_mkwrite

2017-12-12 Thread Yan, Zheng
We recently got an Oops report: BUG: unable to handle kernel NULL pointer dereference at (null) IP: jbd2__journal_start+0x38/0x1a2 [...] Call Trace: ext4_page_mkwrite+0x307/0x52b _ext4_get_block+0xd8/0xd8 do_page_mkwrite+0x6e/0xd8 handle_mm_fault+0x686/0xf9b mntput_no_expire+0x1f/0x21e

linux-next: build warning after merge of the cgroup tree

2017-12-12 Thread Stephen Rothwell
Hi Tejun, After merging the cgroup tree, today's linux-next build (arm multi_v7_defconfig) produced this warning: kernel/cgroup/cgroup.c: In function 'init_cgroup_root': kernel/cgroup/cgroup.c:1867:3: warning: ignoring return value of 'strscpy', declared with attribute warn_unused_result

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

2017-12-12 Thread David Rientjes
On Tue, 12 Dec 2017, Randy Dunlap wrote: > Sure, but I didn't keep the patch emails. > > Acked-by: Randy Dunlap > You may have noticed changing functions like is_file_lru() to bool when it is used to index into an array or as part of an arithmetic operation for ZVC

[BUG] renesas/ravb: two possible sleep-in-atomic bugs in ravb_set_link_ksettings and ravb_nway_reset

2017-12-12 Thread Jia-Ju Bai
Accoring to drivers/net/ethernet/renesas/ravb_main.c, the driver may sleep under a spinlock. The function call paths are: ravb_set_link_ksettings (acquire the spinlock) phy_ethtool_ksettings_set phy_start_aneg phy_start_aneg_priv mutex_lock --> may sleep ravb_nway_reset

[PATCH] openrisc: Fix conflicting types for _exext and _stext

2017-12-12 Thread Joel Stanley
The printk tree in linux-next has a patch "symbol lookup: introduce dereference_symbol_descriptor()" that includes sections.h in kallsyms.h, so arch/openrisc/kernel/traps.c gets a second extern definition for _etext and _stext. Remove the local definitions and include sections.h directly in

[BUG] renesas/sh_eth: two possible sleep-in-atomic bugs in sh_eth_set_link_ksettings and sh_eth_nway_reset

2017-12-12 Thread Jia-Ju Bai
Accoring to drivers/net/ethernet/renesas/sh_eth.c, the driver may sleep under a spinlock. The function call paths are: sh_eth_set_link_ksettings (acquire the spinlock) phy_ethtool_ksettings_set phy_start_aneg phy_start_aneg_priv mutex_lock --> may sleep sh_eth_nway_reset

Will mdadm make a backup if it takes two steps to covert raid5 to raid6?

2017-12-12 Thread Amy Chiang
Hi, I was trying to grow my raid5 array to raid6 with one more disk. And I was wondering about when would the backup take place and when would the backup file is required (or not required). And I found this thread "Re: Backup file size when migrating from raid5 to raid6?" two steps were

Re: [PATCH 1/6] blk-mq: protect completion path with RCU

2017-12-12 Thread jianchao.wang
Hello tejun Sorry for missing the V2, same comment again. On 12/13/2017 03:01 AM, Tejun Heo wrote: > Currently, blk-mq protects only the issue path with RCU. This patch > puts the completion path under the same RCU protection. This will be > used to synchronize issue/completion against timeout

[BUG] b44: two possible sleep-in-atomic bugs in b44_set_link_ksettings and b44_ioctl

2017-12-12 Thread Jia-Ju Bai
The driver may sleep under a spinlock. The function call paths are: b44_set_link_ksettings (acquire the spinlock) phy_ethtool_ksettings_set phy_start_aneg phy_start_aneg_priv mutex_lock --> may sleep b44_ioctl (acquire the spinlock) phy_mii_ioctl mdiobus_read

Re: [alsa-devel] [PATCH] ASoC: Intel: bxt: Refine the HW contraint of Ref capture.

2017-12-12 Thread Vinod Koul
On Wed, Dec 06, 2017 at 03:21:27AM +0800, Harry Pan wrote: > The patch restricts the HW contraint of the refcap of WoV stream > in single channel (mono) and 16k Hz based on platform implementation. > > Such that, the userspace program can rely on correct HW parameters > through the ALSA library

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

2017-12-12 Thread Paul E. McKenney
On Wed, Dec 13, 2017 at 10:17:41AM +0800, Huang, Ying wrote: > "Paul E. McKenney" writes: > > > On Tue, Dec 12, 2017 at 09:12:20AM +0800, Huang, Ying wrote: > >> Hi, Pual, > >> > >> "Paul E. McKenney" writes: > >> > >> > On Mon, Dec 11,

[BUG] scsi/qla2xxx: a possible sleep-in-atomic bug in qlt_get_tag

2017-12-12 Thread Jia-Ju Bai
The driver may sleep under a spinlock. The function call paths are: qlt_handle_abts_recv_work (acquire the spinlock) qlt_response_pkt_all_vps qlt_response_pkt qlt_handle_cmd_for_atio qlt_get_tag percpu_ida_alloc --> may sleep qla82xx_msix_rsp_q (acquire the

[PATCH 08/12] staging: lustre: libcfs: remove unused rounding functions.

2017-12-12 Thread NeilBrown
These are all unused except cfs_size_round(). So discard the others, and use the kernel-standard round_up() function to implement cfs_size_round(). Signed-off-by: NeilBrown --- .../lustre/include/linux/libcfs/libcfs_private.h | 29 +--- 1 file changed, 1

[PATCH 11/12] staging: lustre: libcfs: discard KLASSERT()

2017-12-12 Thread NeilBrown
This appears to be intended for assertions that only make sense in the kernel, but as this code is now kernel-only, it doesn't make sense any more. Signed-off-by: NeilBrown --- .../lustre/include/linux/libcfs/libcfs_private.h |2 --

[PATCH 09/12] staging: lustre: libcfs: discard MKSTR() macro

2017-12-12 Thread NeilBrown
This is only used for tracing when some strings might be NULL. NULL strings are not a problem for tracing, vnsprintf() will report them as "(null)" which is probably better (easier to parse) than an empty string. Also remove a nearby comment that doesn't relate to the (remaining) code at all.

[PATCH 12/12] staging: lustre: libcfs: discard LASSERT_CHECKED

2017-12-12 Thread NeilBrown
This macro isn't used, and comment is about some earlier version of the lustre code that never reached the mainline kernel. Just discard it. Signed-off-by: NeilBrown --- .../lustre/include/linux/libcfs/libcfs_private.h |7 --- 1 file changed, 7 deletions(-) diff --git

[PATCH 10/12] staging: lustre: libcfs: discard MAX_NUMERIC_VALUE

2017-12-12 Thread NeilBrown
This is unused. drivers/staging/lustre/lnet/lnet/nidstrings.c does use the name, but it includes its own local definition. Signed-off-by: NeilBrown --- .../lustre/include/linux/libcfs/libcfs_private.h |3 --- 1 file changed, 3 deletions(-) diff --git

[PATCH 07/12] staging: lustre: libcfs: simplify memory allocation.

2017-12-12 Thread NeilBrown
1/ Use kvmalloc() instead of kmalloc or vmalloc 2/ Discard the _GFP() interfaces that are never used. We only ever do GFP_NOFS and GFP_ATOMIC allocations, so support each of those explicitly. Signed-off-by: NeilBrown --- .../lustre/include/linux/libcfs/libcfs_private.h |

[PATCH 00/13] Assorted lustre clean-ups

2017-12-12 Thread NeilBrown
This series includes a resend of some "list_entry" cleanup patches which now how better commit comments, and also includes some simplifications, removing unnecessary macros and code. Thanks, NeilBrown --- NeilBrown (12): staging: lustre: use list_last_entry to simplify fld_cache_shrink

[PATCH 02/12] staging: lustre: ldlm: use list_for_each_entry in ldlm_extent_shift_kms()

2017-12-12 Thread NeilBrown
Using list_for_each_entry() means we don't need 'tmp'. Signed-off-by: NeilBrown --- drivers/staging/lustre/lustre/ldlm/ldlm_extent.c |4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_extent.c

[PATCH 04/12] staging: lustre: ldlm: minor list_entry improvements in ldlm_request.c

2017-12-12 Thread NeilBrown
Small clarify improvements, and one local variable avoided. Signed-off-by: NeilBrown --- drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_request.c

[PATCH 03/12] staging: lustre: ldlm: use list_first_entry in ldlm_lockd.c

2017-12-12 Thread NeilBrown
This is only a small simplification, but it makes the code a little clearer. Signed-off-by: NeilBrown --- drivers/staging/lustre/lustre/ldlm/ldlm_lockd.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git

[PATCH 06/12] staging: lustre: lov: use list_for_each_entry in lov_obd.c

2017-12-12 Thread NeilBrown
Using the *_entry macro simplifies the code slightly. Signed-off-by: NeilBrown --- drivers/staging/lustre/lustre/lov/lov_obd.c |6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre/lustre/lov/lov_obd.c

[PATCH 05/12] staging: lustre: ldlm: use list_for_each_entry in ldlm_resource.c

2017-12-12 Thread NeilBrown
Having a stand-alone "list_entry()" call is often a sign that something like "list_for_each_entry()" would make the code clearer. Signed-off-by: NeilBrown --- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 20 +++- 1 file changed, 7 insertions(+), 13

[PATCH 01/12] staging: lustre: use list_last_entry to simplify fld_cache_shrink

2017-12-12 Thread NeilBrown
Using list_empty() and list_last_entry() makes the code clearer, and allows a local variable to be discarded. Signed-off-by: NeilBrown --- drivers/staging/lustre/lustre/fld/fld_cache.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git

  1   2   3   4   5   6   7   8   9   10   >