Re: [RFC] prctl: Deprecate non PR_SET_MM_MAP operations

2018-04-04 Thread Cyrill Gorcunov
On Wed, Apr 04, 2018 at 07:35:41AM +0200, Michal Hocko wrote: > > > Or we can simply drop it off because PR_SET_MM_MAP covers all needs, > > > and I would rather prefer to do that asap. > > > > Thanks for making it deprecated. I'd prefer just drop it off if nobody > > objects. The change will get

Re: [PATCH v3 6/6] spi: sun6i: add DMA transfers support

2018-04-04 Thread Maxime Ripard
On Tue, Apr 03, 2018 at 06:44:49PM +0300, Sergey Suloev wrote: > DMA transfers are now available for sun6i and sun8i SoCs. > The DMA mode is used automatically as soon as requested > transfer length is more than FIFO length. > > Changes in v3: > 1) Debug log enhancements. > > Signed-off-by: Serge

Re: [PATCH v3 5/6] spi: sun6i: introduce register set/unset helpers

2018-04-04 Thread Maxime Ripard
On Tue, Apr 03, 2018 at 06:44:48PM +0300, Sergey Suloev wrote: > Two helper functions were added in order to set/unset > specified flags in registers. > > Signed-off-by: Sergey Suloev Again, I'm not really sure what the benefit of that is. Your diffstat is pretty much identical, so it's basicall

Re: [PATCH v2] perf stat: enable 1ms interval for printing event counters values

2018-04-04 Thread Jiri Olsa
On Tue, Apr 03, 2018 at 09:18:33PM +0300, Alexey Budankov wrote: > > Currently print count interval for performance counters values is > limited by 10ms so reading the values at frequencies higher than 100Hz > is restricted by the tool. > > This change makes perf stat -I possible on frequencies

Re: [PATCH v2 1/6] spi: core: handle timeout error from transfer_one()

2018-04-04 Thread Maxime Ripard
On Tue, Apr 03, 2018 at 07:24:11PM +0300, Sergey Suloev wrote: > On 04/03/2018 07:18 PM, Mark Brown wrote: > > On Tue, Apr 03, 2018 at 07:00:55PM +0300, Sergey Suloev wrote: > > > On 04/03/2018 06:52 PM, Mark Brown wrote: > > > > On Tue, Apr 03, 2018 at 06:29:00PM +0300, Sergey Suloev wrote: > > >

Re: [PATCH v2 2/6] spi: sun4i: restrict transfer length in PIO-mode

2018-04-04 Thread Maxime Ripard
On Tue, Apr 03, 2018 at 06:29:01PM +0300, Sergey Suloev wrote: > There is no need to handle the 3/4 FIFO empty interrupt > as the maximum supported transfer length in PIO mode > is 64 bytes. > As long as a problem was reported previously with filling FIFO > on A10s we want to stick with 63 bytes de

Re: [PATCH v2 3/6] spi: sun4i: coding style/readability improvements

2018-04-04 Thread Maxime Ripard
On Tue, Apr 03, 2018 at 06:29:02PM +0300, Sergey Suloev wrote: > Minor changes to fulfill the coding style and > improve the readability. Again, this is a very subjective statement. Overall, I'm not convinced by the idea of moving code around and shuffling variables assignments just for the sake o

Re: [PATCH v2 4/6] spi: sun4i: use completion provided by SPI core driver

2018-04-04 Thread Maxime Ripard
On Tue, Apr 03, 2018 at 06:29:03PM +0300, Sergey Suloev wrote: > As long as the completion already provided by the SPI core > then there is no need to waste extra-memory on this. > Also a waiting function was added to avoid code duplication. This should also be split in two parts. Maxime -- Max

Re: [PATCH v2 5/6] spi: sun4i: introduce register set/unset helpers

2018-04-04 Thread Maxime Ripard
On Tue, Apr 03, 2018 at 06:29:04PM +0300, Sergey Suloev wrote: > Two helper functions were added in order to set/unset > specified flags in registers. Again, the diffstat is pretty neutral, so what is the benefit? Maxime -- Maxime Ripard, Bootlin (formerly Free Electrons) Embedded Linux and Ker

Re: KASAN: use-after-free Read in radix_tree_next_chunk

2018-04-04 Thread Pavel Machek
Come on? Two copies of one mail, to lkml with thousands of subscribers? > Hello, > > syzbot hit the following crash on upstream commit > 9dd2326890d89a5179967c947dab2bab34d7ddee (Fri Mar 30 17:29:47 2018 +) > Merge tag 'ceph-for-4.16-rc8' of git://github.com/ceph/ceph-client > syzbot dashboa

[PATCH RT 5/7] crypto: limit more FPU-enabled sections

2018-04-04 Thread Daniel Wagner
From: Sebastian Andrzej Siewior Those crypto drivers use SSE/AVX/… for their crypto work and in order to do so in kernel they need to enable the "FPU" in kernel mode which disables preemption. There are two problems with the way they are used: - the while loop which processes X bytes may create l

[PATCH RT 6/7] Revert "memcontrol: Prevent scheduling while atomic in cgroup code"

2018-04-04 Thread Daniel Wagner
From: "Steven Rostedt (VMware)" The commit "memcontrol: Prevent scheduling while atomic in cgroup code" fixed this issue: refill_stock() get_cpu_var() drain_stock() res_counter_uncharge() res_counter_uncharge_until() spin

[PATCH RT 2/7] timer: Invoke timer_start_debug() where it makes sense

