Re: [PATCH v5 15/15] scsi: ufs: add wrapper for retrying sending query attribute

2015-10-27 Thread Gilad Broner
Reviewed-by: Gilad Broner > Sometimes queries from the device might return a failure so it is > recommended to retry sending the query, before giving up. > This change adds a wrapper to retry sending a query attribute, > in cases where we need to wait longer, before we continue, > or before

Re: [PATCH v5 10/15] scsi: ufs: fix error recovery after the hibern8 exit failure

2015-10-27 Thread Gilad Broner
Reviewed-by: Gilad Broner > Hibern8 exit can be called from 3 different context: > - ufshcd_hibern8_exit_work > - ufshcd_ungate_work > - runtime/system resume > > If hibern8 exit fails for some reason then we try to bring the link to > active state by link startup but this recovery

Re: [PATCH v5 12/15] scsi: ufs: reduce the interrupts for power mode change requests

2015-10-27 Thread Gilad Broner
Reviewed-by: Gilad Broner > DME commands such as Hibern8 enter/exit and gear switch generate 2 > completion interrupts, one for confirmation that command is received > by local UniPro and 2nd one is the final confirmation after communication > with remote UniPro. Currently both of these

[PATCH v12 2/5] gennvm: Generic NVM manager

2015-10-27 Thread Matias Bjørling
The implementation for Open-Channel SSDs is divided into media management and targets. This patch implements a generic media manager for open-channel SSDs. After a media manager has been initialized, single or multiple targets can be instantiated with the media managed as the backend.

Re: [PATCH v5 08/15] scsi: ufs: add retries to dme_peer get and set attribute

2015-10-27 Thread Gilad Broner
Reviewed-by: Gilad Broner > The dme_peer get/set attribute commands are prone to errors, therefore > we add three retries for the UIC command sending. > Error code returned from ufshcd_send_uic_cmd() is checked, and unless > it was successful or the retries have finished, another command will be

[PATCH v12 3/5] rrpc: Round-robin sector target with cost-based gc

2015-10-27 Thread Matias Bjørling
This target allows an Open-Channel SSD to be exposed asas a block device. It implements a round-robin approach for sector allocation, together with a greedy cost-based garbage collector. Signed-off-by: Matias Bjørling --- drivers/lightnvm/Kconfig |7 + drivers/lightnvm/Makefile |1 +

Re: [PATCH V4 0/8] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

2015-10-27 Thread Rafael J. Wysocki
On Tuesday, October 27, 2015 09:52:50 AM Bjorn Helgaas wrote: > Hi Suravee, > > On Wed, Oct 21, 2015 at 08:52:03AM -0700, Suravee Suthikulpanit wrote: > > This patch series adds support to setup DMA coherency for PCI device using > > the ACPI _CCA attribute. According to the ACPI spec, the _CCA

Re: [PATCH v5 09/15] scsi: ufs: add retries for hibern8 enter

2015-10-27 Thread Gilad Broner
Reviewed-by: Gilad Broner > If hibern8 enter command fails then UFS link state may be unknown which > may result into timeout of all the commands issued after failure. > > This change does 2 things (for pre-defined number of retry counts) after > hibern8 enter failure: > 1. Recovers the UFS link

Re: [PATCH v5 06/15] scsi: ufs: avoid exception event handler racing with PM callbacks

2015-10-27 Thread Gilad Broner
Reviewed-by: Gilad Broner > If device raises the exception event in the response to the commands > sent during the runtime/system PM callbacks, exception event handler > might run in parallel with PM callbacks and may see unclocked register > accesses. This change fixes this issue by not

Re: character driver - poll() timeout

2015-10-27 Thread Muni Sekhar
On Tue, Oct 27, 2015 at 1:41 PM, Clemens Ladisch wrote: > Muni Sekhar wrote: >> Is it possible to print the timeout value in character driver poll() API? > > No. Your driver's poll callback never waits. > > Why do you think you need this value? I need to find out when exactly driver's poll

Re: [PATCH v5 05/15] scsi: ufs: increase fDeviceInit query response timeout

2015-10-27 Thread Gilad Broner
Reviewed-by: Gilad Broner > fDeviceInit query response time for some devices is too long that default > query request timeout of 100ms may not be enough. Experiments show that > fDeviceInit response sometimes takes 500ms so to be on safer side this > change sets the timeout to 600ms. Without

[PATCH v12 5/5] nvme: LightNVM support

2015-10-27 Thread Matias Bjørling
The first generation of Open-Channel SSDs is based on NVMe. The NVMe driver is extended with support for the LightNVM command set. Detection is made through PCI IDs. Current supported devices are the qemu nvme simulator and CNEX Labs Westlake SSD. The qemu nvme enables support through vendor

[PATCH v12 0/5] Support for Open-Channel SSDs

2015-10-27 Thread Matias Bjørling
This patchset implements support for Open-Channel SSDs. Applies against Jens' for-4.4/drivers and is available through the lkml_v12 branch at: https://github.com/OpenChannelSSD/linux Changes since v11: - Add support for group identifiers. - Fixed kbuild error on nvme_nvm_unregister.

Re: [PATCH] staging: board: Set PM domain before probe

2015-10-27 Thread Geert Uytterhoeven
On Tue, Oct 27, 2015 at 3:27 PM, Tomeu Vizoso wrote: > PM domains shouldn't be modified after a device is probed, so set it > before device registration to be sure of that. > > In the future the PM domain pointer will be set through a setter that > will WARN if the device has been probed already.

