Re: [RFC 0/3] cpufreq: cppc: Add support for frequency invariance

2020-07-09 Thread Ionela Voinescu
Hi Viresh, I'll put all my comments here for now, as they refer more to the design of the solution. I hope it won't be too repetitive compared to what we previously discussed offline. I understand you want to get additional points of view. On Thursday 09 Jul 2020 at 15:43:32 (+0530), Viresh Kuma

Re: [tip:sched/fifo 44/45] ERROR: modpost: "sched_setscheduler" undefined!

2020-07-09 Thread Peter Zijlstra
On Fri, Jun 19, 2020 at 10:15:51PM +0800, kernel test robot wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git sched/fifo > head: 8b700983de82f79e05b2c1136d6513ea4c9b22c4 > commit: 616d91b68cd56bcb1954b6a5af7d542401fde772 [44/45] sched: Remove > sched_setscheduler*() EX

Re: [PATCH] arm64: topology: Don't support AMU without cpufreq

2020-07-09 Thread Ionela Voinescu
Hey, On Thursday 09 Jul 2020 at 16:10:48 (+0530), Viresh Kumar wrote: [..] > > I agree that this happening is a cornercase and a reason for which > > cpufreq_get_hw_max_freq() was made weak. If some platform has entirely > > firmware driven frequency control, but it enables CONFIG_CPU_FREQ > > (as

Re: [RFC PATCH for 5.8 3/4] rseq: Introduce RSEQ_FLAG_RELIABLE_CPU_ID

2020-07-09 Thread Christian Brauner
On Wed, Jul 08, 2020 at 01:34:48PM -0400, Mathieu Desnoyers wrote: > - On Jul 8, 2020, at 12:22 PM, Christian Brauner > christian.brau...@ubuntu.com wrote: > [...] > > I've been following this a little bit. The kernel version itself doesn't > > really mean anything and the kernel version is im

Re: [PATCH 08/32] usb: typec: tcpm: tcpm: Remove dangling unused 'struct tcpm_altmode_ops'

2020-07-09 Thread Hans de Goede
Hi, On 7/6/20 4:20 PM, Lee Jones wrote: On Mon, 06 Jul 2020, Hans de Goede wrote: Hi, On 7/6/20 3:33 PM, Lee Jones wrote: Looks as though a079973f462a3 ("usb: typec: tcpm: Remove tcpc_config configuration mechanism") pulled out the only use of 'tcpm_altmode_ops' last year. No need to keep i

drivers/net/ethernet/aeroflex/greth.c:362:9: sparse: sparse: incorrect type in argument 2 (different address spaces)

2020-07-09 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 0bddd227f3dc55975e2b8dfa7fc6f959b062a2c7 commit: 670d0a4b10704667765f7d18f7592993d02783aa sparse: use identifiers to define address spaces date: 3 weeks ago config: sparc-randconfig-s032-20200709

Re: [PATCH 1/1] crypto: ux500: hash: Add namespacing to hash_init()

2020-07-09 Thread Herbert Xu
On Mon, Jun 29, 2020 at 01:30:03PM +0100, Lee Jones wrote: > A recent change to the Regulator consumer API (which this driver > utilises) add prototypes for the some suspend functions. These > functions require including header file include/linux/suspend.h. > > The following tree of includes affe

Re: [PATCH v6 4/7] pidfd: Replace open-coded partial receive_fd()

2020-07-09 Thread Christian Brauner
On Wed, Jul 08, 2020 at 11:35:39PM -0700, Kees Cook wrote: > On Tue, Jul 07, 2020 at 02:22:20PM +0200, Christian Brauner wrote: > > So while the patch is correct it leaves 5.6 and 5.7 with a bug in the > > pidfd_getfd() implementation and that just doesn't seem right. I'm > > wondering whether we s

Re: [PATCH] hwrng: ba431 - HW_RANDOM_BA431 should not default to y

2020-07-09 Thread Herbert Xu
On Tue, Jun 30, 2020 at 03:39:41PM +0200, Geert Uytterhoeven wrote: > As HW_RANDOM_BA431 does not have any platform dependency, it should not > default to enabled. > > Fixes: 0289e9be5dc26d84 ("hwrng: ba431 - add support for BA431 hwrng") > Signed-off-by: Geert Uytterhoeven > --- > drivers/char/

[PATCH v3] kvm,x86: Exit to user space in case of page fault error

2020-07-09 Thread Vivek Goyal
Page fault error handling behavior in kvm seems little inconsistent when page fault reports error. If we are doing fault synchronously then we capture error (-EFAULT) returned by __gfn_to_pfn_memslot() and exit to user space and qemu reports error, "error: kvm run failed Bad address". But if we ar

Re: [PATCH] crypto: virtio - constify features[] and id_table[]

2020-07-09 Thread Herbert Xu
On Wed, Jul 01, 2020 at 10:29:36PM +0200, Rikard Falkeborn wrote: > features[] and id_table[] are not modified and can be made const to > allow the compiler to put them in read-only memory. > > Before: >textdata bss dec hex filename > 115342056 160 1375035b6 >

Re: [PATCH 0/5] drivers/char: Constify static variables

2020-07-09 Thread Herbert Xu
On Wed, Jul 01, 2020 at 10:09:45PM +0200, Rikard Falkeborn wrote: > Constify some static variables (mostly structs) that are not modified. > > Rikard Falkeborn (5): > hwrng: bcm2835 - Constify bcm2835_rng_devtype[] > hwrng: nomadik - Constify nmk_rng_ids[] > hwrng: virtio - Constify id_table

Re: [PATCH v5] x86/crypto: aesni: Fix build with LLVM_IAS=1

2020-07-09 Thread Herbert Xu
On Fri, Jul 03, 2020 at 04:32:06PM +0200, Sedat Dilek wrote: > When building with LLVM_IAS=1 means using Clang's Integrated Assembly (IAS) > from LLVM/Clang >= v10.0.1-rc1+ instead of GNU/as from GNU/binutils > I see the following breakage in Debian/testing AMD64: > > :15:74: error: too many posit

Re: [PATCH v3 0/3] crypto: allow users to specify acomp hardware from a desired NUMA node

2020-07-09 Thread Herbert Xu
On Sun, Jul 05, 2020 at 09:18:57PM +1200, Barry Song wrote: > For a typical Linux server, probably there are multiple ZIP modules. > For example, numa node0 has a compressor, numa node2 has a same module. > Some drivers are automatically using the module near the CPU calling > acomp_alloc. > But it

net/sched/act_bpf.c:132:35: sparse: sparse: incorrect type in argument 2 (different address spaces)

2020-07-09 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 0bddd227f3dc55975e2b8dfa7fc6f959b062a2c7 commit: 670d0a4b10704667765f7d18f7592993d02783aa sparse: use identifiers to define address spaces date: 3 weeks ago config: sh-randconfig-s031-20200709 (attached

Re: printk of non NULL terminated strings ?

2020-07-09 Thread Andreas Schwab
On Jul 09 2020, Joakim Tjernlund wrote: > Is there a format (or other function) that lets me > print strings without an \0 terminator using an explicit length arg instead? Use the precision. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510 2552

[PATCH] interconnect: msm8916: Fix buswidth of pcnoc_s nodes

2020-07-09 Thread Georgi Djakov
The buswidth of the pcnoc_s_* nodes is actually not 8, but 4 bytes. Let's fix it. Reported-by: Jun Nie Signed-off-by: Georgi Djakov --- drivers/interconnect/qcom/msm8916.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/interconnect/qcom/msm8916.c b/d

Re: [PATCH] fpga: dfl: pci: add device id for Intel FPGA PAC N3000

2020-07-09 Thread Tom Rix
On 7/9/20 3:14 AM, Wu, Hao wrote: >> On Thu, Jul 09, 2020 at 05:10:49PM +0800, Wu, Hao wrote: Subject: [PATCH] fpga: dfl: pci: add device id for Intel FPGA PAC N3000 Add PCIe Device ID for Intel FPGA PAC N3000. Signed-off-by: Wu Hao Signed-off-by: Xu Yilun Sig

Re: [PATCH v2 2/2] soc: mediatek: add mtk-devapc driver

2020-07-09 Thread Chun-Kuang Hu
Hi, Neal: Neal Liu 於 2020年7月9日 週四 下午5:13寫道: > > MediaTek bus fabric provides TrustZone security support and data > protection to prevent slaves from being accessed by unexpected > masters. > The security violation is logged and sent to the processor for > further analysis or countermeasures. > >

[Linux-kernel-mentees] [PATCH v2] net/bluetooth: Fix slab-out-of-bounds read in hci_extended_inquiry_result_evt()

2020-07-09 Thread Peilin Ye
Check upon `num_rsp` is insufficient. A malformed event packet with a large `num_rsp` number makes hci_extended_inquiry_result_evt() go out of bounds. Fix it. This patch fixes the following syzbot bug: https://syzkaller.appspot.com/bug?id=4bf11aa05c4ca51ce0df86e500fce486552dc8d2 Reported-by

Re: [PATCH 1/1] MAINTAINERS: add Hridya and myself into Android driver maintainers list

2020-07-09 Thread Joel Fernandes
On Thu, Jul 9, 2020 at 4:18 AM Greg KH wrote: > And I thought we were deleting ashmem soon? This seems harder than initially thought. The Android userspace is intertwined with ashmem in various ways (security policies, apps etc). There were various concerns with old apps using /dev/ashmem directl

Re: [PATCH 08/32] usb: typec: tcpm: tcpm: Remove dangling unused 'struct tcpm_altmode_ops'

2020-07-09 Thread Lee Jones
On Thu, 09 Jul 2020, Hans de Goede wrote: > Hi, > > On 7/6/20 4:20 PM, Lee Jones wrote: > > On Mon, 06 Jul 2020, Hans de Goede wrote: > > > > > Hi, > > > > > > On 7/6/20 3:33 PM, Lee Jones wrote: > > > > Looks as though a079973f462a3 ("usb: typec: tcpm: Remove tcpc_config > > > > configuration

Re: [PATCH] sched/fair: handle case of task_h_load() returning 0

2020-07-09 Thread Valentin Schneider
On 02/07/20 15:42, Vincent Guittot wrote: > task_h_load() can return 0 in some situations like running stress-ng > mmapfork, which forks thousands of threads, in a sched group on a 224 cores > system. The load balance doesn't handle this correctly because > env->imbalance never decreases and it w

Re: [printk] 18a2dc6982: ltp.kmsg01.fail

2020-07-09 Thread Sergey Senozhatsky
On (20/07/09 14:25), Petr Mladek wrote: > On Thu 2020-07-09 13:23:07, John Ogness wrote: > > On 2020-07-09, Petr Mladek wrote: > > > I though more about it. IMHO, it will be better to modify > > > prb_first_seq() to do the same cycle as prb_next_seq() > > > and return seq number of the first valid

[PATCH] perf-c2c: Fix the wrong description.

2020-07-09 Thread qianjun
From: qianjun Use L1Miss to replace L1Hit to describe the correct scene Signed-off-by: qianjun --- tools/perf/Documentation/perf-c2c.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf/Documentation/perf-c2c.txt b/tools/perf/Documentation/perf-c2c.txt index 98ef

Re: [PATCH 3/4] i3c: master: svc: Add Silvaco I3C master driver

2020-07-09 Thread kernel test robot
Hi Miquel, I love your patch! Yet something to improve: [auto build test ERROR on robh/for-next] [also build test ERROR on linus/master v5.8-rc4 next-20200709] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use as documented in

linux-next: Signed-off-by missing for commits in the btrfs tree

2020-07-09 Thread Stephen Rothwell
Hi all, Commits 708dd6a863c3 ("btrfs: switch to iomap_dio_rw() for dio") 11b399d70e53 ("iomap: IOMAP_DIO_RWF_NO_STALE_PAGECACHE return if page invalidation fails") b65c4e58e476 ("iomap: Convert wait_for_completion to flags") are missing a Signed-off-by from their committer. -- Cheers, S

Re: [PATCH v6 6/7] seccomp: Introduce addfd ioctl to seccomp user notifier

2020-07-09 Thread Christian Brauner
On Wed, Jul 08, 2020 at 11:12:02PM -0700, Kees Cook wrote: > On Tue, Jul 07, 2020 at 03:30:49PM +0200, Christian Brauner wrote: > > Hm, maybe change that description to sm like: > > > > [...] > > Cool, yeah. Thanks! I've tweaked it a little more > > > > + /* 24 is original sizeof(struct seccomp_

Re: [PATCH 1/1] MAINTAINERS: add Hridya and myself into Android driver maintainers list

2020-07-09 Thread Christian Brauner
On Thu, Jul 09, 2020 at 09:03:13AM -0400, Joel Fernandes wrote: > On Thu, Jul 9, 2020 at 4:18 AM Greg KH wrote: > > And I thought we were deleting ashmem soon? > > This seems harder than initially thought. The Android userspace is > intertwined with ashmem in various ways (security policies, apps

[PATCH] security: selinux: ss: conditional.c fixed a checkpatch warning

2020-07-09 Thread Ethan Edwards
`sizeof buf` changed to `sizeof(buf)` Signed-off-by: Ethan Edwards --- security/selinux/ss/conditional.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/security/selinux/ss/conditional.c b/security/selinux/ss/conditional.c index 0cc7cdd58465..90a2f5927e55 100644 --- a/se

[PATCH V1] mmc: sdhci: Fix potential null pointer access while accessing vqmmc

2020-07-09 Thread Veerabhadrarao Badiganti
Don't access vqmmc regulator handler, if it's already invalidated. Fixes: f870b6d480d3 (mmc: sdhci: Allow platform controlled voltage switching) Reported-by: Dan Carpenter Signed-off-by: Veerabhadrarao Badiganti --- drivers/mmc/host/sdhci.c | 12 +++- 1 file changed, 7 insertions(+), 5

Re: [RESEND PATCH 1/2] fpga: dfl: pci: reduce the scope of variable 'ret'

2020-07-09 Thread Tom Rix
I think a better change is to use the ret variable, like this --- a/drivers/fpga/dfl-pci.c +++ b/drivers/fpga/dfl-pci.c @@ -312,7 +312,7 @@ static int cci_pci_sriov_configure(struct pci_dev *pcidev, int num_vfs)     }     }   -   return num_vfs; +   return ret;  } The exi

Re: [PATCH 00/17] gpiolib: cdev: pre-uAPI v2 cleanups

2020-07-09 Thread Bartosz Golaszewski
On Wed, Jul 8, 2020 at 6:18 AM Kent Gibson wrote: > > This collection of patches provides improvements to or > address minor problems in gpiolib-cdev. > > The majority of the patches (1-7, 9-11) have been pulled directly from > my "gpio: cdev: add uAPI V2" patch set, as they are not related to any

Re: [RESEND PATCH 2/2] fpga: dfl: fix bug in port reset handshake

2020-07-09 Thread Tom Rix
On 7/9/20 1:12 AM, Xu Yilun wrote: > From: Matthew Gerlach > > When putting the port in reset, driver must wait for the soft reset > acknowledgment bit instead of the soft reset bit. > > Fixes: 47c1b19c160f (fpga: dfl: afu: add port ops support) > Signed-off-by: Matthew Gerlach > Signed-off-by:

Re: UART/TTY console deadlock

2020-07-09 Thread Sergey Senozhatsky
On (20/07/08 11:52), Petr Mladek wrote: > > I still believe that this is a false positive. I've no problems with that. The thing that I'm trying to improve here is the fact that from lockdep's point of view there are no false positives. lockdep reports the problem and then waves good bye. It disa

[PATCH v5 2/4] printk: add lockless ringbuffer

2020-07-09 Thread John Ogness
Introduce a multi-reader multi-writer lockless ringbuffer for storing the kernel log messages. Readers and writers may use their API from any context (including scheduler and NMI). This ringbuffer will make it possible to decouple printk() callers from any context, locking, or console constraints.

[PATCH v5 1/4] crash: add VMCOREINFO macro to define offset in a struct declared by typedef

2020-07-09 Thread John Ogness
The existing macro VMCOREINFO_OFFSET() can't be used for structures declared via typedef because "struct" is not part of type definition. Create another macro for this purpose. Signed-off-by: John Ogness Acked-by: Baoquan He Acked-by: Sergey Senozhatsky Reviewed-by: Petr Mladek --- include/l

[PATCH v5 0/4] printk: replace ringbuffer

2020-07-09 Thread John Ogness
Hello, Here is a v5 for the first series to rework the printk subsystem. The v4 is here [0]. This first series only replaces the existing ringbuffer implementation. No locking is removed. The semantics/behavior of printk are kept the same except for a minor optimization that is reverted (patch 3).

[PATCH v5 3/4] Revert "printk: lock/unlock console only for new logbuf entries"

2020-07-09 Thread John Ogness
This reverts commit 3ac37a93fa9217e576bebfd4ba3e80edaaeb2289. This optimization will not apply once the transition to a lockless printk is complete. Rather than porting this optimization through the transition only to remove it anyway, just revert it now to simplify the transition. Signed-off-by:

[PATCH v5 4/4] printk: use the lockless ringbuffer

2020-07-09 Thread John Ogness
Replace the existing ringbuffer usage and implementation with lockless ringbuffer usage. Even though the new ringbuffer does not require locking, all existing locking is left in place. Therefore, this change is purely replacing the underlining ringbuffer. Changes that exist due to the ringbuffer r

Re: [PATCH 0/2] Remove kiocb ki_complete

2020-07-09 Thread Christoph Hellwig
On Thu, Jul 09, 2020 at 12:10:36PM +0100, Matthew Wilcox wrote: > On Thu, Jul 09, 2020 at 11:17:05AM +0100, Christoph Hellwig wrote: > > I really don't like this series at all. If saves a single pointer > > but introduces a complicated machinery that just doesn't follow any > > natural flow. And

Re: printk of non NULL terminated strings ?

2020-07-09 Thread Joakim Tjernlund
On Thu, 2020-07-09 at 14:56 +0200, Andreas Schwab wrote: > CAUTION: This email originated from outside of the organization. Do not click > links or open attachments unless you recognize the sender and know the > content is safe. > > > On Jul 09 2020, Joakim Tjernlund wrote: > > > Is there a fo

[GIT PULL] EFI fixes for v5.8-rc

2020-07-09 Thread Ard Biesheuvel
The following changes since commit 2a55280a3675203496d302463b941834228b9875: efi/libstub: arm: Print CPU boot mode and MMU state at boot (2020-06-17 15:29:11 +0200) are available in the Git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git tags/efi-urgent-for-v5.8-rc

Re: [PATCH 2/7] mailbox: keembay-scmi-mailbox: Add support for Keem Bay mailbox

2020-07-09 Thread Paul Murphy
On 7/9/20 13:23, Daniele Alessandrelli wrote: Hi Sudeep, Thanks for your review. On Wed, 2020-07-08 at 21:34 +0100, Sudeep Holla wrote: On Tue, Jun 16, 2020 at 04:56:08PM +0100, Daniele Alessandrelli wrote: From: Paul Murphy Keem Bay SoC has a ARM trusted firmware-based secure monitor whi

Re: printk of non NULL terminated strings ?

2020-07-09 Thread Rasmus Villemoes
On 09/07/2020 15.26, Joakim Tjernlund wrote: > On Thu, 2020-07-09 at 14:56 +0200, Andreas Schwab wrote: >> CAUTION: This email originated from outside of the organization. Do not >> click links or open attachments unless you recognize the sender and know the >> content is safe. >> >> >> On Jul 09

Re: [PATCH 0/2] Remove kiocb ki_complete

2020-07-09 Thread Matthew Wilcox
On Thu, Jul 09, 2020 at 02:26:11PM +0100, Christoph Hellwig wrote: > On Thu, Jul 09, 2020 at 12:10:36PM +0100, Matthew Wilcox wrote: > > On Thu, Jul 09, 2020 at 11:17:05AM +0100, Christoph Hellwig wrote: > > > I really don't like this series at all. If saves a single pointer > > > but introduces a

WARNING in submit_audio_out_urb/usb_submit_urb

2020-07-09 Thread syzbot
Hello, syzbot found the following crash on: HEAD commit:768a0741 usb: dwc2: gadget: Remove assigned but never used.. git tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing console output: https://syzkaller.appspot.com/x/log.txt?x=1568d11f10 kernel conf

Re: [PATCH] sched/fair: handle case of task_h_load() returning 0

2020-07-09 Thread Dietmar Eggemann
On 08/07/2020 11:47, Vincent Guittot wrote: > On Wed, 8 Jul 2020 at 11:45, Dietmar Eggemann > wrote: >> >> On 02/07/2020 16:42, Vincent Guittot wrote: >>> task_h_load() can return 0 in some situations like running stress-ng >>> mmapfork, which forks thousands of threads, in a sched group on a 224

Re: [PATCH v2] bus: fsl-mc: fix invalid free in fsl_mc_device_add

2020-07-09 Thread Greg KH
On Thu, Jul 09, 2020 at 05:41:15AM -0700, t...@redhat.com wrote: > From: Tom Rix > > clang static analysis flags this error > > fsl-mc-bus.c:695:2: warning: Attempt to free released memory [unix.Malloc] > kfree(mc_dev); > ^ > > The problem block of code is > >

Re: [PATCH v6] xfs: Fix false positive lockdep warning with sb_internal & fs_reclaim

2020-07-09 Thread Christoph Hellwig
Looks good, Reviewed-by: Christoph Hellwig

Re: [PATCH 0/2] Remove kiocb ki_complete

2020-07-09 Thread Pavel Begunkov
On 09/07/2020 16:26, Christoph Hellwig wrote: > On Thu, Jul 09, 2020 at 12:10:36PM +0100, Matthew Wilcox wrote: >> On Thu, Jul 09, 2020 at 11:17:05AM +0100, Christoph Hellwig wrote: >>> I really don't like this series at all. If saves a single pointer >>> but introduces a complicated machinery tha

Re: [mm] 4e2c82a409: ltp.overcommit_memory01.fail

2020-07-09 Thread Qian Cai
On Thu, Jul 09, 2020 at 12:55:54PM +0800, Feng Tang wrote: > On Tue, Jul 07, 2020 at 01:41:20PM +0800, Feng Tang wrote: > > On Tue, Jul 07, 2020 at 12:00:09PM +0800, Huang, Ying wrote: > > > Feng Tang writes: > > > > > > > On Mon, Jul 06, 2020 at 06:34:34AM -0700, Andi Kleen wrote: > > > >> >

Re: [PATCH v2] net: ethernet: mtk_eth_soc: fix mtu warning

2020-07-09 Thread Andrew Lunn
On Thu, Jul 09, 2020 at 07:57:42AM +0200, Frank Wunderlich wrote: > From: René van Dorst > > in recent Kernel-Versions there are warnings about incorrect MTU-Size > like these: > > mt7530 mdio-bus:00: nonfatal error -95 setting MTU on port x > eth0: mtu greater than device maximum > mtk_soc_eth

Re: [PATCH 0/2] Remove kiocb ki_complete

2020-07-09 Thread Matthew Wilcox
On Thu, Jul 09, 2020 at 04:37:59PM +0300, Pavel Begunkov wrote: > On 09/07/2020 16:26, Christoph Hellwig wrote: > > On Thu, Jul 09, 2020 at 12:10:36PM +0100, Matthew Wilcox wrote: > >> On Thu, Jul 09, 2020 at 11:17:05AM +0100, Christoph Hellwig wrote: > >>> I really don't like this series at all.

[RFC/RFT PATCH v2 0/1] arc: add sparsemem support

2020-07-09 Thread Mike Rapoport
From: Mike Rapoport Hi, It took me a while to get back to it, but better late than never :) This patch adds SPARSEMEM support as an alternative to DISCONTIGMEM in the case when there are two DRAM banks populated. I've verified that it boots on nsim, but I could not find a way to make nsim emul

