Re: [PATCH v2] tpm_tis: Remove the HID IFX0102

2020-07-02 Thread Jarkko Sakkinen
On Fri, Jun 26, 2020 at 07:36:31AM -0700, James Bottomley wrote: > On Fri, 2020-06-26 at 16:15 +0300, Jarkko Sakkinen wrote: > > I have an obstacle with that. > > > > I lost my previous PGP key a year ago and created a new one, which is > > not trusted yet by anyone [*]. I've backed this up now an

[PATCH v5 5/5] scsi: ufs: Prepare HPB read for cached sub-region

2020-07-02 Thread Daejun Park
This patch changes the read I/O to the HPB read I/O. If the logical address of the read I/O belongs to active sub-region, the HPB driver modifies the read I/O command to HPB read. It modifies the upiu command of UFS instead of modifying the existing SCSI command. In the HPB version 1.0, the maxim

[PATCH v5 4/5] scsi: ufs: L2P map management for HPB read

2020-07-02 Thread Daejun Park
This is a patch for managing L2P map in HPB module. The HPB divides logical addresses into several regions. A region consists of several sub-regions. The sub-region is a basic unit where L2P mapping is managed. The driver loads L2P mapping data of each sub-region. The loaded sub-region is called a

[PATCH v5 3/5] scsi: ufs: Introduce HPB module

2020-07-02 Thread Daejun Park
This is a patch for the HPB module. The HPB module queries UFS for device information during initialization. We added the export symbol to two functions in ufshcd.c to initialize the HPB module. The HPB module can be loaded or built-in as needed. The mininum size of the memory pool used in the HPB

[PATCH v5 2/5] scsi: ufs: Add UFS-feature layer

2020-07-02 Thread Daejun Park
This patch is adding UFS feature layer to UFS core driver. UFS Driver data structure (struct ufs_hba) │ ┌--┐ │ UFS feature │ <-- HPB module │layer │ <-- other extended feature module └--┘ Each extended UFS-Feature module has a bus of ufs-ext feature type. T

[PATCH 0/5] Refactor kallsyms_show_value() users for correct cred

2020-07-02 Thread Kees Cook
Hi, I'm not sure who should carry this tree (me? Greg? akpm? Linus?), but it fixes a kernel address exposure bug reported by Dominik Czarnota, where /sys/modules/*/sections/* contents were visible to uid-0 without CAP_SYSLOG (e.g. in containers): This is correct, with CAP_SYSLOG: # cat /sys/modu

[PATCH 1/5] kallsyms: Refactor kallsyms_show_value() to take cred

2020-07-02 Thread Kees Cook
In order to perform future tests against the cred saved during open(), switch kallsyms_show_value() to operate on a cred, and have all current callers pass current_cred(). This makes it very obvious where callers are checking the wrong credential in their "read" contexts. These will be fixed in the

[PATCH v2] vmlinux.lds.h: Coalesce transient LLVM dead code elimination sections

2020-07-02 Thread Danny Lin
A recent LLVM 11 commit [1] made LLD stop implicitly coalescing some temporary LLVM sections, namely .{data,bss}..compoundliteral.XXX: [30] .data..compoundli PROGBITS 9ac9a000 19e9a000 cea0 WA 0 0 32 [31] .rela.data..compo RE

Re: [PATCH v2 1/4] x86/xen: remove 32-bit Xen PV guest support

2020-07-02 Thread Boris Ostrovsky
On 7/2/20 7:24 PM, Andrew Cooper wrote: > On 02/07/2020 23:59, Boris Ostrovsky wrote: >> On 7/1/20 7:06 AM, Juergen Gross wrote: >>> >>> -#ifdef CONFIG_X86_PAE >>> -static void xen_set_pte_atomic(pte_t *ptep, pte_t pte) >>> -{ >>> - trace_xen_mmu_set_pte_atomic(ptep, pte); >>> - __xen_set_pte

[PATCH 3/5] module: Do not expose section addresses to non-CAP_SYSLOG

2020-07-02 Thread Kees Cook
The printing of section addresses in /sys/module/*/sections/* was not using the correct credentials to evaluate visibility. Before: # cat /sys/module/*/sections/.*text 0xc0458000 ... # capsh --drop=CAP_SYSLOG -- -c "cat /sys/module/*/sections/.*text" 0xc0458000 ... After:

[PATCH 2/5] module: Refactor section attr into bin attribute

2020-07-02 Thread Kees Cook
In order to gain access to the open file's f_cred for kallsym visibility permission checks, refactor the module section attributes to use the bin_attribute instead of attribute interface. Additionally removes the redundant "name" struct member. Cc: sta...@vger.kernel.org Signed-off-by: Kees Cook

[PATCH 4/5] kprobes: Do not expose probe addresses to non-CAP_SYSLOG

