Re: [PATCH v7 3/4] perf: xgene: Add APM X-Gene SoC Performance Monitoring Unit driver

2016-07-09 Thread Paul Gortmaker
On Wed, Jul 6, 2016 at 8:07 PM, Tai Nguyen wrote: > Signed-off-by: Tai Nguyen > --- > Documentation/perf/xgene-pmu.txt | 48 ++ > drivers/perf/Kconfig |7 + > drivers/perf/Makefile|1 + > drivers/perf/xgene_pmu.c | 1398 > +++

[RFC PATCH v3 0/2] mm, thp: convert from optimistic swapin collapsing to conservative

2016-07-09 Thread Ebru Akagunduz
This patch series fixes comment inconsistency and supplies to decide to swapin looking the amount of young pages. Changes in v2: - Don't change thp design, notice young pages if needs to swapin - Add comment line fixing patch Changes in v3: - Remove revert patch (allocstall), the patch auto

[PATCH v3 1/2] mm, thp: fix comment inconsistency for swapin readahead functions

2016-07-09 Thread Ebru Akagunduz
After fixing swapin issues, comment lines stayed as in old version. This patch updates the comments. Signed-off-by: Ebru Akagunduz Cc: Hillf Danton --- Changes in v2: - Newly created in this version. Changes in v3: - Replace Reported-by with Cc (Hillf Danton) - Remove RFC tag (Hillf Danton)

[RFC PATCH v3 2/2] mm, thp: convert from optimistic swapin collapsing to conservative

2016-07-09 Thread Ebru Akagunduz
To detect whether khugepaged swapin worthwhile, this patch checks the amount of young pages. There should be at least half of HPAGE_PMD_NR to swapin. Signed-off-by: Ebru Akagunduz Suggested-by: Minchan Kim --- Changes in v2: - Don't change thp design, only notice amount of young pages, if kh

[PATCH] media: solo6x10: increase FRAME_BUF_SIZE

2016-07-09 Thread Andrey Utkin
In practice, devices sometimes return frames larger than current buffer size, leading to failure in solo_send_desc(). It is not clear which minimal increase in buffer size would be enough, so this patch doubles it, this should be safely assumed as sufficient. Signed-off-by: Andrey Utkin --- driv

Re: [PATCH v6 00/10] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-07-09 Thread Rafael J. Wysocki
On Saturday, July 09, 2016 11:44:47 AM Hanjun Guo wrote: > On 2016/7/8 21:22, Lorenzo Pieralisi wrote: > > On Thu, Jul 07, 2016 at 03:58:04PM +0200, Rafael J. Wysocki wrote: > > > > [...] > > > >>> Anyway let's avoid these petty arguments, I agree there must be some > >>> sort of ARM64 ACPI maintai

Re: [PATCH] mtd: nand: brcmnand: Change BUG_ON in brcmnand_send_cmd

2016-07-09 Thread Brian Norris
On Fri, Jul 08, 2016 at 10:36:39AM -0700, Florian Fainelli wrote: > Change the BUG_ON() condition in brcmnand_send_cmd() which checks for > the interrupt status "controller ready" bit to a WARN_ON. > > There is no good reason to kill the system when this condition occur > because we could have sys

[PATCH] x86 / hibernate: Use hlt_play_dead() when resuming from hibernation

2016-07-09 Thread Rafael J. Wysocki
From: Rafael J. Wysocki On Intel hardware, native_play_dead() uses mwait_play_dead() by default and only falls back to the other methods if that fails. That also happens during resume from hibernation, when the restore (boot) kernel runs disable_nonboot_cpus() to take all of the CPUs except for t

Re: [PATCH] mtd: Replace if and BUG with BUG_ON

2016-07-09 Thread Brian Norris
Hi, On Tue, May 31, 2016 at 07:41:23AM +0200, Julia Lawall wrote: > On Mon, 30 May 2016, Ezequiel Garcia wrote: > > On 28 May 2016 at 13:41, Amitoj Kaur Chawla wrote: > > > Replace if condition and BUG() with a BUG_ON having the conditional > > > expression of the if statement as argument. [...]

