Re: [PATCH 02/10] mm: remove write/force parameters from __get_user_pages_unlocked()

2016-10-12 Thread Paolo Bonzini
On 13/10/2016 02:20, Lorenzo Stoakes wrote: > This patch removes the write and force parameters from > __get_user_pages_unlocked() to make the use of FOLL_FORCE explicit in callers > as > use of this flag can result in surprising behaviour (and hence bugs) within > the > mm subsystem. > > Sign

Re: [PATCH v3 0/1] man/set_mempolicy.2,mbind.2: add MPOL_LOCAL NUMA memory policy documentation

2016-10-12 Thread Michael Kerrisk (man-pages)
On 10/12/2016 09:55 PM, Christoph Lameter wrote: > On Wed, 12 Oct 2016, Piotr Kwapulinski wrote: > >> That's right. This could be "local allocation" or any other memory policy. > > Correct. > Thanks, Piotr and Christoph. Cheers, Michael -- Michael Kerrisk Linux man-pages maintainer; http:/

[PATCH v2] mm: exclude isolated non-lru pages from NR_ISOLATED_ANON or NR_ISOLATED_FILE.

2016-10-12 Thread ming.ling
From: Ming Ling Non-lru pages don't belong to any lru, so counting them to NR_ISOLATED_ANON or NR_ISOLATED_FILE doesn't make any sense. It may misguide functions such as pgdat_reclaimable_pages and too_many_isolated. On mobile devices such as 512M ram android Phone, it may use a big zram swap. In

Re: [RFC PATCH 1/1] mm/vmalloc.c: correct logic errors when insert vmap_area

