Re: [PATCH v6 7/9] KVM: arm/arm64: vgic-its: Free caches when GITS_BASER Valid bit is cleared

2017-10-30 Thread Christoffer Dall
On Mon, Oct 30, 2017 at 03:19:54AM +, Marc Zyngier wrote: > On Thu, Oct 26 2017 at 6:23:09 pm BST, Eric Auger > wrote: > > When the GITS_BASER.Valid gets cleared, the data structures in > > guest RAM are not valid anymore. The device, collection > > and LPI lists

Re: [PATCH v6 8/9] KVM: arm/arm64: Document KVM_DEV_ARM_ITS_CTRL_RESET

2017-10-30 Thread Christoffer Dall
On Thu, Oct 26, 2017 at 05:23:10PM +0200, Eric Auger wrote: > At the moment, the in-kernel emulated ITS is not properly reset. > On guest restart/reset some registers keep their old values and > internal structures like device, ITE, and collection lists are not > freed. > > This may lead to

Re: [PATCH v4] KVM: arm/arm64: fix the incompatible matching for external abort

2017-10-30 Thread Christoffer Dall
On Mon, Oct 30, 2017 at 02:05:18PM +0800, Dongjiu Geng wrote: > kvm_vcpu_dabt_isextabt() tries to match a full fault syndrome, but > calls kvm_vcpu_trap_get_fault_type() that only returns the fault class, > thus reducing the scope of the check. This doesn't cause any observable > bug yet as we

[PATCH] PCI: rcar: Use common error handling code in rcar_pcie_enable_msi()

2017-10-30 Thread SF Markus Elfring
From: Markus Elfring Date: Mon, 30 Oct 2017 08:28:06 +0100 Adjust a jump target so that a specific error message is stored only once at the end of this function implementation. Replace two calls of the function "dev_err" by goto statements. This issue was detected

[PATCH v5 5/5] clk: aspeed: Add reset controller

2017-10-30 Thread Joel Stanley
There are some resets that are not associated with gates. These are represented by a reset controller. Reviewed-by: Andrew Jeffery Signed-off-by: Joel Stanley --- v5: - Add Andrew's Reviewed-by v3: - Add named initalisers for the reset defines - Add define

[PATCH v5 2/5] clk: aspeed: Register core clocks

2017-10-30 Thread Joel Stanley
This registers the core clocks; those which are required to calculate the rate of the timer peripheral so the system can load a clocksource driver. Signed-off-by: Joel Stanley --- v5: - Add Andrew's Reviewed-by v4: - Add defines to document the BIT() macros v3: - Fix

[PATCH v5 3/5] clk: aspeed: Add platform driver and register PLLs

2017-10-30 Thread Joel Stanley
This registers a platform driver to set up all of the non-core clocks. The clocks that have configurable rates are now registered. Signed-off-by: Joel Stanley -- v5: - Remove eclk configuration. We do not have enough information to correctly implement the mux and divisor, so

[PATCH v5 1/5] clk: Add clock driver for ASPEED BMC SoCs

2017-10-30 Thread Joel Stanley
This adds the stub of a driver for the ASPEED SoCs. The clocks are defined and the static registration is set up. Reviewed-by: Andrew Jeffery Signed-off-by: Joel Stanley --- v5: - Add Andrew's reviewed-by - Make aspeed_gates not initconst to avoid section

[PATCH v5 4/5] clk: aspeed: Register gated clocks

2017-10-30 Thread Joel Stanley
The majority of the clocks in the system are gates paired with a reset controller that holds the IP in reset. This borrows from clk_hw_register_gate, but registers two 'gates', one to control the clock enable register and the other to control the reset IP. This allows us to enforce the ordering:

[PATCH v5 0/5] clk: Add Aspeed clock driver

2017-10-30 Thread Joel Stanley
This driver supports the ast2500, ast2400 (and derivative) BMC SoCs from Aspeed. This is v5. See patches for detailed changelogs. v5: Address review from Andrew v4: Address review from Andrew and Stephen. v3: Address review from Andrew and has seen more testing on hardware v2: split the driver

Re: [PATCH v6 0/9] vITS Migration fixes and reset

2017-10-30 Thread Christoffer Dall
Hi Eric, On Thu, Oct 26, 2017 at 05:23:02PM +0200, Eric Auger wrote: > This series fixes various bugs observed when saving/restoring the > ITS state before the guest writes the ITS registers (on first boot or > after reset/reboot). > > This is a follow up of Wanghaibin's series [1] plus

[migration_cpu_stop] WARNING: CPU: 0 PID: 11 at arch/x86/kernel/smp.c:128 native_smp_send_reschedule+0x69/0x9e

2017-10-30 Thread Fengguang Wu
On Sun, Oct 29, 2017 at 11:51:55PM +0100, Fengguang Wu wrote: Hi Linus, Up to now we see the below boot error/warnings when testing v4.14-rc6. They hit the RC release mainly due to various imperfections in 0day's auto bisection. So I manually list them here and CC the likely easy to debug ones

Re: [PATCH v5 01/26] irqchip/gic-v3-its: Setup VLPI properties at map time

2017-10-30 Thread Christoffer Dall
On Fri, Oct 27, 2017 at 03:28:30PM +0100, Marc Zyngier wrote: > So far, we require the hypervisor to update the VLPI properties > once the the VLPI mapping has been established. While this > makes it easy for the ITS driver, it creates a window where > an incoming interrupt can be delivered with