Re: [PATCH] drivers/mtd/chips/cfi_cmdset_0020.c: Deinline do_write_buffer, save 5316 bytes

2016-07-09 Thread Brian Norris
On Fri, Apr 08, 2016 at 08:35:43PM +0200, Denys Vlasenko wrote: > This function compiles to 2554 bytes of machine code. > In C, the function is almost 200 lines long. > > It has only one callsite, but forced inlining that much code > makes gcc generate significantly worse code. Let gcc itself deci

[PATCH v2 05/17] libnvdimm, nfit: move flush hint mapping to region-device driver-data

2016-07-09 Thread Dan Williams
In preparation for triggering flushes of a DIMM's writes-posted-queue (WPQ) via the pmem driver move mapping of flush hint addresses to the region driver. Since this uses devm_nvdimm_memremap() the flush addresses will remain mapped while any region to which the dimm belongs is active. We need to

[PATCH v2 00/17] replace pcommit with ADR or directed flushing

2016-07-09 Thread Dan Williams
Changes since v1 [1]: 1/ Move flush address data from nvdimm_drvdata to nd_region_data (Greg, Toshi) 2/ Add more detail to cover letter and patch descriptions (Linda, Jeff) 3/ Account for s/REQ_FLUSH/REQ_PREFLUSH/ rename pending in -next. 4/ Add a directed flush at pmem ->remove() and ->shut

[PATCH v2 13/17] libnvdimm, pmem: use nvdimm_flush() for namespace I/O writes

2016-07-09 Thread Dan Williams
nsio_rw_bytes() is used to write info block metadata to the namespace, so it should trigger a flush after every write. Replace wmb_pmem() with nvdimm_flush() in this path. Cc: Ross Zwisler Signed-off-by: Dan Williams --- drivers/nvdimm/claim.c |2 +- 1 file changed, 1 insertion(+), 1 delet

[PATCH v2 16/17] x86/insn: remove pcommit

2016-07-09 Thread Dan Williams
The pcommit instruction is being deprecated in favor of either ADR (asynchronous DRAM refresh: flush-on-power-fail) at the platform level, or posted-write-queue flush addresses as defined by the ACPI 6.x NFIT (NVDIMM Firmware Interface Table). Cc: Thomas Gleixner Cc: Ingo Molnar Cc: "H. Peter An

[PATCH v2 17/17] pmem: kill __pmem address space

2016-07-09 Thread Dan Williams
The __pmem address space was meant to annotate codepaths that touch persistent memory and need to coordinate a call to wmb_pmem(). Now that wmb_pmem() is gone, there is little need to keep this annotation. Cc: Christoph Hellwig Cc: Ross Zwisler Signed-off-by: Dan Williams --- Documentation/fi

[PATCH v2 14/17] pmem: kill wmb_pmem()

2016-07-09 Thread Dan Williams
All users have been replaced with flushing in the pmem driver. Cc: Ross Zwisler Signed-off-by: Dan Williams --- arch/x86/include/asm/pmem.h | 36 ++--- include/linux/pmem.h| 47 --- 2 files changed, 6 insertions(+),

[PATCH v2 08/17] libnvdimm: introduce nvdimm_flush() and nvdimm_has_flush()

2016-07-09 Thread Dan Williams
nvdimm_flush() is a replacement for the x86 'pcommit' instruction. It is an optional write flushing mechanism that an nvdimm bus can provide for the pmem driver to consume. In the case of the NFIT nvdimm-bus-provider nvdimm_flush() is implemented as a series of flush-hint-address [1] writes to ea

[PATCH v2 15/17] Revert "KVM: x86: add pcommit support"