Re: [PATCH v5 02/15] scsi: ufs: clear fields UTRD, UPIU req and rsp before new transfers

2015-10-27 Thread Gilad Broner
Looks OK. Reviewed-by: Gilad Broner > Some of the data structures (like response UPIU) and/or its elements > (unused fields) should be cleared before sending out the respective > command to UFS device. > > This change clears the UPIU response data structure for query commands > and NOP command

Re: [PATCH v5 04/15] scsi: ufs: clear outstanding_request bit in case query timeout

2015-10-27 Thread Gilad Broner
Reviewed-by: Gilad Broner > When sending a query to the device returns with a timeout error, > we clear the corresponding bit in the DOORBELL register but > we don't clear the outstanding_request field as we should. > This patch fixes this bug. > > Signed-off-by: Yaniv Gardi > > --- >

Re: Inline hunt results for 4.3.0-rc1

2015-10-27 Thread Hannes Frederic Sowa
Hello, On Tue, Oct 27, 2015, at 15:32, Denys Vlasenko wrote: > I have created a set of semi-automated scripts which look for > large inlines in the kernel. > > Recently I taught it to even generate "git format-patch" patches > (unfortunately, only for inlines in *.c files, not *.h), > and here

[RFD] Functional dependencies between devices

2015-10-27 Thread Rafael J. Wysocki
Hi All, As discussed in the recent "On-demand device probing" thread and in a Kernel Summit session earlier today, there is a problem with handling cases where functional dependencies between devices are involved. What I mean by a "functional dependency" is when the driver of device B needs both

RE: [PATCH v4] ARM: configs: Add Freescale LS1021A defconfig

2015-10-27 Thread Huan Wang
> On Thursday 15 October 2015 02:11:57 Huan Wang wrote: > > > On Wednesday 14 October 2015 10:18:47 Huan Wang wrote: > > > > > On Thursday 24 September 2015 07:27:10 Huan Wang wrote: > > > > > > > On Fri, Sep 18, 2015 at 4:38 AM, Huan Wang > wrote: > > > > > > Any suggestion? Thanks. > > > > > >

Re: [PATCH v2 02/32] devicetree: bindings: scsi: HiSi SAS

2015-10-27 Thread zhangfei
On 10/27/2015 10:39 PM, Mark Rutland wrote: On Tue, Oct 27, 2015 at 01:09:15PM +, John Garry wrote: On 26/10/2015 14:45, Mark Rutland wrote: On Mon, Oct 26, 2015 at 10:14:33PM +0800, John Garry wrote: Add devicetree bindings for HiSilicon SAS driver. Signed-off-by: John Garry ---

Re: [PATCH v5 01/15] scsi: ufs: clear UTRD, UPIU req and rsp before new transfers

2015-10-27 Thread Gilad Broner
Looks OK. Reviewed-by: Gilad Broner > Clear the UFS data structures before sending new request. > > The SCSI command is sent to the device within the UFS UPIU request. > As part of the transfer UPIU preparation, the SCSI command is copied > to the UPIU structure according to the SCSI command

Re: [PATCH V4 0/8] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

2015-10-27 Thread Bjorn Helgaas
Hi Suravee, On Wed, Oct 21, 2015 at 08:52:03AM -0700, Suravee Suthikulpanit wrote: > This patch series adds support to setup DMA coherency for PCI device using > the ACPI _CCA attribute. According to the ACPI spec, the _CCA attribute > is required for ARM64. Therefore, this patch is a pre-req for

[PATCH 5/6] net: dsa: Do not reschedule polling if driver removed

2015-10-27 Thread Neil Armstrong
Do not reschedule the delayed work used for polling when the driver is removed, by testing the 'poll_link_needed' flag. Avoids this crash: dsa dsa ethmv2 (unregistering): Link is Down device eth1 left promiscuous mode Unable to handle kernel paging request at virtual address bacc5cf6 ...

[PATCH 0/6] Further fix for dsa unbinding

2015-10-27 Thread Neil Armstrong
This serie fixes further issues for DSA dynamic unbinding. Frode Isaksen's patches make usage of delayed work and fixes kernel crashes when dsa is unbind. The other patches are simple fixes to permit cleanup and avoid netdev related crashes. Frode Isaksen (3): net: dsa: Use delayed work instead

Re: [PATCH v6 03/22] of/platform: Point to struct device from device node

2015-10-27 Thread Tomeu Vizoso
On 24 October 2015 at 15:57, Rafael J. Wysocki wrote: > On Thursday, October 22, 2015 03:01:45 PM Tomeu Vizoso wrote: >> On 22 October 2015 at 03:02, Rafael J. Wysocki wrote: >> > On Monday, September 21, 2015 04:02:43 PM Tomeu Vizoso wrote: >> >> When adding platform and AMBA devices, set the

[PATCH 3/6] net: dsa: cleanup resources upon module removal

2015-10-27 Thread Neil Armstrong
Make sure that we unassign the master_netdev dsa_ptr to make the packet processing go through the regular Ethernet receive path. Suggested-by: Florian Fainelli Signed-off-by: Neil Armstrong --- net/dsa/dsa.c | 8 1 file changed, 8 insertions(+) diff --git a/net/dsa/dsa.c

[PATCH 6/6] net: phy: Stop 'phy-state-machine' and 'phy_change' work on remove

