[PATCH 1/7] mfd: max8997-irq: 'inline' should be at the beginning of the declaration

2016-09-15 Thread Lee Jones
Reported by [-Wold-style-declaration]. Signed-off-by: Lee Jones --- drivers/mfd/max8997-irq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/max8997-irq.c b/drivers/mfd/max8997-irq.c index b95a46d7..326f17b 100644 --- a/drivers/mfd/max8997-irq.c +++ b/drivers/mfd

[tip:x86/asm] sched/core: Allow putting thread_info into task_struct

2016-09-15 Thread tip-bot for Andy Lutomirski
Commit-ID: c65eacbe290b8141554c71b2c94489e73ade8c8d Gitweb: http://git.kernel.org/tip/c65eacbe290b8141554c71b2c94489e73ade8c8d Author: Andy Lutomirski AuthorDate: Tue, 13 Sep 2016 14:29:24 -0700 Committer: Ingo Molnar CommitDate: Thu, 15 Sep 2016 08:25:13 +0200 sched/core: Allow puttin

[PATCH] soc: rockchip: power-domain: Handle errors from of_genpd_add_provider_onecell

2016-09-15 Thread Tomeu Vizoso
It was a bit surprising that the device was reported to have probed just fine, but the provider hadn't been registered. So handle any errors when registering the provider and fail the probe accordingly. Signed-off-by: Tomeu Vizoso Cc: Caesar Wang --- drivers/soc/rockchip/pm_domains.c | 6 +

[PATCH 5/7] mfd: ab8500-debugfs: Remove 'weak' function suspend_test_wake_cause_interrupt_is_mine()

2016-09-15 Thread Lee Jones
There are no other functions which can over-ride it. Signed-off-by: Lee Jones --- drivers/mfd/ab8500-debugfs.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/drivers/mfd/ab8500-debugfs.c b/drivers/mfd/ab8500-debugfs.c index b948088..acf6c00 100644 --- a/drivers/mfd

[tip:x86/asm] x86: Move thread_info into task_struct

2016-09-15 Thread tip-bot for Andy Lutomirski
Commit-ID: 15f4eae70d365bba26854c90b6002aaabb18c8aa Gitweb: http://git.kernel.org/tip/15f4eae70d365bba26854c90b6002aaabb18c8aa Author: Andy Lutomirski AuthorDate: Tue, 13 Sep 2016 14:29:25 -0700 Committer: Ingo Molnar CommitDate: Thu, 15 Sep 2016 08:25:13 +0200 x86: Move thread_info in

[tip:perf/urgent] perf/x86/intel: Don't disable "intel_bts" around "intel" event batching

2016-09-15 Thread tip-bot for Alexander Shishkin
Commit-ID: cecf62352aee2b4fe114aafd1b8c5f265a4243ce Gitweb: http://git.kernel.org/tip/cecf62352aee2b4fe114aafd1b8c5f265a4243ce Author: Alexander Shishkin AuthorDate: Thu, 15 Sep 2016 11:22:33 +0300 Committer: Ingo Molnar CommitDate: Thu, 15 Sep 2016 11:25:26 +0200 perf/x86/intel: Don't

[PATCH 7/7] mfd: cros_ec_spi: Remove unused variable 'request'

2016-09-15 Thread Lee Jones
Signed-off-by: Lee Jones --- drivers/mfd/cros_ec_spi.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/mfd/cros_ec_spi.c b/drivers/mfd/cros_ec_spi.c index ebe9b94..a518832 100644 --- a/drivers/mfd/cros_ec_spi.c +++ b/drivers/mfd/cros_ec_spi.c @@ -366,7 +366,6 @@ static int cros_ec_sp

[PATCH 6/7] mfd: omap-usb-host: Return value is not 'const int'

2016-09-15 Thread Lee Jones
Change from 'const int' to just 'int'. Cc: Signed-off-by: Lee Jones --- drivers/mfd/omap-usb-host.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/omap-usb-host.c b/drivers/mfd/omap-usb-host.c index 1d924d1..7aab376 100644 --- a/drivers/mfd/omap-usb-host.c +++ b

[PATCH 3/7] mfd: db8500-prcmu: Remove unused *prcmu_set_ddr_opp() calls

2016-09-15 Thread Lee Jones
There are no call sites for these functions. Strip them out. Signed-off-by: Lee Jones --- drivers/mfd/db8500-prcmu.c | 19 --- include/linux/mfd/db8500-prcmu.h | 6 -- include/linux/mfd/dbx500-prcmu.h | 9 - 3 files changed, 34 deletions(-) diff --git a/driv

[PATCH 0/7] mfd: Fix all W=1 warnings

2016-09-15 Thread Lee Jones
It really is as simple as it sounds! Lee Jones (7): mfd: max8997-irq: 'inline' should be at the beginning of the declaration mfd: ab8500-debugfs: Prevent initialised field from being over-written mfd: db8500-prcmu: Remove unused *prcmu_set_ddr_opp() calls mfd: ab8500-debugfs: Remove ab

[PATCH 4/7] mfd: ab8500-debugfs: Remove ab8500_dump_all_banks_to_mem()

2016-09-15 Thread Lee Jones
Doesn't appear to be used. No call sites exist. Signed-off-by: Lee Jones --- drivers/mfd/ab8500-debugfs.c | 54 1 file changed, 54 deletions(-) diff --git a/drivers/mfd/ab8500-debugfs.c b/drivers/mfd/ab8500-debugfs.c index 6c913c4..b948088 100644 --

Re: [PATCH v13 7/8] clocksource/drivers/arm_arch_timer: Add GTDT support for memory-mapped timer

