[PATCH 1/3] net: __sock_cmsg_send(): Remove unused parameter `msg'

2017-09-08 Thread Michael Witten
Date: Thu, 7 Sep 2017 03:21:38 + Signed-off-by: Michael Witten --- include/net/sock.h | 2 +- net/core/sock.c| 4 ++-- net/ipv4/ip_sockglue.c | 2 +- net/ipv6/datagram.c| 2 +- 4 files changed, 5 insertions(+), 5 deletions(-) diff --git a/include/net/sock.h

[PATCH 2/3] net: inet_recvmsg(): Remove unnecessary bitwise operation

2017-09-08 Thread Michael Witten
Date: Fri, 8 Sep 2017 00:47:49 + The flag `MSG_DONTWAIT' is handled by passing an argument through the dedicated parameter `nonblock' of the function `tcp_recvmsg()'. Presumably because `MSG_DONTWAIT' is handled so explicitly, it is unset in the collection of flags that are passed to

[PATCH 3/3] net: skb_queue_purge(): lock/unlock the list only once

2017-09-08 Thread Michael Witten
Date: Thu, 7 Sep 2017 20:07:40 + With this commit, the list's lock is locked/unlocked only once for the duration of `skb_queue_purge()'. Hitherto, the list's lock has been locked/unlocked every time an item is dequeued; this seems not only inefficient, but also incorrect, as the whole point

[PATCH v3 0/6] Exynos-gsc: Support the hardware rotation limits

2017-09-08 Thread Hoegeun Kwon
Hello all, The gscaler has hardware rotation limits. So this patch set support the rotate hardware limits of gsc. To avoid problems with bisectability, patches 1~4 must be merged and then merged 5 and 6. Changes for V3: - Fixed of_match_node() to of_device_get_match_data() in drm gsc driver. -

[PATCH v3 1/6] [media] exynos-gsc: Add compatible for Exynos 5250 and 5420 specific version

2017-09-08 Thread Hoegeun Kwon
Exynos 5250 and 5420 have different hardware rotation limits. Since we have to distinguish between these two, we add different compatible(samsung,exynos5250-gsc and samsung,exynos5420-gsc). Signed-off-by: Hoegeun Kwon --- Documentation/devicetree/bindings/media/exynos5-gsc.txt | 1 + 1 file

[PATCH v3 2/6] ARM: dts: exynos: Add clean name of compatible.

2017-09-08 Thread Hoegeun Kwon
Exynos 5250 and 5420 have different hardware rotation limits. However, currently it uses only one compatible - "exynos5-gsc". Since we have to distinguish between these two, we add different compatible. Signed-off-by: Hoegeun Kwon --- arch/arm/boot/dts/exynos5250.dtsi | 8

Re: [Outreachy kernel] [PATCH] Staging: iio: adc: Added Space around binary op.

2017-09-08 Thread Julia Lawall
On Fri, 8 Sep 2017, Himanshi Jain wrote: > Added space around(one on each side of) binary I think that just around would be clear enough. In the previous patches on this file, found with git log --oneline, a subject line of staging: iio: ad7192: seems to be more popular when the patch affects

[PATCH v3 3/6] drm/exynos/gsc: Add hardware rotation limits

2017-09-08 Thread Hoegeun Kwon
The gscaler has hardware rotation limits that need to be hardcoded into driver. Distinguish them and add them to the property list. The hardware rotation limits are related to the cropped source size. When swap occurs, use rot_max size instead of crop_max size. Also the scaling limits are

[PATCH v3 4/6] [media] exynos-gsc: Add hardware rotation limits

2017-09-08 Thread Hoegeun Kwon
The hardware rotation limits of gsc depends on SOC (Exynos 5250/5420/5433). Distinguish them and add them to the driver data. Signed-off-by: Hoegeun Kwon --- drivers/media/platform/exynos-gsc/gsc-core.c | 96 1 file changed, 83 insertions(+), 13 deletions(-) diff

[PATCH v3 6/6] ARM: dts: exynos: Remove unnecessary compatible

2017-09-08 Thread Hoegeun Kwon
Currently, the compatible('samsung,exynos5-gsc') is not used. Remove unnecessary compatible. Signed-off-by: Hoegeun Kwon --- arch/arm/boot/dts/exynos5250.dtsi | 8 arch/arm/boot/dts/exynos5420.dtsi | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git

[PATCH v3 5/6] [media] exynos-gsc: Remove unnecessary compatible

2017-09-08 Thread Hoegeun Kwon
Currently, the compatible('samsung,exynos5-gsc') is not used. Remove unnecessary compatible. Signed-off-by: Hoegeun Kwon --- Documentation/devicetree/bindings/media/exynos5-gsc.txt | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

Re: [PATCH v8 3/3] drm/rockchip: Add support for Rockchip Soc LVDS

2017-09-08 Thread Mark yao
On 2017年09月08日 13:04, Heiko Stuebner wrote: Am Freitag, 8. September 2017, 09:23:09 CEST schrieb Mark yao: On 2017年09月07日 20:09, Heiko Stübner wrote: Hi Mark, Am Donnerstag, 7. September 2017, 18:16:29 CEST schrieb Mark yao: Looks good for me, so: Reviewed-by: Mark Yao I'd like to apply

Re: [PATCH] ASoC: fsl_ssi: Override bit clock rate based on slot number

