Re: [RFC PATCH] mm, oom: introduce vm.sacrifice_hugepage_on_oom

2021-02-16 Thread Michal Hocko
On Tue 16-02-21 14:30:15, Mike Kravetz wrote: [...] > However, this is an 'opt in' feature. So, I would not expect anyone who > carefully plans the size of their hugetlb pool to enable such a feature. > If there is a use case where hugetlb pages are used in a non-essential > application, this

Re: [RFC PATCH] mm, oom: introduce vm.sacrifice_hugepage_on_oom

2021-02-16 Thread Michal Hocko
On Tue 16-02-21 13:53:12, David Rientjes wrote: > On Tue, 16 Feb 2021, Michal Hocko wrote: [...] > > Overall, I am not really happy about this feature even when above is > > fixed, but let's hear more the actual problem first. > > Shouldn't this behavior be possible as an oomd plugin instead,

Re: [Intel-gfx] [PATCH v2] drm/i915/gen9bc: Handle TGP PCH during suspend/resume

2021-02-16 Thread Imre Deak
On Tue, Feb 16, 2021 at 09:36:01PM -0500, Lyude Paul wrote: > On Tue, 2021-02-16 at 20:08 +0200, Imre Deak wrote: > > Hi, > > > > thanks for respinning this patchset, some comments below. > > > > On Fri, Feb 12, 2021 at 01:50:53PM -0500, Lyude Paul wrote: > > > From: Tejas Upadhyay > > > > > >

[PATCH v2 bpf-next] bpf: fix a warning message in mark_ptr_not_null_reg()

2021-02-16 Thread Dan Carpenter
The WARN_ON() argument is a condition, not an error message. So this code will print a stack trace but will not print the warning message. Fix that and also change it to only WARN_ONCE(). Fixes: 4ddb74165ae5 ("bpf: Extract nullable reg type conversion into a helper function") Signed-off-by: Dan

Re: [PATCH v5 1/5] dt-bindings: aspeed-lpc: Remove LPC partitioning

2021-02-16 Thread Joel Stanley
Hi Chaiwei, On Wed, 17 Feb 2021 at 07:40, ChiaWei Wang wrote: > > Hi All, > > Do you have update on this patch series? > Aspeed has subsequent LPC module upstream plan. > We hope that the following patches can be on the basis of the fixed LPC > layout. Andrew has expressed his support for your

RE: [PATCH v5 1/5] dt-bindings: aspeed-lpc: Remove LPC partitioning

2021-02-16 Thread ChiaWei Wang
Hi All, Do you have update on this patch series? Aspeed has subsequent LPC module upstream plan. We hope that the following patches can be on the basis of the fixed LPC layout. Thanks. Chiawei > -Original Message- > From: Andrew Jeffery > Sent: Wednesday, January 27, 2021 8:25 AM > To:

[PATCH][RESEND] iio: dac: ad5686: Add support for AD5673R/AD5677R

2021-02-16 Thread Alexandru Ardelean
From: Mircea Caprioru The AD5673R/AD5677R are low power, 16-channel, 12-/16-bit buffered voltage output digital-to-analog converters (DACs). They include a 2.5 V internal reference (enabled by default). These devices are very similar to AD5674R/AD5679R, except that they have an i2c interface.

[PATCH v4 6/6] tools: iio: add example for high-speed buffer support

2021-02-16 Thread Alexandru Ardelean
Following a recent update to the IIO buffer infrastructure, this change adds a basic example on how to access an IIO buffer via the new mmap() interface. The ioctl() for the high-speed mode needs to be enabled right from the start, before setting any parameters via sysfs (length, enable, etc), to

[PATCH v4 5/6] iio: buffer-dma: Add mmap support

2021-02-16 Thread Alexandru Ardelean
From: Lars-Peter Clausen Add support for the new mmap interface to IIO DMA buffer. This interface allows to directly map the backing memory of a block to userspace. This is especially advantageous for high-speed devices where the extra copy from kernel space to userspace of the data incurs a

[PATCH v4 4/6] iio: buffer-dma: reduce the type of block.size to u32

2021-02-16 Thread Alexandru Ardelean
The type of the 'size' parameter in the block struct is size_t which is 64 bit wide on 64 bit archs. When the mmap interface gets introduced, we will use a 32 bit field in the block descriptor for the block size, which will cause some compiler-checks to fail. 32 bits (4 GB) block sizes should be

[PATCH v4 3/6] iio: buffer-dma: split iio_dma_buffer_fileio_free() function

2021-02-16 Thread Alexandru Ardelean
A part of the logic in the iio_dma_buffer_exit() is required for the change to add mmap support to IIO buffers. This change splits the logic into a separate function, which will be re-used later. Signed-off-by: Alexandru Ardelean --- drivers/iio/buffer/industrialio-buffer-dma.c | 45

[PATCH v4 2/6] Documentation: iio: add doc for high-speed buffer API

2021-02-16 Thread Alexandru Ardelean
This change takes the comment from the commit that introduces the IIO high-speed buffer API, and formats it into rst format. Signed-off-by: Alexandru Ardelean --- Documentation/iio/iio_high_speed_buffers.rst | 100 +++ Documentation/iio/index.rst | 2 +

[PATCH v4 1/6] iio: core: Add mmap interface infrastructure