2016-10-12 Thread zijun_hu
Hi Nicholas, i find __insert_vmap_area() is introduced by you could you offer comments for this patch related to that funciton thanks On 10/12/2016 10:46 PM, Michal Hocko wrote: > [Let's CC Nick who has written this code] > > On Wed 12-10-16 22:30:13, zijun_hu wrote: >> From: zijun_hu >> >> th

Re: [PATCH v3 05/11] x86/tracing: fix compat syscall handling

2016-10-12 Thread Marcin Nowakowski
On 12.10.2016 15:53, Thomas Gleixner wrote: On Tue, 11 Oct 2016, Marcin Nowakowski wrote: -/* - * Because ia32 syscalls do not map to x86_64 syscall numbers - * this screws up the trace output when tracing a ia32 task. - * Instead of reporting bogus syscalls, just do not trace them. - * - * If t

Re: slab corruption with current -git

2016-10-12 Thread Markus Trippelsdorf
On 2016.10.12 at 23:18 -0700, Linus Torvalds wrote: > On Oct 12, 2016 23:07, "Markus Trippelsdorf" wrote: > > > > This is nf_register_net_hook at net/netfilter/core.c:106 > > The "*regs" access? Yeah. 105 entry->orig_ops = reg; 106 entry->ops = *reg; 107 entry->next

Re: [PATCH] cpufreq: conservative: Fix next frequency selection

2016-10-12 Thread Viresh Kumar
On 12-10-16, 21:47, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > Commit d352cf47d93e (cpufreq: conservative: Do not use transition > notifications) overlooked the case when the "frequency step" used > by the conservative governor is small relative to the distances > between the availabl

Re: linux.git: printk() problem

2016-10-12 Thread Michal Hocko
On Wed 12-10-16 09:08:58, Joe Perches wrote: > On Wed, 2016-10-12 at 16:35 +0200, Michal Hocko wrote: > > On Wed 12-10-16 22:30:03, Tetsuo Handa wrote: > > > Hello. > > > > > > I noticed that current linux.git generates hardly readable console output > > > due to KERN_CONT changes. Are you suggest

Re: [PATCH] mm, compaction: allow compaction for GFP_NOFS requests

2016-10-12 Thread Michal Hocko
On Thu 13-10-16 08:19:53, Vlastimil Babka wrote: > On 10/12/2016 01:47 PM, Michal Hocko wrote: [...] > Acked-by: Vlastimil Babka Thanks! > > Small nitpick below. > > > @@ -1696,14 +1703,16 @@ enum compact_result try_to_compact_pages(gfp_t > > gfp_mask, unsigned int order, > > unsi

Re: [PATCH] mm, compaction: allow compaction for GFP_NOFS requests

2016-10-12 Thread Vlastimil Babka
On 10/12/2016 01:47 PM, Michal Hocko wrote: From: Michal Hocko compaction has been disabled for GFP_NOFS and GFP_NOIO requests since the direct compaction was introduced by 56de7263fcf3 ("mm: compaction: direct compact when a high-order allocation fails"). The main reason is that the migration

Re: slab corruption with current -git

2016-10-12 Thread Markus Trippelsdorf
On 2016.10.13 at 08:02 +0200, Markus Trippelsdorf wrote: > On 2016.10.11 at 04:57 -0400, David Miller wrote: > > From: Linus Torvalds > > Date: Mon, 10 Oct 2016 22:47:50 -0700 > > > > > On Mon, Oct 10, 2016 at 10:39 PM, Linus Torvalds > > > wrote: > > >> > > >> I guess I will have to double-chec

[PATCH 04/10] mm: replace get_user_pages_locked() write/force parameters with gup_flags

2016-10-12 Thread Lorenzo Stoakes
This patch removes the write and force parameters from get_user_pages_locked() and replaces them with a gup_flags parameter to make the use of FOLL_FORCE explicit in callers as use of this flag can result in surprising behaviour (and hence bugs) within the mm subsystem. Signed-off-by: Lorenzo Stoa

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-12 Thread Andy Lutomirski
On Wed, Oct 12, 2016 at 7:22 AM, Johannes Berg wrote: > >> > Can you elaborate on how exactly it kills your system? >> >> the last time I saw it it was a NULL deref at >> ieee80211_aes_ccm_decrypt. > > Hm. I was expecting something within the crypto code would cause the > crash, this seems strange

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-12 Thread Johannes Berg
On Wed, 2016-10-12 at 22:39 -0700, Andy Lutomirski wrote: > In a pinch, I have these patches sitting around: > > https://git.kernel.org/cgit/linux/kernel/git/luto/linux.git/commit/?h=x86/vmap_stack&id=0a39cfa6fbb5d5635c85253cc7d6b44b54822afd > https://git.kernel.org/cgit/linux/kernel/git/luto/lin

Re: slab corruption with current -git

2016-10-12 Thread Markus Trippelsdorf
On 2016.10.11 at 04:57 -0400, David Miller wrote: > From: Linus Torvalds > Date: Mon, 10 Oct 2016 22:47:50 -0700 > > > On Mon, Oct 10, 2016 at 10:39 PM, Linus Torvalds > > wrote: > >> > >> I guess I will have to double-check that the slub corruption is gone > >> still with that fixed. > > > > S

Re: [PATCH 6/10] mmc: sdhci-xenon: Add Marvell Xenon SDHC core functionality

2016-10-12 Thread Ziji Hu
Hi Adrian, On 2016/10/12 21:07, Adrian Hunter wrote: > On 12/10/16 14:58, Ziji Hu wrote: >> Hi Adrian, >> >> Thank you very much for your review. >> I will firstly fix the typo. >> >> On 2016/10/11 20:37, Adrian Hunter wrote: + +static int xenon_start_signal_voltage_switch(str

Re: [GIT PULL REQUEST] watchdog - v4.9 merge window

2016-10-12 Thread Wim Van Sebroeck
Hi Linus, It's an old server with only 512Mb RAM. I'm in the progress off replacing it (in another datacenter). Meanwhile I'll restart git-daemon. Sorry, Wim. > On Wed, Oct 12, 2016 at 11:35 AM, Wim Van Sebroeck wrote: > > Hi Linus, > > > > Please pull from 'master' branch of > > git:/

Re: [PATCH v3 3/6] pwm: imx: support output polarity inversion

2016-10-12 Thread Lukasz Majewski
Hi Stefan, > On 2016-10-12 15:15, Lukasz Majewski wrote: > > Hi Stefan, > > > >> On 2016-10-07 08:11, Bhuvanchandra DV wrote: > >> > From: Lothar Wassmann > >> > > >> > The i.MX pwm unit on i.MX27 and newer SoCs provides a > >> > configurable output polarity. This patch adds support to utilize >

Re: [PATCH v4 0/3] Broadcom STB AVS CPUfreq driver

2016-10-12 Thread Viresh Kumar
On 12-10-16, 15:12, Markus Mayer wrote: > This series contains the CPUfreq driver for Broadcom SoCs that use "AVS > Firmware" for voltage and frequency scaling. All voltage and frequency > transitions are performed by the firmware and are therefore hidden from > Linux. > > The driver provides a st

Re: [PATCH 3/8] PM / OPP: Manage supply's voltage/current in a separate structure

2016-10-12 Thread Viresh Kumar
On 12-10-16, 16:15, Dave Gerlach wrote: > > /** > >+ * struct dev_pm_opp_supply - Power supply voltage/current values > >+ * @u_volt: Target voltage in microvolts corresponding to this OPP > >+ * @u_volt_min: Minimum voltage in microvolts corresponding to thisq OPP > >+ * @u_volt_max: Maxi

lz4 v lz4hc compression discrepancy

2016-10-12 Thread Srinath Krishna Ananthakrishnan
Hi, I'm trying to use the lz4 library in the kernel for compression and I have a case where for a candidate payload worth 16k bytes (attached), the data that is retrieved after compression + decompression is different from the original data when using lz4hc. On the other hand, using the lz4 varian

[PATCH 3/3] ARM: dts: mt2701: Add mtk-cirq node for mt2701

2016-10-12 Thread Youlin Pei
This commit add mtk-cirq node to mt2701 dtsi. Signed-off-by: Youlin Pei --- arch/arm/boot/dts/mt2701.dtsi | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/mt2701.dtsi b/arch/arm/boot/dts/mt2701.dtsi index 7eab6f4..a314df9 100644 --- a/arch/arm/b

[PATCH 0/3] Add Mediatek CIRQ interrupt controller

2016-10-12 Thread Youlin Pei
In Mediatek SOCs, the CIRQ is a low power interrupt controller designed to works outside MCUSYS which comprises with Cortex-Ax cores,CCI and GIC. The CIRQ controller is integrated in between MCUSYS and interrupt sources as the second level interrupt controller. The external interrupts which outsi

[PATCH 2/3] irqchip: mtk-cirq: Add mediatek mtk-cirq implement

2016-10-12 Thread Youlin Pei
This commit add the mtk-cirq implement for mt2701. Signed-off-by: Youlin Pei --- drivers/irqchip/Makefile |2 +- drivers/irqchip/irq-mtk-cirq.c | 257 2 files changed, 258 insertions(+), 1 deletion(-) create mode 100644 drivers/irqchip/irq-mtk

[PATCH 1/3] binding: irqchip: mtk-cirq: Add binding document

2016-10-12 Thread Youlin Pei
This commit adds the device tree binding document for the mediatek cirq. Signed-off-by: Youlin Pei --- base on v4.8-rc1 --- .../interrupt-controller/mediatek,cirq.txt | 30 1 file changed, 30 insertions(+) create mode 100644 Documentation/devicetree/bindings/int

Re: [PATCH v8 2/2] clocksource: add J-Core timer/clocksource driver

2016-10-12 Thread Rich Felker
On Wed, Oct 12, 2016 at 11:31:26PM +0200, Daniel Lezcano wrote: > > > I understand the goal is to have one single configuration and everything > > > DT based and it sounds great but what is missing here is just a subarch, > > > not an option to enable/disable the timer. > > > > > > Give a try with

Re: [PATCH] staging: greybus: timesync.c: Fixed WARNING for brace issue

2016-10-12 Thread Viresh Kumar
On Thu, Oct 13, 2016 at 1:20 AM, Chase Metzger wrote: > Removed a checkpatch warning for braces on single argument if and else > statement. > > Signed-off-by: Chase Metzger > --- > drivers/staging/greybus/timesync.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/d

[PATCH 10/10] mm: replace access_process_vm() write parameter with gup_flags

2016-10-12 Thread Lorenzo Stoakes
This patch removes the write parameter from access_process_vm() and replaces it with a gup_flags parameter as use of this function previously _implied_ FOLL_FORCE, whereas after this patch callers explicitly pass this flag. We make this explicit as use of FOLL_FORCE can result in surprising behavi

[PATCH 06/10] mm: replace get_user_pages() write/force parameters with gup_flags

2016-10-12 Thread Lorenzo Stoakes
This patch removes the write and force parameters from get_user_pages() and replaces them with a gup_flags parameter to make the use of FOLL_FORCE explicit in callers as use of this flag can result in surprising behaviour (and hence bugs) within the mm subsystem. Signed-off-by: Lorenzo Stoakes --

[PATCH 05/10] mm: replace get_vaddr_frames() write/force parameters with gup_flags

2016-10-12 Thread Lorenzo Stoakes
This patch removes the write and force parameters from get_vaddr_frames() and replaces them with a gup_flags parameter to make the use of FOLL_FORCE explicit in callers as use of this flag can result in surprising behaviour (and hence bugs) within the mm subsystem. Signed-off-by: Lorenzo Stoakes

[PATCH 08/10] mm: replace __access_remote_vm() write parameter with gup_flags

2016-10-12 Thread Lorenzo Stoakes
This patch removes the write parameter from __access_remote_vm() and replaces it with a gup_flags parameter as use of this function previously _implied_ FOLL_FORCE, whereas after this patch callers explicitly pass this flag. We make this explicit as use of FOLL_FORCE can result in surprising behav

[PATCH 09/10] mm: replace access_remote_vm() write parameter with gup_flags

2016-10-12 Thread Lorenzo Stoakes
This patch removes the write parameter from access_remote_vm() and replaces it with a gup_flags parameter as use of this function previously _implied_ FOLL_FORCE, whereas after this patch callers explicitly pass this flag. We make this explicit as use of FOLL_FORCE can result in surprising behavio

Intel PT: Address filtering not working. (cc me)

2016-10-12 Thread Muhammad Usman Nadeem
Usage: sudo perf record -e intel_pt//u --filter='filter 0x40 / 0x1000 @./a.out' ./a.out 123 a.out is my program (for loop, call using function pointer and use of longjump) and 123 is the argument. Output: [ perf record: Woken up 1 times to write data ] [ perf record: Captured and wrote 1.171

[PATCH v2] z3fold: fix the potential encode bug in encod_handle

2016-10-12 Thread zhongjiang
From: zhong jiang At present, zhdr->first_num plus bud can exceed the BUDDY_MASK in encode_handle, it will lead to the the caller handle_to_buddy return the error value. The patch fix the issue by changing the BUDDY_MASK to PAGE_MASK, it will be consistent with handle_to_z3fold_header. At the sa

Re: [PATCH] Staging:greybus:arche-apb-ctrl: fixed some coding style issue

2016-10-12 Thread Alexander Alemayhu
On Wed, Oct 12, 2016 at 04:45:54PM +0200, Greg KH wrote: > > You can subscribe to the driverdev mailing list to see what othes are > working on, and you can follow my staging-testing kernel branch in the > staging.git tree. But sometimes you just get unlucky, lots of people > are working on clean

Re: [PATCH] staging: android: ion: Make ion_free asynchronous more actively.

2016-10-12 Thread Laura Abbott
On 10/10/2016 07:31 PM, ming.ling wrote: From: Ming Ling So far some ion heaps such as carveout_heap, chunk_heap, system_heap have freed buffers asynchrounously. Freed buffers are placed on a free list and freed from a low priority background thread. If allocations from a particular heap fail,

Re: [PATCH] z3fold: fix the potential encode bug in encod_handle

2016-10-12 Thread zhong jiang
On 2016/10/13 11:33, zhongjiang wrote: > From: zhong jiang > > At present, zhdr->first_num plus bud can exceed the BUDDY_MASK > in encode_handle, it will lead to the the caller handle_to_buddy > return the error value. > > The patch fix the issue by changing the BUDDY_MASK to PAGE_MASK, > it will

[PATCH] staging: i4l: pcbit: replace some include asm/*.h with linux/*.h

2016-10-12 Thread Alexander Alemayhu
Fixes the following checkpatch output: ./pcbit/capi.c:39: WARNING: Use #include instead of ./pcbit/capi.c:40: WARNING: Use #include instead of ./pcbit/callbacks.c:25: WARNING: Use #include instead of ./pcbit/layer2.c:39: WARNING: Use #include instead of ./pcbit/drv.c:34: WARNING: Use #inc

[PATCH] acer-wmi: setup accelerometer when machine has appropriate notify event

2016-10-12 Thread Lee, Chun-Yi
The accelerometer event relies on on the ACERWMID_EVENT_GUID notify. So, this patch changes the codes to setup accelerometer input device when detected ACERWMID_EVENT_GUID. It avoids that the accel input device created on every acer machines. In addition, patch adds a clearly parsing logic of acce

Re: [PATCH 1/3] mfd: cros_ec: Add EC console read structures definitions

2016-10-12 Thread Nicolas Boichat
On Wed, Aug 31, 2016 at 4:45 AM, Lee Jones wrote: > On Tue, 23 Aug 2016, Nicolas Boichat wrote: > >> ec_params_console_read_v1 is used to capture EC logs from kernel, >> and ec_params_get_cmd_versions_v1 is used to probe whether EC >> supports that command. >> >> Signed-off-by: Nicolas Boichat >>

[PATCH] acer-wmi: only supports AMW0_GUID1 on acer family

2016-10-12 Thread Lee, Chun-Yi
The AMW0_GUID1 wmi is not only found on Acer family but also other machines like Lenovo, Fujitsu and Medion. In the past days, acer-wmi driver handled those non-Acer machines by quirks list. But actually acer-wmi driver was loaded on any machines that have AMW0_GUID1. This behavior is strange beca

[PATCH] z3fold: fix the potential encode bug in encod_handle

2016-10-12 Thread zhongjiang
From: zhong jiang At present, zhdr->first_num plus bud can exceed the BUDDY_MASK in encode_handle, it will lead to the the caller handle_to_buddy return the error value. The patch fix the issue by changing the BUDDY_MASK to PAGE_MASK, it will be consistent with handle_to_z3fold_header. At the sa

Re: [PATCH] ext4: do not advertise encryption support when disabled

2016-10-12 Thread Theodore Ts'o
On Mon, Oct 10, 2016 at 12:05:31PM -0700, Eric Biggers wrote: > The sysfs file /sys/fs/ext4/features/encryption was present on kernels > compiled with CONFIG_EXT4_FS_ENCRYPTION=n. This was misleading because > such kernels do not actually support ext4 encryption. Therefore, only > provide this fi

Re: [PATCH 1/1] jbd2: fix incorrect unlock on j_list_lock

2016-10-12 Thread Theodore Ts'o
On Wed, Oct 12, 2016 at 04:58:35PM -0600, Andreas Dilger wrote: > On Mar 17, 2015, at 8:08 PM, Taesoo Kim wrote: > > > > When 'jh->b_transaction == transaction' (asserted by below) > > > > J_ASSERT_JH(jh, (jh->b_transaction == transaction || ... > > > > 'journal->j_list_lock' will be in

Re: [PATCH] ext4: super.c: Update logging style using PR_CONT

2016-10-12 Thread Theodore Ts'o
Applied, thanks. - Ted

Re: [PATCH 3.4 096/125] xen/pciback: Save xen_pci_op commands before processing it

2016-10-12 Thread Zefan Li
On 2016/10/12 20:59, Konrad Rzeszutek Wilk wrote: > On Wed, Oct 12, 2016 at 08:33:32PM +0800, l...@kernel.org wrote: >> From: Konrad Rzeszutek Wilk >> >> 3.4.113-rc1 review patch. If anyone has any objections, please let me know. > > You also need: > > > commit d159457b84395927b5a52adb72f748dd

[PATCH] usb: ohci-da8xx: Remove code that references mach

2016-10-12 Thread David Lechner
Including mach/* is frowned upon in device drivers, so get rid of it. This replaces usb20_clk and code that pokes CFGCHIP2 with a proper phy driver. Signed-off-by: David Lechner Acked-by: Alan Stern --- Greg, This is a resend of the last patch from the "[PATCH v6 0/3] da8xx USB PHY platform d

Re: [PATCH 3.4 088/125] ser_gigaset: fix deallocation of platform device structure

2016-10-12 Thread Zefan Li
On 2016/10/12 20:52, Paul Bolle wrote: > Zefan Li, > > On Wed, 2016-10-12 at 20:33 +0800, l...@kernel.org wrote: >> When shutting down the device, the struct ser_cardstate must not be >> kfree()d immediately after the call to platform_device_unregister() >> since the embedded struct platform_devic

[PATCH 07/10] mm: replace get_user_pages_remote() write/force parameters with gup_flags

2016-10-12 Thread Lorenzo Stoakes
This patch removes the write and force parameters from get_user_pages_remote() and replaces them with a gup_flags parameter to make the use of FOLL_FORCE explicit in callers as use of this flag can result in surprising behaviour (and hence bugs) within the mm subsystem. Signed-off-by: Lorenzo Stoa

Re: [PATCH 3.4 065/125] drm/ttm: Fixed a read/write lock imbalance

2016-10-12 Thread Zefan Li
On 2016/10/12 21:04, Thomas Hellstrom wrote: > Li, > > IIRC This one goes hand in hand with a vmwgfx (the only user) patch. > Please don't apply until I've figured out whether that patch is also in 3.4. > I'll drop it for now. Thanks!

Re: [PATCH 3.4 081/125] mm, vmstat: allow WQ concurrency to discover memory reclaim doesn't make any progress

2016-10-12 Thread Zefan Li
On 2016/10/12 21:29, Michal Hocko wrote: > On Wed 12-10-16 20:33:17, l...@kernel.org wrote: >> From: Michal Hocko >> >> 3.4.113-rc1 review patch. If anyone has any objections, please let me know. > > Do not forget to take the follow up fix 564e81a57f97 ("mm, vmstat: fix > wrong WQ sleep when mem

[PATCH v2] kvm: x86: remove the misleading comment in vmx_handle_external_intr

2016-10-12 Thread Longpeng(Mike)
Since Paolo has removed irq-enable-operation in vmx_handle_external_intr (KVM: x86: use guest_exit_irqoff), the original comment about the IF bit in rflags is incorrect and stale now, so remove it. Signed-off-by: Longpeng(Mike) --- Hi Radim, Changes since v1: - remove this stale comment instead

linux-next: Tree for Oct 13

2016-10-12 Thread Stephen Rothwell
Hi all, Please do *not* add any v4.10 material to your linux-next included trees until v4.9-rc1 has been released i.e. the merge window closes. Changes since 20161012: The akpm-current tree still had its build failures for which I applied 2 patches. Non-merge commits (relative to Linus'

Re: [RFC v2 1/2] sdhci: Add device tree property broken-highspeed

2016-10-12 Thread Jaehoon Chung
On 10/13/2016 03:40 AM, Zach Brown wrote: > Certain board configurations can make highspeed malfuction due to timing > issues. In these cases a way is needed to force the controller and card > into standard speed even if they otherwise appear to be capable of > highspeed. I'm not sure "broken-hish

[PATCH] perf jit: Fix build issue on Ubuntu

2016-10-12 Thread Anton Blanchard
From: Anton Blanchard When building on Ubuntu 16.04, I get the following error: Makefile:49: *** the openjdk development package appears to me missing, install and try again. Stop. The problem is that update-java-alternatives has multiple spaces between fields, and cut treats each space as a

Re: Question: Re: [PATCH] drm/bridge: analogix: protect power when get_modes or detect

2016-10-12 Thread Mark yao
On 2016年10月12日 22:51, Sean Paul wrote: On Wed, Oct 12, 2016 at 6:22 AM, Mark yao wrote: I'm not familiar with the analogix driver, maybe use a power reference count would better then direct power on/off analogix_dp. Does anyone has the idea to protect detect and get_modes context? I'm not su

Re: [RFC v2 PATCH] mm/percpu.c: fix panic triggered by BUG_ON() falsely

2016-10-12 Thread zijun_hu
On 10/13/2016 05:41 AM, Andrew Morton wrote: > On Tue, 11 Oct 2016 22:00:28 +0800 zijun_hu wrote: > >> as shown by pcpu_build_alloc_info(), the number of units within a percpu >> group is educed by rounding up the number of CPUs within the group to >> @upa boundary, therefore, the number of CPUs

[PATCH v2] drm/bridge: analogix: protect power when get_modes or detect

2016-10-12 Thread Mark Yao
The drm callback ->detect and ->get_modes seems is not power safe, they may be called when device is power off, do register access on detect or get_modes will cause system die. Here is the path call ->detect before analogix_dp power on [] analogix_dp_detect+0x44/0xdc [] drm_helper_probe_single_co

Re: [PATCH v7 2/8] power: add power sequence library

2016-10-12 Thread Peter Chen
On Wed, Oct 12, 2016 at 12:30:29PM +0200, Heiko Stuebner wrote: > Hi, > > Am Dienstag, 20. September 2016, 11:36:41 CEST schrieb Peter Chen: > > We have an well-known problem that the device needs to do some power > > sequence before it can be recognized by related host, the typical > > example li

Re: [PATCH v6] net: ip, diag -- Add diag interface for raw sockets

2016-10-12 Thread David Miller
From: Cyrill Gorcunov Date: Wed, 12 Oct 2016 09:53:29 +0300 > I can't rename the field, neither a can use union. Remind me again what is wrong with using an anonymous union?

RE: [PATCH] xen-netback: fix type mismatch warning

2016-10-12 Thread Paul Durrant
> -Original Message- > From: Arnd Bergmann [mailto:a...@arndb.de] > Sent: 12 October 2016 10:54 > To: Wei Liu ; Paul Durrant > Cc: Arnd Bergmann ; David S. Miller > ; David Vrabel ; xen- > de...@lists.xenproject.org; net...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject: [PATCH

Re: [PATCH] sc16is7xx: Drop bogus use of IRQF_ONESHOT

2016-10-12 Thread Jakub Kicinski
On Mon, 3 Oct 2016 10:14:32 -0500, Julia Cartwright wrote: > The use of IRQF_ONESHOT when registering an interrupt handler with > request_irq() is non-sensical. > ... Disabling the irq when kthread is pending as mentioned here: http://www.spinics.net/lists/linux-rt-users/msg14673.html could be b

Re: [PATCH 3.4 000/125] 3.4.113-rc1 review

2016-10-12 Thread Zefan Li
On 2016/10/13 0:56, Guenter Roeck wrote: > On Wed, Oct 12, 2016 at 08:33:20PM +0800, l...@kernel.org wrote: >> From: Zefan Li >> >> This is the start of the stable review cycle for the 3.4.113 release. >> There are 125 patches in this series, all will be posted as a response >> to this one. If an

Re: [PATCH 3.4 007/125] recordmcount: Fix endianness handling bug for nop_mcount

2016-10-12 Thread Zefan Li
On 2016/10/12 21:10, Steven Rostedt wrote: > Hi Li, > > What is your plan for EOL of 3.4? I see on kernel.org that the EOL of > 3.4 was suppose to be September 2016 (which has passed). I just gave a > talk yesterday saying it is no longer supported. But I appear to be > mistaken. > I think I'll

Re: [GIT pull] locking fix for 4.9

2016-10-12 Thread Steven Rostedt
On Wed, 12 Oct 2016 10:22:59 -0700 Linus Torvalds wrote: > On Wed, Oct 12, 2016 at 1:28 AM, Steven Rostedt wrote: > > > > But I agree. We have lived a long time without the need for this > > warning. I'm not strongly advocating keeping the warning around and > > just disabling it totally. But it

Re: [PATCH/RFT 07/12] USB: ohci-da8xx: Request gpios and handle interrupt in the driver

2016-10-12 Thread David Lechner
On 10/12/2016 10:01 AM, Axel Haslam wrote: I agree that we should use a regulator for the vbus power. i will make that change. However, im not so sure about using the regulator for the overcurrent handling. There seems to be no other driver doing this, and as you mention, we would need to chang

Re: [PATCH] ext4: super.c: Update logging style using PR_CONT

2016-10-12 Thread Theodore Ts'o
On Wed, Oct 12, 2016 at 07:32:10AM -0700, Joe Perches wrote: > On Wed, 2016-10-12 at 12:41 +0200, Jan Kara wrote: > > On Tue 11-10-16 18:57:58, Joe Perches wrote: > > > Recent commit require line continuing printks to use PR_CONT. > > > > > > Update super.c to use PR_CONT and use vsprintf extensio

Re: [RFC PATCH] mm, compaction: allow compaction for GFP_NOFS requests

2016-10-12 Thread Dave Chinner
On Fri, Oct 07, 2016 at 03:18:14PM +0200, Michal Hocko wrote: > On Thu 06-10-16 13:11:42, Dave Chinner wrote: > > On Wed, Oct 05, 2016 at 01:38:45PM +0200, Michal Hocko wrote: > > > On Wed 05-10-16 07:32:02, Dave Chinner wrote: > > > > On Tue, Oct 04, 2016 at 10:12:15AM +0200, Michal Hocko wrote: >

[PATCH 00/10] mm: adjust get_user_pages* functions to explicitly pass FOLL_* flags

2016-10-12 Thread Lorenzo Stoakes
This patch series adjusts functions in the get_user_pages* family such that desired FOLL_* flags are passed as an argument rather than implied by flags. The purpose of this change is to make the use of FOLL_FORCE explicit so it is easier to grep for and clearer to callers that this flag is being u

Re: [PATCH v2] z3fold: add shrinker

2016-10-12 Thread Dave Chinner
On Wed, Oct 12, 2016 at 10:26:34AM +0200, Vitaly Wool wrote: > On Wed, 12 Oct 2016 09:52:06 +1100 > Dave Chinner wrote: > > > > > > > +static unsigned long z3fold_shrink_scan(struct shrinker *shrink, > > > + struct shrink_control *sc) > > > +{ > > > + struct z3fold_pool

[PATCH 03/10] mm: replace get_user_pages_unlocked() write/force parameters with gup_flags

2016-10-12 Thread Lorenzo Stoakes
This patch removes the write and force parameters from get_user_pages_unlocked() and replaces them with a gup_flags parameter to make the use of FOLL_FORCE explicit in callers as use of this flag can result in surprising behaviour (and hence bugs) within the mm subsystem. Signed-off-by: Lorenzo St

[PATCH 01/10] mm: remove write/force parameters from __get_user_pages_locked()

2016-10-12 Thread Lorenzo Stoakes
This patch removes the write and force parameters from __get_user_pages_locked() to make the use of FOLL_FORCE explicit in callers as use of this flag can result in surprising behaviour (and hence bugs) within the mm subsystem. Signed-off-by: Lorenzo Stoakes --- mm/gup.c | 47 +++

[PATCH 02/10] mm: remove write/force parameters from __get_user_pages_unlocked()

2016-10-12 Thread Lorenzo Stoakes
This patch removes the write and force parameters from __get_user_pages_unlocked() to make the use of FOLL_FORCE explicit in callers as use of this flag can result in surprising behaviour (and hence bugs) within the mm subsystem. Signed-off-by: Lorenzo Stoakes --- include/linux/mm.h | 3 +--

Re: [RFC v2 PATCH] mm/percpu.c: fix panic triggered by BUG_ON() falsely

2016-10-12 Thread zijun_hu
On 10/13/2016 05:41 AM, Andrew Morton wrote: > On Tue, 11 Oct 2016 22:00:28 +0800 zijun_hu wrote: > >> as shown by pcpu_build_alloc_info(), the number of units within a percpu >> group is educed by rounding up the number of CPUs within the group to >> @upa boundary, therefore, the number of CPUs

Re: [PATCH v3 3/4] mm: try to exhaust highatomic reserve before the OOM

2016-10-12 Thread Minchan Kim
Hi Michal, On Wed, Oct 12, 2016 at 10:34:50AM +0200, Michal Hocko wrote: > Looks much better. Thanks! I am wondering whether we want to have this > marked for stable. The patch is quite non-intrusive and fires only when > we are really OOM. It is definitely better to try harder than go and > disru

Re: [PATCH v3 3/6] pwm: imx: support output polarity inversion

2016-10-12 Thread Stefan Agner
On 2016-10-12 15:15, Lukasz Majewski wrote: > Hi Stefan, > >> On 2016-10-07 08:11, Bhuvanchandra DV wrote: >> > From: Lothar Wassmann >> > >> > The i.MX pwm unit on i.MX27 and newer SoCs provides a configurable >> > output polarity. This patch adds support to utilize this feature >> > where avail

[PATCH] f2fs: fix wrong sum_page pointer in f2fs_gc

2016-10-12 Thread Jaegeuk Kim
This patch fixes using a wrong pointer for sum_page in f2fs_gc. Signed-off-by: Jaegeuk Kim --- fs/f2fs/gc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c index e48142f..9c18917 100644 --- a/fs/f2fs/gc.c +++ b/fs/f2fs/gc.c @@ -854,16 +8

Re: [PATCH] exynos-drm: Fix display manager failing to start without IOMMU problem

2016-10-12 Thread Shuah Khan
Hi Inki, On 08/15/2016 10:40 PM, Inki Dae wrote: >> >> okay the very first commit that added IOMMU support >> introduced the code that rejects non-contig gem memory >> type without IOMMU. >> >> commit 0519f9a12d0113caab78980c48a7902d2bd40c2c >> Author: Inki Dae >> Date: Sat Oct 20 07:53:42 201

Re: [PATCHv2 1/4] arm64: dump: Make ptdump debugfs a separate option

2016-10-12 Thread Laura Abbott
On 10/12/2016 03:45 PM, Kees Cook wrote: On Wed, Oct 12, 2016 at 3:31 PM, Laura Abbott wrote: ptdump_register currently initializes a set of page table information and registers debugfs. There are uses for the ptdump option without wanting the debugfs options. Split this out to make it a separ

Re: [PATCH] exynos-drm: Fix display manager failing to start without IOMMU problem

2016-10-12 Thread Shuah Khan
On 10/12/2016 05:11 PM, Shuah Khan wrote: + Fixing Krzysztof Kozlowski address. > Hi Inki, > > On 08/15/2016 10:40 PM, Inki Dae wrote: > >>> >>> okay the very first commit that added IOMMU support >>> introduced the code that rejects non-contig gem memory >>> type without IOMMU. >>> >>> commit

Re: [PATCHv2 1/4] arm64: dump: Make ptdump debugfs a separate option

2016-10-12 Thread Kees Cook
On Wed, Oct 12, 2016 at 3:57 PM, Laura Abbott wrote: > On 10/12/2016 03:45 PM, Kees Cook wrote: >> >> On Wed, Oct 12, 2016 at 3:31 PM, Laura Abbott wrote: >>> >>> >>> ptdump_register currently initializes a set of page table information and >>> registers debugfs. There are uses for the ptdump opt

Re: [PATCH v3 3/6] pwm: imx: support output polarity inversion

2016-10-12 Thread Lukasz Majewski
Hi Stefan, > On 2016-10-07 08:11, Bhuvanchandra DV wrote: > > From: Lothar Wassmann > > > > The i.MX pwm unit on i.MX27 and newer SoCs provides a configurable > > output polarity. This patch adds support to utilize this feature > > where available. > > > > Signed-off-by: Lothar Waßmann > > Sig

[PATCH] perf: dynamically change verbosity level in perf top

2016-10-12 Thread Alexis Berlemont
Signed-off-by: Alexis Berlemont --- tools/perf/ui/browsers/hists.c | 5 - tools/perf/util/map.c | 17 - 2 files changed, 16 insertions(+), 6 deletions(-) diff --git a/tools/perf/ui/browsers/hists.c b/tools/perf/ui/browsers/hists.c index fb8e42c..6016909 100644 --- a

[PATCH v2] sparc64: Multi-page size support

2016-10-12 Thread Nitin Gupta
Add support for using multiple hugepage sizes simultaneously on mainline. Currently, support for 256M has been added which can be used along with 8M pages. Page tables are set like this (e.g. for 256M page): VA + (8M * x) -> PA + (8M * x) (sz bit = 256M) where x in [0, 31] and TSB is set simi

Re: [PATCH 1/1] jbd2: fix incorrect unlock on j_list_lock

2016-10-12 Thread Andreas Dilger
On Mar 17, 2015, at 8:08 PM, Taesoo Kim wrote: > > When 'jh->b_transaction == transaction' (asserted by below) > > J_ASSERT_JH(jh, (jh->b_transaction == transaction || ... > > 'journal->j_list_lock' will be incorrectly unlocked, since > spin_lock() is called only in the 'if' and 'else-i

Re: [PATCH v2 01/31] af9005: don't do DMA on stack

2016-10-12 Thread Milton Miller II
On Tue, 11 Oct 2016 07:09:16 -0300, Mauro wrote: > struct af9005_device_state { > > u8 sequence; > > int led_state; > > + unsigned char data[256]; > > + struct mutex data_mutex; > > }; This will not work on DMA incoherent architectures. When the data cache is invalida

[PATCH v6 03/17] dax: remove buffer_size_valid()

2016-10-12 Thread Ross Zwisler
Now that ext4 properly sets bh.b_size when we call get_block() for a hole, rely on that value and remove the buffer_size_valid() sanity check. Signed-off-by: Ross Zwisler Reviewed-by: Jan Kara Reviewed-by: Christoph Hellwig --- fs/dax.c | 22 +- 1 file changed, 1 insertion(

[PATCH v6 04/17] ext2: remove support for DAX PMD faults

2016-10-12 Thread Ross Zwisler
DAX PMD support was added via the following commit: commit e7b1ea2ad658 ("ext2: huge page fault support") I believe this path to be untested as ext2 doesn't reliably provide block allocations that are aligned to 2MiB. In my testing I've been unable to get ext2 to actually fault in a PMD. It alw

[PATCH v6 06/17] dax: remove the last BUG_ON() from fs/dax.c

2016-10-12 Thread Ross Zwisler
Don't take down the kernel if we get an invalid 'from' and 'length' argument pair. Just warn once and return an error. Signed-off-by: Ross Zwisler Reviewed-by: Christoph Hellwig Reviewed-by: Jan Kara --- fs/dax.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/fs/dax.c b

[PATCH v6 08/17] dax: coordinate locking for offsets in PMD range

2016-10-12 Thread Ross Zwisler
DAX radix tree locking currently locks entries based on the unique combination of the 'mapping' pointer and the pgoff_t 'index' for the entry. This works for PTEs, but as we move to PMDs we will need to have all the offsets within the range covered by the PMD to map to the same bit lock. To accompl

[PATCH v6 14/17] dax: move put_(un)locked_mapping_entry() in dax.c

2016-10-12 Thread Ross Zwisler
No functional change. The static functions put_locked_mapping_entry() and put_unlocked_mapping_entry() will soon be used in error cases in grab_mapping_entry(), so move their definitions above this function. Signed-off-by: Ross Zwisler --- fs/dax.c | 50 +

[PATCH v6 07/17] dax: consistent variable naming for DAX entries

2016-10-12 Thread Ross Zwisler
No functional change. Consistently use the variable name 'entry' instead of 'ret' for DAX radix tree entries. This was already happening in most of the code, so update get_unlocked_mapping_entry(), grab_mapping_entry() and dax_unlock_mapping_entry(). Signed-off-by: Ross Zwisler Reviewed-by: Chr

[PATCH v6 13/17] dax: move RADIX_DAX_* defines to dax.h

2016-10-12 Thread Ross Zwisler
The RADIX_DAX_* defines currently mostly live in fs/dax.c, with just RADIX_DAX_ENTRY_LOCK being in include/linux/dax.h so it can be used in mm/filemap.c. When we add PMD support, though, mm/filemap.c will also need access to the RADIX_DAX_PTE type so it can properly construct a 4k sized empty entr

[PATCH v6 10/17] dax: correct dax iomap code namespace

2016-10-12 Thread Ross Zwisler
The recently added DAX functions that use the new struct iomap data structure were named iomap_dax_rw(), iomap_dax_fault() and iomap_dax_actor(). These are actually defined in fs/dax.c, though, so should be part of the "dax" namespace and not the "iomap" namespace. Rename them to dax_iomap_rw(), d

[PATCH v6 05/17] dax: make 'wait_table' global variable static

2016-10-12 Thread Ross Zwisler
The global 'wait_table' variable is only used within fs/dax.c, and generates the following sparse warning: fs/dax.c:39:19: warning: symbol 'wait_table' was not declared. Should it be static? Make it static so it has scope local to fs/dax.c, and to make sparse happy. Signed-off-by: Ross Zwisler

[PATCH v6 00/17] re-enable DAX PMD support

2016-10-12 Thread Ross Zwisler
DAX PMDs have been disabled since Jan Kara introduced DAX radix tree based locking. This series allows DAX PMDs to participate in the DAX radix tree based locking scheme so that they can be re-enabled. For now I'm still using the same baseline for this series as I did with v5. I'll update the bas

[PATCH v6 01/17] ext4: allow DAX writeback for hole punch

2016-10-12 Thread Ross Zwisler
Currently when doing a DAX hole punch with ext4 we fail to do a writeback. This is because the logic around filemap_write_and_wait_range() in ext4_punch_hole() only looks for dirty page cache pages in the radix tree, not for dirty DAX exceptional entries. Signed-off-by: Ross Zwisler Reviewed-by:

[PATCH v6 11/17] dax: add dax_iomap_sector() helper function

2016-10-12 Thread Ross Zwisler
To be able to correctly calculate the sector from a file position and a struct iomap there is a complex little bit of logic that currently happens in both dax_iomap_actor() and dax_iomap_fault(). This will need to be repeated yet again in the DAX PMD fault handler when it is added, so break it out

[PATCH v6 09/17] dax: remove dax_pmd_fault()

2016-10-12 Thread Ross Zwisler
dax_pmd_fault() is the old struct buffer_head + get_block_t based 2 MiB DAX fault handler. This fault handler has been disabled for several kernel releases, and support for PMDs will be reintroduced using the struct iomap interface instead. Signed-off-by: Ross Zwisler Reviewed-by: Christoph Hell

[PATCH v6 02/17] ext4: tell DAX the size of allocation holes

2016-10-12 Thread Ross Zwisler
When DAX calls _ext4_get_block() and the file offset points to a hole we currently don't set bh->b_size. This is current worked around via buffer_size_valid() in fs/dax.c. _ext4_get_block() has the hole size information from ext4_map_blocks(), so populate bh->b_size so we can remove buffer_size_v

  1   2   3   4   5   6   7   >