[PATCH] rpl_iptunnel: simplify the return expression of rpl_do_srh()

2020-08-18 Thread Xu Wang
Simplify the return expression. Signed-off-by: Xu Wang --- net/ipv6/rpl_iptunnel.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/net/ipv6/rpl_iptunnel.c b/net/ipv6/rpl_iptunnel.c index 5fdf3ebb953f..e58ad9ac987c 100644 --- a/net/ipv6/rpl_iptunnel.c +++

[PATCH 8/8] tty: serial, fix kernel-doc

2020-08-18 Thread Jiri Slaby
As in the previous patches, fix kernel-doc in serial drivers. Signed-off-by: Jiri Slaby --- drivers/tty/serial/ifx6x60.c | 7 +++ drivers/tty/serial/men_z135_uart.c | 8 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/drivers/tty/serial/ifx6x60.c

[PATCH 6/8] tty: vt, fix kernel-doc

2020-08-18 Thread Jiri Slaby
As in the previous patches, fix kernel-doc in vt. Signed-off-by: Jiri Slaby --- drivers/tty/vt/consolemap.c | 2 +- drivers/tty/vt/vt.c | 7 --- drivers/tty/vt/vt_ioctl.c | 3 +-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/tty/vt/consolemap.c

[PATCH 2/8] newport_con: fix no return statement in newport_show_logo

2020-08-18 Thread Jiri Slaby
When CONFIG_LOGO_SGI_CLUT224 is unset, newport_show_logo contains no return, despite it should return a pointer. Add one returning NULL to fix a compiler warning: drivers/video/console/newport_con.c: In function 'newport_show_logo': drivers/video/console/newport_con.c:132:1: warning: no

[PATCH 1/8] tty: n_gsm, eliminate indirection for gsm->{output,error}()

2020-08-18 Thread Jiri Slaby
gsm->output and ->error are set only to gsmld_output and gsm_error, respectively. Call these functions directly and remove error and output function pointers from struct gsm_mux completely. Note: we need a forward declaration of gsmld_output now. Signed-off-by: Jiri Slaby ---

[PATCH 01/16] vt: make vc_data pointers const in selection.h

2020-08-18 Thread Jiri Slaby
There are many functions declared in selection.h which only read from struct vc_data passed as a parameter. Make all those uses const to hint the compiler a bit. Signed-off-by: Jiri Slaby --- drivers/tty/vt/consolemap.c | 2 +- drivers/tty/vt/vt.c | 15 ---

[PATCH 4/8] tty: fix kernel-doc

2020-08-18 Thread Jiri Slaby
With W=1, the kernel-doc checker complains quite a lot in the tty layer. Over the time, many documented parameters were renamed, removed or switched from tty to tty_port and similar. Some were mistyped in the doc too. So fix all these in the tty core. (But do not add the missing ones which the

[PATCH 08/16] vc_screen: extract vcs_write_buf

2020-08-18 Thread Jiri Slaby
This is the counterpart of the previous patch: here, we extract buffer writing with attributes from vcs_write. Now, there is no need for org to be initialized to NULL. The org0 check before update_region() confuses compilers, so check org instead. It provides the same semantics. And it also

[PATCH 09/16] vc_screen: eliminate ifdefs from vcs_write_buf

2020-08-18 Thread Jiri Slaby
Introduce a new inline function called vc_compile_le16 and do the shifts and ORs there. Depending on LE x BE. I tried cpu_to_le16, but it ends up with worse assembly on BE for whatever reason -- the compiler seems to be unable to optimize the swap. Signed-off-by: Jiri Slaby ---

Re: [PATCH V2] ARM: dts: omap3: Add cpu trips and cooling map for omap34/36 families

2020-08-18 Thread Tony Lindgren
* Andreas Kemnade [200817 20:15]: > On Mon, 17 Aug 2020 15:02:27 -0500 > Adam Ford wrote: > > > On Mon, Aug 17, 2020 at 2:59 PM Andreas Kemnade > > wrote: > > > > > > On Mon, 17 Aug 2020 08:39:31 -0500 > > > Adam Ford wrote: > > > > > > > The OMAP3530, OMAP3630, and DM3730 all show

[PATCH 11/16] vs_screen: kill tmp_count from vcs_read

2020-08-18 Thread Jiri Slaby
Both tmp_count computations and the single use can be eliminated using min(). Do so. Side note: we need HEADER_SIZE to be unsigned for min() not to complain. Fix that too as all its other uses do not mind. Signed-off-by: Jiri Slaby --- drivers/tty/vt/vc_screen.c | 10 ++ 1 file

[PATCH 14/16] vc_screen: extract vcs_read_buf

2020-08-18 Thread Jiri Slaby
And finally, move the attributes buffer handling to a separate function. Leaving vcs_read quite compact. Signed-off-by: Jiri Slaby --- drivers/tty/vt/vc_screen.c | 146 - 1 file changed, 78 insertions(+), 68 deletions(-) diff --git

[PATCH 15/16] vc_screen: extract vcs_read_buf_header

2020-08-18 Thread Jiri Slaby
The attribute header handling is terrible in vcs_read_buf. Separate it to a new function and simply do memmove (of up to 4 bytes) to the start of the con_buf -- if user seeked. Signed-off-by: Jiri Slaby --- drivers/tty/vt/vc_screen.c | 31 ++- 1 file changed, 18

[PATCH 12/16] vc_screen: extract vcs_read_buf_uni

2020-08-18 Thread Jiri Slaby
The same as making write more readable, extract unicode handling from vcs_read. The other two cases (w/ and w/o attributes) will follow. Signed-off-by: Jiri Slaby --- drivers/tty/vt/vc_screen.c | 49 +++--- 1 file changed, 30 insertions(+), 19 deletions(-) diff

[PATCH 05/16] vc_screen: rewrite vcs_size to accept vc, not inode

2020-08-18 Thread Jiri Slaby
It is weird to fetch the information from the inode over and over. Read and write already have the needed information, so rewrite vcs_size to accept a vc, attr and unicode and adapt vcs_lseek to that. Also make sure all sites check the return value of vcs_size for errors. And document it using

[PATCH 16/16] vc_screen: prune macros

2020-08-18 Thread Jiri Slaby
Do not undefine random words. I guess this was here as there were macros with such generic names somewhere. I very doubt they still exist. So drop these. And remove a spare blank line. Signed-off-by: Jiri Slaby --- drivers/tty/vt/vc_screen.c | 4 1 file changed, 4 deletions(-) diff --git

[PATCH 07/16] vc_screen: extract vcs_write_buf_noattr

2020-08-18 Thread Jiri Slaby
vcs_write is too long to be readable. Extract buffer handling w/o attributes from there to a separate function. Signed-off-by: Jiri Slaby --- drivers/tty/vt/vc_screen.c | 55 +++--- 1 file changed, 34 insertions(+), 21 deletions(-) diff --git

[PATCH 06/16] vc_screen: sanitize types in vcs_write

2020-08-18 Thread Jiri Slaby
* ret can carry error codes, so make it signed, i.e. ssize_t * pos is derived from the passed ppos, so make it long enough, i.e. loff_t * attr is a boolean, so... * size is limited by vcs_size() which returns an int * written, p, orig_count and this_round are always ">= 0" and "< size", so

[PATCH 02/16] vt: declare xy for get/putconsxy properly

2020-08-18 Thread Jiri Slaby
That is: 1) call the parameter 'xy' to denote what it really is, not generic 'p' 2) tell the compiler and users that we expect an array: * with at least 2 chars (static 2) * which we don't modify in putconsxy (const) Signed-off-by: Jiri Slaby --- drivers/tty/vt/vt.c | 10 +-