[RFC/RFT PATCH v2 1/1] arc: add sparsemem support

2020-07-09 Thread Mike Rapoport
From: Mike Rapoport Signed-off-by: Mike Rapoport --- arch/arc/Kconfig | 10 ++ arch/arc/include/asm/sparsemem.h | 13 + arch/arc/mm/init.c | 7 ++- 3 files changed, 29 insertions(+), 1 deletion(-) create mode 100644 arch/arc/include/asm/sp

[RFC PATCH v1] arm64: kvm: flush tlbs by range in unmap_stage2_range function

2020-07-09 Thread Zhenyu Ye
Now in unmap_stage2_range(), we unmap a page by the following steps: p*d_clear(); kvm_tlb_flush_vmid_ipa(); # take 2us; kvm_flush_dcache_p*d();# take 0.5us; put_page(); When the range is very large, such as 1G, then unmap_stage2_range() may take more than 500ms

[PATCH v6 1/2] gpio: add GPO driver for PCA9570

2020-07-09 Thread Sungbo Eo
NXP PCA9570 is a 4-bit I2C GPO expander without interrupt functionality. Its ports are controlled only by a data byte without register address. Datasheet: https://www.nxp.com/docs/en/data-sheet/PCA9570.pdf Signed-off-by: Sungbo Eo --- v6: * removed client * re-added mutex * removed template_chip

