Re: [RFC, PATCHv1 00/28] 5-level paging

2016-12-09 Thread Kirill A. Shutemov
On Fri, Dec 09, 2016 at 08:40:11AM -0800, Andi Kleen wrote: > > On other hand, large virtual address space would put more pressure on > > cache -- at least one more page table per process, if we make 56-bit VA > > default. > > The top level page always has to be there unless you disable it at boot

Re: [PATCH] sched/pid fix use-after free in task_tgid_vnr

2016-12-09 Thread Oleg Nesterov
On 12/09, EunTaik Lee wrote: > > There is a use-after-free case with below call stack. > > pid_nr_ns+0x10/0x38 > cgroup_pidlist_start+0x144/0x400 > cgroup_seqfile_start+0x1c/0x24 > kernfs_seq_start+0x54/0x90 > seq_read+0x15c/0x3a8 > kernfs_fop_read+0x38/0x160 > __vfs_read+0x28/0xc8 > vfs_read+0x84/

RE: [PATCH v3 1/9] staging: fsl-mc: move bus driver out of staging

2016-12-09 Thread Stuart Yoder
> > > Where is the device freed? I see you trying to do some "odd" stuff in > > > fsl_mc_device_remove() by deleting and then putting a device structure. > > > I can't find a "release()" callback anywhere for your bus, where is it? > > > > > > What happens when the reference count falls to 0 for y

Re: [PATCH 1/2] mm, page_alloc: don't convert pfn to idx when merging

2016-12-09 Thread Mel Gorman
On Fri, Dec 09, 2016 at 10:37:53AM +0100, Vlastimil Babka wrote: > In __free_one_page() we do the buddy merging arithmetics on "page/buddy > index", > which is just the lower MAX_ORDER bits of pfn. The operations we do that > affect > the higher bits are bitwise AND and subtraction (in that order

Re: Tearing down DMA transfer setup after DMA client has finished

2016-12-09 Thread Måns Rullgård
Vinod Koul writes: > On Fri, Dec 09, 2016 at 11:25:57AM +0100, Sebastian Frias wrote: >> >> What concrete solution do you propose? > > I have already proposed two solutions. > > A) Request a channel only when you need it. Obviously we can't do virtual > channels with this (though we should still

Re: [PATCH] uio-hv-generic: store physical addresses instead of virtual

2016-12-09 Thread Stephen Hemminger
On Fri, 9 Dec 2016 12:44:40 +0100 Arnd Bergmann wrote: > gcc warns about the newly added driver when phys_addr_t is wider than > a pointer: > > drivers/uio/uio_hv_generic.c: In function 'hv_uio_mmap': > drivers/uio/uio_hv_generic.c:71:17: error: cast to pointer from integer of > different size

Re: Still OOM problems with 4.9er kernels

2016-12-09 Thread Michal Hocko
On Fri 09-12-16 17:58:14, Gerhard Wiesinger wrote: > On 09.12.2016 17:09, Michal Hocko wrote: [...] > > > [97883.882611] Mem-Info: > > > [97883.883747] active_anon:2915 inactive_anon:3376 isolated_anon:0 > > > active_file:3902 inactive_file:3639 isolated_file:0 > > >

Re: [patch 5/6] [RFD] timekeeping: Provide optional 128bit math

2016-12-09 Thread Chris Metcalf
On 12/9/2016 3:30 AM, Peter Zijlstra wrote: On Fri, Dec 09, 2016 at 07:38:47AM +0100, Peter Zijlstra wrote: On Fri, Dec 09, 2016 at 06:26:38AM +0100, Peter Zijlstra wrote: Just for giggles, on tilegx the branch is actually slower than doing the mult unconditionally. The problem is that the two

[PATCH] scripts/spelling.txt: add several more common spelling mistakes

2016-12-09 Thread Colin King
From: Colin Ian King Lately I've been cleaning up spelling mistakes in kernel error messages and here are some of the more common spelling mistakes that I've found which probably should be added to this list so we don't keep on seeing them appearing again. Signed-off-by: Colin Ian King --- scr

Re: Tearing down DMA transfer setup after DMA client has finished

2016-12-09 Thread Mason
On 09/12/2016 18:17, Vinod Koul wrote: > On Fri, Dec 09, 2016 at 11:25:57AM +0100, Sebastian Frias wrote: >> >> What concrete solution do you propose? > > I have already proposed two solutions. > > A) Request a channel only when you need it. Obviously we can't do virtual > channels with this (th

Re: [PATCH] x86/tsc: RFC: re-synchronize TSCs to boot cpu TSC

2016-12-09 Thread Thomas Gleixner
On Fri, 9 Dec 2016, Roland Scheidegger wrote: > Am 09.12.2016 um 10:59 schrieb Thomas Gleixner: > > On Fri, 9 Dec 2016, Roland Scheidegger wrote: > >> > >> I saw some system lockups though: > >> When doing a cold boot, this kernel never managed to boot up. The last > >> message seen is: > >> x86: B

Re: [patch 5/6] [RFD] timekeeping: Provide optional 128bit math

2016-12-09 Thread Chris Metcalf
On 12/9/2016 5:18 AM, Peter Zijlstra wrote: On Fri, Dec 09, 2016 at 07:38:47AM +0100, Peter Zijlstra wrote: Turns out using GCC-6.2.1 we have the same problem on i386, GCC doesn't recognise the 32x32 mults and generates crap. This used to work :/ I tried: gcc-4.4: good gcc-4.6, gcc-4.8, gcc-

Re: [PATCH v2 1/2] devicetree: i2c-hid: Add Wacom digitizer + regulator support

