Re: NFS crash, hashed pointers in backtrace

2017-12-08 Thread Geert Uytterhoeven
On Wed, Dec 6, 2017 at 5:19 PM, Geert Uytterhoeven wrote: > On Wed, Dec 6, 2017 at 5:10 PM, Trond Myklebust > wrote: >> On Wed, 2017-12-06 at 15:31 +0100, Geert Uytterhoeven wrote: >>> On Tue, Dec 5, 2017 at 5:02 PM, Geert Uytterhoeven >> org> wrote: >>> Got another nfsroot crash: >>> >>> Unable

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

2017-12-08 Thread Philippe Ombredanne
Wenyou, On Fri, Dec 8, 2017 at 2:55 AM, Wenyou Yang wrote: > The ov7740 (color) image sensor is a high performance VGA CMOS > image snesor, which supports for output formats: RAW RGB and YUV > and image sizes: VGA, and QVGA, CIF and any size smaller. > > Signed-off-by: Songjun Wu > Signed-off-by

[PATCH 0/2] of: overlay: Crash fix and improvement

2017-12-08 Thread Geert Uytterhoeven
Hi Pantelis, Rob, Frank, This patch series fixes memory corruption when applying overlays. I first noticed this when using OF configfs. After lots of failed debugging attempts, I bisected it to "of: overlay: add per overlay sysfs attributes", which is not upstream. But that was a red he

[PATCH 1/2] of: overlay: Fix out-of-bounds write in init_overlay_changeset()

2017-12-08 Thread Geert Uytterhoeven
If an overlay has no "__symbols__" node, but it has nodes without "__overlay__" subnodes at the end (e.g. a "__fixups__" node), after filling in all fragments for nodes with "__overlay__" subnodes, "fragment = &fragments[cnt]" will point beyond the end of the allocated array. Hence writing to "fra

[PATCH 2/2] of: overlay: Make node skipping in init_overlay_changeset() clearer

2017-12-08 Thread Geert Uytterhoeven
Make it more clear that nodes without "__overlay__" subnodes are skipped, by reverting the logic and using continue. This also reduces indentation level. Signed-off-by: Geert Uytterhoeven --- This applies to Rob's for-next branch only. --- drivers/of/overlay.c | 21 +++-- 1 file

Re: [PATCH v4 09/12] clk: qcom: Add Krait clock controller driver

2017-12-08 Thread Philippe Ombredanne
Sricharan, On Fri, Dec 8, 2017 at 12:00 PM, Sricharan R wrote: > Hi Philippe, > > > On 12/8/2017 3:53 PM, Philippe Ombredanne wrote: >> Sricharan, Stephen, >> >> On Fri, Dec 8, 2017 at 10:29 AM, Sricharan R >> wrote: >>> From: Stephen Boyd >>> >>> The Krait CPU clocks are made up of a primary

[PATCHv5 0/3] x86: 5-level related changes into decompression code

2017-12-08 Thread Kirill A. Shutemov
Here's few changes to x86 decompression code. The first patch is pure cosmetic change: it gives file with KASLR helpers a proper name. The last two patches bring support of booting into 5-level paging mode if a bootloader put the kernel above 4G. Patch 2/3 handles allocation of space for trampol

[PATCHv5 1/3] x86/boot/compressed/64: Rename pagetable.c to kaslr_64.c

2017-12-08 Thread Kirill A. Shutemov
The name of the file -- pagetable.c -- is misleading: it only contains helpers used for KASLR in 64-bit mode. Let's rename the file to reflect its content. Signed-off-by: Kirill A. Shutemov --- arch/x86/boot/compressed/Makefile| 2 +- arch/x86/boot/compressed/{pagetable.c =>

Re: [PATCH] dt-bindings: at24/eeprom: add an undocumented compatible string

2017-12-08 Thread Bartosz Golaszewski
2017-12-08 1:03 GMT+01:00 Javier Martinez Canillas : > Hello Rob and Bartosz, > > On Thu, Dec 7, 2017 at 11:50 PM, Rob Herring wrote: >> On Wed, Dec 06, 2017 at 11:12:19AM +0100, Bartosz Golaszewski wrote: >>> The "atmel,sdp" compatible is reported by checkpatch as undocumented. >>> >>> Add it to

[PATCHv5 2/3] x86/boot/compressed/64: Introduce place_trampoline()

2017-12-08 Thread Kirill A. Shutemov
If a bootloader enables 64-bit mode with 4-level paging, we might need to switch over to 5-level paging. The switching requires the disabling paging. It works fine if kernel itself is loaded below 4G. But if the bootloader put the kernel above 4G (not sure if anybody does this), we would lose cont

Re: [PATCH v7 19/37] tracing: Generalize per-element hist trigger data

2017-12-08 Thread Namhyung Kim
On Wed, Dec 06, 2017 at 04:38:00PM -0600, Tom Zanussi wrote: > Up until now, hist triggers only needed per-element support for saving > 'comm' data, which was saved directly as a private data pointer. > > In anticipation of the need to save other data besides 'comm', add a > new hist_elt_data stru

Re: [RFC] Linux Lab: lightweight board farm for daily kernel development and testing

2017-12-08 Thread cee1
Hi Zhangjin, Cool~ With Docker all necessary bits are encapsulated in images, which saves lots of time of setting up a development environment. The online "web console", http://tinylab.cloud:6080/labs/, makes it further more convenient. 2017-12-08 20:29 GMT+08:00 Wu Zhangjin : > > Hi all, > >

Re: [PATCH v9 3/6] clocksource: stm32: only use 32 bits timers