[ANNOUNCE] util-linux v2.36-rc2

2020-07-09 Thread Karel Zak
The util-linux release v2.36-rc2 is available at http://www.kernel.org/pub/linux/utils/util-linux/v2.36 Feedback and bug reports, as always, are welcomed. Karel -- Karel Zak http://karelzak.blogspot.com

Re: [PATCH] fbtft-bus.c: Removing that prohibited space before ')'

2020-07-09 Thread kernel test robot
Hi K, Thank you for the patch! Yet something to improve: [auto build test ERROR on staging/staging-testing] [also build test ERROR on v5.8-rc4 next-20200709] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use as documented in

[PATCH] irqchip/gic-v4.1: Ensure accessing the correct RD when writing INVALLR

2020-07-09 Thread Zenghui Yu
The GICv4.1 spec tells us that it's CONSTRAINED UNPREDICTABLE to issue a register-based invalidation operation for a vPEID not mapped to that RD, or another RD within the same CommonLPIAff group. To follow this rule, commit f3a059219bc7 ("irqchip/gic-v4.1: Ensure mutual exclusion between vPE affin

Re: [PATCH] sched/fair: handle case of task_h_load() returning 0

2020-07-09 Thread Vincent Guittot
On Thu, 9 Jul 2020 at 15:06, Valentin Schneider wrote: > > > On 02/07/20 15:42, Vincent Guittot wrote: > > task_h_load() can return 0 in some situations like running stress-ng > > mmapfork, which forks thousands of threads, in a sched group on a 224 cores > > system. The load balance doesn't handl