2016-07-09 Thread Dan Williams
This reverts commit 8b3e34e46aca9b6d349b331cd9cf71ccbdc91b2e. Given the deprecation of the pcommit instruction, revert its usage as a vm exit source in kvm. Cc: Xiao Guangrong Cc: Paolo Bonzini Cc: Ross Zwisler Signed-off-by: Dan Williams --- arch/x86/include/asm/vmx.h |1 - arch/x8

[PATCH v2 07/17] libnvdimm: keep region data alive over namespace removal

2016-07-09 Thread Dan Williams
nd_region device driver data will be used in the namespace i/o path. Re-order nd_region_remove() to ensure this data stays live across namespace device removal Signed-off-by: Dan Williams --- drivers/nvdimm/region.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers

[PATCH v2 11/17] libnvdimm, pmem: flush posted-write queues on shutdown

2016-07-09 Thread Dan Williams
Commit writes to media on system shutdown or pmem driver unload. Signed-off-by: Dan Williams --- drivers/nvdimm/bus.c | 16 drivers/nvdimm/pmem.c |8 include/linux/nd.h|1 + 3 files changed, 25 insertions(+) diff --git a/drivers/nvdimm/bus.c b/drivers/nv

[PATCH v2 10/17] libnvdimm, pmem: use REQ_FUA, REQ_FLUSH for nvdimm_flush()

2016-07-09 Thread Dan Williams
Given that nvdimm_flush() has higher overhead than wmb_pmem() (pointer chasing through nd_region), and that we otherwise assume a platform has ADR capability when flush hints are not present, move nvdimm_flush() to REQ_FLUSH context. Cc: Ross Zwisler Signed-off-by: Dan Williams --- drivers/nvdi

[PATCH v2 12/17] fs/dax: remove wmb_pmem()

2016-07-09 Thread Dan Williams
Flushing posted-write queues is now deferred to REQ_FLUSH context, or otherwise handled by an ADR event at the platform level. Cc: Ross Zwisler Signed-off-by: Dan Williams --- fs/dax.c |7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/fs/dax.c b/fs/dax.c index 761495b

[PATCH v2 06/17] tools/testing/nvdimm: simulate multiple flush hints per-dimm

2016-07-09 Thread Dan Williams
Sample nfit data to test the kernel's handling of the multiple flush-hint case. Signed-off-by: Dan Williams --- tools/testing/nvdimm/test/nfit.c | 55 +++--- 1 file changed, 33 insertions(+), 22 deletions(-) diff --git a/tools/testing/nvdimm/test/nfit.c b/tools

[PATCH v2 04/17] libnvdimm, nfit: remove nfit_spa_map() infrastructure

2016-07-09 Thread Dan Williams
Now that all shared mappings are handled by devm_nvdimm_memremap() we no longer need nfit_spa_map() nor do we need to trigger a callback to the bus provider at region disable time. Signed-off-by: Dan Williams --- drivers/acpi/nfit.c | 146 -- dri

[PATCH v2 09/17] libnvdimm: cycle flush hints

2016-07-09 Thread Dan Williams
When the NFIT provides multiple flush hint addresses per-dimm it is expressing that the platform is capable of processing multiple flush requests in parallel. There is some fixed cost per flush request, let the cost be shared in parallel on multiple cpus. Since there may not be enough flush hint

[PATCH v2 02/17] nfit: don't override return value of nfit_mem_init

2016-07-09 Thread Dan Williams
We were needlessly converting nfit_mem_init() errors to -ENOMEM. Signed-off-by: Dan Williams --- drivers/acpi/nfit.c |5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/acpi/nfit.c b/drivers/acpi/nfit.c index d79837b9d07e..f8c1a850effc 100644 --- a/drivers/acpi/nf

[PATCH v2 01/17] nfit: always associate flush hints

2016-07-09 Thread Dan Williams
Before enabling use of flush hints for pmem regions, we need to make sure they are always associated. Move the initialization of nfit_flush out of the block-window specific init path to the general init path. Cc: Ross Zwisler Signed-off-by: Dan Williams --- drivers/acpi/nfit.c | 17 -

[PATCH v2 03/17] libnvdimm: introduce devm_nvdimm_memremap(), convert nfit_spa_map() users