2015-10-27 Thread Neil Armstrong
Avoids: Unable to handle kernel NULL pointer dereference at virtual address 0064 Workqueue: events_power_efficient phy_state_machine PC is at phy_state_machine+0x28/0x480 Signed-off-by: Frode Isaksen Signed-off-by: Neil Armstrong --- drivers/net/phy/phy_device.c | 6 ++ 1 file

[PATCH 1/6] net: dsa: Use delayed work instead of timer+work for polling

2015-10-27 Thread Neil Armstrong
Simplifies the code and avoids a crash when removing the module: dsa dsa ethmv2 (unregistering): Link is Down device eth1 left promiscuous mode Unable to handle kernel paging request at virtual address bacc5cf6 ... (run_timer_softirq) from [] (__do_softirq+0xcc/0x320) (__do_softirq) from []

Re: [PATCHv5 00/52] perf stat: Add scripting support

2015-10-27 Thread Arnaldo Carvalho de Melo
Em Tue, Oct 27, 2015 at 02:24:03PM +, Liang, Kan escreveu: > > > > > > Em Sun, Oct 25, 2015 at 03:51:16PM +0100, Jiri Olsa escreveu: > > > hi, > > > sending another version of stat scripting. > > > > > > v5 changes: > > > - several patches from v4 already taken > > > - using u16 for cpu

[PATCH 4/6] net: dsa: Add missing master netdev dev_put() calls

2015-10-27 Thread Neil Armstrong
Upon probe failure or unbinding, add missing dev_put() calls. Signed-off-by: Neil Armstrong --- net/dsa/dsa.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/net/dsa/dsa.c b/net/dsa/dsa.c index acbf854..9240a46 100644 --- a/net/dsa/dsa.c +++ b/net/dsa/dsa.c @@ -931,8

[PATCH 2/6] net: dsa: allow switch drivers to cleanup their resources

2015-10-27 Thread Neil Armstrong
Some switch drivers might request interrupts, remap register ranges, allow such drivers to implement a "remove" callback doing just that. Suggested-by: Florian Fainelli Signed-off-by: Neil Armstrong --- include/net/dsa.h | 1 + net/dsa/dsa.c | 4 2 files changed, 5 insertions(+) diff

Re: [PATCH V4 8/8] PCI: ACPI: Add support for PCI device DMA coherency

2015-10-27 Thread Bjorn Helgaas
On Wed, Oct 21, 2015 at 08:52:11AM -0700, Suravee Suthikulpanit wrote: > This patch adds support for setting up PCI device DMA coherency from > ACPI _CCA object that should normally be specified in the DSDT node > of its PCI host bridge. > > Signed-off-by: Suravee Suthikulpanit > CC: Bjorn

Re: [v3, 3/3] MIPS: VDSO: Add implementations of gettimeofday() and clock_gettime()

2015-10-27 Thread Ralf Baechle
On Thu, Oct 22, 2015 at 06:41:30PM -0700, Leonid Yegoshin wrote: > You can not use R4K CP0_count in SMP (multicore) without core-specific > adjustment. > After first power-saving with core clock off or core down the values in > CP0_count > in different cores are absolutely different. > > Until

[PATCH 02/11] parport: remove trailing white space

2015-10-27 Thread Sudip Mukherjee
From: Sudip Mukherjee Trailing white space is not accepted in kernel coding style. Remove them. Signed-off-by: Sudip Mukherjee --- drivers/parport/share.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/parport/share.c b/drivers/parport/share.c index

Re: [PATCH v6 04/22] of: add function to allow probing a device from a OF node

2015-10-27 Thread Tomeu Vizoso
On 26 October 2015 at 09:16, Geert Uytterhoeven wrote: > On Mon, Sep 21, 2015 at 4:02 PM, Tomeu Vizoso > wrote: >> Walks the OF tree up and finds the closest ancestor that has a struct >> device associated with it, probing it if isn't bound to a driver yet. >> >> The above should ensure that the

[PATCH 04/11] parport: fix coding style

2015-10-27 Thread Sudip Mukherjee
From: Sudip Mukherjee The multi-line comments were not according to the kernel coding style. Signed-off-by: Sudip Mukherjee --- drivers/parport/share.c | 69 +++-- 1 file changed, 44 insertions(+), 25 deletions(-) diff --git

[PATCH 01/11] parport: fix a trivial typo

2015-10-27 Thread Sudip Mukherjee
From: Geliang Tang s/regsiter/register/ Signed-off-by: Geliang Tang Signed-off-by: Sudip Mukherjee --- drivers/parport/share.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/parport/share.c b/drivers/parport/share.c index 5ce5ef2..8931696 100644 ---

[PATCH 03/11] parport: EXPORT_SYMBOL should follow function

2015-10-27 Thread Sudip Mukherjee
From: Sudip Mukherjee All symbols were exported at the end of the file but they are supposed to be exported just after the function. And checkpatch was complaining about it. Signed-off-by: Sudip Mukherjee --- drivers/parport/share.c | 29 + 1 file changed, 13

[PATCH 06/11] parport: quoted strings should not be split

2015-10-27 Thread Sudip Mukherjee
From: Sudip Mukherjee user visible strings should not be split as that affects the ability to grep. Signed-off-by: Sudip Mukherjee --- drivers/parport/share.c | 18 -- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/drivers/parport/share.c

[PATCH 10/11] parport: remove unneeded space