Re: [PATCH 0/2] Remove kiocb ki_complete

2020-07-09 Thread Pavel Begunkov
On 09/07/2020 16:43, Matthew Wilcox wrote: > On Thu, Jul 09, 2020 at 04:37:59PM +0300, Pavel Begunkov wrote: >> On 09/07/2020 16:26, Christoph Hellwig wrote: >>> On Thu, Jul 09, 2020 at 12:10:36PM +0100, Matthew Wilcox wrote: On Thu, Jul 09, 2020 at 11:17:05AM +0100, Christoph Hellwig wrote: >

Re: [PATCH] sched/fair: handle case of task_h_load() returning 0

2020-07-09 Thread Vincent Guittot
On Thu, 9 Jul 2020 at 15:34, Dietmar Eggemann wrote: > > On 08/07/2020 11:47, Vincent Guittot wrote: > > On Wed, 8 Jul 2020 at 11:45, Dietmar Eggemann > > wrote: > >> > >> On 02/07/2020 16:42, Vincent Guittot wrote: > >>> task_h_load() can return 0 in some situations like running stress-ng > >>>

Re: [PATCH 0/2] Remove kiocb ki_complete

2020-07-09 Thread Jens Axboe
On 7/9/20 7:32 AM, Matthew Wilcox wrote: > On Thu, Jul 09, 2020 at 02:26:11PM +0100, Christoph Hellwig wrote: >> On Thu, Jul 09, 2020 at 12:10:36PM +0100, Matthew Wilcox wrote: >>> On Thu, Jul 09, 2020 at 11:17:05AM +0100, Christoph Hellwig wrote: I really don't like this series at all. If sa