2016-09-15 Thread Fu Wei
HI On 15 September 2016 at 04:09, wrote: > From: Fu Wei > > The patch add memory-mapped timer register support by using the information > provided by the new GTDT driver of ACPI. > Also refactor original memory-mapped timer dt support for reusing some common > code. > > Signed-off-by: Fu Wei >

Re: [Nbd] [RESEND][PATCH 0/5] nbd improvements

2016-09-15 Thread Wouter Verhelst
Hi, On Fri, Sep 09, 2016 at 10:02:03PM +0200, Wouter Verhelst wrote: > I see some practical problems with this: [...] One more that I didn't think about earlier: A while back, we spent quite some time defining the semantics of the various commands in the face of the NBD_CMD_FLUSH and NBD_CMD_FLA

Re: [PATCH v4 1/4] tpm/tpm_crb: implement tpm crb idle state

2016-09-15 Thread Jarkko Sakkinen
On Thu, Sep 15, 2016 at 10:27:38AM +0300, Tomas Winkler wrote: > The register TPM_CRB_CTRL_REQ_x contains bits goIdle and cmdReady for > SW to indicate that the device can enter or should exit the idle state. > > The legacy ACPI-start (SMI + DMA) based devices do not support these > bits and the i

Re: [PATCH v3 1/4] tpm/tpm_crb: implement tpm crb idle state

2016-09-15 Thread Jarkko Sakkinen
On Thu, Sep 15, 2016 at 08:23:03AM +, Winkler, Tomas wrote: > > > > Subject: Re: [PATCH v3 1/4] tpm/tpm_crb: implement tpm crb idle > > > > state > > > > > > > > On Mon, Sep 12, 2016 at 04:04:18PM +0300, Tomas Winkler wrote: > > > > > The register TPM_CRB_CTRL_REQ_x contains bits goIdle and cmd

Re: [GIT PULL] phy: for 4.9

2016-09-15 Thread Kishon Vijay Abraham I
On Thursday 15 September 2016 04:06 PM, Greg KH wrote: > On Thu, Sep 15, 2016 at 03:52:56PM +0530, Kishon Vijay Abraham I wrote: >> +Chanwoo >> >> Hi Greg, >> >> On Thursday 15 September 2016 02:08 PM, Greg KH wrote: >>> On Wed, Sep 14, 2016 at 01:13:41PM +0530, Kishon Vijay Abraham I wrote:

[PATCH] PM / Domains: Allow holes in genpd_data.domains array

2016-09-15 Thread Tomeu Vizoso
In platforms such as Rockchip's, the array of domains isn't always filled without holes, as which domains are present depend on the particular SoC revision. By allowing holes to be in the array, such SoCs can still use a single set of constants to index the array of power domains. Fixes: 0159ec67

Re: [PATCH V6 2/5] PCI/ACPI: Check platform specific ECAM quirks

2016-09-15 Thread Lorenzo Pieralisi
On Tue, Sep 13, 2016 at 07:38:39PM +0800, Dongdong Liu wrote: [...] > Our host bridge is non ECAM only for the RC bus config space; > for any other bus underneath the root bus we support ECAM access. > > RC config resource with hardcode as DEFINE_RES_MEM(0xb007, SZ_4K), > EP config resource

Re: [GIT PULL] Greybus driver subsystem for 4.9-rc1

2016-09-15 Thread Bryan O'Donoghue
On Thu, 2016-09-15 at 12:17 +0200, Greg KH wrote: > This > > is the first I've heard of timesync having a binding. I can't > imagine > > why it needs one. > > Ah, I'll let Bryan answer that one :) It's possible we could drop the binding. It was needed to describe the register location of the MMIO

Re: 转发: [PATCH] usb: serial: update CH34x driver in drivers/usb/serial

2016-09-15 Thread Johan Hovold
On Thu, Sep 15, 2016 at 11:19:26AM +0100, Aidan Thornton wrote: > On Thu, Sep 15, 2016 at 6:56 AM, Greg KH wrote: > > On Thu, Sep 15, 2016 at 12:03:48AM +0100, Aidan Thornton wrote: > >> > >> It looks like someone by the name of Grigori Goronzy (CCed) had a patch > >> series > >> or four attempti

Re: [PATCH] PM / Domains: Allow holes in genpd_data.domains array

2016-09-15 Thread Jon Hunter
On 15/09/16 11:39, Tomeu Vizoso wrote: > In platforms such as Rockchip's, the array of domains isn't always > filled without holes, as which domains are present depend on the > particular SoC revision. > > By allowing holes to be in the array, such SoCs can still use a single > set of constants t

Re: [GIT PULL] phy: for 4.9

2016-09-15 Thread Kishon Vijay Abraham I
Hi Greg, On Thursday 15 September 2016 04:26 PM, Kishon Vijay Abraham I wrote: > > > On Thursday 15 September 2016 04:06 PM, Greg KH wrote: >> On Thu, Sep 15, 2016 at 03:52:56PM +0530, Kishon Vijay Abraham I wrote: >>> +Chanwoo >>> >>> Hi Greg, >>> >>> On Thursday 15 September 2016 02:08 PM, Gre

Re: [Nbd] [RESEND][PATCH 0/5] nbd improvements

2016-09-15 Thread Alex Bligh
Wouter, Josef, (& Eric) > On 15 Sep 2016, at 11:49, Wouter Verhelst wrote: > > Hi, > > On Fri, Sep 09, 2016 at 10:02:03PM +0200, Wouter Verhelst wrote: >> I see some practical problems with this: > [...] > > One more that I didn't think about earlier: > > A while back, we spent quite some tim