2021-02-16 Thread Alexandru Ardelean
From: Lars-Peter Clausen Add the necessary infrastructure to the IIO core to support an mmap based interface to access the capture data. The advantage of the mmap based interface compared to the read() based interface is that it avoids an extra copy of the data between kernel and userspace.

[PATCH v4 0/6] iio: core: Add mmap interface infrastructure

2021-02-16 Thread Alexandru Ardelean
Changelog v3 -> v4: * https://lore.kernel.org/linux-iio/20210215143234.3248-5-alexandru.ardel...@analog.com/T/ * added patch 'iio: buffer-dma: reduce the type of block.size to u32' - resolves error on 64 bit archs; 32 bit block size should be enough * in patch 'iio: buffer-dma: Add mmap

[PATCH 4/4] ipmi: kcs_bmc: Simplify state machine

2021-02-16 Thread William A. Kennington III
This makes a few changes, notably: - Removes the mutex primitive as it is not needed in the file operation path. The read path had multiple spinlock acquisitions with state changes that don't appear safe to reordering, specifically around forced aborts. - Removes multiple

[PATCH 3/4] ipmi: kcs_bmc: Invert allocation

2021-02-16 Thread William A. Kennington III
This is a more typical pattern for having the platform specific drivers store their own data around the generic driver data. Signed-off-by: William A. Kennington III --- drivers/char/ipmi/kcs_bmc.c | 40 +++--- drivers/char/ipmi/kcs_bmc.h | 13 +-

[PATCH 3/4] ipmi: kcs_bmc: Invert allocation pattern

2021-02-16 Thread William A. Kennington III
The generic kcs_bmc code shouldn't need to store data on behalf of the platform specific driver that initializes it. Instead, have the platform specific driver allocate memory and call a generic initialization routine. Signed-off-by: William A. Kennington III --- drivers/char/ipmi/kcs_bmc.c

[PATCH 2/4] ipmi: kcs_bmc: Remove register abstraction

2021-02-16 Thread William A. Kennington III
The generic kcs_bmc code doesn't care about how byte reads / writes are implemented. This pushes that logic to the device specific handlers, so that kcs_bmc doesn't need to understand registers and how to read or write them. Signed-off-by: William A. Kennington III ---

[PATCH 1/4] ipmi: kcs_bmc: Simplify irq handling

2021-02-16 Thread William A. Kennington III
Platforms specific IRQ handles repeat the same logic, calling a sub-handler in the kcs_bmc generic code that should just conform to the irqhandler callback. Signed-off-by: William A. Kennington III --- drivers/char/ipmi/kcs_bmc.c | 10 +- drivers/char/ipmi/kcs_bmc.h | 3

Re: [PATCH v1 01/12] gna: add driver module

2021-02-16 Thread Maciej Kwapulinski
Greg Kroah-Hartman writes: > On Tue, Feb 16, 2021 at 05:05:14PM +0100, Maciej Kwapulinski wrote: >> --- /dev/null >> +++ b/drivers/misc/gna/gna_driver.c >> @@ -0,0 +1,65 @@ >> +// SPDX-License-Identifier: GPL-2.0-only >> +// Copyright(c) 2017-2021 Intel Corporation >> + >> +#define pr_fmt(fmt)

Re: [PATCH 2/2] ath9k: fix ath_tx_process_buffer() potential null ptr dereference

2021-02-16 Thread Kalle Valo
Shuah Khan writes: > On 2/16/21 12:53 AM, Felix Fietkau wrote: >> >> On 2021-02-16 08:03, Kalle Valo wrote: >>> Shuah Khan wrote: >>> ath_tx_process_buffer() references ieee80211_find_sta_by_ifaddr() return pointer (sta) outside null check. Fix it by moving the code block under

Re: [PATCH v3 0/5] iio: core: Add mmap interface infrastructure

2021-02-16 Thread Alexandru Ardelean
On Tue, Feb 16, 2021 at 10:02 PM Jonathan Cameron wrote: > > On Mon, 15 Feb 2021 16:32:29 +0200 > Alexandru Ardelean wrote: > > > Hi Alex, > > I'm a little nervous about adding the userspace interface used in this > series, but it seems reasonable and as you say below, is close to what > Analog

Re: [PATCH 0/2] block: avoid to drop & re-add partitions if partitions aren't changed

2021-02-16 Thread Christoph Hellwig
On Wed, Feb 17, 2021 at 11:07:14AM +0800, Ming Lei wrote: > Do you think it is correct for ioctl(BLKRRPART) to always drop/re-add > partition device node? Yes, that is what it is designed to do. The only reason to call this ioctl is when userspace software has written new partition table

Re: [PATCH net-next v2] misc: Add Renesas Synchronization Management Unit (SMU) support

2021-02-16 Thread Greg KH
On Tue, Feb 16, 2021 at 07:50:49PM +, Min Li wrote: > > > > Yes, that line. > > > > The documentation should tell you how to do that, as per my patch bot: > > > > - This looks like a new version of a previously submitted patch, but you > > did not list below the --- line any changes from

Re: [PATCH v2] Staging: mt7621-pci: fixed a blank line coding style issue

2021-02-16 Thread Sergio Paracuellos
On Wed, Feb 17, 2021 at 8:07 AM Selvakumar Elangovan wrote:> > Removed an unnecessary blank line before closing brace reported by > checkpatch.pl > > Signed-off-by: Selvakumar Elangovan > --- > drivers/staging/mt7621-pci/pci-mt7621.c | 1 - > 1 file changed, 1 deletion(-) Reviewed-by: Sergio