[PATCH][next] scsi: cxgb4i: fix dereference of pointer tdata before it is null checked

2020-07-09 Thread Colin King
From: Colin Ian King Currently pointer tdata is being dereferenced on the initialization of pointer skb before tdata is null checked. This could lead to a potential null pointer dereference. Fix this by dereferencing tdata after tdata has been null pointer sanity checked. Addresses-Coverity: ("

Re: [PATCH 0/2] Remove kiocb ki_complete

2020-07-09 Thread Matthew Wilcox
On Thu, Jul 09, 2020 at 04:49:51PM +0300, Pavel Begunkov wrote: > On 09/07/2020 16:43, Matthew Wilcox wrote: > > On Thu, Jul 09, 2020 at 04:37:59PM +0300, Pavel Begunkov wrote: > >> Kanchan, could you take a look if you can hide it in req->cflags? > > > > No, that's not what cflags are for. And b

[PATCH v3 07/14] dt-bindings: clock: add SM8150 QCOM Graphics clock bindings

2020-07-09 Thread Jonathan Marek
Add device tree bindings for graphics clock controller for Qualcomm Technology Inc's SM8150 SoCs. Signed-off-by: Jonathan Marek --- .../devicetree/bindings/clock/qcom,gpucc.yaml | 4 ++- include/dt-bindings/clock/qcom,gpucc-sm8150.h | 33 +++ 2 files changed, 36 insertions(+), 1

[PATCH v3 14/14] arm64: dts: qcom: add sm8250 GPU nodes

2020-07-09 Thread Jonathan Marek
This brings up the GPU. Tested on HDK865 by running vulkan CTS. Signed-off-by: Jonathan Marek --- arch/arm64/boot/dts/qcom/sm8250.dtsi | 143 +++ 1 file changed, 143 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi i

[PATCH v3 12/14] dt-bindings: power: Add missing rpmpd rpmh regulator level

2020-07-09 Thread Jonathan Marek
Add RPMH_REGULATOR_LEVEL_SVS_L0, used by sm8250. Signed-off-by: Jonathan Marek --- include/dt-bindings/power/qcom-rpmpd.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/dt-bindings/power/qcom-rpmpd.h b/include/dt-bindings/power/qcom-rpmpd.h index dc146e44228b..5e61eaf73bdd 100644 -

[PATCH v3 01/14] clk: qcom: gcc: fix sm8150 GPU and NPU clocks

2020-07-09 Thread Jonathan Marek
Fix the parents and set BRANCH_HALT_SKIP. From the downstream driver it should be a 500us delay and not skip, however this matches what was done for other clocks that had 500us delay in downstream. Fixes: f73a4230d5bb ("clk: qcom: gcc: Add GPU and NPU clocks for SM8150") Signed-off-by: Jonathan M

[PATCH v3 11/14] clk: qcom: Add graphics clock controller driver for SM8250

2020-07-09 Thread Jonathan Marek
Add support for the graphics clock controller found on SM8250 based devices. This is initially copied from the downstream kernel, but has been modified to more closely match the upstream sc7180 driver. Signed-off-by: Jonathan Marek --- drivers/clk/qcom/Kconfig| 8 + drivers/clk/qcom/M

[PATCH v3 05/14] clk: qcom: gcc: remove unnecessary vco_table from SM8150

2020-07-09 Thread Jonathan Marek
The fixed alpha pll ops only use it for clamping in round_rate, which is unnecessary. This is consistent with SM8250 GCC not using vco_table. Signed-off-by: Jonathan Marek --- drivers/clk/qcom/gcc-sm8150.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/clk/qcom/gcc-sm8150

[PATCH v3 03/14] clk: qcom: clk-alpha-pll: same regs and ops for trion and lucid

2020-07-09 Thread Jonathan Marek
Fixed ops were already identical, this adds support for non-fixed ops by sharing between trion and lucid. This also changes the names for trion ops to be consistent with the rest. Note LUCID_PCAL_DONE is renamed to TRION_PCAL_DONE because it is wrong for lucid, LUCID_PCAL_DONE should be BIT(27).

[PATCH v3 00/14] Enable GPU for SM8150 and SM8250

2020-07-09 Thread Jonathan Marek
This series adds the missing clock drivers and dts nodes to enable the GPU on both SM8150 and SM8250. Note an extra drm/msm patch [1] is required for SM8250. As noted by Dmitry, GMU init fails with newer firmware, needs this patch [2]. [1] https://patchwork.freedesktop.org/series/78968/ [2] htt

[PATCH v3 08/14] dt-bindings: clock: add SM8250 QCOM Graphics clock bindings

2020-07-09 Thread Jonathan Marek
Add device tree bindings for graphics clock controller for Qualcomm Technology Inc's SM8250 SoCs. Signed-off-by: Jonathan Marek --- .../devicetree/bindings/clock/qcom,gpucc.yaml | 4 ++- include/dt-bindings/clock/qcom,gpucc-sm8250.h | 34 +++ 2 files changed, 37 insertions(+), 1

[PATCH v3 13/14] arm64: dts: qcom: add sm8150 GPU nodes

2020-07-09 Thread Jonathan Marek
This brings up the GPU. Tested on HDK855 by running vulkan CTS. Signed-off-by: Jonathan Marek --- arch/arm64/boot/dts/qcom/sm8150.dtsi | 136 +++ 1 file changed, 136 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/sm8150.dtsi b/arch/arm64/boot/dts/qcom/sm8150.dtsi i

[PATCH v3 09/14] clk: qcom: add common gdsc_gx_do_nothing_enable for gpucc drivers

2020-07-09 Thread Jonathan Marek
All gpucc drivers need this, so move it to common code instead of duplicating it in every gpucc driver. Signed-off-by: Jonathan Marek --- drivers/clk/qcom/gdsc.c | 25 + drivers/clk/qcom/gdsc.h | 1 + drivers/clk/qcom/gpucc-sc7180.c | 27 +

[PATCH v3 10/14] clk: qcom: Add graphics clock controller driver for SM8150

2020-07-09 Thread Jonathan Marek
Add support for the graphics clock controller found on SM8150 based devices. This is initially copied from the downstream kernel, but has been modified to more closely match the upstream sc7180 driver. Signed-off-by: Jonathan Marek --- drivers/clk/qcom/Kconfig| 8 + drivers/clk/qcom/M

[PATCH v3 02/14] clk: qcom: clk-alpha-pll: remove unused/incorrect PLL_CAL_VAL

2020-07-09 Thread Jonathan Marek
0x44 isn't a register offset, it is the value that goes into CAL_L_VAL. Fixes: 548a909597d5 ("clk: qcom: clk-alpha-pll: Add support for Trion PLLs") Signed-off-by: Jonathan Marek --- drivers/clk/qcom/clk-alpha-pll.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/clk/qcom/clk-alpha

[PATCH v3 06/14] dt-bindings: clock: combine qcom,sdm845-gpucc and qcom,sc7180-gpucc

2020-07-09 Thread Jonathan Marek
These two bindings are almost identical, so combine them into one. This will make it easier to add the sm8150 and sm8250 gpucc bindings. Signed-off-by: Jonathan Marek --- ...qcom,sdm845-gpucc.yaml => qcom,gpucc.yaml} | 14 ++-- .../bindings/clock/qcom,sc7180-gpucc.yaml | 74 -

