[PATCH v4 3/4] drivers/tty/serial/8250: add aspeed,lpc-io-reg and aspeed,lpc-interrupts DT properties

2021-04-02 Thread Zev Weiss
These allow describing all the Aspeed VUART attributes currently available via sysfs. aspeed,sirq provides a replacement for the deprecated aspeed,sirq-polarity-sense property. Signed-off-by: Zev Weiss --- drivers/tty/serial/8250/8250_aspeed_vuart.c | 44 - 1 file changed, 4

[PATCH v4 4/4] dt-bindings: serial: 8250: add aspeed,lpc-io-reg and aspeed,lpc-interrupts

2021-04-02 Thread Zev Weiss
These correspond to the existing lpc_address, sirq, and sirq_polarity sysfs attributes; the second element of aspeed,lpc-interrupts provides a replacement for the deprecated aspeed,sirq-polarity-sense property. Signed-off-by: Zev Weiss --- .../devicetree/bindings/serial/8250.yaml | 27 +

[PATCH v4 2/4] drivers/tty/serial/8250: refactor sirq and lpc address setting code

2021-04-02 Thread Zev Weiss
This splits dedicated aspeed_vuart_set_{sirq,lpc_address}() functions out of the sysfs store functions in preparation for adding DT properties that will be poking the same registers. While we're at it, these functions now provide some basic bounds-checking on their arguments. Signed-off-by: Zev W

[PATCH v4 1/4] dt-bindings: serial: 8250: deprecate aspeed,sirq-polarity-sense

2021-04-02 Thread Zev Weiss
This property ties SIRQ polarity to SCU register bits that don't necessarily have any direct relationship to it; the only use of it was removed in commit c82bf6e133d30e0f9172a20807814fa28aef0f67. Signed-off-by: Zev Weiss Reviewed-by: Joel Stanley --- Documentation/devicetree/bindings/serial/825

Re: [PATCH 1/4] mm/page_reporting: Introduce free page reported counters

2021-04-02 Thread Alexander Duyck
On Fri, Mar 26, 2021 at 2:45 AM Xunlei Pang wrote: > > It's useful to know how many memory has been actually reported, > so add new zone::reported_pages to record that. > > Add "/sys/kernel/mm/page_reporting/reported_kbytes" for the > actual memory has been reported. > > Add "/sys/kernel/mm/page_r

Compiling Linux kernel into a build directory

2021-04-02 Thread James Courtier-Dutton
Hi, Currently, when one builds the linux kernel, it places .o files all over the source code tree. Is there a way to have the linux kernel build, but place all the .o files into a separate build folder? Similar to how cmake or ninja work when building C source code. One possible advantage of this

Re: [PATCH 1/2] zram: fix crashes due to use of cpu hotplug multistate

2021-04-02 Thread Luis Chamberlain
On Fri, Apr 02, 2021 at 09:54:12AM +0200, Greg KH wrote: > On Thu, Apr 01, 2021 at 11:59:25PM +, Luis Chamberlain wrote: > > As for the syfs deadlock possible with drivers, this fixes it in a generic > > way: > > > > commit fac43d8025727a74f80a183cc5eb74ed902a5d14 > > Author: Luis Chamberlain

Re: [PATCH] qemu_fw_cfg: Make fw_cfg_rev_attr a proper kobj_attribute

2021-04-02 Thread Nathan Chancellor
On Fri, Apr 02, 2021 at 11:25:42AM -0700, Kees Cook wrote: > On Fri, Apr 02, 2021 at 08:42:07AM +0200, Sedat Dilek wrote: > > On Thu, Feb 25, 2021 at 10:25 PM Kees Cook wrote: > > > > > > On Thu, 11 Feb 2021 12:42:58 -0700, Nathan Chancellor wrote: > > > > fw_cfg_showrev() is called by an indirect

Re: [PATCH RESEND 0/3] x86/sgx: eextend ioctl

2021-04-02 Thread Jethro Beekman
On 2021-04-02 17:53, Dave Hansen wrote: > On 4/2/21 1:38 AM, Jethro Beekman wrote: >>> So, we're talking here about pages that have been EEADDED, but for >>> which we do not want to include the entire contents of the page? >>> Do these contents always include the beginning of the page, or can >>> t

Re: [PATCH bpf-next 3/5] libbpf: add low level TC-BPF API

2021-04-02 Thread Alexei Starovoitov
On Fri, Apr 2, 2021 at 8:27 AM Kumar Kartikeya Dwivedi wrote: > > This would be fine, because it's not a fast path or anything, but right now we > return the id using the netlink response, otherwise for query we have to open > the socket, prepare the msg, send and recv again. So it's a minor > op

Re: [PATCH] riscv: Bump COMMAND_LINE_SIZE value to 1024