[PATCH 1/2] mfd: wm8994-core: disable regulators before removing them

2016-09-15 Thread Viresh Kumar
The order in which resources were freed in wm8994_device_exit() isn't correct. The regulators are removed before they are disabled. Fix it by reordering code a bit, which makes it exact opposite of wm8994_device_init() as well. Signed-off-by: Viresh Kumar --- drivers/mfd/wm8994-core.c | 2 +- 1

[PATCH 2/2] mfd: wm8994-core: Don't use managed regulator bulk get API

2016-09-15 Thread Viresh Kumar
The kernel WARNs and then crashes today if wm8994_device_init() fails after calling devm_regulator_bulk_get(). That happens because there are multiple devices involved here and the order in which resources are freed isn't correct. The regulators are added as children of wm8994->dev. devm_regulato

Re: [PATCH] coresight: tmc: Cleanup operation mode handling

2016-09-15 Thread Venkatesh Vivekanandan
On Wed, Sep 14, 2016 at 7:23 PM, Suzuki K Poulose wrote: > The mode of operation of the TMC tracked in drvdata->mode is defined > as a local_t type. This is always checked and modified under the > drvdata->spinlock and hence we don't need local_t for it and the > unnecessary synchronisation instru

Re: [GIT PULL] Greybus driver subsystem for 4.9-rc1

2016-09-15 Thread Mark Rutland
Hi, More questions below. Perhaps some of these will be implicitly answered when the linearised patches appear, and I'm happy to wait until then to continue the discussion, as I suspect otherwise we're all likely to end up exasperated. Please do Cc me on those. Regardless, until those appear and

[PATCH v4 0/9] ima: carry the measurement list across kexec

2016-09-15 Thread Mimi Zohar
The TPM PCRs are only reset on a hard reboot. In order to validate a TPM's quote after a soft reboot (eg. kexec -e), the IMA measurement list of the running kernel must be saved and then restored on the subsequent boot, possibly of a different architecture. The existing securityfs binary_runtime_

[PATCH v4 2/9] ima: permit duplicate measurement list entries

2016-09-15 Thread Mimi Zohar
Measurements carried across kexec need to be added to the IMA measurement list, but should not prevent measurements of the newly booted kernel from being added to the measurement list. This patch adds support for allowing duplicate measurements. The "boot_aggregate" measurement entry is the delimi

[PATCH v4 3/9] ima: maintain memory size needed for serializing the measurement list

2016-09-15 Thread Mimi Zohar
In preparation for serializing the binary_runtime_measurements, this patch maintains the amount of memory required. Changelog v3: - include the ima_kexec_hdr size in the binary_runtime_measurement size. Signed-off-by: Mimi Zohar --- security/integrity/ima/Kconfig | 12 + security/in

[PATCH v4 5/9] ima: on soft reboot, save the measurement list

2016-09-15 Thread Mimi Zohar
From: Thiago Jung Bauermann This patch uses the kexec buffer passing mechanism to pass the serialized IMA binary_runtime_measurements to the next kernel. Changelog v4: - Revert the skip_checksum change. Instead calculate the checksum with the measurement list segment, on update validate the exi

[PATCH v4 6/9] ima: store the builtin/custom template definitions in a list

2016-09-15 Thread Mimi Zohar
The builtin and single custom templates are currently stored in an array. In preparation for being able to restore a measurement list containing multiple builtin/custom templates, this patch stores the builtin and custom templates as a linked list. This will permit defining more than one custom t

[PATCH v4 9/9] ima: platform-independent hash value

2016-09-15 Thread Mimi Zohar
From: Andreas Steffen For remote attestion it is important for the ima measurement values to be platform-independent. Therefore integer fields to be hashed must be converted to canonical format. Changelog: - Define canonical format as little endian (Mimi) Signed-off-by: Andreas Steffen Signed-

[PATCH v4 1/9] ima: on soft reboot, restore the measurement list

2016-09-15 Thread Mimi Zohar
The TPM PCRs are only reset on a hard reboot. In order to validate a TPM's quote after a soft reboot (eg. kexec -e), the IMA measurement list of the running kernel must be saved and restored on boot. This patch restores the measurement list. Changelog v2: - redefined ima_kexec_hdr to use types w

[PATCH v4 4/9] ima: serialize the binary_runtime_measurements

2016-09-15 Thread Mimi Zohar
The TPM PCRs are only reset on a hard reboot. In order to validate a TPM's quote after a soft reboot (eg. kexec -e), the IMA measurement list of the running kernel must be saved and restored on boot. This patch serializes the IMA measurement list in the binary_runtime_measurements format. Signed

[PATCH v4 7/9] ima: support restoring multiple template formats

2016-09-15 Thread Mimi Zohar
The configured IMA measurement list template format can be replaced at runtime on the boot command line, including a custom template format. This patch adds support for restoring a measuremement list containing multiple builtin/custom template formats. Signed-off-by: Mimi Zohar --- security/inte

[PATCH v4 8/9] ima: define a canonical binary_runtime_measurements list format

2016-09-15 Thread Mimi Zohar
The IMA binary_runtime_measurements list is currently in platform native format. To allow restoring a measurement list carried across kexec with a different endianness than the targeted kernel, this patch defines little-endian as the canonical format. For big endian systems wanting to save/restor

Re: [Nbd] [RESEND][PATCH 0/5] nbd improvements