Re: [PATCH 0/2] Remove kiocb ki_complete

2020-07-09 Thread Jens Axboe
On 7/9/20 7:26 AM, Christoph Hellwig wrote: > On Thu, Jul 09, 2020 at 12:10:36PM +0100, Matthew Wilcox wrote: >> On Thu, Jul 09, 2020 at 11:17:05AM +0100, Christoph Hellwig wrote: >>> I really don't like this series at all. If saves a single pointer >>> but introduces a complicated machinery that

[PATCH v3 04/14] clk: qcom: clk-alpha-pll: use the right PCAL_DONE value for lucid pll

2020-07-09 Thread Jonathan Marek
Lucid PCAL_DONE is different from trion. Signed-off-by: Jonathan Marek --- drivers/clk/qcom/clk-alpha-pll.c | 28 ++-- drivers/clk/qcom/clk-alpha-pll.h | 2 +- 2 files changed, 27 insertions(+), 3 deletions(-) diff --git a/drivers/clk/qcom/clk-alpha-pll.c b/drivers/clk/

[PATCH v2] serial: mxs-auart: add missed iounmap() in probe failure and remove

2020-07-09 Thread Chuhong Yuan
This driver calls ioremap() in probe, but it misses calling iounmap() in probe's error handler and remove. Add the missed calls to fix it. Fixes: 47d37d6f94cc ("serial: Add auart driver for i.MX23/28") Signed-off-by: Chuhong Yuan --- Changes in v2: - Use iounmap() instead of devm_ioremap() to