Re: [PATCH v5 02/26] KVM: arm/arm64: register irq bypass consumer on ARM/ARM64

2017-10-30 Thread Christoffer Dall
On Fri, Oct 27, 2017 at 03:28:31PM +0100, Marc Zyngier wrote: > From: Eric Auger > > This patch selects IRQ_BYPASS_MANAGER and HAVE_KVM_IRQ_BYPASS > configs for ARM/ARM64. > > kvm_arch_has_irq_bypass() now is implemented and returns true. > As a consequence the irq bypass

Re: [PATCH] irqdomain: Update the comments of fwnode field of irq_domain structure

2017-10-30 Thread Marc Zyngier
On Mon, Oct 30 2017 at 10:15:00 am GMT, Dou Liyang wrote: > Commit: > > f110711a6053 ("irqdomain: Convert irqdomain-%3Eof_node to fwnode") > > converted of_node field to fwnode, but didn't update its comments. > > Update it. > > Signed-off-by: Dou Liyang

[lkp-robot] [locking/lockdep] 2dcd5adfb7: WARNING:possible_circular_locking_dependency_detected

2017-10-30 Thread kernel test robot
FYI, we noticed the following commit (built with gcc-5): commit: 2dcd5adfb7401b762ddbe4b86dcacc2f3de6b97b ("locking/lockdep: Remove the BROKEN flag from CONFIG_LOCKDEP_CROSSRELEASE and CONFIG_LOCKDEP_COMPLETIONS") https://git.kernel.org/cgit/linux/kernel/git/tip/tip.git locking/core in

Re: [lkp-robot] [android/ion] 5fb70554d6: kernel_selftests.android.make_fail

2017-10-30 Thread Pintu Kumar
On Sun, Oct 29, 2017 at 7:51 PM, kernel test robot wrote: > > FYI, we noticed the following commit (built with gcc-6): > > commit: 5fb70554d68e2ea032b6a28b082801d8b7b76cb8 ("android/ion: userspace > test utility for ion buffer sharing") > url: >

[PATCH] iio: adc: aspeed: Deassert reset in probe

2017-10-30 Thread Joel Stanley
The ASPEED SoC must deassert a reset in order to use the ADC peripheral. The device tree bindings are updated to document the resets phandle, and the example is updated to match what is expected for both the reset and clock phandle. Signed-off-by: Joel Stanley ---

[lkp-robot] [proc] eb948c71f7: WARNING:at_mm/hugetlb.c:#hugetlb_add_hstate

2017-10-30 Thread kernel test robot
FYI, we noticed the following commit (built with gcc-5): commit: eb948c71f790f4fd27440895ca19180a78fd3016 ("proc: mm: export PTE sizes directly in smaps") url: https://github.com/0day-ci/linux/commits/Fan-Du/proc-mm-export-PTE-sizes-directly-in-smaps/20171027-233355 in testcase: trinity with

[lkp-robot] [x86/topology] 45d87f5bb7: WARNING:at_kernel/locking/lockdep.c:#lockdep_trace_alloc

2017-10-30 Thread kernel test robot
FYI, we noticed the following commit (built with gcc-6): commit: 45d87f5bb794a19baa7f92fdf2fabdcb0575c66d ("x86/topology: Avoid wasting 128k for package id array") url:

Re: [RFC][PATCH 1/8] epoll: remove epmutex from ep_free() & eventpoll_release_file()

2017-10-30 Thread Hou Tao
Hi, On 2017/10/28 21:58, Davidlohr Bueso wrote: > On Sat, 28 Oct 2017, Hou Tao wrote: > >> Remove the global epmutex from ep_free() and eventpoll_release_file(). >> In the later patches, we will add locks with a smaller granularity >> to serve the same purposes of epmutex. >> >> Signed-off-by:

Re: [lkp-robot] [locking/lockdep] 2dcd5adfb7: WARNING:possible_circular_locking_dependency_detected

2017-10-30 Thread Byungchul Park
On Sun, Oct 29, 2017 at 10:18:55PM +0800, kernel test robot wrote: > > FYI, we noticed the following commit (built with gcc-5): > > commit: 2dcd5adfb7401b762ddbe4b86dcacc2f3de6b97b ("locking/lockdep: Remove > the BROKEN flag from CONFIG_LOCKDEP_CROSSRELEASE and > CONFIG_LOCKDEP_COMPLETIONS") >

[btrfs] WARNING: CPU: 0 PID: 6379 at fs/direct-io.c:293 dio_complete+0x1d4/0x220

2017-10-30 Thread Fengguang Wu
CC fsdevel. On Sun, Oct 29, 2017 at 11:51:55PM +0100, Fengguang Wu wrote: Hi Linus, Up to now we see the below boot error/warnings when testing v4.14-rc6. They hit the RC release mainly due to various imperfections in 0day's auto bisection. So I manually list them here and CC the likely easy

Re: [PATCH v6 9/9] KVM: arm/arm64: vgic-its: Implement KVM_DEV_ARM_ITS_CTRL_RESET

2017-10-30 Thread Christoffer Dall
On Thu, Oct 26, 2017 at 05:23:11PM +0200, Eric Auger wrote: > On reset we clear the valid bits of GITS_CBASER and GITS_BASER. > We also clear command queue registers and free the cache (device, > collection, and lpi lists). > > As we need to take the same locks as save/restore functions, we >

Re: [PATCH v4 05/13] irqchip: add initial support for ompic