2015-10-27 Thread Sudip Mukherjee
From: Sudip Mukherjee checkpatch complains that space is prohibited between function name and open parenthesis '('. Signed-off-by: Sudip Mukherjee --- drivers/parport/share.c | 94 - 1 file changed, 47 insertions(+), 47 deletions(-) diff --git

[PATCH 09/11] parport: change style of NULL comparison

2015-10-27 Thread Sudip Mukherjee
From: Sudip Mukherjee checkpatch was complaining about NULL comparisons. Signed-off-by: Sudip Mukherjee --- drivers/parport/share.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/parport/share.c b/drivers/parport/share.c index 441333b..ccd7df4 100644 ---

Re: [PATCH V5 1/1] usb:serial add Fintek F81532/534 driver

2015-10-27 Thread Johan Hovold
On Thu, Oct 22, 2015 at 10:25:26AM +0800, Peter Hung wrote: > Hi Johan, > > Johan Hovold 於 2015/9/14 下午 09:33 寫道: > > On Tue, Jul 21, 2015 at 09:58:19AM +0800, Peter Hung wrote: > > >>4. RS422 Mode > >> 1. The RTS mode is dont care. > >> 2. Set M2/M1/M0 as 0/0/0 > > >

[PATCH 08/11] parport: remove unnecessary out of memory message

2015-10-27 Thread Sudip Mukherjee
From: Sudip Mukherjee If kmalloc() or kzalloc() fails we will get sufficient messages in the logs, no need to print these extra messages. Signed-off-by: Sudip Mukherjee --- drivers/parport/share.c | 13 +++-- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git

Re: [PATCH v10] gpio: Add GPIO support for the ACCES 104-IDIO-16

2015-10-27 Thread Mika Westerberg
On Tue, Oct 27, 2015 at 03:40:38PM +0100, Simon Guinot wrote: > Should this driver not be moved under the "MFD GPIO expanders" menu ? Maybe. It is still part of the PCH (Platform Controller Hub) as far as I can tell so I would not call it a GPIO expander either ;-) -- To unsubscribe from this

[PATCH 07/11] parport: remove braces

2015-10-27 Thread Sudip Mukherjee
From: Sudip Mukherjee checkpatch was complaining about braces for single statement block. Signed-off-by: Sudip Mukherjee --- drivers/parport/share.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/parport/share.c b/drivers/parport/share.c index

[PATCH 11/11] parport: avoid assignment in if

2015-10-27 Thread Sudip Mukherjee
From: Sudip Mukherjee It is not an usual practise to assign some value to a variable in the if test condition. Signed-off-by: Sudip Mukherjee --- drivers/parport/share.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/parport/share.c b/drivers/parport/share.c

[PATCH 05/11] parport: code indent should use tabs

2015-10-27 Thread Sudip Mukherjee
From: Sudip Mukherjee Code should be indented using tabs and not by space. Signed-off-by: Sudip Mukherjee --- drivers/parport/share.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/parport/share.c b/drivers/parport/share.c index 840c730..388c138 100644 ---

[PATCH] ARM: add v7 LPAE multi-platform defconfig

2015-10-27 Thread Alison Wang
v7 LPAE multi-platform defconfig is based on v7 multi-platform defconfig and adds LPAE support. This defconfig is verified on LS1021A which enables GIANFAR, I2C, WATCHDOG, AUDIO, EDMA and DSPI drivers, etc. Signed-off-by: Alison Wang --- arch/arm/configs/multi_v7_lpae_defconfig | 168

[PATCH 00/11] parport cleanup

2015-10-27 Thread Sudip Mukherjee
Hi Greg, A small series of checkpatch cleanup on parport. This is the first set. Its only cleanup and no other change is done. All the patches are also sent along with this pull request for your convenience. The following changes since commit 32b88194f71d6ae7768a29f87fbba454728273ee: Linux

Re: [PATCH 31/52] perf stat record: Synthesize stat record data

2015-10-27 Thread Arnaldo Carvalho de Melo
Em Sun, Oct 25, 2015 at 03:51:47PM +0100, Jiri Olsa escreveu: > Synthesizing needed stat record data for report/script: > - cpu/thread maps > - stat config After this it gets a bit better, but then I expected that to specify an event I would be able to use: [root@zoo linux]# perf stat record

[PATCH v11 0/4] Allow USB devices to remain runtime-suspended when sleeping

2015-10-27 Thread Tomeu Vizoso
Hi, this is v11 of an attempt to make it easier for devices to remain in runtime PM when the system goes to sleep, mainly to reduce the time spent resuming devices. For this, we interpret the absence of all PM callback implementations as it being safe to do direct_complete, so their ancestors

[PATCH v11 1/4] device core: add device_is_bound()

2015-10-27 Thread Tomeu Vizoso
Adds a function that tells whether a device is already bound to a driver. This is needed to warn when there is an attempt to change the PM domain of a device that has finished probing already. The reason why we want to enforce that is because in the general case that can cause problems and also

Re: [PATCH V4 1/8] acpi: Honor ACPI _CCA attribute setting

2015-10-27 Thread Bjorn Helgaas
>From "git log --oneline include/acpi/acpi_bus.h", I see the convention is to capitalize "ACPI:" in the summary (also applies to other patches in this series). On Wed, Oct 21, 2015 at 08:52:04AM -0700, Suravee Suthikulpanit wrote: > From: Jeremy Linton > > ACPI configurations can now mark

Re: [PATCH v10 3/4] PM / sleep: Go direct_complete if driver has no callbacks