2016-09-15 Thread Wouter Verhelst
On Thu, Sep 15, 2016 at 12:09:28PM +0100, Alex Bligh wrote: > Wouter, Josef, (& Eric) > > > On 15 Sep 2016, at 11:49, Wouter Verhelst wrote: > > > > Hi, > > > > On Fri, Sep 09, 2016 at 10:02:03PM +0200, Wouter Verhelst wrote: > >> I see some practical problems with this: > > [...] > > > > One

Re: [PATCH 1/7] mfd: max8997-irq: 'inline' should be at the beginning of the declaration

2016-09-15 Thread Arnd Bergmann
On Thursday, September 15, 2016 11:45:15 AM CEST Lee Jones wrote: > -static const inline struct max8997_irq_data * > +inline static const struct max8997_irq_data * I think this causes another warning, the order should be "stastic inline const struct max8997_irq_data". Arnd

Re: [PATCH 2/7] mfd: ab8500-debugfs: Prevent initialised field from being over-written

2016-09-15 Thread Arnd Bergmann
On Thursday, September 15, 2016 11:45:16 AM CEST Lee Jones wrote: > Due to the lack of parity in the way array fields have been named/ > numbered, a mistake was made where more debug fields were declared > than actually existed. In doing so, 2 fields were added, which > although unclear, were alre

Re: [PATCH 6/7] mfd: omap-usb-host: Return value is not 'const int'

2016-09-15 Thread Arnd Bergmann
On Thursday, September 15, 2016 11:45:20 AM CEST Lee Jones wrote: > Change from 'const int' to just 'int'. > > Cc: > Signed-off-by: Lee Jones > Looks good, I seem to have done the same thing. Also didn't submit, so keep your version. I tend to include the exact warning in the changelog, which

Re: [PATCH 0/7] mfd: Fix all W=1 warnings

2016-09-15 Thread Arnd Bergmann
On Thursday, September 15, 2016 11:45:14 AM CEST Lee Jones wrote: > It really is as simple as it sounds! > I seem to have one that you didn't find. Arnd 8< >From 6c63171eba728fc6d8bc6358884deaccd4cee33d Mon Sep 17 00:00:00 2001 From: Arnd Bergmann Date: Thu, 4 Aug 2016 15:28:51

[PATCH] drm/sti: mark symbols static where possible

2016-09-15 Thread Baoyou Xie
We get 4 warnings when building kernel with W=1: drivers/gpu/drm/sti/sti_gdp.c:476:5: warning: no previous prototype for 'sti_gdp_field_cb' [-Wmissing-prototypes] drivers/gpu/drm/sti/sti_hqvdp.c:786:5: warning: no previous prototype for 'sti_hqvdp_vtg_cb' [-Wmissing-prototypes] drivers/gpu/drm/st

Re: [Nbd] [RESEND][PATCH 0/5] nbd improvements

2016-09-15 Thread Christoph Hellwig
On Thu, Sep 15, 2016 at 12:09:28PM +0100, Alex Bligh wrote: > A more general point is that with multiple queues requests > may be processed in a different order even by those servers that > currently process the requests in strict order, or in something > similar to strict order. The server is perm

Re: [Nbd] [RESEND][PATCH 0/5] nbd improvements

2016-09-15 Thread Christoph Hellwig
On Thu, Sep 15, 2016 at 01:29:36PM +0200, Wouter Verhelst wrote: > Yes, and that is why I was asking about this. If the write barriers > are expected to be shared across connections, we have a problem. If, > however, they are not, then it doesn't matter that the commands may be > processed out of o

Re: DAX mapping detection (was: Re: [PATCH] Fix region lost in /proc/self/smaps)

2016-09-15 Thread Nicholas Piggin
On Thu, 15 Sep 2016 20:32:10 +1000 Dave Chinner wrote: > On Thu, Sep 15, 2016 at 01:49:45PM +1000, Nicholas Piggin wrote: > > On Thu, 15 Sep 2016 12:31:33 +1000 > > Dave Chinner wrote: > > > > > On Wed, Sep 14, 2016 at 08:19:36PM +1000, Nicholas Piggin wrote: > > > > On Wed, 14 Sep 2016 17:

Re: [PATCH v3 00/14] Clock improvement for video playback

2016-09-15 Thread Patrice Chotard
On 09/15/2016 08:57 AM, Patrice Chotard wrote: > > > On 09/14/2016 08:36 PM, Stephen Boyd wrote: >> On 08/29, gabriel.fernan...@st.com wrote: >>> From: Gabriel Fernandez >>> >>> v3: >>> - Rebase to v4.8-rc1 >>> - Tipo fix in st,clkgen-pll.txt >>> - Add Ack of Peter for the series >>> -

Re: [Nbd] [RESEND][PATCH 0/5] nbd improvements

2016-09-15 Thread Alex Bligh
Christoph, > On 15 Sep 2016, at 12:38, Christoph Hellwig wrote: > > On Thu, Sep 15, 2016 at 12:49:35PM +0200, Wouter Verhelst wrote: >> A while back, we spent quite some time defining the semantics of the >> various commands in the face of the NBD_CMD_FLUSH and NBD_CMD_FLAG_FUA >> write barriers

Re: [GIT PULL] Greybus driver subsystem for 4.9-rc1

2016-09-15 Thread Bryan O'Donoghue
On Thu, 2016-09-15 at 12:20 +0100, Mark Rutland wrote: > Hi, > > More questions below. Perhaps some of these will be implicitly > answered > when the linearised patches appear, and I'm happy to wait until then > to > continue the discussion, as I suspect otherwise we're all likely to > end > up ex

Re: [Nbd] [RESEND][PATCH 0/5] nbd improvements