2016-07-09 Thread Dan Williams
In preparation for generically mapping flush hint addresses for both the BLK and PMEM use case, provide a generic / reference counted mapping api. Given the fact that a dimm may belong to multiple regions (PMEM and BLK), the flush hint addresses need to be held valid as long as any region associat

Re: [PATCH 11/13] blk-mq: allow the driver to pass in an affinity mask

2016-07-09 Thread Christoph Hellwig
On Mon, Jul 04, 2016 at 11:35:28AM +0200, Alexander Gordeev wrote: > > mq_map is initialized to zero already, so we don't really need the > > assignment for queue 0. The reason why this check exists is because > > we start with queue = -1 and we never want to assignment -1 to mq_map. > > Would th

Re: [PATCH] intel_pstate: Fix MSR_CONFIG_TDP_x addressing in core_get_max_pstate

2016-07-09 Thread Srinivas Pandruvada
On Sat, 2016-07-09 at 02:45 +0200, Rafael J. Wysocki wrote: > On Friday, July 08, 2016 12:39:07 PM Srinivas Pandruvada wrote: > > On Fri, 2016-07-08 at 20:42 +0200, Jan Kiszka wrote: > > > If MSR_CONFIG_TDP_CONTROL is locked, we currently try to address > > > some > > > MSR 0x8648 or so. Mask o

Re: [PATCH 07/13] pci: Provide sensible irq vector alloc/free routines

2016-07-09 Thread Christoph Hellwig
On Wed, Jul 06, 2016 at 10:05:45AM +0200, Alexander Gordeev wrote: > > + pci_enable_msi, pci_enable_msi_range, pci_enable_msi_exact, > > pci_disable_msi, > > + pci_msi_vec_count, pci_enable_msix_range, pci_enable_msix_exact, > > + pci_disable_msix, pci_msix_vec_count > > Description of these func

Re: [PATCH 08/13] pci: spread interrupt vectors in pci_alloc_irq_vectors

2016-07-09 Thread Christoph Hellwig
On Thu, Jul 07, 2016 at 01:05:01PM +0200, Alexander Gordeev wrote: > irq_create_affinity_mask() bails out with no affinity in case of single > vector, but alloc_descs() (see below (*)) assigns the whole affinity > mask. It should be consistent instead. I don't understand the comment. If we only h

Re: [PATCH 12/13] nvme: switch to use pci_alloc_irq_vectors

2016-07-09 Thread Christoph Hellwig
On Thu, Jul 07, 2016 at 09:30:19PM +0200, Alexander Gordeev wrote: > On Mon, Jul 04, 2016 at 05:39:33PM +0900, Christoph Hellwig wrote: > > @@ -1575,6 +1546,7 @@ static int nvme_dev_add(struct nvme_dev *dev) > > dev->tagset.cmd_size = nvme_cmd_size(dev); > > dev->tagset.flag

Re: [PATCH v3 0/7] lib: string: add functions to case-convert strings

2016-07-09 Thread Markus Mayer
On 9 July 2016 at 20:13, Chris Metcalf wrote: > On 7/8/2016 6:43 PM, Markus Mayer wrote: >> >> This series introduces a family of generic string case conversion >> functions. This kind of functionality is needed in several places in >> the kernel. Right now, everybody seems to be implementing thei

Odd performance results

2016-07-09 Thread Paul E. McKenney
Hello! So I ran a quick benchmark which showed stair-step results. I immediately thought "Ah, this is due to CPU 0 and 1, 2 and 3, 4 and 5, and 6 and 7 being threads in a core." Then I thought "Wait, this is an x86!" Then I dumped out cpu*/topology/thread_siblings_list, getting the following:

Re: [PATCH v2 08/17] libnvdimm: introduce nvdimm_flush() and nvdimm_has_flush()

2016-07-09 Thread kbuild test robot
Hi, [auto build test ERROR on linux-nvdimm/libnvdimm-for-next] [also build test ERROR on next-20160708] [cannot apply to v4.7-rc6] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Dan-Williams/rep