2015-10-27 Thread Tomeu Vizoso
On 26 October 2015 at 13:56, Daniel Kurtz wrote: > Hi Tomeu, > > On Wed, Oct 21, 2015 at 11:34 PM, Tomeu Vizoso > wrote: >> If a suitable prepare callback cannot be found for a given device and >> its driver has no PM callbacks at all, assume that it can go direct to >> complete when the system

[PATCH v11 4/4] USB / PM: Allow USB devices to remain runtime-suspended when sleeping

2015-10-27 Thread Tomeu Vizoso
Have dev_pm_ops.prepare return 1 for USB devices and ports so that USB devices can remain runtime-suspended when the system goes to a sleep state, if their wakeup state is correct and they have runtime PM enabled. Signed-off-by: Tomeu Vizoso Reviewed-by: Ulf Hansson Acked-by: Alan Stern ---

[PATCH v11 2/4] PM / Domains: add setter for dev.pm_domain

2015-10-27 Thread Tomeu Vizoso
Adds a function that sets the pointer to dev_pm_domain in struct device and that warns if the device has already finished probing. The reason why we want to enforce that is because in the general case that can cause problems and also that we can simplify code quite a bit if we can always assume

Re: [PATCH v10] gpio: Add GPIO support for the ACCES 104-IDIO-16

2015-10-27 Thread Simon Guinot
On Tue, Oct 27, 2015 at 04:23:43PM +0200, Mika Westerberg wrote: > On Tue, Oct 27, 2015 at 11:06:58AM +0100, Linus Walleij wrote: > > > +menu "ISA GPIO drivers" > > > + > > > +config GPIO_104_IDIO_16 > > > + tristate "ACCES 104-IDIO-16 GPIO support" > > > + depends on X86 > > > +

Re: [PATCH] w1: w1_process() is not freezable kthread

2015-10-27 Thread Evgeniy Polyakov
Hi 26.10.2015, 08:53, "Jiri Kosina" : > From: Jiri Kosina > > w1_process() calls try_to_freeze(), but the thread doesn't mark itself > freezable through set_freezable(), so the try_to_freeze() call is useless. I believe it is better to mark it freezable, what do you think? Its task is useless

Re: [PATCH v2 02/32] devicetree: bindings: scsi: HiSi SAS

2015-10-27 Thread Mark Rutland
On Tue, Oct 27, 2015 at 01:09:15PM +, John Garry wrote: > On 26/10/2015 14:45, Mark Rutland wrote: > >On Mon, Oct 26, 2015 at 10:14:33PM +0800, John Garry wrote: > >>Add devicetree bindings for HiSilicon SAS driver. > >> > >>Signed-off-by: John Garry > >>--- > >>

[PATCH v11 3/4] PM / sleep: Go direct_complete if driver has no callbacks

2015-10-27 Thread Tomeu Vizoso
If a suitable prepare callback cannot be found for a given device and its driver has no PM callbacks at all, assume that it can go direct to complete when the system goes to sleep. The reason for this is that there's lots of devices in a system that do no PM at all and there's no reason for them

Re: [PATCH V4 7/8] PCI: OF: Move of_pci_dma_configure() to pci_dma_configure()