[PATCH 10/16] vc_screen: sanitize types in vcs_read

2020-08-18 Thread Jiri Slaby
* pos is derived from the passed ppos, so make it long enough, i.e. loff_t * attr and uni_mode are booleans, so... * size is limited by vcs_size() which returns an int * read, p, orig_count and this_round are always ">= 0" and "< size", so uint is enough * row, col, and max_col are derived

[PATCH 04/16] vc_screen: document and cleanup vcs_vc

2020-08-18 Thread Jiri Slaby
Document parameters of vcs_vc and make viewed a bool. Signed-off-by: Jiri Slaby --- drivers/tty/vt/vc_screen.c | 20 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/drivers/tty/vt/vc_screen.c b/drivers/tty/vt/vc_screen.c index 778f83ea2249..adc3c786b621

[PATCH 3/8] newport_con: make module's init & exit static using module_driver

2020-08-18 Thread Jiri Slaby
The compiler complains that newport_console_init and newport_console_exit are not declared: drivers/video/console/newport_con.c:745:12: warning: no previous prototype for 'newport_console_init' drivers/video/console/newport_con.c:750:13: warning: no previous prototype for

[PATCH 03/16] vc: propagate "viewed as bool" from screenpos up

2020-08-18 Thread Jiri Slaby
viewed is used as a flag, i.e. bool. So treat is as such in most of the places. vcs_vc is handled in the next patch. Note: the last parameter of invert_screen was misnamed in the declaration since 1.1.92. Signed-off-by: Jiri Slaby --- drivers/accessibility/speakup/main.c | 4 ++--

[PATCH 13/16] vc_screen: extract vcs_read_buf_noattr

2020-08-18 Thread Jiri Slaby
Now, move the code for no-attributes handling to a separate function. Signed-off-by: Jiri Slaby --- drivers/tty/vt/vc_screen.c | 33 ++--- 1 file changed, 22 insertions(+), 11 deletions(-) diff --git a/drivers/tty/vt/vc_screen.c b/drivers/tty/vt/vc_screen.c index

[PATCH 7/8] tty: synclink, fix kernel-doc

2020-08-18 Thread Jiri Slaby
As in the previous patches, fix kernel-doc in synclink drivers. This is done separately from others, as kernel-doc comments were heavily broken in these drivers. Convert them to proper ones. Signed-off-by: Jiri Slaby --- drivers/tty/synclink.c| 82 +++---

[PATCH 5/8] tty: ldiscs, fix kernel-doc

2020-08-18 Thread Jiri Slaby
As in the previous patch, fix kernel-doc in line disciplines. Signed-off-by: Jiri Slaby --- drivers/tty/n_gsm.c | 23 +++--- drivers/tty/n_hdlc.c | 72 ++-- drivers/tty/n_tty.c | 4 +-- 3 files changed, 49 insertions(+), 50 deletions(-) diff

RE: [PATCH] Makefile: Yes. Finally remove '-Wdeclaration-after-statement'