Re: [PATCH v3 0/7] lib: string: add functions to case-convert strings

2016-07-09 Thread Chris Metcalf
On 7/8/2016 6:43 PM, Markus Mayer wrote: This series introduces a family of generic string case conversion functions. This kind of functionality is needed in several places in the kernel. Right now, everybody seems to be implementing their own copy of this functionality. Based on the discussion

Re: [PATCH v2 08/17] libnvdimm: introduce nvdimm_flush() and nvdimm_has_flush()

2016-07-09 Thread Dan Williams
On Sat, Jul 9, 2016 at 9:47 PM, kbuild test robot wrote: > Hi, > > [auto build test ERROR on linux-nvdimm/libnvdimm-for-next] > [also build test ERROR on next-20160708] > [cannot apply to v4.7-rc6] > [if your patch is applied to the wrong git tree, please drop us a note to > help improve the syst

Re: Odd performance results

2016-07-09 Thread Peter Zijlstra
On 10 July 2016 06:26:39 CEST, "Paul E. McKenney" wrote: >Hello! > >So I ran a quick benchmark which showed stair-step results. I >immediately >thought "Ah, this is due to CPU 0 and 1, 2 and 3, 4 and 5, and 6 and 7 >being threads in a core." Then I thought "Wait, this is an x86!" >Then I dump

Re: [RFC PATCH 0/3] doc-rst: customize HTML (RTD) theme

2016-07-09 Thread Jonathan Corbet
On Tue, 5 Jul 2016 14:55:09 -0300 Mauro Carvalho Chehab wrote: > I hope you don't mind. I'm merging those three patches on my tree > (for now, they're on an experimental tree that I can easily rebase, if > needed). If OK for you, my plan is to merge it on a separate branch, > together with the ot

Re: [PATCH v2 03/17] libnvdimm: introduce devm_nvdimm_memremap(), convert nfit_spa_map() users

2016-07-09 Thread kbuild test robot
Hi, [auto build test ERROR on linux-nvdimm/libnvdimm-for-next] [also build test ERROR on v4.7-rc6 next-20160708] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Dan-Williams/replace-pcommit-with-

Re: [CRIU] Introspecting userns relationships to other namespaces?

2016-07-09 Thread Andrew Vagin
On Fri, Jul 08, 2016 at 10:13:08PM -0500, Eric W. Biederman wrote: > "W. Trevor King" writes: > > > On Thu, Jul 07, 2016 at 08:01:52AM -0700, James Bottomley wrote: > >> In theory, we could get nsfs to show this information as an option > >> (just add a show_options entry to the superblock ops),

Re: Missing include file in include/uapi/linux/errqueue.h?

2016-07-09 Thread Brooks Moses
On Sat, Jul 9, 2016 at 10:36 AM, Brooks Moses wrote: > I've been attempting to qualify the Linux 4.5.2 user-space headers for > a toolchain release, and ran into what looks like a missing include > file in include/uapi/linux/errqueue.h. In particular, > https://github.com/torvalds/linux/commit/f2

[char-misc 4.7] mei: me: disable driver on SPT SPS firmware

2016-07-09 Thread Tomas Winkler
Sunrise Point PCH with SPS Firmware doesn't expose working MEI interface, we need to quirk it out. Cc: #4.4+ Signed-off-by: Tomas Winkler --- drivers/misc/mei/pci-me.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/misc/mei/pci-me.c b/drivers/misc/mei/pci-me.c index

Re: [tip:x86/debug] printk: Make the printk*once() variants return a value

2016-07-09 Thread Borislav Petkov
On Sat, Jul 09, 2016 at 10:56:55AM -0700, Joe Perches wrote: > defconfigs both with and without CONFIG_PRINTK build > properly with the proposed change to this specific patch. Did you try latest tip/master? > Borislav, your delightful personality always impresses. > Never change. What goes aroun

<    1   2