2020-07-02 Thread Kees Cook
The kprobe show() functions were using "current"'s creds instead of the file opener's creds for kallsyms visibility. Fix to use seq_file->file->f_cred. Cc: sta...@vger.kernel.org Fixes: 81365a947de4 ("kprobes: Show address of kprobes if kallsyms does") Fixes: ffb9bd68ebdb ("kprobes: Show blacklist

[PATCH 5/5] bpf: Check correct cred for CAP_SYSLOG in bpf_dump_raw_ok()

2020-07-02 Thread Kees Cook
When evaluating access control over kallsyms visibility, credentials at open() time need to be used, not the "current" creds (though in BPF's case, this has likely always been the same). Plumb access to associated file->f_cred down through bpf_dump_raw_ok() and its callers now that kallsysm_show_va

Re: [PATCH 01/16] init: remove the bstat helper

2020-07-02 Thread Song Liu
Hi Christoph, On Mon, Jun 15, 2020 at 5:53 AM Christoph Hellwig wrote: > > The only caller of the bstat function becomes cleaner and simpler when > open coding the function. > > Signed-off-by: Christoph Hellwig Thanks for the set. md parts of the set look good to me. How should we route this s

[PATCH v5 1/5] scsi: ufs: Add UFS feature related parameter

2020-07-02 Thread Daejun Park
This is a patch for parameters to be used for UFS features layer and HPB module. Signed-off-by: Daejun Park --- drivers/scsi/ufs/ufs.h | 12 1 file changed, 12 insertions(+) diff --git a/drivers/scsi/ufs/ufs.h b/drivers/scsi/ufs/ufs.h index f8ab16f30fdc..ae557b8d3eba 100644 --- a/d

Re: [PATCH v2 1/4] x86/xen: remove 32-bit Xen PV guest support

2020-07-02 Thread Andrew Cooper
On 02/07/2020 23:59, Boris Ostrovsky wrote: > On 7/1/20 7:06 AM, Juergen Gross wrote: >> >> -#ifdef CONFIG_X86_PAE >> -static void xen_set_pte_atomic(pte_t *ptep, pte_t pte) >> -{ >> -trace_xen_mmu_set_pte_atomic(ptep, pte); >> -__xen_set_pte(ptep, pte); > > Probably not for this series b

Re: weird loadavg on idle machine post 5.7

2020-07-02 Thread Dave Jones
On Thu, Jul 02, 2020 at 10:36:27PM +0100, Mel Gorman wrote: > I'm thinking that the !!task_contributes_to_load(p) should still happen > after smp_cond_load_acquire() when on_cpu is stable and the pi_lock is > held to stabilised p->state against a parallel wakeup or updating the > task rq. I d

[PATCH v5 0/5] scsi: ufs: Add Host Performance Booster Support

2020-07-02 Thread Daejun Park
Changelog: v4 -> v5 Delete unused macro define. v3 -> v4 1. Cleanup. v2 -> v3 1. Add checking input module parameter value. 2. Change base commit from 5.8/scsi-queue to 5.9/scsi-queue. 3. Cleanup for unused variables and label. v1 -> v2 1. Change the full boilerplate text to SPDX style. 2. Adop

Re: [PATCH v3] x86/speculation/l1tf: Add KConfig for setting the L1D cache flush mode

2020-07-02 Thread Randy Dunlap
On 7/2/20 3:12 PM, Abhishek Bhardwaj wrote: > This change adds a new kernel configuration that sets the l1d cache > flush setting at compile time rather than at run time. > > Signed-off-by: Abhishek Bhardwaj > > --- > > Changes in v3: > - Change depends on to only x86_64. > - Remove copy paste

[PATCH 06/15] options: move helpers up

2020-07-02 Thread Luc Van Oostenryck
The helpers for parsing the options are often situated just above the first function using them. As result, these helpers can be found a bit everywhere in the code, it's messy and doesn't help to reuse these helpers. So, move all these helpers to the top. Signed-off-by: Luc Van Oostenryck --- l

[PATCH 05/15] options: handle_onoff_switch() can handle any flags, not only warnings

2020-07-02 Thread Luc Van Oostenryck
So, use 'flag' instead of 'warning' for variable and function names. Signed-off-by: Luc Van Oostenryck --- lib.c | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/lib.c b/lib.c index 9acdc60fc416..c27773097127 100644 --- a/lib.c +++ b/lib.

[PATCH 08/15] options: avoid spaces between function name and arguments list

2020-07-02 Thread Luc Van Oostenryck
It's a stylistic detail but a lot of the strcmp() calls used for the processing of the options are written 'strcmp (...)'. Two other functions calls are also in the case. Reformat them to the usual style for function calls: without the space between the function name and the arguments. Signed-off

[PATCH 13/15] cleanup: move predefines in a separate file

2020-07-02 Thread Luc Van Oostenryck
Now that option parsing have moved to a separate file, move everything related to predefined macros to a separate file too. Signed-off-by: Luc Van Oostenryck --- Makefile| 1 + lib.c | 221 --- lib.h | 1 + predefine.c | 225 +++

[PATCH 14/15] cleanup: move parsing helpers to parse.c

2020-07-02 Thread Luc Van Oostenryck
lib.c contains 2-3 helpers fro parsing. Move these to parse.c. Signed-off-by: Luc Van Oostenryck --- lib.c | 38 -- parse.c | 38 ++ 2 files changed, 38 insertions(+), 38 deletions(-) diff --git a/lib.c b/lib.c index fd1f

[PATCH 15/15] cleanup: move hexval() to utils.c

2020-07-02 Thread Luc Van Oostenryck
Now lib.c contains almost nothing else than library entrypoints. Move a small utility, hexval(), to utils.c to complete this cleanup. Signed-off-by: Luc Van Oostenryck --- lib.c | 17 - lib.h | 2 -- utils.c | 17 + utils.h | 4 4 files changed, 21 ins

[PATCH 10/15] options: add a small helper: handle_switch_finalize()

2020-07-02 Thread Luc Van Oostenryck
This is just to isolate the details about which switch need an extra 'finalization' in a separate function in preparation to moving all the parsing code in a separate file. Signed-off-by: Luc Van Oostenryck --- lib.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/li

[PATCH 12/15] options: keep the options sorted

2020-07-02 Thread Luc Van Oostenryck
The declarations and definitions of the variables corresponding to the options half-sorted half-unsorted. Sort them a little more. Signed-off-by: Luc Van Oostenryck --- options.c | 98 +++--- options.h | 137 --

[PATCH 04/15] options: make Wsparse_error less special

2020-07-02 Thread Luc Van Oostenryck
-Wsparse-error should not be enabled with -Wsparse-all, this is special cased in the condition in loop handling -Wsparse-all. However, the condition already handle warnings forced to off. So instead of explicitly checking for &Wsparse_error, it's enough to force Wsparse_error off. Signed-off-by:

Re: weird loadavg on idle machine post 5.7

2020-07-02 Thread Michal Kubecek
On Thu, Jul 02, 2020 at 10:36:27PM +0100, Mel Gorman wrote: > > It builds, not booted, it's for discussion but maybe Dave is feeling brave! > > diff --git a/kernel/sched/core.c b/kernel/sched/core.c > index ca5db40392d4..52c73598b18a 100644 > --- a/kernel/sched/core.c > +++ b/kernel/sched/core.c

[PATCH 11/15] options: move option parsing in a separate file

2020-07-02 Thread Luc Van Oostenryck
lib.c contains to much things and is too hard to keep tidy. So, move everything related to option parsing in it's own file. Signed-off-by: Luc Van Oostenryck --- Makefile | 1 + lib.c | 982 + lib.h | 117 +-- options.c | 998 +++

[PATCH 03/15] options: move on top the definition of warning type enums

2020-07-02 Thread Luc Van Oostenryck
This allows to reuse these enums in earlier helpers. Signed-off-by: Luc Van Oostenryck --- lib.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/lib.c b/lib.c index 5128a5b64e9e..43d55a0648ee 100644 --- a/lib.c +++ b/lib.c @@ -250,6 +250,12 @@ void die(const char

[PATCH 07/15] options: alphasort the handle_switch_[a-zA_Z]()

2020-07-02 Thread Luc Van Oostenryck
These function have probably been added in 'historical order' and as result it's not easy to quickly see where they're defined. Change this arranging them in asciibetical order. Signed-off-by: Luc Van Oostenryck --- lib.c | 656 +- 1 file

[PATCH 02/15] options: move -Wsparse-all's processing out of handle_onoff_switch()

2020-07-02 Thread Luc Van Oostenryck
Since handle_onoff_switch() can be used for other flags than the warnings, the processing of -Wsparse-all should move elsewhere. So move it into handle_switch_W(). Signed-off-by: Luc Van Oostenryck --- lib.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/li

[PATCH 09/15] options: move declaration of tabstop out of "token.h"

2020-07-02 Thread Luc Van Oostenryck
'tabstop' is unusual in the sense that it's one the few (the only?) variable defined via an option flag which is not declared in "lib.h" but in "token.h". This for to have to include "token.h" in the code doing the parsing of the options ... Move this declaration to "lib.h". Signed-off-by: Luc Va

[PATCH 00/14] tidy-up options / reorganize lib.c

2020-07-02 Thread Luc Van Oostenryck
A lot of content in lib.c have been added by just appending at the bottom of what was already present. As consequence, things are now not well organized at all, especially when related to the options. So, reorganize things a little bit here: *) move all helpers on top *) keep things alphabetically

[PATCH 01/15] options: let handle_onoff_switch() use null terminated arrays

2020-07-02 Thread Luc Van Oostenryck
This makes things slightly easier to use. Signed-off-by: Luc Van Oostenryck --- lib.c | 20 +++- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/lib.c b/lib.c index e56788260cb7..709dd5176112 100644 --- a/lib.c +++ b/lib.c @@ -529,14 +529,14 @@ enum { WARNI

Re: [PATCH v10 02/14] KVM: Cache as_id in kvm_memory_slot

2020-07-02 Thread Sean Christopherson
On Mon, Jun 01, 2020 at 07:59:45AM -0400, Peter Xu wrote: > Cache the address space ID just like the slot ID. It will be used in > order to fill in the dirty ring entries. > > Suggested-by: Paolo Bonzini > Suggested-by: Sean Christopherson > Signed-off-by: Peter Xu > --- > include/linux/kvm_h

Re: [PATCH 0/3] selftests: tpm: fixes

2020-07-02 Thread Jarkko Sakkinen
On Mon, Jun 29, 2020 at 02:21:47PM -0600, Shuah Khan wrote: > On 6/22/20 3:20 PM, Jarkko Sakkinen wrote: > > A few fixes for tools/testing/selftests/tpm. > > > > Jarkko Sakkinen (3): > >Revert "tpm: selftest: cleanup after unseal with wrong auth/policy > > test" > >selftests: tpm: Use

Re: [RFC PATCH 4/7] crypto: remove ARC4 support from the skcipher API

2020-07-02 Thread Ard Biesheuvel
On Thu, 2 Jul 2020 at 20:21, Ard Biesheuvel wrote: > > On Thu, 2 Jul 2020 at 19:50, Eric Biggers wrote: > > > > [+linux-wireless, Marcel Holtmann, and Denis Kenzior] > > > > On Thu, Jul 02, 2020 at 12:19:44PM +0200, Ard Biesheuvel wrote: > > > Remove the generic ecb(arc4) skcipher, which is sligh

Re: [PATCH 2/2] perf tools: Fix record failure when mixed with ARM SPE event

2020-07-02 Thread Mathieu Poirier
Hi Li, On Tue, Jun 23, 2020 at 08:31:41PM +0800, Wei Li wrote: > When recording with cache-misses and arm_spe_x event, i found that > it will just fail without showing any error info if i put cache-misses > after arm_spe_x event. > > [root@localhost 0620]# perf record -e cache-misses -e \ > arm_s

Re: [PATCH v2 1/4] x86/xen: remove 32-bit Xen PV guest support

2020-07-02 Thread Boris Ostrovsky
On 7/1/20 7:06 AM, Juergen Gross wrote: > Xen is requiring 64-bit machines today and since Xen 4.14 it can be > built without 32-bit PV guest support. There is no need to carry the > burden of 32-bit PV guest support in the kernel any longer, as new > guests can be either HVM or PVH, or they can us

[PATCH] tpm: Define TPM2_SPACE_BUFFER_SIZE to replace the use of PAGE_SIZE

2020-07-02 Thread Jarkko Sakkinen
The size of the buffers for storing context's and sessions can vary from arch to arch as PAGE_SIZE can be anything between 4 kB and 256 kB (the maximum for PPC64). Define a fixed buffer size set to 16 kB. This should be enough for most use with three handles (that is how many we allow at the moment

[PATCH v4] drivers/nvme: Add support for ACPI StorageD3Enable property

2020-07-02 Thread David E. Box
This patch implements a solution for a BIOS hack used on some currently shipping Intel systems to change driver power management policy for PCIe NVMe drives. Some newer Intel platforms, like some Comet Lake systems, require that PCIe devices use D3 when doing suspend-to-idle in order to allow the p

Re: [PATCH 0/5] RFC: connector: Add network namespace awareness

2020-07-02 Thread Aleksa Sarai
On 2020-07-02, Christian Brauner wrote: > On Thu, Jul 02, 2020 at 08:17:38AM -0500, Eric W. Biederman wrote: > > Matt Bennett writes: > > > > > Previously the connector functionality could only be used by processes > > > running in the > > > default network namespace. This meant that any proces

Re: [PATCH v5 03/10] dmaengine: Actions: Add support for S700 DMA engine

2020-07-02 Thread kernel test robot
Hi Amit, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on robh/for-next] [also build test WARNING on clk/clk-next pza/reset/next linus/master v5.8-rc3 next-20200702] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting

Re: [PATCH 1/2] perf tools: ARM SPE code cleanup

2020-07-02 Thread Mathieu Poirier
On Tue, Jun 23, 2020 at 08:31:40PM +0800, Wei Li wrote: > Remove the useless check code to make it clear. > > Signed-off-by: Wei Li > --- > tools/perf/arch/arm/util/auxtrace.c | 6 ++ > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/tools/perf/arch/arm/util/auxtrace.c > b

[PATCH v2] igb: reinit_locked() should be called with rtnl_lock

2020-07-02 Thread Francesco Ruggeri
We observed two panics involving races with igb_reset_task. The first panic is caused by this race condition: kworker reboot -f igb_reset_task igb_reinit_locked igb_down napi_synchronize __igb_shutdown

Re: [PATCH v2 7/7] KVM: X86: Move kvm_apic_set_version() to kvm_update_vcpu_model()

2020-07-02 Thread Xiaoyao Li
On 7/3/2020 3:00 AM, Sean Christopherson wrote: On Tue, Jun 23, 2020 at 07:58:16PM +0800, Xiaoyao Li wrote: Obviously, kvm_apic_set_version() fits well in kvm_update_vcpu_model(). Same as the last patch, it would be nice to explicitly document that there are no dependencies between kvm_apic_se

Re: [GIT PULL] io_uring fixes for 5.8-rc4

2020-07-02 Thread pr-tracker-bot
The pull request you sent on Wed, 1 Jul 2020 22:30:24 -0600: > git://git.kernel.dk/linux-block.git tags/io_uring-5.8-2020-07-01 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/c93493b7cd40c20708e3373a7cc8e8049460d7ce Thank you! -- Deet-doot-dot, I am a bot. https://k

Re: [PATCH v5 3/3] ARM: imx6plus: optionally enable internal routing of clk_enet_ref

2020-07-02 Thread Fabio Estevam
Hi Sven, On Thu, Jul 2, 2020 at 2:53 PM Sven Van Asbroeck wrote: > + /* > +* On imx6 plus, enet_ref from ANATOP/CCM can be internally routed to > +* be the PTP clock source, instead of having to be routed through > +* pads. > +*/ > + if (of_machine_is_

Re: [PATCH v2 6/7] KVM: X86: Move kvm_x86_ops.update_vcpu_model() into kvm_update_vcpu_model()

2020-07-02 Thread Xiaoyao Li
On 7/3/2020 2:59 AM, Sean Christopherson wrote: On Tue, Jun 23, 2020 at 07:58:15PM +0800, Xiaoyao Li wrote: kvm_x86_ops.update_vcpu_model() is used to update vmx/svm vcpu settings based on updated CPUID settings. So it's supposed to be called after CPUIDs are fully updated, i.e., kvm_update_cpui

Re: [PATCH v2 2/7] KVM: X86: Go on updating other CPUID leaves when leaf 1 is absent

2020-07-02 Thread Xiaoyao Li
On 7/3/2020 3:02 AM, Sean Christopherson wrote: On Thu, Jul 02, 2020 at 11:54:03AM -0700, Sean Christopherson wrote: On Tue, Jun 23, 2020 at 07:58:11PM +0800, Xiaoyao Li wrote: As handling of bits other leaf 1 added over time, kvm_update_cpuid() should not return directly if leaf 1 is absent, b

Re: [PATCH 4.19 114/131] ocfs2: avoid inode removal while nfsd is accessing it

2020-07-02 Thread Linus Torvalds
On Thu, Jul 2, 2020 at 2:17 PM Pavel Machek wrote: > > > > commit 4cd9973f9ff69e37dd0ba2bd6e6423f8179c329a upstream. > > > > Patch series "ocfs2: fix nfsd over ocfs2 issues", v2. > > This causes locking imbalance: This sems to be true upstream too. > When ocfs2_nfs_sync_lock() returns error, cal

[git pull] m68knommu changes for v5.8-rc4

2020-07-02 Thread Greg Ungerer
Hi Linus, Please pull important m68knommu fixes for v5.8-rc4 Regards Greg The following changes since commit 9ebcfadb0610322ac537dd7aa5d9cbc2b2894c68: Linux 5.8-rc3 (2020-06-28 15:00:24 -0700) are available in the Git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/gerg/m6

[PATCH v2] lib: update DEBUG_SHIRQ docs to match reality

2020-07-02 Thread Wolfram Sang
There is no extra interrupt when registering a shared interrupt handler since 2011. Update the Kconfig text to make it clear and to avoid wrong assumptions when debugging issues found by it. Fixes: 6d83f94db95c ("genirq: Disable the SHIRQ_DEBUG call in request_threaded_irq for now") Link: https:

Re: [PATCH v2 09/11] ima: Move validation of the keyrings conditional into ima_validate_rule()

2020-07-02 Thread Tyler Hicks
On 2020-06-30 19:07:29, Mimi Zohar wrote: > On Fri, 2020-06-26 at 17:38 -0500, Tyler Hicks wrote: > > Use ima_validate_rule() to ensure that the combination of a hook > > function and the keyrings conditional is valid and that the keyrings > > conditional is not specified without an explicit KEY_CH

Re: [PATCH v2] x86/speculation/l1tf: Add KConfig for setting the L1D cache flush mode

2020-07-02 Thread Abhishek Bhardwaj
On Thu, Jul 2, 2020 at 3:01 PM Randy Dunlap wrote: > > Hi-- > > On 7/2/20 2:38 PM, Abhishek Bhardwaj wrote: > > This change adds a new kernel configuration that sets the l1d cache > > flush setting at compile time rather than at run time. > > > > Signed-off-by: Abhishek Bhardwaj > > > > --- > > >

Re: [PATCH 4.19 082/131] nvme: fix possible deadlock when I/O is blocked

2020-07-02 Thread Sagi Grimberg
Hi! From: Sagi Grimberg [ Upstream commit 3b4b19721ec652ad2c4fe51dfbe5124212b5f581 ] Revert fab7772bfbcf ("nvme-multipath: revalidate nvme_ns_head gendisk in nvme_validate_ns") When adding a new namespace to the head disk (via nvme_mpath_set_live) we will see partition scan which triggers

[PATCH] hwmon: (pmbus) fix a typo in Kconfig SENSORS_IR35221 option

2020-07-02 Thread rentao . bupt
From: Tao Ren Fix a typo in SENSORS_IR35221 option: module name should be "ir35221" instead of "ir35521". Fixes: 8991ebd9c9a6 ("hwmon: (pmbus) Add client driver for IR35221") Cc: Samuel Mendoza-Jonas Signed-off-by: Tao Ren --- drivers/hwmon/pmbus/Kconfig | 2 +- 1 file changed, 1 insertion(+

[PATCH v3] x86/speculation/l1tf: Add KConfig for setting the L1D cache flush mode

2020-07-02 Thread Abhishek Bhardwaj
This change adds a new kernel configuration that sets the l1d cache flush setting at compile time rather than at run time. Signed-off-by: Abhishek Bhardwaj --- Changes in v3: - Change depends on to only x86_64. - Remove copy paste errors at the end of the KConfig. Changes in v2: - Fix typo in

Re: [RFC] many-files Documentation patch

2020-07-02 Thread Jonathan Corbet
On Thu, 2 Jul 2020 15:05:01 -0700 Randy Dunlap wrote: > This patch series eliminates/corrects many doubled words, such as > "the the", "of of", "with with", and "and and". > It mostly drops the doubled word, but sometimes it modifies one or two > words so that they make sense. > > > Can (will)

[PATCH v2] firmware: improve description of firmware_request_nowarn

2020-07-02 Thread Wolfram Sang
The doubled 'however' is confusing. Simplify the comment a little and reformat the paragraph. Signed-off-by: Wolfram Sang Acked-by: Luis Chamberlain --- Changes since v1: * removed uneeded empty line (sorry!) * added Luis' ack drivers/base/firmware_loader/main.c | 12 ++-- 1 file ch

Re: [PATCH] Replace HTTP links with HTTPS ones: BPF (Safe dynamic programs and tools)

2020-07-02 Thread Alexei Starovoitov
On Thu, Jul 2, 2020 at 1:05 PM Alexander A. Klimov wrote: > > 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 contai

[RFC] many-files Documentation patch

2020-07-02 Thread Randy Dunlap
This patch series eliminates/corrects many doubled words, such as "the the", "of of", "with with", and "and and". It mostly drops the doubled word, but sometimes it modifies one or two words so that they make sense. Can (will) Jon merge this patch series or do I send many separate patches to rele

Re: [PATCH] vmlinux.lds.h: Coalesce transient LLVM dead code elimination sections

2020-07-02 Thread Danny Lin
Jul 2, 2020 9:04:25 AM Nathan Chancellor : > On Thu, Jul 02, 2020 at 08:54:53AM -0700, Kees Cook wrote: >> On Thu, Jul 02, 2020 at 01:54:00AM -0700, Danny Lin wrote: >>> A recent LLVM 11 commit [1] made LLD stop implicitly coalescing some >>> temporary LLVM sections, namely .{data,bss}..compoundli

Re: [PATCH v2] x86/speculation/l1tf: Add KConfig for setting the L1D cache flush mode

2020-07-02 Thread Randy Dunlap
Hi-- On 7/2/20 2:38 PM, Abhishek Bhardwaj wrote: > This change adds a new kernel configuration that sets the l1d cache > flush setting at compile time rather than at run time. > > Signed-off-by: Abhishek Bhardwaj > > --- > > Changes in v2: > - Fix typo in the help of the new KConfig. > > arc

Re: [PATCH v4 3/3] prctl: Allow ptrace capable processes to change /proc/self/exe

2020-07-02 Thread Paul Moore
On Thu, Jul 2, 2020 at 5:16 PM Serge E. Hallyn wrote: > On Wed, Jul 01, 2020 at 08:49:06AM +0200, Adrian Reber wrote: > > From: Nicolas Viennot > > > > Previously, the current process could only change the /proc/self/exe > > link with local CAP_SYS_ADMIN. > > This commit relaxes this restriction

[for-next][PATCH 00/18] tracing: Updatse for 5.9

2020-07-02 Thread Steven Rostedt
git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git for-next Head SHA1: 29ce24519c0692ca7d998d7444a9e016a4c44fa7 Nicholas Piggin (1): ring-buffer: speed up buffer resets by avoiding synchronize_rcu for each CPU Steven Rostedt (VMware) (13): tracing: Only allow t

[for-next][PATCH 09/18] tracing: Move pipe reference to trace array instead of current_tracer

2020-07-02 Thread Steven Rostedt
From: "Steven Rostedt (VMware)" If a process has the trace_pipe open on a trace_array, the current tracer for that trace array should not be changed. This was original enforced by a global lock, but when instances were introduced, it was moved to the current_trace. But this structure is shared by

[for-next][PATCH 01/18] tracing: Only allow trace_array_printk() to be used by instances

2020-07-02 Thread Steven Rostedt
From: "Steven Rostedt (VMware)" To prevent default "trace_printks()" from spamming the top level tracing ring buffer, only allow trace instances to use trace_array_printk() (which can be used without the trace_printk() start up warning). Signed-off-by: Steven Rostedt (VMware) --- kernel/trace/

[for-next][PATCH 07/18] tracing: define DEFINE_EVENT_PRINT not related to DEFINE_EVENT

2020-07-02 Thread Steven Rostedt
From: Wei Yang Current definition define DEFINE_EVENT_PRINT to be DEFINE_EVENT. Actually, at this point DEFINE_EVENT is already an empty macro. Let's cut the relationship between DEFINE_EVENT_PRINT and DEFINE_EVENT. Link: http://lkml.kernel.org/r/20200612092844.56107-4-richard.weiy...@linux.ali

[for-next][PATCH 02/18] x86/ftrace: Make non direct case the default in ftrace_regs_caller

2020-07-02 Thread Steven Rostedt
From: "Steven Rostedt (VMware)" If a direct function is hooked along with one of the ftrace registered functions, then the ftrace_regs_caller is attached to the function that shares the direct hook as well as the ftrace hook. The ftrace_regs_caller will call ftrace_ops_list_func() that iterates t

[for-next][PATCH 04/18] x86/ftrace: Do not jump to direct code in created trampolines

2020-07-02 Thread Steven Rostedt
From: "Steven Rostedt (VMware)" When creating a trampoline based on the ftrace_regs_caller code, nop out the jnz test that would jmup to the code that would return to a direct caller (stored in the ORIG_RAX field) and not back to the function that called it. Link: http://lkml.kernel.org/r/202004

[for-next][PATCH 08/18] tracing: not necessary to define DEFINE_EVENT_PRINT to be empty again

2020-07-02 Thread Steven Rostedt
From: Wei Yang After the previous cleanup, DEFINE_EVENT_PRINT's definition has no relationship with DEFINE_EVENT. So After we re-define DEFINE_EVENT, it is not necessary to define DEFINE_EVENT_PRINT to be empty again. Link: http://lkml.kernel.org/r/20200612092844.56107-5-richard.weiy...@linux.a

[for-next][PATCH 17/18] ring-buffer: Call trace_clock_local() directly for RETPOLINE kernels

2020-07-02 Thread Steven Rostedt
From: "Steven Rostedt (VMware)" After doing some benchmarks and examining the code, I found that the ring buffer clock calls were quite expensive, and noticed that it uses retpolines. This is because the ring buffer clock is programmable, and can be set. But in most cases it simply uses the faste

[for-next][PATCH 13/18] ring-buffer: speed up buffer resets by avoiding synchronize_rcu for each CPU

2020-07-02 Thread Steven Rostedt
From: Nicholas Piggin On a 144 thread system, `perf ftrace` takes about 20 seconds to start up, due to calling synchronize_rcu() for each CPU. cat /proc/108560/stack 0xc0003e7eb336f470 __switch_to+0x2e0/0x480 __wait_rcu_gp+0x20c/0x220 synchronize_rcu+0x9c/0xc0 ring_buffer_r

[for-next][PATCH 15/18] ring-buffer: Consolidate add_timestamp to remove some branches

2020-07-02 Thread Steven Rostedt
From: "Steven Rostedt (VMware)" Reorganize a little the logic to handle adding the absolute time stamp, extended and forced time stamps, in such a way to remove a branch or two. This is just a micro optimization. Also add before and after time stamps to the rb_event_info structure to display tho

[for-next][PATCH 11/18] ring-buffer: Incorporate absolute timestamp into add_timestamp logic

2020-07-02 Thread Steven Rostedt
From: "Steven Rostedt (VMware)" Instead of calling out the absolute test for each time to check if the ring buffer wants absolute time stamps for all its recording, incorporate it with the add_timestamp field and turn it into flags for faster processing between wanting a absolute tag and needing

[for-next][PATCH 06/18] tracing: not necessary re-define DEFINE_EVENT_PRINT

2020-07-02 Thread Steven Rostedt
From: Wei Yang The definition of DEFINE_EVENT_PRINT is not changed after previous one, so not necessary to re-define is as the same form. Link: http://lkml.kernel.org/r/20200612092844.56107-3-richard.weiy...@linux.alibaba.com Signed-off-by: Wei Yang Signed-off-by: Steven Rostedt (VMware) ---

[for-next][PATCH 03/18] x86/ftrace: Only have the builtin ftrace_regs_caller call direct hooks

2020-07-02 Thread Steven Rostedt
From: "Steven Rostedt (VMware)" If a direct hook is attached to a function that ftrace also has a function attached to it, then it is required that the ftrace_ops_list_func() is used to iterate over the registered ftrace callbacks. This will also include the direct ftrace_ops helper, that tells f

Re: [PATCH v4 3/3] prctl: Allow ptrace capable processes to change /proc/self/exe

2020-07-02 Thread Serge E. Hallyn
On Wed, Jul 01, 2020 at 10:55:37AM +0200, Christian Brauner wrote: > On Wed, Jul 01, 2020 at 08:49:06AM +0200, Adrian Reber wrote: > > From: Nicolas Viennot > > > > Previously, the current process could only change the /proc/self/exe > > link with local CAP_SYS_ADMIN. > > This commit relaxes this

[for-next][PATCH 10/18] ring-buffer: Have nested events still record running time stamp

2020-07-02 Thread Steven Rostedt
From: "Steven Rostedt (VMware)" Up until now, if an event is interrupted while it is recorded by an interrupt, and that interrupt records events, the time of those events will all be the same. This is because events only record the delta of the time since the previous event (or beginning of a pag

[for-next][PATCH 12/18] ring-buffer: Add rb_time_t 64 bit operations for speeding up 32 bit

2020-07-02 Thread Steven Rostedt
From: "Steven Rostedt (VMware)" After a discussion with the new time algorithm to have nested events still have proper time keeping but required using local64_t atomic operations. Mathieu was concerned about the performance this would have on 32 bit machines, as in most cases, atomic 64 bit opera

[for-next][PATCH 16/18] ring-buffer: Move the add_timestamp into its own function

2020-07-02 Thread Steven Rostedt
From: "Steven Rostedt (VMware)" Make a helper function rb_add_timestamp() that moves the adding of the extended time stamps into its own function. Also, remove the noinline and inline for the functions it calls, as recent benchmarks appear they do not make a difference (just let gcc decide). Sig

[for-next][PATCH 05/18] tracing: not necessary to undefine DEFINE_EVENT again

2020-07-02 Thread Steven Rostedt
From: Wei Yang After un-define DEFINE_EVENT in Stage 2, DEFINE_EVENT is not defined to a specific form. It is not necessary to un-define it again. Let's skip this. Link: http://lkml.kernel.org/r/20200612092844.56107-2-richard.weiy...@linux.alibaba.com Signed-off-by: Wei Yang Signed-off-by: S

[for-next][PATCH 18/18] ring-buffer: Do not trigger a WARN if clock going backwards is detected

2020-07-02 Thread Steven Rostedt
From: "Steven Rostedt (VMware)" After tweaking the ring buffer to be a bit faster, a warning is triggering on one of my machines, and causing my tests to fail. This warning is caused when the delta (current time stamp minus previous time stamp), is larger than the max time held by the ring buffer

[for-next][PATCH 14/18] ring-buffer: Mark the !tail (crossing a page) as unlikely

2020-07-02 Thread Steven Rostedt
From: "Steven Rostedt (VMware)" It is the uncommon case where an event crosses a sub buffer boundary (page) mark that check at the end of reserving an event as unlikely. Suggested-by: Mathieu Desnoyers Signed-off-by: Steven Rostedt (VMware) --- kernel/trace/ring_buffer.c | 2 +- 1 file change

Re: objtool clac/stac handling change..

2020-07-02 Thread Linus Torvalds
On Thu, Jul 2, 2020 at 1:59 PM Al Viro wrote: > > I'm not sure it's the right solution in this case. Look at the call chain > and the stuff done nearby (that __clear_user(), for example)... > > I'm not saying that this code is not awful - it certainly is. But it's > not that simple, unfortunatel

Re: [PATCH] docs/core-api: memory-allocation: describe reclaim behaviour

2020-07-02 Thread Mike Rapoport
Gentle ping. On Fri, Jun 26, 2020 at 05:29:50PM +0300, Mike Rapoport wrote: > From: Mike Rapoport > > Changelog of commit dcda9b04713c ("mm, tree wide: replace __GFP_REPEAT by > __GFP_RETRY_MAYFAIL with more useful semantic") has very nice description > of GFP flags that affect reclaim behaviour

RE: Writing to a const pointer: is this supposed to happen?

2020-07-02 Thread Kars Mulder
On Thursday, July 02, 2020 09:55 CEST, David Laight wrote: > Hmm... sscanf() is also horrid. > Surprisingly difficult to use correctly. > > It is usually best to use strchr() (and maybe str[c]scn()) > to parse strings. > For numbers use whatever the kernels current 'favourite' implementation > of

Re: [PATCH 0/8] mm: cleanup usage of

2020-07-02 Thread Mike Rapoport
Gentle ping. On Sat, Jun 27, 2020 at 05:34:45PM +0300, Mike Rapoport wrote: > From: Mike Rapoport > > Hi, > > Most architectures have very similar versions of pXd_alloc_one() and > pXd_free_one() for intermediate levels of page table. > These patches add generic versions of these functions in

Re: [PATCH] MAINTAINERS: Add Shengjiu to reviewer list of sound/soc/fsl

2020-07-02 Thread Fabio Estevam
On Thu, Jul 2, 2020 at 4:31 PM Nicolin Chen wrote: > > Add Shengjiu who's actively working on the latest fsl/nxp audio drivers. > > Signed-off-by: Nicolin Chen > Cc: Shengjiu Wang Reviewed-by: Fabio Estevam

Re: [PATCH 1/2] [RFC] media: rcar-vin: send a V4L2 event to vdev if no frame captured after a timeout

2020-07-02 Thread Niklas Söderlund
Hi Michael, On 2020-07-02 14:33:41 +0200, Michael Rodin wrote: > Hi Niklas, > > On Wed, Jul 01, 2020 at 12:17:10AM +0200, Niklas Söderlund wrote: > > Hi Michael, > > > > Thanks for your RFC. > > Thanks your your feedback! > > > On 2020-06-19 19:46:10 +0200, Michael Rodin wrote: > > > Data flow

[GIT PULL] Devicetree fixes for 5.8, v2

2020-07-02 Thread Rob Herring
Linus, Please pull DT fixes for 5.8. I wouldn't normally update dtc at this point in the cycle, but I've been waiting on fixes to land upstream for a while now. Rob The following changes since commit b3a9e3b9622ae10064826dccb4f7a52bd88c7407: Linux 5.8-rc1 (2020-06-14 12:45:04 -0700) are a

Re: [PATCH][next] xen-netfront: remove redundant assignment to variable 'act'

2020-07-02 Thread David Miller
From: Colin King Date: Thu, 2 Jul 2020 15:22:23 +0100 > From: Colin Ian King > > The variable act is being initialized with a value that is > never read and it is being updated later with a new value. The > initialization is redundant and can be removed. > > Addresses-Coverity: ("Unused value

[PATCH v2] x86/speculation/l1tf: Add KConfig for setting the L1D cache flush mode

2020-07-02 Thread Abhishek Bhardwaj
This change adds a new kernel configuration that sets the l1d cache flush setting at compile time rather than at run time. Signed-off-by: Abhishek Bhardwaj --- Changes in v2: - Fix typo in the help of the new KConfig. arch/x86/kernel/cpu/bugs.c | 8 arch/x86/kvm/Kconfig | 17 +

Re: weird loadavg on idle machine post 5.7

2020-07-02 Thread Mel Gorman
On Thu, Jul 02, 2020 at 01:15:48PM -0400, Dave Jones wrote: > When I upgraded my firewall to 5.7-rc2 I noticed that on a mostly > idle machine (that usually sees loadavg hover in the 0.xx range) > that it was consistently above 1.00 even when there was nothing running. > All that perf showed was th

Re: [PATCH] MAINTAINERS: net: macb: add Claudiu as co-maintainer

2020-07-02 Thread David Miller
From: Date: Thu, 2 Jul 2020 15:00:21 +0200 > From: Nicolas Ferre > > I would like that Claudiu becomes co-maintainer of the Cadence macb > driver. He's already participating to lots of reviews and enhancements > to this driver and knows the different versions of this controller. > > Signed-off

Re: [PATCH] x86/speculation/l1tf: Add KConfig for setting the L1D cache flush mode

2020-07-02 Thread kernel test robot
Hi Abhishek, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on tip/auto-latest] [also build test WARNING on linux/master tip/x86/core kvm/linux-next linus/master v5.8-rc3 next-20200702] [If your patch is applied to the wrong git tree, kindly drop us a note. And

Re: [PATCH v4] selftests: tpm: upgrade TPM2 tests from Python 2 to Python 3

2020-07-02 Thread Shuah Khan
On 7/2/20 1:44 PM, Jarkko Sakkinen wrote: On Fri, Jun 26, 2020 at 11:40:52AM +0800, Pengfei Xu wrote: Python 2 is no longer supported by the Python upstream project, so upgrade TPM2 tests to Python 3. Signed-off-by: Pengfei Xu I think that it's perfect now. Thank you. Also 1. I checked tha

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