2017-09-08 Thread Arnaud Mouiche
On 08/09/2017 07:42, Nicolin Chen wrote: On Thu, Sep 07, 2017 at 10:23:43PM -0700, Nicolin Chen wrote: The set_sysclk() now is used to override the output bit clock rate. But this is not a common way to implement a set_dai_sysclk(). And this creates a problem when a general machine driver

Re: [RFC PATCH 0/3] kbuild: generate intermediate C files instead of copying _shipped files

2017-09-08 Thread Masahiro Yamada
Hi Linus, Very sorry that I had not responded quickly. When I was digging into tool version dependency, as you pointed out, gperf is a problem (but seemed one time breakage for gperf 3.1) flex seemed very stable for a long time. bison seemed a bit problem if old version is used. But, I did

Re: [PATCH 00/14] Fix wrong %pF and %pS printk format specifier usages

2017-09-08 Thread Sergey Senozhatsky
On (09/07/17 16:05), Luck, Tony wrote: [..] > >>if (not_a_function_descriptor(ptr)) > >>return ptr; > > > > I'm not sure if it's possible on ia64/ppc64/parisc64 > > to reliably detect if it's a function descriptor or not. > > Agreed. I don't know how to write this test (without

Re: [PATCH 00/14] Fix wrong %pF and %pS printk format specifier usages

2017-09-08 Thread Sergey Senozhatsky
On (09/06/17 22:27), Helge Deller wrote: > This patch series fixes the wrong usages of the %pF and %pS printk format > specifiers throughout the kernel code. > > Both specifiers have the same result on most architectures. But on ia64, ppc64 > and parisc64 architectures the %pF specifier does an

Re: [PATCH] spi: Use ida to manage SPI bus number

2017-09-08 Thread Baolin Wang
Hi Mark, On 6 September 2017 at 21:28, Mark Brown wrote: > On Wed, Sep 06, 2017 at 07:26:51PM +0800, Baolin Wang wrote: >> This patch adds one static ida variable to manage SPI bus number, moreover >> we can get dynamic bus number from ida instead of one "dynamic" IDs, which >> can be removed. >

Re: Current mainline git (24e700e291d52bd2) hangs when building e.g. perf

2017-09-08 Thread Thomas Gleixner
On Fri, 8 Sep 2017, Markus Trippelsdorf wrote: CC+ Borislav. He might have access to such a beast > On 2017.09.07 at 08:28 +0200, Markus Trippelsdorf wrote: > > On 2017.09.06 at 15:15 +0200, Markus Trippelsdorf wrote: > > > On 2017.09.06 at 14:52 +0200, Thomas Gleixner wrote: > > > > On Tue, 5

Re: [PATCH] Staging: iio: adc: Added Space around binary op.

2017-09-08 Thread Jonathan Cameron
On 8 September 2017 05:47:52 BST, Himanshi Jain wrote: >Added space around(one on each side of) binary >operator(-) as preferred according to kernel >coding style. > >Signed-off-by: Himanshi Jain Take a closer look at that macro. It isn't doing what you think... To give a hint, changing

Re: [PATCH] [RESEND] drm/stm: fix warning about multiplication in condition

2017-09-08 Thread Daniel Vetter
On Thu, Sep 07, 2017 at 10:59:49AM +0530, Archit Taneja wrote: > Hi Benjamin, > > This should be pushed to drm-misc by you, right? I applied it. I guess Philippe Cornu should also become drm-misc committer instead of just acking a patch and hoping fairies will pick it up. -Daniel > > Thanks, >

[PATCH] scsi: shost->async_scan should be protected by mutex_lock

2017-09-08 Thread Ouyangzhaowei (Charles)
shost->async_scan should be protected by mutex_lock, otherwise the check of "called twice" won't work. Signed-off-by: Ouyang Zhaowei --- drivers/scsi/scsi_scan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/scsi_scan.c b/drivers/scsi/scsi_scan.c index

[PATCH] mfd: rtsx: Add support for RTS5260

2017-09-08 Thread rui_feng
From: rui_feng Add support for new chip rts5260. Signed-off-by: Rui Feng --- drivers/mfd/Makefile | 2 +- drivers/mfd/rts5260.c| 543 +++ drivers/mfd/rtsx_pcr.c | 8 + drivers/mfd/rtsx_pcr.h | 1 +

[PATCH V2] ttyport: trivial fix for some typo in comments

2017-09-08 Thread Antonio Borneo
Fix some minor typo spotted in comments: - s/wit ha/with a/ - s/doestroyed/destroyed/ Signed-off-by: Antonio Borneo --- Changes: V1 -> V2 add commit message To: Greg Kroah-Hartman To: Jiri Slaby Cc: Jiri Kosina Cc: linux-kernel@vger.kernel.org --- drivers/tty/tty_port.c | 4 ++-- 1 file

Re: [PATCH] thermal/drivers/generic-iio-adc: Switch tz request to devm version

2017-09-08 Thread Daniel Lezcano
On 08/09/2017 04:56, Eduardo Valentin wrote: > Hey Daniel, Hi Valentin, >> static int gadc_thermal_remove(struct platform_device *pdev) > > Any particular reason why this patch does not include a removal of > gadc_thermal_remove()? Ah, yes. Missed it. Thanks. -- Daniel --

[PATCH V2] thermal/drivers/generic-iio-adc: Switch tz request to devm version

2017-09-08 Thread Daniel Lezcano
Everything mentionned here: https://lkml.org/lkml/2016/4/20/850 This driver was added before the devm_iio_channel_get() function version was merged. The sensor should be released before the iio channel, thus we had to use the non-devm version of thermal_zone_of_sensor_register(). Now the

Re: [PATCH v8 3/3] drm/rockchip: Add support for Rockchip Soc LVDS

2017-09-08 Thread Mark yao
On 2017年09月08日 14:12, Mark yao wrote: On 2017年09月08日 13:04, Heiko Stuebner wrote: Am Freitag, 8. September 2017, 09:23:09 CEST schrieb Mark yao: On 2017年09月07日 20:09, Heiko Stübner wrote: Hi Mark, Am Donnerstag, 7. September 2017, 18:16:29 CEST schrieb Mark yao: Looks good for me, so:

Re: [PATCH] powerpc/mm: Fix missing mmap_sem release

2017-09-08 Thread Laurent Dufour
On 07/09/2017 22:51, Davidlohr Bueso wrote: > On Thu, 07 Sep 2017, Laurent Dufour wrote: > >> The commit b5c8f0fd595d ("powerpc/mm: Rework mm_fault_error()") reviewed >> the way the error path is managed in __do_page_fault() but it was a bit too >> agressive when handling a case by returning

Re: [GIT PULL] Security subsystem updates for 4.14

2017-09-08 Thread Christoph Hellwig
The reason why I send out the original version of this patch is because IMA used to call ->read under i_rwsem, and that deadlocked on XFS and NFS, or ext3/4 with DAX. The call path for that is process_measurement (takes i_rwsem) -> ima_collect_measurement -> ima_calc_file_hash ->

Re: Re: [PATCH] mm/vmstats: add counters for the page frag cache

2017-09-08 Thread Kyeongdon Kim
On 2017-09-04 오후 5:30, Konstantin Khlebnikov wrote: On 04.09.2017 04:35, Kyeongdon Kim wrote: > Thanks for your reply, > But I couldn't find "NR_FRAGMENT_PAGES" in linux-next.git .. is that vmstat counter? or others? > I mean rather than adding bunch vmstat counters for operations it might

Re: [PATCH v6 04/11] swiotlb: Map the buffer if it was unmapped by XPFO

2017-09-08 Thread Christoph Hellwig
On Thu, Sep 07, 2017 at 12:44:14PM -0600, Tycho Andersen wrote: > On Thu, Sep 07, 2017 at 11:10:15AM -0700, Christoph Hellwig wrote: > > > - if (PageHighMem(pfn_to_page(pfn))) { > > > + if (PageHighMem(page) || xpfo_page_is_unmapped(page)) { > > > > Please don't sprinkle xpfo details over various

[PATCH v5 10/10] net: stmmac: dwmac-sun8i: Handle integrated/external MDIOs

2017-09-08 Thread Corentin Labbe
The Allwinner H3 SoC have two distinct MDIO bus, only one could be active at the same time. The selection of the active MDIO bus are done via some bits in the EMAC register of the system controller. This patch implement this MDIO switch via a custom MDIO-mux. Signed-off-by: Corentin Labbe ---

[PATCH v5 07/10] arm64: dts: allwinner: add snps,dwmac-mdio compatible to emac/mdio

2017-09-08 Thread Corentin Labbe
stmmac bindings docs said that its mdio node must have compatible = "snps,dwmac-mdio"; Since dwmac-sun8i does not have any good reasons to not doing it, all their MDIO node must have it. Signed-off-by: Corentin Labbe --- arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 1 + 1 file changed, 1

[PATCH v5 09/10] net: stmmac: snps,dwmac-mdio MDIOs are automatically registered

2017-09-08 Thread Corentin Labbe
stmmac bindings docs said that its mdio node must have compatible = "snps,dwmac-mdio"; Since dwmac-sun8i does not have any good reasons to not doing it, all their MDIO node must have it. Since these compatible is automatically registered, dwmac-sun8i compatible does not need to be in

[PATCH v5 02/10] dt-bindings: net: Restore sun8i dwmac binding

2017-09-08 Thread Corentin Labbe
This patch restore dt-bindings documentation about dwmac-sun8i This reverts commit 8aa33ec2f481 ("dt-bindings: net: Revert sun8i dwmac binding") Signed-off-by: Corentin Labbe --- .../devicetree/bindings/net/dwmac-sun8i.txt| 84 ++ 1 file changed, 84 insertions(+)

[PATCH v5 06/10] ARM: dts: sunxi: h3/h5: represent the mdio switch used by sun8i-h3-emac

2017-09-08 Thread Corentin Labbe
Since dwmac-sun8i could use either an integrated PHY or an external PHY (which could be at same MDIO address), we need to represent this selection by a MDIO switch. Signed-off-by: Corentin Labbe --- arch/arm/boot/dts/sunxi-h3-h5.dtsi | 30 +- 1 file changed, 25

[PATCH v5 08/10] net: stmmac: dwmac-sun8i: choose internal PHY via phy-is-integrated

2017-09-08 Thread Corentin Labbe
The current way to find if the phy is internal is to compare DT phy-mode and emac_variant/internal_phy. But it will negate a possible future SoC where an external PHY use the same phy mode than the internal one. This patch adds a new way to find if the PHY is internal, via the phy-is-integrated

[PATCH v5 05/10] dt-bindings: net: dwmac-sun8i: update documentation about integrated PHY

2017-09-08 Thread Corentin Labbe
This patch add documentation about the MDIO switch used on sun8i-h3-emac for integrated PHY. Signed-off-by: Corentin Labbe --- .../devicetree/bindings/net/dwmac-sun8i.txt| 127 +++-- 1 file changed, 120 insertions(+), 7 deletions(-) diff --git

Re: [PATCH V2] thermal/drivers/generic-iio-adc: Switch tz request to devm version

2017-09-08 Thread Laxman Dewangan
On Friday 08 September 2017 12:31 PM, Daniel Lezcano wrote: Everything mentionned here: https://lkml.org/lkml/2016/4/20/850 This driver was added before the devm_iio_channel_get() function version was merged. The sensor should be released before the iio channel, thus we had to use the

Re: XFS mounted with 'discard' option - deleting fio test files slow

2017-09-08 Thread Christoph Hellwig
On Fri, Sep 08, 2017 at 09:34:02AM +1000, Dave Chinner wrote: > 4k random write with direct IO. 5GB file. Probably got a million 4k > extents in it. Which means XFS has sent a million tiny 4k discards > to the device. Run 'xfs_bmap -vvp fio_test_file.*' to confirm. > > Don't use "-o discard" if

[PATCH v5 04/10] net: stmmac: sun8i: Restore the compatibles

2017-09-08 Thread Corentin Labbe
This patch restore compatibles about dwmac-sun8i This reverts commit ad4540cc5aa3 ("net: stmmac: sun8i: Remove the compatibles") Signed-off-by: Corentin Labbe --- drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 8 1 file changed, 8 insertions(+) diff --git

Re: [PATCH] HID: rmi: Make sure the HID device is opened on resume

2017-09-08 Thread Benjamin Tissoires
On Jul 24 2017 or thereabouts, Lyude Paul wrote: > On Mon, 2017-07-24 at 10:15 +0200, Benjamin Tissoires wrote: > > On Jul 22 2017 or thereabouts, Lyude wrote: > > > So it looks like that suspend/resume has actually always been > > > broken on > > > hid-rmi. The fact it worked was a rather silly

[PATCH v5 03/10] arm: dts: sunxi: Restore EMAC changes

2017-09-08 Thread Corentin Labbe
This patch restore arm DT about dwmac-sun8i This reverts commit fe45174b72ae ("arm: dts: sunxi: Revert EMAC changes") Signed-off-by: Corentin Labbe --- arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 9 arch/arm/boot/dts/sun8i-h3-bananapi-m2-plus.dts | 19 +

[PATCH v5 00/10] net: stmmac: dwmac-sun8i: Handle integrated PHY

2017-09-08 Thread Corentin Labbe
Hello The current way to find if the PHY is internal is to compare DT phy-mode and emac_variant/internal_phy. But it will negate a possible future SoC where an external PHY use the same phy mode than the integrated one. This patchs series adds a new way to find if the PHY is integrated, via the

[PATCH v5 01/10] arm64: dts: allwinner: Restore EMAC changes

2017-09-08 Thread Corentin Labbe
This patch restore arm64 DT about dwmac-sun8i This reverts commit 87e1f5e8bb4b ("arm64: dts: allwinner: Revert EMAC changes") Signed-off-by: Corentin Labbe --- .../boot/dts/allwinner/sun50i-a64-bananapi-m64.dts | 16 .../boot/dts/allwinner/sun50i-a64-pine64-plus.dts| 15

Re: [PATCH v5 01/10] arm64: dts: allwinner: Restore EMAC changes

2017-09-08 Thread Maxime Ripard
On Fri, Sep 08, 2017 at 09:11:47AM +0200, Corentin Labbe wrote: > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts > b/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts > index 1c2387bd5df6..968908761194 100644 > ---

Re: [PATCH 1/2] xfs: always use DAX if mount option is used

2017-09-08 Thread Christoph Hellwig
On Thu, Sep 07, 2017 at 03:08:31PM -0600, Ross Zwisler wrote: > Before support for the per-inode DAX flag was disabled the XFS the code had > an issue where the user couldn't reliably tell whether or not DAX was being > used to service page faults and I/O when the DAX mount option was used. In >

Re: [PATCH 2/2] xfs: validate bdev support for DAX inode flag

2017-09-08 Thread Christoph Hellwig
Looks good, Reviewed-by: Christoph Hellwig

Re: [PATCH] arm64: KVM: VHE: reset PSTATE.UAO when switch to host

2017-09-08 Thread gengdongjiu
On 2017/9/7 23:23, Marc Zyngier wrote: > On 07/09/17 16:03, gengdongjiu wrote: >>> On 07/09/17 12:49, gengdongjiu wrote: [...] > > I really cannot think of a good reason why we'd want to do that. Playing > with set_fs() is almost universally wrong, and I'm certainly going to > oppose to

Re: [PATCH v5 05/10] dt-bindings: net: dwmac-sun8i: update documentation about integrated PHY

2017-09-08 Thread Maxime Ripard
On Fri, Sep 08, 2017 at 09:11:51AM +0200, Corentin Labbe wrote: > This patch add documentation about the MDIO switch used on sun8i-h3-emac > for integrated PHY. > > Signed-off-by: Corentin Labbe > --- > .../devicetree/bindings/net/dwmac-sun8i.txt| 127 > +++-- > 1 file

Re: [PATCH v1 0/4] perf annotate: Display multiple events on the left side of annotate view

2017-09-08 Thread Jin, Yao
Hi, Any comments for this patch-set? I just find a merge conflict with the latest perf/core. Do I need to send the v2 to fix the merge issue right now? Thanks Jin Yao On 8/16/2017 6:18 PM, Jin Yao wrote: perf record -e cycles,branches ... perf annotate main --stdio The result only shows

Re: [PATCH v5 01/10] arm64: dts: allwinner: Restore EMAC changes

2017-09-08 Thread Corentin Labbe
On Fri, Sep 08, 2017 at 09:19:54AM +0200, Maxime Ripard wrote: > On Fri, Sep 08, 2017 at 09:11:47AM +0200, Corentin Labbe wrote: > > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts > > b/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts > > index

Re: [PATCH 10/14] fs/xfs: Use %pS printk format for direct addresses

2017-09-08 Thread Christoph Hellwig
Looks fine, Reviewed-by: Christoph Hellwig

Re: [PATCH v5 01/10] arm64: dts: allwinner: Restore EMAC changes

2017-09-08 Thread Chen-Yu Tsai
On Fri, Sep 8, 2017 at 3:36 PM, Corentin Labbe wrote: > On Fri, Sep 08, 2017 at 09:19:54AM +0200, Maxime Ripard wrote: >> On Fri, Sep 08, 2017 at 09:11:47AM +0200, Corentin Labbe wrote: >> > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts >> >

Re: [PATCH v6 01/11] mm: add MAP_HUGETLB support to vm_mmap

2017-09-08 Thread Christoph Hellwig
On Thu, Sep 07, 2017 at 11:35:59AM -0600, Tycho Andersen wrote: > vm_mmap is exported, which means kernel modules can use it. In particular, > for testing XPFO support, we want to use it with the MAP_HUGETLB flag, so > let's support it via vm_mmap. > } else if (flags & MAP_HUGETLB) { > +

Re: [PATCH v5 05/10] dt-bindings: net: dwmac-sun8i: update documentation about integrated PHY

2017-09-08 Thread Corentin Labbe
On Fri, Sep 08, 2017 at 09:25:38AM +0200, Maxime Ripard wrote: > On Fri, Sep 08, 2017 at 09:11:51AM +0200, Corentin Labbe wrote: > > This patch add documentation about the MDIO switch used on sun8i-h3-emac > > for integrated PHY. > > > > Signed-off-by: Corentin Labbe > > --- > >

Re: [PATCH v6 03/11] mm, x86: Add support for eXclusive Page Frame Ownership (XPFO)

2017-09-08 Thread Christoph Hellwig
I think this patch needs to be split into the generic mm code, and the x86 arch code at least. > +/* > + * The current flushing context - we pass it instead of 5 arguments: > + */ > +struct cpa_data { > + unsigned long *vaddr; > + pgd_t *pgd; > + pgprot_tmask_set;

Re: [PATCH v6 05/11] arm64/mm: Add support for XPFO

2017-09-08 Thread Christoph Hellwig
> +/* > + * Lookup the page table entry for a virtual address and return a pointer to > + * the entry. Based on x86 tree. > + */ > +static pte_t *lookup_address(unsigned long addr) Seems like this should be moved to common arm64 mm code and used by kernel_page_present.

Re: [PATCH v6 10/11] mm: add a user_virt_to_phys symbol

2017-09-08 Thread Christoph Hellwig
On Thu, Sep 07, 2017 at 11:36:08AM -0600, Tycho Andersen wrote: > We need someting like this for testing XPFO. Since it's architecture > specific, putting it in the test code is slightly awkward, so let's make it > an arch-specific symbol and export it for use in LKDTM. We really should not add

Re: [PATCH 2/2] x86/hibernate/64: Mask off CR3's PCID bits in the saved CR3

2017-09-08 Thread Jiri Kosina
On Thu, 7 Sep 2017, Andy Lutomirski wrote: > Jiri reported a resume-from-hibernation failure triggered by PCID. > The root cause appears to be rather odd. The hibernation asm > restores a CR3 value that comes from the image header. If the image > kernel has PCID on, it's entirely reasonable for

[PATCH] scsi: lpfc: remove redundant null check on eqe

2017-09-08 Thread Colin King
From: Colin Ian King The pointer eqe is always non-null inside the while loop, so the check to see if eqe is NULL is redudant and hence can be removed. Detected by CoverityScan CID#1248693 ("Logically Dead Code") Signed-off-by: Colin Ian King --- drivers/scsi/lpfc/lpfc_sli.c | 3 --- 1 file

Re: Current mainline git (24e700e291d52bd2) hangs when building e.g. perf

2017-09-08 Thread Borislav Petkov
On Fri, Sep 08, 2017 at 08:26:44AM +0200, Thomas Gleixner wrote: > On Fri, 8 Sep 2017, Markus Trippelsdorf wrote: > > CC+ Borislav. He might have access to such a beast Can I have /proc/cpuinfo and dmesg pls, in order to see whether I have something similar? Private mail's fine too. Thx. --

[PATCH v4 0/3] Workaround for bus/slot reset on Cavium cn8xxx root ports

2017-09-08 Thread Jan Glauber
Using vfio-pci on a combination of cn8xxx and some PCI devices results in a kernel panic. This is triggered by issuing a bus or a slot reset on the PCI device. With this series both checks indicate that the reset is not possible preventing the kernel panic. David Daney (2): PCI: Allow

[PATCH v4 3/3] PCI: Avoid slot reset if bus reset is not possible

2017-09-08 Thread Jan Glauber
When checking to see if a PCI slot can safely be reset, we check to see if any of the children have their PCI_DEV_FLAGS_NO_BUS_RESET flag set. Some PCIe root port bridges do not behave well after a slot reset, and may cause the device in the slot to become unusable. Add a check for the

Re: [PATCH 33/35] perf bench futex: Fix build on musl + clang

2017-09-08 Thread Jörg Krause
Hi Arnaldo, On Mon, 2017-03-06 at 16:38 -0300, Arnaldo Carvalho de Melo wrote: > From: Arnaldo Carvalho de Melo > > When building with clang on a musl libc system, Alpine Linux, we end up > hitting a problem where memset() is used but its prototype is not > present, add it to avoid this: > >

Re: [PATCH] block: tolerate tracing of NULL bio

2017-09-08 Thread Christoph Hellwig
Looks good, Reviewed-by: Christoph Hellwig

[PATCH v4 1/3] PCI: Allow PCI_DEV_FLAGS_NO_BUS_RESET to be used on bus device

2017-09-08 Thread Jan Glauber
From: David Daney When checking to see if a PCI bus can safely be reset, we check to see if any of the children have their PCI_DEV_FLAGS_NO_BUS_RESET flag set. As these devices are known not to behave well after a bus reset. Some PCIe root port bridges also do not behave well after a bus reset,

[PATCH v4 2/3] PCI: Avoid bus reset for Cavium cn8xxx root ports

2017-09-08 Thread Jan Glauber
From: David Daney Root ports of cn8xxx do not function after bus reset when used with some e1000e and LSI HBA devices. Add a quirk to prevent bus reset on these root ports. Signed-off-by: David Daney [jglau...@cavium.com: fixed typo and whitespaces] Signed-off-by: Jan Glauber ---

[PATCH] staging: pi433: pi433_if: Fixes minor typo errors

2017-09-08 Thread harsha
Fixes checkpatch warning -- "occured" and "succesfully" are misspelled Signed-off-by: Harsha Sharma --- drivers/staging/pi433/pi433_if.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c index

Re: [PATCH v2] Add driver to force WMI Thunderbolt controller power status

2017-09-08 Thread Mika Westerberg
On Thu, Sep 07, 2017 at 01:47:22PM -0500, Mario Limonciello wrote: I have a small comment which you might want to consider. > +static int intel_wmi_thunderbolt_probe(struct wmi_device *wdev) > +{ > + return sysfs_create_group(>dev.kobj, _attribute_group); > +} > + > +static int

FINALE WAARSCHUWING VAN MICROSOFT

2017-09-08 Thread Mishra, Jatadhari
MICROSOFT VERIFICATION UPDATE Geachte klant, Lees en volg de instructies om uw Microsoft Privacy te beschermen. Als onderdeel van onze inspanningen om uw ervaring te verbeteren in onze consumentendiensten, updaten we de Microsoft Services Agreement en de Microsoft Privacy Statement. We

[PATCH] x86/idt: Unbreak MS HyperV hypercall vector install

2017-09-08 Thread Thomas Gleixner
The rework of the IDT setup changed the way how unused system gates are accounted. At the end of the gate initialization all unused gates in the system vector area are set to the spurios handler and marked as used for consistency reasons. Marking them as used breaks MS HyperV because it installs

Re: [PATCH] arm64: KVM: VHE: reset PSTATE.UAO when switch to host

2017-09-08 Thread Marc Zyngier
On Fri, 8 Sep 2017 15:19:21 +0800 gengdongjiu wrote: > On 2017/9/7 23:23, Marc Zyngier wrote: > > On 07/09/17 16:03, gengdongjiu wrote: > >>> On 07/09/17 12:49, gengdongjiu wrote: > > [...] > > > > > I really cannot think of a good reason why we'd want to do that. Playing > > with

Re: [PATCH] asm-generic/io.h: remove unnecessary include of linux/vmalloc.h

2017-09-08 Thread Tobias Klauser
gt; https://github.com/0day-ci/linux/commits/Tobias-Klauser/asm-generic-io-h-remove-unnecessary-include-of-linux-vmalloc-h/20170908-012111 > config: arm-shmobile_defconfig (attached as .config) > compiler: arm-linux-gnueabi-gcc (Debian 6.1.1-9) 6.1.1 20160705 > reproduce: > wg

Re: [RFC Part2 PATCH v3 02/26] crypto: ccp: Add Platform Security Processor (PSP) device support

2017-09-08 Thread Borislav Petkov
On Thu, Sep 07, 2017 at 06:15:55PM -0500, Gary R Hook wrote: > I would prefer that we not shorten this. The prior incarnation, > ccp_alloc_struct(), has/had been around for a while. And there are a > number of similarly named allocation functions in the driver that we > like to keep sorted. If

Re: [PATCH 4/4] sched/deadline: use C bitfields for the state flags

2017-09-08 Thread Daniel Bristot de Oliveira
On 09/07/2017 12:09 PM, luca abeni wrote: > Ask the compiler to use a single bit for storing true / false values, > instead of wasting the size of a whole int value. > Tested with gcc 5.4.0 on x86_64, and the compiler produces the expected > Assembly (similar to the Assembly code generated when

Re: [PATCH 3/4] sched/deadline: rename __dl_clear() to __dl_sub()

2017-09-08 Thread Daniel Bristot de Oliveira
On 09/07/2017 12:09 PM, luca abeni wrote: > From: Peter Zijlstra > > __dl_sub() is more meaningful as a name, and is more consistent > with the naming of the dual function (__dl_add()). > > Signed-off-by: Peter Zijlstra (Intel) > Signed-off-by: luca abeni Reviewed-by: Daniel Bristot de

Re: [alsa-devel] [PATCH 23/25 v3] ALSA/dummy: Replace tasklet with softirq hrtimer

2017-09-08 Thread Takashi Iwai
On Wed, 06 Sep 2017 06:30:03 +0200, Takashi Sakamoto wrote: > > On Sep 6 2017 01:18, Sebastian Andrzej Siewior wrote: > > From: Thomas Gleixner > > > > The tasklet is used to defer the execution of snd_pcm_period_elapsed() to > > the softirq context. Using the CLOCK_MONOTONIC_SOFT base invokes

Re: [PATCH 1/4] sched/sched.h: remove duplicate prototype of __dl_clear_params()

2017-09-08 Thread Daniel Bristot de Oliveira
Hi, I wonder if a commit log, even an one line comment, explaining that this function was already declared in the file could help to understand this patch... But, the patch is so trivial... that... not sure it is worth... anyway... On 09/07/2017 12:09 PM, luca abeni wrote: > Signed-off-by: luca

[PATCH v2 1/2] dt-bindings: spi: Add Spreadtrum ADI controller documentation

2017-09-08 Thread Baolin Wang
This patch adds the binding documentation for Spreadtrum ADI controller device. Signed-off-by: Baolin Wang --- Changes since v1: - Add more documentation the 'sprd,hw-channels' property and why need one hardware spinlock. --- .../devicetree/bindings/spi/spi-sprd-adi.txt | 58

[PATCH v2 2/2] spi: Add ADI driver for Spreadtrum platform

2017-09-08 Thread Baolin Wang
This patch adds ADI driver based on SPI framework for Spreadtrum SC9860 platform. Signed-off-by: Baolin Wang --- - Add COMPILE_TEST config as dependency. - Remove spi_controller_put() function when removing driver. - Change to module_init() level. --- drivers/spi/Kconfig|6 +

[PATCH] staging: rt8712: xmit_linux: Avoid multiple assignments in a single line

2017-09-08 Thread harsha
Fixed coding style issue Signed-off-by: Harsha Sharma --- drivers/staging/rtl8712/xmit_linux.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8712/xmit_linux.c b/drivers/staging/rtl8712/xmit_linux.c index d13fd15..03c6b0c 100644 ---

Re: [RFC Part2 PATCH v3 02/26] crypto: ccp: Add Platform Security Processor (PSP) device support

2017-09-08 Thread Borislav Petkov
On Thu, Sep 07, 2017 at 05:19:32PM -0500, Brijesh Singh wrote: > At high level, AMD-SP (AMD Secure Processor) (i.e CCP driver) will provide the > support for CCP, SEV and TEE FW commands. > > > +--- CCP > | > AMD-SP --| > |+--- SEV > ||

Re: [PATCH] selftests: breakpoints: re-order TEST_GEN_PROGS targets

2017-09-08 Thread Fathi Boudra
Shuah, should I resend? On 29 June 2017 at 12:39, Fathi Boudra wrote: > breakpoint_test can fail on arm64 with older/unpatched glibc: > > breakpoint_test_arm64.c: In function 'run_test': > breakpoint_test_arm64.c:170:25: error: 'TRAP_HWBKPT' undeclared (first use > in this function) > > due

Re: [PATCH] ASoC: fsl_ssi: Override bit clock rate based on slot number

2017-09-08 Thread Łukasz Majewski
Hi Nicolin, On Thu, Sep 07, 2017 at 10:23:43PM -0700, Nicolin Chen wrote: The set_sysclk() now is used to override the output bit clock rate. But this is not a common way to implement a set_dai_sysclk(). And this creates a problem when a general machine driver (simple-card for example) tries

Re: [Outreachy kernel] [PATCH] staging: pi433: pi433_if: Fixes minor typo errors

2017-09-08 Thread Julia Lawall
The subject line above is still not what is shown with git log --oneline for this file. Also, this patch fixes a previous submission so it should be v2. I suggest to read through the tutorial carefully again to get these small details. julia On Fri, 8 Sep 2017, harsha wrote: > Fixes

[PATCH 2/3] perf tools: Open perf.data with O_CLOEXEC flag

2017-09-08 Thread Jiri Olsa
Do not carry the perf.data file descriptor into the workload process and close it when perf executes the workload. Link: http://lkml.kernel.org/n/tip-j7qdp4ucreain1cqvy485...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/util/data.c | 3 ++- 1 file changed, 2 insertions(+), 1

[PATCH v2 2/2] lockdep: Remove unnecessary acquisitions wrt workqueue flush

2017-09-08 Thread Byungchul Park
The workqueue added manual acquisitions to catch deadlock cases. Now crossrelease was introduced, some of those are redundant, since wait_for_completion() already includes the acquisition for itself. Removed it. Signed-off-by: Byungchul Park --- include/linux/workqueue.h | 4 ++--

Re: [RFC][PATCH] perf: Rewrite enabled/running timekeeping

2017-09-08 Thread Alexey Budankov
On 05.09.2017 19:03, Peter Zijlstra wrote: > On Tue, Sep 05, 2017 at 03:06:26PM +0300, Alexey Budankov wrote: >> [ 6614.226305] WARNING: CPU: 45 PID: 43385 at kernel/events/core.c:239 >> event_function+0xb3/0xe0 > > I think I avoided that problem by not radically rewriting > perf_event_read()

Re: [Outreachy kernel] [PATCH] staging: rt8712: xmit_linux: Avoid multiple assignments in a single line

2017-09-08 Thread Julia Lawall
On Fri, 8 Sep 2017, harsha wrote: > Fixed coding style issue > > Signed-off-by: Harsha Sharma > --- > drivers/staging/rtl8712/xmit_linux.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/rtl8712/xmit_linux.c >

[PATCH v2 1/2] completion: Add support for initializing completion with lockdep_map

2017-09-08 Thread Byungchul Park
To peter, Does it work? Chagnes from v1 - Add a completion initialization function with a lockdep map - Enhance readability of the workqueue code ->8- >From e148617e20ebc9c9eefe7bb222b9bba07cb963bc Mon Sep 17 00:00:00 2001 From: Byungchul Park Date: Fri, 8 Sep 2017 17:39:48 +0900

[PATCH 3/3] perf tools: Add python-clean target

2017-09-08 Thread Jiri Olsa
To be able to cleanup only python related binaries. Link: http://lkml.kernel.org/n/tip-j7qdp4ucreain1cqvy485...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/Makefile.perf | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/tools/perf/Makefile.perf

[PATCH 1/3] tools lib api: Fix make DEBUG=1 build

2017-09-08 Thread Jiri Olsa
Do not use -D_FORTIFY_SOURCE=2 for DEBUG build as it seems to mess up with debuginfo, which results in bad gdb experience. We already do that for perf. Link: http://lkml.kernel.org/n/tip-mfg2ly8i01dyx1tku813b...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/lib/api/Makefile | 8 +++- 1

Re: 4.13 on thinkpad x220: oops when writing to SD card

2017-09-08 Thread Pavel Machek
On Thu 2017-09-07 14:06:52, Ulf Hansson wrote: > [...] > > > > > From: Adrian Hunter > > Date: Thu, 7 Sep 2017 10:40:35 +0300 > > Subject: [PATCH] mmc: block: Fix incorrectly initialized requests > > > > mmc_init_request() depends on card->bouncesz so it must be calculated > > before

Re: [PATCH RFC 0/6] Modify KSZ9477 DSA driver in preparation to add other KSZ switch drivers.

2017-09-08 Thread Pavel Machek
Hi! > From: Tristram Ha > > This series of patches is to modify the original KSZ9477 DSA driver so that > other KSZ switch drivers can be added and use the common code. > Please wrap the lines from time to time... > This patch set is against net-next. > >

Re: [RFC PATCH v4 3/6] i2c: add docs to clarify DMA handling

2017-09-08 Thread Wolfram Sang
Hi Mauro, thanks for your comments. Much appreciated! > There are also a couple of things here that Sphinx would complain. > So, it could be worth to rename it to *.rst, while you're writing > it, and see what: > make htmldocs > will complain and how it will look in html. OK, I'll check

Re: [PATCH RFC 1/6] The file ksz_common.c will be used by other KSZ switch drivers.

2017-09-08 Thread Pavel Machek
On Thu 2017-09-07 21:08:58, tristram...@microchip.com wrote: > From: Tristram Ha > > Break ksz_common.c into 2 files so that the common code can be used by other > KSZ switch drivers. > > Signed-off-by: Tristram Ha > --- > diff --git a/drivers/net/dsa/microchip/Makefile >

Re: [PATCH RFC 1/6] The file ksz_common.c will be used by other KSZ switch drivers.

2017-09-08 Thread Pavel Machek
> > > Signed-off-by: Tristram Ha > > > --- > > > diff --git a/drivers/net/dsa/microchip/Makefile > > > b/drivers/net/dsa/microchip/Makefile > > > index ed335e2..0961c30 100644 > > > --- a/drivers/net/dsa/microchip/Makefile > > > +++ b/drivers/net/dsa/microchip/Makefile > > > @@ -1,2 +1,2 @@ > >

Re: [PATCH RFC] Update documentation for KSZ DSA drivers so that new drivers can be added

2017-09-08 Thread Pavel Machek
On Thu 2017-09-07 21:11:45, tristram...@microchip.com wrote: > From: Tristram Ha > > Add other KSZ switches support so that patch check does not complain. Nice; unlike Andrew I believe that adding compatible strings is ok before the support is added. Device tree description is

Re: [PATCH RFC 2/5] Add KSZ8795 switch driver support in Makefile

2017-09-08 Thread Pavel Machek
On Fri 2017-09-08 00:43:09, Andrew Lunn wrote: > On Thu, Sep 07, 2017 at 10:29:34PM +, tristram...@microchip.com wrote: > > > -Original Message- > > > From: Andrew Lunn [mailto:and...@lunn.ch] > > > Sent: Thursday, September 07, 2017 2:56 PM > > > To: Tristram Ha - C24268 > > > Cc:

[PATCH V2] thermal/drivers/step_wise: Fix temperature regulation misbehavior

2017-09-08 Thread Daniel Lezcano
There is a particular situation when the cooling device is cpufreq and the heat dissipation is not efficient enough where the temperature increases little by little until reaching the critical threshold and leading to a SoC reset. The behavior is reproducible on a hikey6220 with bad heat

  1   2   3   4   5   6   7   8   9   10   >