2017-10-30 Thread Marc Zyngier
On Mon, Oct 30 2017 at 1:18:06 pm GMT, Stafford Horne wrote: > On Mon, Oct 30, 2017 at 02:29:18AM +, Marc Zyngier wrote: >> On Mon, Oct 30 2017 at 8:11:15 am GMT, Stafford Horne >> wrote: >> > From: Stefan Kristiansson

RE: [PATCH] hp_accel: Add accelerometer quirk for HP ProBook 440 G4

2017-10-30 Thread Osama Khan
Thanks for reviewing the patch. I'll keep your feedback in mind for future submissions. -Original Message- From: platform-driver-x86-ow...@vger.kernel.org [mailto:platform-driver-x86-ow...@vger.kernel.org] On Behalf Of Andy Shevchenko Sent: Friday, October 27, 2017 9:31 PM To: Osama

[PATCH] locking/lockdep: Revise Documentation/locking/crossrelease.txt

2017-10-30 Thread Byungchul Park
I'm afraid the revision is not perfect yet. Of course, the document can have got much better english by others than me. But, I think I should enhance it as much as I can, before they can help it starting with a better one. In addition, I removed verboseness as much as possible. ->8-

[ANNOUNCE] Git v2.15.0

2017-10-30 Thread Junio C Hamano
The latest feature release Git v2.15.0 is now available at the usual places. It is comprised of 769 non-merge commits since v2.14.0, contributed by 91 people, 28 of which are new faces. The tarballs are found at: https://www.kernel.org/pub/software/scm/git/ The following public

Re: [PATCH 2/3] drivers: phy: broadcom: Add driver for Cygnus USB phy controller

2017-10-30 Thread Chanwoo Choi
Hi, On 2017년 10월 30일 14:02, Raveendra Padasalagi wrote: > Hi Chanwoo, > > On Mon, Oct 30, 2017 at 5:32 AM, Chanwoo Choi wrote: >> Hi, >> >> On 2017년 10월 27일 17:35, Kishon Vijay Abraham I wrote: >>> +Chanwoo, for reviewing extcon >>> >>> Hi. >>> >>> On Tuesday 24 October

Re: [PATCH v2 1/2] media: i2c: Add the ov7740 image sensor driver

2017-10-30 Thread kbuild test robot
-ci/linux/commits/Wenyou-Yang/media-i2c-Add-the-ov7740-image-sensor-driver/20171030-140347 base: git://linuxtv.org/media_tree.git master config: blackfin-allyesconfig (attached as .config) compiler: bfin-uclinux-gcc (GCC) 6.2.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp

Re: [PATCH v2 1/1] mtd: intel-spi: Add Intel Lewisburg PCH SPI super SKU PCI ID

2017-10-30 Thread Mika Westerberg
On Sun, Oct 29, 2017 at 03:17:35PM -0700, sathyanarayanan.kuppusw...@linux.intel.com wrote: > From: Kuppuswamy Sathyanarayanan > > This patch adds Intel Lewisburg PCH SPI serial flash controller super > SKU PCI ID. > > Signed-off-by: Kuppuswamy

Linux 4.14-rc6: WARNING: CPU: 9 PID: 5377 at arch/x86/events/intel/core.c:2228 intel_pmu_handle_irq+0x4a8/0x4c0

2017-10-30 Thread Fengguang Wu
CC perf maintainers. On Sun, Oct 29, 2017 at 11:51:55PM +0100, Fengguang Wu wrote: Hi Linus, Up to now we see the below boot error/warnings when testing v4.14-rc6. They hit the RC release mainly due to various imperfections in 0day's auto bisection. So I manually list them here and CC the

Re: [PATCH v4 1/1] xen/time: do not decrease steal time after live migration on xen

2017-10-30 Thread Juergen Gross
On 30/10/17 06:26, Dongli Zhang wrote: > After guest live migration on xen, steal time in /proc/stat > (cpustat[CPUTIME_STEAL]) might decrease because steal returned by > xen_steal_lock() might be less than this_rq()->prev_steal_time which is > derived from previous return value of

Re: [PATCH v5 24/26] KVM: arm/arm64: GICv4: Prevent userspace from changing doorbell affinity

2017-10-30 Thread Christoffer Dall
On Fri, Oct 27, 2017 at 03:28:53PM +0100, Marc Zyngier wrote: > We so far allocate the doorbell interrupts without taking any > special measure regarding the affinity of these interrupts. We > simply move them around as required when the vcpu gets scheduled > on a different CPU. > > But that's

Re: [PATCH] hw_random: Include device.h instead of declaring struct device

2017-10-30 Thread Herbert Xu
On Thu, Oct 26, 2017 at 07:12:08PM +0530, PrasannaKumar Muralidharan wrote: > Include linux/device.h instead of declaring struct device. > > Signed-off-by: PrasannaKumar Muralidharan Nack. We should not include a header file when a simple forward declaration is

Re: [PATCH v2] kbuild: fix linker feature test macros when cross compiling with Clang

2017-10-30 Thread Masahiro Yamada
2017-10-29 0:00 GMT+09:00 Masahiro Yamada : > 2017-10-28 5:13 GMT+09:00 Nick Desaulniers : >> I was not seeing my linker flags getting added when using ld-option when >> cross compiling with Clang. Upon investigation, this seems to be due to

[mainline][ppc] sysfs: cannot create duplicate filename '/devices/hv_24x7/events/PM_CAPP1_APC_UOP_SEND_PB_CMD'

