Re: md/raid1: Improve another size determination in setup_conf()

2016-10-10 Thread Jes Sorensen
SF Markus Elfring writes: Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer. >>> >>> Isn't this pure matter of taste? >>> Some

Re: btrfs_direct_IO oops

2016-10-10 Thread Dave Jones
On Mon, Oct 10, 2016 at 04:43:57AM +0100, Al Viro wrote: > Very interesting. Could you slap something like > diff --git a/lib/iov_iter.c b/lib/iov_iter.c > index 0ce3411..1ef00e7 100644 > --- a/lib/iov_iter.c > +++ b/lib/iov_iter.c > @@ -682,8 +682,9 @@ static void pipe_advance(struct

Re: [PATCH 24/54] md/raid1: Improve another size determination in setup_conf()

2016-10-10 Thread Jes Sorensen
Dan Carpenter writes: > On Thu, Oct 06, 2016 at 11:29:20AM +0200, Richard Weinberger wrote: >> On Thu, Oct 6, 2016 at 11:22 AM, SF Markus Elfring >> wrote: >> > From: Markus Elfring >> > Date: Tue, 4 Oct

Re: [PATCH v4 10/10] ARM: sunxi: Enable sun8i-emac driver on multi_v7_defconfig

2016-10-10 Thread Jean-Francois Moine
On Mon, 10 Oct 2016 14:35:11 +0200 LABBE Corentin wrote: > On Mon, Oct 10, 2016 at 02:30:46PM +0200, Maxime Ripard wrote: > > On Fri, Oct 07, 2016 at 10:25:57AM +0200, Corentin Labbe wrote: > > > Enable the sun8i-emac driver in the multi_v7 default configuration > > >

Re: md/raid1: Improve another size determination in setup_conf()

2016-10-10 Thread Jes Sorensen
SF Markus Elfring writes: but patches that just fix coding style are a bad thing >>> >>> When you find such a change opportunity so "bad", are there any >>> circumstances left over where you would dare to touch the corresponding >>> source code line. >> >>

Re: [PATCHv2] ceph: Fix error handling in ceph_read_iter

2016-10-10 Thread Yan, Zheng
> On 10 Oct 2016, at 20:56, Nikolay Borisov wrote: > > In case __ceph_do_getattr returns an error and the retry_op in > ceph_read_iter is not READ_INLINE, then it's possible to invoke > __free_page on a page which is NULL, this naturally leads to a crash. > This can happen

Re: [PATCH v3 1/2] Input: synaptics-rmi4 - factor out functions from probe

2016-10-10 Thread Benjamin Tissoires
On Sep 20 2016 or thereabouts, Nick Dyer wrote: > Signed-off-by: Nick Dyer > --- > drivers/input/rmi4/rmi_driver.c | 139 > +--- > 1 file changed, 86 insertions(+), 53 deletions(-) > > diff --git a/drivers/input/rmi4/rmi_driver.c

Re: [PATCHv2] ceph: Fix error handling in ceph_read_iter

2016-10-10 Thread Yan, Zheng
> On 10 Oct 2016, at 21:13, Nikolay Borisov wrote: > > > > On 10/10/2016 04:11 PM, Yan, Zheng wrote: >> >>> On 10 Oct 2016, at 20:56, Nikolay Borisov wrote: >>> >>> In case __ceph_do_getattr returns an error and the retry_op in >>> ceph_read_iter is not

Re: Oops on Power8 (was Re: [PATCH v2 1/7] workqueue: make workqueue available early during boot)

2016-10-10 Thread Tejun Heo
On Mon, Oct 10, 2016 at 09:02:53AM -0400, Tejun Heo wrote: > Hello, Michael. > > On Mon, Oct 10, 2016 at 09:22:55PM +1100, Michael Ellerman wrote: > > This patch seems to be causing one of my Power8 boxes not to boot. > > > > Specifically commit 3347fa092821 ("workqueue: make workqueue available

CONFIG_DEBUG_TEST_DRIVER_REMOVE causes unremovable drivers to bind devices twice

2016-10-10 Thread Laszlo Ersek
Hi, Greg asked me to stick to email with this bug report, so I'm reposting the original kernel bugzilla report to personal addresses, and lkml. Thanks, Laszlo https://bugzilla.kernel.org/show_bug.cgi?id=177021 Bug ID: 177021 Summary: [driver core]

Re: [PATCH 2/3] dt-bindings: mtd: add DT binding for s3c2410 flash controller

2016-10-10 Thread Rob Herring
On Wed, Oct 05, 2016 at 08:46:56PM -0300, Sergio Prado wrote: > Adds the device tree bindings description for Samsung S3C2410 and > compatible NAND flash controller. > > Signed-off-by: Sergio Prado > --- > .../devicetree/bindings/mtd/samsung-s3c2410.txt| 57 >

Re: [PATCH v2 2/4] power: add axp20x-battery driver

2016-10-10 Thread Maxime Ripard
Adding the real Quentin address On Mon, Oct 10, 2016 at 02:57:26PM +0200, Maxime Ripard wrote: > Hi Icenowy, > > On Sun, Oct 09, 2016 at 02:27:12PM +0800, Icenowy Zheng wrote: > > + case POWER_SUPPLY_PROP_CURRENT_NOW: > > + ret = regmap_read(power->regmap, AXP20X_BATT_DISCHRG_I_H, );

Re: [PATCH v3 1/4] net: phy: dp83867: Add documentation for optional impedance control

2016-10-10 Thread Rob Herring
On Thu, Oct 06, 2016 at 10:43:52AM +0530, Mugunthan V N wrote: > Add documention of ti,impedance-control which can be used to Needs updating. > correct MAC impedance mismatch using phy extended registers. > > Signed-off-by: Mugunthan V N > --- >

Re: [PATCH] perf powerpc: Don't call perf_event_disable from atomic context

2016-10-10 Thread Will Deacon
On Tue, Oct 04, 2016 at 09:06:18AM +0200, Peter Zijlstra wrote: > On Tue, Oct 04, 2016 at 03:29:33PM +1100, Michael Ellerman wrote: > > Peter Zijlstra writes: > > > So it would be good to also explain why PPC needs this in the first > > > place. > > > > Unfortunately I

Re: [PATCHv3] hwmon: Add tc654 driver

2016-10-10 Thread Guenter Roeck
On 10/09/2016 03:12 PM, Chris Packham wrote: Add support for the tc654 and tc655 fan controllers from Microchip. http://ww1.microchip.com/downloads/en/DeviceDoc/20001734C.pdf Signed-off-by: Chris Packham --- Changes in v3: - typofix in documentation - add

Re: md/raid1: Improve another size determination in setup_conf()

2016-10-10 Thread SF Markus Elfring
>> How do you value compliance with coding styles? > > The Linux Coding Style is not a law, How serious can such guidelines become for software developers? > nor is it at all perfect. I got a similar impression. But are there enough items where a mostly clear guidance is specified? > You

Re: Oops on Power8 (was Re: [PATCH v2 1/7] workqueue: make workqueue available early during boot)

2016-10-10 Thread Balbir Singh
On 10/10/16 23:53, Tejun Heo wrote: > On Mon, Oct 10, 2016 at 10:17:16PM +1100, Balbir Singh wrote: >> rest_init() >> { >> ... >> kernel_thread(kernel_init, NULL, CLONE_FS); >> numa_default_policy(); >> pid = kernel_thread(kthreadd, NULL, CLONE_FS | CLONE_FILES); >>

Re: dm: free io_barrier after blk_cleanup_queue call

2016-10-10 Thread Mike Snitzer
On Mon, Oct 10 2016 at 8:35am -0400, Tahsin Erdogan wrote: > dm_old_request_fn() has paths that access md->io_barrier. The party > destroying io_barrier should ensure that no future execution > of dm_old_request_fn() is possible. Move destruction to below >

Re: [PATCH v7 1/6] perf annotate: Add cross arch annotate support

2016-10-10 Thread Ravi Bangoria
Hi Arnaldo, Sorry for little late replies, I was off last week. Please find my comments. On Wednesday 05 October 2016 04:49 PM, Arnaldo Carvalho de Melo wrote: > Em Wed, Sep 21, 2016 at 09:17:51PM +0530, Ravi Bangoria escreveu: >> Change current data structures and function to enable cross arch

Re: [PATCHv2] ceph: Fix error handling in ceph_read_iter

2016-10-10 Thread Nikolay Borisov
On 10/10/2016 04:11 PM, Yan, Zheng wrote: > >> On 10 Oct 2016, at 20:56, Nikolay Borisov wrote: >> >> In case __ceph_do_getattr returns an error and the retry_op in >> ceph_read_iter is not READ_INLINE, then it's possible to invoke >> __free_page on a page which is NULL, this

Re: [PATCH v11 2/4] drm: Add API for capturing frame CRCs

2016-10-10 Thread Emil Velikov
On 6 October 2016 at 16:21, Tomeu Vizoso wrote: > Adds files and directories to debugfs for controlling and reading frame > CRCs, per CRTC: > > dri/0/crtc-0/crc > dri/0/crtc-0/crc/control > dri/0/crtc-0/crc/data > > Drivers can implement the set_crc_source callback()

Re: [PATCH v7 3/6] perf annotate: Show raw form for jump instruction with indirect target

2016-10-10 Thread Ravi Bangoria
On Wednesday 05 October 2016 04:57 PM, Arnaldo Carvalho de Melo wrote: > Em Wed, Sep 21, 2016 at 09:17:53PM +0530, Ravi Bangoria escreveu: >> For jump instructions that does not include target address as direct >> operand, use raw value for that. This is needed for certain powerpc > "use raw

Re: [PATCH] kbuild: honor '-s' option for tools/*

2016-10-10 Thread Joe Perches
On Mon, 2016-10-10 at 14:36 +0200, Arnd Bergmann wrote: > This changes all three of the above to behave just like Kbuild does, > and print no output with 'make -s' regardless of the version of that > tool, but otherwise behaves as before. In case of > tools/scripts/Makefile.include, I decided to

Re: CONFIG_DEBUG_TEST_DRIVER_REMOVE causes unremovable drivers to bind devices twice

2016-10-10 Thread Rob Herring
On Mon, Oct 10, 2016 at 8:17 AM, Laszlo Ersek wrote: > Hi, > > Greg asked me to stick to email with this bug report, so I'm reposting > the original kernel bugzilla report to personal addresses, and lkml. > > Thanks, > Laszlo > >

Re: [PATCH v4 2/2] iommu/exynos: Add proper runtime pm support

2016-10-10 Thread Marek Szyprowski
Hi Luis On 2016-10-06 19:37, Luis R. Rodriguez wrote: On Thu, Sep 29, 2016 at 10:12:31AM +0200, Marek Szyprowski wrote: This patch uses recently introduced device links to track the runtime pm state of the master's device. This way each SYSMMU controller is runtime activated when its master's

Re: [PATCH v7 4/6] perf annotate: Support jump instruction with target as second operand

2016-10-10 Thread Ravi Bangoria
On Wednesday 05 October 2016 04:58 PM, Arnaldo Carvalho de Melo wrote: > Em Wed, Sep 21, 2016 at 09:17:54PM +0530, Ravi Bangoria escreveu: >> Current perf is not able to parse jump instruction when second operand >> contains target address. Arch like powerpc has such instructions. For >>

Re: [PATCH V1 06/10] Documentation: dt: input: temperature driver

2016-10-10 Thread Rob Herring
On Thu, Oct 06, 2016 at 09:43:30AM +0100, Steve Twiss wrote: > From: Steve Twiss The subject is wrong. > > Add binding information for DA9061 onkey. > > This patch updates the compatible string "dlg,da9061-onkey" to support > DA9061, removes ther reference to

Re: [PATCH v7 5/6] perf annotate: Fix jump target outside of function address range

2016-10-10 Thread Ravi Bangoria
On Wednesday 05 October 2016 05:01 PM, Arnaldo Carvalho de Melo wrote: > Em Wed, Sep 21, 2016 at 09:17:55PM +0530, Ravi Bangoria escreveu: >> If jump target is outside of function range, perf is not handling it >> correctly. Especially when target address is lesser than function start >>

Re: [RFC PATCH 2/3] PM / Domains: Add support for devices with multiple domains

2016-10-10 Thread Jon Hunter
On 07/10/16 10:14, Kevin Hilman wrote: > Jon Hunter writes: > >> Some devices may require more than one PM domain to operate and this is >> not currently by the PM domain framework. Furthermore, the current Linux >> 'device' structure only allows devices to be associated

Re: [PATCH v1 1/2] ARM: dts: add rockchip PX3 Evaluation board

2016-10-10 Thread Andy Yan
Hi Heiko: On 2016年10月10日 17:54, Heiko Stuebner wrote: Hi Andreas, Andy, Am Dienstag, 13. September 2016, 14:14:01 CEST schrieb Andreas Färber: Hi Andy, This patch didn't make it to linux-rockchip list somehow... Not sure why I'm CC'ed, I don't have access to such a board to check, so just a

Re: [PATCH v11 2/4] drm: Add API for capturing frame CRCs

2016-10-10 Thread Tomeu Vizoso
Adding Benjamin Gaignard to CC in case he wants to comment on the usage of the registration functions, as suggested by Daniel Vetter. Regards, Tomeu On 6 October 2016 at 17:21, Tomeu Vizoso wrote: > Adds files and directories to debugfs for controlling and reading

Re: [PATCH] Reorganize STM32 clocks in order to prepare them for PLLI2S and PLLSAI

2016-10-10 Thread Alexandre Torgue
Hi Radoslaw, I add Gabriel in the discussion. Gabriel is updating PLL management for STM32F429. Regards Alex On 10/10/2016 12:31 PM, Daniel Thompson wrote: On 10/10/16 10:56, Radosław Pietrzyk wrote: Hi, all plls have the same clock parent which is after a main divider. Currently the

Re: [RFC][PATCH 4/4] futex: Rewrite FUTEX_UNLOCK_PI

2016-10-10 Thread Peter Zijlstra
On Mon, Oct 10, 2016 at 12:17:48PM +0200, Thomas Gleixner wrote: > There is another problem with all that racing against fixup_owner() > resp. fixup_pi_state_owner(). > > I fear, we need to rethink this whole locking/protection scheme from > scratch. So for pi_state (ie, the

[PATCH RESEND v3] dma: omap-dma: add support for pause of non-cyclic transfers

2016-10-10 Thread Vignesh R
From: Sebastian Andrzej Siewior This DMA driver is used by 8250-omap on DRA7-evm. There is one requirement that is to pause a transfer. This is currently used on the RX side. It is possible that the UART HW aborted the RX (UART's RX-timeout) but the DMA controller starts

Re: [PATCH 2/2] intel_pmc_core: avoid boot time warning for !CONFIG_DEBUGFS_FS

2016-10-10 Thread Greg Kroah-Hartman
On Mon, Oct 10, 2016 at 01:12:58PM +0200, Arnd Bergmann wrote: > While looking at a patch that introduced a compile-time warning > "‘pmc_core_dev_state_get’ defined but not used" (I sent a patch > for debugfs to fix it), I noticed that the same patch caused > it in intel_pmc_core also introduced a

Re: md/raid1: Improve another size determination in setup_conf()

2016-10-10 Thread SF Markus Elfring
> I am ignoring Markus patches It's a pity that you chose such a reaction. > and have told him that he should focus on bug fixes. I find that I suggest to improve something. Could you admit a few times that I found a "bug" you care also about at other source code places? > These patches

Re: [PATCH] sched/fair: Do not decay new task load on first enqueue

2016-10-10 Thread Vincent Guittot
On 10 October 2016 at 12:01, Matt Fleming wrote: > On Sun, 09 Oct, at 11:39:27AM, Wanpeng Li wrote: >> >> The difference between this patch and Peterz's is your patch have a >> delta since activate_task()->enqueue_task() does do update_rq_clock(), >> so why don't have

Re: [PATCH v4 03/10] ARM: sun8i: dt: Add DT bindings documentation for Allwinner sun8i-emac

2016-10-10 Thread Maxime Ripard
On Fri, Oct 07, 2016 at 10:25:50AM +0200, Corentin Labbe wrote: > This patch adds documentation for Device-Tree bindings for the > Allwinner sun8i-emac driver. > > Signed-off-by: Corentin Labbe > --- > .../bindings/net/allwinner,sun8i-emac.txt | 70 >

Re: [PATCH v4 10/10] ARM: sunxi: Enable sun8i-emac driver on multi_v7_defconfig

2016-10-10 Thread Maxime Ripard
On Fri, Oct 07, 2016 at 10:25:57AM +0200, Corentin Labbe wrote: > Enable the sun8i-emac driver in the multi_v7 default configuration > > Signed-off-by: Corentin Labbe > --- > arch/arm/configs/multi_v7_defconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git

Re: [PATCH v4 04/10] ARM: dts: sun8i-h3: Add dt node for the syscon control module

2016-10-10 Thread Maxime Ripard
Hi, On Fri, Oct 07, 2016 at 10:25:51AM +0200, Corentin Labbe wrote: > This patch add the dt node for the syscon register present on the > Allwinner H3. > > Only two register are present in this syscon and the only one useful is > the one dedicated to EMAC clock. > > Signed-off-by: Corentin

[PATCH] ASoC: Intel: haswell depends on sst-firmware

2016-10-10 Thread Arnd Bergmann
The Intel Haswell audio support fails to link if CONFIG_SND_SOC_INTEL_SST_FIRMWARE is disabled: sst-haswell-dsp.c: undefined reference to `sst_mem_block_register' sst-haswell-dsp.c: undefined reference to `sst_mem_block_unregister_all' sst-haswell-dsp.c: undefined reference to

Re: igb driver can cause cache invalidation of non-owned memory?

2016-10-10 Thread Nikita Yushchenko
>> All the data has been synced > > Non-synced data is write done by CPU executing upper layers of network stack, Upper layers shall never get area that is still dma_map()ed and will be dma_unmap()ed in future. But with igb, this is exactly what happens.

Re: [PATCH v4 10/10] ARM: sunxi: Enable sun8i-emac driver on multi_v7_defconfig

2016-10-10 Thread LABBE Corentin
On Mon, Oct 10, 2016 at 02:30:46PM +0200, Maxime Ripard wrote: > On Fri, Oct 07, 2016 at 10:25:57AM +0200, Corentin Labbe wrote: > > Enable the sun8i-emac driver in the multi_v7 default configuration > > > > Signed-off-by: Corentin Labbe > > --- > >

[PATCH] tlan: avoid unused label with PCI=n

2016-10-10 Thread Arnd Bergmann
While build testing with randconfig on x86, I ran into this warning that appears to have been around forever drivers/net/ethernet/ti/tlan.c: In function ‘tlan_probe1’: drivers/net/ethernet/ti/tlan.c:614:1: error: label ‘err_out’ defined but not used [-Werror=unused-label] This can be trivially

[PATCH] dm: free io_barrier after blk_cleanup_queue call

2016-10-10 Thread Tahsin Erdogan
dm_old_request_fn() has paths that access md->io_barrier. The party destroying io_barrier should ensure that no future execution of dm_old_request_fn() is possible. Move destruction to below blk_cleanup_queue() to ensure this. Signed-off-by: Tahsin Erdogan --- drivers/md/dm.c

[PATCH] gpio: mockup: add sysfs dependency

2016-10-10 Thread Arnd Bergmann
Building the gpio-mockup driver without SYSFS results in a harmless Kconfig warning: warning: (GPIO_MOCKUP) selects GPIO_SYSFS which has unmet direct dependencies (GPIOLIB && SYSFS) We can easily avoid that warning by adding a dependency on SYSFS. Fixes: 0f98dd1b27d2 ("gpio/mockup: add virtual

Re: [PATCH] mmc: core, annotate cmd_hdr as __le32

2016-10-10 Thread Ulf Hansson
+ stable On 3 October 2016 at 10:58, Jiri Slaby wrote: > Commit f68381a70b (mmc: block: fix packed command header endianness) > correctly fixed endianness handling of packed_cmd_hdr in > mmc_blk_packed_hdr_wrq_prep. > > But now, sparse complains about incorrect types: >

[PATCH] ceph: Fix error handling in ceph_read_iter

2016-10-10 Thread Nikolay Borisov
In case __ceph_do_getattr returns an error and the retry_op in ceph_read_iter is not READ_INLINE, then it's possible to invoke __free_page on a page which is NULL, this naturally leads to a crash. This can happen when, for example, a process waiting on a MDS reply receives sigterm. Fix this by

Re: [PATCH 1/2] mmc: block: add missing header dependencies

2016-10-10 Thread Ulf Hansson
On 30 September 2016 at 03:37, Baoyou Xie wrote: > We get 1 warning when building kernel with W=1: > drivers/mmc/card/block.c:2147:5: warning: no previous prototype for > 'mmc_blk_issue_rq' [-Wmissing-prototypes] > > In fact, this function is declared in

[PATCH] kbuild: honor '-s' option for tools/*

2016-10-10 Thread Arnd Bergmann
When building with 'make -s' on x86, we always see the output for descending into the tools/objtool directory when CONFIG_STACK_VALIDATION is set, but we should not see any output. There are three related problems causing this: * we override the MAKEFLAGS variable when entering tools, keeping

Re: ppc64 qemu test failure since commit f9aa67142 ("powerpc/64s: Consolidate Alignment 0x600 interrupt")

2016-10-10 Thread Guenter Roeck
Hi Michael, On 10/09/2016 11:00 PM, Michael Ellerman wrote: Guenter Roeck writes: Nicholas, some of my qemu tests for ppc64 started failing on mainline (and -next). You can find a test log at

Re: [PATCH v4 04/10] ARM: dts: sun8i-h3: Add dt node for the syscon control module

2016-10-10 Thread Jean-Francois Moine
On Mon, 10 Oct 2016 14:31:51 +0200 Maxime Ripard wrote: > Hi, > > On Fri, Oct 07, 2016 at 10:25:51AM +0200, Corentin Labbe wrote: > > This patch add the dt node for the syscon register present on the > > Allwinner H3. > > > > Only two register are present in

Re: Oops on Power8 (was Re: [PATCH v2 1/7] workqueue: make workqueue available early during boot)

2016-10-10 Thread Tejun Heo
On Mon, Oct 10, 2016 at 10:17:16PM +1100, Balbir Singh wrote: > rest_init() > { > ... > kernel_thread(kernel_init, NULL, CLONE_FS); > numa_default_policy(); > pid = kernel_thread(kthreadd, NULL, CLONE_FS | CLONE_FILES); > rcu_read_lock(); > kthreadd_task =

Re: [PATCH 4.8 00/10] 4.8.1-stable review

2016-10-10 Thread Guenter Roeck
On 10/10/2016 02:38 AM, Mark Brown wrote: On Sat, Oct 08, 2016 at 10:22:41AM -0700, Guenter Roeck wrote: On 10/08/2016 09:11 AM, Kevin Hilman wrote: vexpress-v2p-ca15-tc1: 1 failed lab vexpress-v2p-ca15_a7: 1 failed lab These builds are known broken an QEMU need to be

[PATCHv2] ceph: Fix error handling in ceph_read_iter

2016-10-10 Thread Nikolay Borisov
In case __ceph_do_getattr returns an error and the retry_op in ceph_read_iter is not READ_INLINE, then it's possible to invoke __free_page on a page which is NULL, this naturally leads to a crash. This can happen when, for example, a process waiting on a MDS reply receives sigterm. Fix this by

Re: [RESEND PATCH v2 3/3] dt-bindings: reset: oxnas: Update for OX820

2016-10-10 Thread Rob Herring
On Wed, Oct 05, 2016 at 05:27:10PM +0200, Neil Armstrong wrote: > Add new compatible string for reset and sys-ctrl for the Oxford > Semiconductor OX820 Support. > Drop the OX810SE indices since they moved in a dedicated include file. > > Signed-off-by: Neil Armstrong >

[PATCH v5 4/5] PM / runtime: Use device links

2016-10-10 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Modify the runtime PM framework to use device links to ensure that supplier devices will not be suspended if any of their consumer devices are active. The idea is to reference count suppliers on the consumer's resume and drop references to

[Rebase][PATCH v5 5/5] PM / runtime: Optimize the use of device links

2016-10-10 Thread Rafael J. Wysocki
From: Rafael J. Wysocki If the device has no links to suppliers that should be used for runtime PM (links with DEVICE_LINK_PM_RUNTIME set), there is no reason to walk the list of suppliers for that device during runtime suspend and resume. Add a simple mechanism to

[PATCH v5 2/5] driver core: Functional dependencies tracking support

2016-10-10 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Currently, there is a problem with taking functional dependencies between devices into account. What I mean by a "functional dependency" is when the driver of device B needs device A to be functional and (generally) its driver to be present in

[PATCH v5 3/5] PM / sleep: Make async suspend/resume of devices use device links

2016-10-10 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Make the device suspend/resume part of the core system suspend/resume code use device links to ensure that supplier and consumer devices will be suspended and resumed in the right order in case of async suspend/resume. The idea, roughly, is to

[Resend][PATCH v5 1/5] driver core: Add a wrapper around __device_release_driver()

2016-10-10 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Add an internal wrapper around __device_release_driver() that will acquire device locks and do the necessary checks before calling it. The next patch will make use of it. Signed-off-by: Rafael J. Wysocki ---

Re: [PATCH 6/6] dt-bindings: clk: oxnas,stdclk: Add OX820 bindings

2016-10-10 Thread Rob Herring
On Wed, Oct 05, 2016 at 05:07:52PM +0200, Neil Armstrong wrote: > Add OX820 bindings and remove clock indices from bindings since they are > present > in the dt-bindings headers files. > > Signed-off-by: Neil Armstrong > --- >

Re: [PATCH] iommu/vt-d: Fix the size calculation of pasid table

2016-10-10 Thread Xunlei Pang
Ping David for confirmation On 2016/09/19 at 20:18, Joerg Roedel wrote: > [Cc'ing David] > > On Mon, Sep 12, 2016 at 10:49:11AM +0800, Xunlei Pang wrote: >> According to the vt-d spec, the size of pasid (state) entry is 8B >> which equals 3 in power of 2, the number of pasid (state) entries >> is

[PATCH v5 0/5] Functional dependencies between devices

2016-10-10 Thread Rafael J. Wysocki
Hi Everyone, > On Thursday, September 08, 2016 11:25:44 PM Rafael J. Wysocki wrote: > [cut] > > Time for another update. :-) > > Fewer changes this time, mostly to address issues found by Lukas and Marek. > > The most significant one is to make device_link_add() cope with the case > when >

[PATCH] [RFC] x86: avoid -mtune=atom for objtool warnings

2016-10-10 Thread Arnd Bergmann
I have no idea what is actually going on here, but building an x86 kernel with CONFIG_MATOM results in countless warnings from objtool, such as arch/x86/events/intel/ds.o: warning: objtool: intel_pmu_pebs_del()+0x43: call without frame pointer save/setup security/keys/keyring.o: warning:

Re: [PATCH v2] ASoC: sgtl5000: add headphone and LINEOUT mute controls

2016-10-10 Thread Richard Leitner
On 09/09/2016 12:17 PM, Mark Brown wrote: On Fri, Sep 09, 2016 at 09:46:45AM +0200, Richard Leitner wrote: Hi, any comments, updates or ideas for improvement on that patch? On Wed, 31 Aug 2016 09:26:31 +0200 Richard Leitner wrote: Please don't top post, reply in line with

Re: [PATCH v10 0/3] Secure Memory Allocation Framework

2016-10-10 Thread John Einar Reitan
On Fri, Oct 07, 2016 at 10:42:17AM -0400, Rob Clark wrote: > probably should keep the discussion on github (USAGE.md was updated a > bit more and merged into https://github.com/cubanismo/allocator so > look there for the latest).. > > but briefly: > > 1) my expectation is if the user is

Re: [PATCH v2 2/4] power: add axp20x-battery driver

2016-10-10 Thread Maxime Ripard
Hi Icenowy, On Sun, Oct 09, 2016 at 02:27:12PM +0800, Icenowy Zheng wrote: > + case POWER_SUPPLY_PROP_CURRENT_NOW: > + ret = regmap_read(power->regmap, AXP20X_BATT_DISCHRG_I_H, ); > + if (ret) > + return ret; > + ret =

Re: Oops on Power8 (was Re: [PATCH v2 1/7] workqueue: make workqueue available early during boot)

2016-10-10 Thread Tejun Heo
Hello, Michael. On Mon, Oct 10, 2016 at 09:22:55PM +1100, Michael Ellerman wrote: > This patch seems to be causing one of my Power8 boxes not to boot. > > Specifically commit 3347fa092821 ("workqueue: make workqueue available > early during boot") in linux-next. > > If I revert this on top of

[PATCH 2/2] intel_pmc_core: avoid boot time warning for !CONFIG_DEBUGFS_FS

2016-10-10 Thread Arnd Bergmann
While looking at a patch that introduced a compile-time warning "‘pmc_core_dev_state_get’ defined but not used" (I sent a patch for debugfs to fix it), I noticed that the same patch caused it in intel_pmc_core also introduced a bogus run-time warning: "PMC Core: debugfs register failed". The

Re: [RFC][PATCH 6/7] printk: use alternative printk buffers

2016-10-10 Thread Petr Mladek
On Mon 2016-10-10 13:09:57, Sergey Senozhatsky wrote: > On (10/06/16 13:32), Petr Mladek wrote: > > On Thu 2016-10-06 13:22:48, Sergey Senozhatsky wrote: > > > On (10/05/16 11:50), Petr Mladek wrote: > > > [..] > > > > > well, it solves a number of problems that the existing implementation > > > >

Re: [PATCH 24/54] md/raid1: Improve another size determination in setup_conf()

2016-10-10 Thread Dan Carpenter
On Thu, Oct 06, 2016 at 11:29:20AM +0200, Richard Weinberger wrote: > On Thu, Oct 6, 2016 at 11:22 AM, SF Markus Elfring > wrote: > > From: Markus Elfring > > Date: Tue, 4 Oct 2016 21:46:18 +0200 > > > > Replace the specification of a

Best Regards

2016-10-10 Thread Abdallah Bintou
ATTENTION: I received encouraging information about you and how trusts worthy you are. I am delighted with such useful information. My name is Mr. Abdallah Bintou I am the manager of Bill and Exchange department of BANK OF AFRICA (B.O.A) here in Ouagadougou, Burkina Faso. In my department I

Re: tg3 BUG: spinlock lockup suspected

2016-10-10 Thread Siva Reddy Kallam
On Sun, Oct 9, 2016 at 12:35 AM, Meelis Roos wrote: >> > That did not go well - bisect found the following commit but that does >> > not seem to be related at all. So probably the reproducibility is not >> > 100% but more random. >> >> Now I reproduced the bug even with 4.7-rc1 so

Re: About group scheduling for SCHED_DEADLINE

2016-10-10 Thread Peter Zijlstra
On Mon, Oct 10, 2016 at 12:15:58PM +0200, Peter Zijlstra wrote: > Right, so the problem we have is unspecified SCHED_FIFO on SMP and > historical behaviour. > > As you know we've extended FIFO to SMP by G-FIFO (run the m highest prio > tasks on m CPUs). But along with that, we allow arbitrary

[PATCH] softirq: Display IRQ_POLL for irq-poll statistics

2016-10-10 Thread Sagi Grimberg
This library was moved to the generic area and was renamed to irq-poll. Hence, update proc/softirqs output accordingly. Signed-off-by: Sagi Grimberg --- kernel/softirq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/softirq.c b/kernel/softirq.c

Re: [PATCH] Reorganize STM32 clocks in order to prepare them for PLLI2S and PLLSAI

2016-10-10 Thread Gabriel Fernandez
Hi Radosław, Yes i m nearly ready to push a patch-set to manage LCD-TFT clock. In my patch-set i introduced PLLI2S and PLLSAI in generic way, and offer the possibility to change the vco frequency (in order to cover all frequencies for any LCD). And then, the vco is no longer a fixed factor.

Re: [PATCH] drivers: staging: greybus: audio_codec.c: Fixed CHECKS for brace issues

2016-10-10 Thread Vaibhav Agarwal
On Mon, Oct 10, 2016 at 3:56 AM, Chase Metzger wrote: > Added braces to else statement where checkpatch complained. > > Signed-off-by: Chase Metzger > --- > drivers/staging/greybus/audio_codec.c | 4 +++- > 1 file changed, 3 insertions(+), 1

Re: [PATCH] sched/core: Fix kick offline cpu to do nohz idle load balance

2016-10-10 Thread Peter Zijlstra
On Mon, Oct 10, 2016 at 04:34:48PM +0800, Wanpeng Li wrote: > > If there is a need to kick the idle load balancer, an ILB will be selected > > to perform nohz idle load balance, however, if the selected ILB is in the > > process of offline, smp_sched_reschedule() which generates a sched IPI will >

Re: [PATCH v1 2/2] arm64: dts: rockchip: Add PX5 Evaluation board

2016-10-10 Thread Heiko Stuebner
Hi Andreas, Andy, Am Dienstag, 13. September 2016, 14:23:35 CEST schrieb Andreas Färber: > Hi, > > Am 10.09.2016 um 19:47 schrieb Andy Yan: > > PX5 EVB is designed by Rockchip for automotive field, > > which intergrated with CVBS(TP2825)/MIPI DSI/CSI/LVDS > > HDMI video input/output interface,

[PATCH 1/2] debugfs: improve DEFINE_DEBUGFS_ATTRIBUTE for !CONFIG_DEBUGFS_FS

2016-10-10 Thread Arnd Bergmann
The slp_s0_residency_usec debugfs file currently uses DEFINE_DEBUGFS_ATTRIBUTE(), but that macro cannot really be used to define files outside of the debugfs code, as it has no reference to the get/set functions if CONFIG_DEBUGFS_FS is not defined: drivers/platform/x86/intel_pmc_core.c:80:12:

Re: [RFC PATCH 0/3] PM / Domains: Add support for devices that require multiple domains

2016-10-10 Thread Jon Hunter
On 06/10/16 13:22, Ulf Hansson wrote: > On 20 September 2016 at 12:28, Jon Hunter wrote: >> The Tegra124/210 XUSB subsystem (that consists of both host and device >> controllers) is partitioned across 3 PM domains which are: >> - XUSBA: Superspeed logic (for USB 3.0) >> -

Re: Oops on Power8 (was Re: [PATCH v2 1/7] workqueue: make workqueue available early during boot)

2016-10-10 Thread Balbir Singh
On 10/10/16 21:22, Michael Ellerman wrote: > Hi Tejun, > > Tejun Heo writes: >> From f85002f627f7fdc7b3cda526863f5c9a8d36b997 Mon Sep 17 00:00:00 2001 >> From: Tejun Heo >> Date: Fri, 16 Sep 2016 15:49:32 -0400 >> Subject: [PATCH] workqueue: make workqueue

[PATCH] base memory: introduce CONFIG_MEMORY_DEVICE

2016-10-10 Thread Yisheng Xie
MEMORY_FAILURE do not depend on SPARSEMEM_MANUAL, nor MEMORY_HOTPLUG_SPARSE. However, when I tried to use sysfs: /sys/devices/system/memory/soft_offline_page /sys/devices/system/memory/hard_offline_page to test memory failure function with FLATMEM_MANUAL && MEMORY_FAILURE enabled on arch like

Re: igb driver can cause cache invalidation of non-owned memory?

2016-10-10 Thread David Miller
From: Nikita Yushchenko Date: Mon, 10 Oct 2016 12:51:28 +0300 > Hmm... I'm not about device writing to memory. This absolutely is about whether the device wrote into the area or not. > Sequence in igb driver is: > > dma_map(full_page) > >

[PATCH] qed: fix old-style function definition

2016-10-10 Thread Arnd Bergmann
The definition of qed_get_rdma_ops() is not a prototype unless we add 'void' here, as indicated by this W=1 warning: drivers/net/ethernet/qlogic/qed/qed_roce.c: In function ‘qed_get_rdma_ops’: drivers/net/ethernet/qlogic/qed/qed_roce.c:2950:28: error: old-style function definition

Re: Suspend/Resume issue in 4.8-rc8

2016-10-10 Thread Jarkko Sakkinen
On Mon, Oct 10, 2016 at 02:36:43AM -0400, Genki Marshall wrote: > Hello, > > On 4.8-rc8, I'm having an issue with laptop suspend/resume for the > Chromebook Pixel (2015). Specifically: > > When on commit 24532f7 on Linus's tree (latest commit at time of > writing) I'm having the following

[PATCH] dmaengine: mmp_tdma: select GENERIC_ALLOCATOR

2016-10-10 Thread Arnd Bergmann
build testing on x86 revealed a missing dependency that leads to a link error: drivers/dma/built-in.o: In function `mmp_tdma_prep_dma_cyclic': mmp_tdma.c:(.text+0x9564): undefined reference to `gen_pool_dma_alloc' drivers/dma/built-in.o: In function `mmp_tdma_free_chan_resources':

[PATCH RFC] perf bench futex: Support running benchmark while some cpus are offline

2016-10-10 Thread Pan Xinhui
i%ncpus might not be the correct cpu number which we want to set. If we run command "perf bench futex all" while some cpus are not online. We will fail to create pthreads. Lets fix this by adding int cpu_at_index(int i) which just do a simple traverse on the cpu_set. And return the correct cpu

Re: kernel v4.8: iptables logs are truncated with the 4.8 kernel?

2016-10-10 Thread Liping Zhang
Hi Chris, 2016-10-10 15:02 GMT+08:00 Chris Caputo : > On Tue, 4 Oct 2016, Justin Piszcz wrote: >> kernel 4.8 with ulogd-2.0.5- IPs are no longer logged: >> >> Oct 4 17:51:30 atom INPUT_BLOCK IN=eth1 OUT= >> MAC=00:1b:21:9c:3b:fa:3e:94:d5:d2:49:1e:08:00 LEN=0 TOS=00 PREC=0x00 >>

Re: igb driver can cause cache invalidation of non-owned memory?

2016-10-10 Thread Nikita Yushchenko
>> Hmm... I'm not about device writing to memory. > > This absolutely is about whether the device wrote into the > area or not. Not only. >> Sequence in igb driver is: >> >> dma_map(full_page) >> >> sync_to_cpu(half_page); >> skb_add_rx_frag(skb, half_page); >> napi_gro_receive(skb); >>

Re: [RFC PATCH 0/3] PM / Domains: Add support for devices that require multiple domains

2016-10-10 Thread Ulf Hansson
On 10 October 2016 at 13:18, Jon Hunter wrote: > > On 06/10/16 13:22, Ulf Hansson wrote: >> On 20 September 2016 at 12:28, Jon Hunter wrote: >>> The Tegra124/210 XUSB subsystem (that consists of both host and device >>> controllers) is partitioned

Re: [RFC][PATCH 4/4] futex: Rewrite FUTEX_UNLOCK_PI

2016-10-10 Thread Peter Zijlstra
On Sun, Oct 09, 2016 at 01:17:50PM +0200, Thomas Gleixner wrote: > On Fri, 7 Oct 2016, Peter Zijlstra wrote: > > top_waiter = futex_top_waiter(hb, ); > > if (top_waiter) { > > - ret = wake_futex_pi(uaddr, uval, top_waiter, hb); > > + struct futex_pi_state *pi_state =

[PATCH 0/5] Replace data type declarations with variables of same types in several source files.

2016-10-10 Thread Sergio Paracuellos
This patch series replaces several sizeof(struct XXX) stuff in favour of sizeof(var) which is the preferred one. Sergio Paracuellos (5): staging: wlan-ng: Replace data type declaration with variable of same type in cfg80211.c staging: wlan-ng: Replace data type declaration with variable

[PATCH RESEND] ARM: dts: keystone-k2*: Increase SPI Flash partition size for U-Boot

2016-10-10 Thread Vignesh R
U-Boot SPI Boot image is now more than 512KB for Keystone2 devices and cannot fit into existing partition. So, increase the SPI Flash partition for U-Boot to 1MB for all Keystone2 devices. Signed-off-by: Vignesh R --- This was submitted to v4.9 merge window but was never picked

[PATCH 2/5] staging: wlan-ng: Replace data type declaration with variable of same type in hfa384x_usb.c

2016-10-10 Thread Sergio Paracuellos
sizeof(var) instead of sizeof(struct XXX) is preferred. Fix it in hfa384x_usb.c file. Signed-off-by: Sergio Paracuellos --- drivers/staging/wlan-ng/hfa384x_usb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH RESEND v3] dma: omap-dma: add support for pause of non-cyclic transfers

2016-10-10 Thread Peter Ujfalusi
On 10/10/16 15:07, Vignesh R wrote: > From: Sebastian Andrzej Siewior > > This DMA driver is used by 8250-omap on DRA7-evm. There is one > requirement that is to pause a transfer. This is currently used on the RX > side. It is possible that the UART HW aborted the RX

Re: [PATCH 1/4] base: soc: Early register bus when needed

2016-10-10 Thread Arnd Bergmann
On Tuesday, October 4, 2016 11:09:24 AM CEST Geert Uytterhoeven wrote: > If soc_device_register() is called before soc_bus_register(), it crashes > with a NULL pointer dereference. > > soc_bus_register() is already a core_initcall(), but drivers/base/ is > entered later than e.g. drivers/pinctrl/

Re: [PATCH 3/4] base: soc: Check for NULL SoC device attributes

2016-10-10 Thread Arnd Bergmann
On Tuesday, October 4, 2016 11:09:26 AM CEST Geert Uytterhoeven wrote: > If soc_device_match() is used to check the value of a specific > attribute that is not present for the current SoC, the kernel crashes > with a NULL pointer dereference. > > Fix this by explicitly checking for the absence of

Re: [PATCH 1/2] host: ehci-exynos: Convert to use the SET_SYSTEM_SLEEP_PM_OPS

2016-10-10 Thread Anand Moon
hi Alan/Krzysztof, On 10 October 2016 at 02:47, Alan Stern wrote: > On Sun, 9 Oct 2016, Krzysztof Kozlowski wrote: > >> On Sun, Oct 09, 2016 at 02:34:14PM +, Anand Moon wrote: >> > Move the ehci-exynos system PM callbacks within #ifdef CONFIG_PM_SLEEP >> > as to

  1   2   3   4   5   6   7   8   9   10   >