2015-10-27 Thread Bjorn Helgaas
Hi Suravee, On Wed, Oct 21, 2015 at 08:52:10AM -0700, Suravee Suthikulpanit wrote: > This patch move of_pci_dma_configure() to a more generic > pci_dma_configure(), which can be extended by non-OF code (e.g. ACPI). > -void of_pci_dma_configure(struct pci_dev *pci_dev) > -{ > - struct device

Re: [PATCH] namei: permit linking with CAP_FOWNER in userns

2015-10-27 Thread Seth Forshee
On Tue, Oct 20, 2015 at 04:09:19PM +0200, Dirk Steinmetz wrote: > Attempting to hardlink to an unsafe file (e.g. a setuid binary) from > within an unprivileged user namespace fails, even if CAP_FOWNER is held > within the namespace. This may cause various failures, such as a gentoo > installation

Re: [PATCH v10 1/6] PM / Domains: prepare for multiple states

2015-10-27 Thread Ulf Hansson
On 20 October 2015 at 15:26, wrote: > From: Axel Haslam > > prepare generic power domain init function parameters to accept a pointer > to the states structure that describes the possible states that a power > domain can enter. > > Also, as most platforms are not initializing states or

Inline hunt results for 4.3.0-rc1

2015-10-27 Thread Denys Vlasenko
Hi, I have created a set of semi-automated scripts which look for large inlines in the kernel. Recently I taught it to even generate "git format-patch" patches (unfortunately, only for inlines in *.c files, not *.h), and here are they for 4.3.0-rc1 - i.e. current Linus tree. Submitting 300+

Re: [PATCH v4 4/5] regulator: tps65912: Add regulator driver for the TPS65912 PMIC

2015-10-27 Thread Andrew F. Davis
On 10/26/2015 07:16 PM, Mark Brown wrote: On Mon, Oct 26, 2015 at 10:47:41AM -0500, Andrew F. Davis wrote: On 10/25/2015 07:43 PM, Mark Brown wrote: .of_compatible = "mediatek,mt6397-regulator", This is in the MFD, this is not used in actual systems. Not sure what you mean by "actual

[PATCH] staging: board: Set PM domain before probe

2015-10-27 Thread Tomeu Vizoso
PM domains shouldn't be modified after a device is probed, so set it before device registration to be sure of that. In the future the PM domain pointer will be set through a setter that will WARN if the device has been probed already. Signed-off-by: Tomeu Vizoso Cc: Geert Uytterhoeven ---

Re: [PATCH 29/52] perf stat record: Add record command

2015-10-27 Thread Arnaldo Carvalho de Melo
Em Sun, Oct 25, 2015 at 03:51:45PM +0100, Jiri Olsa escreveu: > Add 'perf stat record' command support. It creates simple > (header only) perf.data file ATM. Huh? Couldn't we have the tools providing a sensible message at this point? [root@zoo linux]# rm -f perf.data [root@zoo linux]# perf stat

Re: [PATCH v12 0/6] Altera PCIe host controller driver with MSI support

2015-10-27 Thread Bjorn Helgaas
Hi Ley, On Fri, Oct 23, 2015 at 06:27:09PM +0800, Ley Foon Tan wrote: > This is the 12th version of patch set to add support for Altera PCIe host > controller with MSI feature on Altera FPGA device families. This patchset > mainly resovle the comments from Bjorn. > > > This patchset is based on

RE: [PATCHv5 00/52] perf stat: Add scripting support

2015-10-27 Thread Liang, Kan
> > Em Sun, Oct 25, 2015 at 03:51:16PM +0100, Jiri Olsa escreveu: > > hi, > > sending another version of stat scripting. > > > > v5 changes: > > - several patches from v4 already taken > > - using u16 for cpu number in cpu_map_event > > - renamed PERF_RECORD_HEADER_ATTR_UPDATE to >

Re: [PATCH v10] gpio: Add GPIO support for the ACCES 104-IDIO-16

2015-10-27 Thread Mika Westerberg
On Tue, Oct 27, 2015 at 11:06:58AM +0100, Linus Walleij wrote: > > +menu "ISA GPIO drivers" > > + > > +config GPIO_104_IDIO_16 > > + tristate "ACCES 104-IDIO-16 GPIO support" > > + depends on X86 > > + help > > + Enables GPIO support for the ACCES 104-IDIO-16 family. > >

Re: [PATCH 2/2] serial: 8250_ingenic: allow to be independent of SERIAL_8250_CONSOLE

2015-10-27 Thread Peter Hurley
On 10/23/2015 09:31 AM, Masahiro Yamada wrote: > This UART driver should not depend on the console. They should be > orthogonal. > > Surround the earlycon code with CONFIG_SERIAL_EARLYCON conditional > and rip off "depends on SERIAL_8250_CONSOLE". Reviewed-by: Peter Hurley PS - While

Re: [PATCH -rt] Revert "net: use synchronize_rcu_expedited()"

2015-10-27 Thread Eric Dumazet
On Tue, 2015-10-27 at 07:31 -0500, Josh Cartwright wrote: > Okay, yes, I like the first suggestion better as well, I've included a > patch below that does just that. I hope you don't mind me turning it > into a Suggested-by :). > > Thanks for taking a look! > Josh > @@ -6969,7 +6969,7 @@

Re: [PATCH 1/2] serial: 8250_ingenic: delete redundant "select SERIAL_EARLYCON"

2015-10-27 Thread Peter Hurley
On 10/23/2015 09:31 AM, Masahiro Yamada wrote: > SERIAL_8250_INGENIC depends on SERIAL_8250_CONSOLE, which already > selects SERIAL_EARLYCON. > > This line is redundant. Reviewed-by: Peter Hurley -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a

Re: [PATCH v4 0/7] USB MIDI Gadget improvements and bug fixes

2015-10-27 Thread Felipe Ferreri Tonello
Hi Balbi, On 26/10/15 16:55, Felipe F. Tonello wrote: > Patch 7 has changes on how to transmit IN USB requests. It implements a FIFO > of pre-allocated usb requests and uses then as needed, instead of allocating > then on demand. This is my initial implementation and is open for > suggestions and

Re: [PATCH 11/52] perf tools: Add stat config event

2015-10-27 Thread Arnaldo Carvalho de Melo
Em Tue, Oct 27, 2015 at 02:51:54PM +0100, Jiri Olsa escreveu: > On Tue, Oct 27, 2015 at 10:44:40AM -0300, Arnaldo Carvalho de Melo wrote: > > > I think we could easily add record specific event for this > > > once it's needed.. there's plenty of free numbers in user > > > events area ;-) > >

Re: [PATCH 1/4 v2] staging: ipath: ipath_driver: Use setup_timer

2015-10-27 Thread Muhammad Falak R Wani
On October 27, 2015 4:40:42 PM GMT+05:30, Dan Carpenter wrote: >On Tue, Oct 27, 2015 at 11:45:18AM +0200, Leon Romanovsky wrote: >> On Tue, Oct 27, 2015 at 11:19 AM, Dan Carpenter >> wrote: >> > On Sun, Oct 25, 2015 at 01:21:11PM +0200, Leon Romanovsky wrote: >> >> On Sun, Oct 25, 2015 at

Re: [PATCH] iommu/amd: Fix amd_iommu_detect() (does not fix any issues).

2015-10-27 Thread Konrad Rzeszutek Wilk
On Tue, Oct 27, 2015 at 09:53:30AM +0900, Jerome Glisse wrote: > On Tue, Oct 27, 2015 at 09:47:48AM +0900, Jerome Glisse wrote: > > On Mon, Oct 26, 2015 at 12:07:17PM -0400, Konrad Rzeszutek Wilk wrote: > > > On Mon, Aug 31, 2015 at 06:13:03PM -0400, j.gli...@gmail.com wrote: > > > > From: Jérôme