[PATCH v2] Staging: mt7621-pci: fixed a blank line coding style issue

2021-02-16 Thread Selvakumar Elangovan
Removed an unnecessary blank line before closing brace reported by checkpatch.pl Signed-off-by: Selvakumar Elangovan --- drivers/staging/mt7621-pci/pci-mt7621.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/mt7621-pci/pci-mt7621.c b/drivers/staging/mt7621-pci/pci-mt7621.c

Re: [PATCH] Makefile: Remove # characters from compiler string

2021-02-16 Thread Masahiro Yamada
On Wed, Feb 17, 2021 at 6:33 AM Nathan Chancellor wrote: > > When using AMD's Optimizing C/C++ Compiler (AOCC), the build fails due > to a # character in the version string, which is interpreted as a > comment: > > $ make CC=clang defconfig init/main.o > include/config/auto.conf.cmd:1374: ***

Re: [PATCH] Staging: mt7621-pci: pci-mt7621: fixed a blank line coding style issue

2021-02-16 Thread Sergio Paracuellos
Hi Selvakumar, On Wed, Feb 17, 2021 at 7:53 AM Selvakumar Elangovan wrote: > > Removed an unecessary blank line before closing brace reported by > checkpatch.pl Typo: s/unecessary/unnecessary/g > > Signed-off-by: Selvakumar Elangovan > --- > drivers/staging/mt7621-pci/pci-mt7621.c | 1 - >

RE: [Intel-gfx] [drm/i915] 04ff178484: phoronix-test-suite.supertuxkart.1024x768.Fullscreen.Ultimate.1.GranParadisoIsland.frames_per_second -30.4% regression

2021-02-16 Thread Chiou, Cooper
Hi upstream committee, Please consider to merge this patch which resolved VP8 hardware encoding GPU hang critical issue on Gen9 sku, and there is no performance regression on this fix. Best Regards, Cooper -Original Message- From: kernel test robot Sent: Monday, February 8, 2021

[PATCH v4 net-next] net: socket: use BIT() for MSG_*

2021-02-16 Thread menglong8 . dong
From: Menglong Dong The bit mask for MSG_* seems a little confused here. Replace it with BIT() to make it clear to understand. Signed-off-by: Menglong Dong --- v4: - CC netdev v3: - move changelog here v2: - use BIT() instead of BIT_MASK() --- include/linux/socket.h | 71

[PATCH] Staging: mt7621-pci: pci-mt7621: fixed a blank line coding style issue

2021-02-16 Thread Selvakumar Elangovan
Removed an unecessary blank line before closing brace reported by checkpatch.pl Signed-off-by: Selvakumar Elangovan --- drivers/staging/mt7621-pci/pci-mt7621.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/mt7621-pci/pci-mt7621.c b/drivers/staging/mt7621-pci/pci-mt7621.c

Re: [PATCH v1] vdpa/mlx5: Restore the hardware used index after change map

2021-02-16 Thread Eli Cohen
On Tue, Feb 16, 2021 at 04:25:20PM -0800, Si-Wei Liu wrote: > > > > The saved mvq->avail_idx will be used to recreate hardware virtq object > > > and > > > the used index in create_virtqueue(), once status DRIVER_OK is set. I > > > suspect we should pass the index to mvq->used_idx in > > >

[PATCH V0 6/6] MAINTAINERS:Added the entry for DCC(Data Capture and Compare) driver support

2021-02-16 Thread Souradeep Chowdhury
Added the entries for all the files added as a part of driver support for DCC(Data Capture and Compare). Signed-off-by: Souradeep Chowdhury --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 546aa66..f0d208a 100644 --- a/MAINTAINERS +++

[PATCH V0 2/6] arm64: dts: qcom: sm8150: Add Data Capture and Compare(DCC) support node

2021-02-16 Thread Souradeep Chowdhury
Add the DCC(Data Capture and Compare) device tree node entry along with the addresses for register regions. Signed-off-by: Souradeep Chowdhury --- arch/arm64/boot/dts/qcom/sm8150.dtsi | 7 +++ 1 file changed, 7 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/sm8150.dtsi

[PATCH V0 3/6] soc: qcom: dcc:Add driver support for Data Capture and Compare unit(DCC)

2021-02-16 Thread Souradeep Chowdhury
The DCC is a DMA Engine designed to capture and store data during system crash or software triggers.The DCC operates based on link list entries which provides it with data and addresses and the function it needs to perform.These functions are read,write and loop.Added the basic driver in this

[PATCH V0 5/6] DCC:Added the sysfs entries for DCC(Data Capture and Compare) driver

2021-02-16 Thread Souradeep Chowdhury
The DCC is a DMA engine designed to store register values either in case of a system crash or in case of software triggers manually done by the user.Using DCC hardware and the sysfs interface of the driver the user can exploit various functionalities of DCC.The user can specify the register

[PATCH V0 4/6] soc: qcom: dcc:Add the sysfs variables to the Data Capture and Compare driver(DCC)

2021-02-16 Thread Souradeep Chowdhury
Added the sysfs variables to expose the user space functionalities like DCC enable,disable,configure addresses and software triggers. Also added the necessary methods along with the same. Signed-off-by: Souradeep Chowdhury --- drivers/soc/qcom/dcc.c | 519

