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

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: [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 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

[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

[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

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

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_

[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: [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

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

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: [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 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

[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 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. > >

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

[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: 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

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: [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 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 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

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 >

[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] 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/

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 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

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 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

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] 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: [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: [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: [LKP] [x86, sched] 1567c3e346: vm-scalability.median -15.8% regression

2020-07-09 Thread Giovanni Gherdovich
On Tue, 2020-07-07 at 10:58 +0800, Xing Zhengjun wrote: > > On 6/12/2020 4:11 PM, Xing Zhengjun wrote: > > Hi Giovanni, > > > > I test the regression, it still existed in v5.7. Do you have time > > to take a look at this? Thanks. > > > > Ping... > Hello, I haven't sat down to reproduce

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

2020-07-09 Thread trix
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 mc_bus = kzalloc(sizeof(*mc_bus), GFP_KERNEL); if (!mc_bus)

Re: [PATCH] powercap: idle_inject: Fix the comment about play_idle

2020-07-09 Thread Rafael J. Wysocki
On Sun, Jun 21, 2020 at 10:04 AM Frank Lee wrote: > > From: Yangtao Li > > After commit 333cff6c963fbc ("powercap/drivers/idle_inject: Specify > idle state max latency"), we convert to use play_idle_precise() with > max allowed latency to specify the idle state. > > Some function comments still u

[rcu:dev.2020.07.06a] BUILD SUCCESS 3b69dc4d0fad073eb5554dacdccddf6c21577802

2020-07-09 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git dev.2020.07.06a branch HEAD: 3b69dc4d0fad073eb5554dacdccddf6c21577802 lib: Add backtrace_idle parameter to force backtrace of idle CPUs elapsed time: 730m configs tested: 111 configs skipped: 7 The following

Re: [PATCH] freezer: Add unsafe versions of freezable_schedule_timeout_interruptible for NFS

2020-07-09 Thread Rafael J. Wysocki
On Mon, Jul 6, 2020 at 11:57 AM wrote: > > From: He Zhe > > commit 0688e64bc600 ("NFS: Allow signal interruption of NFS4ERR_DELAYed > operations") > introduces nfs4_delay_interruptible which also needs an _unsafe version to > avoid the following call trace for the same reason explained in > comm

printk of non NULL terminated strings ?

2020-07-09 Thread Joakim Tjernlund
Is there a format (or other function) that lets me print strings without an \0 terminator using an explicit length arg instead? Something like: printk("%s...", str, str_len); /* replace ... */ or some other funktion perhaps ? Jocke

Re: [PATCH v4 1/2] linux/bits.h: fix unsigned less than zero warnings

2020-07-09 Thread Herbert Xu
Rikard Falkeborn wrote: > When calling the GENMASK and GENMASK_ULL macros with zero lower bit and > an unsigned unknown high bit, some gcc versions warn due to the > comparisons of the high and low bit in GENMASK_INPUT_CHECK. > > To silence the warnings, only perform the check if both inputs are

[PATCH] powerpc/watchpoint/ptrace: Introduce PPC_DEBUG_FEATURE_DATA_BP_DAWR_ARCH_31

2020-07-09 Thread Ravi Bangoria
PPC_DEBUG_FEATURE_DATA_BP_DAWR_ARCH_31 can be used to determine whether we are running on an ISA 3.1 compliant machine. Which is needed to determine DAR behaviour, 512 byte boundary limit etc. This was requested by Pedro Miraglia Franco de Carvalho for extending watchpoint features in gdb. Note tha

RE: [PATCH] arm64: mm: free unused memmap for sparse memory model that define VMEMMAP

2020-07-09 Thread Song Bao Hua (Barry Song)
> -Original Message- > From: liwei (CM) > Sent: Wednesday, July 8, 2020 7:52 PM > To: Song Bao Hua (Barry Song) ; > catalin.mari...@arm.com; w...@kernel.org > Cc: fengbaopeng ; nsaenzjulie...@suse.de; > steve.cap...@arm.com; r...@linux.ibm.com; > linux-arm-ker...@lists.infradead.org; linu

Re: [PATCH net-next v2 08/12] bridge: mrp: Implement the MRP Interconnect API

2020-07-09 Thread Nikolay Aleksandrov
On 09/07/2020 13:00, Horatiu Vultur wrote: > Thie patch adds support for MRP Interconnect. Similar with the MRP ring, > if the HW can't generate MRP_InTest frames, then the SW will try to > generate them. And if also the SW fails to generate the frames then an > error is return to userspace. > > T

Re: [PATCH v2] genpd: Fix up terminology with parent/child

2020-07-09 Thread Rafael J. Wysocki
On Thu, Jul 9, 2020 at 1:32 AM Kees Cook wrote: > > The genpd infrastructure uses the terms master/slave, but such uses have > no external exposures (not even in Documentation/driver-api/pm/*) and are > not mandated by nor associated with any external specifications. Change > the language used thr

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

2020-07-09 Thread Petr Mladek
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 entry. > > Exactly! > > Here is a patch that do

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

2020-07-09 Thread Daniele Alessandrelli
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 which > > acts > > as the SCP for the p

[PATCH] HARDWARE RANDOM NUMBER GENERATOR CORE: Replace HTTP links with HTTPS ones

2020-07-09 Thread Alexander A. Klimov
Rationale: Reduces attack surface on kernel devs opening the links for MITM as HTTPS traffic is much harder to manipulate. Deterministic algorithm: For each file: If not .svg: For each line: If doesn't contain `\bxmlns\b`: For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:

Re: [PATCH v3] PCI: aardvark: Don't touch PCIe registers if no card connected

2020-07-09 Thread Pali Rohár
On Thursday 09 July 2020 12:35:09 Lorenzo Pieralisi wrote: > On Thu, Jul 02, 2020 at 10:30:36AM +0200, Pali Rohár wrote: > > When there is no PCIe card connected and advk_pcie_rd_conf() or > > advk_pcie_wr_conf() is called for PCI bus which doesn't belong to emulated > > root bridge, the aardvark d

Re: [PATCH v2] cpuidle: change enter_s2idle() prototype

2020-07-09 Thread Rafael J. Wysocki
On Mon, Jul 6, 2020 at 5:13 AM Neal Liu wrote: > > Control Flow Integrity(CFI) is a security mechanism that disallows > changes to the original control flow graph of a compiled binary, > making it significantly harder to perform such attacks. > > init_state_node() assign same function callback to

Re: [PATCH 1/9] timer: Move trigger_dyntick_cpu() to enqueue_timer()

2020-07-09 Thread Anna-Maria Behnsen
Hi Frederic, On Tue, 7 Jul 2020, Frederic Weisbecker wrote: > Consolidate the code by calling trigger_dyntick_cpu() from > enqueue_timer() instead of calling it from all its callers. Looks good, but maybe you could also update the comments in the code (see remarks below)? > Signed-off-by: Frede

Re: [PATCH v1 2/2] romfs: address performance regression since v3.10

2020-07-09 Thread Sven Van Asbroeck
Hello Al, You are the closest I could find to a romfs maintainer. get_maintainer.pl doesn't appear to list any. This attempted performance regression fix didn't generate much feedback (to say the least). It's however a real issue for us when supporting a legacy product where we don't have the lux

RE: [PATCH v4] mm/zswap: move to use crypto_acomp API for hardware acceleration

2020-07-09 Thread Song Bao Hua (Barry Song)
> -Original Message- > From: linux-crypto-ow...@vger.kernel.org > [mailto:linux-crypto-ow...@vger.kernel.org] On Behalf Of Sebastian Andrzej > Siewior > Sent: Thursday, July 9, 2020 7:17 PM > To: Song Bao Hua (Barry Song) > Cc: a...@linux-foundation.org; herb...@gondor.apana.org.au; > da

Re: [PATCH v3 00/21] KVM: Cleanup and unify kvm_mmu_memory_cache usage

2020-07-09 Thread Paolo Bonzini
On 03/07/20 04:35, Sean Christopherson wrote: > The only interesting delta from v2 is that patch 18 is updated to handle > a conflict with arm64's p4d rework. Resolution was straightforward > (famous last words). > > > This series resurrects Christoffer Dall's series[1] to provide a common > MMU

Re: [PATCH] ACPI: Use fallthrough pseudo-keyword

2020-07-09 Thread Rafael J. Wysocki
On Tue, Jul 7, 2020 at 10:04 PM Gustavo A. R. Silva wrote: > > Replace the existing /* fall through */ comments and its variants with > the new pseudo-keyword macro fallthrough[1]. Also, remove unnecessary > fall-through markings when it is the case. > > [1] > https://www.kernel.org/doc/html/late

[PATCH resend v2 6/8] virt: vbox: Add a few new vmmdev request types to the userspace whitelist

2020-07-09 Thread Hans de Goede
Upstream VirtualBox has defined and is using a few new request types for vmmdev requests passed through /dev/vboxguest to the hypervisor. Add the defines for these to vbox_vmmdev_types.h and add add them to the whitelists of vmmdev requests which userspace is allowed to make. BugLink: https://bug

[SchedulerTaskPacking] Small background task packing

2020-07-09 Thread Parth Shah
> A) Name: Small background task packing > B) Target behaviour: All fair task wakeup follows a procedure of finding an idle CPU and waking the task on this idle CPU. There are two major wakeup paths: 1. Slow-path: Wake up the task on an idle CPU which is in the shallowest idle states by searchin

[PATCH resend v2 8/8] virt: vbox: Fix some comments which talk about the "session spinlock"

2020-07-09 Thread Hans de Goede
The session lock is a mutex, not a spinlock, fix the comments to match. Acked-by: Arnd Bergmann Signed-off-by: Hans de Goede --- drivers/virt/vboxguest/vboxguest_core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/virt/vboxguest/vboxguest_core.c b/drivers/vir

[PATCH resend v2 3/8] virt: vbox: Rename guest_caps struct members to set_guest_caps

2020-07-09 Thread Hans de Goede
Rename guest_caps[_tracker] struct members to set_guest_caps[_tracker] this is a preparation patch for adding support for the VBGL_IOCTL_GUEST_CAPS_ACQUIRE ioctl. Acked-by: Arnd Bergmann Signed-off-by: Hans de Goede --- drivers/virt/vboxguest/vboxguest_core.c | 20 ++-- drivers/

[PATCH resend v2 4/8] virt: vbox: Add vbg_set_host_capabilities() helper function

2020-07-09 Thread Hans de Goede
Add vbg_set_host_capabilities() helper function, this is a preparation patch for adding support for the VBGL_IOCTL_GUEST_CAPS_ACQUIRE ioctl. Acked-by: Arnd Bergmann Signed-off-by: Hans de Goede --- drivers/virt/vboxguest/vboxguest_core.c | 79 ++--- 1 file changed, 46 inser

[PATCH resend v2 7/8] virt: vbox: Log unknown ioctl requests as error

2020-07-09 Thread Hans de Goede
Every now and then upstream adds new ioctls without notifying us, log unknown ioctl requests as an error to catch these. Acked-by: Arnd Bergmann Signed-off-by: Hans de Goede --- Changes in v2: - Rate-limit the error logging to avoid userspace potentially DOS-ing the log --- drivers/virt/vboxgue

[PATCH resend v2 5/8] virt: vbox: Add support for the new VBG_IOCTL_ACQUIRE_GUEST_CAPABILITIES ioctl

2020-07-09 Thread Hans de Goede
Add support for the new VBG_IOCTL_ACQUIRE_GUEST_CAPABILITIES ioctl, this is necessary for automatic resizing of the guest resolution to match the VM-window size to work with the new VMSVGA virtual GPU which is now the new default in VirtualBox. BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=

[PATCH resend v2 0/8] virt: vbox: Add support for the new VBG_IOCTL_ACQUIRE_GUEST_CAPABILITIES ioctl

2020-07-09 Thread Hans de Goede
Hi Greg, As requested by Arnd, here is a resend of v2 of my vboxguest driver patch-set, adding support for the new VBG_IOCTL_ACQUIRE_GUEST_CAPABILITIES ioctl, with Arnd's Acked-by added. Can you pick this up and merge it into the next branch of your char/misc tree? Regards, Hans

[PATCH resend v2 2/8] virt: vbox: Fix guest capabilities mask check

2020-07-09 Thread Hans de Goede
Check the passed in capabilities against VMMDEV_GUEST_CAPABILITIES_MASK instead of against VMMDEV_EVENT_VALID_EVENT_MASK. This tightens the allowed mask from 0x7ff to 0x7. Fixes: 0ba002bc4393 ("virt: Add vboxguest driver for Virtual Box Guest integration") Cc: sta...@vger.kernel.org Acked-by: Arn

[PATCH resend v2 1/8] virt: vbox: Fix VBGL_IOCTL_VMMDEV_REQUEST_BIG and _LOG req numbers to match upstream

2020-07-09 Thread Hans de Goede
Until this commit the mainline kernel version (this version) of the vboxguest module contained a bug where it defined VBGL_IOCTL_VMMDEV_REQUEST_BIG and VBGL_IOCTL_LOG using _IOC(_IOC_READ | _IOC_WRITE, 'V', ...) instead of _IO(V, ...) as the out of tree VirtualBox upstream version does. Since the

[PATCH] media: atmel: atmel-isc: rework component offsets

2020-07-09 Thread Eugen Hristev
The component offsets were computed in a negative way: they were subtracted from the actual color component value. So, a higher offset was reducing the component value. This is not really desirable, as the offset is a 2's complements number with 1 bit for sign and 12 value bits, so we would like to

Re: [PATCH] clk: rockchip: mark pclk_uart2 as critical on rk3328

2020-07-09 Thread Johan Jonker
Hi Elaine, Robin, Thank you for your help! This patch can go in the garbage bin. It turns out that with SERIAL_8250 also SERIAL_8250_DW must be selected... ;) It's not in the Kconfig help description. Shouldn't that be automatically be included for Rockchip? Example: config SERIAL_8250 t

[PATCH v2] scsi: fcoe: add missed kfree() in an error path

2020-07-09 Thread Jing Xiangfeng
fcoe_fdmi_info() misses to call kfree() in an error path. Add a label 'free_fdmi' and jump to it. Fixes: f07d46bbc9ba ("fcoe: Fix smatch warning in fcoe_fdmi_info function") Signed-off-by: Jing Xiangfeng --- drivers/scsi/fcoe/fcoe.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff -

Re: [PATCH v4] platform: x86: Add ACPI driver for ChromeOS

2020-07-09 Thread Rafael J. Wysocki
On Wed, Jun 10, 2020 at 11:21 PM Enric Balletbo i Serra wrote: > > Hi Rafael, > > Many thanks for your feedback. See my answers inline. > > On 5/6/20 13:17, Rafael J. Wysocki wrote: > > On Tuesday, April 14, 2020 4:35:38 PM CEST Enric Balletbo i Serra wrote: > >> Hi Rafael, > >> > >> On 13/4/20 22

Re: [PATCH] irqdomain/treewide: Keep firmware node unconditionally allocated

2020-07-09 Thread Marc Zyngier
Hi Thomas, Catching up on email... On 2020-07-09 10:53, Thomas Gleixner wrote: Quite some non OF/ACPI users of irqdomains allocate firmware nodes of type IRQCHIP_FWNODE_NAMED or IRQCHIP_FWNODE_NAMED_ID and free them right after creating the irqdomain. The only purpose of these FW nodes is to

RE: [REPORT] possible circular locking dependency when booting a VM on arm64 host

2020-07-09 Thread Salil Mehta
> From: yuzenghui > Sent: Thursday, July 9, 2020 12:50 PM > To: Salil Mehta > Cc: Marc Zyngier ; Thomas Gleixner ; > Linux > Kernel Mailing List ; > linux-arm-ker...@lists.infradead.org; Zhuangyuzeng (Yisen) > ; Wanghaibin (D) > Subject: Re: [REPORT] possible circular locking dependency when boo

Re: [PATCH v3 0/2] synquacer: add TPM support

2020-07-09 Thread Jarkko Sakkinen
On Thu, Jul 09, 2020 at 08:46:35AM +0900, Masahisa Kojima wrote: > Hi Jarkko, > > > Hi Jakko, > I apologize for mis-spelling of your name. > Same mistake also appears in my another reply to "[PATCH v3 1/2] tpm: > tis: add support for MMIO TPM on SynQuacer" No worries :-) /Jarkko

[PATCH 1/2] firmware: qcom_scm: Add memory protect virtual address ranges

2020-07-09 Thread Stanimir Varbanov
This adds a new SCM memprotect command to set virtual address ranges. Signed-off-by: Stanimir Varbanov --- drivers/firmware/qcom_scm.c | 24 drivers/firmware/qcom_scm.h | 1 + include/linux/qcom_scm.h| 8 +++- 3 files changed, 32 insertions(+), 1 deletion(-) d

[PATCH 2/2] venus: firmware: Set virtual address ranges

2020-07-09 Thread Stanimir Varbanov
In order to boot some of the new Venus firmware versions TZ call to set virtual address ranges is needed. Add virtual address ranges for CP and CP_NONPIX in resource structure and use them when loading and booting the firmware on remote processor. Signed-off-by: Stanimir Varbanov --- drivers/med

Re: [PATCH v4] platform: x86: Add ACPI driver for ChromeOS

2020-07-09 Thread Rafael J. Wysocki
On Thu, Jul 9, 2020 at 11:31 AM Enric Balletbo i Serra wrote: > > Hi Rafael, > > On 11/6/20 13:06, Enric Balletbo i Serra wrote: > > Hi, > > > > On 11/6/20 0:43, Dmitry Torokhov wrote: > >> On Wed, Jun 10, 2020 at 09:52:12PM +, mario.limoncie...@dell.com wrote: > -Original Message

[PATCH 0/2] Venus - fix firmware load failure

2020-07-09 Thread Stanimir Varbanov
Hi, These two patches fixed the problem with "failed to reset venus core" seen with various firmware versions (including the one from linux-firmware) on sdm845 and sdm850. regards, Stan Stanimir Varbanov (2): firmware: qcom_scm: Add memory protect virtual address ranges venus: firmware: Set

[PATCH bpf] selftests: bpf: fix detach from sockmap tests

2020-07-09 Thread Lorenz Bauer
Fix sockmap tests which rely on old bpf_prog_dispatch behaviour. In the first case, the tests check that detaching without giving a program succeeds. Since these are not the desired semantics, invert the condition. In the second case, the clean up code doesn't supply the necessary program fds. Rep

Re: [REPORT] possible circular locking dependency when booting a VM on arm64 host

2020-07-09 Thread Zenghui Yu
On 2020/7/9 18:54, Salil Mehta wrote: Hi Yuzenghui, I will try to reproduce it today at our platform. Just one question is it easily reproducible or is a rare occurrence? Salil, it's 100% reproducible once you start a guest. You don't even need to assign hostdev to the VM. Thanks, Zenghui

Re: [PATCH] kernel/smp: Fix an off by one in csd_lock_wait_toolong()

2020-07-09 Thread Sebastian Andrzej Siewior
On 2020-07-09 12:59:06 [+0200], Peter Zijlstra wrote: > On Thu, Jul 09, 2020 at 01:48:18PM +0300, Dan Carpenter wrote: > > The __per_cpu_offset[] array has "nr_cpu_ids" elements so change the > > > >= to prevent a read one element beyond the end of the array. > > > > Fixes: 0504bc41a62c ("kernel/s

Re: [PATCH v2] kernel: Implement selective syscall userspace redirection

2020-07-09 Thread Matthew Wilcox
On Thu, Jul 09, 2020 at 12:38:40AM -0400, Gabriel Krisman Bertazi wrote: > The proposed interface looks like this: > > prctl(PR_SET_SYSCALL_USER_DISPATCH, , , [selector]) > > Dispatcher is the address of a syscall instruction that is allowed to > by-pass the blockage, such that in fast paths yo

Re: [PATCH v3 0/2] synquacer: add TPM support

2020-07-09 Thread Jarkko Sakkinen
On Thu, Jul 09, 2020 at 08:34:27AM +0900, Masahisa Kojima wrote: > Hi Jakko, > > > Overally the code looks great. You've run it through checkpatch.pl? > > Yes, I have run checkpatch.pl and removed errors. OK, cool. /Jarkko

Re: [PATCH v3] PCI: aardvark: Don't touch PCIe registers if no card connected

2020-07-09 Thread Lorenzo Pieralisi
On Thu, Jul 02, 2020 at 10:30:36AM +0200, Pali Rohár wrote: > When there is no PCIe card connected and advk_pcie_rd_conf() or > advk_pcie_wr_conf() is called for PCI bus which doesn't belong to emulated > root bridge, the aardvark driver throws the following error message: > > advk-pcie d007

Re: [PATCH v4 08/10] net: eth: altera: add support for ptp and timestamping

2020-07-09 Thread Richard Cochran
On Wed, Jul 08, 2020 at 03:23:59PM +0800, Ooi, Joyce wrote: > @@ -222,6 +223,32 @@ static void tse_get_regs(struct net_device *dev, struct > ethtool_regs *regs, > buf[i] = csrrd32(priv->mac_dev, i * 4); > } > > +static int tse_get_ts_info(struct net_device *dev, > +

Re: [PATCH] Replace HTTP links with HTTPS ones: XDP (eXpress Data Path)

2020-07-09 Thread Jesper Dangaard Brouer
On Wed, 8 Jul 2020 20:58:39 +0200 "Alexander A. Klimov" wrote: > Am 08.07.20 um 16:02 schrieb Jonathan Corbet: > > On Wed, 8 Jul 2020 15:57:37 +0200 > > "Alexander A. Klimov" wrote: > > > >> Documentation/arm/ixp4xx.rst | 4 ++-- > >> 1 file changed, 2 insertions(+), 2 deletions(-) > >

[PATCH] hwmon: corsair-cpro: add fan_target

2020-07-09 Thread Marius Zachmann
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. - send_usb_cmd now has one more argument which is needed for the fan_target

Re: [PATCH 2/2] powerpc/powernv/idle: save-restore DAWR0,DAWRX0 for P10

2020-07-09 Thread Pratik Sampat
On 09/07/20 2:39 pm, Gautham R Shenoy wrote: On Fri, Jul 03, 2020 at 06:16:40PM +0530, Pratik Rajesh Sampat wrote: Additional registers DAWR0, DAWRX0 may be lost on Power 10 for stop levels < 4. Adding Ravi Bangoria to the cc. Therefore save the values of these SPRs before entering a "st

Re: [PATCH] spi: spi-geni-qcom: Set the clock properly at runtime resume

2020-07-09 Thread Mark Brown
On Wed, Jul 08, 2020 at 04:39:39PM -0700, Douglas Anderson wrote: > In the patch ("spi: spi-geni-qcom: Avoid clock setting if not needed") > we avoid a whole pile of clock code. As part of that, we should have > restored the clock at runtime resume. Do that. Acked-by: Mark Brown signature.asc

Re: [PATCH 2/2] doc, mm: clarify /proc//oom_score value range

2020-07-09 Thread Yafang Shao
On Thu, Jul 9, 2020 at 5:58 PM Michal Hocko wrote: > > On Thu 09-07-20 17:01:06, Yafang Shao wrote: > > On Thu, Jul 9, 2020 at 4:18 PM Michal Hocko wrote: > > > > > > On Thu 09-07-20 15:41:11, Yafang Shao wrote: > > > > On Thu, Jul 9, 2020 at 2:26 PM Michal Hocko wrote: > > > > > > > > > > From:

Re: [PATCH v2] ALSA: Use fallthrough pseudo-keyword

2020-07-09 Thread Takashi Iwai
On Wed, 08 Jul 2020 22:32:36 +0200, Gustavo A. R. Silva wrote: > > Replace the existing /* fall through */ comments and its variants with > the new pseudo-keyword macro fallthrough[1]. Also, remove unnecessary > fall-through markings when it is the case. > > [1] > https://www.kernel.org/doc/html

Re: [PATCH v5 17/17] x86/perf, static_call: Optimize x86_pmu methods

2020-07-09 Thread Peter Zijlstra
On Wed, Jun 24, 2020 at 05:30:41PM +0200, Peter Zijlstra wrote: > Replace many of the indirect calls with static_call(). > > XXX run performance numbers So I finally got around to doing that, and the average PMI time, as measured by perf_sample_event_took()*: PRE:3283.03 [ns] POST: 3145.12

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

2020-07-09 Thread John Ogness
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 entry. Exactly! Here is a patch that does just that. I added a prb_first_valid_seq() function and made

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

2020-07-09 Thread Petr Mladek
On Thu 2020-07-09 12:59:06, Petr Mladek wrote: > On Thu 2020-07-09 12:20:35, John Ogness wrote: > > On 2020-07-09, Sergey Senozhatsky wrote: > > > On (20/07/09 15:14), kernel test robot wrote: > > > [..] > > > > > > Took me a while to find the FAIL-ed test: > > > > > >> kmsg01.c:393: INFO: TEST: r

Re: [PATCH v5 1/4] riscv: Move kernel mapping to vmalloc zone

2020-07-09 Thread Alex Ghiti
Hi Palmer, Le 7/9/20 à 1:05 AM, Palmer Dabbelt a écrit : On Sun, 07 Jun 2020 00:59:46 PDT (-0700), a...@ghiti.fr wrote: This is a preparatory patch for relocatable kernel. The kernel used to be linked at PAGE_OFFSET address and used to be loaded physically at the beginning of the main memory.

Re: [Ksummit-discuss] [PATCH] CodingStyle: Inclusive Terminology

2020-07-09 Thread Mauro Carvalho Chehab
Em Mon, 06 Jul 2020 06:30:01 -0700 Joe Perches escreveu: > On Mon, 2020-07-06 at 09:04 -0400, Matthew Wilcox wrote: > > On Mon, Jul 6, 2020 at 8:59 AM Joe Perches wrote: > > > On Mon, 2020-07-06 at 08:51 -0400, Matthew Wilcox wrote: > > > > In terms of number of lines of code using the word,

Re: [PATCH 2/2] KVM: PPC: Book3S HV: rework secure mem slot dropping

2020-07-09 Thread Michael Ellerman
Laurent Dufour writes: > Le 08/07/2020 à 13:25, Bharata B Rao a écrit : >> On Fri, Jul 03, 2020 at 05:59:14PM +0200, Laurent Dufour wrote: >>> When a secure memslot is dropped, all the pages backed in the secure device >>> (aka really backed by secure memory by the Ultravisor) should be paged out

Re: [PATCH 0/2] Remove kiocb ki_complete

2020-07-09 Thread Matthew Wilcox
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 there doesn't seem to be any good reason for it to > start with. Jens d

Re: [PATCH v4 2/2] Add Intel LGM soc DMA support.

2020-07-09 Thread Andy Shevchenko
On Thu, Jul 09, 2020 at 02:01:06PM +0800, Amireddy Mallikarjuna reddy wrote: > Add DMA controller driver for Lightning Mountain(LGM) family of SoCs. > > The main function of the DMA controller is the transfer of data from/to any > DPlus compliant peripheral to/from the memory. A memory to memory c

Re: [PATCH v2] dt-bindings: media: venus: Add an optional power domain for perf voting

2020-07-09 Thread Rajendra Nayak
On 7/1/2020 4:50 PM, Rajendra Nayak wrote: On 6/1/2020 11:26 AM, Rajendra Nayak wrote: On 5/28/2020 1:06 AM, Rob Herring wrote: On Wed, May 13, 2020 at 11:33:27AM +0530, Rajendra Nayak wrote: Add an optional power domain which when specified can be used for setting the performance state of

RE: [PATCH v1 0/2] scsi: ufs: Fix and simplify setup_xfer_req vop and request's completion timestamp

2020-07-09 Thread Avri Altman
Looks good to me. Thanks, Avri > > Hi, > This small series fixes and simplifies setup_xfer_req vop and request's > completion timestamp. > > Stanley Chu (2): > scsi: ufs: Simplify completion timestamp for SCSI and query commands > scsi: ufs: Fix and simplify setup_xfer_req variant operatio

<    5   6   7   8   9   10   11   12   13   14   >