2020-08-18 Thread David Laight
> I'm a big fan of -Wdeclaration-after-statement and I think C++ style > mixed variables/statements code has several disadvantages: Agreed. Personally I think declarations should either be either right at the top of a function or in a very small code block. Otherwise they are annoying to find.

Kernel build error on BTFIDS vmlinux

2020-08-18 Thread Jesper Dangaard Brouer
On latest DaveM net-git tree (06a4ec1d9dc652), after linking (LD vmlinux) the "BTFIDS vmlinux" fails. Are anybody else experiencing this? Are there already a fix? (just returned from vacation so not fully up-to-date on ML yet) The tool which is called and error message:

Re: [PATCH 2/2] tasks: Add task_struct addr for lx-ps cmd

2020-08-18 Thread Ritesh Harjani
On 8/18/20 11:10 AM, Jan Kiszka wrote: On 18.08.20 06:04, Ritesh Harjani wrote: task_struct addr in lx-ps cmd seems helpful TASK PIDCOMM 0x82c2b8c0 0 swapper/0 0x888a0ba20040 1 systemd 0x888a0ba24040 2 kthreadd 0x888a0ba28040 3 rcu_gp

[PATCH 0/4] *** SUBJECT HERE ***

2020-08-18 Thread Tomer Samara
*** BLURB HERE *** Tomer Samara (4): staging: android: Replace BUG_ON with WARN_ON staging: android: Add error handling to ion_page_pool_shrink staging: android: Convert BUG to WARN staging: android: Add error handling to order_to_index callers

Re: Re: [PATCH] usb-serial:cp210x: add support to software flow control

2020-08-18 Thread Greg KH
On Thu, Jul 30, 2020 at 04:32:37PM +0800, china_shenglong wrote: > Hi, Greg > > > Yeah, It mainly adjusts the code style according to the kernel specification, > and then modifies it according to your previous review comments. Always include that below the --- line, and version your patches so

[PATCH 0/4] *** SUBJECT HERE ***

2020-08-18 Thread Tomer Samara
*** BLURB HERE *** Tomer Samara (4): staging: android: Replace BUG_ON with WARN_ON staging: android: Add error handling to ion_page_pool_shrink staging: android: Convert BUG to WARN staging: android: Add error handling to order_to_index callers

[PATCH 0/4] *** SUBJECT HERE ***

2020-08-18 Thread Tomer Samara
*** BLURB HERE *** Tomer Samara (4): staging: android: Replace BUG_ON with WARN_ON staging: android: Add error handling to ion_page_pool_shrink staging: android: Convert BUG to WARN staging: android: Add error handling to order_to_index callers

[PATCH] mm/memory.c: Replace vmf->vma with variable vma

2020-08-18 Thread yanfei.xu
From: Yanfei Xu The code has declared a vma_struct named vma which is assigned a value of vmf->vma. Thus, use variable vma directly here. Signed-off-by: Yanfei Xu --- mm/memory.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/memory.c b/mm/memory.c index

Re: [PATCH] mm/memory-failure: do pgoff calculation before for_each_process()

2020-08-18 Thread 堀口 直也
On Tue, Aug 18, 2020 at 04:26:47PM +0800, Xianting Tian wrote: > There is no need to calcaulate pgoff in each loop of for_each_process(), > so move it to the place before for_each_process(), which can save some > CPU cycles. > > Signed-off-by: Xianting Tian Looks good to me. Thank you.

Re: [PATCH v3] usb: renesas-xhci: remove version check

2020-08-18 Thread Greg Kroah-Hartman
On Tue, Aug 18, 2020 at 02:09:19PM +0530, Vinod Koul wrote: > On 18-08-20, 09:31, Greg Kroah-Hartman wrote: > > On Tue, Aug 18, 2020 at 12:47:39PM +0530, Vinod Koul wrote: > > > Some devices in wild are reporting bunch of firmware versions, so remove > > > the check for versions in driver > > > >

[Patch v2] mm/page_reporting: drop stale list head check in page_reporting_cycle

2020-08-18 Thread Wei Yang
list_for_each_entry_safe() guarantees that we will never stumble over the list head; ">lru != list" will always evaluate to true. Let's simplify. Signed-off-by: Wei Yang Reviewed-by: David Hildenbrand --- mm/page_reporting.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH] mm/page_reporting: the "page" must not be the list head

2020-08-18 Thread Wei Yang
On Tue, Aug 18, 2020 at 09:23:12AM +0200, David Hildenbrand wrote: >On 18.08.20 05:05, Wei Yang wrote: >> On Mon, Aug 17, 2020 at 07:07:04PM +0200, David Hildenbrand wrote: >>> On 17.08.20 18:05, Alexander Duyck wrote: On 8/17/2020 2:35 AM, David Hildenbrand wrote: > On 17.08.20

Re: [PATCH v1 0/9] extcon: extcon-ptn5150: Add the USB external connector support

2020-08-18 Thread Andy Shevchenko
On Tue, Aug 18, 2020 at 02:57:18PM +0800, Ramuthevar,Vadivel MuruganX wrote: > USB external connector chip PTN5150 used on the Intel LGM SoC > boards to detect the USB type and connection. Internally I meant you can send cleanups, but couple of patches here are the features and were still under

Re: [PATCH v3 0/2] add regmap-spi-avmm & Intel Max10 BMC chip support