2021-04-02 Thread Maciej W. Rozycki
On Fri, 2 Apr 2021, David Abdurachmanov wrote: > > > > This macro is exported as a part of the user API so it must not depend > > > > on > > > > Kconfig. Also changing it (rather than say adding COMMAND_LINE_SIZE_V2 > > > > or > > > > switching to an entirely new data object that has its dimen

Re: [PATCH 14/16] staging: rtl8723bs: remove all RT_TRACE logs in core/rtw_wlan_util.c

2021-04-02 Thread Joe Perches
On Fri, 2021-04-02 at 19:40 +0200, Fabio Aiuto wrote: > On Fri, Apr 02, 2021 at 08:20:17AM -0700, Joe Perches wrote: > > On Fri, 2021-04-02 at 14:51 +0200, Fabio Aiuto wrote: > > > On Fri, Apr 02, 2021 at 03:37:57AM -0700, Joe Perches wrote: > > > > On Fri, 2021-04-02 at 12:01 +0200, Fabio Aiuto wr

Re: [PATCH RESEND 0/3] x86/sgx: eextend ioctl

2021-04-02 Thread Dave Hansen
On 4/2/21 11:31 AM, Jethro Beekman wrote: > On 2021-04-02 17:53, Dave Hansen wrote: >> On 4/2/21 1:38 AM, Jethro Beekman wrote: So, we're talking here about pages that have been EEADDED, but for which we do not want to include the entire contents of the page? Do these contents always

Re: [PATCH] qemu_fw_cfg: Make fw_cfg_rev_attr a proper kobj_attribute

2021-04-02 Thread Sedat Dilek
On Fri, Apr 2, 2021 at 8:31 PM Nathan Chancellor wrote: > > On Fri, Apr 02, 2021 at 11:25:42AM -0700, Kees Cook wrote: > > On Fri, Apr 02, 2021 at 08:42:07AM +0200, Sedat Dilek wrote: > > > On Thu, Feb 25, 2021 at 10:25 PM Kees Cook wrote: > > > > > > > > On Thu, 11 Feb 2021 12:42:58 -0700, Natha

Re: [PATCH V2 3/5] arm64: dts: qcom: pmk8350: Add PMIC peripherals for pmk8350

2021-04-02 Thread Matthias Kaehlcke
On Thu, Apr 01, 2021 at 02:43:14PM +0530, satya priya wrote: > subject: arm64: dts: qcom: pmk8350: Add PMIC peripherals for pmk8350 same nit as for 1/5: maybe just 'arm64: dts: qcom: Add pml7350 support/.dtsi' or similar since this adds the initial .dtsi for the pmk8350? > Add PON, GPIO, RTC and

Re: Compiling Linux kernel into a build directory

2021-04-02 Thread Randy Dunlap
On 4/2/21 11:29 AM, James Courtier-Dutton wrote: > Hi, > > Currently, when one builds the linux kernel, it places .o files all > over the source code tree. > Is there a way to have the linux kernel build, but place all the .o > files into a separate build folder? > Similar to how cmake or ninja wo

Re: [PATCH 2/4] mm/page_reporting: Introduce free page reporting factor

2021-04-02 Thread Alexander Duyck
On Fri, Mar 26, 2021 at 2:45 AM Xunlei Pang wrote: > > Add new "/sys/kernel/mm/page_reporting/reporting_factor" > within [0, 100], and stop page reporting when it reaches > the configured threshold. Default is 100 which means no > limitation is imposed. Percentile is adopted to reflect > the fact

Re: [PATCH] fs: split receive_fd_replace from __receive_fd

2021-04-02 Thread Kees Cook
On Thu, Mar 25, 2021 at 09:22:09AM +0100, Christoph Hellwig wrote: > receive_fd_replace shares almost no code with the general case, so split > it out. Also remove the "Bump the sock usage counts" comment from > both copies, as that is now what __receive_sock actually does. > > Signed-off-by: Chr

Re: [PATCH v5 16/19] regulator: bd71815: use ramp-delay helper

2021-04-02 Thread Mark Brown
On Mon, Mar 29, 2021 at 04:00:13PM +0300, Matti Vaittinen wrote: > Use generic regamp ramp-delay helper function instead of implementing own. This is patching something which was just added in the previous patch... Acked-by: Mark Brown signature.asc Description: PGP signature

Re: [PATCH bpf-next 3/5] libbpf: add low level TC-BPF API

2021-04-02 Thread Kumar Kartikeya Dwivedi
On Sat, Apr 03, 2021 at 12:02:14AM IST, Alexei Starovoitov wrote: > On Fri, Apr 2, 2021 at 8:27 AM Kumar Kartikeya Dwivedi > wrote: > > [...] > > All of these things are messy because of tc legacy. bpf tried to follow tc > style > with cls and act distinction and it didn't quite work. cls with >

Re: [PATCH v3 07/25] x86/sgx: Initialize virtual EPC driver even when SGX driver is disabled

2021-04-02 Thread Kai Huang
On Fri, 2 Apr 2021 15:42:51 + Sean Christopherson wrote: > On Fri, Apr 02, 2021, Borislav Petkov wrote: > > On Fri, Mar 19, 2021 at 08:23:02PM +1300, Kai Huang wrote: > > > Modify sgx_init() to always try to initialize the virtual EPC driver, > > > even if the SGX driver is disabled. The SGX d

Re: [PATCH] kunit: make KUNIT_EXPECT_STREQ() quote values, don't print literals

2021-04-02 Thread Daniel Latypov
On Fri, Apr 2, 2021 at 10:47 AM Shuah Khan wrote: > > On 4/2/21 3:35 AM, Brendan Higgins wrote: > > On Fri, Feb 5, 2021 at 2:18 PM Daniel Latypov wrote: > >> > >> Before: > >>> Expected str == "world", but > >>> str == hello > >>> "world" == world > >> > >> After: > >>> Expected s

[PATCH V12 0/3] Charge loop device i/o to issuing cgroup

2021-04-02 Thread Dan Schatzberg
No major changes, rebased on top of latest mm tree Changes since V12: * Small change to get_mem_cgroup_from_mm to avoid needing get_active_memcg Changes since V11: * Removed WQ_MEM_RECLAIM flag from loop workqueue. Technically, this can be driven by writeback, but this was causing a warning

[PATCH 1/3] loop: Use worker per cgroup instead of kworker

2021-04-02 Thread Dan Schatzberg
Existing uses of loop device may have multiple cgroups reading/writing to the same device. Simply charging resources for I/O to the backing file could result in priority inversion where one cgroup gets synchronously blocked, holding up all other I/O to the loop device. In order to avoid this prior

[PATCH 2/3] mm: Charge active memcg when no mm is set

2021-04-02 Thread Dan Schatzberg
set_active_memcg() worked for kernel allocations but was silently ignored for user pages. This patch establishes a precedence order for who gets charged: 1. If there is a memcg associated with the page already, that memcg is charged. This happens during swapin. 2. If an explicit mm is passed,

[PATCH 3/3] loop: Charge i/o to mem and blk cg

2021-04-02 Thread Dan Schatzberg
The current code only associates with the existing blkcg when aio is used to access the backing file. This patch covers all types of i/o to the backing file and also associates the memcg so if the backing file is on tmpfs, memory is charged appropriately. This patch also exports cgroup_get_e_css a

[PATCH 2/3] habanalabs/gaudi: sync stream add protection to SOB reset flow

2021-04-02 Thread Oded Gabbay
From: farah kassabri Since we moved the SOB reset flow to workqueue and not part of the fence release flow, we might reach a scenario where new context is created while we in the middle of resetting the SOB. in such cases the reset may fail due to idle check. This will mess up the streams sync si

[PATCH 1/3] habanalabs: add custom timeout flag per cs

2021-04-02 Thread Oded Gabbay
From: Alon Mizrahi There is a need to allow to user to send command submissions with custom timeout as some CS take longer than the max timeout that is used by default. Signed-off-by: Alon Mizrahi Reviewed-by: Oded Gabbay Signed-off-by: Oded Gabbay --- .../habanalabs/common/command_submissio

[PATCH 3/3] habanalabs/gaudi: add debugfs to DMA from the device

2021-04-02 Thread Oded Gabbay
When trying to debug program, the user often needs to dump large parts of the device's DRAM, which can reach to tens of GBs. Because reading from the device's internal memory through the PCI BAR is extremely slow, the debug can take hours. Instead, we can provide the user to copy data through one

Re: [PATCH] kunit: make KUNIT_EXPECT_STREQ() quote values, don't print literals

2021-04-02 Thread Shuah Khan
On 4/2/21 1:09 PM, Daniel Latypov wrote: On Fri, Apr 2, 2021 at 10:47 AM Shuah Khan wrote: On 4/2/21 3:35 AM, Brendan Higgins wrote: On Fri, Feb 5, 2021 at 2:18 PM Daniel Latypov wrote: Before: Expected str == "world", but str == hello "world" == world After: Expect

Re: [PATCH v5 00/19] Support ROHM BD71815 PMIC

2021-04-02 Thread Mark Brown
On Tue, Mar 30, 2021 at 11:06:53AM +, Vaittinen, Matti wrote: > Do you think Lee could merge other but the regulator parts to MFD if > Mark is busy? I'd like to be able to squeeze the amount of patches and > recipients for future iterations. It might be easier to work directly > on regulator t

Re: [PATCH v3 07/25] x86/sgx: Initialize virtual EPC driver even when SGX driver is disabled

2021-04-02 Thread Borislav Petkov
On Fri, Apr 02, 2021 at 03:42:51PM +, Sean Christopherson wrote: > Nope! That's wrong, as sgx_epc_init() will not be called if sgx_drv_init() > succeeds. And writing it as "if (sgx_drv_init() || sgx_vepc_init())" is also > wrong since that would kill SGX when one of the drivers is alive and w

[PATCH 1/2] fdt: translate address if #size-cells = <0>

2021-04-02 Thread Dario Binacchi
The __of_translate_address() translates an address from the device tree into a CPU physical address. A note in the description of the routine explains that the crossing of any level with #size-cells = <0> can not be mapped to a CPU physical address not because it's really specified that way, but be

[PATCH 0/2] fdt: translate address if #size-cells = <0>

2021-04-02 Thread Dario Binacchi
The series comes from my commit in U-boot d64b9cdcd4 ("fdt: translate address if #size-cells = <0>") and from the subsequent exchange of emails at the end of which I was suggested to send the patch to the linux kernel (https://patchwork.ozlabs.org/project/uboot/patch/1614324949-61314-1-git-send-e

[PATCH 2/2] clk: ti: get register address from device tree

2021-04-02 Thread Dario Binacchi
Until now, only the register offset was retrieved from the device tree to be added, during access, to a common base address for the clocks. If possible, we try to retrieve the physical address of the register directly from the device tree. Signed-off-by: Dario Binacchi --- drivers/clk/ti/clk.c

[PATCH] ARM: OMAP2+: CM: remove omap2_set_globals_cm()

2021-04-02 Thread Dario Binacchi
The function is no longer used, so let's remove it. Signed-off-by: Dario Binacchi --- arch/arm/mach-omap2/cm.h| 1 - arch/arm/mach-omap2/cm_common.c | 13 - 2 files changed, 14 deletions(-) diff --git a/arch/arm/mach-omap2/cm.h b/arch/arm/mach-omap2/cm.h index d02fe63dab5

Re: [PATCH v2 2/6] dt-bindings: soc: actions: Add Actions Semi Owl socinfo binding

2021-04-02 Thread Cristian Ciocaltea
On Fri, Apr 02, 2021 at 11:34:07PM +0530, Manivannan Sadhasivam wrote: > On Tue, Mar 30, 2021 at 04:48:17PM +0300, Cristian Ciocaltea wrote: > > Add devicetree binding for the Actions Semi Owl socinfo driver. > > > > Devicetree binding shouldn't be added for a driver instead for an IP or hw. > >

Re: Packet gets stuck in NOLOCK pfifo_fast qdisc

2021-04-02 Thread Jiri Kosina
On Thu, 3 Sep 2020, John Fastabend wrote: > > > At this point I fear we could consider reverting the NOLOCK stuff. > > > I personally would hate doing so, but it looks like NOLOCK benefits are > > > outweighed by its issues. > > > > I agree, NOLOCK brings more pains than gains. There are many rac

Re: [PATCH V2 4/5] arm64: dts: qcom: pmr735a: Add PMIC peripherals for pmr735a

2021-04-02 Thread Matthias Kaehlcke
Same comments as '[1/5] arm64: dts: qcom: pm7325: Add PMIC peripherals for pm7325', plus another nit: '[3/5] arm64: dts: qcom: pmk8350: Add PMIC peripherals for pmk8350' has an ADC config for the die temperature of the pmr735a, hence it seems this patch should be before the 'pmk8350' one in this s

[PATCH] ARM: OMAP2+: remove omap2_set_globals_control()

2021-04-02 Thread Dario Binacchi
The function is no longer used, so let's remove it. Signed-off-by: Dario Binacchi --- arch/arm/mach-omap2/control.c | 5 - arch/arm/mach-omap2/control.h | 1 - 2 files changed, 6 deletions(-) diff --git a/arch/arm/mach-omap2/control.c b/arch/arm/mach-omap2/control.c index 73338cf80d76..06

Re: [RFC Part1 PATCH 05/13] X86/sev-es: move few helper functions in common file

2021-04-02 Thread Borislav Petkov
On Wed, Mar 24, 2021 at 11:44:16AM -0500, Brijesh Singh wrote: > The sev_es_terminate() and sev_es_{wr,rd}_ghcb_msr() helper functions > in a common file so that it can be used by both the SEV-ES and SEV-SNP. > > Cc: Thomas Gleixner > Cc: Ingo Molnar > Cc: Borislav Petkov > Cc: Joerg Roedel >

Re: [PATCH] kunit: tool: make --kunitconfig accept dirs, add lib/kunit fragment

2021-04-02 Thread Daniel Latypov
On Fri, Apr 2, 2021 at 11:00 AM Shuah Khan wrote: > > On 4/2/21 3:32 AM, Brendan Higgins wrote: > >> TL;DR > >> $ ./tools/testing/kunit/kunit.py run --kunitconfig=lib/kunit > >> > >> Per suggestion from Ted [1], we can reduce the amount of typing by > >> assuming a convention that these files are

Re: [PATCH v3 07/25] x86/sgx: Initialize virtual EPC driver even when SGX driver is disabled

2021-04-02 Thread Sean Christopherson
On Fri, Apr 02, 2021, Borislav Petkov wrote: > On Fri, Apr 02, 2021 at 03:42:51PM +, Sean Christopherson wrote: > > Nope! That's wrong, as sgx_epc_init() will not be called if sgx_drv_init() > > succeeds. And writing it as "if (sgx_drv_init() || sgx_vepc_init())" is > > also > > wrong since

Re: [PATCH] kunit: tool: make --kunitconfig accept dirs, add lib/kunit fragment

2021-04-02 Thread Shuah Khan
On 4/2/21 1:27 PM, Daniel Latypov wrote: On Fri, Apr 2, 2021 at 11:00 AM Shuah Khan wrote: On 4/2/21 3:32 AM, Brendan Higgins wrote: TL;DR $ ./tools/testing/kunit/kunit.py run --kunitconfig=lib/kunit Per suggestion from Ted [1], we can reduce the amount of typing by assuming a convention tha

Re: (subset) [PATCH v5 00/19] Support ROHM BD71815 PMIC

2021-04-02 Thread Mark Brown
On Mon, 29 Mar 2021 15:52:38 +0300, Matti Vaittinen wrote: > Patch series introducing support for ROHM BD71815 PMIC > > ROHM BD71815 is a power management IC used in some battery powered > systems. It contains regulators, GPO(s), charger + coulomb counter, RTC > and a clock gate. > > All regulato

Re: Packet gets stuck in NOLOCK pfifo_fast qdisc

2021-04-02 Thread Josh Hunt
On 4/2/21 12:25 PM, Jiri Kosina wrote: On Thu, 3 Sep 2020, John Fastabend wrote: At this point I fear we could consider reverting the NOLOCK stuff. I personally would hate doing so, but it looks like NOLOCK benefits are outweighed by its issues. I agree, NOLOCK brings more pains than gains. T

[PATCH v2] kunit: make KUNIT_EXPECT_STREQ() quote values, don't print literals

2021-04-02 Thread Daniel Latypov
Before: > Expected str == "world", but > str == hello > "world" == world After: > Expected str == "world", but > str == "hello" Note: like the literal ellision for integers, this doesn't handle the case of KUNIT_EXPECT_STREQ(test, "hello", "world") since we don't expect it to

Re: [PATCH] kunit: make KUNIT_EXPECT_STREQ() quote values, don't print literals

2021-04-02 Thread Daniel Latypov
On Fri, Apr 2, 2021 at 12:19 PM Shuah Khan wrote: > > On 4/2/21 1:09 PM, Daniel Latypov wrote: > > On Fri, Apr 2, 2021 at 10:47 AM Shuah Khan > > wrote: > >> > >> On 4/2/21 3:35 AM, Brendan Higgins wrote: > >>> On Fri, Feb 5, 2021 at 2:18 PM Daniel Latypov wrote: > > Before: > >

Re: [PATCH v2 3/6] soc: actions: Add Actions Semi Owl socinfo driver

2021-04-02 Thread Cristian Ciocaltea
On Fri, Apr 02, 2021 at 11:46:24PM +0530, Manivannan Sadhasivam wrote: > On Tue, Mar 30, 2021 at 04:48:18PM +0300, Cristian Ciocaltea wrote: > > The driver provides information about the Action Semi Owl family of > > SoCs (S500, S700 and S900) to user space via sysfs: machine, family, > > soc_id, s

Re: [PATCH RESEND 0/3] x86/sgx: eextend ioctl

2021-04-02 Thread Jethro Beekman
On 2021-04-02 20:42, Dave Hansen wrote: > On 4/2/21 11:31 AM, Jethro Beekman wrote: >> On 2021-04-02 17:53, Dave Hansen wrote: >>> On 4/2/21 1:38 AM, Jethro Beekman wrote: > So, we're talking here about pages that have been EEADDED, but for > which we do not want to include the entire conte

Re: [PATCH] lib/string: Introduce sysfs_streqcase

2021-04-02 Thread Gioh Kim
On Fri, Apr 2, 2021 at 8:17 PM Nick Desaulniers wrote: > > Thanks for the patch! > > + akpm (please remember to run ./scripts/get_maintainer.pl on your patch > files) > > On Fri, Apr 2, 2021 at 2:41 AM Gioh Kim wrote: > > > > As the name shows, it checks if strings are equal in case insensitive

Re: [PATCH v2 1/6] dt-bindings: reserved-memory: Add Owl SoC serial number binding

2021-04-02 Thread Cristian Ciocaltea
On Fri, Apr 02, 2021 at 11:36:07PM +0530, Manivannan Sadhasivam wrote: > On Thu, Apr 01, 2021 at 08:40:01PM +0300, Cristian Ciocaltea wrote: > > On Thu, Apr 01, 2021 at 12:07:04PM -0500, Rob Herring wrote: > > > On Tue, Mar 30, 2021 at 04:48:16PM +0300, Cristian Ciocaltea wrote: > > > > Add devicet

Re: [PATCH] lib/string: Introduce sysfs_streqcase

2021-04-02 Thread Gioh Kim
On Fri, Apr 2, 2021 at 8:24 PM Kees Cook wrote: > > On Fri, Apr 02, 2021 at 11:17:13AM -0700, Nick Desaulniers wrote: > > Thanks for the patch! > > > > + akpm (please remember to run ./scripts/get_maintainer.pl on your patch > > files) > > > > On Fri, Apr 2, 2021 at 2:41 AM Gioh Kim wrote: > > >

Re: [PATCH v3 07/25] x86/sgx: Initialize virtual EPC driver even when SGX driver is disabled

2021-04-02 Thread Borislav Petkov
On Fri, Apr 02, 2021 at 07:30:23PM +, Sean Christopherson wrote: > Heh, that's what I had originally and used for literally years. IIRC, I > suggested the "!! & !!" abomination after internal review complained about the > oddness of the above. Whut? > FWIW, I think the above is far less like

Re: [PATCH v6 12/21] selftests/resctrl: Check for resctrl mount point only if resctrl FS is supported

2021-04-02 Thread Shuah Khan
On 3/16/21 8:22 PM, Fenghua Yu wrote: check_resctrlfs_support() does the following 1. Checks if the platform supports resctrl file system or not by looking for resctrl in /proc/filesystems 2. Calls opendir() on default resctrl file system path (i.e. /sys/fs/resctrl) 3. Checks if resctrl f

Re: [PATCH v6 4/7] pwm: pca9685: Support staggered output ON times

2021-04-02 Thread Uwe Kleine-König
Hello, On Wed, Mar 31, 2021 at 06:21:32PM +0200, Thierry Reding wrote: > However, I'm a bit hesitant about this staggering output mode. From what > I understand what's going to happen for these is basically that overall > each PWM will be running at the requested duty cycle, but the on/off > times

Re: [PATCH RESEND 0/3] x86/sgx: eextend ioctl

2021-04-02 Thread Dave Hansen
On 4/2/21 12:38 PM, Jethro Beekman wrote: > On 2021-04-02 20:42, Dave Hansen wrote: >> On 4/2/21 11:31 AM, Jethro Beekman wrote: >>> On 2021-04-02 17:53, Dave Hansen wrote: But, why would an enclave loader application ever do this? >>> >>> e.g. to save space >> >> How does this save space,

Re: [PATCH mvebu v2 01/10] arm64: dts: marvell: armada-37xx: add syscon compatible to NB clk node

2021-04-02 Thread Gregory CLEMENT
Hi Pali, > From: Marek Behún > > Add "syscon" compatible to the North Bridge clocks node to allow the > cpufreq driver to access these registers via syscon API. > > This is needed for a fix of cpufreq driver. > > Signed-off-by: Marek Behún > Fixes: e8d66e7927b2 ("arm64: dts: marvell: armada-37xx

CFI violation in drivers/infiniband/core/sysfs.c

2021-04-02 Thread Nathan Chancellor
Hi all, I am testing the Clang Control Flow Integrity series that is being worked on right now [1] and I encounter a violation in the Infiniband sysfs core (drivers/infiniband/core/sysfs.c) on an arm64 server with mlx5: $ cat /sys/class/infiniband/mlx5_bond_0/ports/1/hw_counters/lifespan 12 $ ec

Re: [PATCH v6 5/7] dt-bindings: pwm: pca9685: Add nxp,staggered-outputs property

2021-04-02 Thread Uwe Kleine-König
Hello Clemens, On Mon, Mar 29, 2021 at 02:57:05PM +0200, Clemens Gruber wrote: > The pca9685 driver supports a new nxp,staggered-outputs property for > reduced current surges and EMI. This adds documentation for the new DT > property. > > Signed-off-by: Clemens Gruber This patch would need an a

Re: [PATCH 4/4] mm/page_reporting: Fix possible user allocation failure

2021-04-02 Thread Alexander Duyck
On Fri, Mar 26, 2021 at 2:45 AM Xunlei Pang wrote: > > We encountered user memory allocation failure(OOM) on our > 512MiB tiny instances, it didn't happen after turning off > the page reporting. > > After some debugging, it turns out 32*4MB=128MB(order-10) > free pages were isolated during reporti

[PATCH net v3] atl1c: move tx cleanup processing out of interrupt

2021-04-02 Thread Gatis Peisenieks
Tx queue cleanup happens in interrupt handler on same core as rx queue processing. Both can take considerable amount of processing in high packet-per-second scenarios. Sending big amounts of packets can stall the rx processing which is unfair and also can lead to to out-of-memory condition sin

Re: [PATCH v5 00/18] Add support for Clang CFI

2021-04-02 Thread Nathan Chancellor
On Thu, Apr 01, 2021 at 04:31:58PM -0700, Sami Tolvanen wrote: > This series adds support for Clang's Control-Flow Integrity (CFI) > checking. With CFI, the compiler injects a runtime check before each > indirect function call to ensure the target is a valid function with > the correct static type.

Re: [PATCH v6 00/21] Miscellaneous fixes for resctrl selftests

2021-04-02 Thread Shuah Khan
On 4/2/21 12:18 PM, Fenghua Yu wrote: Hi, Shuah, On Fri, Apr 02, 2021 at 12:17:17PM -0600, Shuah Khan wrote: On 3/26/21 1:45 PM, Fenghua Yu wrote: Hi, Shuah, On Wed, Mar 17, 2021 at 02:22:34AM +, Fenghua Yu wrote: This patch set has several miscellaneous fixes to resctrl selftest tool th

Re: [PATCH] perf/core: Mark perf_pmu_snapshot_aux() as static

2021-04-02 Thread Souptick Joarder
On Sat, Jan 9, 2021 at 7:47 AM Souptick Joarder wrote: > > Kernel test robot throws below warning -> > > kernel/events/core.c:6535:6: warning: no previous prototype for > 'perf_pmu_snapshot_aux' [-Wmissing-prototypes] > 6535 | long perf_pmu_snapshot_aux(struct perf_buffer *rb, > |

Re: [PATCH] ARM: dts: turris-omnia: fix hardware buffer management

2021-04-02 Thread Gregory CLEMENT
Hi Rui, > Hardware buffer management has never worked on the Turris Omnia, as the > required MBus window hadn't been reserved. Fix thusly. > > Fixes: 018b88eee1a2 ("ARM: dts: turris-omnia: enable HW buffer management") > > Signed-off-by: Rui Salvaterra Applied on mvebu/fixes Thanks, Gregory >

Re: [PATCH RESEND 0/3] x86/sgx: eextend ioctl

2021-04-02 Thread Jethro Beekman
On 2021-04-02 21:50, Dave Hansen wrote: > > On 4/2/21 12:38 PM, Jethro Beekman wrote: >> On 2021-04-02 20:42, Dave Hansen wrote: >>> On 4/2/21 11:31 AM, Jethro Beekman wrote: On 2021-04-02 17:53, Dave Hansen wrote: > But, why would an enclave loader application ever do this? e.g

[PATCH-next 1/5] lib/test_vmalloc.c: remove two kvfree_rcu() tests

2021-04-02 Thread Uladzislau Rezki (Sony)
Remove two test cases related to kvfree_rcu() and SLAB. Those are considered as redundant now, because similar test functionality has recently been introduced in the "rcuscale" RCU test-suite. Signed-off-by: Uladzislau Rezki (Sony) --- lib/test_vmalloc.c | 40

[PATCH-next 4/5] mm/vmalloc: refactor the preloading loagic

2021-04-02 Thread Uladzislau Rezki (Sony)
Instead of keeping open-coded style, move the code related to preloading into a separate function. Therefore introduce the preload_this_cpu_lock() routine that prelaods a current CPU with one extra vmap_area object. There is no functional change as a result of this patch. Signed-off-by: Uladzisla

[PATCH-next 3/5] vm/test_vmalloc.sh: adapt for updated driver interface

2021-04-02 Thread Uladzislau Rezki (Sony)
A 'single_cpu_test' parameter is odd and it does not exist anymore. Instead there was introduced a 'nr_threads' one. If it is not set it behaves as the former parameter. That is why update a "stress mode" according to this change specifying number of workers which are equal to number of CPUs. Also

[PATCH-next 5/5] mm/vmalloc: remove an empty line

2021-04-02 Thread Uladzislau Rezki (Sony)
Signed-off-by: Uladzislau Rezki (Sony) --- mm/vmalloc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/vmalloc.c b/mm/vmalloc.c index 093c7e034ca2..1e643280cbcf 100644 --- a/mm/vmalloc.c +++ b/mm/vmalloc.c @@ -1503,7 +1503,6 @@ static struct vmap_area *alloc_vmap_area(unsigned long size,

[PATCH-next 2/5] lib/test_vmalloc.c: add a new 'nr_threads' parameter

2021-04-02 Thread Uladzislau Rezki (Sony)
By using this parameter we can specify how many workers are created to perform vmalloc tests. By default it is one CPU. The maximum value is set to 1024. As a result of this change a 'single_cpu_test' one becomes obsolete, therefore it is no longer needed. Signed-off-by: Uladzislau Rezki (Sony)

Re: [PATCH v6 4/7] pwm: pca9685: Support staggered output ON times

2021-04-02 Thread Uwe Kleine-König
On Wed, Mar 31, 2021 at 02:26:14PM +0200, Clemens Gruber wrote: > On Mon, Mar 29, 2021 at 08:02:06PM +0200, Uwe Kleine-König wrote: > > On Mon, Mar 29, 2021 at 07:16:38PM +0200, Clemens Gruber wrote: > > > On Mon, Mar 29, 2021 at 07:03:57PM +0200, Uwe Kleine-König wrote: > > > > On Mon, Mar 29, 202

linux-kernel@vger.kernel.org

2021-04-02 Thread Hugh Dickins
On Fri, 2 Apr 2021, Matthew Wilcox wrote: > OK, more competent testing, and that previous bug now detected and fixed. > I have a reasonable amount of confidence this will solve your problem. > If you do apply this patch, don't enable CONFIG_TEST_XARRAY as the new > tests assume that attempting to

[PATCH v3] nvme: disallow passthru cmd from targeting a nsid != nsid of the block dev

2021-04-02 Thread Niklas Cassel
From: Niklas Cassel When a passthru command targets a specific namespace, the ns parameter to nvme_user_cmd()/nvme_user_cmd64() is set. However, there is currently no validation that the nsid specified in the passthru command targets the namespace/nsid represented by the block device that the ioc

Re: [PATCH v5 clocksource] Do not mark clocks unstable due to delays for v5.13

2021-04-02 Thread Paul E. McKenney
Hello! If there is a sufficient delay between reading the watchdog clock and the clock under test, the clock under test will be marked unstable through no fault of its own. This series checks for this, doing limited retries to get a good set of clock reads. If the clock is marked unstable and is

Re: [PATCH-next 5/5] mm/vmalloc: remove an empty line

2021-04-02 Thread Souptick Joarder
On Sat, Apr 3, 2021 at 1:53 AM Uladzislau Rezki (Sony) wrote: > > Signed-off-by: Uladzislau Rezki (Sony) How about merging it with patch [4/5] ? > --- > mm/vmalloc.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/mm/vmalloc.c b/mm/vmalloc.c > index 093c7e034ca2..1e643280cbcf 100644 >

[PATCH v6 clocksource] Do not mark clocks unstable dueclocksource: Retry clock read if long delays detected

2021-04-02 Thread paulmck
From: "Paul E. McKenney" When the clocksource watchdog marks a clock as unstable, this might be due to that clock being unstable or it might be due to delays that happen to occur between the reads of the two clocks. Yes, interrupts are disabled across those two reads, but there are no shortage o

[PATCH v6 clocksource] Do not mark clocks unstable dueclocksource: Provide module parameters to inject delays in watchdog

2021-04-02 Thread paulmck
From: "Paul E. McKenney" When the clocksource watchdog marks a clock as unstable, this might be due to that clock being unstable or it might be due to delays that happen to occur between the reads of the two clocks. Yes, interrupts are disabled across those two reads, but there are no shortage o

[PATCH v6 clocksource] Do not mark clocks unstable dueclocksource: Check per-CPU clock synchronization when marked unstable

2021-04-02 Thread paulmck
From: "Paul E. McKenney" Some sorts of per-CPU clock sources have a history of going out of synchronization with each other. However, this problem has purportedy been solved in the past ten years. Except that it is all too possible that the problem has instead simply been made less likely, whic

[PATCH v6 clocksource] Do not mark clocks unstable dueclocksource: Provide a module parameter to fuzz per-CPU clock checking

2021-04-02 Thread paulmck
From: "Paul E. McKenney" Code that checks for clock desynchronization must itself be tested, so this commit creates a new clocksource.inject_delay_shift_percpu= kernel boot parameter that adds or subtracts a large value from the check read, using the specified bit of the CPU ID to determine wheth

[PATCH v6 clocksource] Do not mark clocks unstable dueclocksource: Do pairwise clock-desynchronization checking

2021-04-02 Thread paulmck
From: "Paul E. McKenney" Although smp_call_function() has the advantage of simplicity, using it to check for cross-CPU clock desynchronization means that any CPU being slow reduces the sensitivity of the checking across all CPUs. And it is not uncommon for smp_call_function() latencies to be in t

Re: [PATCH V2 1/5] arm64: dts: qcom: pm7325: Add PMIC peripherals for pm7325

2021-04-02 Thread Matthias Kaehlcke
On Fri, Apr 02, 2021 at 10:35:54AM -0700, Matthias Kaehlcke wrote: > On Thu, Apr 01, 2021 at 02:43:12PM +0530, satya priya wrote: > > > subject: arm64: dts: qcom: pm7325: Add PMIC peripherals for pm7325 > > nit: maybe just 'arm64: dts: qcom: Add pm7325 support/.dtsi' or similar? > > > Add temp-a

Re: [PATCH v3] sched/debug: Use sched_debug_lock to serialize use of cgroup_path[] only

2021-04-02 Thread Steven Rostedt
On Thu, 1 Apr 2021 14:10:30 -0400 Waiman Long wrote: > The handling of sysrq key can be activated by echoing the key to > /proc/sysrq-trigger or via the magic key sequence typed into a terminal > that is connected to the system in some way (serial, USB or other mean). > In the former case, the h

[tip:perf/core] BUILD SUCCESS cface0326a6c2ae5c8f47bd466f07624b3e348a7

2021-04-02 Thread kernel test robot
allyesconfig sparc defconfig mips allyesconfig powerpc allyesconfig powerpc allmodconfig x86_64 randconfig-a004-20210402 x86_64 randconfig-a005-20210402 x86_64

[PATCH] habanalabs: remove the store jobs array from CS IOCTL

2021-04-02 Thread Oded Gabbay
The store part was never implemented in the code and never been used by the userspace applications. We currently use the related parameters to a different purpose with a defined union. However, there is no point in that and it is better to just remove the union and the store parameters. Signed-of

Re: [PATCH RESEND 0/3] x86/sgx: eextend ioctl

2021-04-02 Thread Dave Hansen
On 4/2/21 1:20 PM, Jethro Beekman wrote: > On 2021-04-02 21:50, Dave Hansen wrote: >> Again, how does this save space? >> >> Are you literally talking about the temporary cost of allocating *one* page? > > No I'm talking about the amount of disk space/network traffic needed > to distribute the app

Re: [PATCH v3 0/5] Add support for CP110 UTMI PHY

2021-04-02 Thread Gregory CLEMENT
Hi Kosta, > From: Konstantin Porotchkin > > This series of patches adds a new PHY driver for supporting CP110 UTMI > PHY in Linux. Currently the functionality of USB ports connected to > this PHY depends on boot loader setup. > The new driver eliminates kernel configuration dependency from the bo

Re: [PATCH 3/3] srcu: Fix broken node geometry after early ssp init

2021-04-02 Thread Frederic Weisbecker
On Fri, Apr 02, 2021 at 08:03:57AM -0700, Paul E. McKenney wrote: > On Fri, Apr 02, 2021 at 12:02:21PM +0200, Frederic Weisbecker wrote: > > Arguably that's a quite a corner case and I don't expect anyone to call > > start_poll_synchronize_srcu() so early but who knows. An alternative is to > > for

Re: [PATCH] net: initialize local variables in net/ipv6/mcast.c and net/ipv4/igmp.c

2021-04-02 Thread Eric Dumazet
On 4/2/21 8:10 PM, Phillip Potter wrote: > On Fri, Apr 02, 2021 at 07:49:44PM +0200, Eric Dumazet wrote: >> >> >> On 4/2/21 7:36 PM, Phillip Potter wrote: >>> Use memset to initialize two local buffers in net/ipv6/mcast.c, >>> and another in net/ipv4/igmp.c. Fixes a KMSAN found uninit-value >>>

Re: [External] [PATCH v3 7/8] hugetlb: make free_huge_page irq safe

2021-04-02 Thread Mike Kravetz
On 4/2/21 5:47 AM, Muchun Song wrote: > On Wed, Mar 31, 2021 at 11:42 AM Mike Kravetz wrote: >> >> Commit c77c0a8ac4c5 ("mm/hugetlb: defer freeing of huge pages if in >> non-task context") was added to address the issue of free_huge_page >> being called from irq context. That commit hands off fre

Re: [PATCH-next 5/5] mm/vmalloc: remove an empty line

2021-04-02 Thread Uladzislau Rezki
> On Sat, Apr 3, 2021 at 1:53 AM Uladzislau Rezki (Sony) > wrote: > > > > Signed-off-by: Uladzislau Rezki (Sony) > > How about merging it with patch [4/5] ? > I had in mind such concern. Yes we can do a squashing with [4/5]. If there are other comments i will rework whole series if not we can

Re: [PATCH v6 00/21] Miscellaneous fixes for resctrl selftests

2021-04-02 Thread Fenghua Yu
On Fri, Apr 02, 2021 at 02:04:16PM -0600, Shuah Khan wrote: > On 4/2/21 12:18 PM, Fenghua Yu wrote: > > Hi, Shuah, > > > > On Fri, Apr 02, 2021 at 12:17:17PM -0600, Shuah Khan wrote: > > > On 3/26/21 1:45 PM, Fenghua Yu wrote: > > > > Hi, Shuah, > > > > > > > > On Wed, Mar 17, 2021 at 02:22:34AM

[PATCH v2] drm/msm: Drop mm_lock in scan loop

2021-04-02 Thread Rob Clark
From: Rob Clark lock_stat + mmm_donut[1] say that this reduces contention on mm_lock significantly (~350x lower waittime-max, and ~100x lower waittime-avg) [1] https://chromium.googlesource.com/chromiumos/platform/microbenchmarks/+/refs/heads/main/mmm_donut.py Signed-off-by: Rob Clark --- dr

Re: [PATCH] net: initialize local variables in net/ipv6/mcast.c and net/ipv4/igmp.c

2021-04-02 Thread Eric Dumazet
On 4/2/21 10:53 PM, Eric Dumazet wrote: > > > On 4/2/21 8:10 PM, Phillip Potter wrote: >> On Fri, Apr 02, 2021 at 07:49:44PM +0200, Eric Dumazet wrote: >>> >>> >>> On 4/2/21 7:36 PM, Phillip Potter wrote: Use memset to initialize two local buffers in net/ipv6/mcast.c, and another in

Re: [PATCH 3/3] kbuild: fix false-positive modpost warning when all symbols are trimmed

2021-04-02 Thread Nathan Chancellor
On Fri, Mar 26, 2021 at 03:54:11AM +0900, Masahiro Yamada wrote: > Nathan reports that the mips defconfig emits the following warning: > > WARNING: modpost: Symbol info of vmlinux is missing. Unresolved symbol > check will be entirely skipped. > > This false-positive happens under the followin

linux-kernel@vger.kernel.org

2021-04-02 Thread Hugh Dickins
On Fri, 2 Apr 2021, Hugh Dickins wrote: > > There is a "Put holes back where they were" xas_store(&xas, NULL) on > the failure path, which I think we would expect to delete empty nodes. > But it only goes as far as nr_none. Is it ok to xas_store(&xas, NULL) > where there was no non-NULL entry bef

[PATCH v5 0/2] kunit: fail tests on UBSAN errors

2021-04-02 Thread Daniel Latypov
v1 by Uriel is here: [1]. Since it's been a while, I've dropped the Reviewed-By's. It depended on commit 83c4e7a0363b ("KUnit: KASAN Integration") which hadn't been merged yet, so that caused some kerfuffle with applying them previously and the series was reverted. This revives the series but mak

[PATCH v5 1/2] kunit: support failure from dynamic analysis tools

2021-04-02 Thread Daniel Latypov
From: Uriel Guajardo Add a kunit_fail_current_test() function to fail the currently running test, if any, with an error message. This is largely intended for dynamic analysis tools like UBSAN and for fakes. E.g. say I had a fake ops struct for testing and I wanted my `free` function to complain

[PATCH v5 2/2] kunit: ubsan integration

2021-04-02 Thread Daniel Latypov
From: Uriel Guajardo Integrates UBSAN into the KUnit testing framework. It fails KUnit tests whenever it reports undefined behavior. When CONFIG_KUNIT=n, nothing is printed or even formatted, so this has no behavioral impact outside of tests. kunit_fail_current_test() effectively does a pr_err(

<    2   3   4   5   6   7   8   9   >