2016-09-15 Thread Alex Bligh
> On 15 Sep 2016, at 12:40, Christoph Hellwig wrote: > > On Thu, Sep 15, 2016 at 01:29:36PM +0200, Wouter Verhelst wrote: >> Yes, and that is why I was asking about this. If the write barriers >> are expected to be shared across connections, we have a problem. If, >> however, they are not, then

Re: [Nbd] [RESEND][PATCH 0/5] nbd improvements

2016-09-15 Thread Christoph Hellwig
On Thu, Sep 15, 2016 at 12:43:35PM +0100, Alex Bligh wrote: > Sure, it's at: > > https://github.com/yoe/nbd/blob/master/doc/proto.md#ordering-of-messages-and-writes > > and that link takes you to the specific section. > > The treatment of FLUSH and FUA is meant to mirror exactly the > linux bloc

Re: [PATCH] leds: Only remove trigger if one is actually configured.

2016-09-15 Thread Jacek Anaszewski
Hi Daniel, On 09/14/2016 11:23 AM, Daniel Romell wrote: In led_trigger_remove(), check if there is a trigger configured before calling led_trigger_set(). This will prevent some unnecessary work, and more importantly stop sending uevent change events when no change has in fact occurred. Signed-o

Re: [PATCH 1/6] leds: gpio: fix an unhandled error case in create_gpio_led

2016-09-15 Thread Jacek Anaszewski
Hi Heiner, On 09/13/2016 08:53 PM, Heiner Kallweit wrote: gpiod_get_value_cansleep returns 0, 1, or an error code. So far errors are not handled and treated the same as 1. Change this to bail out if an error code is returned and remove the double negation. Signed-off-by: Heiner Kallweit --- d

Re: [Nbd] [RESEND][PATCH 0/5] nbd improvements

2016-09-15 Thread Christoph Hellwig
On Thu, Sep 15, 2016 at 12:46:07PM +0100, Alex Bligh wrote: > Essentially NBD does supports FLUSH/FUA like this: > > https://www.kernel.org/doc/Documentation/block/writeback_cache_control.txt > > IE supports the same FLUSH/FUA primitives as other block drivers (AIUI). > > Link to protocol (per l

[PATCHv3 03/41] radix-tree: Add radix_tree_join

2016-09-15 Thread Kirill A. Shutemov
From: Matthew Wilcox This new function allows for the replacement of many smaller entries in the radix tree with one larger multiorder entry. From the point of view of an RCU walker, they may see a mixture of the smaller entries and the large entry during the same walk, but they will never see N

[PATCHv3 14/41] filemap: allocate huge page in page_cache_read(), if allowed

2016-09-15 Thread Kirill A. Shutemov
This patch adds basic functionality to put huge page into page cache. At the moment we only put huge pages into radix-tree if the range covered by the huge page is empty. We ignore shadow entires for now, just remove them from the tree before inserting huge page. Later we can add logic to accumu

[PATCHv3 38/41] ext4: fix SEEK_DATA/SEEK_HOLE for huge pages

2016-09-15 Thread Kirill A. Shutemov
ext4_find_unwritten_pgoff() needs few tweaks to work with huge pages. Mostly trivial page_mapping()/page_to_pgoff() and adjustment to how we find relevant block. Signe-off-by: Kirill A. Shutemov --- fs/ext4/file.c | 18 ++ 1 file changed, 14 insertions(+), 4 deletions(-) diff --

[PATCHv3 08/41] Revert "radix-tree: implement radix_tree_maybe_preload_order()"

2016-09-15 Thread Kirill A. Shutemov
This reverts commit 356e1c23292a4f63cfdf1daf0e0ddada51f32de8. After conversion of huge tmpfs to multi-order entries, we don't need this anymore. Signed-off-by: Kirill A. Shutemov --- include/linux/radix-tree.h | 1 - lib/radix-tree.c | 74 -

[PATCHv3 22/41] thp: do not threat slab pages as huge in hpage_{nr_pages,size,mask}

2016-09-15 Thread Kirill A. Shutemov
Slab pages can be compound, but we shouldn't threat them as THP for pupose of hpage_* helpers, otherwise it would lead to confusing results. For instance, ext4 uses slab pages for journal pages and we shouldn't confuse them with THPs. The easiest way is to exclude them in hpage_* helpers. Signed-

[PATCHv3 06/41] radix-tree: Handle multiorder entries being deleted by replace_clear_tags

2016-09-15 Thread Kirill A. Shutemov
From: Matthew Wilcox radix_tree_replace_clear_tags() can be called with NULL as the replacement value; in this case we need to delete sibling entries which point to the slot. Signed-off-by: Matthew Wilcox Signed-off-by: Kirill A. Shutemov --- lib/radix-tree.c | 11 ++- 1 file changed,

[PATCHv3 34/41] ext4: handle huge pages in ext4_da_write_end()

2016-09-15 Thread Kirill A. Shutemov
Call ext4_da_should_update_i_disksize() for head page with offset relative to head page. Signed-off-by: Kirill A. Shutemov --- fs/ext4/inode.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c index 31560b52eff2..deacd3499ec7 100644 ---

Re: [Nbd] [RESEND][PATCH 0/5] nbd improvements

2016-09-15 Thread Wouter Verhelst
On Thu, Sep 15, 2016 at 04:38:07AM -0700, Christoph Hellwig wrote: > On Thu, Sep 15, 2016 at 12:49:35PM +0200, Wouter Verhelst wrote: > > A while back, we spent quite some time defining the semantics of the > > various commands in the face of the NBD_CMD_FLUSH and NBD_CMD_FLAG_FUA > > write barrier