Re: [PATCH v3 4/4] io_uring: add support for zone-append

2020-07-09 Thread Jens Axboe
On 7/9/20 4:15 AM, Christoph Hellwig wrote: > On Sun, Jul 05, 2020 at 03:00:47PM -0600, Jens Axboe wrote: >>> diff --git a/fs/io_uring.c b/fs/io_uring.c >>> index 155f3d8..cbde4df 100644 >>> --- a/fs/io_uring.c >>> +++ b/fs/io_uring.c >>> @@ -402,6 +402,8 @@ struct io_rw { >>> struct kiocb

Re: remove dead bdi congestion leftovers

2020-07-09 Thread Jens Axboe
On 7/8/20 11:32 PM, Christoph Hellwig wrote: > On Wed, Jul 08, 2020 at 05:14:29PM -0600, Jens Axboe wrote: >> On 7/1/20 3:06 AM, Christoph Hellwig wrote: >>> Hi Jens, >>> >>> we have a lot of bdi congestion related code that is left around without >>> any use. This series removes it in preparation

Re: [PATCH v3 4/4] io_uring: add support for zone-append

2020-07-09 Thread Christoph Hellwig
On Thu, Jul 09, 2020 at 07:58:04AM -0600, Jens Axboe wrote: > > We don't actually need any new field at all. By the time the write > > returned ki_pos contains the offset after the write, and the res > > argument to ->ki_complete contains the amount of bytes written, which > > allow us to triviall

Re: remove dead bdi congestion leftovers

2020-07-09 Thread Christoph Hellwig
On Thu, Jul 09, 2020 at 07:58:58AM -0600, Jens Axboe wrote: > On 7/8/20 11:32 PM, Christoph Hellwig wrote: > > On Wed, Jul 08, 2020 at 05:14:29PM -0600, Jens Axboe wrote: > >> On 7/1/20 3:06 AM, Christoph Hellwig wrote: > >>> Hi Jens, > >>> > >>> we have a lot of bdi congestion related code that is

Re: [PATCH 0/2] Remove kiocb ki_complete

2020-07-09 Thread Pavel Begunkov
On 09/07/2020 16:53, Matthew Wilcox wrote: > On Thu, Jul 09, 2020 at 04:49:51PM +0300, Pavel Begunkov wrote: >> On 09/07/2020 16:43, Matthew Wilcox wrote: >>> On Thu, Jul 09, 2020 at 04:37:59PM +0300, Pavel Begunkov wrote: Kanchan, could you take a look if you can hide it in req->cflags? >>> >

Re: [PATCH] hwmon: corsair-cpro: add fan_target

2020-07-09 Thread Guenter Roeck
On Thu, Jul 09, 2020 at 01:24:39PM +0200, Marius Zachmann wrote: > This adds fan_target entries to the corsair-cpro driver. > Reading the attribute from the device does not seem possible, so > it returns the last set value. (same as pwm) > > Furthermore: > - removes unnecessary kernel.h include. >

Re: [PATCH v14 07/20] mm/thp: narrow lru locking

2020-07-09 Thread Alex Shi
在 2020/7/7 下午12:52, Hugh Dickins 写道: > My vote, for dispatch of the series, is to leave this patch in; > but cannot object if consensus were that it should be taken out. hi Johannes & Kirill, What's your comments for this patch or others? I am ok to withdraw this patch if no one like to give m

[PATCH v3 13/14] [DO NOT MERGE] arm64: dts: allwinner: h6: Add GPU OPP table

2020-07-09 Thread Clément Péron
Add an Operating Performance Points table for the GPU to enable Dynamic Voltage & Frequency Scaling on the H6. The voltage range is set with minival voltage set to the target and the maximal voltage set to 1.2V. This allow DVFS framework to work properly on board with fixed regulator. Signed-off-

[PATCH v3 11/14] arm64: defconfig: Enable devfreq cooling device

2020-07-09 Thread Clément Péron
Devfreq cooling device framework is used in Panfrost to throttle GPU in order to regulate its temperature. Enable this driver for ARM64 SoC. Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- arch/arm64/configs/defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm

[PATCH v3 00/14] Add regulator devfreq support to Panfrost

2020-07-09 Thread Clément Péron
Hi, This serie cleans and adds regulator support to Panfrost devfreq. This is mostly based on comment for the freshly introduced lima devfreq. We need to add regulator support because on Allwinner the GPU OPP table defines both frequencies and voltages. First patches [01-07] should not change th

[PATCH v3 03/14] drm/panfrost: don't use pfdevfreq.busy_count to know if hw is idle

2020-07-09 Thread Clément Péron
This use devfreq variable that will be lock with spinlock in future patches. We should either introduce a function to access this one but as devfreq is optional let's just remove it. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- drivers/gpu/drm/panfr

[PATCH v3 14/14] [DO NOT MERGE] arm64: dts: allwinner: force GPU regulator to be always

2020-07-09 Thread Clément Péron
Signed-off-by: Clément Péron --- arch/arm64/boot/dts/allwinner/sun50i-h6-beelink-gs1.dts | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6-beelink-gs1.dts b/arch/arm64/boot/dts/allwinner/sun50i-h6-beelink-gs1.dts index 3f7ceeb1a767..14257f7476b8 100644 -

[PATCH v3 12/14] arm64: dts: allwinner: h6: Add cooling map for GPU

2020-07-09 Thread Clément Péron
Add a simple cooling map for the GPU. Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 22 1 file changed, 22 insertions(+) diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi b/arch/arm64/boot/dts/allwin

[PATCH v3 10/14] drm/panfrost: add regulators to devfreq

2020-07-09 Thread Clément Péron
Some OPP tables specify voltage for each frequency. Devfreq can handle these regulators but they should be get only 1 time to avoid issue and know who is in charge. If OPP table is probe don't init regulator. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron

[PATCH v3 06/14] drm/panfrost: properly handle error in probe

2020-07-09 Thread Clément Péron
Introduce a boolean to know if opp table has been added. With this, we can call panfrost_devfreq_fini() in case of error and release what has been initialised. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- drivers/gpu/drm/panfrost/panfrost_devfreq.c

[PATCH v3 09/14] drm/panfrost: dynamically alloc regulators

2020-07-09 Thread Clément Péron
We will later introduce regulators managed by OPP. Only alloc regulators when it's needed. This also help use to release the regulators only when they are allocated. Reviewed-by: Steven Price Reviewed-by: Alyssa Rosenzweig Signed-off-by: Clément Péron --- drivers/gpu/drm/panfrost/panfrost_dev

<    1   2   3   4   5   6   7   8   9   10   >