2020-08-18 Thread Xu Yilun
On Mon, Aug 17, 2020 at 10:12:52AM +0100, Lee Jones wrote: > On Mon, 17 Aug 2020, Xu Yilun wrote: > > > Hi Brown & jones: > > > > I tried to refacor the regmap code and add comments in this patchset. I > > made big changes to the rx flow and remove some tricky parts in it. > > > > Would it be

[PATCH bpf] net-tun: add type safety to tun_xdp_to_ptr() and tun_ptr_to_xdp()

2020-08-18 Thread Maciej Żenczykowski
From: Maciej Żenczykowski Test: builds Signed-off-by: Maciej Żenczykowski --- drivers/net/tun.c | 6 +++--- include/linux/if_tun.h | 8 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/net/tun.c b/drivers/net/tun.c index 3c11a77f5709..5dd7f353eeef 100644 ---

Re: [PATCH v7 1/3] devres: provide devm_krealloc()

2020-08-18 Thread Andy Shevchenko
On Mon, Aug 17, 2020 at 10:02:05PM +0200, Bartosz Golaszewski wrote: > On Mon, Aug 17, 2020 at 7:43 PM Andy Shevchenko > wrote: > > On Mon, Aug 17, 2020 at 07:05:33PM +0200, Bartosz Golaszewski wrote: > > > From: Bartosz Golaszewski ... > > > +static struct devres *to_devres(void *data) > > >

Re: [PATCH v3] usb: renesas-xhci: remove version check

2020-08-18 Thread Vinod Koul
On 18-08-20, 09:31, Greg Kroah-Hartman wrote: > On Tue, Aug 18, 2020 at 12:47:39PM +0530, Vinod Koul wrote: > > Some devices in wild are reporting bunch of firmware versions, so remove > > the check for versions in driver > > > > Reported by: Anastasios Vacharakis > > Reported by: Glen Journeay

Re: [PATCH v7 0/2] pwm: intel: Add PWM driver for a new SoC

2020-08-18 Thread Andy Shevchenko
On Tue, Aug 18, 2020 at 01:48:59PM +0800, Rahul Tanwar wrote: > Patch 1 adds dt binding document in YAML format. > Patch 2 add PWM fan controller driver for LGM SoC. > > v7: > - Address code quality related review concerns. > - Rename fan related property to pwm-*. > - Fix one make

Re: [PATCH 3/5] arm64: dts: renesas: r8a774a1: Add PCIe EP nodes

2020-08-18 Thread Sergei Shtylyov
On 18.08.2020 10:23, Lad, Prabhakar wrote: [...] Add PCIe EP nodes to R8A774A1 (RZ/G2M) SoC dtsi. Signed-off-by: Lad Prabhakar Reviewed-by: Biju Das --- arch/arm64/boot/dts/renesas/r8a774a1.dtsi | 38 +++ 1 file changed, 38 insertions(+) diff --git

Re: [PATCH 1/2] dma-mapping: introduce relaxed version of dma sync

2020-08-18 Thread Christoph Hellwig
On Tue, Aug 18, 2020 at 09:28:53AM +0100, Will Deacon wrote: > On Tue, Aug 18, 2020 at 04:43:10PM +0900, Cho KyongHo wrote: > > Cache maintenance operations in the most of CPU architectures needs > > memory barrier after the cache maintenance for the DMAs to view the > > region of the memory

[PATCH v1 2/2] pinctrl: mediatek: make MediaTek MT6765 pinctrl driver support race-free register access

2020-08-18 Thread light.hsieh
From: Light Hsieh This patch make MediaTek MT6765 pinctrl driver support race-free register access Signed-off-by: Light Hsieh --- drivers/pinctrl/mediatek/pinctrl-mt6765.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/pinctrl/mediatek/pinctrl-mt6765.c

[PATCH v1 1/2] pinctrl: mediatek: support access registers without race-condition

2020-08-18 Thread light.hsieh
From: Light Hsieh Some MediaTek SOC provide more control registers other than value register. Generanll, a value register need read-modify-write is at offset 0x0. A corresponding SET register is at offset 0xXXX4. Write 1s' to some bits of SET register will set same bits in value

Re: [PATCH] tty: ipwireless: fix error handling

2020-08-18 Thread Jiri Slaby
On 16. 08. 20, 9:43, Tong Zhang wrote: > ipwireless_send_packet() can only return 0 on success and -ENOMEM on > error, the caller should check non zero for error condition > > Signed-off-by: Tong Zhang > --- > drivers/tty/ipwireless/network.c | 4 ++-- > drivers/tty/ipwireless/tty.c | 2 +-

Re: [PATCH] bitops, kcsan: Partially revert instrumentation for non-atomic bitops

2020-08-18 Thread Marco Elver
On Thu, 13 Aug 2020 at 18:39, Marco Elver wrote: > Previous to the change to distinguish read-write accesses, when > CONFIG_KCSAN_ASSUME_PLAIN_WRITES_ATOMIC=y is set, KCSAN would consider > the non-atomic bitops as atomic. We want to partially revert to this > behaviour, but with one important

[PATCH] mm/memory-failure: do pgoff calculation before for_each_process()

2020-08-18 Thread Xianting Tian
There is no need to calcaulate pgoff in each loop of for_each_process(), so move it to the place before for_each_process(), which can save some CPU cycles. Signed-off-by: Xianting Tian --- mm/memory-failure.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH] net/bluetooth/rfcomm/sock.c: add CAP_NET_RAW check.