[PATCH V0 0/6] Add driver support for Data Capture and Compare Engine(DCC) for SM8150

2021-02-16 Thread Souradeep Chowdhury
DCC(Data Capture and Compare) is a DMA engine designed for debugging purposes.In case of a system crash or manual software triggers by the user the DCC hardware stores the value at the register addresses which can be used for debugging purposes.The DCC driver provides the user with sysfs

[PATCH V0 1/6] dt-bindings: Added the yaml bindings for DCC

2021-02-16 Thread Souradeep Chowdhury
Documentation for Data Capture and Compare(DCC) device tree bindings in yaml format. Signed-off-by: Souradeep Chowdhury --- .../devicetree/bindings/arm/msm/qcom,dcc.yaml | 49 ++ 1 file changed, 49 insertions(+) create mode 100644

Re: [PATCH] mm: slub: Convert sys slab alloc_calls, free_calls to bin attribute

2021-02-16 Thread Faiyaz Mohammed
Hi Vlastimil, On 1/13/2021 9:35 PM, Vlastimil Babka wrote: > On 1/12/21 10:21 AM, Faiyaz Mohammed wrote: >> Reading the sys slab alloc_calls, free_calls returns the available object >> owners, but the size of this file is limited to PAGE_SIZE >> because of the limitation of sysfs attributes, it

Re: [RESEND PATCH] drm/tilcdc: fix raster control register setting

2021-02-16 Thread Tomi Valkeinen
On 16/02/2021 22:22, Dario Binacchi wrote: > The fdd property of the tilcdc_panel_info structure must set the reqdly > bit field (bit 12 to 19) of the raster control register. The previous > statement set the least significant bit instead. > > Signed-off-by: Dario Binacchi > > --- > >

Re: [v8 PATCH 10/13] mm: vmscan: use per memcg nr_deferred of shrinker

2021-02-16 Thread Kirill Tkhai
On 17.02.2021 03:13, Yang Shi wrote: > Use per memcg's nr_deferred for memcg aware shrinkers. The shrinker's > nr_deferred > will be used in the following cases: > 1. Non memcg aware shrinkers > 2. !CONFIG_MEMCG > 3. memcg is disabled by boot parameter > > Signed-off-by: Yang Shi

Re: [PATCH v2 0/5] dax-device: Some cleanups

2021-02-16 Thread Uwe Kleine-König
Hello Dan, On 2/17/21 4:55 AM, Dan Williams wrote: One small comment on patch5, otherwise looks good. I take it back, patch5 looks good. I was going to ask about the return value removal for dax_bus_remove(), but that would need struct bus_type to change prototypes. Changing struct

Re: KASAN: invalid-access Write in enqueue_timer

2021-02-16 Thread Dmitry Vyukov
On Tue, Feb 16, 2021 at 7:15 PM Dmitry Vyukov wrote: > > On Tue, Feb 16, 2021 at 06:50:20PM +0100, Jason A. Donenfeld wrote: > > > On Tue, Feb 16, 2021 at 6:46 PM Jason A. Donenfeld > > > wrote: > > > > On Tue, Feb 16, 2021 at 6:28 PM Catalin Marinas > > > > wrote: > > > > > > hlist_add_head

Re: [v8 PATCH 09/13] mm: vmscan: add per memcg shrinker nr_deferred

2021-02-16 Thread Kirill Tkhai
On 17.02.2021 03:13, Yang Shi wrote: > Currently the number of deferred objects are per shrinker, but some slabs, > for example, > vfs inode/dentry cache are per memcg, this would result in poor isolation > among memcgs. > > The deferred objects typically are generated by __GFP_NOFS

Re: [PATCH] mm: slub: Convert sys slab alloc_calls, free_calls to bin attribute

