Re: performance delta after VFS i_mutex=>i_rwsem conversion

2016-06-09 Thread Ingo Molnar
* Ingo Molnar wrote: > > * Waiman Long wrote: > > > I do have a patchset that allow us to more accurately determine the state of > > the lock owner. > > > > locking/rwsem: Add reader-owned state to the owner field > >

[PATCH] pci: make bus_attr_resource_alignemnt static

2016-06-09 Thread Ben Dooks
The symbol bus_attr_resource_alignment is not exported or declared elsewhere, so make it static to fix the following warning: drivers/pci/pci.c:4900:1: warning: symbol 'bus_attr_resource_alignment' was not declared. Should it be static? Signed-off-by: Ben Dooks ---

Re: linux-next: build warnings after merge of the kbuild tree

2016-06-09 Thread Michael Ellerman
On Thu, 2016-06-09 at 14:05 +1000, Stephen Rothwell wrote: > Hi Kees, > > On Wed, 8 Jun 2016 19:56:38 -0700 Kees Cook wrote: > > > > Congratulations on having the gcc plugin development headers > > successfully installed! ;) > > Thanks :-) And on ppc64le too! (I think)

Re: [RESEND PATCH v5 1/5] mfd: RK808: Add RK818 support

2016-06-09 Thread Lee Jones
On Thu, 09 Jun 2016, Wadim Egorov wrote: > On 08.06.2016 16:17, Lee Jones wrote: > > On Thu, 02 Jun 2016, Wadim Egorov wrote: > > > >> The RK818 chip is a power management IC for multimedia and handheld > > "Power Management IC (PMIC)" > > > >> devices. It contains the following components: > >> >

Re: [PATCH 3/3 V2] pvclock: Get rid of __pvclock_read_cycles in function pvclock_read_flags

2016-06-09 Thread Borislav Petkov
On Thu, Jun 09, 2016 at 01:21:18PM +0200, Paolo Bonzini wrote: > This is basically implementing a seqcount. It needs two barriers and, Why does it need the two barriers? More details please. > technically, they should be virt_rmb() -- it really doesn't matter of > course because reads are never

[PATCH 03/10] proc, oom_adj: extract oom_score_adj setting into a helper

2016-06-09 Thread Michal Hocko
From: Michal Hocko Currently we have two proc interfaces to set oom_score_adj. The legacy /proc//oom_adj and /proc//oom_score_adj which both have their specific handlers. Big part of the logic is duplicated so extract the common code into __set_oom_adj helper. Legacy knob still

Re: [PATCH v3] powerpc: spinlock: Fix spin_unlock_wait()

2016-06-09 Thread Michael Ellerman
On Wed, 2016-06-08 at 15:59 +0200, Peter Zijlstra wrote: > On Wed, Jun 08, 2016 at 11:49:20PM +1000, Michael Ellerman wrote: > > > > Ok; what tree does this go in? I have this dependent series which I'd > > > like to get sorted and merged somewhere. > > > > Ah sorry, I didn't realise. I was going

Re: [PATCH][RT] netpoll: Always take poll_lock when doing polling

2016-06-09 Thread Sebastian Andrzej Siewior
* Alison Chaiken | 2016-06-07 17:19:43 [-0700]: >Sorry to be obscure; I had applied that patch to v4.1.6-rt5. Using the latest is often not a bad choice compared to the random tree you have here. >> What I remember from testing the two patches on am335x was that before a >> ping flood on gbit

Re: [PATCH] arm64: Implement arch_setup_pdev_archdata hook

2016-06-09 Thread Arnd Bergmann
On Tuesday, June 7, 2016 4:29:21 PM CEST Baolin Wang wrote: > Now on ARM64 platform, it will set 'dummy_dma_ops' for device dma_ops if > it did not call 'arch_setup_dma_ops' at device creation time by issuing > platform_device_alloc() function, that will cause failure when setting > the dma mask

Re: [PATCH v2 2/2] PCI: Rockchip: Add Rockchip PCIe controller support