2020-08-18 Thread Qingyu Li
When creating a raw PF_BLUETOOTH socket, CAP_NET_RAW needs to be checked first. Signed-off-by: Qingyu Li --- net/bluetooth/rfcomm/sock.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/net/bluetooth/rfcomm/sock.c b/net/bluetooth/rfcomm/sock.c index ae6f80730561..d67d49e5aa00 100644 ---

Re: [PATCH v2] lib/string.c: implement stpcpy

2020-08-18 Thread Joe Perches
On Tue, 2020-08-18 at 08:24 +, David Laight wrote: > From: Nick Desaulniers > > Sent: 17 August 2020 19:37 > .. > > That said, this libcall optimization/transformation (sprintf->stpcpy) > > does look useful to me. > > I'd rather get a cow if I ask for a cow... > > Maybe checkpatch (etc)

Re: [PATCH v3 4/5] binfmt_elf, binfmt_elf_fdpic: Use a VMA list snapshot

2020-08-18 Thread Jann Horn
On Tue, Aug 18, 2020 at 10:18 AM Linus Torvalds wrote: > On Mon, Aug 17, 2020 at 11:13 PM Jann Horn wrote: > > > > /* > > * If this looks like the beginning of a DSO or executable mapping, > > +* we'll check for an ELF header. If we find one, we'll dump the > > first >

Re: [PATCH 1/2] soundwire: add definition for maximum number of ports

2020-08-18 Thread Vinod Koul
On 18-08-20, 06:53, Liao, Bard wrote: > > -Original Message- > > From: Vinod Koul > > Sent: Tuesday, August 18, 2020 2:36 PM > > To: Bard Liao > > Cc: alsa-de...@alsa-project.org; linux-kernel@vger.kernel.org; > > ti...@suse.de; > > broo...@kernel.org; gre...@linuxfoundation.org;

Re: [PATCH 1/2] PM / Domains: Add GENPD_FLAG_SUSPEND_ON flag

2020-08-18 Thread Ulf Hansson
On Mon, 17 Aug 2020 at 18:49, Sibi Sankar wrote: > > On 2020-08-17 14:14, Ulf Hansson wrote: > > On Thu, 13 Aug 2020 at 19:26, Sibi Sankar wrote: > >> > >> On 2020-08-13 18:04, Ulf Hansson wrote: > >> > On Wed, 12 Aug 2020 at 19:03, Sibi Sankar wrote: > >> >> > >> >> Uffe, > >> >> Thanks for

Re: [PATCH 1/2] drm: allow limiting the scatter list size.

2020-08-18 Thread Christian König
Am 18.08.20 um 10:27 schrieb Gerd Hoffmann: On Tue, Aug 18, 2020 at 09:57:59AM +0200, Christian König wrote: Am 18.08.20 um 09:48 schrieb Gerd Hoffmann: Add max_segment argument to drm_prime_pages_to_sg(). When set pass it through to the __sg_alloc_table_from_pages() call, otherwise use

Re: [LKP] Re: [x86/mce] 1de08dccd3: will-it-scale.per_process_ops -14.1% regression

2020-08-18 Thread Feng Tang
Hi Borislav, On Sat, Apr 25, 2020 at 03:01:36PM +0200, Borislav Petkov wrote: > On Sat, Apr 25, 2020 at 07:44:14PM +0800, kernel test robot wrote: > > Greeting, > > > > FYI, we noticed a -14.1% regression of will-it-scale.per_process_ops due to > > commit: > > > > > > commit:

Re: [PATCH 0/3] Simplify PM for am3/4, drop RTC pdata for am3/4/dra7

2020-08-18 Thread Tony Lindgren
Hi Santosh, * Tony Lindgren [200703 19:08]: > Hi all, > > Here are patches to simplify the RTC+DDR PM code for am3 and am4. We want > to do this to drop the RTC related legacy platform data for am3 and am4. > We also drop RTC legacy platform data for dra7. > > Please test the RTC+DDR suspend

Re: [PATCH 1/2] dma-mapping: introduce relaxed version of dma sync

2020-08-18 Thread Will Deacon
On Tue, Aug 18, 2020 at 04:43:10PM +0900, Cho KyongHo wrote: > Cache maintenance operations in the most of CPU architectures needs > memory barrier after the cache maintenance for the DMAs to view the > region of the memory correctly. The problem is that memory barrier is > very expensive and

Re: [PATCH] f2fs: fix indefinite loop scanning for free nid

2020-08-18 Thread Chao Yu
On 2020/8/14 16:05, Sahitya Tummala wrote: If the sbi->ckpt->next_free_nid is not NAT block aligned and if there are free nids in that NAT block between the start of the block and next_free_nid, then those free nids will not be scanned in scan_nat_page(). This results into mismatch between

Re: [PATCH 1/2] drm: allow limiting the scatter list size.

2020-08-18 Thread Gerd Hoffmann
On Tue, Aug 18, 2020 at 09:57:59AM +0200, Christian König wrote: > Am 18.08.20 um 09:48 schrieb Gerd Hoffmann: > > Add max_segment argument to drm_prime_pages_to_sg(). When set pass it > > through to the __sg_alloc_table_from_pages() call, otherwise use > > SCATTERLIST_MAX_SEGMENT. > > > > Also

Re: [PATCHv4 0/6] Add initial genpd support for omap PRM driver

2020-08-18 Thread Tony Lindgren
* santosh.shilim...@oracle.com [200817 15:25]: > On 8/16/20 11:53 PM, Tony Lindgren wrote: > > Hi Santosh, > > > > * Tony Lindgren [200702 18:46]: > > > Hi all, > > > > > > Here's v4 set of patches to add genpd support to the PRM (Power and Reset > > > Module) driver. > > > > > > Initially we

Re: [PATCH] n_gsm: Fix write handling for zero bytes written

2020-08-18 Thread Jiri Slaby
On 17. 08. 20, 15:54, Tony Lindgren wrote: > If write returns zero we currently end up removing the message > from the queue. Instead of removing the message, we want to just > break out of the loop just like we already do for error codes. When exactly does the only writer (gsmld_output) return

Re: [PATCH 16/16] memory: samsung: exynos5422-dmc: Correct white space issues

2020-08-18 Thread Lukasz Luba
On 7/24/20 7:23 PM, Krzysztof Kozlowski wrote: Remove unneeded blank line and align indentation with open parenthesis. Signed-off-by: Krzysztof Kozlowski --- drivers/memory/samsung/exynos5422-dmc.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

Re: [PATCH v5 1/3] net: introduce helper sendpage_ok() in include/linux/net.h

2020-08-18 Thread Coly Li
On 2020/8/18 16:08, Eric Dumazet wrote: > > > On 8/16/20 12:08 AM, Coly Li wrote: >> The original problem was from nvme-over-tcp code, who mistakenly uses >> kernel_sendpage() to send pages allocated by __get_free_pages() without >> __GFP_COMP flag. Such pages don't have refcount (page_count is

[PATCH] net/bluetooth/hidp/sock.c: add CAP_NET_RAW check.

2020-08-18 Thread Qingyu Li
When creating a raw PF_BLUETOOTH socket, CAP_NET_RAW needs to be checked first. Signed-off-by: Qingyu Li --- net/bluetooth/hidp/sock.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/net/bluetooth/hidp/sock.c b/net/bluetooth/hidp/sock.c index 595fb3c9d6c3..3dd9c8e0419a 100644 ---

[PATCH] libbpf: simplify the return expression of build_map_pin_path()

2020-08-18 Thread Xu Wang
Simplify the return expression. Signed-off-by: Xu Wang --- tools/lib/bpf/libbpf.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/tools/lib/bpf/libbpf.c b/tools/lib/bpf/libbpf.c index 5055e1531e43..b423fdaae0b6 100644 --- a/tools/lib/bpf/libbpf.c +++

Re: linux-next: manual merge of the fsinfo tree with the kbuid tree

2020-08-18 Thread David Howells
Stephen Rothwell wrote: > This is now a conflict between the fsinfo tree and Linus' tree. Please drop the fsinfo branch for now, thanks. David

Re: [RESEND PATCH] media: camss: vfe: Use trace_printk for debugging only

2020-08-18 Thread Nicolas Boichat
On Fri, Jul 10, 2020 at 2:48 PM Nicolas Boichat wrote: > > trace_printk should not be used in production code. Since > tracing interrupts is presumably latency sensitive, pr_dbg is > not appropriate, so guard the call with a preprocessor symbol > that can be defined for debugging purpose. > >

Re: [PATCH v3 4/5] binfmt_elf, binfmt_elf_fdpic: Use a VMA list snapshot

2020-08-18 Thread Linus Torvalds
On Mon, Aug 17, 2020 at 11:13 PM Jann Horn wrote: > > /* > * If this looks like the beginning of a DSO or executable mapping, > +* we'll check for an ELF header. If we find one, we'll dump the first > +* page to aid in determining what was mapped here. > +

[PATCH v4] checkpatch: add --kconfig-prefix

2020-08-18 Thread Jerome Forissier
Kconfig allows to customize the CONFIG_ prefix via the $CONFIG_ environment variable. Out-of-tree projects may therefore use Kconfig with a different prefix, or they may use a custom configuration tool which does not use the CONFIG_ prefix at all. Such projects may still want to adhere to the

[PATCH] net/bluetooth/cmtp/sock.c: add CAP_NET_RAW check.

2020-08-18 Thread Qingyu Li
When creating a raw PF_BLUETOOTH socket, CAP_NET_RAW needs to be checked first. Signed-off-by: Qingyu Li --- net/bluetooth/cmtp/sock.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/net/bluetooth/cmtp/sock.c b/net/bluetooth/cmtp/sock.c index 96d49d9fae96..476ccc9bee7c 100644 ---

Re: [PATCH v3 3/3] selinux: add permission names to trace event

2020-08-18 Thread peter enderborg
On 8/17/20 10:16 PM, Stephen Smalley wrote: > On 8/17/20 1:07 PM, Thiébaud Weksteen wrote: > >> From: Peter Enderborg >> >> In the print out add permissions, it will look like: >> <...>-1042  [007]    201.965142: selinux_audited: >> requested=0x400 denied=0x400

Re: [PATCH v5 1/3] net: introduce helper sendpage_ok() in include/linux/net.h

2020-08-18 Thread Eric Dumazet
On 8/16/20 12:08 AM, Coly Li wrote: > The original problem was from nvme-over-tcp code, who mistakenly uses > kernel_sendpage() to send pages allocated by __get_free_pages() without > __GFP_COMP flag. Such pages don't have refcount (page_count is 0) on > tail pages, sending them by

Re: [PATCH v2 3/9] ASoC: audio-graph: Identify 'no_pcm' DAI links for DPCM

2020-08-18 Thread Sameer Pujar
Hi Kuninori, On 8/18/2020 8:11 AM, Kuninori Morimoto wrote: External email: Use caution opening links or attachments Hi Sameer PCM devices are created for FE dai links with 'no-pcm' flag as '0'. Such DAI links have CPU component which implement either pcm_construct() or pcm_new() at

Re: [PATCH v3] checkpatch: add --kconfig-prefix

2020-08-18 Thread Joe Perches
On Tue, 2020-08-18 at 09:50 +0200, Jerome Forissier wrote: > > On 8/18/20 9:43 AM, Jerome Forissier wrote: > > Kconfig allows to customize the CONFIG_ prefix via the $CONFIG_ > > environment variable. Out-of-tree projects may therefore use Kconfig > > with a different prefix, or they may use a

[PATCH] net/bluetooth/bnep/sock.c: add CAP_NET_RAW check.

2020-08-18 Thread Qingyu Li
When creating a raw PF_BLUETOOTH socket, CAP_NET_RAW needs to be checked first. Signed-off-by: Qingyu Li --- net/bluetooth/bnep/sock.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/net/bluetooth/bnep/sock.c b/net/bluetooth/bnep/sock.c index d515571b2afb..e06787a3b5ce 100644 ---

[PATCH] Remove unneeded variable t1

2020-08-18 Thread Xu Wang
Remove unneeded variable t1 seed_encrypt() and seed_decrypt(). Signed-off-by: Xu Wang --- crypto/seed.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/crypto/seed.c b/crypto/seed.c index 5e3bef3a617d..69b3058d6a32 100644 --- a/crypto/seed.c +++ b/crypto/seed.c @@ -366,7

Re: [PATCH v2 3/9] ASoC: audio-graph: Identify 'no_pcm' DAI links for DPCM

2020-08-18 Thread Sameer Pujar
Hi Kuninori, On 8/18/2020 10:53 AM, Kuninori Morimoto wrote: External email: Use caution opening links or attachments Hi again PCM devices are created for FE dai links with 'no-pcm' flag as '0'. Such DAI links have CPU component which implement either pcm_construct() or pcm_new() at

Re: [PATCH v3] checkpatch: add --kconfig-prefix

2020-08-18 Thread Joe Perches
On Tue, 2020-08-18 at 09:43 +0200, Jerome Forissier wrote: > Kconfig allows to customize the CONFIG_ prefix via the $CONFIG_ > environment variable. Out-of-tree projects may therefore use Kconfig > with a different prefix, or they may use a custom configuration tool > which does not use the

drivers/media/i2c/max9286.c:1033:6: error: 'struct gpio_chip' has no member named 'of_node'

2020-08-18 Thread kernel test robot
-randconfig-p002-20200818 (attached as .config) compiler: nios2-linux-gcc (GCC) 9.3.0 reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross git checkout

Re: [PATCH 1/2] drm: allow limiting the scatter list size.

2020-08-18 Thread Christian König
Am 18.08.20 um 09:48 schrieb Gerd Hoffmann: Add max_segment argument to drm_prime_pages_to_sg(). When set pass it through to the __sg_alloc_table_from_pages() call, otherwise use SCATTERLIST_MAX_SEGMENT. Also add max_segment field to gem objects and pass it to drm_prime_pages_to_sg() calls in

[PATCH] net/bluetooth/hci_sock.c: add CAP_NET_RAW check.

2020-08-18 Thread Qingyu Li
When creating a raw PF_BLUETOOTH socket, CAP_NET_RAW needs to be checked first. Signed-off-by: Qingyu Li --- net/bluetooth/hci_sock.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/net/bluetooth/hci_sock.c b/net/bluetooth/hci_sock.c index 251b9128f530..c0919e209f05 100644 ---

[char-misc V5] mei: hdcp: fix mei_hdcp_verify_mprime() input parameter

2020-08-18 Thread Tomas Winkler
wired_cmd_repeater_auth_stream_req_in has a variable length array at the end. we use struct_size() overflow macro to determine the size for the allocation and sending size. This also fixes bug in case number of streams is > 0 in the original submission. This bug was not triggered as the number of

Re: [PATCH v2 0/5] Add support of SECVIO from SNVS on iMX8q/x

2020-08-18 Thread Franck LENORMAND (OSS)
Hello, Peng was able to do a firt pass of review on my patchset which led to this second version. I hope a maintainer will be able to take a look at this patchset once rested after all the work they did for 5.9. On mar., 2020-07-21 at 17:20 +0200, franck.lenorm...@oss.nxp.com wrote: > From:

[PATCH 1/2] dma-mapping: introduce relaxed version of dma sync

2020-08-18 Thread Cho KyongHo
Cache maintenance operations in the most of CPU architectures needs memory barrier after the cache maintenance for the DMAs to view the region of the memory correctly. The problem is that memory barrier is very expensive and dma_[un]map_sg() and dma_sync_sg_for_{device|cpu}() involves the memory

[PATCH 2/2] arm64: dma-mapping: add relaxed DMA sync

2020-08-18 Thread Cho KyongHo
__dma_[un]map_area() is the implementation of cache maintenance operations for DMA in arm64. 'dsb sy' in the subroutine guarantees the view of given memory area is consistent to all memory observers. So, it is required. However, dma_sync_sg_for_{device|cpu}() and dma_[un]map_sg() calls

Re: [PATCH v3] checkpatch: add --kconfig-prefix

2020-08-18 Thread Jerome Forissier
On 8/18/20 9:43 AM, Jerome Forissier wrote: > Kconfig allows to customize the CONFIG_ prefix via the $CONFIG_ > environment variable. Out-of-tree projects may therefore use Kconfig > with a different prefix, or they may use a custom configuration tool > which does not use the CONFIG_ prefix at

Re: [PATCH v5 3/3] mm/page_alloc: Keep memoryless cpuless node 0 offline

2020-08-18 Thread Srikar Dronamraju
* Michal Hocko [2020-08-18 09:37:12]: > On Tue 18-08-20 09:32:52, David Hildenbrand wrote: > > On 12.08.20 08:01, Srikar Dronamraju wrote: > > > Hi Andrew, Michal, David > > > > > > * Andrew Morton [2020-08-06 21:32:11]: > > > > > >> On Fri, 3 Jul 2020 18:28:23 +0530 Srikar Dronamraju > > >>

[PATCH 1/2] drm: allow limiting the scatter list size.

2020-08-18 Thread Gerd Hoffmann
Add max_segment argument to drm_prime_pages_to_sg(). When set pass it through to the __sg_alloc_table_from_pages() call, otherwise use SCATTERLIST_MAX_SEGMENT. Also add max_segment field to gem objects and pass it to drm_prime_pages_to_sg() calls in drivers and helpers. Signed-off-by: Gerd

[PATCH 2/2] drm/virtio: set max_segment

2020-08-18 Thread Gerd Hoffmann
When initializing gem objects call virtio_max_dma_size() to figure the scatter list limit. Needed to make virtio-gpu work properly with SEV. Signed-off-by: Gerd Hoffmann --- drivers/gpu/drm/virtio/virtgpu_object.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 2/3] dma-buf: heaps: add chunk heap to dmabuf heaps

2020-08-18 Thread Hyesoo Yu
This patch adds support for a chunk heap that allows for buffers that are made up of a list of fixed size chunks taken from a CMA. Chunk sizes are configuratd when the heaps are created. Signed-off-by: Hyesoo Yu --- drivers/dma-buf/heaps/Kconfig | 9 ++ drivers/dma-buf/heaps/Makefile

[PATCH 3/3] dma-heap: Devicetree binding for chunk heap

2020-08-18 Thread Hyesoo Yu
Document devicetree binding for chunk heap on dma heap framework Signed-off-by: Hyesoo Yu --- .../devicetree/bindings/dma-buf/chunk_heap.yaml| 46 ++ 1 file changed, 46 insertions(+) create mode 100644 Documentation/devicetree/bindings/dma-buf/chunk_heap.yaml diff

[PATCH 0/3] Chunk Heap Support on DMA-HEAP

2020-08-18 Thread Hyesoo Yu
These patch series to introduce a new dma heap, chunk heap. That heap is needed for special HW that requires bulk allocation of fixed high order pages. For example, 64MB dma-buf pages are made up to fixed order-4 pages * 1024. The chunk heap uses alloc_pages_bulk to allocate high order page.

[PATCH 1/3] dma-buf: add missing EXPORT_SYMBOL_GPL() for dma heaps

2020-08-18 Thread Hyesoo Yu
The interface of dma heap is used from kernel module to register dma heaps, otherwize we will get compile error. Signed-off-by: Hyesoo Yu --- drivers/dma-buf/dma-heap.c | 2 ++ drivers/dma-buf/heaps/heap-helpers.c | 2 ++ 2 files changed, 4 insertions(+) diff --git

[PATCH] ide/macide: Convert Mac IDE driver to platform driver

2020-08-18 Thread Finn Thain
Add platform devices for the Mac IDE controller variants. Convert the macide module into a platform driver to support two of those variants. For the third, use a generic "pata_platform" driver instead. This enables automatic loading of the appropriate module and begins the process of replacing the

Re: [RFC-PATCH 1/2] mm: Add __GFP_NO_LOCKS flag

2020-08-18 Thread Michal Hocko
On Mon 17-08-20 15:28:03, Paul E. McKenney wrote: > On Mon, Aug 17, 2020 at 10:28:49AM +0200, Michal Hocko wrote: > > On Mon 17-08-20 00:56:55, Uladzislau Rezki wrote: > > [ . . . ] > > > > wget > > >

[PATCH v3] checkpatch: add --kconfig-prefix

2020-08-18 Thread Jerome Forissier
Kconfig allows to customize the CONFIG_ prefix via the $CONFIG_ environment variable. Out-of-tree projects may therefore use Kconfig with a different prefix, or they may use a custom configuration tool which does not use the CONFIG_ prefix at all. Such projects may still want to adhere to the

Re: [PATCH drm/hisilicon 0/4] Use drv_err instead of DRM_ERROR in hibmc driver

2020-08-18 Thread Thomas Zimmermann
Hi Am 18.08.20 um 08:51 schrieb Tian Tao: > patch #1 is using the drv_err instead of DRM_ERROR in hibmc_ttm.c > patch #2 is using the drv_err instead of DRM_ERROR in hibmc_drm_vdac.c > patch #3 is using the drv_err and drm_dbg_atomic instead of DRM_ERROR > and DRM_DEBUG_ATOMIC in hibmc_drm_de.c

<    7   8   9   10   11   12   13   14   >