2021-02-16 Thread Faiyaz Mohammed
Hi Matthew, On 1/12/2021 5:52 PM, Matthew Wilcox wrote: > On Tue, Jan 12, 2021 at 02:51:27PM +0530, Faiyaz Mohammed wrote: >> @@ -5180,6 +5187,7 @@ static int any_slab_objects(struct kmem_cache *s) >> >> struct slab_attribute { >> struct attribute attr; >> +struct bin_attribute

Re: [v8 PATCH 05/13] mm: vmscan: use kvfree_rcu instead of call_rcu

2021-02-16 Thread Kirill Tkhai
On 17.02.2021 03:13, Yang Shi wrote: > Using kvfree_rcu() to free the old shrinker_maps instead of call_rcu(). > We don't have to define a dedicated callback for call_rcu() anymore. > > Signed-off-by: Yang Shi Acked-by: Kirill Tkhai > --- > mm/vmscan.c | 7 +-- > 1 file changed, 1

Re: [PATCH] remove unused variable driver_desc

2021-02-16 Thread Greg Kroah-Hartman
On Tue, Feb 16, 2021 at 10:59:51PM -0500, Sean Behan wrote: > --- > drivers/staging/emxx_udc/emxx_udc.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/staging/emxx_udc/emxx_udc.c > b/drivers/staging/emxx_udc/emxx_udc.c > index 3536c03ff523..741147a4f0fe 100644 > ---

arch/sh/kernel/ftrace.c:368:17: sparse: sparse: incorrect type in argument 1 (different address spaces)

2021-02-16 Thread kernel test robot
Hi Luc, First bad commit (maybe != root cause): tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: f40ddce88593482919761f74910f42f4b84c004b commit: e5fc436f06eef54ef512ea55a9db8eb9f2e76959 sparse: use static inline for __chk_{user,io}_ptr() date: 6

[PATCH v3 net-next 6/7] ptp: ptp_clockmatrix: Simplify code - remove unnecessary `err` variable.

2021-02-16 Thread vincent.cheng.xh
From: Vincent Cheng Code clean-up. Signed-off-by: Vincent Cheng --- drivers/ptp/ptp_clockmatrix.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/ptp/ptp_clockmatrix.c b/drivers/ptp/ptp_clockmatrix.c index 241bff0..dc42c36 100644 ---

[PATCH v3 net-next 0/7] ptp: ptp_clockmatrix: Fix output 1 PPS alignment.

2021-02-16 Thread vincent.cheng.xh
From: Vincent Cheng This series fixes a race condition that may result in the output clock not aligned to internal 1 PPS clock. Part of device initialization is to align the rising edge of output clocks to the internal rising edge of the 1 PPS clock. If the system APLL and DPLL are not locked

[PATCH v3 net-next 2/7] ptp: ptp_clockmatrix: Add alignment of 1 PPS to idtcm_perout_enable.

2021-02-16 Thread vincent.cheng.xh
From: Vincent Cheng When enabling output using PTP_CLK_REQ_PEROUT, need to align the output clock to the internal 1 PPS clock. Signed-off-by: Vincent Cheng Acked-by: Richard Cochran --- drivers/ptp/ptp_clockmatrix.c | 16 +--- 1 file changed, 13 insertions(+), 3 deletions(-)

[PATCH v3 net-next 4/7] ptp: ptp_clockmatrix: Clean-up dev_*() messages.

2021-02-16 Thread vincent.cheng.xh
From: Vincent Cheng Code clean-up. * Remove unnecessary \n termination from dev_*() messages. * Remove 'char *fmt' to define strings to stay within 80 column limit. Not needed since coding guidelines increased to 100 columns limit. Keeping format in place allows static code checkers to

Re: [PATCH v2 1/2] exfat: add initial ioctl function

2021-02-16 Thread Hyeongseok Kim
On 2/17/21 2:39 PM, Namjae Jeon wrote: Hi Hyeongseok, Do you have any other opinion about this? I also think this patch should be combined with the 2/2 patch. If you agree, I'll merge these as one. Yep, Agreed. Please do that:) Thanks! Thank you for the opinion. I sent out v3.

[PATCH v3 net-next 5/7] ptp: ptp_clockmatrix: Coding style - tighten vertical spacing.

2021-02-16 Thread vincent.cheng.xh
From: Vincent Cheng Code clean-up. * Remove blank line between variable declarations. * Remove blank line between: err = blah(...) if (err) ... * Remove unnecessary blank line before/after loop constructs. Signed-off-by: Vincent Cheng ---

[PATCH v3 0/1] Add FITRIM ioctl support for exFAT filesystem

2021-02-16 Thread Hyeongseok Kim
This is for adding FITRIM ioctl functionality to exFAT filesystem. To do that, add generic ioctl function and FITRIM handler. Changelog = v2->v3: - Remove unnecessary local variable - Merge all changes to a single patch v1->v2: - Change variable declaration order as reverse tree style. -

[PATCH v3 1/1] exfat: add support ioctl and FITRIM function

2021-02-16 Thread Hyeongseok Kim
Add FITRIM ioctl to enable discarding unused blocks while mounted. As current exFAT doesn't have generic ioctl handler, add empty ioctl function first, and add FITRIM handler. Signed-off-by: Hyeongseok Kim --- fs/exfat/balloc.c | 81 +

[PATCH v3 net-next 1/7] ptp: ptp_clockmatrix: Add wait_for_sys_apll_dpll_lock.

2021-02-16 Thread vincent.cheng.xh
From: Vincent Cheng Part of the device initialization aligns the rising edge of the output clock to the internal 1 PPS clock. If the system APLL and DPLL is not locked, then the alignment will fail and there will be a fixed offset between the internal 1 PPS clock and the output clock. After

[PATCH v3 net-next 3/7] ptp: ptp_clockmatrix: Remove unused header declarations.

2021-02-16 Thread vincent.cheng.xh
From: Vincent Cheng Removed unused header declarations. Signed-off-by: Vincent Cheng --- drivers/ptp/ptp_clockmatrix.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/ptp/ptp_clockmatrix.h b/drivers/ptp/ptp_clockmatrix.h index 0233236..fb32327 100644 ---

[PATCH v3 net-next 7/7] ptp: ptp_clockmatrix: clean-up - parenthesis around a == b are unnecessary

2021-02-16 Thread vincent.cheng.xh
From: Vincent Cheng Code clean-up. Signed-off-by: Vincent Cheng --- drivers/ptp/ptp_clockmatrix.c | 18 -- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/drivers/ptp/ptp_clockmatrix.c b/drivers/ptp/ptp_clockmatrix.c index dc42c36..75463c2 100644 ---

Re: [PATCH 2/2] mmc: core: Add no single read retries

2021-02-16 Thread Adrian Hunter
On 17/02/21 7:46 am, Adrian Hunter wrote: > On 17/02/21 7:22 am, DooHyun Hwang wrote: >> This makes to handle read errors faster by not retrying >> multiple block read(CMD18) errors with single block reads(CMD17). >> >> On some bad SD Cards that have problem with read operations, >> it is not

Re: [PATCH] cpufreq: exclude boost frequencies from valid count if not enabled

2021-02-16 Thread Viresh Kumar
Hi Thara, On 16-02-21, 19:00, Thara Gopinath wrote: > This is a fix for a regression observed on db845 platforms with 5.7-rc11 > kernel. On these platforms running stress tests with 5.11-rc7 kernel > causes big cpus to overheat and ultimately shutdown the system due to > hitting critical

[PATCH] thermal: cpufreq_cooling: freq_qos_update_request() returns < 0 on error

2021-02-16 Thread Viresh Kumar
freq_qos_update_request() returns 1 if the effective constraint value has changed, 0 if the effective constraint value has not changed, or a negative error code on failures. The frequency constraints for CPUs can be set by different parts of the kernel. If the maximum frequency constraint set by

[PATCH v2] psi: Optimize task switch inside shared cgroups

2021-02-16 Thread Chengming Zhou
The commit 36b238d57172 ("psi: Optimize switching tasks inside shared cgroups") only update cgroups whose state actually changes during a task switch only in task preempt case, not in task sleep case. We actually don't need to clear and set TSK_ONCPU state for common cgroups of next and prev task

Re: [PATCH 2/2] mmc: core: Add no single read retries

2021-02-16 Thread Adrian Hunter
On 17/02/21 7:22 am, DooHyun Hwang wrote: > This makes to handle read errors faster by not retrying > multiple block read(CMD18) errors with single block reads(CMD17). > > On some bad SD Cards that have problem with read operations, > it is not helpful to retry multiple block read errors with >

Re: {standard input}:577: Error: unsupported relocation against base

2021-02-16 Thread Michael Ellerman
Segher Boessenkool writes: > Hi! > > On Tue, Feb 16, 2021 at 08:36:02PM +1100, Michael Ellerman wrote: >> Feng Tang writes: >> > {standard input}:577: Error: unsupported relocation against base >> > {standard input}:580: Error: unsupported relocation against base >> > {standard input}:583:

Re: memory leak in path_openat (2)

2021-02-16 Thread Al Viro
On Tue, Feb 16, 2021 at 09:21:14PM -0800, syzbot wrote: > Hello, > > syzbot found the following issue on: > > HEAD commit:dcc0b490 Merge tag 'powerpc-5.11-8' of git://git.kernel.or.. > git tree: upstream > console output: https://syzkaller.appspot.com/x/log.txt?x=1316c614d0 >

Re: [PATCH v2 1/2] exfat: add initial ioctl function

2021-02-16 Thread Namjae Jeon
2021-02-17 9:33 GMT+09:00, Hyeongseok Kim : > On 2/17/21 9:17 AM, Chaitanya Kulkarni wrote: >> On 2/16/21 16:13, Hyeongseok Kim wrote: >>> Sorry, I don't understand exactly. >>> You're saying that these 2 patch should be merged to a single patch? >>> Would it be better? >> I think so unless there

[PATCH 1/2] dt-bindings: mmc: Add no-single-read-retry property

2021-02-16 Thread DooHyun Hwang
Add an optional property to not retry multiple block read error with several single block reads. This property makes to handle read errors faster by not retrying multiple block read errors with single block reads. Signed-off-by: DooHyun Hwang ---

[PATCH 2/2] mmc: core: Add no single read retries

2021-02-16 Thread DooHyun Hwang
This makes to handle read errors faster by not retrying multiple block read(CMD18) errors with single block reads(CMD17). On some bad SD Cards that have problem with read operations, it is not helpful to retry multiple block read errors with several single block reads, and it is delayed to treat

[PATCH 0/2] mmc: core: add a new property

2021-02-16 Thread DooHyun Hwang
Add an optional property to not retry multiple block read error with several single block reads. DooHyun Hwang (2): dt-bindings: mmc: Add no-single-read-retry property mmc: core: Add no single read retries Documentation/devicetree/bindings/mmc/mmc-controller.yaml | 6 ++

Re: DT overlay applied via pinctrl description

2021-02-16 Thread Frank Rowand
+Frank, Rob, devicetree list On 2/16/21 9:35 AM, Michal Simek wrote: > Hi, > > I have a question about expectations when pinctrl setting is applied. In > DTS all nodes are described in the order available in DT. > > uart-default { > mux { > ... > }; > > conf { >

[PATCH 1/2] dt-bindings: mmc: Add no-single-read-retry property

2021-02-16 Thread DooHyun Hwang
Add an optional property to not retry multiple block read error with several single block reads. This property makes to handle read errors faster by not retrying multiple block read errors with single block reads. Signed-off-by: DooHyun Hwang ---

[PATCH 2/2] mmc: core: Add no single read retries

2021-02-16 Thread DooHyun Hwang
This makes to handle read errors faster by not retrying multiple block read(CMD18) errors with single block reads(CMD17). On some bad SD Cards that have problem with read operations, it is not helpful to retry multiple block read errors with several single block reads, and it is delayed to treat

[PATCH 0/2] mmc: core: add a new property

2021-02-16 Thread DooHyun Hwang
Add an optional property to not retry multiple block read error with several single block reads. DooHyun Hwang (2): dt-bindings: mmc: Add no-single-read-retry property mmc: core: Add no single read retries Documentation/devicetree/bindings/mmc/mmc-controller.yaml | 6 ++

memory leak in path_openat (2)

2021-02-16 Thread syzbot
Hello, syzbot found the following issue on: HEAD commit:dcc0b490 Merge tag 'powerpc-5.11-8' of git://git.kernel.or.. git tree: upstream console output: https://syzkaller.appspot.com/x/log.txt?x=1316c614d0 kernel config: https://syzkaller.appspot.com/x/.config?x=a2fbb1a71525e1d5

Re: [External] Re: [PATCH 4/4] mm: memcontrol: fix swap uncharge on cgroup v2

2021-02-16 Thread Muchun Song
On Wed, Feb 17, 2021 at 1:28 AM Johannes Weiner wrote: > > On Tue, Feb 16, 2021 at 11:59:01AM -0500, Johannes Weiner wrote: > > Hello Muchun, > > > > On Sat, Feb 13, 2021 at 01:01:59AM +0800, Muchun Song wrote: > > > The swap charges the actual number of swap entries on cgroup v2. > > > If a swap

Re: [External] Re: [PATCH 4/4] mm: memcontrol: fix swap uncharge on cgroup v2

2021-02-16 Thread Muchun Song
On Wed, Feb 17, 2021 at 12:59 AM Johannes Weiner wrote: > > Hello Muchun, > > On Sat, Feb 13, 2021 at 01:01:59AM +0800, Muchun Song wrote: > > The swap charges the actual number of swap entries on cgroup v2. > > If a swap cache page is charged successful, and then we uncharge > > the swap

Re: [External] Re: [PATCH] psi: Optimize task switch inside shared cgroups

2021-02-16 Thread Chengming Zhou
Hello Johannes, 在 2021/2/17 上午4:00, Johannes Weiner 写道: > Hello Chengming, > > This patch looks useful to me. A couple of comments below: > > On Tue, Feb 09, 2021 at 03:14:13PM +0800, Chengming Zhou wrote: >> The commit 36b238d57172 ("psi: Optimize switching tasks inside shared >> cgroups") only

linux-next: manual merge of the tip tree with the pm tree

2021-02-16 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the tip tree got a conflict in: arch/x86/platform/intel-mid/device_libs/platform_bt.c between commit: 4590d98f5a4f ("sfi: Remove framework for deprecated firmware") from the pm tree and commit: bdb154f074a6 ("x86/platform/intel-mid: Convert comma to

Re: [SPAM?]Re: arch/m68k/68000/dragen2.c:73:16: error: 'screen_bits' undeclared

2021-02-16 Thread Greg Ungerer
Hi Arnd, On 12/2/21 8:05 pm, Arnd Bergmann wrote: On Fri, Feb 12, 2021 at 6:48 AM kernel test robot wrote: FYI, the error/warning still remains. | ^~~~ arch/m68k/68000/dragen2.c: In function 'init_dragen2': arch/m68k/68000/dragen2.c:73:16: error:

Re: [PATCH] opp: fix dev_pm_opp_set_rate for different frequency at the same opp level

2021-02-16 Thread Viresh Kumar
On 16-02-21, 15:10, Jonathan Marek wrote: > There is not "nothing to do" when the opp is the same. The frequency can > be different from opp->rate. I am sorry but I am not sure what are you trying to fix here and what exactly is broken here. Can you provide a usecase for your platform where this

Re: [PATCH] dt-bindings: cpufreq: cpufreq-qcom-hw: Document SM8350 CPUfreq compatible

2021-02-16 Thread Viresh Kumar
On 16-02-21, 16:42, Vinod Koul wrote: > Add the CPUfreq compatible for SM8350 SoC along with note for using the > specific compatible for SoCs > > Signed-off-by: Vinod Koul > --- > Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.txt | 4 +++- > 1 file changed, 3 insertions(+), 1

Re: [PATCH v2] vfs: prevent copy_file_range to copy across devices

2021-02-16 Thread Nicolas Boichat
On Mon, Feb 15, 2021 at 11:42 PM Luis Henriques wrote: > > Nicolas Boichat reported an issue when trying to use the copy_file_range > syscall on a tracefs file. It failed silently because the file content is > generated on-the-fly (reporting a size of zero) and copy_file_range needs > to know in

Re: riscv+KASAN does not boot

2021-02-16 Thread Dmitry Vyukov
On Tue, Feb 16, 2021 at 9:42 PM Alex Ghiti wrote: > > Hi Dmitry, > > Le 2/16/21 à 6:25 AM, Dmitry Vyukov a écrit : > > On Tue, Feb 16, 2021 at 12:17 PM Dmitry Vyukov wrote: > >> > >> On Fri, Jan 29, 2021 at 9:11 AM Dmitry Vyukov wrote: > I was fixing KASAN support for my sv48 patchset so I

[tip:x86/entry] BUILD SUCCESS 3aac798a917be3b8f2f647b834bb06bf2f8df4f1

2021-02-16 Thread kernel test robot
powerpc allmodconfig powerpc allnoconfig x86_64 randconfig-a003-20210216 x86_64 randconfig-a002-20210216 x86_64 randconfig-a004-20210216 x86_64 randconfig-a001-20210216 x86_64

[tip:master] BUILD SUCCESS 74b68949724a504c579e0073e23beba7a71602ba

2021-02-16 Thread kernel test robot
allyesconfig powerpc allmodconfig powerpc allnoconfig i386 randconfig-a003-20210216 i386 randconfig-a005-20210216 i386 randconfig-a002-20210216 i386 randconfig-a006

Re: [PATCH v9 0/7] FPGA Security Manager Class Driver

2021-02-16 Thread Moritz Fischer
Hi Russ, On Tue, Feb 16, 2021 at 09:46:53AM -0800, Russ Weight wrote: > I believe all of the dependencies have been accepted now. > > - Russ Sorry for dropping the ball on this, I'll get to this ASAP after -rc1 is tagged. > > On 2/15/21 6:56 AM, Tom Rix wrote: > > Russ, Moritz > > > > This

Re: [PATCH V3 1/2] topology: Allow multiple entities to provide sched_freq_tick() callback

2021-02-16 Thread Viresh Kumar
On 17-02-21, 00:24, Ionela Voinescu wrote: > I think it could be merged in patch 1/2 as it's part of enabling the use > of multiple sources of information for FIE. Up to you! Sure. > > static void amu_fie_setup(const struct cpumask *cpus) > > { > > - bool invariant; > > int cpu; > > >

[PATCH net-next v2 5/5] net: stmmac: dwmac-sun8i: Add a shutdown callback

2021-02-16 Thread Samuel Holland
The Ethernet MAC and PHY are usually major consumers of power on boards which may not be able to fully power off (those with no PMIC). Powering down the MAC and internal PHY saves power while these boards are "off". Reviewed-by: Chen-Yu Tsai Signed-off-by: Samuel Holland ---

[PATCH net-next v2 1/5] net: stmmac: dwmac-sun8i: Return void from PHY unpower

2021-02-16 Thread Samuel Holland
This is a deinitialization function that always returned zero, and that return value was always ignored. Have it return void instead. Reviewed-by: Chen-Yu Tsai Signed-off-by: Samuel Holland --- drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 5 ++--- 1 file changed, 2 insertions(+), 3

[PATCH net-next v2 3/5] net: stmmac: dwmac-sun8i: Use reset_control_reset

2021-02-16 Thread Samuel Holland
Use the appropriate function instead of reimplementing it, and update the error message to match the code. Reviewed-by: Chen-Yu Tsai Signed-off-by: Samuel Holland --- drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH net-next v2 0/5] dwmac-sun8i cleanup and shutdown hook

2021-02-16 Thread Samuel Holland
These patches clean up some things I noticed while fixing suspend/resume behavior. The first four are minor code improvements. The last one adds a shutdown hook to minimize power consumption on boards without a PMIC. Changes v1 to v2: - Note the assumption of exclusive reset controller access

[PATCH net-next v2 4/5] net: stmmac: dwmac-sun8i: Minor probe function cleanup

2021-02-16 Thread Samuel Holland
Adjust the spacing and use an explicit "return 0" in the success path to make the function easier to parse. Reviewed-by: Chen-Yu Tsai Signed-off-by: Samuel Holland --- drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git

[PATCH net-next v2 2/5] net: stmmac: dwmac-sun8i: Remove unnecessary PHY power check

2021-02-16 Thread Samuel Holland
sun8i_dwmac_unpower_internal_phy already checks if the PHY is powered, so there is no need to do it again here. Reviewed-by: Chen-Yu Tsai Signed-off-by: Samuel Holland --- drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git

[RFC PATCH v5 9/9] cxl/mem: Add payload dumping for debug

2021-02-16 Thread Ben Widawsky
It's often useful in debug scenarios to see what the hardware has dumped out. As it stands today, any device error will result in the payload not being copied out, so there is no way to triage commands which weren't expected to fail (and sometimes the payload may have that information). The

[PATCH] remove unused variable driver_desc

2021-02-16 Thread Sean Behan
--- drivers/staging/emxx_udc/emxx_udc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging/emxx_udc/emxx_udc.c index 3536c03ff523..741147a4f0fe 100644 --- a/drivers/staging/emxx_udc/emxx_udc.c +++ b/drivers/staging/emxx_udc/emxx_udc.c @@

[PATCH v5 7/9] cxl/mem: Add set of informational commands

2021-02-16 Thread Ben Widawsky
Add initial set of formal commands beyond basic identify and command enumeration. Signed-off-by: Ben Widawsky Reviewed-by: Dan Williams Reviewed-by: Jonathan Cameron (v2) --- drivers/cxl/mem.c| 9 + include/uapi/linux/cxl_mem.h | 5 + 2 files changed, 14 insertions(+)

[PATCH v5 6/9] cxl/mem: Enable commands via CEL

2021-02-16 Thread Ben Widawsky
CXL devices identified by the memory-device class code must implement the Device Command Interface (described in 8.2.9 of the CXL 2.0 spec). While the driver already maintains a list of commands it supports, there is still a need to be able to distinguish between commands that the driver knows

[PATCH v5 8/9] MAINTAINERS: Add maintainers of the CXL driver

2021-02-16 Thread Ben Widawsky
Cc: Dan Williams Cc: Vishal Verma Cc: Ira Weiny Cc: Alison Schofield Signed-off-by: Ben Widawsky --- MAINTAINERS | 11 +++ 1 file changed, 11 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 6eff4f720c72..93c8694a8f04 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -,6

  1   2   3   4   5   6   7   8   9   10   >