2017-10-30 Thread Abdul Haleem
Hi, A warning is being triggered while booting mainline kernel on ppc machine. Machine Type: Power 9 Kernel : 4.14.0-rc6 gcc: 4.8.5 Test : Boot Boot logs: -- hv-24x7: found a duplicate event PM_CAPP1_XPT_MSG_SENT_GT_16_LE_64, ct=1 hv-24x7: found a duplicate event

[PATCH v2] cgroup/cpuset: remove circular dependency deadlock

2017-10-30 Thread Prateek Sood
Remove circular dependency deadlock in a scenario where hotplug of CPU is being done while there is updation in cgroup and cpuset triggered from userspace. Process A => kthreadd => Process B => Process C => Process A Process A cpu_subsys_offline(); cpu_down(); _cpu_down();

[PATCH v2 2/2] gpio: aspeed: Add support for reset tolerance

2017-10-30 Thread Andrew Jeffery
Use the new pinconf parameter for state persistence to expose the associated capability of the Aspeed GPIO controller. Signed-off-by: Andrew Jeffery --- drivers/gpio/gpio-aspeed.c | 39 +-- 1 file changed, 37 insertions(+), 2 deletions(-)

[PATCH v2 0/2] gpio: Generalise state persistence (was: Expose reset tolerance capability)

2017-10-30 Thread Andrew Jeffery
Hello, This series provides an API to configure general GPIO state persistence in gpiolib. Previously, only sleep persistence was considered, but controllers like one found in Aspeed BMCs also support persistence of state across controller resets. There is some prior discussion on v1[1] and the

[PATCH v2 1/2] gpio: gpiolib: Generalise state persistence beyond sleep

2017-10-30 Thread Andrew Jeffery
General support for state persistence is added to gpiolib with the introduction of a new pinconf parameter to propagate the request to hardware. The existing persistence support for sleep is adapted to include hardware support if the GPIO driver provides it. Persistence continues to be enabled by

[RFC] ALSA: vsnd: Add Xen para-virtualized frontend driver

2017-10-30 Thread Oleksandr Andrushchenko
Hi, all! This is an attempt to summarize previous discussions on Xen para-virtual sound driver. A first attempt has been made to upstream the driver [1] which brought number of fundamental questions, one of the biggest ones was that the frontend driver has no means to synchronize its period

[PATCH] PM / QoS: Fix default runtime_pm device resume latency

2017-10-30 Thread Tero Kristo
The recent change to the PM QoS framework to introduce a proper no constraint value overlooked to handle the devices which don't implement PM QoS OPS. Runtime PM is one of the more severely impacted subsystems, failing every attempt to runtime suspend a device. This leads into some nasty second

Re: [PATCH] sound: fix kernel-doc build warning

2017-10-30 Thread Takashi Iwai
On Mon, 30 Oct 2017 01:08:52 +0100, Randy Dunlap wrote: > > From: Randy Dunlap > > Fix kernel-doc build error. A symbol that ends with an underscore > character ('_') has special meaning in reST (reStructuredText), so add > a '*' to prevent this error and to indicate that

Crypto Fixes for 4.14

2017-10-30 Thread Herbert Xu
Hi Linus: This push fixes an objtool regression. Please pull from git://git.kernel.org/pub/scm/linux/kernel/git/herbert/crypto-2.6.git linus Jason A. Donenfeld (1): crypto: x86/chacha20 - satisfy stack validation 2.0 arch/x86/crypto/chacha20-avx2-x86_64.S |4 ++--

[f2fs-dev] [PATCH] f2fs: optimize __update_nat_bits

2017-10-30 Thread Fan Li
Make three modification for __update_nat_bits: 1. Take the codes of dealing the nat with nid 0 out of the loop Such nat only needs to be dealt with once at beginning. 2. Use " nat_index == 0" instead of " start_nid == 0" to decide if it's the first nat block It's better that we don't

Re: [PATCH RFC] random: fix syzkaller fuzzer test int overflow

2017-10-30 Thread Greg KH
On Sun, Oct 29, 2017 at 02:25:29PM -0400, Theodore Ts'o wrote: > On Sat, Oct 28, 2017 at 11:22:00AM +0800, Chen Feng wrote: > > > > I checked the ioctl. What's the purpose of RNDADDTOENTCNT ioctl to > > userspace? > > It's a legacy ioctl which is probably not used anywhere; it's been > replaced

Re: [btrfs] WARNING: CPU: 0 PID: 6379 at fs/direct-io.c:293 dio_complete+0x1d4/0x220

2017-10-30 Thread Eryu Guan
Hi Fengguang, On Mon, Oct 30, 2017 at 08:20:21AM +0100, Fengguang Wu wrote: > CC fsdevel. > > On Sun, Oct 29, 2017 at 11:51:55PM +0100, Fengguang Wu wrote: > > Hi Linus, > > > > Up to now we see the below boot error/warnings when testing v4.14-rc6. > > > > They hit the RC release mainly due to

Re: [RFC] NAND: Optimize NAND_ECC_HW_OOB_FIRST read

2017-10-30 Thread Boris Brezillon
Hi PrasannaKumar, On Sat, 28 Oct 2017 13:13:51 +0530 PrasannaKumar Muralidharan wrote: > From: Lars-Peter Clausen > > Avoid sending unnecessary READ commands to the chip. > > Signed-off-by: Lars-Peter Clausen > Signed-off-by:

Re: IB/mlx4: Use common error handling code in __mlx4_ib_create_flow()

2017-10-30 Thread Leon Romanovsky
On Mon, Oct 30, 2017 at 09:04:36AM +0100, SF Markus Elfring wrote: > >>> I mean you aren't really making the code any smaller > >> > >> Would anybody like to check corresponding effects in more detail > >> after a specific function call was replaced by a goto statement? > > > > You are supposed to

Re: [PATCH 03/14] VFS: Implement a filesystem superblock creation/configuration context [ver #6]

2017-10-30 Thread Miklos Szeredi
On Fri, Oct 27, 2017 at 6:03 PM, David Howells wrote: > Miklos Szeredi wrote: > >> Yes I did mean vfs_parse_sb_flag_option(). >> >> Yes, I understand its purpose, but it would be cleaner if all the >> option parsing was done in fc->ops->parse_option().

Re: [PATCH 10/17] hyper-v: trace vmbus_open()

2017-10-30 Thread Greg KH
On Mon, Oct 30, 2017 at 09:16:19AM +0100, Vitaly Kuznetsov wrote: > Greg KH writes: > > > On Sun, Oct 29, 2017 at 12:21:09PM -0700, k...@exchange.microsoft.com wrote: > >> From: Vitaly Kuznetsov > >> > >> Add tracepoint to CHANNELMSG_OPENCHANNEL

Re: [PATCH 4/5] staging: fsl-dpaa2/eth: Change RX buffer alignment

2017-10-30 Thread Dan Carpenter
On Fri, Oct 27, 2017 at 02:44:37PM +, Bogdan Purcareata wrote: > > -Original Message- > > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > > Sent: Friday, October 27, 2017 5:30 PM > > To: Bogdan Purcareata > > Cc: Ruxandra Ioana Radulescu

Re: [PATCH 07/27] kexec_file: Disable at runtime if securelevel has been set

2017-10-30 Thread David Howells
Mimi Zohar wrote: > Yes, that works.  Thanks!  Remember is_ima_appraise_enabled() is > dependent on the "ima: require secure_boot rules in lockdown mode" > patch - http://kernsec.org/pipermail/linux-security-module-archive/201 > 7-October/003910.html. What happens if

Re: Adjustments for a lot of function implementations

2017-10-30 Thread SF Markus Elfring
> While we do not mind cleanup patches, the way you post them (one fix per file) I find it safer in this way while I was browsing through the landscape of Linux software components. > is really annoying and takes us too much time to review. It is just the case that there are so many remaining

RE: [PATCH 4/5] staging: fsl-dpaa2/eth: Change RX buffer alignment

2017-10-30 Thread Bogdan Purcareata
> -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Monday, October 30, 2017 10:56 AM > To: Bogdan Purcareata > Cc: de...@driverdev.osuosl.org; gre...@linuxfoundation.org; linux- > ker...@vger.kernel.org > Subject: Re: [PATCH

Re: [PATCH] net: hns: set correct return value

2017-10-30 Thread Tobias Klauser
On 2017-10-30 at 09:50:01 +0100, Pan Bian wrote: > The function of_parse_phandle() returns a NULL pointer if it cannot > resolve a phandle property to a device_node pointer. In function > hns_nic_dev_probe(), its return value is passed to PTR_ERR to extract > the error code.

Re: [f2fs-dev] [PATCH] f2fs: add a function to move nid

2017-10-30 Thread Jaegeuk Kim
On 10/28, Chao Yu wrote: > On 2017/10/28 19:03, Fan Li wrote: > > This patch add a new function to move nid from one state to another. > > Move operation is heavily used, by adding a new function for it > > we can cut down some branches from several flow. > > > > > > Signed-off-by: Fan li

Re: Adjustments for a lot of function implementations

2017-10-30 Thread Julia Lawall
On Mon, 30 Oct 2017, SF Markus Elfring wrote: > > While we do not mind cleanup patches, the way you post them (one fix per > > file) > > I find it safer in this way while I was browsing through the landscape of > Linux > software components. > > > > is really annoying and takes us too much

Re: [PATCH 0/3] arm64: defconfig: remove some Qualcomm USB options

2017-10-30 Thread Amit Kucheria
On Thu, Oct 26, 2017 at 3:23 AM, Alex Elder wrote: > This series deletes three config options related to USB on Qualcomm > SoCs from the arm64 "defconfig". The code enabled by the options is > no longer needed by any Qualcomm hardware. > >

Re: [PATCH] virtio_balloon: fix deadlock on OOM

2017-10-30 Thread Wei Wang
On 10/13/2017 09:21 PM, Michael S. Tsirkin wrote: fill_balloon doing memory allocations under balloon_lock can cause a deadlock when leak_balloon is called from virtballoon_oom_notify and tries to take same lock. To fix, split page allocation and enqueue and do allocations outside the lock.

Re: [PATCH v2 1/2] gpio: gpiolib: Generalise state persistence beyond sleep

2017-10-30 Thread Charles Keepax
On Mon, Oct 30, 2017 at 04:52:17PM +1030, Andrew Jeffery wrote: > General support for state persistence is added to gpiolib with the > introduction of a new pinconf parameter to propagate the request to > hardware. The existing persistence support for sleep is adapted to > include hardware support

RE: [PATCH] net: tipc: Convert timers to use timer_setup()

2017-10-30 Thread Jon Maloy
> -Original Message- > From: keesc...@google.com [mailto:keesc...@google.com] On Behalf Of > Kees Cook > Sent: Friday, October 27, 2017 06:58 > To: Jon Maloy > Cc: David S. Miller ; Ying Xue > ; net...@vger.kernel.org;

[PATCH] arm64: defconfig: enable CONFIG_GPIO_UNIPHIER

2017-10-30 Thread Masahiro Yamada
Enable the GPIO controller driver used for UniPhier SoC family. Signed-off-by: Masahiro Yamada --- arch/arm64/configs/defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig index ceb9ca2..6b4df78

[PATCH] ARM: multi_v7_defconfig: enable CONFIG_GPIO_UNIPHIER

2017-10-30 Thread Masahiro Yamada
Enable the GPIO controller driver used for UniPhier SoC family. Signed-off-by: Masahiro Yamada --- arch/arm/configs/multi_v7_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig

Re: [PATCH v2 1/2] media: i2c: Add the ov7740 image sensor driver

2017-10-30 Thread kbuild test robot
-ci/linux/commits/Wenyou-Yang/media-i2c-Add-the-ov7740-image-sensor-driver/20171030-140347 base: git://linuxtv.org/media_tree.git master config: sh-allmodconfig (attached as .config) compiler: sh4-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705 reproduce: wget https

Re: Linux 4.14-rc6: WARNING: CPU: 9 PID: 5377 at arch/x86/events/intel/core.c:2228 intel_pmu_handle_irq+0x4a8/0x4c0

2017-10-30 Thread Peter Zijlstra
On Mon, Oct 30, 2017 at 07:27:36AM +0100, Fengguang Wu wrote: > [ 189.480568] perf: interrupt took too long (5132 > 4982), lowering > kernel.perf_event_max_sample_rate to 38000 > [ 189.690660] perf: interrupt took too long (6582 > 6415), lowering > kernel.perf_event_max_sample_rate to 3 >

Re: [PATCH 10/17] hyper-v: trace vmbus_open()

2017-10-30 Thread Vitaly Kuznetsov
Greg KH writes: > On Mon, Oct 30, 2017 at 09:16:19AM +0100, Vitaly Kuznetsov wrote: >> Greg KH writes: >> >> > On Sun, Oct 29, 2017 at 12:21:09PM -0700, k...@exchange.microsoft.com >> > wrote: >> >> From: Vitaly Kuznetsov

[PATCH net-next 1/6] net: mvpp2: set the Rx FIFO size depending on the port speeds for PPv2.2

2017-10-30 Thread Antoine Tenart
The Rx FIFO size was set to the same value for all ports. This patch sets it depending on the maximum speed a given port can handle. This is only working for PPv2.2. Signed-off-by: Antoine Tenart --- drivers/net/ethernet/marvell/mvpp2.c | 52

[PATCH net-next 0/6] net: ppv2: various improvements

2017-10-30 Thread Antoine Tenart
Hi all, This series includes various patches improving the Marvell PPv2 driver. I send them as a series to avoid any possible merge conflict. - Patches 1 and 2 improve the initializing of the Tx and Rx FIFO. - Patch 3 initialize the RSS table to evenly distribute the ingress packets across

[PATCH net-next 6/6] net: mvpp2: simplify the Tx desc set DMA logic

2017-10-30 Thread Antoine Tenart
Two functions were always used to set the DMA addresses in Tx descriptors, because this address is split into a base+offset in the descriptors. A mask was used to come up with the base and offset addresses and two functions were called, mvpp2_txdesc_dma_addr_set() and mvpp2_txdesc_offset_set().

Re: possible deadlock in lru_add_drain_all

2017-10-30 Thread Byungchul Park
On Mon, Oct 30, 2017 at 09:22:03AM +0100, Michal Hocko wrote: > [Cc Byungchul. The original full report is > http://lkml.kernel.org/r/089e0825eec8955c1f055c83d...@google.com] > > Could you have a look please? This smells like a false positive to me. > > On Fri 27-10-17 15:42:34, Michal Hocko

Re: [PATCH] powerpc/kernel/sysfs: Export ldbar spr to sysfs

2017-10-30 Thread kbuild test robot
/commits/Anju-T-Sudhakar/powerpc-kernel-sysfs-Export-ldbar-spr-to-sysfs/20171030-155220 base: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next config: powerpc-storcenter_defconfig (attached as .config) compiler: powerpc-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705 reproduce

Re: [PATCH -mm -V2] mm, swap: Fix false error message in __swp_swapcount()

2017-10-30 Thread Michal Hocko
On Mon 30-10-17 08:57:13, Minchan Kim wrote: [...] > Although it's better than old, we can make it simple, still. > > diff --git a/include/linux/swapops.h b/include/linux/swapops.h > index 291c4b534658..f50d5a48f03a 100644 > --- a/include/linux/swapops.h > +++ b/include/linux/swapops.h > @@ -41,6

Re: [PATCH linux-next 1/1] ASoC: rsnd: ssiu: reset SSI_MODE register

2017-10-30 Thread Jiada Wang
Hello Morimoto-san On 10/30/2017 08:38 AM, Kuninori Morimoto wrote: Hi Jiada register SSI_MODE is set when SSI works in TDM Extended or TDM (Ex-)Split mode, but it isn't reset after SSI stops. this will cause issue, if SSI starts to work in other modes which requie SSI_MODE to have

Re: [perf_event_ctx_lock_nested] BUG: sleeping function called from invalid context at kernel/locking/mutex.c:97

2017-10-30 Thread Peter Zijlstra
On Mon, Oct 30, 2017 at 12:02:01AM +0100, Fengguang Wu wrote: > The dmesg fragment is > > [ 40.886662] done > [ 40.886686] [ 40.905102] capability: warning: `turbostat' uses 32-bit > capabilities (legacy support in use) > [ 40.940087] IPMI Device Information > [ 41.073351] [

[PATCH] lib: hint GCC to inlilne _find_next_bit() helper

2017-10-30 Thread Clement Courbet
Hi Yury, I've tried your benchmark on x86-64 (haswell). Inlining is a pretty small increase in binary size: 48B (2%). In terms of speed, results are not very stable from one run to another (I've included two runs to give you an idea), but overall there seems to be small improvement on the

Re: [PATCH v2] pids: introduce find_get_task_by_vpid helper

2017-10-30 Thread Balbir Singh
On Sat, Oct 28, 2017 at 4:52 AM, Mike Rapoport wrote: > There are several functions that do find_task_by_vpid() followed by > get_task_struct(). We can use a helper function instead. > > Signed-off-by: Mike Rapoport > --- I did a quick grep and

[PATCH] net: hns: set correct return value

2017-10-30 Thread Pan Bian
The function of_parse_phandle() returns a NULL pointer if it cannot resolve a phandle property to a device_node pointer. In function hns_nic_dev_probe(), its return value is passed to PTR_ERR to extract the error code. However, in this case, the extracted error code will always be zero, which is

Re: [PATCH v2] mm: mlock: remove lru_add_drain_all()

2017-10-30 Thread Vlastimil Babka
On 10/20/2017 12:25 AM, Shakeel Butt wrote: > lru_add_drain_all() is not required by mlock() and it will drain > everything that has been cached at the time mlock is called. And > that is not really related to the memory which will be faulted in > (and cached) and mlocked by the syscall itself. >

Re: [PATCH RFC] random: fix syzkaller fuzzer test int overflow

2017-10-30 Thread Theodore Ts'o
On Mon, Oct 30, 2017 at 08:39:56AM +0100, Greg KH wrote: > > No "Reported-by:"? Good point, fixed in my tree. - Ted

Re: [PATCH v3 2/2] watchdog: Add Spreadtrum watchdog driver

2017-10-30 Thread Baolin Wang
Hi Guenter, (There are some problem with Eric's email, he can not receive this email, so I help to reply his comments following yours. sorry for troubles.) +#define SPRD_WDT_MAX_TIMEOUT 60 >>> >>> >>> Is that really the maximum supported timeout ? Seems a bit low. >>> Shouldn't it

Re: [PATCH] drivers/firmware: psci_checker: Add missing destroy_timer_on_stack()

2017-10-30 Thread Arnd Bergmann
On Tue, Oct 24, 2017 at 5:02 PM, Lorenzo Pieralisi wrote: > The PSCI checker suspend_test_thread() function (ie executed for the > suspend test) requires an on-stack timer to carry out the test it > executes; it sets it up through the setup_timer_on_stack() API. > >

Re: [PATCH RFC 3/5] cpufreq: schedutil: Use idle_calls counter of the remote CPU

2017-10-30 Thread Viresh Kumar
On 28-10-17, 02:59, Joel Fernandes wrote: > Since the recent remote cpufreq callback work, its possible that a cpufreq > update is triggered from a remote CPU. For single policies however, the > current > code uses the local CPU when trying to determine if the remote sg_cpu entered > idle or is

Re: [GIT PULL] ARM: uniphier: fixes for v4.14

2017-10-30 Thread Arnd Bergmann
On Fri, Oct 20, 2017 at 7:16 PM, Masahiro Yamada wrote: > Hi Arnd, Olof, > > Please pull small fixes of ARM UniPhier DT. > It turned out one clock was missing that is actually > necessary for EHCI nodes. Pulled into fixes, thanks! Arnd

Re: [f2fs-dev] [PATCH 2/2] f2fs: relax EIO injection for quota file

2017-10-30 Thread Jaegeuk Kim
On 10/28, Chao Yu wrote: > Hi Jaegeuk, > > On 2017/10/23 0:51, Jaegeuk Kim wrote: > > On 10/22, Chao Yu wrote: > >> On 2017/10/20 0:56, Jaegeuk Kim wrote: > >>> This case is not happening easily. > >> > >> Actually it can happen, so why not just keep it? > > > > Okay, so let me keep this patch

Re: [PATCH] mtd: cfi_cmdset_0002: fix Cypress S29GL flash erase suspend

2017-10-30 Thread Boris Brezillon
Hi Chen, On Thu, 14 Sep 2017 21:58:11 +0800 Chen Bin wrote: > On UBIFS, Cypress S29GL01GT flash is failed to access occasionally > , and throw below error information and call trace: > MTD get_chip(): chip not ready after erase suspend > UBI error: ubi_io_write: error -5

Re: [PATCH v3 2/2] watchdog: Add Spreadtrum watchdog driver

2017-10-30 Thread Guenter Roeck
On 10/30/2017 02:18 AM, Baolin Wang wrote: Hi Guenter, (There are some problem with Eric's email, he can not receive this email, so I help to reply his comments following yours. sorry for troubles.) +#define SPRD_WDT_MAX_TIMEOUT 60 Is that really the maximum supported timeout ?

Build regressions/improvements in v4.14-rc7

2017-10-30 Thread Geert Uytterhoeven
Below is the list of build error/warning regressions/improvements in v4.14-rc7[1] compared to v4.13[2]. Summarized: - build errors: +6/-0 - build warnings: +1861/-923 JFYI, when comparing v4.14-rc7[1] to v4.14-rc6[3], the summaries are: - build errors: +0/-0 - build warnings: +521/-388

Re: [PATCH v2 RESEND] Revert "f2fs: handle dirty segments inside refresh_sit_entry"

2017-10-30 Thread Jaegeuk Kim
On 10/30, Chao Yu wrote: > On 2017/10/30 9:33, Yunlong Song wrote: > > This reverts commit 5e443818fa0b2a2845561ee25bec181424fb2889 > > > > The commit should be reverted because call sequence of below two parts > > of code must be kept: > > a. update sit information, it needs to be updated before

Re: [PATCH] media: pvrusb2: Convert timers to use timer_setup()

2017-10-30 Thread Hans Verkuil
On 10/24/2017 05:22 PM, Kees Cook wrote: > In preparation for unconditionally passing the struct timer_list pointer to > all timer callbacks, switch to using the new timer_setup() and from_timer() > to pass the timer pointer explicitly. > > Cc: Mike Isely > Cc: Mauro Carvalho

Re: [PATCH 3/3] spi: spi-axi: take extra controller reference before deregistration

2017-10-30 Thread Lars-Peter Clausen
On 10/29/2017 12:56 PM, Johan Hovold wrote: > Take an extra reference to the controller to avoid use-after-free in > free_irq() which is called only after the controller has been > deregistered and freed. > > Note that this is not an issue for this particular driver which does not > use shared

Re: [PATCH] drivers/firmware: psci_checker: Add missing destroy_timer_on_stack()

2017-10-30 Thread Lorenzo Pieralisi
On Mon, Oct 30, 2017 at 10:17:54AM +0100, Arnd Bergmann wrote: > On Tue, Oct 24, 2017 at 5:02 PM, Lorenzo Pieralisi > wrote: > > The PSCI checker suspend_test_thread() function (ie executed for the > > suspend test) requires an on-stack timer to carry out the test it >

Re: possible deadlock in lru_add_drain_all

2017-10-30 Thread Byungchul Park
On Mon, Oct 30, 2017 at 09:22:03AM +0100, Michal Hocko wrote: > [Cc Byungchul. The original full report is > http://lkml.kernel.org/r/089e0825eec8955c1f055c83d...@google.com] > > Could you have a look please? This smells like a false positive to me. +cc pet...@infradead.org Hello, IMHO, the

[GIT PULL] MMC fixes for v.4.14-rc8

2017-10-30 Thread Ulf Hansson
Hi Linus, Here's a PR with a couple of MMC fixes intended for v4.14-rc8. Details about the highlights are as usual found in the signed tag. Please pull this in! Kind regards Ulf Hansson The following changes since commit eb701ce16a45ed9880897c48f05ee608d77c72e3: mmc: sdhci-pci: Fix default

Re: [PATCH v2 2/3] mailbox: Add support for Hi3660 mailbox

2017-10-30 Thread Mark Rutland
Hi, On Mon, Oct 30, 2017 at 12:45:06PM +0800, Leo Yan wrote: > On Fri, Oct 27, 2017 at 11:46:00AM +0100, Mark Rutland wrote: > > On Fri, Oct 27, 2017 at 02:15:03PM +0800, Kaihua Zhong wrote: > > > +static int hi3660_mbox_check_state(struct mbox_chan *chan) > > > +{ > > > + /* Ensure channel is

Re: [PATCH] PM / QoS: Fix default runtime_pm device resume latency

2017-10-30 Thread Rafael J. Wysocki
On Mon, Oct 30, 2017 at 8:10 AM, Tero Kristo wrote: > The recent change to the PM QoS framework to introduce a proper > no constraint value overlooked to handle the devices which don't > implement PM QoS OPS. Runtime PM is one of the more severely > impacted subsystems, failing

[PATCH net-next 2/6] net: mvpp2: initialize the Tx FIFO size

2017-10-30 Thread Antoine Tenart
So far only the Rx FIFO size was initialized. For PPv2.2 the Tx FIFO size can be set as well. This patch initializes the Tx FIFO size for PPv2.2 controllers to 3K. Signed-off-by: Antoine Tenart --- drivers/net/ethernet/marvell/mvpp2.c | 25

Re: Adjustments for a lot of function implementations

2017-10-30 Thread Hans Verkuil
On 10/30/2017 11:40 AM, SF Markus Elfring wrote: > Feel free to repost, but only if you organize the patch as either fixing > the same type of > issue for a whole subdirectory (media/usb, media/pci, etc) >>> >>> Just for the record, while this may work for media, it won't work for all

Re: [PATCH v2 1/1] mtd: intel-spi: Add Intel Lewisburg PCH SPI super SKU PCI ID

2017-10-30 Thread Cyrille Pitchen
Le 30/10/2017 à 08:32, Mika Westerberg a écrit : > On Sun, Oct 29, 2017 at 03:17:35PM -0700, > sathyanarayanan.kuppusw...@linux.intel.com wrote: >> From: Kuppuswamy Sathyanarayanan >> >> This patch adds Intel Lewisburg PCH SPI serial flash controller

  1   2   3   4   5   6   7   8   9   10   >