2016-12-09 Thread Rob Herring
On Fri, Dec 9, 2016 at 10:05 AM, Doug Anderson wrote: > Hi, > > On Thu, Dec 8, 2016 at 8:01 AM, Rob Herring wrote: >>> Just my $0.02. Feel free to ignore. >>> >>> One thought is that I would say that the need to power on the device >>> explicitly seems more like a board level difference and less

[PATCH perf/core] samples/bpf: Drop unnecessary build targets.

2016-12-09 Thread Joe Stringer
Commit f72179ef11db ("samples/bpf: Switch over to libbpf") added these two makefile changes that were unnecessary for switching samples to use libbpf. The extra make is already handled by the build dependency, and libbpf target doesn't build because it lacks main(). Remove these. Reported-by: Wang

Re: [PATCH 6/9] dt-bindings: Document rk3399 Gru/Kevin

2016-12-09 Thread Rob Herring
On Thu, Dec 01, 2016 at 06:27:30PM -0800, Brian Norris wrote: > Gru is a base dev board for a family of devices, including Kevin. Both > utilize Rockchip RK3399, and they share much of their design. > > Signed-off-by: Brian Norris > --- > Documentation/devicetree/bindings/arm/rockchip.txt | 20 +

Re: Tearing down DMA transfer setup after DMA client has finished

2016-12-09 Thread Vinod Koul
On Fri, Dec 09, 2016 at 05:28:01PM +, Måns Rullgård wrote: > Vinod Koul writes: > > > On Fri, Dec 09, 2016 at 11:25:57AM +0100, Sebastian Frias wrote: > >> > >> What concrete solution do you propose? > > > > I have already proposed two solutions. > > > > A) Request a channel only when you ne

Re: Tearing down DMA transfer setup after DMA client has finished

2016-12-09 Thread Vinod Koul
On Fri, Dec 09, 2016 at 06:34:15PM +0100, Mason wrote: > On 09/12/2016 18:17, Vinod Koul wrote: > > > On Fri, Dec 09, 2016 at 11:25:57AM +0100, Sebastian Frias wrote: > >> > >> What concrete solution do you propose? > > > > I have already proposed two solutions. > > > > A) Request a channel only

Re: [PATCHv3 perf/core 5/7] samples/bpf: Switch over to libbpf

2016-12-09 Thread Joe Stringer
On 8 December 2016 at 21:18, Wangnan (F) wrote: > > > On 2016/12/9 13:04, Wangnan (F) wrote: >> >> >> >> On 2016/12/9 10:46, Joe Stringer wrote: >> >> [SNIP] >> >>> diff --git a/tools/lib/bpf/Makefile b/tools/lib/bpf/Makefile >>> index 62d89d50fcbd..616bd55f3be8 100644 >>> --- a/tools/lib/bpf/Ma

Re: [PATCH v2 2/2] dt-bindings: Add DT bindings info for FlexRM ring manager

2016-12-09 Thread Rob Herring
On Fri, Dec 02, 2016 at 10:08:56AM +0530, Anup Patel wrote: > This patch adds device tree bindings document for the FlexRM > ring manager found on Broadcom iProc SoCs. > > Reviewed-by: Ray Jui > Reviewed-by: Scott Branden > Signed-off-by: Anup Patel > --- > .../bindings/mailbox/brcm,iproc-flex

[PATCH v6 0/3] Add platform clock for BayTrail platforms