2016-06-09 Thread Arnd Bergmann
On Wednesday, June 8, 2016 10:21:27 AM CEST Marc Zyngier wrote: > > + rockchip_pcie_enable_interrupts(port); > > + if (!IS_ENABLED(CONFIG_PCI_MSI)) { > > + err = rockchip_pcie_init_irq_domain(port); > > + if (err < 0) > > + goto err_vpcie; > >

Re: [PATCH] pvclock: introduce seqcount-like API

2016-06-09 Thread Paolo Bonzini
On 09/06/2016 14:43, Roman Kagan wrote: > On Thu, Jun 09, 2016 at 01:23:23PM +0200, Paolo Bonzini wrote: >> The version field in struct pvclock_vcpu_time_info basically implements >> a seqcount. Wrap it with the usual read_begin and read_retry functions, >> and use these APIs instead of

Re: [PATCH net-next 5/8] net: dsa: mv88e6xxx: add switch register helpers

2016-06-09 Thread Vivien Didelot
Andrew Lunn writes: > On Wed, Jun 08, 2016 at 08:44:53PM -0400, Vivien Didelot wrote: >> Extract the allocation and registration code related to the dsa_switch >> structure in a mv88e6xxx_register_switch helper function. >> >> For symmetry in the code, add a

Re: [PATCH v9 08/22] IB/hns: Add icm support

2016-06-09 Thread Leon Romanovsky
On Wed, Jun 01, 2016 at 11:37:50PM +0800, Lijun Ou wrote: > This patch mainly added icm support for RoCE. It initializes icm > which managers the relative memory blocks for RoCE. The data > structures of RoCE will be located in it. For example, CQ table, > QP table and MTPT table so on. I wonder

Re: [PATCH] Add .set_antenna callback in ath6kl driver to fix wireless core warns

2016-06-09 Thread Valo, Kalle
Prasun Maiti writes: > Since add more warnings for inconsistent ops in cfg80211, the wireless > core warns if a driver implements a cfg80211 callback but doesn't > implements the inverse operation. The ath6kl driver implements a cfg80211 > .get_antenna operation handler

Re: [PATCH v2 08/15] clk: sunxi-ng: Add divider

2016-06-09 Thread Jean-Francois Moine
On Tue, 7 Jun 2016 22:41:47 +0200 Maxime Ripard wrote: > Add support for the various dividers (linear, table or pow-of-two based) > found in the CCU. > > Signed-off-by: Maxime Ripard > --- > drivers/clk/sunxi-ng/Makefile |

Re: [PATCH v2 11/15] clk: sunxi-ng: Add N-M-factor clock support

2016-06-09 Thread Jean-Francois Moine
On Tue, 7 Jun 2016 22:41:50 +0200 Maxime Ripard wrote: > Introduce support for clocks that multiply and divide using linear factors. > > Signed-off-by: Maxime Ripard > > --- > Changes from v1: > - Fixed the maximums for

Re: [PATCH 06/21] fs: udf: Replace CURRENT_TIME with current_fs_time()

2016-06-09 Thread Jan Kara
On Wed 08-06-16 22:04:50, Deepa Dinamani wrote: > CURRENT_TIME is not y2038 safe. > > CURRENT_TIME macro is also not appropriate for filesystems > as it doesn't use the right granularity for filesystem > timestamps. > > Logical Volume Integrity format is described to have the > same timestamp

[PATCH v5 0/5] sched/fair: Fix attach and detach sched avgs for task group change and sched class change

2016-06-09 Thread Yuyang Du
Hi Peter, The attach/detach twice problem is worse than Vincent reported. The attach twice issue can happen not only as Vincent raised when task moves between groups, but also when switching to fair class. In addition, for newly forked task, the detach also has a problem. This patchset attempts

[PATCH v5 1/5] sched/fair: Clean up attach_entity_load_avg()

2016-06-09 Thread Yuyang Du
attach_entity_load_avg() is called (indirectly) from: - switched_to_fair(): switch between classes to fair - task_move_group_fair(): move between task groups - enqueue_entity_load_avg(): enqueue entity Only in switched_to_fair() is it possible that the task's last_update_time is not 0 and

Re: [PATCH v3 1/1] net: ethernet: Add TSE PCS support to dwmac-socfpga

2016-06-09 Thread Tien Hock Loh
Hi Peppe, On Tue, 2016-06-07 at 10:30 +0200, Giuseppe CAVALLARO wrote: > Hello > > On 6/7/2016 5:01 AM, th...@altera.com wrote: > > From: Tien Hock Loh > > > > This adds support for TSE PCS that uses SGMII adapter when the phy-mode of > > the dwmac is set to sgmii > > > >

Re: Using DT overlays for adding virtual hardware

2016-06-09 Thread Arnd Bergmann
On Wednesday, June 8, 2016 6:39:08 PM CEST Jan Kiszka wrote: > >> > > > > I just don’t see how an ACPI based hypervisor can ever be certified for > > safety critical applications. It might be possible but it should be > > an enormous undertaking; perhaps a subset without AML, but then again > >

[RFC PATCH v2 2/2] power/idle: enhance the precision of sleep_length

2016-06-09 Thread zhaoyang . huang
From: Zhaoyang Huang There should be a gap between tick_nohz_idle_enter and tick_nohz_get_sleep_length when idle, which will cause the sleep_length is not very precised. Change it in this patch. Signed-off-by: Zhaoyang Huang ---

[RFC PATCH v2 1/2] power/cpuidle: enhance the precision of state select

2016-06-09 Thread zhaoyang . huang
From: Zhaoyang Huang In previous version, cpu_pm_enter is invoked after the governor select the state, which cause the executing time of cpu_pm_enter is included in the idle time. Moving it before the state selection. Signed-off-by: Zhaoyang Huang

Re: [PATCH 0/9] clocksource/drivers/clksrc-of: Improve error handling

2016-06-09 Thread Geert Uytterhoeven
Hi Daniel, On Wed, Jun 8, 2016 at 4:10 PM, Daniel Lezcano wrote: > On 06/07/2016 11:54 AM, Geert Uytterhoeven wrote: > > [ ... ] > >> in clockevents/clockevents/next breaks the boot on e.g. r8a7791/koelsch >> (arm32) and r8a7795/salvator-x (arm64). >> >> Using

Re: [PATCH v3] power_supply: power_supply_read_temp only if use_cnt > 0

2016-06-09 Thread Krzysztof Kozlowski
On 06/08/2016 05:26 PM, Rhyland Klein wrote: > On 6/8/2016 2:35 AM, Krzysztof Kozlowski wrote: >> On 06/07/2016 10:26 PM, Rhyland Klein wrote: >>> Change power_supply_read_temp() to use power_supply_get_property() >>> so that it will check the use_cnt and ensure it is > 0. The use_cnt >>> will be

Re: [PATCH] x86:pvclock: add missing barriers

2016-06-09 Thread Roman Kagan
On Thu, Jun 09, 2016 at 12:01:13AM +0300, Roman Kagan wrote: > On Wed, Jun 08, 2016 at 09:45:09PM +0200, Borislav Petkov wrote: > > On Wed, Jun 08, 2016 at 09:11:39PM +0300, Roman Kagan wrote: > > > --- a/arch/x86/include/asm/pvclock.h > > > +++ b/arch/x86/include/asm/pvclock.h > > > @@ -80,18

[PATCH] regulator: Remove regulator_can_change_voltage()

2016-06-09 Thread Mark Brown
There is little obvious use case for a regualtor driver to know if it is possible to vary voltages at all by itself. If a consumer needs to limit what voltages it tries to set based on the system configuration then it will need to enumerate the possible voltages, and without that even if it is

Re: [alsa-devel] [PATCH v2 2/9] ASoC: mediatek: implement mediatek common structure

2016-06-09 Thread Mark Brown
On Wed, Jun 08, 2016 at 10:06:12AM +0800, Garlic Tseng wrote: > On Tue, 2016-06-07 at 16:54 +0100, Mark Brown wrote: > > Do you need to export the individual ops rather than just the ops > > structure? > Yes, in 2701 driver we modify some ops. OK. signature.asc Description: PGP signature

[PATCH] pvclock: introduce seqcount-like API

2016-06-09 Thread Paolo Bonzini
The version field in struct pvclock_vcpu_time_info basically implements a seqcount. Wrap it with the usual read_begin and read_retry functions, and use these APIs instead of peppering the code with smp_rmb()s. While at it, change it to the more pedantically correct virt_rmb(). With this change,

[PATCH 06/10] mm, oom: kill all tasks sharing the mm

2016-06-09 Thread Michal Hocko
From: Michal Hocko Currently oom_kill_process skips both the oom reaper and SIG_KILL if a process sharing the same mm is unkillable via OOM_ADJUST_MIN. After "mm, oom_adj: make sure processes sharing mm have same view of oom_score_adj" all such processes are sharing the same

[PATCH 01/10] proc, oom: drop bogus task_lock and mm check

2016-06-09 Thread Michal Hocko
From: Michal Hocko both oom_adj_write and oom_score_adj_write are using task_lock, check for task->mm and fail if it is NULL. This is not needed because the oom_score_adj is per signal struct so we do not need mm at all. The code has been introduced by 3d5992d2ac7d ("oom: add

[PATCH 05/10] mm, oom: skip vforked tasks from being selected

2016-06-09 Thread Michal Hocko
From: Michal Hocko vforked tasks are not really sitting on any memory. They are sharing the mm with parent until they exec into a new code. Until then it is just pinning the address space. OOM killer will kill the vforked task along with its parent but we still can end up

[PATCH 09/10] mm, oom_reaper: do not attempt to reap a task more than twice

2016-06-09 Thread Michal Hocko
From: Michal Hocko oom_reaper relies on the mmap_sem for read to do its job. Many places which might block readers have been converted to use down_write_killable and that has reduced chances of the contention a lot. Some paths where the mmap_sem is held for write can take other

[PATCH 10/10] mm, oom: hide mm which is shared with kthread or global init

2016-06-09 Thread Michal Hocko
From: Michal Hocko The only case where the oom_reaper is not triggered for the oom victim is when it shares the memory with a kernel thread (aka use_mm) or with the global init. After "mm, oom: skip vforked tasks from being selected" the victim cannot be a vforked task of the

Re: [PATCH] nfsd: Always lock state exclusively.

2016-06-09 Thread Jeff Layton
On Wed, 2016-06-08 at 22:55 -0400, Oleg Drokin wrote: > It used to be the case that state had an rwlock that was locked for write > by downgrades, but for read for upgrades (opens). Well, the problem is > if there are two competing opens for the same state, they step on > each other toes

RE: [PATCH v9 00/14] USB OTG/dual-role framework

2016-06-09 Thread Yoshihiro Shimoda
Hi Roger, > From: Roger Quadros > Sent: Thursday, June 09, 2016 4:31 PM > > Peter & Yoshihiro, > > On 09/06/16 08:27, Peter Chen wrote: > > On Wed, Jun 8, 2016 at 5:03 PM, Roger Quadros wrote: > >> Hi, > >> > >> This series centralizes OTG/Dual-role functionality in the kernel.

Re: [RFC v4 01/14] regulator: of: Add helper for getting all supplies

2016-06-09 Thread Mark Brown
On Thu, Jun 09, 2016 at 11:44:18AM +0200, Krzysztof Kozlowski wrote: > Few drivers have a need of getting regulator supplies without knowing > their names: > 1. The Simple Framebuffer driver works on setup provided by bootloader >(outside of scope of kernel); > 2. Generic power sequence driver

Re: Divide-by-zero in post_init_entity_util_avg

2016-06-09 Thread Peter Zijlstra
On Thu, Jun 09, 2016 at 10:01:42AM +0100, Chris Wilson wrote: > > [15774.966082] divide error: [#1] SMP > [15774.966137] Modules linked in: i915 intel_gtt > [15774.966208] CPU: 1 PID: 15319 Comm: gemscript Not tainted 4.7.0-rc1+ #330 > [15774.966252] Hardware name:

Re: [tip:x86/urgent] x86/quirks: Add early quirk to reset Apple AirPort card

2016-06-09 Thread Lukas Wunner
On Thu, Jun 09, 2016 at 08:48:03AM +0200, Ingo Molnar wrote: > * Lukas Wunner wrote: > > On Wed, Jun 08, 2016 at 11:56:14AM -0700, Yinghai Lu wrote: > > > How do you know that sec is valid ? > > > How about on the system that have one bridge that still have sec num > > > register

[PATCH] net: diag: add missing declarations

2016-06-09 Thread Ben Dooks
The functions inet_diag_msg_common_fill and inet_diag_msg_attrs_fill seem to have been missed from the include/linux/inet_diag.h header file. Add them to fix the following warnings: net/ipv4/inet_diag.c:69:6: warning: symbol 'inet_diag_msg_common_fill' was not declared. Should it be static?

Re: [PATCH 3/3 V2] pvclock: Get rid of __pvclock_read_cycles in function pvclock_read_flags

2016-06-09 Thread Paolo Bonzini
On 07/06/2016 15:16, Minfei Huang wrote: > ping. Any comment is appreciate. The patch looks good, I'll follow up with another to introduce the seqcount-like read_begin/read_retry API. Thanks, Paolo > On 05/28/16 at 08:27P, Minfei Huang wrote: >> There is a generic function

Re: [PATCH v2 4/4] dell-wmi: Rework code for generating sparse keymap and processing WMI events

2016-06-09 Thread Michał Kępień
> > - case 0x10: > > - /* Keys pressed */ > > + case 0x0010: > > + /* Sequence of keys pressed */ > > for (i = 2; i < len; ++i) > > - dell_wmi_process_key(buffer_entry[i]); > > +

[PATCH] x86/apic: fix misspelled APIC

2016-06-09 Thread hw . claudio
From: Claudio Fontana Signed-off-by: Claudio Fontana --- arch/x86/kernel/apic/apic.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/x86/kernel/apic/apic.c b/arch/x86/kernel/apic/apic.c index

[PATCH] lib: lcm.c: make __attribute_const__ consistent

2016-06-09 Thread Ben Dooks
The lcm and lcm_not_zero() are producing sparse warnings due to the use of __attribute_const__ in the header but not in the lcm.c file. Add the __attribute_const__ to lcm.c and make the header consistent with using the attribute before the function to fix the following sparse warnings:

[PATCH] kernel/cpu: Distinct name for cpu_hotplug.dep_map

2016-06-09 Thread Joonas Lahtinen
Use distinctive name for cpu_hotplug.dep_map to avoid the actual cpu_hotplug.lock appearing as cpu_hotplug.lock#2 in lockdep splats. Cc: Ingo Molnar Cc: Peter Zijlstra Cc: Gautham R. Shenoy Cc: intel-...@lists.freedesktop.org Cc:

[PATCH 0/2] dmaengine: bcm2835: Cyclic DMA fixes

2016-06-09 Thread Matthias Reichl
In downstream Raspberry Pi kernel we noticed that audio didn't work as expected, we got stuttering and overruns/underruns. Here's the link to the original discussion on GitHub: https://github.com/raspberrypi/linux/issues/1517 This issue is caused by a small bug in the period-splitting-code and

Re: [PATCH 5/6] drivers/media/media-device: add "release" callback

2016-06-09 Thread Mauro Carvalho Chehab
Em Mon, 21 Mar 2016 14:30:38 +0100 Max Kellermann escreveu: > Allow the client to free its data structures only after all files have > been closed (fixing use-after-free bugs). Hmm... Shuah is also working on fixing such issues at the media controller stuff, and I made a few

RE: [PATCH v4 09/29] staging: unisys: visorinput: remove unnecessary locking

2016-06-09 Thread Sell, Timothy C
> -Original Message- > From: Thomas Gleixner [mailto:t...@linutronix.de] > Sent: Thursday, June 09, 2016 5:01 AM > To: Kershner, David A > Cc: cor...@lwn.net; mi...@redhat.com; h...@zytor.com; > gre...@linuxfoundation.org; Arfvidson, Erik; Sell, Timothy C; > hof...@osadl.org;

Re: [PATCH 07/10] mm: base LRU balancing on an explicit cost model

2016-06-09 Thread Michal Hocko
On Wed 08-06-16 12:16:05, Johannes Weiner wrote: > On Wed, Jun 08, 2016 at 02:51:37PM +0200, Michal Hocko wrote: > > On Mon 06-06-16 15:48:33, Johannes Weiner wrote: > > > Rename struct zone_reclaim_stat to struct lru_cost, and move from two > > > separate value ratios for the LRU lists to a

Re: [PATCH 3/3 V2] pvclock: Get rid of __pvclock_read_cycles in function pvclock_read_flags

2016-06-09 Thread Paolo Bonzini
On 09/06/2016 14:16, Peter Zijlstra wrote: > On Sat, May 28, 2016 at 08:27:43PM +0800, Minfei Huang wrote: >> +++ b/arch/x86/kernel/pvclock.c >> @@ -61,11 +61,14 @@ void pvclock_resume(void) >> u8 pvclock_read_flags(struct pvclock_vcpu_time_info *src) >> { >> unsigned version; >> -

[PATCH v2] kernel/sysrq: reset watchdog on all CPUs while processing sysrq-w

2016-06-09 Thread Andrey Ryabinin
Lengthy output of sysrq-w may take a lot of time on slow serial console. Currently we reset NMI-watchdog on the current CPU to avoid spurious lockup messages. Sometimes this doesn't work since softlockup watchdog might trigger on another CPU which is waiting for an IPI to proceed. We reset

Re: [RESEND PATCH 1/3] rfkill: Create "rfkill-airplane-mode" LED trigger

2016-06-09 Thread Johannes Berg
On Thu, 2016-05-19 at 09:16 +0200, Pavel Machek wrote: > > In the original situation, without these patches, userspace has to > > have a list of all LEDs that are supposed to indicate airplane > > mode. > Well, that's situation for many LEDs. That doesn't make it a *good* situation though. > >

Re: [PATCH] pvclock: introduce seqcount-like API

2016-06-09 Thread Roman Kagan
On Thu, Jun 09, 2016 at 01:23:23PM +0200, Paolo Bonzini wrote: > The version field in struct pvclock_vcpu_time_info basically implements > a seqcount. Wrap it with the usual read_begin and read_retry functions, > and use these APIs instead of peppering the code with smp_rmb()s. > While at it,

[PATCH v2] drm/dsi: Implement dcs set/get display brightness

2016-06-09 Thread Vinay Simha BN
Provide a small convenience wrapper that set/get the display brightness value Cc: John Stultz Cc: Sumit Semwal Cc: Archit Taneja Cc: Rob Clark Cc: Jani Nikula

[PATCH] drivers: of: add definition of early_init_dt_alloc_reserved_memory_arch

2016-06-09 Thread Ben Dooks
The function early_init_dt_alloc_reserved_memory_arch is defined in drivers/of/of_reserved_mem.c but is not declared in any of the header files. Add the declaration of this to avoid the warning: drivers/of/of_reserved_mem.c:31:19: warning: symbol 'early_init_dt_alloc_reserved_memory_arch' was

RE: [PATCH v9 13/14] usb: gadget: udc: adapt to OTG core

2016-06-09 Thread Yoshihiro Shimoda
Hi Roger, > From: Roger Quadros > Sent: Wednesday, June 08, 2016 6:04 PM > > The OTG state machine needs a mechanism to start and > stop the gadget controller as well as connect/disconnect > from the bus. Add usb_gadget_start(), usb_gadget_stop() > and usb_gadget_connect_control(). > >

Re: [PATCH 3/3 V2] pvclock: Get rid of __pvclock_read_cycles in function pvclock_read_flags

2016-06-09 Thread Paolo Bonzini
On 08/06/2016 10:17, Borislav Petkov wrote: >> > - version = __pvclock_read_cycles(src, , ); >> > + version = src->version; >> > + /* Make the latest version visible */ >> > + smp_rmb(); >> > + >> > + flags = src->flags; >> >/* Make sure

Re: [PATCH v2 4/4] dell-wmi: Rework code for generating sparse keymap and processing WMI events

2016-06-09 Thread Pali Rohár
On Thursday 09 June 2016 13:27:21 Michał Kępień wrote: > > > - case 0x10: > > > - /* Keys pressed */ > > > + case 0x0010: > > > + /* Sequence of keys pressed */ > > > > > > for (i = 2; i < len; ++i) > > > > > > -

Re: [PATCH v3 20/20] dt-bindings: pwm: sti: Update DT bindings with recent changes

2016-06-09 Thread Lee Jones
On Wed, 08 Jun 2016, Rob Herring wrote: > On Wed, Jun 08, 2016 at 10:21:35AM +0100, Lee Jones wrote: > > We're renaming the 'st,pwm-num-chan' binding to 'st,pwm-num-devs' to > > be more inline with the naming conventions of the subsystem. Where > > we used to treat each line as a channel, the

Re: [PATCH 3/3 V2] pvclock: Get rid of __pvclock_read_cycles in function pvclock_read_flags

2016-06-09 Thread Peter Zijlstra
On Sat, May 28, 2016 at 08:27:43PM +0800, Minfei Huang wrote: > +++ b/arch/x86/kernel/pvclock.c > @@ -61,11 +61,14 @@ void pvclock_resume(void) > u8 pvclock_read_flags(struct pvclock_vcpu_time_info *src) > { > unsigned version; > - cycle_t ret; > u8 flags; > > do { > -

Re: [PATCH] cpufreq: governor: Drop gov_cancel_work()

2016-06-09 Thread Rafael J. Wysocki
On Thu, Jun 9, 2016 at 5:21 AM, Viresh Kumar wrote: > On 09-06-16, 01:45, Rafael J. Wysocki wrote: >> From: Rafael J. Wysocki >> >> There's no reason for gov_cancel_work() to exist at all, as it only >> has one caller and the only thing done

Re: [PATCH 1/1] mm/swap.c: flush lru_add pvecs on compound page arrival

2016-06-09 Thread Michal Hocko
On Wed 08-06-16 09:34:01, Dave Hansen wrote: > On 06/08/2016 09:06 AM, Michal Hocko wrote: > >> > Do we have any statistics that tell us how many pages are sitting the > >> > lru pvecs? Although this helps the problem overall, don't we still have > >> > a problem with memory being held in such an

Re: [PATCH 01/21] fs: Replace CURRENT_TIME_SEC with current_fs_time()

2016-06-09 Thread Bob Copeland
On Wed, Jun 08, 2016 at 10:04:45PM -0700, Deepa Dinamani wrote: > CURRENT_TIME_SEC is not y2038 safe. current_fs_time() will > be transitioned to use 64 bit time along with vfs in a > separate patch. > There is no plan to transistion CURRENT_TIME_SEC to use > y2038 safe time interfaces. [...] >

Re: [PATCH 3/3 V2] pvclock: Get rid of __pvclock_read_cycles in function pvclock_read_flags

2016-06-09 Thread Paolo Bonzini
On 09/06/2016 13:28, Borislav Petkov wrote: >> > technically, they should be virt_rmb() -- it really doesn't matter of >> > course because reads are never reordered on x86. > You mean > > version = src->version; > flags = src->flags; > > are not reordered? > > I don't think so.

Re: [PATCH 3/3 V2] pvclock: Get rid of __pvclock_read_cycles in function pvclock_read_flags

2016-06-09 Thread Peter Zijlstra
On Thu, Jun 09, 2016 at 02:26:59PM +0200, Paolo Bonzini wrote: > > > On 09/06/2016 14:16, Peter Zijlstra wrote: > > On Sat, May 28, 2016 at 08:27:43PM +0800, Minfei Huang wrote: > >> +++ b/arch/x86/kernel/pvclock.c > >> @@ -61,11 +61,14 @@ void pvclock_resume(void) > >> u8

[PATCH] binfmt_em86: Fix incompatible pointer type

2016-06-09 Thread Daniel Wagner
From: Daniel Wagner Since the -Wincompatible-pointer-types is reported as error, alpha doesn't build anymore. Let's fix it in a minimal way. fs/binfmt_em86.c:73:35: error: passing argument 2 of ‘copy_strings_kernel’ from incompatible pointer type

Re: [RFC v4 01/14] regulator: of: Add helper for getting all supplies

2016-06-09 Thread Rafael J. Wysocki
On Thu, Jun 9, 2016 at 12:29 PM, Mark Brown wrote: > On Thu, Jun 09, 2016 at 11:44:18AM +0200, Krzysztof Kozlowski wrote: >> Few drivers have a need of getting regulator supplies without knowing >> their names: >> 1. The Simple Framebuffer driver works on setup provided by

Re: [PATCH v9 03/14] usb: hcd.h: Add OTG to HCD interface

2016-06-09 Thread Sergei Shtylyov
Hello. On 6/9/2016 10:31 AM, Roger Quadros wrote: The OTG core will use struct otg_hcd_ops to interface with the HCD controller. Host controller driver (HCD) controller? Maybe just HC? :-) OK. OTOH, my googling has shown that HCD may stand for both HC driver and HC device... The

Re: [PATCH 1/2] max8903: adds support for initiation via device tree.

2016-06-09 Thread Krzysztof Kozlowski
Hi, Thanks for your contribution. Few comments below: On Thu, Jun 2, 2016 at 8:44 AM, wrote: > From: Chris Lapa > > This commit also adds requesting gpio's via devm_gpio_request() to ensure > the gpio is available for usage by the driver. > >

Re: [RFC v4 01/14] regulator: of: Add helper for getting all supplies

2016-06-09 Thread Krzysztof Kozlowski
On 06/09/2016 12:29 PM, Mark Brown wrote: > On Thu, Jun 09, 2016 at 11:44:18AM +0200, Krzysztof Kozlowski wrote: >> Few drivers have a need of getting regulator supplies without knowing >> their names: >> 1. The Simple Framebuffer driver works on setup provided by bootloader >>(outside of

[PATCH 0/10 -v4] Handle oom bypass more gracefully

2016-06-09 Thread Michal Hocko
Hi, this is the v4 version of the patchse. Previous version was posted http://lkml.kernel.org/r/1464945404-30157-1-git-send-email-mho...@kernel.org It's been mostly my screwups during the rebase that were fixed and patch 10 reworked. The following 10 patches should put some order to very rare

[PATCH 04/10] mm, oom_adj: make sure processes sharing mm have same view of oom_score_adj

2016-06-09 Thread Michal Hocko
From: Michal Hocko oom_score_adj is shared for the thread groups (via struct signal) but this is not sufficient to cover processes sharing mm (CLONE_VM without CLONE_SIGHAND) and so we can easily end up in a situation when some processes update their oom_score_adj and confuse

[PATCH 08/10] mm, oom: task_will_free_mem should skip oom_reaped tasks

2016-06-09 Thread Michal Hocko
From: Michal Hocko 0-day robot has encountered the following: [ 82.694232] Out of memory: Kill process 3914 (trinity-c0) score 167 or sacrifice child [ 82.695110] Killed process 3914 (trinity-c0) total-vm:55864kB, anon-rss:1512kB, file-rss:1088kB, shmem-rss:25616kB [

[PATCH 07/10] mm, oom: fortify task_will_free_mem

2016-06-09 Thread Michal Hocko
From: Michal Hocko task_will_free_mem is rather weak. It doesn't really tell whether the task has chance to drop its mm. 98748bd72200 ("oom: consider multi-threaded tasks in task_will_free_mem") made a first step into making it more robust for multi-threaded applications so now

[PATCH 02/10] proc, oom: drop bogus sighand lock

2016-06-09 Thread Michal Hocko
From: Michal Hocko Oleg has pointed out that can simplify both oom_adj_{read,write} and oom_score_adj_{read,write} even further and drop the sighand lock. The main purpose of the lock was to protect p->signal from going away but this will not happen since ea6d290ca34c ("signals:

Re: [PATCH v2 0/4] dell-wmi: Changes in WMI event code handling

2016-06-09 Thread Michał Kępień
> First patch describe problem about 0xe045 code. Second and third are just > cosmetic and last rework code which processing WMI events. It should be > properly tested on more Dell machines, to check that everything is still > working correctly. > > Changes since v1: > * Fixed comments > * Fixed

[PATCH 1/1] Staging: comedi: dmm32at: fix BIT macro issue.

2016-06-09 Thread Ravishankar Karkala Mallikarjunayya
This Replace all occurences of (1< --- drivers/staging/comedi/drivers/dmm32at.c | 98

Re: [PATCH 3/3 V2] pvclock: Get rid of __pvclock_read_cycles in function pvclock_read_flags

2016-06-09 Thread Peter Zijlstra
On Thu, Jun 09, 2016 at 02:16:03PM +0200, Peter Zijlstra wrote: > What's wrong with: > > u8 flags = READ_ONCE(src->flags); > > ? > > (and have the flags store be done using WRITE_ONCE() of course). > > Sure, if your total state is larger than one word you need the > seqcount for

Re: [PATCH v10 08/14] usb: otg: add OTG/dual-role core

2016-06-09 Thread Sergei Shtylyov
On 6/9/2016 10:53 AM, Roger Quadros wrote: It provides APIs for the following tasks - Registering an OTG/dual-role capable controller - Registering Host and Gadget controllers to OTG core - Providing inputs to and kicking the OTG state machine Provide a dual-role device (DRD) state machine.

Re: [PATCH] mmc: dw_mmc: remove UBSAN warning in dw_mci_setup_bus()

2016-06-09 Thread Jaehoon Chung
Hi Seung-Woo, On 06/08/2016 01:07 PM, Seung-Woo Kim wrote: > This patch removes following UBSAN warnings in dw_mci_setup_bus(). > The warnings are caused because of shift with more than 31 on 32 > bit variable, so this patch fixes to shift only for less than 32. > > UBSAN: Undefined behaviour

Re: [Qemu-devel] [RFC PATCH 0/3] x86: Add support for guest DMA dirty page tracking

2016-06-09 Thread Zhou Jie
TO Alex TO Michael In your solution you add a emulate PCI bridge to act as a bridge between direct assigned devices and the host bridge. Do you mean put all direct assigned devices to one emulate PCI bridge? If yes, this maybe bring some problems. We are writing a patchset to

Re: [PATCH] ARM: Keystone: Introduce Kconfig option to compile in typical Keystone features

2016-06-09 Thread Mark Brown
On Wed, Jun 08, 2016 at 11:50:35PM -0700, Tony Lindgren wrote: > * Arnd Bergmann [160608 09:06]: > > The downside would be any users that want the options we enable there > > turned off and have them silently enabled when upgrading the kernel. > I think the long term solution

Re: [alsa-devel] [PATCH v2 6/9] ASoC: mediatek: add mt2701 platform driver implementation.

2016-06-09 Thread Mark Brown
On Wed, Jun 08, 2016 at 11:32:22AM +0800, Garlic Tseng wrote: > On Tue, 2016-06-07 at 17:31 +0100, Mark Brown wrote: > > > + SOC_DAPM_SINGLE_AUTODISABLE("Multi ch asrc out3", PWR2_TOP_CON, 7, 1, > > > + 1), > > > +}; > > On/off controls should end in Switch. > Do you

[PATCH net-next v3] netvsc: get rid of completion timeouts

2016-06-09 Thread Vitaly Kuznetsov
I'm hitting 5 second timeout in rndis_filter_set_rss_param() while setting RSS parameters for the device. When this happens we end up returning -ETIMEDOUT from the function and rndis_filter_device_add() falls back to setting net_device->max_chn = 1; net_device->num_chn = 1;

Re: [PATCH v3 4/4] soc: Add SoC driver for Freescale Vybrid platform

2016-06-09 Thread maitysanchayan
Hello Rob, On 16-05-27 15:38:17, maitysancha...@gmail.com wrote: > On 16-05-27 10:31:55, Arnd Bergmann wrote: > > On Friday, May 27, 2016 12:03:01 PM CEST maitysancha...@gmail.com wrote: > > > > > > So if I understand correctly, the binding at the SoC level is fine. > > > Keeping that but

Re: linux-next: build warnings after merge of the kbuild tree

2016-06-09 Thread Michal Marek
Dne 9.6.2016 v 06:05 Stephen Rothwell napsal(a): > On Wed, 8 Jun 2016 19:56:38 -0700 Kees Cook wrote: >> Ah, yes, that should default to off. We'll get a fix landed ASAP. > > Note that this was an allmodconfig build. The default is 'n', but > allmodconfig will turn it on

[PATCH 2/2 v9] ASoC: dwc: Add irq parameter to DOCUMENTATION

2016-06-09 Thread Jose Abreu
A parameter description for the interruptions of the I2S controller was added. Signed-off-by: Jose Abreu Acked-by: Rob Herring Cc: Carlos Palminha Cc: Mark Brown Cc: Liam Girdwood Cc:

[PATCH 1/2 v9] ASoC: dwc: Add PIO PCM extension

2016-06-09 Thread Jose Abreu
A PCM extension was added to I2S driver so that audio samples are transferred using PIO mode. The PCM supports two channels @ 16 or 32 bits with rates 32k, 44.1k and 48k. Although the mainline I2S driver uses ALSA DMA engine the I2S controller can be built without DMA support, therefore this is

[PATCH 1/2] dmaengine: bcm2835: Fix cyclic DMA period splitting

2016-06-09 Thread Matthias Reichl
The code responsible for splitting periods into chunks that can be handled by the DMA controller missed to update total_len, the number of bytes processed in the current period, when there are more chunks to follow. Therefore total_len was stuck at 0 and the code didn't work at all. This resulted

[PATCH 0/2 v9] Add I2S audio support for ARC AXS10x boards

2016-06-09 Thread Jose Abreu
ARC AXS10x platforms consist of a mainboard with several peripherals. One of those peripherals is an HDMI output port controlled by the ADV7511 transmitter. This patch set adds I2S audio for the AXS10x platform. NOTE: Although the mainline I2S driver uses ALSA DMA engine, this controller can be

[PATCH 2/2] dmaengine: bcm2835: Avoid splitting periods into very small chunks

2016-06-09 Thread Matthias Reichl
The current cyclic DMA period splitting implementation can generate very small chunks at the end of each period. For example a 65536 byte period will be split into a 65532 byte chunk and a 4 byte chunk on the "lite" DMA channels. This increases pressure on the RAM controller as the DMA controller

Re: [RFC v4 4/7] extcon: usb-gpio: Add support for VBUS detection

2016-06-09 Thread Roger Quadros
On 09/06/16 11:43, Krzysztof Kozlowski wrote: > On 06/09/2016 10:38 AM, Roger Quadros wrote: >> Hi, >> >> On 08/06/16 16:48, Krzysztof Kozlowski wrote: >>> Add VBUS pin detection support to extcon-usb-gpio driver for boards >>> which have both VBUS and ID pins, or only one of them. >>> >>> The

Re: Files leak from nfsd in 4.7.1-rc1 (and more?)

2016-06-09 Thread Andrew W Elble
> But...suppose we have two racing OPEN calls. They're both in > nfs4_get_vfs_file. One opens the file and succeeds and the other fails > and ends up in out_put_access. At that point, you could end up > clobbering the successful update to st_access_bmap from the other task, > and we'd end up not

RE: [PATCH 1/1] mm/swap.c: flush lru_add pvecs on compound page arrival

2016-06-09 Thread Odzioba, Lukasz
On Wed 08-07-16 17:04:00, Michal Hocko wrote: > I do not see how a SIGTERM would make any difference. But see below. This is how we encounter this problem initially, by hitting ctr-c while running parallel memory intensive workload, which ended up not calling munmap on allocated memory. > Is

Re: [RFC PATCH 15/15] iommu/exynos: update to use iommu big-endian

2016-06-09 Thread Ben Dooks
On 09/06/16 07:51, Marek Szyprowski wrote: > Hi > > > On 2016-06-08 20:31, Matthew Leach wrote: >> From: Ben Dooks >> >> Add initial support for big endian by always writing the pte >> in le32. Note, revisit if hardware capable of doing big endian >> fetches. >> >>

Re: [PATCH 00/18] staging/android: clean up SW_SYNC

2016-06-09 Thread Daniel Vetter
On Wed, Jun 08, 2016 at 10:45:34PM -0700, Greg Kroah-Hartman wrote: > On Wed, Jun 08, 2016 at 04:07:59PM -0300, Gustavo Padovan wrote: > > Hi Greg, > > > > Any comment on this? > > I am just starting to catch up on patches, please give me some time, > staging patches are at the bottom of my

Re: [RFC PATCH v1.9 07/14] x86/stacktrace: add function for detecting reliable stack traces

2016-06-09 Thread Jiri Slaby
On 04/12/2016, 05:56 PM, Josh Poimboeuf wrote: >> We had been using unwinder for over a decade in SUSE but it stopped >> working for assembly recently (for obvious reasons). So having a working >> and reliable unwinder again is one of the top priorities for us. > > Since you already have an

Re: [RFC PATCH 2/4] trace: Introduce an output interface from ftrace to STM

2016-06-09 Thread Alexander Shishkin
Chunyan Zhang writes: > On Tue, Jun 7, 2016 at 6:04 PM, Alexander Shishkin > wrote: >> Chunyan Zhang writes: >> >>> This patch is introducing a new function to print Ftrace messages >>> to STM buffer when

Re: [RFC 1/1] ARM: print MHz in /proc/cpuinfo

2016-06-09 Thread Sudeep Holla
On 08/06/16 20:31, Jon Mason wrote: On Wed, Jun 08, 2016 at 09:34:06AM +0100, Sudeep Holla wrote: On 07/06/16 22:08, Jon Mason wrote: Query the CPU core clock in the device tree to determine the core clock speed. How do guarantee that it's the current frequency of the CPU ? I am basing

Re: [v2,01/10] i2c-i801: Remove hwpec from block byte-by-byte function

2016-06-09 Thread Benjamin Tissoires
Hi Corey, I wanted to send a global rev-by, but it looks like some changes in this series are required from my point of view, so I'll just send individual rev-by and change requests as my review progresses. On May 29 2016 or thereabouts, Corey Minyard wrote: > From: Corey Minyard

  1   2   3   4   5   6   7   8   9   10   >