2017-12-08 Thread Benjamin Gaignard
2017-12-08 13:51 GMT+01:00 Daniel Lezcano : > On 08/12/2017 12:32, Benjamin Gaignard wrote: >> From: Benjamin Gaignard >> >> The clock driving counters is at 90MHz so the maximum period >> for 16 bis counters is around 728us (2^16 / 90.000.000). >> For 32 bits counters this period is close 47 seco

Re: [PATCH v6 09/11] fs/pipe.c: export create_pipe_files() and replace_fd()

2017-12-08 Thread Jarkko Sakkinen
On Thu, Dec 07, 2017 at 07:37:38PM +0200, Jarkko Sakkinen wrote: > I think I could replace pipes with anonymous inodes. Is that a better > idea than pipes? I can work on that v8 if the export is a show stopper > as it seems. We are using that to do some other stuff in tpm_vtpm_proxy. I'll go with

Re: [PATCH v7 15/37] tracing: Add variable support to hist triggers

2017-12-08 Thread Namhyung Kim
Hi Tom, On Wed, Dec 06, 2017 at 04:37:56PM -0600, Tom Zanussi wrote: > Add support for saving the value of a current event's event field by > assigning it to a variable that can be read by a subsequent event. > > The basic syntax for saving a variable is to simply prefix a unique > variable name

[GIT PULL] phy: for 4.15 -rc