[PATCHv3 05/41] radix-tree: Add radix_tree_split_preload()

2016-09-15 Thread Kirill A. Shutemov
From: Matthew Wilcox Calculate how many nodes we need to allocate to split an old_order entry into multiple entries, each of size new_order. The test suite checks that we allocated exactly the right number of nodes; neither too many (checked by rtp->nr == 0), nor too few (checked by comparing nr

[PATCHv3 12/41] thp: handle write-protection faults for file THP

2016-09-15 Thread Kirill A. Shutemov
For filesystems that wants to be write-notified (has mkwrite), we will encount write-protection faults for huge PMDs in shared mappings. The easiest way to handle them is to clear the PMD and let it refault as wriable. Signed-off-by: Kirill A. Shutemov --- mm/memory.c | 11 ++- 1 file c

[PATCHv3 16/41] filemap: allocate huge page in pagecache_get_page(), if allowed

2016-09-15 Thread Kirill A. Shutemov
Write path allocate pages using pagecache_get_page(). We should be able to allocate huge pages there, if it's allowed. As usually, fallback to small pages, if failed. Signed-off-by: Kirill A. Shutemov --- mm/filemap.c | 18 -- 1 file changed, 16 insertions(+), 2 deletions(-) dif

Re: [PATCH 7/9] net: ethernet: ti: cpts: calc mult and shift from refclk freq

2016-09-15 Thread Richard Cochran
On Wed, Sep 14, 2016 at 10:26:19PM +0200, Richard Cochran wrote: > On Wed, Sep 14, 2016 at 04:02:29PM +0300, Grygorii Strashko wrote: > > + clocks_calc_mult_shift(&mult, &shift, freq, NSEC_PER_SEC, maxsec); > > + > > + cpts->cc_mult = mult; > > + cpts->cc.mult = mult; > > In order to get goo

Re: [PATCH 1/8] gpio: palmas: fix implicit assumption module.h is present

2016-09-15 Thread Linus Walleij
On Tue, Sep 13, 2016 at 12:16 AM, Paul Gortmaker wrote: > This file is currently getting module.h from a global gpio header > and it will fail to build once we remove module.h from that. > > However, the driver is controlled with the following Kconfig: > > drivers/gpio/Kconfig:config GPIO_PALMAS

[PATCHv3 13/41] truncate: make sure invalidate_mapping_pages() can discard huge pages

2016-09-15 Thread Kirill A. Shutemov
invalidate_inode_page() has expectation about page_count() of the page -- if it's not 2 (one to caller, one to radix-tree), it will not be dropped. That condition almost never met for THPs -- tail pages are pinned to the pagevec. Let's drop them, before calling invalidate_inode_page(). Signed-off

Re: [PATCH 04/26] drm/omap: omap_display_timings: rename x_res to hactive

2016-09-15 Thread Tomi Valkeinen
On 01/09/16 14:22, Peter Ujfalusi wrote: > In preparation to move the stack to use the generic videmode > struct for display timing information. I really don't like the description to depend on the subject =). Tomi signature.asc Description: OpenPGP digital signature

Re: [Nbd] [RESEND][PATCH 0/5] nbd improvements

2016-09-15 Thread Alex Bligh
> On 15 Sep 2016, at 12:46, Christoph Hellwig wrote: > > On Thu, Sep 15, 2016 at 12:43:35PM +0100, Alex Bligh wrote: >> Sure, it's at: >> >> https://github.com/yoe/nbd/blob/master/doc/proto.md#ordering-of-messages-and-writes >> >> and that link takes you to the specific section. >> >> The tre

[PATCHv3 25/41] fs: make block_page_mkwrite() aware about huge pages

2016-09-15 Thread Kirill A. Shutemov
Adjust check on whether part of the page beyond file size and apply compound_head() and page_mapping() where appropriate. Signed-off-by: Kirill A. Shutemov --- fs/buffer.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/fs/buffer.c b/fs/buffer.c index 7f50e5a63670.

[PATCHv3 31/41] ext4: handle huge pages in ext4_page_mkwrite()

2016-09-15 Thread Kirill A. Shutemov
Trivial: remove assumption on page size. Signed-off-by: Kirill A. Shutemov --- fs/ext4/inode.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c index c9c05840f6cc..9ed715ed00e7 100644 --- a/fs/ext4/inode.c +++ b/fs/ext4/inode.c @

[PATCHv3 40/41] mm, fs, ext4: expand use of page_mapping() and page_to_pgoff()

2016-09-15 Thread Kirill A. Shutemov
With huge pages in page cache we see tail pages in more code paths. This patch replaces direct access to struct page fields with macros which can handle tail pages properly. Signed-off-by: Kirill A. Shutemov --- fs/buffer.c | 2 +- fs/ext4/inode.c | 4 ++-- mm/filemap.c| 26

Re: [PATCH 2/8] gpio: sx150x: fix implicit assumption module.h is present

2016-09-15 Thread Linus Walleij
On Tue, Sep 13, 2016 at 12:16 AM, Paul Gortmaker wrote: > This file is currently getting module.h from a global gpio header > and it will faii to build once we remove module.h from that. > > However, the driver is controlled with the following Kconfig: > > drivers/gpio/Kconfig:config GPIO_SX150X

Re: [Nbd] [RESEND][PATCH 0/5] nbd improvements

2016-09-15 Thread Wouter Verhelst
On Thu, Sep 15, 2016 at 04:52:17AM -0700, Christoph Hellwig wrote: > On Thu, Sep 15, 2016 at 12:46:07PM +0100, Alex Bligh wrote: > > Essentially NBD does supports FLUSH/FUA like this: > > > > https://www.kernel.org/doc/Documentation/block/writeback_cache_control.txt > > > > IE supports the same F

Re: [PATCH 3/8] gpio: ts4800: fix implicit assumption module.h is present

2016-09-15 Thread Linus Walleij
On Tue, Sep 13, 2016 at 12:16 AM, Paul Gortmaker wrote: > The Kconfig for this file is: > > drivers/gpio/Kconfig:config GPIO_TS4800 > drivers/gpio/Kconfig: tristate "TS-4800 DIO blocks and compatibles" > > ...but however it does not include module.h -- it in turn gets it from > another header (gp

Re: [PATCH] documentation: fix broken lkml archive links in RCU requirements

2016-09-15 Thread Michael Opdenacker
Richard, Paul, Thank you for your replies! On 09/09/2016 16:33, Paul E. McKenney wrote: On Fri, Sep 09, 2016 at 04:17:14PM +0200, Richard Weinberger wrote: Please don't add lkml.org. It does not use message ids for indexing. With knowing the message id you can query any other archive. e.g. http

[PATCHv3 29/41] ext4: make ext4_mpage_readpages() hugepage-aware

2016-09-15 Thread Kirill A. Shutemov
This patch modifies ext4_mpage_readpages() to deal with huge pages. We read out 2M at once, so we have to alloc (HPAGE_PMD_NR * blocks_per_page) sector_t for that. I'm not entirely happy with kmalloc in this codepath, but don't see any other option. Signed-off-by: Kirill A. Shutemov --- fs/ext4

Re: [PATCH 00/19] [RESEND] Remove STiH415 and STiH416 SoC platform support

2016-09-15 Thread Arnd Bergmann
On Thursday, September 15, 2016 8:01:39 AM CEST Peter Griffin wrote: > > STiH415 I'm sure never shipped. I'm reasonably sure STiH416 didn't > either. These SoCs were considered legacy even when I was at ST > ~3 years ago. > > Also remember these are STB SoC's, so JTAG fuses are blown in > product

Re: [Nbd] [RESEND][PATCH 0/5] nbd improvements

2016-09-15 Thread Christoph Hellwig
On Thu, Sep 15, 2016 at 01:55:14PM +0200, Wouter Verhelst wrote: > Maybe I'm not using the correct terminology here. The point is that > after a FLUSH, the server asserts that all write commands *for which a > reply has already been sent to the client* will also have reached > permanent storage. No

Re: [PATCH 4/8] gpio: altera: fix implicit assumption module.h is present

2016-09-15 Thread Linus Walleij
On Tue, Sep 13, 2016 at 12:16 AM, Paul Gortmaker wrote: > The Kconfig for this file is: > > drivers/gpio/Kconfig:config GPIO_ALTERA > drivers/gpio/Kconfig: tristate "Altera GPIO" > > ...but however it does not include module.h -- it in turn gets it from > another header (gpio/driver.h) and we'd

[PATCHv3 41/41] ext4, vfs: add huge= mount option

2016-09-15 Thread Kirill A. Shutemov
The same four values as in tmpfs case. Encyption code is not yet ready to handle huge page, so we disable huge pages support if the inode has EXT4_INODE_ENCRYPT. Signed-off-by: Kirill A. Shutemov --- fs/ext4/ext4.h | 5 + fs/ext4/inode.c | 26 +- fs/ext4/super.c |

[PATCHv3 37/41] ext4: make EXT4_IOC_MOVE_EXT work with huge pages

2016-09-15 Thread Kirill A. Shutemov
Adjust how we find relevant block within page and how we clear the required part of the page. Signed-off-by: Kirill A. Shutemov --- fs/ext4/move_extent.c | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/fs/ext4/move_extent.c b/fs/ext4/move_extent.c index a920c5d29

Re: [PATCH 6/8] gpio: loongson1: fix implicit assumption module.h is present

2016-09-15 Thread Linus Walleij
On Tue, Sep 13, 2016 at 12:16 AM, Paul Gortmaker wrote: > The Kconfig for this file is: > > drivers/gpio/Kconfig:config GPIO_LOONGSON1 > drivers/gpio/Kconfig: tristate "Loongson1 GPIO support" > > ...but however it does not include module.h -- it in turn gets it from > another header (gpio/driv

[PATCHv3 20/41] mm: make write_cache_pages() work on huge pages

2016-09-15 Thread Kirill A. Shutemov
We writeback whole huge page a time. Let's adjust iteration this way. Signed-off-by: Kirill A. Shutemov --- include/linux/mm.h | 1 + include/linux/pagemap.h | 1 + mm/page-writeback.c | 17 - 3 files changed, 14 insertions(+), 5 deletions(-) diff --git a/include/linu

[PATCHv3 39/41] ext4: make fallocate() operations work with huge pages

2016-09-15 Thread Kirill A. Shutemov
__ext4_block_zero_page_range() adjusted to calculate starting iblock correctry for huge pages. ext4_{collapse,insert}_range() requires page cache invalidation. We need the invalidation to be aligning to huge page border if huge pages are possible in page cache. Signed-off-by: Kirill A. Shutemov

Re: [PATCH 12/26] drm/omap: HDMI5: Use pointer to cfg->v_fc_config.timings in hdmi_core_video_config

2016-09-15 Thread Tomi Valkeinen
On 01/09/16 14:23, Peter Ujfalusi wrote: > By using a pointer to the omap_vode_timings struct we can unwrap lines to s/vode/mode/ Tomi signature.asc Description: OpenPGP digital signature

Re: [PATCH 5/8] gpio: ath79: fix implicit assumption module.h is present

2016-09-15 Thread Linus Walleij
On Tue, Sep 13, 2016 at 12:16 AM, Paul Gortmaker wrote: > The Kconfig for this file is: > > drivers/gpio/Kconfig:config GPIO_ATH79 > drivers/gpio/Kconfig: tristate "Atheros AR71XX/AR724X/AR913X GPIO support" > > ...but however it does not include module.h -- it in turn gets it from > another head

[PATCHv3 27/41] truncate: make invalidate_inode_pages2_range() aware about huge pages

2016-09-15 Thread Kirill A. Shutemov
For huge pages we need to unmap whole range covered by the huge page. Signed-off-by: Kirill A. Shutemov --- mm/truncate.c | 27 +++ 1 file changed, 19 insertions(+), 8 deletions(-) diff --git a/mm/truncate.c b/mm/truncate.c index 9c339e6255f2..6a445278aaaf 100644 --- a/m

[PATCHv3 15/41] filemap: handle huge pages in do_generic_file_read()

2016-09-15 Thread Kirill A. Shutemov
Most of work happans on head page. Only when we need to do copy data to userspace we find relevant subpage. We are still limited by PAGE_SIZE per iteration. Lifting this limitation would require some more work. Signed-off-by: Kirill A. Shutemov --- mm/filemap.c | 5 - 1 file changed, 4 inse

[PATCHv3 24/41] fs: make block_write_{begin,end}() be able to handle huge pages

2016-09-15 Thread Kirill A. Shutemov
It's more or less straight-forward. Most changes are around getting offset/len withing page right and zero out desired part of the page. Signed-off-by: Kirill A. Shutemov --- fs/buffer.c | 53 +++-- 1 file changed, 31 insertions(+), 22 deletions(-

[PATCHv3 18/41] HACK: readahead: alloc huge pages, if allowed

2016-09-15 Thread Kirill A. Shutemov
Most page cache allocation happens via readahead (sync or async), so if we want to have significant number of huge pages in page cache we need to find a ways to allocate them from readahead. Unfortunately, huge pages doesn't fit into current readahead design: 128 max readahead window, assumption o

[PATCHv3 02/41] radix tree test suite: Allow GFP_ATOMIC allocations to fail

2016-09-15 Thread Kirill A. Shutemov
From: Matthew Wilcox In order to test the preload code, it is necessary to fail GFP_ATOMIC allocations, which requires defining GFP_KERNEL and GFP_ATOMIC properly. Remove the obsolete __GFP_WAIT and copy the definitions of the __GFP flags which are used from the kernel include files. We also nee

[PATCHv3 35/41] ext4: make ext4_da_page_release_reservation() aware about huge pages

2016-09-15 Thread Kirill A. Shutemov
For huge pages 'stop' must be within HPAGE_PMD_SIZE. Let's use hpage_size() in the BUG_ON(). We also need to change how we calculate lblk for cluster deallocation. Signed-off-by: Kirill A. Shutemov --- fs/ext4/inode.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/fs/e

[PATCHv3 36/41] ext4: handle writeback with huge pages

2016-09-15 Thread Kirill A. Shutemov
Modify mpage_map_and_submit_buffers() and mpage_release_unused_pages() to deal with huge pages. Mostly result of try-and-error. Critical view would be appriciated. Signed-off-by: Kirill A. Shutemov --- fs/ext4/inode.c | 61 - 1 file change

Re: [Nbd] [RESEND][PATCH 0/5] nbd improvements

2016-09-15 Thread Alex Bligh
> On 15 Sep 2016, at 12:52, Christoph Hellwig wrote: > > On Thu, Sep 15, 2016 at 12:46:07PM +0100, Alex Bligh wrote: >> Essentially NBD does supports FLUSH/FUA like this: >> >> https://www.kernel.org/doc/Documentation/block/writeback_cache_control.txt >> >> IE supports the same FLUSH/FUA primi

Re: [PATCH] documentation: fix broken lkml archive links in RCU requirements

2016-09-15 Thread Michael Opdenacker
Hi Steve, On 09/09/2016 16:34, Steven Rostedt wrote: Correct, we avoid any links to lkml.org at all costs. Simple do a s,/g/,/r/, and all your links should work. For example, using the above mentioned link: https://lkml.kernel.org/r/20100319013024.GA28456@Krystal Works as expected. I conf

[PATCHv3 33/41] ext4: make ext4_block_write_begin() aware about huge pages

2016-09-15 Thread Kirill A. Shutemov
It simply matches changes to __block_write_begin_int(). Signed-off-by: Kirill A. Shutemov --- fs/ext4/inode.c | 24 1 file changed, 16 insertions(+), 8 deletions(-) diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c index a07c055d8e78..31560b52eff2 100644 --- a/fs/ext4/inod

Re: [PATCH 8/8] gpio: don't include module.h in shared driver header

2016-09-15 Thread Linus Walleij
On Tue, Sep 13, 2016 at 12:16 AM, Paul Gortmaker wrote: > Most shared headers in include/linux don't need to know what the > internals of a struct module are; all they care about is that it > is a struct and hence they may require a pointer to one. > > The advantage in this is that module.h is in

[PATCHv3 32/41] ext4: handle huge pages in __ext4_block_zero_page_range()

2016-09-15 Thread Kirill A. Shutemov
As the function handles zeroing range only within one block, the required changes are trivial, just remove assuption on page size. Signed-off-by: Kirill A. Shutemov --- fs/ext4/inode.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c in

<    1   2   3   4   5   6   7   8   9   >