2018-04-04 Thread Daniel Wagner
From: Thomas Gleixner The timer start debug function is called before the proper timer base is set. As a consequence the trace data contains the stale CPU and flags values. Call the debug function after setting the new base and flags. Fixes: 500462a9de65 ("timers: Switch to a non-cascading whee

[PATCH RT 7/7] Linux 4.4.126-rt141-rc1

2018-04-04 Thread Daniel Wagner
Signed-off-by: Daniel Wagner --- localversion-rt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/localversion-rt b/localversion-rt index 09a4d4e4f010..564532c76548 100644 --- a/localversion-rt +++ b/localversion-rt @@ -1 +1 @@ --rt141 +-rt141-rc1 -- 2.14.3

Re: [PATCH 2/2] kfree_rcu() should use kfree_bulk() interface

2018-04-04 Thread Rao Shoaib
On 04/03/2018 07:23 PM, Matthew Wilcox wrote: On Tue, Apr 03, 2018 at 05:55:55PM -0700, Rao Shoaib wrote: On 04/03/2018 01:58 PM, Matthew Wilcox wrote: I think you might be better off with an IDR. The IDR can always contain one entry, so there's no need for this 'rbf_list_head' or __rcu_bulk

[PATCH RT 4/7] arm*: disable NEON in kernel mode

2018-04-04 Thread Daniel Wagner
From: Sebastian Andrzej Siewior NEON in kernel mode is used by the crypto algorithms and raid6 code. While the raid6 code looks okay, the crypto algorithms do not: NEON is enabled on first invocation and may allocate/free/map memory before the NEON mode is disabled again. This needs to be changed

[PATCH RT 0/7] 4.4.126-rt141-rc1

2018-04-04 Thread Daniel Wagner
Dear RT Folks, This is the RT stable review cycle of patch 4.4.126-rt141-rc1. The series contains a few patches which never made it to the v4.4-rt tree. I identified the patches using Steven's workflow. Next thing I try to do is to use Julia's script to idendtify missing patches. In theory nothin

[PATCH RT 1/7] locking: add types.h

2018-04-04 Thread Daniel Wagner
From: Sebastian Andrzej Siewior During the stable update the arm architecture did not compile anymore due to missing definition of u16/32. Cc: stable...@vger.kernel.org Signed-off-by: Sebastian Andrzej Siewior --- include/linux/spinlock_types_raw.h | 2 ++ 1 file changed, 2 insertions(+) diff

[PATCH RT 3/7] mm/slub: close possible memory-leak in kmem_cache_alloc_bulk()

2018-04-04 Thread Daniel Wagner
From: Sebastian Andrzej Siewior Under certain circumstances we could leak elements which were moved to the local "to_free" list. The damage is limited since I can't find any users here. Cc: stable...@vger.kernel.org Signed-off-by: Sebastian Andrzej Siewior --- mm/slub.c | 1 + 1 file changed,

Re: [RFC PATCH 3/4] acpi: apei: Do not panic() in NMI because of GHES messages

2018-04-04 Thread James Morse
Hi Alexandru, On 03/04/18 18:08, Alexandru Gagniuc wrote: > BIOSes like to send NMIs for a number of silly reasons often deemed > to be "fatal". For example pin bounce during a PCIE hotplug/unplug > might cause the link to go down and retrain, with fatal PCI errors > being generated while the link

Build failure due commit 9217e566bdee

2018-04-04 Thread Jarkko Nikula
Hi Today's head and linux-next doesn't compile due commit 9217e566bdee ("of_net: Implement of_get_nvmem_mac_address helper"): drivers/of/of_net.o: In function `of_get_nvmem_mac_address': .../drivers/of/of_net.c:100: undefined reference to `of_nvmem_cell_get' .../drivers/of/of_net.c:104: undefi

Re: [PATCH v3] staging: vt6655: check for memory allocation failures

2018-04-04 Thread Ji-Hun Kim
On Tue, Apr 03, 2018 at 01:40:52PM +0300, Dan Carpenter wrote: > > desc->rd_info = kzalloc(sizeof(*desc->rd_info), GFP_KERNEL); > > - > > + if (!desc->rd_info) { > > + ret = -ENOMEM; > > + goto error; > > + } > > if (!d

Re: v4.16+ seeing many unaligned access in dequeue_task_fair() on IA64

2018-04-04 Thread Peter Zijlstra
On Wed, Apr 04, 2018 at 12:04:00AM +, Luck, Tony wrote: > > bisect says: > > > > d519329f72a6 ("sched/fair: Update util_est only on util_avg updates") > > > > Reverting just this commit makes the problem go away. > > The unaligned read and write seem to come from: > > struct util_est ue = RE

Re: [PATCH 2/2] kfree_rcu() should use kfree_bulk() interface

2018-04-04 Thread Rao Shoaib
On 04/02/2018 10:20 AM, Christopher Lameter wrote: On Sun, 1 Apr 2018, rao.sho...@oracle.com wrote: kfree_rcu() should use the new kfree_bulk() interface for freeing rcu structures as it is more efficient. It would be even better if this approach could also use kmem_cache_free_bulk(

Re: Multiple generic PHY instances for DWC3 USB IP

2018-04-04 Thread Masahiro Yamada
2018-04-04 15:04 GMT+09:00 Felipe Balbi : > > Hi, > > Masahiro Yamada writes: >> 2018-04-04 14:36 GMT+09:00 Felipe Balbi : >>> >>> Hi, >>> >>> Masahiro Yamada writes: Currently, DWC3 core IP (drivers/usb/dwc3/core.c) can take only one PHY phandle for each of SS, HS. (phy-names DT p

Re: [PATCH] mmc: sdhci-cadence: send tune request twice to work around errata

2018-04-04 Thread Ulf Hansson
On 27 March 2018 at 11:29, Masahiro Yamada wrote: > Cadence sent out an errata report to their customers of this IP. > This errata is not so severe, but the tune request should be sent > twice to avoid the potential issue. > > Quote from the report: > > Problem Summary > --- > The IP61

Re: [PATCH RT 1/3] alarmtimer: Prevent live lock in alarm_cancel()

2018-04-04 Thread Sebastian Andrzej Siewior
On 2018-04-03 20:32:36 [+0200], Daniel Wagner wrote: > Hi Sebastian, > > On 03/28/2018 12:07 PM, Sebastian Andrzej Siewior wrote: > > If alarm_try_to_cancel() requires a retry, then depending on the > > priority setting the retry loop might prevent timer callback completion > > on RT. Prevent that

[PATCH 1/1] drm/i915: Do not use kfree() to free kmem_cache_alloc() return value

2018-04-04 Thread Xidong Wang
In eb_lookup_vmas(), the return value of kmem_cache_alloc() is freed with kfree(). I think the expected paired function is kmem_cahce_free(). Signed-off-by: Xidong Wang --- drivers/gpu/drm/i915/i915_gem_execbuffer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/

Re: [PATCH 11/11] x86/pti: leave kernel text global for !PCID

2018-04-04 Thread kbuild test robot
: https://github.com/0day-ci/linux/commits/Dave-Hansen/Use-global-pages-with-PTI/20180404-135611 reproduce: # apt-get install sparse make ARCH=x86_64 allmodconfig make C=1 CF=-D__CHECK_ENDIAN__ sparse warnings: (new ones prefixed by >>) >> arch/x86/mm/pti.c:2

[RFC PATCH] x86/pti: pti_clone_pmds can be static

2018-04-04 Thread kbuild test robot
Fixes: a7e2701bf2b2 ("x86/pti: leave kernel text global for !PCID") Signed-off-by: Fengguang Wu --- pti.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/mm/pti.c b/arch/x86/mm/pti.c index 3ee9ceb..057c8ff 100644 --- a/arch/x86/mm/pti.c +++ b/arch/x86/mm/pti.c @@

Re: [PATCH v5 08/13] ARM: sunxi: Add initialization of CNTVOFF

2018-04-04 Thread Maxime Ripard
On Tue, Apr 03, 2018 at 10:06:28PM +0200, Mylène Josserand wrote: > Hello, > > Thank you for the review. > > On Tue, 3 Apr 2018 11:12:18 +0200 > Maxime Ripard wrote: > > > On Tue, Apr 03, 2018 at 08:18:31AM +0200, Mylène Josserand wrote: > > > Add the initialization of CNTVOFF for sun8i-a83t. >

[PATCH] ARM: decompressor: restore r1 and r2 just before jumping to the kernel

2018-04-04 Thread Łukasz Stelmach
Hypervisor setup before __enter_kernel destroys the value sotred in r1. The value needs to be restored just before the jump. Signed-off-by: Łukasz Stelmach --- arch/arm/boot/compressed/head.S | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/boot/compressed/head.S

Re: [PATCH v2] ARM: dts: tpc: Device tree description of the iMX6Q TPC board

2018-04-04 Thread Lukasz Majewski
Hi Fabio, Thanks for the feedback, > Hi Lukasz, > > On Tue, Apr 3, 2018 at 1:59 PM, Lukasz Majewski wrote: > > > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt > > b/Documentation/devicetree/bindings/vendor-prefixes.txt index > > ae850d6c0ad3..8ff7eadc8bef 100644 --- > > a/

ARM compile failure in Re: linux-next: Tree for Apr 4

2018-04-04 Thread Pavel Machek
Hi! > Please do not add any v4.18 destined stuff to your linux-next included > trees until after v4.17-rc1 has been released. > > Changes since 20180403: > > The vfs tree still had its build failure for which I reverted a commit. > > Non-merge commits (relative to Linus' tree): 8505 > 8493 fil

Re: [RFC][PATCH] tracing, printk: Force no hashing when trace_printk() is used

2018-04-04 Thread Peter Zijlstra
On Tue, Apr 03, 2018 at 05:06:12PM -0400, Steven Rostedt wrote: > If you are concerned about attack surface, I could make it a bit more > difficult to tweak by malicious software. What about the patch below? > It would be much more difficult to modify this knob from an attack > vector. Not if you

x32 suspend failuer in Re: linux-next: Tree for Apr 4

2018-04-04 Thread Pavel Machek
Hi! > Please do not add any v4.18 destined stuff to your linux-next included > trees until after v4.17-rc1 has been released. On thinkpad x60, suspend does not suspend at all with this -next version. Previous versions suspended/resumed fine but broke networking. Any ideas? I guess bisecting on n

[PATCH] doc: Add vendor prefix for Kieback & Peter GmbH

2018-04-04 Thread Lukasz Majewski
The 'kiebackpeter' entry has been added to vendor-prefixes.txt to indicate products from Kieback & Peter GmbH. Signed-off-by: Lukasz Majewski --- Documentation/devicetree/bindings/vendor-prefixes.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/vendor-pre

Re: [PATCH 1/1] drm/i915: Do not use kfree() to free kmem_cache_alloc() return value

2018-04-04 Thread Chris Wilson
Quoting Xidong Wang (2018-04-04 08:37:54) > In eb_lookup_vmas(), the return value of kmem_cache_alloc() is freed > with kfree(). I think the expected paired function is kmem_cahce_free(). > > Signed-off-by: Xidong Wang That is indeed what it should be doing, Fixes: d1b48c1e7184 ("drm/i915: Repl

Re: x32 suspend failuer in Re: linux-next: Tree for Apr 4

2018-04-04 Thread Rafael J. Wysocki
On Wed, Apr 4, 2018 at 9:50 AM, Pavel Machek wrote: > Hi! > >> Please do not add any v4.18 destined stuff to your linux-next included >> trees until after v4.17-rc1 has been released. > > On thinkpad x60, suspend does not suspend at all with this -next > version. Previous versions suspended/resume

Re: [PATCH v9 00/24] Speculative page faults

2018-04-04 Thread Laurent Dufour
Hi Jerome, Thanks for reviewing this series. On 03/04/2018 22:37, Jerome Glisse wrote: > On Tue, Mar 13, 2018 at 06:59:30PM +0100, Laurent Dufour wrote: >> This is a port on kernel 4.16 of the work done by Peter Zijlstra to >> handle page fault without holding the mm semaphore [1]. >> >> The idea

Re: Multiple generic PHY instances for DWC3 USB IP

2018-04-04 Thread Felipe Balbi
Hi, Masahiro Yamada writes: >>> Each DWC3 instance is connected with >>> multiple HS PHYs and multiple SS PHYs, >>> depending on the number of ports. >> >> in that case, you shouldn't need dwc3 at all. A Host-only dwc3 is xHCI >> compliant. If you really don't have the gadget block, there's no n

Re: [GIT PULL] arch: remove obsolete architecture ports

2018-04-04 Thread Pavel Machek
On Tue 2018-04-03 11:18:15, Geert Uytterhoeven wrote: > On Mon, Apr 2, 2018 at 9:54 PM, Arnd Bergmann wrote: > > On Mon, Apr 2, 2018 at 8:57 PM, Linus Torvalds > > wrote: > > Regarding a possible revert, that would indeed involve reverting > > multiple patches for most architectures, plus parts o

Re: [PATCH v4 5/8] mfd: cros_ec_dev: Register cros_ec_accel_legacy driver as a subdevice.

2018-04-04 Thread Enric Balletbo Serra
Hi Lee, 2018-03-28 13:03 GMT+02:00 Lee Jones : > On Tue, 20 Mar 2018, Enric Balletbo i Serra wrote: > >> With this patch, the cros_ec_ctl driver will register the legacy >> accelerometer driver (named cros_ec_accel_legacy) if it fails to >> register sensors through the usual path cros_ec_sensors_r

Re: [GIT PULL] Kernel lockdown for secure boot

2018-04-04 Thread David Howells
Andy Lutomirski wrote: > As far as I can tell, what's really going on here is that there's a > significant contingent here that wants to prevent Linux from > chainloading something that isn't Linux. You have completely the wrong end of the stick. No one has said that or even implied that. You

Re: [lustre-devel] [PATCH 11/17] staging: lustre: libcfs: discard cfs_time_shift().

2018-04-04 Thread Dilger, Andreas
On Apr 2, 2018, at 16:26, NeilBrown wrote: > On Mon, Apr 02 2018, Dilger, Andreas wrote: >> On Mar 30, 2018, at 13:02, James Simmons wrote: This function simply multiplies by HZ and adds jiffies. This is simple enough to be opencoded, and doing so makes the code easier to read. >>>

KVM: Questions regarding page_track and memory slots

2018-04-04 Thread David Hildenbrand
Hi, I can see that we allocate under x86 for each KVM memslot "sizeof(unsigned short) * npages" for page_track. So 1 byte for each 4096 bytes of memory slot size. This doesn't sound a lot, but if we have very big memory slots (e.g. for NVDIMM), this can quickly get out of hand. E.g. for 4TB, we w

Re: [PATCH v2 2/6] i2c: i2c-stm32f7: Add slave support

2018-04-04 Thread Pierre Yves MORDRET
On 04/03/2018 05:26 PM, Wolfram Sang wrote: > On Mon, Mar 26, 2018 at 10:41:51AM +0200, Pierre Yves MORDRET wrote: >> >> >> On 03/25/2018 08:16 PM, Wolfram Sang wrote: >>> On Wed, Mar 21, 2018 at 05:48:56PM +0100, Pierre-Yves MORDRET wrote: This patch adds slave support for I2C controller em

Re: [PATCH v3 6/7] arm: dts: sun8i: a33: Add the DSI-related nodes

2018-04-04 Thread Maxime Ripard
Hi, On Thu, Mar 22, 2018 at 10:23:32AM +0800, Chen-Yu Tsai wrote: > On Tue, Mar 6, 2018 at 9:56 PM, Maxime Ripard > wrote: > > From: Maxime Ripard > > > > The A33 has a MIPI-DSI block, along with its D-PHY. Let's add it in order > > to use it in the relevant boards. > > > > Signed-off-by: Maxim

Re: [PATCH 1/1] drm/i915:Do not use kfree() to free kmem_cache_alloc() return value

2018-04-04 Thread Jani Nikula
On Wed, 04 Apr 2018, Xidong Wang wrote: > In eb_lookup_vmas(), lut, the return value of kmem_cache_alloc(), is freed > with kfree().I think the expected paired function is kmem_cache_free(). Agreed. But did you try to compile your patch before sending? Fixes: d1b48c1e7184 ("drm/i915: Replace exe

Re: [PATCH v2 3/6] i2c: i2c-stm32f7: Add initial SMBus protocols support

2018-04-04 Thread Pierre Yves MORDRET
On 04/03/2018 05:31 PM, Wolfram Sang wrote: > All SMBus protocols are implemented except SMBus-specific protocols. >>> >>> What does that mean? >> >> It miss SMBus Host Notification and SMBBus Alert. They are almost ready but >> I'm >> struggling to put them back to operational state after

Re: [PATCH v2 char-misc 1/1] x86/hyperv: Add interrupt handler annotations

2018-04-04 Thread Greg KH
On Tue, Apr 03, 2018 at 01:59:08PM -0700, mhkelle...@gmail.com wrote: > From: Michael Kelley > > Add standard interrupt handler annotations to > hyperv_vector_handler(). > > Signed-off-by: Michael Kelley > --- > Changes in v2: > * Fixed From: line > --- > arch/x86/kernel/cpu/mshyperv.c | 2 +-

[PATCH 1/3] adp5061: New driver for ADP5061 I2C battery charger

2018-04-04 Thread Stefan Popa
This patch adds basic support for Analog Devices I2C programmable linear battery charger. With this driver, some parameters can be read and configured such as: * trickle charge current level * trickle charge voltage threshold * weak charge threshold * constant current * constant charge voltage lim

[PATCH 2/3] adp5061: Add support for battery charging enable

2018-04-04 Thread Stefan Popa
This patch adds the option to enable/disable battery charging. This option is not configurable via the power_supply properties, therefore, access via sysfs was provided to examine and modify this attribute on the fly. Signed-off-by: Stefan Popa --- .../ABI/testing/sysfs-class-power-adp5061

[PATCH] media: dvb_frontend: fix wrong cast in compat_ioctl

2018-04-04 Thread Katsuhiro Suzuki
FE_GET_PROPERTY has always failed as following situations: - Use compatible ioctl - The array of 'struct dtv_property' has 2 or more items This patch fixes wrong cast to a pointer 'struct dtv_property' from a pointer of 2nd or after item of 'struct compat_dtv_property' array. Signed-off-by: K

Re: [PATCH] usb: dwc3: of-simple: use managed and shared reset control

2018-04-04 Thread Philipp Zabel
On Wed, 2018-04-04 at 10:25 +0900, Masahiro Yamada wrote: > 2018-04-03 19:35 GMT+09:00 Vivek Gautam : > > > > > > On 4/3/2018 3:49 PM, Masahiro Yamada wrote: > > > > > > 2018-04-03 17:46 GMT+09:00 Philipp Zabel : > > > > > > > > On Tue, 2018-04-03 at 17:30 +0900, Masahiro Yamada wrote: > > > >

[PATCH 3/3] adp5061: Add support for charging voltage limit enable

2018-04-04 Thread Stefan Popa
This patch adds the option to activate/deactivate the charging voltage limit. If activated, the charger prevents charging until the battery voltage drops below the VCHG_VLIM threshold. This option is not configurable via the power_supply properties, therefore, access via sysfs was provided to exam

Re: [PATCH v2 5/6] i2c: i2c-stm32f7: Add DMA support

2018-04-04 Thread Pierre Yves MORDRET
On 04/03/2018 05:39 PM, Wolfram Sang wrote: > >> +#define STM32F7_I2C_DMA_LEN_MIN 0x1 > ... > >> +if (i2c_dev->dma && f7_msg->count >= STM32F7_I2C_DMA_LEN_MIN) { > > Are you using DMA for every message with a length >= 1? The setup of > that might be more expensive than

Re: linux-next: Signed-off-by missing for commit in the overlayfs tree

2018-04-04 Thread Miklos Szeredi
On Fri, Mar 30, 2018 at 3:08 AM, Stephen Rothwell wrote: > Hi Miklos, > > Commit > > cbf293becfa4 ("ovl: cleanup setting OVL_INDEX") > > is missing a Signed-off-by from its author. Hi Steven, AFAIK for trivial work we are not required to get a signoff from the author. Thanks, Miklos

Re: [PATCH v9 09/24] mm: protect mremap() against SPF hanlder

2018-04-04 Thread Laurent Dufour
On 28/03/2018 23:21, David Rientjes wrote: > On Wed, 28 Mar 2018, Laurent Dufour wrote: > @@ -326,7 +336,10 @@ static unsigned long move_vma(struct vm_area_struct *vma, mremap_userfaultfd_prep(new_vma, uf); arch_remap(mm, old_addr, old_addr + old_len, >

Re: [PATCH V3 4/4] genirq/affinity: irq vector spread among online CPUs as far as possible

2018-04-04 Thread Thomas Gleixner
On Wed, 4 Apr 2018, Ming Lei wrote: > On Tue, Apr 03, 2018 at 03:32:21PM +0200, Thomas Gleixner wrote: > > On Thu, 8 Mar 2018, Ming Lei wrote: > > > 1) before 84676c1f21 ("genirq/affinity: assign vectors to all possible > > > CPUs") > > > irq 39, cpu list 0 > > > irq 40, cpu list 1 > > > irq

Re: [RFC] drm/atomic+msm: add helper to implement legacy dirtyfb

2018-04-04 Thread Thomas Hellstrom
Hi, On 04/04/2018 08:58 AM, Daniel Vetter wrote: On Wed, Apr 4, 2018 at 12:42 AM, Rob Clark wrote: Add an atomic helper to implement dirtyfb support. This is needed to support DSI command-mode panels with x11 userspace (ie. when we can't rely on pageflips to trigger a flush to the panel). To

Re: [RESEND PATCH V4] pidns: introduce syscall translate_pid

2018-04-04 Thread Konstantin Khlebnikov
On 04.04.2018 00:51, Nagarathnam Muthusamy wrote: On 04/03/2018 02:52 PM, Andrew Morton wrote: On Tue, 3 Apr 2018 14:45:28 -0700 Nagarathnam Muthusamy wrote: This changelog doesn't explain what the value is to our users.  I assume it is a performance optimization because "backward translat

[PATCH v2 6/9] trace_uprobe: Support SDT markers having reference count (semaphore)

2018-04-04 Thread Ravi Bangoria
Userspace Statically Defined Tracepoints[1] are dtrace style markers inside userspace applications. These markers are added by developer at important places in the code. Each marker source expands to a single nop instruction in the compiled code but there may be additional overhead for computing th

[PATCH v2 4/9] Uprobe: Rename map_info to uprobe_map_info

2018-04-04 Thread Ravi Bangoria
map_info is very generic name, rename it to uprobe_map_info. Renaming will help to export this structure outside of the file. Also rename free_map_info() to uprobe_free_map_info() and build_map_info() to uprobe_build_map_info(). Signed-off-by: Ravi Bangoria Reviewed-by: Jérôme Glisse --- kerne

[PATCH v2 3/9] Uprobe: Move mmput() into free_map_info()

2018-04-04 Thread Ravi Bangoria
From: Oleg Nesterov build_map_info() has a side effect like one need to perform mmput() when done with the mm. Add mmput() in free_map_info() so that user does not have to call it explicitly. Signed-off-by: Oleg Nesterov Signed-off-by: Ravi Bangoria --- kernel/events/uprobes.c | 9 ++---

[PATCH v2 0/9] trace_uprobe: Support SDT markers having reference count (semaphore)

2018-04-04 Thread Ravi Bangoria
Userspace Statically Defined Tracepoints[1] are dtrace style markers inside userspace applications. Applications like PostgreSQL, MySQL, Pthread, Perl, Python, Java, Ruby, Node.js, libvirt, QEMU, glib etc have these markers embedded in them. These markers are added by developer at important places

[PATCH v2 8/9] trace_uprobe/sdt: Document about reference counter

2018-04-04 Thread Ravi Bangoria
Reference counter gate the invocation of probe. If present, by default reference count is 0. Kernel needs to increment it before tracing the probe and decrement it when done. This is identical to semaphore in Userspace Statically Defined Tracepoints (USDT). Document usage of reference counter. Si

[PATCH v2 7/9] trace_uprobe/sdt: Fix multiple update of same reference counter

2018-04-04 Thread Ravi Bangoria
When virtual memory map for binary/library is being prepared, there is no direct one to one mapping between mmap() and virtual memory area. Ex, when loader loads the library, it first calls mmap(size = total_size), where total_size is addition of size of all elf sections that are going to be mapped

[PATCH v2 1/9] Uprobe: Export vaddr <-> offset conversion functions

2018-04-04 Thread Ravi Bangoria
These are generic functions which operates on file offset and virtual address. Make these functions available outside of uprobe code so that other can use it as well. Signed-off-by: Ravi Bangoria Reviewed-by: Jérôme Glisse --- include/linux/mm.h | 12 kernel/events/uprobes.c |

Re: [PATCH] drm/imx: Remove last traces of struct imx_drm_crtc

2018-04-04 Thread Philipp Zabel
On Tue, 2018-04-03 at 20:56 -0300, Fabio Estevam wrote: > On Tue, Mar 27, 2018 at 6:39 PM, Leonard Crestez > wrote: > > When the definition of this struct was removed a forward declaration and an > > unused struct member were still left around. Remove them because they serve > > no purpose. > > >

[PATCH v2 9/9] perf probe: Support SDT markers having reference counter (semaphore)

2018-04-04 Thread Ravi Bangoria
With this, perf buildid-cache will save SDT markers with reference counter in probe cache. Perf probe will be able to probe markers having reference counter. Ex, # readelf -n /tmp/tick | grep -A1 loop2 Name: loop2 ... Semaphore: 0x10020036 # ./perf buildid-cache --add /tmp/tic

[PATCH v2 2/9] mm: Prefix vma_ to vaddr_to_offset() and offset_to_vaddr()

2018-04-04 Thread Ravi Bangoria
Make function names more meaningful by adding vma_ prefix to them. Signed-off-by: Ravi Bangoria Reviewed-by: Jérôme Glisse --- include/linux/mm.h | 4 ++-- kernel/events/uprobes.c | 14 +++--- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/include/linux/mm.h b/incl

[PATCH v2 5/9] Uprobe: Export uprobe_map_info along with uprobe_{build/free}_map_info()

2018-04-04 Thread Ravi Bangoria
Given the file(inode) and offset, build_map_info() finds all existing mm that map the portion of file containing offset. Exporting these functions and data structure will help to use them in other set of files. Signed-off-by: Ravi Bangoria Reviewed-by: Jérôme Glisse --- include/linux/uprobes.h

Re: [PATCH] usbip: vhc_hcd: prevent module being removed while device are attached

2018-04-04 Thread Oliver Neukum
Am Dienstag, den 03.04.2018, 09:56 -0600 schrieb Shuah Khan: > This is a virtual device associated to a real physical device on a different > system. My concern is that if the module gets removed accidentally then it > could disrupt access to the remote device. The remote nature of the device > wit

Re: [PATCH] drivers: gpio: pca953x: add compatibility for pcal6524 and pcal9555a

2018-04-04 Thread H. Nikolaus Schaller
Hi, > Am 30.03.2018 um 23:39 schrieb Andy Shevchenko : > > On Thu, Mar 29, 2018 at 10:29 PM, H. Nikolaus Schaller > wrote: > >> Another issue I have fixed is that the pcal6524 has 4 registers per >> bank while the pcal9555a has only 2. > > So, you mean it's still in non-working case in Linus'

Re: [PATCH] HID: input: fix battery level reporting on BT mice

2018-04-04 Thread Jiri Kosina
On Tue, 3 Apr 2018, Martin van Es wrote: > Hi Dimitry, > > I reapplied the 3 commits I had to revert earlier and applied your patch. > Have > correct battery level reading on my BT mouse back! > > /sys/class/power_supply/hid-00:1f:20:fd:cb:be-battery# grep "" * > capacity:53 > grep: device: Is

Re: [GIT PULL] arch: remove obsolete architecture ports

2018-04-04 Thread Arnd Bergmann
On Wed, Apr 4, 2018 at 10:01 AM, Pavel Machek wrote: > On Tue 2018-04-03 11:18:15, Geert Uytterhoeven wrote: >> In reality, a resurrection may not be implemented as a pure revert, but as >> the addition of a new architecture, implemented using modern features (DT, >> CCF, ...). > > By insisting o

Re: [PATCH 2/2] kfree_rcu() should use kfree_bulk() interface

2018-04-04 Thread Rao Shoaib
On 04/04/2018 12:16 AM, Rao Shoaib wrote: On 04/03/2018 07:23 PM, Matthew Wilcox wrote: On Tue, Apr 03, 2018 at 05:55:55PM -0700, Rao Shoaib wrote: On 04/03/2018 01:58 PM, Matthew Wilcox wrote: I think you might be better off with an IDR.  The IDR can always contain one entry, so there's n

Re: Multiple generic PHY instances for DWC3 USB IP

2018-04-04 Thread Arnd Bergmann
On Wed, Apr 4, 2018 at 10:00 AM, Felipe Balbi wrote: > > Hi, > > Masahiro Yamada writes: Each DWC3 instance is connected with multiple HS PHYs and multiple SS PHYs, depending on the number of ports. >>> >>> in that case, you shouldn't need dwc3 at all. A Host-only dwc3 is xHCI >>>

Re: [RFC] drm/atomic+msm: add helper to implement legacy dirtyfb

2018-04-04 Thread Daniel Vetter
On Wed, Apr 04, 2018 at 10:22:21AM +0200, Thomas Hellstrom wrote: > Hi, > > On 04/04/2018 08:58 AM, Daniel Vetter wrote: > > On Wed, Apr 4, 2018 at 12:42 AM, Rob Clark wrote: > > > Add an atomic helper to implement dirtyfb support. This is needed to > > > support DSI command-mode panels with x11

Re: x32 suspend failuer in Re: linux-next: Tree for Apr 4

2018-04-04 Thread Pavel Machek
On Wed 2018-04-04 09:58:17, Rafael J. Wysocki wrote: > On Wed, Apr 4, 2018 at 9:50 AM, Pavel Machek wrote: > > Hi! > > > >> Please do not add any v4.18 destined stuff to your linux-next included > >> trees until after v4.17-rc1 has been released. > > > > On thinkpad x60, suspend does not suspend a

Re: [PATCH V2 6/7] thermal/drivers/cpu_cooling: Introduce the cpu idle cooling driver

2018-04-04 Thread Daniel Lezcano
On 26/02/2018 05:30, Viresh Kumar wrote: [ ... ] + + for_each_possible_cpu(cpu) { + cpumask = topology_core_cpumask(cpu); + + cct = per_cpu_ptr(&cpuidle_cooling_tsk, cpu); + + /* + * This condition makes the first cpu b

Re: [PATCH] HID: input: fix battery level reporting on BT mice

2018-04-04 Thread Martin van Es
On Wednesday, April 4, 2018 10:33:16 AM CEST Jiri Kosina wrote: > Can I add your Tested-by: while applying the commit? That's ok. Best regards, Martin

[PATCH v9 02/10] sched: idle: Do not stop the tick upfront in the idle loop

2018-04-04 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Push the decision whether or not to stop the tick somewhat deeper into the idle loop. Stopping the tick upfront leads to unpleasant outcomes in case the idle governor doesn't agree with the nohz code on the duration of the upcoming idle period. Specifically, if the tick

[PATCH v9 00/10] sched/cpuidle: Idle loop rework

2018-04-04 Thread Rafael J. Wysocki
Hi All, Thanks a lot for the feedback so far! For the motivation/summary, please refer to the BZ entry at https://bugzilla.kernel.org/show_bug.cgi?id=199227 created for collecting information related to this patch series. Some v7.3 testing results from Len and Doug are in there already. The

[PATCH v9 01/10] time: tick-sched: Reorganize idle tick management code

2018-04-04 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Prepare the scheduler tick code for reworking the idle loop to avoid stopping the tick in some cases. The idea is to split the nohz idle entry call to decouple the idle time stats accounting and preparatory work from the actual tick stop code, in order to later be able to

[PATCH v9 09/10] cpuidle: menu: Refine idle state selection for running tick

2018-04-04 Thread Rafael J. Wysocki
From: Rafael J. Wysocki If the tick isn't stopped, the target residency of the state selected by the menu governor may be greater than the actual time to the next tick and that means lost energy. To avoid that, make tick_nohz_get_sleep_length() return the current time to the next event (before s

[PATCH v9 04/10] jiffies: Introduce USER_TICK_USEC and redefine TICK_USEC

2018-04-04 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Since the subsequent changes will need a TICK_USEC definition analogous to TICK_NSEC, rename the existing TICK_USEC as USER_TICK_USEC, update its users and redefine TICK_USEC accordingly. Suggested-by: Peter Zijlstra Signed-off-by: Rafael J. Wysocki --- v8 -> v9: No ch

Re: [PATCH] lib: logic_pio: Fix potential NULL pointer dereference

2018-04-04 Thread John Garry
On 03/04/2018 22:15, Gustavo A. R. Silva wrote: new_range is being dereferenced before it is null checked, hence there is a potential null pointer dereference. Fix this by moving the pointer dereference after new_range has been properly null checked. Hi Gustavo, In fact we expect new_range t

[PATCH v9 03/10] sched: idle: Do not stop the tick before cpuidle_idle_call()

2018-04-04 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Make cpuidle_idle_call() decide whether or not to stop the tick. First, the cpuidle_enter_s2idle() path deals with the tick (and with the entire timekeeping for that matter) by itself and it doesn't need the tick to be stopped beforehand. Second, to address the issue wit

[PATCH v9 05/10] cpuidle: Return nohz hint from cpuidle_select()

2018-04-04 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Add a new pointer argument to cpuidle_select() and to the ->select cpuidle governor callback to allow a boolean value indicating whether or not the tick should be stopped before entering the selected state to be returned from there. Make the ladder governor ignore that po

[PATCH v9 06/10] time: tick-sched: Split tick_nohz_stop_sched_tick()

2018-04-04 Thread Rafael J. Wysocki
From: Rafael J. Wysocki In order to address the issue with short idle duration predictions by the idle governor after the scheduler tick has been stopped, split tick_nohz_stop_sched_tick() into two separate routines, one computing the time to the next timer event and the other simply stopping the

[PATCH] f2fs: enlarge block plug coverage

2018-04-04 Thread Chao Yu
This patch enlarges block plug coverage in __issue_discard_cmd, in order to collect more pending bios before issuing them, to avoid being disturbed by previous discard I/O in IO aware discard mode. Signed-off-by: Chao Yu --- fs/f2fs/segment.c | 7 +-- 1 file changed, 5 insertions(+), 2 delet

[PATCH v9 10/10] cpuidle: menu: Avoid selecting shallow states with stopped tick

2018-04-04 Thread Rafael J. Wysocki
From: Rafael J. Wysocki If the scheduler tick has been stopped already and the governor selects a shallow idle state, the CPU can spend a long time in that state if the selection is based on an inaccurate prediction of idle time. That effect turns out to be relevant, so it needs to be mitigated.

[PATCH v9 08/10] sched: idle: Select idle state before stopping the tick

2018-04-04 Thread Rafael J. Wysocki
From: Rafael J. Wysocki In order to address the issue with short idle duration predictions by the idle governor after the scheduler tick has been stopped, reorder the code in cpuidle_idle_call() so that the governor idle state selection runs before tick_nohz_idle_go_idle() and use the "nohz" hint

[PATCH v9 07/10] time: hrtimer: Introduce hrtimer_next_event_without()

2018-04-04 Thread Rafael J. Wysocki
From: Rafael J. Wysocki The next set of changes will need to compute the time to the next hrtimer event over all hrtimers except for the scheduler tick one. To that end introduce a new helper function, hrtimer_next_event_without(), for computing the time until the next hrtimer event over all tim

Re: [PATCH] ARC: Improve cmpxchng syscall implementation

2018-04-04 Thread Alexey Brodkin
Hi Vineet, Peter, On Wed, 2018-03-21 at 14:54 +0300, Alexey Brodkin wrote: > Hi Vineet, > > On Mon, 2018-03-19 at 11:29 -0700, Vineet Gupta wrote: > > On 03/19/2018 04:00 AM, Alexey Brodkin wrote: > > > arc_usr_cmpxchg syscall is supposed to be used on platforms > > > that lack support of Load-Lo

[PATCH v4 4/9] vsprintf: Consolidate handling of unknown pointer specifiers

2018-04-04 Thread Petr Mladek
There are few printk formats that make sense only with two or more specifiers. Also some specifiers make sense only when a kernel feature is enabled. The handling of unknown specifiers is strange, inconsistent, and even leaking the address. For example, netdev_bits() prints the non-hashed pointer

[PATCH v4 1/9] vsprintf: Shuffle ptr_to_id() code

2018-04-04 Thread Petr Mladek
This patch just moves ptr_to_id() implementation up in the source file. We will want to call it earlier. This patch does not change the existing behavior. Signed-off-by: Petr Mladek --- lib/vsprintf.c | 134 - 1 file changed, 67 insertions

[PATCH v4 8/9] vsprintf: Prevent crash when dereferencing invalid pointers

2018-04-04 Thread Petr Mladek
We already prevent crash when dereferencing some obviously broken pointers. But the handling is not consistent. Sometimes we print "(null)" only for pure NULL pointer, sometimes for pointers in the first page and sometimes also for pointers in the last page (error codes). Note that printk() call t

  1   2   3   4   5   6   7   8   9   10   >