Re: [PATCH v2 5/5] ACPI / processor_idle: Add support for Low Power Idle(LPI) states

2015-10-27 Thread Sudeep Holla
Hi Prashanth, On 27/10/15 00:21, Prakash, Prashanth wrote: Hi Sudeep, While testing with these patches everything looked fine except that we are flattening all the LPI states even if it is disabled. I added a simple check to fix it. Please let me know if you have any feedback on the same.

Re: [PATCH 2/7 v2] powerpc/dma-mapping: override dma_get_page_shift

2015-10-27 Thread Busch, Keith
On Tue, Oct 27, 2015 at 05:02:16PM +1100, Alexey Kardashevskiy wrote: > >+unsigned long dma_get_page_shift(struct device *dev) > >+{ > >+struct iommu_table *tbl = get_iommu_table_base(dev); > >+if (tbl) > >+return tbl->it_page_shift; > > > All PCI devices have this

[PATCH v2 1/3] lib/string_helpers: change blk_size to u32 for string_get_size() interface

2015-10-27 Thread Vitaly Kuznetsov
string_get_size() can't really handle huge block sizes, especially blk_size > U32_MAX but string_get_size() interface states the opposite. Change blk_size from u64 to u32 to reflect the reality. Signed-off-by: Vitaly Kuznetsov --- include/linux/string_helpers.h | 2 +- lib/string_helpers.c

[PATCH v2 2/3] lib/string_helpers.c: don't lose precision in string_get_size()

2015-10-27 Thread Vitaly Kuznetsov
string_get_size() loses precision when there is a remainder for blk_size / divisor[units] and size is big enough. E.g string_get_size(8192, 4096, STRING_UNITS_10, ...) returns "32.7 MB" while it is supposed to return "33.5 MB". For some artificial inputs the result can be ridiculously wrong, e.g.

[PATCH v2 3/3] lib/test-string_helpers.c: add string_get_size() tests

2015-10-27 Thread Vitaly Kuznetsov
Add a couple of simple tests for string_get_size(). Signed-off-by: Vitaly Kuznetsov --- Changes since v1: - Make test_string_get_size_one() check both STRING_UNITS_2 and STRING_UNITS_10 in one shot [Rasmus Villemoes] - Check U64_MAX instead of some-very-big-number. ---

[PATCH v2 0/3] lib/string_helpers: fix precision issues and introduce tests

2015-10-27 Thread Vitaly Kuznetsov
Linux always lies about your storage size when it has 4k sectors and its size is big enough. E.g. a device with 8192 4k sectors will be reported as "32.7 MB/32 MiB" while "33.5 MB/32 MiB" is expected. This series is supposed to fix the issue by fixing calculation precision in string_get_size() for

Re: [PATCH v2 0/4] x86: sigcontext fixes, again

2015-10-27 Thread Stas Sergeev
27.10.2015 03:52, Andy Lutomirski пишет: > On Mon, Oct 26, 2015 at 4:45 AM, Stas Sergeev wrote: >> 26.10.2015 04:25, Andy Lutomirski пишет: >>> This is take 2 at fixing x86 64-bit signals wrt SS. After a lot of >>> thought, this is not controlled by any flags -- I would much prefer >>> to avoid

Re: [PATCH 2/2] arm64: validate the delta of cycle_now and cycle_last

2015-10-27 Thread Mark Rutland
On Tue, Oct 27, 2015 at 09:21:13PM +0800, Yang Yingliang wrote: > In multi-core system, if the clock is not sync perfectly, it > will make cycle_last that recorded by CPU-A is a little more > than cycle_now that read by CPU-B. If that is happening, that sounds like a hardware and/or firmware bug.

Re: [alsa-devel] [PATCH V2 02/10] ASoC: img: Add driver for I2S input controller

2015-10-27 Thread Damien Horsley
On 23/10/15 23:57, Mark Brown wrote: > On Thu, Oct 22, 2015 at 08:09:38PM +0100, Damien Horsley wrote: >> On 19/10/15 18:47, Mark Brown wrote: >>> On Mon, Oct 12, 2015 at 01:40:29PM +0100, Damien Horsley wrote: > >>> The APIs here all seem a bit odd - for example the enable API taking a >>>

Re: [PATCH v2 1/2] serial: 8250_early: do not save and restore IER in write callback

2015-10-27 Thread Peter Hurley
Hi Masahiro, On 10/24/2015 12:17 AM, Masahiro Yamada wrote: > The IER has already been masked in early_serial8250_setup(), there is > no reason to save and restore it every time early_serial8250_write() > is called. > > Signed-off-by: Masahiro Yamada > --- > >

Re: [PATCH] keys, trusted: select TPM2 hash algorithm

2015-10-27 Thread Mimi Zohar
On Tue, 2015-10-27 at 12:42 +0200, Jarkko Sakkinen wrote: > n Mon, Oct 26, 2015 at 07:44:39AM +0200, Jarkko Sakkinen wrote: > > On Sun, Oct 25, 2015 at 03:21:31PM -0400, Mimi Zohar wrote: > > > On Sat, 2015-10-24 at 15:42 +0300, Jarkko Sakkinen wrote: > > > > Added 'hashalg=' option for selecting

Re: [PATCH v3] serial: support 16-bit register interface for console

2015-10-27 Thread Peter Hurley
Hi Masahiro, On 10/26/2015 12:11 AM, Masahiro Yamada wrote: > Currently, 8-bit (MMIO) and 32-bit (MMIO32) register interfaces are > supported for the 8250 console, but the 16-bit (MMIO16) is not. > The 8250 UART device on my board is connected to a 16-bit bus > and my main motivation is to use

Re: [PATCH 11/52] perf tools: Add stat config event

2015-10-27 Thread Jiri Olsa
On Tue, Oct 27, 2015 at 10:44:40AM -0300, Arnaldo Carvalho de Melo wrote: > Em Tue, Oct 27, 2015 at 02:30:43PM +0100, Jiri Olsa escreveu: > > On Tue, Oct 27, 2015 at 10:16:05AM -0300, Arnaldo Carvalho de Melo wrote: > > > Em Sun, Oct 25, 2015 at 03:51:27PM +0100, Jiri Olsa escreveu: > > > > Adding

Re: [PATCH 11/52] perf tools: Add stat config event

2015-10-27 Thread Arnaldo Carvalho de Melo
Em Tue, Oct 27, 2015 at 02:30:43PM +0100, Jiri Olsa escreveu: > On Tue, Oct 27, 2015 at 10:16:05AM -0300, Arnaldo Carvalho de Melo wrote: > > Em Sun, Oct 25, 2015 at 03:51:27PM +0100, Jiri Olsa escreveu: > > > Adding stat config event to pass/store stat config data, > > > so report tools

Re: [PATCH v2] spi: Setup the master controller driver before setting the chipselect

2015-10-27 Thread Grygorii Strashko
On 10/27/2015 03:11 PM, Grygorii Strashko wrote: On 10/27/2015 02:22 PM, Franklin S Cooper Jr. wrote: On 10/16/2015 10:55 AM, Andy Shevchenko wrote: On Fri, Oct 16, 2015 at 6:47 PM, Franklin S Cooper Jr. wrote: On 10/16/2015 10:45 AM, Andy Shevchenko wrote: On Fri, Oct 16, 2015 at 6:29

Re: [PATCH 5/8] mm: memcontrol: account socket memory on unified hierarchy

2015-10-27 Thread David Miller
From: Michal Hocko Date: Tue, 27 Oct 2015 13:26:47 +0100 > On Mon 26-10-15 12:56:19, Johannes Weiner wrote: > [...] >> Or any other combination of pick-and-choose consumers. But >> honestly, nowadays all our paths are lockless, and the counting is an >> atomic-add-return with a per-cpu batch

Re: [PATCH 11/52] perf tools: Add stat config event

2015-10-27 Thread Jiri Olsa
On Tue, Oct 27, 2015 at 10:16:05AM -0300, Arnaldo Carvalho de Melo wrote: > Em Sun, Oct 25, 2015 at 03:51:27PM +0100, Jiri Olsa escreveu: > > Adding stat config event to pass/store stat config data, > > so report tools (report/script) know how to interpret > > stat data. > > > > The config data

Re: [PATCH v5 5/6] iommu/mediatek: Add mt8173 IOMMU driver

2015-10-27 Thread Robin Murphy
On 09/10/15 03:23, Yong Wu wrote: [...] +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include Nit: ordering? +#include +#include "io-pgtable.h" [...] +struct mtk_iommu_data { + void __iomem

Re: [PATCH v5 4/6] memory: mediatek: Add SMI driver

2015-10-27 Thread Robin Murphy
On 09/10/15 03:23, Yong Wu wrote: [...] +static int mtk_smi_probe(struct platform_device *pdev) +{ + struct device *dev = >dev; + struct mtk_smi_data *smidata; + int ret; + + if (!dev->pm_domain) + return -EPROBE_DEFER; + + smidata = devm_kzalloc(dev,

[PATCH 1/2] clocksource: replace cycle_last validation with an equal way

2015-10-27 Thread Yang Yingliang
Mask the cycle values before subtraction. So we can use this validation while the clocksource mask is not 64-bits. Signed-off-by: Yang Yingliang Cc: Thomas Gleixner --- kernel/time/timekeeping_internal.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 0/2] validate the delta of cycle_now and cycle_last on arm64

2015-10-27 Thread Yang Yingliang
In multi-core system, if the clock is not sync perfectly, it will make cycle_last that recorded by CPU-A is a little more than cycle_now that read by CPU-B. With the negative result, hrtimer_update_base() return a huge and wrong time. It leads to the cpu can not finish the while loop in

[PATCH 2/2] arm64: validate the delta of cycle_now and cycle_last

2015-10-27 Thread Yang Yingliang
In multi-core system, if the clock is not sync perfectly, it will make cycle_last that recorded by CPU-A is a little more than cycle_now that read by CPU-B. With the negative result, hrtimer_update_base() return a huge and wrong time. It leads to the cpu can not finish the while loop in

Re: [PATCH 11/52] perf tools: Add stat config event

2015-10-27 Thread Arnaldo Carvalho de Melo
Em Sun, Oct 25, 2015 at 03:51:27PM +0100, Jiri Olsa escreveu: > Adding stat config event to pass/store stat config data, > so report tools (report/script) know how to interpret > stat data. > > The config data are stored in 'tag|value' way to allow > easy extension and backward compatibility. I

<    5   6   7   8   9   10   11   12   13   14   >