2017-12-08 Thread Kishon Vijay Abraham I
Hi Greg, Please find the pull request for 4.15 -rc cycle below. It fixes a compilation error, fix in handling error case and there is a fix in exynos5-usbdrd to enumerate superspeed devices on Odroid XU3 (This includes a fix in usb/dwc3/core and I've got Acked-by from Felipe Balbi for this). Con

Re: [PATCH v2 2/2] uio: Introduce UIO driver dt-binding documentation

2017-12-08 Thread Andrey Zhizhikin
Hello Rob, Thanks for your reply here! > > No. See prior discussions: > > https://lkml.org/lkml/2016/5/18/457 > https://lkml.org/lkml/2017/10/8/238 Just for my clarify; to understand how to move on with the patch: Since UIO is not considered as a real HW and rather a aggregate, which could be us

Re: [PATCH v9 3/6] clocksource: stm32: only use 32 bits timers

2017-12-08 Thread Daniel Lezcano
On 08/12/2017 12:32, Benjamin Gaignard wrote: > From: Benjamin Gaignard > > The clock driving counters is at 90MHz so the maximum period > for 16 bis counters is around 728us (2^16 / 90.000.000). > For 32 bits counters this period is close 47 secondes which is > more acceptable. > > When using 1

[PATCH] futex: Avoid violating the 10th rule of futex

2017-12-08 Thread Peter Zijlstra
On Thu, Dec 07, 2017 at 05:02:40PM -0600, Gratian Crisan wrote: > Yep ... looks good to me. I've been running two targets with the > original reproducer for 8 hours now plus a target running the C test. > All of them are still going. > > I'm going to let them run overnight to make sure but I'm go

[PATCH 2/6] staging: fsl-dpaa2/eth: Don't set netdev->needed_headroom

2017-12-08 Thread Ioana Radulescu
Commit 4b2d9fe87950 ("staging: fsl-dpaa2/eth: Extra headroom in RX buffers") tried to avoid the performance penalty of doing skb reallocations in the network stack for IP forwarded frames between two DPAA2 Ethernet interfaces. This led to a (too) complicated formula that relies on the stack's inter

[PATCH 4/6] staging: fsl-dpaa2/eth: Don't enable FAS on Tx

2017-12-08 Thread Ioana Radulescu
For Tx confirmed frames that have an error indication in the frame descriptor, we look at the Frame Annotation Status field (in the buffer headroom) for details on the root cause and then print a debug message with that information. While useful in initial development stages, it doesn't bring enou

[PATCH 5/6] staging: fsl-dpaa2/eth: Compute needed headroom per frame

2017-12-08 Thread Ioana Radulescu
For non-linear skbs we build scatter-gather frames and allocate a new buffer for the S/G table in which we reserve the required headroom, so the actual skb headroom size doesn't matter. Rather than use a one-size-fits-all approach, decide when to enforce headroom requirements on a frame by frame b

[PATCH 0/6] staging: fsl-dpaa2/eth: Frame buffer optimizations

2017-12-08 Thread Ioana Radulescu
This patchset continues the work related to frame buffer layout. The first two patches fix an issue and simplify some rather convoluted formulas introduced by a previous commit. The third adds a new counter for TX skb reallocations due to insufficient headroom. This helped us notice that most of

[PATCH 1/6] staging: fsl-dpaa2/eth: Fix access to FAS field

2017-12-08 Thread Ioana Radulescu
Commit 4b2d9fe87950 ("staging: fsl-dpaa2/eth: Extra headroom in RX buffers") removes the software annotation (SWA) area from the RX buffer layout, as it's not used by anyone, but fails to update the macros for accessing hardware annotation (HWA) fields, which is right after the SWA in the buffer he

[PATCH 6/6] staging: fsl-dpaa2/eth: Make Tx buffer alignment optional

2017-12-08 Thread Ioana Radulescu
Aligning the Tx buffers at 64B is a performance optimization recommendation, not a hard requirement. Make optional the alignment of Tx FD buffers, without enforcing a reallocation in case there is not enough headroom for it. On Rx, we keep allocating buffers with enough headroom to allow Tx align

[PATCH 3/6] staging: fsl-dpaa2/eth: Add counter for skb reallocs

2017-12-08 Thread Ioana Radulescu
Add a counter for the number of egress frames that need to be realloc'ed due to insufficient headroom space. Signed-off-by: Ioana Radulescu --- drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c | 1 + drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.h | 1 + drivers/staging/fsl-dpaa2/ethernet/d

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

2017-12-08 Thread Greentime Hu
Hi, Marc: 2017-12-08 20:29 GMT+08:00 Marc Zyngier : > On 08/12/17 11:54, Greentime Hu wrote: >> Hi, Mark: >> >> 2017-12-08 18:21 GMT+08:00 Mark Rutland : >>> On Fri, Dec 08, 2017 at 05:12:00PM +0800, Greentime Hu wrote: From: Greentime Hu This patch adds VDSO support. The VDSO code

[GIT PULL] Please pull powerpc/linux.git powerpc-4.15-4 tag

2017-12-08 Thread Michael Ellerman
Hi Linus, Please pull some more powerpc fixes for 4.15: The following changes since commit ae64f9bd1d3621b5e60d7363bc20afb46aede215: Linux 4.15-rc2 (2017-12-03 11:01:47 -0500) are available in the Git repository at: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git tags/po

Re: [PATCH] futex: use fault_in to avoid infinite loop

2017-12-08 Thread chengjian (D)
On 2017/12/7 5:40, Peter Zijlstra wrote: @@ -3262,6 +3262,8 @@ static int futex_wait_requeue_pi(u32 __user *uaddr, unsigned int flags, SYSCALL_DEFINE2(set_robust_list, struct robust_list_head __user *, head, size_t, len) { + unsigned long address = (unsigned long)head

Re: [RFC PATCH v3 1/7] ktask: add documentation

2017-12-08 Thread Michal Hocko
On Wed 06-12-17 15:32:48, Daniel Jordan wrote: > On 12/06/2017 09:35 AM, Michal Hocko wrote: [...] > > There is also no mention about other > > characteristics (e.g. power management), resource isloataion etc. So > let > > me ask again. How do you control that the parallelized operation > > doesn'

AF_ALG: skb limits

2017-12-08 Thread Stephan Mueller
Am Freitag, 8. Dezember 2017, 12:39:06 CET schrieb Jonathan Cameron: Hi Jonathan, > > As a heads up, the other nasties we've found so far are around hitting > limits on the various socket buffers. When you run into those you can end > up with parts of the data to be encrypted going through with

Re: [PATCH] KVM: X86: Fix host dr6 miss restore

2017-12-08 Thread David Hildenbrand
On 08.12.2017 10:12, Wanpeng Li wrote: > From: Wanpeng Li > > Reported by syzkaller: > >WARNING: CPU: 0 PID: 12927 at arch/x86/kernel/traps.c:780 > do_debug+0x222/0x250 >CPU: 0 PID: 12927 Comm: syz-executor Tainted: G OE > 4.15.0-rc2+ #16 >RIP: 0010:do_debug+0x222/0x25

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

2017-12-08 Thread Dhaval Shah
SPDX-License-Identifier is used for the Xilinx Video IP and related drivers. Signed-off-by: Dhaval Shah --- drivers/media/platform/xilinx/xilinx-dma.c | 5 + drivers/media/platform/xilinx/xilinx-dma.h | 5 + drivers/media/platform/xilinx/xilinx-tpg.c | 5 + drivers/media/platform/

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

2017-12-08 Thread Huang, Ying
Minchan Kim writes: > On Fri, Dec 08, 2017 at 04:41:38PM +0800, Huang, Ying wrote: >> Minchan Kim writes: >> >> > On Fri, Dec 08, 2017 at 01:41:10PM +0800, Huang, Ying wrote: >> >> Minchan Kim writes: >> >> >> >> > On Thu, Dec 07, 2017 at 04:29:37PM -0800, Andrew Morton wrote: >> >> >> On Thu

Re: [RFC PATCH 2/5] perf jevents: add support for arch recommended events

2017-12-08 Thread Jiri Olsa
On Wed, Dec 06, 2017 at 02:40:10PM +, John Garry wrote: > On 06/12/2017 13:37, Jiri Olsa wrote: > > On Wed, Dec 06, 2017 at 12:13:16AM +0800, John Garry wrote: > > > > SNIP > > > > > --- > > > tools/perf/pmu-events/jevents.c | 215 > > > > > > 1 file

[PATCH 1/1] base: power: runtime: Export pm_runtime_get/put_suppliers

2017-12-08 Thread Vivek Gautam
The device link allows the pm framework to tie the supplier and consumer. So, whenever the consumer is powered-on, the supplier is powered-on first. There are however cases in which the consumer wants to power-on the supplier, but not itself. E.g., A Graphics or multimedia driver wants to power-on

Re: linux-next: manual merge of the wireless-drivers-next tree with the wireless-drivers tree

2017-12-08 Thread Kalle Valo
Stephen Rothwell writes: > Today's linux-next merge of the wireless-drivers-next tree got a > conflict in: > > drivers/net/wireless/intel/iwlwifi/pcie/drv.c > > between commit: > > 567deca8e72d ("iwlwifi: add new cards for 9260 and 22000 series") > > from the wireless-drivers tree and commit:

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

2017-12-08 Thread Marc Zyngier
On 08/12/17 11:54, Greentime Hu wrote: > Hi, Mark: > > 2017-12-08 18:21 GMT+08:00 Mark Rutland : >> On Fri, Dec 08, 2017 at 05:12:00PM +0800, Greentime Hu wrote: >>> From: Greentime Hu >>> >>> This patch adds VDSO support. The VDSO code is currently used for >>> sys_rt_sigreturn() and optimised g

Re: [RFC PATCH 2/5] perf jevents: add support for arch recommended events

2017-12-08 Thread Jiri Olsa
On Wed, Dec 06, 2017 at 03:20:14PM +, John Garry wrote: > On 06/12/2017 13:36, Jiri Olsa wrote: > > On Wed, Dec 06, 2017 at 12:13:16AM +0800, John Garry wrote: > > > For some architectures (like arm64), there are architecture- > > > defined recommended events. Vendors may not be obliged to > >

Re: [PATCH] powerpc/xmon: use ARRAY_SIZE on various array sizing calculations

2017-12-08 Thread Joe Perches
On Fri, 2017-12-08 at 22:51 +1100, Michael Ellerman wrote: > Balbir Singh writes: > > > On Thu, Dec 7, 2017 at 10:01 PM, Colin King > > wrote: > > > From: Colin Ian King > > > > > > Use the ARRAY_SIZE macro on several arrays to determine their size. > > > Improvement suggested by coccinelle.

Re: KASAN: slab-out-of-bounds Read in strcmp

2017-12-08 Thread Dmitry Vyukov
On Tue, Dec 5, 2017 at 11:00 AM, Dmitry Vyukov wrote: >>> > > > > > === >>> > > > > > === >>> > > > > > BUG: KASAN: slab-out-of-bounds in strcmp+0x96/0xb0 >>> > > > > > lib/string.c:328 >>> > > > > > Read of size 1 at addr 8801cd99d2c

[PATCH] clkc: zynq: Missing a blank line after declarations.

2017-12-08 Thread Dhaval Shah
Resolved missing a blank line after declarations checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/clk/zynq/clkc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/clk/zynq/clkc.c b/drivers/clk/zynq/clkc.c index 88a2cab37f62..1cd4705e405e 100644 --

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

2017-12-08 Thread 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_node *timer_node; > >> + > >> + timer_node = o

RE: [Regression 4.15-rc2] New messages `tpm tpm0: A TPM error (2314) occurred continue selftest`

2017-12-08 Thread Alexander.Steffen
> On Thu, Dec 07, 2017 at 03:56:07PM +, alexander.stef...@infineon.com > wrote: > > > > If these are intentional, it’d be great > > > to give some hint to the user, what effect this has. > > > > I agree that those error messages in their current form are not that > > helpful for the users. But

Re: [PATCH v5 3/7] scsi: libsas: make the event threshold configurable

2017-12-08 Thread John Garry
On 08/12/2017 09:42, Jason Yan wrote: Add a sysfs attr that LLDD can configure it for every host. We made a example in hisi_sas. Other LLDDs using libsas can implement it if they want. Suggested-by: Hannes Reinecke Signed-off-by: Jason Yan CC: John Garry CC: Johannes Thumshirn CC: Ewan Milne

Re: USB: hub: Checking communication difficulties

2017-12-08 Thread SF Markus Elfring
> Greg maintains USB and he's has blocked Markus, How do you think about to reconsider this blockage? > because he never listens to feedback I am listening … > but instead just repsonds that he has a different opinion. I choose such a reaction in some cases. My responses can vary. It seems

Re: [PATCH v2 0/5] perf-probe: Improve probing on versioned symbols

2017-12-08 Thread Thomas-Mich Richter
On 12/07/2017 08:19 AM, Masami Hiramatsu wrote: > Hi, > > Here is the 2nd version of the series for probing on > versioned symbols in libraries. This includes 5 patches > to fix the issues discussed on perf-users ML > (https://www.spinics.net/lists/linux-perf-users/msg04637.html) > > The first v

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

2017-12-08 Thread Greentime Hu
Hi, Mark: 2017-12-08 18:21 GMT+08:00 Mark Rutland : > On Fri, Dec 08, 2017 at 05:12:00PM +0800, Greentime Hu wrote: >> From: Greentime Hu >> >> This patch adds VDSO support. The VDSO code is currently used for >> sys_rt_sigreturn() and optimised gettimeofday() (using the SoC timer >> counter). >

Re: [PATCH] perf tests: Handle properly readdir DT_UNKNOWN

2017-12-08 Thread Jiri Olsa
On Thu, Dec 07, 2017 at 12:19:05PM -0300, Arnaldo Carvalho de Melo wrote: > Em Wed, Dec 06, 2017 at 06:45:35PM +0100, Jiri Olsa escreveu: > > Some system can return DT_UNKNOWN in readdir's struct dirent::d_type > > and we must handle it properly. In this case we can directly check > > if the entity

Re: [PATCH v2] perf evsel: Enable ignore_missing_thread for pid option

2017-12-08 Thread Jiri Olsa
On Thu, Dec 07, 2017 at 09:43:33PM +0800, Mengting Zhang wrote: SNIP > diff --git a/tools/perf/util/evsel.c b/tools/perf/util/evsel.c > index f894893..d0ef889 100644 > --- a/tools/perf/util/evsel.c > +++ b/tools/perf/util/evsel.c > @@ -1592,10 +1592,43 @@ static int __open_attr__fprintf(FILE *fp,

Re: [PATCH] powerpc/xmon: use ARRAY_SIZE on various array sizing calculations

2017-12-08 Thread Michael Ellerman
Balbir Singh writes: > On Thu, Dec 7, 2017 at 10:01 PM, Colin King wrote: >> From: Colin Ian King >> >> Use the ARRAY_SIZE macro on several arrays to determine their size. >> Improvement suggested by coccinelle. > > This file is taken from binutils and re-licensed. Keeping the file > as-is help

Re: [PATCH v2 2/3] x86/acpi: take rsdp address for boot params if available

2017-12-08 Thread Juergen Gross
On 08/12/17 12:26, Ingo Molnar wrote: > > * Juergen Gross wrote: > >> On 08/12/17 08:05, Ingo Molnar wrote: >>> >>> * Juergen Gross wrote: >> >> ... >> >>> acpi_physical_address acpi_arch_get_root_pointer(void) >>> { >>> return boot_params.hdr.acpi_rsdp_addr; >>> } >>> >>> 4) >>> >>> Add th

Re: [PATCH v2] mm: terminate shrink_slab loop if signal is pending

2017-12-08 Thread Michal Hocko
On Fri 08-12-17 20:36:16, Tetsuo Handa wrote: > On 2017/12/08 17:22, Michal Hocko wrote: > > On Thu 07-12-17 17:23:05, Suren Baghdasaryan wrote: > >> Slab shrinkers can be quite time consuming and when signal > >> is pending they can delay handling of the signal. If fatal > >> signal is pending the

Re: [v2 PATCH] cpufreq: powernv: Correctly parse the sign of pstates on POWER8 vs POWER9

2017-12-08 Thread Michael Ellerman
"Rafael J. Wysocki" writes: > On Thu, Dec 7, 2017 at 6:59 AM, Gautham R. Shenoy > wrote: >> From: "Gautham R. Shenoy" >> >> On POWERNV platform, Pstates are 8-bit values. On POWER8 they are >> negatively numbered while on POWER9 they are positively >> numbered. Thus, on POWER9, the maximum numb

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

2017-12-08 Thread Prateek Sood
On 12/08/2017 03:10 PM, Prateek Sood wrote: > On 12/05/2017 04:31 AM, Peter Zijlstra wrote: >> On Mon, Dec 04, 2017 at 02:58:25PM -0800, Tejun Heo wrote: >>> Hello, again. >>> >>> On Mon, Dec 04, 2017 at 12:22:19PM -0800, Tejun Heo wrote: Hello, On Mon, Dec 04, 2017 at 10:44:49AM +05

Re: [PATCH v2 5/5] perf-probe: Support escaped character in parser

2017-12-08 Thread Thomas-Mich Richter
On 12/07/2017 08:21 AM, Masami Hiramatsu wrote: > Support the special characters escaped by '\' in parser. > This allows user to specify versions directly like below. > > = > # ./perf probe -x /lib64/libc-2.25.so malloc_get_state\\@GLIBC_2.2.5 > Added new event: > probe_libc:malloc_g

Re: [Intel-gfx] [PATCH] drm/i915: Remove unused IRQ chip data of HDMI LPE audio

2017-12-08 Thread Ville Syrjälä
On Fri, Dec 08, 2017 at 05:33:23PM +0800, Augustine.Chen wrote: > The chip data of HDMI LPE audio is set to drm_i915_private which is not > consistent with the expectation by x86 APIC driver. Hmm. Why is the apic code looking at data for an irq chip it hasn't created? Do we need something like -

Re: [PATCH v2] ubsan: don't handle misaligned address when support unaligned access

2017-12-08 Thread Andrey Ryabinin
On 12/08/2017 02:14 PM, David Laight wrote: > From: Andrey Ryabinin >> Sent: 08 December 2017 10:49 > ... >> CONFIG_UBSAN_ALIGNMENT is already disabled by default for >> HAVE_EFFICIENT_UNALIGNED_ACCESS=y because it's noisy, >> but we still allow users to enable it if they want to. >> >> I don't th

Re: [PATCH] crypto: AF_ALG - fix race accessing cipher request

2017-12-08 Thread Jonathan Cameron
On Fri, 8 Dec 2017 11:50:37 +0100 Stephan Müller wrote: > Hi Herbert, > > This patch would go on top of 7d2c3f54e6f646887d019faa45f35d6fe9fe82ce > "crypto: af_alg - remove locking in async callback" found in Linus' tree > which is not yet in the cryptodev-2.6 tree. > > In addition, this patch i

Re: [PATCH v2] mm: terminate shrink_slab loop if signal is pending

2017-12-08 Thread Tetsuo Handa
On 2017/12/08 17:22, Michal Hocko wrote: > On Thu 07-12-17 17:23:05, Suren Baghdasaryan wrote: >> Slab shrinkers can be quite time consuming and when signal >> is pending they can delay handling of the signal. If fatal >> signal is pending there is no point in shrinking that process >> since it wil

Re: [PATCH] tools/perf/inject: fix dwarf support detection

2017-12-08 Thread Jiri Olsa
On Wed, Dec 06, 2017 at 07:08:04PM -0800, Stephane Eranian wrote: > I ran into problems trying to use the JIT support and display > source-level information. Basically, there was no dwarf debug > info generated in the jitted-XX.so files, yet I had libdw-dev > installed. > > Turns out that the feat

Re: [PATCH AUTOSEL for 4.14 012/135] ASoC: cs42l56: Fix reset GPIO name in example DT binding

2017-12-08 Thread Mark Brown
On Thu, Dec 07, 2017 at 09:03:01PM +, alexander.le...@verizon.com wrote: > On Thu, Dec 07, 2017 at 05:25:02PM +, Mark Brown wrote: > >We shouldn't be getting into adding completely new DT properties in > >stable backports like this. Old kernels have the bindings they have. > I thought th

[PATCH v9 2/6] clocksource: stm32: increase min delta value

2017-12-08 Thread Benjamin Gaignard
From: Benjamin Gaignard The CPU is a CortexM4 @ 200MHZ and the clocks driving the timers are at 90MHZ with a min delta at 1 you could have an interrupt each 0.01 ms which is really to much. By increase it to 0x60 it give more time (around 1 ms) to CPU to handle the interrupt. Signed-off-by: Benj

[PATCH v9 5/6] clocksource: stm32: Update license and copyright

2017-12-08 Thread Benjamin Gaignard
Adopt SPDX License Identifier and add STMicroelectronics copyright Signed-off-by: Benjamin Gaignard --- drivers/clocksource/timer-stm32.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/clocksource/timer-stm32.c b/drivers/clocksource/timer-stm32.c index c9aed23141

[PATCH v9 6/6] arm: dts: stm32: remove useless clocksource nodes

2017-12-08 Thread Benjamin Gaignard
From: Benjamin Gaignard 16 bits timers aren't accurate enough to be used as clocksource, remove them from stm32f4 and stm32f7 devicetree. Signed-off-by: Benjamin Gaignard --- arch/arm/boot/dts/stm32f429.dtsi | 32 arch/arm/boot/dts/stm32f746.dtsi | 32 -

[PATCH v9 3/6] clocksource: stm32: only use 32 bits timers

2017-12-08 Thread Benjamin Gaignard
From: Benjamin Gaignard The clock driving counters is at 90MHz so the maximum period for 16 bis counters is around 728us (2^16 / 90.000.000). For 32 bits counters this period is close 47 secondes which is more acceptable. When using 16 bits counters the kernel may not be able to boot because it

Re: [PATCHv2] tools: hv: hv_set_ifconfig.sh double check before setting ip

2017-12-08 Thread Olaf Hering
On Fri, Dec 08, Eduardo Otubo wrote: > tools/hv/hv_set_ifconfig.sh | 45 > +++-- > 1 file changed, 43 insertions(+), 2 deletions(-) > +# let's wait for 3 minutes Was this codepath runtime tested? Last time this came up, the conclusion was that W

[PATCH v9 1/6] clocksource: stm32: convert driver to timer_of

2017-12-08 Thread Benjamin Gaignard
From: Benjamin Gaignard Convert driver to use timer_of helpers. This allow to remove custom proprietary structure. Signed-off-by: Benjamin Gaignard --- drivers/clocksource/Kconfig | 1 + drivers/clocksource/timer-stm32.c | 160 ++ 2 files changed, 58

[PATCH v9 4/6] clocksource: stm32: add clocksource support

2017-12-08 Thread Benjamin Gaignard
The stm32 timer hardware is currently only used as a clock event device, but it can be utilized as a clocksource as well. Implement this by enabling the free running counter in the hardware block and converting the clock event part from a count down event timer to a comparator based timer. Signed

[PATCH v9 0/6] stm32 clocksource driver rework

2017-12-08 Thread Benjamin Gaignard
version 9: - rebased on timer/master where timer_of_cleanup() exist - fix the comments done by Daniel on v8 - reword commits message to be more explicit about 16 bits counter issue - add one patch about copyrights and licences version 8: - rebased on timers/core - change timer_of_exit() n

Re: [PATCH] LDT improvements

2017-12-08 Thread Ingo Molnar
* Thomas Gleixner wrote: > On Fri, 8 Dec 2017, Ingo Molnar wrote: > > * Thomas Gleixner wrote: > > > > > On Fri, 8 Dec 2017, Ingo Molnar wrote: > > > > * Andy Lutomirski wrote: > > > > > I don't love mucking with user address space. I'm also quite nervous > > > > > about > > > > > putting

Re: [PATCHv4 3/4] x86/boot/compressed/64: Introduce place_trampoline()

2017-12-08 Thread Ingo Molnar
* Kirill A. Shutemov wrote: > On Thu, Dec 07, 2017 at 07:30:48AM +0100, Ingo Molnar wrote: > > > We also need a small stack in the trampoline to re-enable long mode via > > > long return. But stack and code can share the page just fine. > > > > BTW., I'm not sure this is necessarily a good idea

Re: Multiple oom_reaper BUGs: unmap_page_range racing with exit_mmap

2017-12-08 Thread Michal Hocko
On Fri 08-12-17 01:26:46, David Rientjes wrote: > On Thu, 7 Dec 2017, David Rientjes wrote: > > > I'm backporting and testing the following patch against Linus's tree. To > > clarify an earlier point, we don't actually have any change from upstream > > code that allows for free_pgtables() befor

Re: [PATCH v2 2/3] x86/acpi: take rsdp address for boot params if available

2017-12-08 Thread Ingo Molnar
* Juergen Gross wrote: > On 08/12/17 08:05, Ingo Molnar wrote: > > > > * Juergen Gross wrote: > > ... > > > acpi_physical_address acpi_arch_get_root_pointer(void) > > { > > return boot_params.hdr.acpi_rsdp_addr; > > } > > > > 4) > > > > Add this to arch/x86/include/asm/acpi.h: > > > >

Re: [PATCH] [powerpc-next] Fix powerpc64 alignment of .toc section in kernel modules

2017-12-08 Thread Michael Ellerman
Desnes Augusto Nunes do Rosário writes: ... > > Lastly, will you fix it up or do you want me to send a second version > then? Whatever is best for you. No that's fine I've done those changes, no need to send another version. cheers

Re: [RFC patch 7/7] timekeeping: Hack to use fine grained timestamps during boot

2017-12-08 Thread Petr Mladek
On Tue 2017-11-28 19:47:09, Thomas Gleixner wrote: > On Tue, 28 Nov 2017, Prarit Bhargava wrote: > > On 11/23/2017 07:58 AM, Petr Mladek wrote: > > > On Wed 2017-11-15 19:15:38, Thomas Gleixner wrote: > > >> For demonstration purposes only. > > >> > > >> Add a disgusting hack to work around the fac

Re: [PATCH 1/9] sched/autogroup: remove unneeded kallsyms include

2017-12-08 Thread Sergey Senozhatsky
On (12/08/17 11:56), Peter Zijlstra wrote: > On Fri, Dec 08, 2017 at 11:56:08AM +0900, Sergey Senozhatsky wrote: > > Autogroup does not seem to use any of kallsyms functions/defines. > > > > Signed-off-by: Sergey Senozhatsky > > Cc: Peter Zijlstra > > Feel free to take this through whatever tre

Re: [PATCH v3 1/3] PCI: designware-ep: Fix find_first_zero_bit() usage

2017-12-08 Thread Lorenzo Pieralisi
On Mon, Nov 27, 2017 at 04:55:05PM +, Lorenzo Pieralisi wrote: > On Mon, Nov 27, 2017 at 04:49:53PM +0100, Niklas Cassel wrote: > > find_first_zero_bit()'s parameter 'size' is defined in bits, > > not in bytes. > > > > find_first_zero_bit() was called with size in bytes rather than bits, > > w

Re: USB: hub: Delete an error message for a failed memory allocation in usb_hub_clear_tt_buffer()

2017-12-08 Thread Dan Carpenter
On Wed, Dec 06, 2017 at 09:20:05PM +0100, Geert Uytterhoeven wrote: > Hi Markus, > > On Wed, Dec 6, 2017 at 6:51 PM, SF Markus Elfring > wrote: > >> The system will come to a grinding halt anyway if it can't allocate 24 or > >> 40 bytes. > > > > Maybe. > > Since you've been sending zillions of

Re: [PATCH v2 2/3] x86/acpi: take rsdp address for boot params if available

2017-12-08 Thread Juergen Gross
On 08/12/17 08:05, Ingo Molnar wrote: > > * Juergen Gross wrote: ... > acpi_physical_address acpi_arch_get_root_pointer(void) > { > return boot_params.hdr.acpi_rsdp_addr; > } > > 4) > > Add this to arch/x86/include/asm/acpi.h: > > extern acpi_physical_address acpi_arch_get_root_pointer

RE: [PATCH v2] ubsan: don't handle misaligned address when support unaligned access

2017-12-08 Thread David Laight
From: Andrey Ryabinin > Sent: 08 December 2017 10:49 ... > CONFIG_UBSAN_ALIGNMENT is already disabled by default for > HAVE_EFFICIENT_UNALIGNED_ACCESS=y because it's noisy, > but we still allow users to enable it if they want to. > > I don't think we should completely forbid enabling it for > HA

[kernel-hardening][PATCH] arm: mm: idmap: Mark variables as ro_after_init

2017-12-08 Thread Jinbum Park
idmap_pgd, arch_phys_to_idmap_offset are setup once while init stage, and never changed after that. so, it is good candidate for __ro_after_init. Signed-off-by: Jinbum Park --- arch/arm/mm/idmap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/mm/idmap.c b/arch/

Re: [PATCH v2 4/5] perf-probe: Find versioned symbols from map

2017-12-08 Thread Thomas-Mich Richter
On 12/07/2017 08:21 AM, Masami Hiramatsu wrote: > Find versioned symbols correctly from map. > Commit d80406453ad4 ("perf symbols: Allow user probes on > versioned symbols") allows user to find default versioned > symbols (with "@@") in map. However, it did not enable > normal versioned symbol (wit

[PATCH net v3] net: phy: meson-gxl: detect LPA corruption

2017-12-08 Thread Jerome Brunet
The purpose of this change is to fix the incorrect detection of the link partner (LP) advertised capabilities which sometimes happens with this PHY (roughly 1 time in a dozen) This issue may cause the link to be negotiated at 10Mbps/Full or 10Mbps/Half when 100MBps/Full is actually possible. In so

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

2017-12-08 Thread Michael Ellerman
Matthew Wilcox writes: > On Thu, Dec 07, 2017 at 11:14:27AM -0800, Kees Cook wrote: >> On Wed, Dec 6, 2017 at 9:46 PM, Michael Ellerman wrote: >> > Matthew Wilcox writes: >> >> So, just like we currently say "exactly one of MAP_SHARED or MAP_PRIVATE", >> >> we could add a new paragraph saying "

Re: [PATCHv4 3/4] x86/boot/compressed/64: Introduce place_trampoline()

2017-12-08 Thread Kirill A. Shutemov
On Thu, Dec 07, 2017 at 07:30:48AM +0100, Ingo Molnar wrote: > > We also need a small stack in the trampoline to re-enable long mode via > > long return. But stack and code can share the page just fine. > > BTW., I'm not sure this is necessarily a good idea: it means > writable+executable > memo

Re: [PATCH] usb-core: Fix potential null pointer dereference in xhci-debugfs.c

2017-12-08 Thread Alexander Kappner
Hi, > I think we need to dig a bit deeper. It's good to check if spriv is valid but there are probably other reasons than kzalloc failing. I agree -- this small allocation is unlikely to fail in practice. Also, while my patch prevents the kernel oops, it also prevents the debugfs entries fr

Re: [PATCH v5 net-next] net/tcp: trace all TCP/IP state transition with tcp_set_state tracepoint

2017-12-08 Thread Marcelo Ricardo Leitner
On Fri, Dec 08, 2017 at 11:40:23AM +0800, Yafang Shao wrote: > It will looks like these, > > if (sk->sk_protocol == IPPROTO_TCP) > __tcp_set_state(newsk, TCP_SYN_RECV); > else > newsk->sk_state = TCP_SYN_RECV; > > > if (sk->sk_protocol == IPPROTO_TCP) > __tc

[kernel-hardening][PATCH v6 3/3] arm: mm: dump: add checking for writable and executable pages

2017-12-08 Thread Jinbum Park
Page mappings with full RWX permissions are a security risk. x86, arm64 has an option to walk the page tables and dump any bad pages. (1404d6f13e47 ("arm64: dump: Add checking for writable and exectuable pages")) Add a similar implementation for arm. Reviewed-by: Kees Cook Tested-by: Laura Abbot

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

2017-12-08 Thread Catalin Marinas
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/arm64/in

[kernel-hardening][PATCH v6 2/3] arm: mm: dump: make the page table dumping seq_file optional

2017-12-08 Thread Jinbum Park
This patch makes the page table dumping seq_file optional. It makes the page table dumping code usable for other cases. This patch refers below commit of arm64. (ae5d1cf358a5 ("arm64: dump: Make the page table dumping seq_file optional")) Reviewed-by: Kees Cook Tested-by: Laura Abbott Reviewed-

[kernel-hardening][PATCH v6 1/3] arm: mm: dump: make page table dumping reusable

2017-12-08 Thread Jinbum Park
This patch refactors the arm page table dumping code, so multiple tables may be registered with the framework. This patch refers below commits of arm64. (4674fdb9f149 ("arm64: mm: dump: make page table dumping reusable")) (4ddb9bf83349 ("arm64: dump: Make ptdump debugfs a separate option")) Revie

Re: [PATCH v4 09/12] clk: qcom: Add Krait clock controller driver

2017-12-08 Thread Sricharan R
Hi Philippe, On 12/8/2017 3:53 PM, Philippe Ombredanne wrote: > Sricharan, Stephen, > > On Fri, Dec 8, 2017 at 10:29 AM, Sricharan R wrote: >> From: Stephen Boyd >> >> The Krait CPU clocks are made up of a primary mux and secondary >> mux for each CPU and the L2, controlled via cp15 accessors.

[kernel-hardening][PATCH v6 0/3] arm: Makes ptdump resuable and add WX page checking

2017-12-08 Thread Jinbum Park
Hi, Page table dumping code for arm64-x86 is reusable, and they have function for WX page checking. But arm doesn't have that. This path series are to makes ptdump reusable, and add WX page checking for arm. This is heavily based on arm64 version. v2 : Fix a sender name of mail header, there was

Re: [PATCH] sched/autogroup: move sched.h include

2017-12-08 Thread Peter Zijlstra
On Fri, Dec 08, 2017 at 05:24:22PM +0900, Sergey Senozhatsky wrote: > Move local "sched.h" include to the bottom. sched.h defines > several macros that are getting redefined in ARCH-specific > code, for instance, finish_arch_post_lock_switch() and > prepare_arch_switch(), so we need ARCH-specific d

Re: [PATCH 1/9] sched/autogroup: remove unneeded kallsyms include

2017-12-08 Thread Peter Zijlstra
On Fri, Dec 08, 2017 at 11:56:08AM +0900, Sergey Senozhatsky wrote: > Autogroup does not seem to use any of kallsyms functions/defines. > > Signed-off-by: Sergey Senozhatsky > Cc: Peter Zijlstra Feel free to take this through whatever tree you need this in. Acked-by: Peter Zijlstra (Intel) >

Re: [PATCH] kthread: finer-grained lockdep/cross-release completion

2017-12-08 Thread Peter Zijlstra
On Thu, Dec 07, 2017 at 11:08:49AM +0100, Daniel Vetter wrote: > Since -rc1 we're hitting a bunch of lockdep splats using the new > cross-release stuff around the 2 kthread completions. In all cases > they are because totally independent uses of kthread are mixed up by > lockdep into the same locki

[PATCH 0/6] x86/kvm/hyperv: stable clocksorce for L2 guests when running nested KVM on Hyper-V

2017-12-08 Thread Vitaly Kuznetsov
Changes since RFC: - Handle disabled TSC page case [Radim Krčmář] - Function/parameter renames [Radim Krčmář] - Protect against simultaneous CPU shutdown (future-proof) [Radim Krčmář] - BUG() in !CONFIG_HYPERV_TSCPAGE case [Stephen Hemminger, Paolo Bonzini] - Fix build issues (hopefully) [kbuild te

Re: [PATCH v2] usb: host: Implement workaround for Erratum A-009611

2017-12-08 Thread Felipe Balbi
Hi, Greg Kroah-Hartman writes: > On Fri, Dec 08, 2017 at 05:49:41PM +0800, yinbo@nxp.com wrote: >> From: "yinbo.zhu" >> >> Description: This is a occasional problem where the software > > No need for a "Description:" word. That's just assumed here, right? > >> issues an End Transfer comma

[PATCH 2/6] x86/hyper-v: add a function to read both TSC and TSC page value simulateneously

2017-12-08 Thread Vitaly Kuznetsov
This is going to be used from KVM code where we need to get both TSC and TSC page value. When Hyper-V code is compiled out just return rdtsc(), this will allow us to avoid ugly ifdefs in non-Hyper-V code. Signed-off-by: Vitaly Kuznetsov --- RFC -> v1: - EXPORT_SYMBOL_GPL(hv_get_tsc_page) [kbuild

<    2   3   4   5   6   7   8   9   10   >