2016-12-09 Thread Irina Tirdea
These patches specifically enable the audio MCLK required by Baytrail CR devices. It is the remaining part of a bigger set of patches (already merged in Mark's tree) that enable sound for Baytrail CR devices (especially Asus T100TAF) [1]. They include the clock driver and enabling the clock in the

Re: Still OOM problems with 4.9er kernels

2016-12-09 Thread Gerhard Wiesinger
On 09.12.2016 18:30, Michal Hocko wrote: On Fri 09-12-16 17:58:14, Gerhard Wiesinger wrote: On 09.12.2016 17:09, Michal Hocko wrote: [...] [97883.882611] Mem-Info: [97883.883747] active_anon:2915 inactive_anon:3376 isolated_anon:0 active_file:3902 inactive_file:3639 isolated_

[GIT PULL] ARM: SoC fixes

2016-12-09 Thread Olof Johansson
Hi Linus, This should be our last set of fixes for 4.9. Please merge! Thanks, -Olof The following changes since commit 909e481e2467f202b97d42beef246e8829416a85: arm64: dts: juno: fix cluster sleep state entry latency on all SoC versions (2016-12-02 17:28:17 +0100) are available in the git

[PATCH v6 2/3] arch/x86/platform/atom: Move pmc_atom to drivers/platform/x86

2016-12-09 Thread Irina Tirdea
The pmc_atom driver does not contain any architecture specific code. It only enables the SOC Power Management Controller Driver for BayTrail and CherryTrail platforms. Move the pmc_atom driver from arch/x86/platform/atom to drivers/platform/x86. Signed-off-by: Irina Tirdea --- arch/x86/Kconfig

[PATCH v6 1/3] clk: x86: Add Atom PMC platform clocks

2016-12-09 Thread Irina Tirdea
The BayTrail and CherryTrail platforms provide platform clocks through their Power Management Controller (PMC). The SoC supports up to 6 clocks (PMC_PLT_CLK[5:0]) with a frequency of either 19.2 MHz (PLL) or 25 MHz (XTAL) for BayTrail and a frequency of 19.2 MHz (XTAL) for CherryTrail. These clock

[PATCH v6 3/3] platform/x86: Enable Atom PMC platform clocks

2016-12-09 Thread Irina Tirdea
The BayTrail and CherryTrail platforms provide platform clocks through their Power Management Controller (PMC). The SoC supports up to 6 clocks (PMC_PLT_CLK[5:0]) with a frequency of either 19.2 MHz (PLL) or 25 MHz (XTAL) for BayTrail an a frequency of 19.2 MHz (XTAL) for CherryTrail. These clocks

Re: [regression ?] kbuild: fix building bzImage with CONFIG_TRIM_UNUSED_KSYMS enabled

2016-12-09 Thread Nicolas Pitre
On Fri, 9 Dec 2016, Sergey Senozhatsky wrote: > On (12/08/16 22:40), Nicolas Pitre wrote: > > > > after 53924022d8a ("kbuild: fix building bzImage > > > > withCONFIG_TRIM_UNUSED_KSYMS > > > > enabled") all of my build scripts build the kernel without the modules. > > > > > > [..] > > > > +# For

RE: [PATCH v5 2/2] clk: x86: Add Atom PMC platform clocks

2016-12-09 Thread Tirdea, Irina
On 2016-12-09 02:25, Stephen Boyd wrote: > On 12/07, Irina Tirdea wrote: >> The BayTrail and CherryTrail platforms provide platform clocks >> through their Power Management Controller (PMC). >> >> The SoC supports up to 6 clocks (PMC_PLT_CLK[5:0]) with a >> frequency of either 19.2 MHz (PLL) or 25

[PATCH] arm64: mm: Fix NOMAP page initialization

2016-12-09 Thread Robert Richter
On ThunderX systems with certain memory configurations we see the following BUG_ON(): kernel BUG at mm/page_alloc.c:1848! This happens for some configs with 64k page size enabled. The BUG_ON() checks if start and end page of a memmap range belongs to the same zone. The BUG_ON() check fails if a

[PATCH V3 1/2] ASoC: samsung: include gpio consumer.h

2016-12-09 Thread Fabian Frederick
Fix the following build errors on X86_32 !GPIOLIB sound/soc/samsung/tm2_wm5110.c:220:3: error: implicit declaration of function 'gpiod_set_value_cansleep' [-Werror=implicit-function-declaration] sound/soc/samsung/tm2_wm5110.c:438:24: error: implicit declaration of function 'devm_gpiod_get' [-Werro

Re: [PATCH 1/2] of: Add vendor prefix for Tianma Micro-electronics

2016-12-09 Thread Rob Herring
On Fri, Dec 02, 2016 at 09:52:07AM +0100, Gary Bisson wrote: > Tianma Micro-electronics Co., Ltd. (Tianma) specializes in providing > display solutions and efficient support services worldwide. > > More info: > http://en.tianma.com/about.shtml > > Signed-off-by: Gary Bisson > --- > Documentatio

[PATCH V3 2/2] ASoC: samsung: add GPIOLIB dependency

2016-12-09 Thread Fabian Frederick
Both SND_SOC_SMARTQ and SND_SOC_SAMSUNG_TM2_WM5110 use gpio/consumer.h This patch adds GPIOLIB || COMPILE_TEST to Kconfig entries to fix runtime dependency. See commit 638f958baeaf ("extcon: Allow compile test of GPIO consumers if !GPIOLIB") for similar problem and explanations. Reviewed-by: Krz

Re: [PATCH 2/2] drm/panel: simple: Add support for Tianma TM070JDHG30

2016-12-09 Thread Rob Herring
On Fri, Dec 02, 2016 at 09:52:08AM +0100, Gary Bisson wrote: > The Tianma TM070JDHG30 is a 7" LVDS display with a resolution of > 1280x800. > http://usa.tianma.com/products-technology/product/tm070jdhg30-00 > > You can also find this product along with a FT5x06 touch controller > from Boundary Dev

Re: Tearing down DMA transfer setup after DMA client has finished

2016-12-09 Thread Vinod Koul
On Fri, Dec 09, 2016 at 11:26:06PM +0530, Vinod Koul wrote: > On Fri, Dec 09, 2016 at 06:34:15PM +0100, Mason wrote: > > On 09/12/2016 18:17, Vinod Koul wrote: > > > > > On Fri, Dec 09, 2016 at 11:25:57AM +0100, Sebastian Frias wrote: > > >> > > >> What concrete solution do you propose? > > > > >

Re: [PATCH 06/20] hwmon/via-cputemp: Convert to hotplug state machine

2016-12-09 Thread Guenter Roeck
On Fri, Dec 09, 2016 at 12:53:30PM +0100, Thomas Gleixner wrote: > Guenter, > > On Thu, 17 Nov 2016, Sebastian Andrzej Siewior wrote: > > > Install the callbacks via the state machine and let the core invoke the > > callbacks on the already online CPUs. When the hotplug state is > > unregistered

Re: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-09 Thread Stephen Hemminger
On Fri, 9 Dec 2016 08:31:22 +0100 Greg KH wrote: > On Fri, Dec 09, 2016 at 12:05:53AM +, KY Srinivasan wrote: > > > > > > > -Original Message- > > > From: Greg KH [mailto:gre...@linuxfoundation.org] > > > Sent: Thursday, December 8, 2016 7:56 AM > > > To: KY Srinivasan > > > Cc:

Re: [PATCH 3/5] Move dma_ops from archdata into struct device

2016-12-09 Thread Christoph Hellwig
We'll need a bit of a wieder audience for this I think.. On Wed, Dec 07, 2016 at 05:11:28PM -0800, Bart Van Assche wrote: > Additionally, introduce set_dma_ops(). A later patch will introduce a > call to that function in the RDMA drivers that will be modified to use > dma_noop_ops. This looks goo

Re: [PATCH 4.4 00/28] 4.4.38-stable review

2016-12-09 Thread Shuah Khan
On 12/09/2016 09:17 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.38 release. > There are 28 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses sh

[PATCH v4 1/4] x86/asm/32: Make sync_core() handle missing CPUID on all 32-bit kernels

2016-12-09 Thread Andy Lutomirski
We support various non-Intel CPUs that don't have the CPUID instruction, so the M486 test was wrong. For now, fix it with a big hammer: handle missing CPUID on all 32-bit CPUs. Reported-by: One Thousand Gnomes Signed-off-by: Andy Lutomirski --- arch/x86/include/asm/processor.h | 2 +- 1 file c

Re: Tearing down DMA transfer setup after DMA client has finished

2016-12-09 Thread Mason
[ Dropping Mans to preserve his peace-of-mind ] On 09/12/2016 18:56, Vinod Koul wrote: > On Fri, Dec 09, 2016 at 06:34:15PM +0100, Mason wrote: >> On 09/12/2016 18:17, Vinod Koul wrote: >> >>> On Fri, Dec 09, 2016 at 11:25:57AM +0100, Sebastian Frias wrote: What concrete solution do you

Re: [PATCH 7/7] hwrng: core: Remove two unused include

2016-12-09 Thread Corentin Labbe
On Fri, Dec 09, 2016 at 03:21:39PM +0100, Corentin Labbe wrote: > linux/fs.h and linux/sched.h are useless for hw_random/core.c. > This patch remove them. > > Signed-off-by: Corentin Labbe > --- > drivers/char/hw_random/core.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/ch

[PATCH v4 2/4] Revert "x86/boot: Fail the boot if !M486 and CPUID is missing"

2016-12-09 Thread Andy Lutomirski
This reverts commit ed68d7e9b9cfb64f3045ffbcb108df03c09a0f98. The patch wasn't quite correct -- there are non-Intel (and hence non-486) CPUs that we support that don't have CPUID. Since we no longer require CPUID for sync_core(), just revert the patch. I think the relevant CPUs are Geode and Ela

[PATCH v4 4/4] x86/asm: Rewrite sync_core() to use IRET-to-self

2016-12-09 Thread Andy Lutomirski
Aside from being excessively slow, CPUID is problematic: Linux runs on a handful of CPUs that don't have CPUID. Use IRET-to-self instead. IRET-to-self works everywhere, so it makes testing easy. For reference, On my laptop, IRET-to-self is ~110ns, CPUID(eax=1, ecx=0) is ~83ns on native and very

[PATCH v4 3/4] x86/microcode/intel: Replace sync_core() with native_cpuid()

2016-12-09 Thread Andy Lutomirski
The Intel microcode driver is using sync_core() to mean "do CPUID with EAX=1". I want to rework sync_core(), but first the Intel microcode driver needs to stop depending on its current behavior. Reported-by: Henrique de Moraes Holschuh Acked-by: Borislav Petkov Signed-off-by: Andy Lutomirski -

[PATCH v4 0/4] CPUID-less CPU/sync_core fixes and improvements

2016-12-09 Thread Andy Lutomirski
*** PATCHES 1 and 2 MAY BE 4.9 MATERIAL *** Alan Cox pointed out that the 486 isn't the only supported CPU that doesn't have CPUID. Let's clean up the mess and make everything faster while we're at it. Patch 1 is intended to be an easy fix: it makes sync_core() work without CPUID on all 32-bit k

Re: [PATCH 4.8 00/45] 4.8.14-stable review

2016-12-09 Thread Shuah Khan
On 12/09/2016 09:20 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.8.14 release. > There are 45 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses sh

Re: [PATCH 6/9] dt-bindings: Document rk3399 Gru/Kevin

2016-12-09 Thread Heiko Stuebner
Am Freitag, 9. Dezember 2016, 11:54:02 CET schrieb Rob Herring: > On Thu, Dec 01, 2016 at 06:27:30PM -0800, Brian Norris wrote: > > Gru is a base dev board for a family of devices, including Kevin. Both > > utilize Rockchip RK3399, and they share much of their design. > > > > Signed-off-by: Brian

[PATCH 5/5] x86: Remove empty idle.h header

2016-12-09 Thread Borislav Petkov
From: Thomas Gleixner One include less is always a good thing(tm). Good riddance. Signed-off-by: Thomas Gleixner Signed-off-by: Borislav Petkov --- arch/x86/include/asm/apic.h | 1 - arch/x86/include/asm/idle.h | 4 arch/x86/kernel/apic/apic.c | 1 -

[PATCH 0/5] x86/CPU: Improve AMD Erratum 400 workaround

2016-12-09 Thread Borislav Petkov
From: Borislav Petkov Hi, here's the final version of the pile we've been working on for a while now, trying to remove that rdmsr on the idle path. @jolsa: just wait until it lands in tip and then you can run it on your box. Thanks. Borislav Petkov (2): x86/cpufeature: Provide helper to set

Re: [PATCH 06/20] hwmon/via-cputemp: Convert to hotplug state machine

2016-12-09 Thread Thomas Gleixner
On Fri, 9 Dec 2016, Guenter Roeck wrote: > I queued up the second patch as well. Hope it does not blow up on us. > Sorry, I got a bit nervous after the coretemp experience. Sorry for that, but we are watching out for blow ups and are ready to fix any fallout. Thanks tglx

[PATCH 1/5] x86/cpufeature: Provide helper to set bugs bits

2016-12-09 Thread Borislav Petkov
From: Borislav Petkov Will be used in a later patch to set bug bits for bugs which need late detection. Signed-off-by: Borislav Petkov --- arch/x86/include/asm/cpufeature.h | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/x86/include/asm/cpufeature.h b/arch/x86/include/asm/cpufeature.

[PATCH] md: Combine two kmalloc() calls into one in sb_equal()

2016-12-09 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 9 Dec 2016 19:09:13 +0100 The function "kmalloc" was called in one case by the function "sb_equal" without checking immediately if it failed. This issue was detected by using the Coccinelle software. Perform the desired memory allocation (and release at the end) b

[PATCH 2/5] x86/bugs: Separate AMD E400 erratum and C1E bug

2016-12-09 Thread Borislav Petkov
From: Thomas Gleixner We need two bits of information in order to properly apply the Erratum 400 workaround: 1. X86_BUG_AMD_E400 - to be able to select the proper idle routine 2. X86_BUG_AMD_APIC_C1E - to actually note that the platform we're currently running on is actually affected by E400

[PATCH 4/5] x86/amd: Simplify AMD E400 aware idle routine

2016-12-09 Thread Borislav Petkov
Reorganize the E400 detection now that we have everything in place: switch the CPUs to broadcast mode after the LAPIC has been initialized and remove the facilities that were used previously on the idle path. We can't use static_cpu_has_bug() on the idle path because alternatives have been applied

[PATCH 3/5] x86/amd: Check for the C1E bug post ACPI subsystem init

2016-12-09 Thread Borislav Petkov
From: Thomas Gleixner AMD CPUs affected by the E400 erratum suffer from the issue that the local APIC timer stops when the CPU goes into C1E. Unfortunately there is no way to detect the affected CPUs on early boot. It's only possible to determine the range of possibly affected CPUs from the famil

Re: [PATCH 1/2] mm, page_alloc: don't convert pfn to idx when merging

2016-12-09 Thread Vlastimil Babka
On 12/09/2016 06:26 PM, Mel Gorman wrote: > On Fri, Dec 09, 2016 at 10:37:53AM +0100, Vlastimil Babka wrote: >> In __free_one_page() we do the buddy merging arithmetics on "page/buddy >> index", >> which is just the lower MAX_ORDER bits of pfn. The operations we do that >> affect >> the higher bi

[PATCH] siphash: add cryptographically secure hashtable function

2016-12-09 Thread Jason A. Donenfeld
SipHash is a 64-bit keyed hash function that is actually a cryptographically secure PRF, like HMAC. Except SipHash is super fast, and is meant to be used as a hashtable keyed lookup function. SipHash isn't just some new trendy hash function. It's been around for a while, and there really isn't any

Re: [PATCH] md: Combine two kmalloc() calls into one in sb_equal()

2016-12-09 Thread Joe Perches
On Fri, 2016-12-09 at 19:30 +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Fri, 9 Dec 2016 19:09:13 +0100 > > The function "kmalloc" was called in one case by the function "sb_equal" > without checking immediately if it failed. > This issue was detected by using the Coccinelle sof

[PATCH 0/2] usb: gadget: s3c2410: add device tree support

2016-12-09 Thread Sergio Prado
This series adds support for configuring Samsung's s3c2410 and compatible USB device controller via devicetree. Tested on FriendlyARM mini2440, based on s3c2440 SoC. Sergio Prado (2): dt-bindings: usb: add DT binding for s3c2410 USB device controller usb: gadget: s3c2410: allow probing from d

[PATCH 2/2] usb: gadget: s3c2410: allow probing from device tree

2016-12-09 Thread Sergio Prado
Allows configuring Samsung's s3c2410 and compatible USB device controller using a devicetree. Signed-off-by: Sergio Prado --- drivers/usb/gadget/udc/s3c2410_udc.c | 142 +-- drivers/usb/gadget/udc/s3c2410_udc.h | 4 + 2 files changed, 123 insertions(+), 23 delet

[PATCH 1/2] dt-bindings: usb: add DT binding for s3c2410 USB device controller

2016-12-09 Thread Sergio Prado
Adds the device tree bindings description for Samsung S3C2410 and compatible USB device controller. Signed-off-by: Sergio Prado --- .../devicetree/bindings/usb/s3c2410-usb.txt| 28 ++ 1 file changed, 28 insertions(+) diff --git a/Documentation/devicetree/bindings/usb

Re: [PATCH] arm64: mm: Fix NOMAP page initialization

2016-12-09 Thread Russell King - ARM Linux
On Fri, Dec 09, 2016 at 07:10:41PM +0100, Robert Richter wrote: > On ThunderX systems with certain memory configurations we see the > following BUG_ON(): > > kernel BUG at mm/page_alloc.c:1848! > > This happens for some configs with 64k page size enabled. The BUG_ON() > checks if start and end p

Re: [PATCH] md: Combine two kmalloc() calls into one in sb_equal()

2016-12-09 Thread Bernd Schubert
On 09.12.2016 19:30, SF Markus Elfring wrote: From: Markus Elfring Date: Fri, 9 Dec 2016 19:09:13 +0100 The function "kmalloc" was called in one case by the function "sb_equal" without checking immediately if it failed. Err, your patch actually *replaces* the check. So where did you get the

Re: enabling COMPILE_TEST support for GCC plugins in v4.11

2016-12-09 Thread Kees Cook
On Fri, Dec 9, 2016 at 2:40 AM, Arnd Bergmann wrote: > On Thursday, December 8, 2016 11:00:42 AM CET Kees Cook wrote: > >> If you have a moment, applying 215e2aa6c024[1] and reverting >> a519167e753e for an allyesconfig/allmodconfig build should let you >> know if things are working correctly with

Re: kvm: deadlock between kvm_io_bus_register_dev/kvm_hv_set_msr_common

2016-12-09 Thread Paolo Bonzini
On 09/12/2016 17:09, Dmitry Vyukov wrote: > [resending an plain test] > > Hello, > > While running syzkaller fuzzer I am getting tasks deadlocked at the > following stacks. Seems that they are waiting for each other: This is pretty simple, kvm_hv_get/set_msr_common are taking kvm->lock inside

[PATCH 1/1 linux-next] fs: add BLOCKSIZE(inode)

2016-12-09 Thread Fabian Frederick
Add (1 << inode->i_blkbits) macro for more readability #define BLOCKSIZE(node) (1 << inode->i_blkbits) This patch also fixes multiple checkpatch warnings: WARNING: Prefer 'unsigned int' to bare use of 'unsigned' Signed-off-by: Fabian Frederick --- fs/block_dev.c | 2 +- fs/btrfs

Re: enabling COMPILE_TEST support for GCC plugins in v4.11

2016-12-09 Thread Kees Cook
On Fri, Dec 9, 2016 at 3:33 AM, Arnd Bergmann wrote: > On Thursday, December 8, 2016 11:00:42 AM CET Kees Cook wrote: >> Hi, >> >> I'd like to get the GCC plugins building under >> allyesconfig/allmodconfig for -next soon (with the intention of >> landing the change in v4.11). Specifically, I inte

Re: [PATCH 3/5] Move dma_ops from archdata into struct device

2016-12-09 Thread David Woodhouse
On Fri, 2016-12-09 at 19:22 +0100, Christoph Hellwig wrote: > We'll need a bit of a wieder audience for this I think.. > > On Wed, Dec 07, 2016 at 05:11:28PM -0800, Bart Van Assche wrote: > > Additionally, introduce set_dma_ops(). A later patch will introduce a > > call to that function in the RDM

Re: [PATCH] scripts/spelling.txt: add several more common spelling mistakes

2016-12-09 Thread Kees Cook
On Fri, Dec 9, 2016 at 9:33 AM, Colin King wrote: > From: Colin Ian King > > Lately I've been cleaning up spelling mistakes in kernel error > messages and here are some of the more common spelling mistakes > that I've found which probably should be added to this list so > we don't keep on seeing

Re: [PATCH] remoteproc: qcom_adsp_pil: select qcom_scm

2016-12-09 Thread Bjorn Andersson
On Fri 09 Dec 03:47 PST 2016, Arnd Bergmann wrote: > The adsp-pil driver relies on SCM and causes a build error without it: > > ERROR: "qcom_scm_pas_supported" [drivers/remoteproc/qcom_adsp_pil.ko] > undefined! > ERROR: "qcom_scm_is_available" [drivers/remoteproc/qcom_adsp_pil.ko] > undefined!

Re: [PATCH] scripts/spelling.txt: add several more common spelling mistakes

2016-12-09 Thread Joe Perches
On Fri, 2016-12-09 at 17:33 +, Colin King wrote: > From: Colin Ian King > > Lately I've been cleaning up spelling mistakes in kernel error > messages and here are some of the more common spelling mistakes > that I've found which probably should be added to this list so > we don't keep on seei

[PATCH V2 linux-next] fs: add BLOCKSIZE(inode)

2016-12-09 Thread Fabian Frederick
Add (1 << inode->i_blkbits) macro for more readability #define BLOCKSIZE(inode) (1 << inode->i_blkbits) This patch also fixes multiple checkpatch warnings: WARNING: Prefer 'unsigned int' to bare use of 'unsigned' Signed-off-by: Fabian Frederick --- V2: -Fix typo in macro fs/block_dev.c

Re: [PATCH 1/2] mm, page_alloc: don't convert pfn to idx when merging

2016-12-09 Thread Mel Gorman
On Fri, Dec 09, 2016 at 07:32:22PM +0100, Vlastimil Babka wrote: > > As a slight aside, I recently spotted that one of the largest overhead > > in the bulk free path was in the page_is_buddy() checks so pretty much > > anything that helps that is welcome. > > Interesting, the function shouldn't be

Re: enabling COMPILE_TEST support for GCC plugins in v4.11

2016-12-09 Thread Linus Torvalds
On Fri, Dec 9, 2016 at 11:12 AM, Kees Cook wrote: > > I'm starting to wonder if we need to expose the compiler version to > Kconfig so that "all*config" builds for earlier compiler will > automatically leave things like plugins off. But I have no idea what > the right approach for that might be.

Re: [PATCH 3/5] Move dma_ops from archdata into struct device

2016-12-09 Thread Bart Van Assche
On 12/09/2016 11:13 AM, David Woodhouse wrote: On Fri, 2016-12-09 at 19:22 +0100, Christoph Hellwig wrote: We'll need a bit of a wieder audience for this I think.. On Wed, Dec 07, 2016 at 05:11:28PM -0800, Bart Van Assche wrote: Additionally, introduce set_dma_ops(). A later patch will introdu

Re: md: Combine two kmalloc() calls into one in sb_equal()

2016-12-09 Thread SF Markus Elfring
> So where did you get the idea from that it is not checked immediately? Is another variable assignment performed so far before the return value is checked from a previous function call? Regards, Markus

Re: [PATCH] nommu: allow mmap when !CONFIG_MMU

2016-12-09 Thread Russell King - ARM Linux
On Thu, Dec 01, 2016 at 02:48:00PM +0100, Benjamin Gaignard wrote: > commit ab6494f0c96f ("nommu: Add noMMU support to the DMA API") have > add CONFIG_MMU compilation flag but that prohibit to use dma_mmap_wc() > when the platform doesn't have MMU. > > This patch call vm_iomap_memory() in noMMU ca

Re: [PATCH V2 linux-next] fs: add BLOCKSIZE(inode)

2016-12-09 Thread Andrew Morton
On Fri, 9 Dec 2016 20:38:33 +0100 Fabian Frederick wrote: > Add (1 << inode->i_blkbits) macro for more readability > #define BLOCKSIZE(inode) (1 << inode->i_blkbits) > > This patch also fixes multiple checkpatch warnings: > WARNING: Prefer 'unsigned int' to bare use of 'unsigned' > > ... > > -

Re: [RFC 10/10] kmod: add a sanity check on module loading

2016-12-09 Thread Martin Wilck
On Thu, 2016-12-08 at 11:49 -0800, Luis R. Rodriguez wrote: > > Although this does get us in the business of keeping alias maps in > kernel, the the work to support and maintain this is trivial. You've implemented a special treatment for request_module("fs-$X")in finished_kmod_load(), but there a

Re: md: Combine two kmalloc() calls into one in sb_equal()

2016-12-09 Thread SF Markus Elfring
> tmp1 = kmemdup(sb1, MD_SB_GENERIC_CONSTANT_WORDS * sizeof(__u32), > GFP_KERNEL); Is a function available in the Linux programming interface which would duplicate the beginning of two array elements in a single call directly? Regards, Markus

Re: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-09 Thread Stephen Hemminger
On Fri, 9 Dec 2016 20:09:49 + Haiyang Zhang wrote: > > -Original Message- > > From: Stephen Hemminger [mailto:step...@networkplumber.org] > > Sent: Friday, December 9, 2016 1:21 PM > > To: Greg KH > > Cc: KY Srinivasan ; o...@aepfle.de; Haiyang Zhang > > ; linux-kernel@vger.kernel.or

Re: [RFC PATCH 0/3] staging: remove fbdev drivers

2016-12-09 Thread Benjamin Herrenschmidt
On Fri, 2016-12-09 at 14:57 +0100, David Herrmann wrote: > Despite all of this I still see no reason why a driver could not > expose the static, real frambuffers via private ioctls. You can get > all your fancy acceleration that way. Then fix user-space to use this > API. If enough drivers end up w

[tip:x86/idle] x86/cpufeature: Provide helper to set bugs bits

2016-12-09 Thread tip-bot for Borislav Petkov
Commit-ID: a588b9836447ac683a8f63949ad55265813826f5 Gitweb: http://git.kernel.org/tip/a588b9836447ac683a8f63949ad55265813826f5 Author: Borislav Petkov AuthorDate: Fri, 9 Dec 2016 19:29:08 +0100 Committer: Thomas Gleixner CommitDate: Fri, 9 Dec 2016 21:23:20 +0100 x86/cpufeature: Provid

Re: [PATCH 1/3] clkdev: add devm_get_clk_from_child()

2016-12-09 Thread Russell King - ARM Linux
On Mon, Dec 05, 2016 at 05:23:20AM +, Kuninori Morimoto wrote: > > From: Kuninori Morimoto > > Some driver is using this type of DT bindings for clock (more detail, > see ${LINUX}/Documentation/devicetree/bindings/sound/simple-card.txt). > > sound_soc { > ... >

[tip:x86/idle] x86/bugs: Separate AMD E400 erratum and C1E bug

2016-12-09 Thread tip-bot for Thomas Gleixner
Commit-ID: 3344ed30791af66dbbad5f375008f3d1863b6c99 Gitweb: http://git.kernel.org/tip/3344ed30791af66dbbad5f375008f3d1863b6c99 Author: Thomas Gleixner AuthorDate: Fri, 9 Dec 2016 19:29:09 +0100 Committer: Thomas Gleixner CommitDate: Fri, 9 Dec 2016 21:23:20 +0100 x86/bugs: Separate AMD

[tip:x86/idle] x86/amd: Check for the C1E bug post ACPI subsystem init

2016-12-09 Thread tip-bot for Thomas Gleixner
Commit-ID: e7ff3a47630d9512d0bcbdfa73660021087ba445 Gitweb: http://git.kernel.org/tip/e7ff3a47630d9512d0bcbdfa73660021087ba445 Author: Thomas Gleixner AuthorDate: Fri, 9 Dec 2016 19:29:10 +0100 Committer: Thomas Gleixner CommitDate: Fri, 9 Dec 2016 21:23:21 +0100 x86/amd: Check for the

[tip:x86/idle] x86/amd: Simplify AMD E400 aware idle routine

2016-12-09 Thread tip-bot for Borislav Petkov
Commit-ID: 07c94a38125376d70d156bd8bff98ddfe4c8ea95 Gitweb: http://git.kernel.org/tip/07c94a38125376d70d156bd8bff98ddfe4c8ea95 Author: Borislav Petkov AuthorDate: Fri, 9 Dec 2016 19:29:11 +0100 Committer: Thomas Gleixner CommitDate: Fri, 9 Dec 2016 21:23:21 +0100 x86/amd: Simplify AMD

[tip:x86/idle] x86: Remove empty idle.h header

2016-12-09 Thread tip-bot for Thomas Gleixner
Commit-ID: 34bc3560c657d3d4fb17367ed9bfda803166dce0 Gitweb: http://git.kernel.org/tip/34bc3560c657d3d4fb17367ed9bfda803166dce0 Author: Thomas Gleixner AuthorDate: Fri, 9 Dec 2016 19:29:12 +0100 Committer: Thomas Gleixner CommitDate: Fri, 9 Dec 2016 21:23:22 +0100 x86: Remove empty idle

Re: enabling COMPILE_TEST support for GCC plugins in v4.11

2016-12-09 Thread Arnd Bergmann
On Friday, December 9, 2016 11:13:20 AM CET Kees Cook wrote: > On Fri, Dec 9, 2016 at 3:33 AM, Arnd Bergmann wrote: > > On Thursday, December 8, 2016 11:00:42 AM CET Kees Cook wrote: > >> If you have a moment, applying 215e2aa6c024[1] and reverting > >> a519167e753e for an allyesconfig/allmodconfi

Re: [PATCH] uio-hv-generic: store physical addresses instead of virtual

2016-12-09 Thread Arnd Bergmann
On Friday, December 9, 2016 9:28:44 AM CET Stephen Hemminger wrote: > On Fri, 9 Dec 2016 12:44:40 +0100 > Arnd Bergmann wrote: > > Fixes: 95096f2fbd10 ("uio-hv-generic: new userspace i/o driver for VMBus") > > Signed-off-by: Arnd Bergmann > > Thanks, the code was inherited from outside, and on

Re: [PATCH] remoteproc: qcom_adsp_pil: select qcom_scm

2016-12-09 Thread Arnd Bergmann
On Friday, December 9, 2016 11:23:10 AM CET Bjorn Andersson wrote: > On Fri 09 Dec 03:47 PST 2016, Arnd Bergmann wrote: > > > The adsp-pil driver relies on SCM and causes a build error without it: > > > > ERROR: "qcom_scm_pas_supported" [drivers/remoteproc/qcom_adsp_pil.ko] > > undefined! > > ER

[PATCH] scsi/qla2xxx: label endian-ness for many fields

2016-12-09 Thread Michael S. Tsirkin
This adds endian-ness labels for lots of qla structs. Doing this cuts down number of sparse warnings from ~1700 to ~1400. Will help find and resolve some of real issues down the road. Signed-off-by: Michael S. Tsirkin --- Compile-tested only. diff --git a/drivers/scsi/qla2xxx/qla_def.h b/drive

Re: [PATCH] linux/types.h: enable endian checks for all sparse builds

2016-12-09 Thread Michael S. Tsirkin
On Fri, Dec 09, 2016 at 03:18:02PM +, Bart Van Assche wrote: > On 12/08/16 22:40, Madhani, Himanshu wrote: > > We’ll take a look and send patches to resolve these warnings. > > Thanks! > > Bart. > Sounds good. I posted what I have so far so that you can start from that. -- MST

Re: [PATCH] scsi/qla2xxx: label endian-ness for many fields

2016-12-09 Thread Madhani, Himanshu
On 12/9/16, 12:45 PM, "Michael S. Tsirkin" wrote: >This adds endian-ness labels for lots of qla structs. >Doing this cuts down number of sparse warnings from ~1700 to ~1400. >Will help find and resolve some of real issues down the road. > >Signed-off-by: Michael S. Tsirkin > >--- > >Compile-tes

Re: [PATCH] x86/tsc: RFC: re-synchronize TSCs to boot cpu TSC

2016-12-09 Thread Roland Scheidegger
Am 09.12.2016 um 18:33 schrieb Thomas Gleixner: > On Fri, 9 Dec 2016, Roland Scheidegger wrote: >> Am 09.12.2016 um 10:59 schrieb Thomas Gleixner: >>> On Fri, 9 Dec 2016, Roland Scheidegger wrote: I saw some system lockups though: When doing a cold boot, this kernel never managed to

Re: md: Combine two kmalloc() calls into one in sb_equal()

2016-12-09 Thread Joe Perches
On Fri, 2016-12-09 at 21:05 +0100, SF Markus Elfring wrote: > > tmp1 = kmemdup(sb1, MD_SB_GENERIC_CONSTANT_WORDS * sizeof(__u32), > > GFP_KERNEL); > > Is a function available in the Linux programming interface which would > duplicate > the beginning of two array elements in a single call dir

Re: [PATCH SND/USB]: Add QuickCam Communicate Deluxe/S7500 to volume_control_quirks.

2016-12-09 Thread Con Kolivas
On Friday, 9 December 2016 11:22:15 AM AEDT Takashi Iwai wrote: > On Fri, 09 Dec 2016 05:15:57 +0100, > > Con Kolivas wrote: > > The Logitech QuickCam Communicate Deluxe/S7500 microphone fails with the > > following warning. > > > > [6.778995] usb 2-1.2.2.2: Warning! Unlikely big volume range

Re: [PATCH SND/USB]: Add QuickCam Communicate Deluxe/S7500 to volume_control_quirks.

2016-12-09 Thread Takashi Iwai
On Fri, 09 Dec 2016 21:35:17 +0100, Con Kolivas wrote: > > On Friday, 9 December 2016 11:22:15 AM AEDT Takashi Iwai wrote: > > On Fri, 09 Dec 2016 05:15:57 +0100, > > > > Con Kolivas wrote: > > > The Logitech QuickCam Communicate Deluxe/S7500 microphone fails with the > > > following warning. > >

Re: [RFC 10/10] kmod: add a sanity check on module loading

2016-12-09 Thread Linus Torvalds
On Fri, Dec 9, 2016 at 12:03 PM, Martin Wilck wrote: > On Thu, 2016-12-08 at 11:49 -0800, Luis R. Rodriguez wrote: >> >> Although this does get us in the business of keeping alias maps in >> kernel, the the work to support and maintain this is trivial. > > You've implemented a special treatment fo

[PATCH 0/4] x86: Trust firmware a bit more about presence of 8042

2016-12-09 Thread Dmitry Torokhov
Hi, Historically we did not trust PNP data regarding keyboard controllers on X86, but more and more boards get upset with us if they try to tell us that there is no keyboard controller and we still go and try to poke at where we think it might be. To work around this issue let's have a bit more fa

[PATCH 3/4] x86/init: remove i8042_detect() form platform ops

2016-12-09 Thread Dmitry Torokhov
Now that i8042 uses flag in legacy platform data, i8042_detect() is no longer used and can be removed. Signed-off-by: Dmitry Torokhov --- arch/x86/include/asm/x86_init.h | 2 -- arch/x86/kernel/x86_init.c | 2 -- arch/x86/platform/ce4100/ce4100.c | 6 -- arch/x86/p

<    1   2   3   4   5   6   7   >