Re: [PATCH v9 2/4] fs: Add standard casefolding support

2020-06-23 Thread Eric Biggers
On Tue, Jun 23, 2020 at 09:33:39PM -0700, Daniel Rosenberg wrote: > This adds general supporting functions for filesystems that use > utf8 casefolding. It provides standard dentry_operations and adds the > necessary structures in struct super_block to allow this standardization. > > Ext4 and F2fs

Re: [PATCH][next] dmaengine: hisilicon: Use struct_size() in devm_kzalloc()

2020-06-23 Thread Vinod Koul
On 17-06-20, 16:11, Gustavo A. R. Silva wrote: > Make use of the struct_size() helper instead of an open-coded version > in order to avoid any potential type mistakes. > > This code was detected with the help of Coccinelle and, audited and > fixed manually. Applied, thanks -- ~Vinod

Re: [PATCH][next] dmaengine: ti: k3-udma: Use struct_size() in kzalloc()

2020-06-23 Thread Joe Perches
On Wed, 2020-06-24 at 11:25 +0530, Vinod Koul wrote: > On 19-06-20, 17:43, Gustavo A. R. Silva wrote: > > Make use of the struct_size() helper instead of an open-coded version > > in order to avoid any potential type mistakes. > > > > This code was detected with the help of Coccinelle and,

Re: [PATCH v12 00/10] Introduce support for guest CET feature

2020-06-23 Thread Yang Weijiang
On Tue, Jun 23, 2020 at 11:39:19AM -0700, Sean Christopherson wrote: > On Thu, Jun 11, 2020 at 09:29:13AM +0800, Yang Weijiang wrote: > > On Wed, Jun 10, 2020 at 09:56:36AM -0700, Sean Christopherson wrote: > > > On Wed, May 06, 2020 at 04:20:59PM +0800, Yang Weijiang wrote: > > > > Several parts

Re: [PATCH 5.4 000/314] 5.4.49-rc1 review

2020-06-23 Thread Greg Kroah-Hartman
On Tue, Jun 23, 2020 at 09:14:21PM -0700, Nathan Chancellor wrote: > On Tue, Jun 23, 2020 at 09:53:15PM +0200, Greg Kroah-Hartman wrote: > > This is the start of the stable review cycle for the 5.4.49 release. > > There are 314 patches in this series, all will be posted as a response > > to this

Re: [PATCH][next] dmaengine: ti: k3-udma: Use struct_size() in kzalloc()

2020-06-23 Thread Vinod Koul
On 19-06-20, 17:43, Gustavo A. R. Silva wrote: > Make use of the struct_size() helper instead of an open-coded version > in order to avoid any potential type mistakes. > > This code was detected with the help of Coccinelle and, audited and > fixed manually. > > Addresses-KSPP-ID:

Re: [PATCH 5.4 000/314] 5.4.49-rc1 review

2020-06-23 Thread Greg Kroah-Hartman
On Tue, Jun 23, 2020 at 10:05:35PM -0700, Guenter Roeck wrote: > On 6/23/20 12:53 PM, Greg Kroah-Hartman wrote: > > This is the start of the stable review cycle for the 5.4.49 release. > > There are 314 patches in this series, all will be posted as a response > > to this one. If anyone has any

Re: [PATCH 5.7 000/477] 5.7.6-rc1 review

2020-06-23 Thread Greg Kroah-Hartman
On Tue, Jun 23, 2020 at 10:07:08PM -0700, Guenter Roeck wrote: > On 6/23/20 12:49 PM, Greg Kroah-Hartman wrote: > > This is the start of the stable review cycle for the 5.7.6 release. > > There are 477 patches in this series, all will be posted as a response > > to this one. If anyone has any

Re: [PATCH 5.7 318/477] pinctrl: freescale: imx: Use devm_of_iomap() to avoid a resource leak in case of error in imx_pinctrl_probe()

2020-06-23 Thread Greg Kroah-Hartman
On Wed, Jun 24, 2020 at 07:05:05AM +0200, Marion & Christophe JAILLET wrote: > Hi, > > This one must NOT be included. It generates a regression. > This should be removed from 5.4 as well. > > See 13f2d25b951f139064ec2dd53c0c7ebdf8d8007e. > > There is also a thread on ML about it. I couldn't

Re: [PATCH v2 2/2] cpufreq: Specify default governor on command line

2020-06-23 Thread Viresh Kumar
On 23-06-20, 15:21, Quentin Perret wrote: > Currently, the only way to specify the default CPUfreq governor is via > Kconfig options, which suits users who can build the kernel themselves > perfectly. > > However, for those who use a distro-like kernel (such as Android, with > the Generic Kernel

Re: linux-next: build failure after merge of the drm-misc tree

2020-06-23 Thread Dave Airlie
On Wed, 24 Jun 2020 at 11:36, Stephen Rothwell wrote: > > Hi all, > > On Wed, 17 Jun 2020 10:59:29 +1000 Stephen Rothwell > wrote: > > > > After merging the drm-misc tree, today's linux-next build (x86_64 > > allmodconfig) failed like this: > > > >

Re: [PATCH 2/3] nvme-pci: Add controller memory buffer supported macro

2020-06-23 Thread Christoph Hellwig
On Tue, Jun 23, 2020 at 07:58:17PM -0700, Keith Busch wrote: > On Tue, Jun 23, 2020 at 06:27:51PM +0200, Christoph Hellwig wrote: > > On Tue, Jun 23, 2020 at 09:24:33PM +0800, Baolin Wang wrote: > > > Introduce a new capability macro to indicate if the controller > > > supports the memory buffer

Re: [PATCH v9 4/4] ext4: Use generic casefolding support

2020-06-23 Thread Gabriel Krisman Bertazi
Daniel Rosenberg writes: > - > const struct dentry_operations ext4_dentry_ops = { > - .d_hash = ext4_d_hash, > - .d_compare = ext4_d_compare, > + .d_hash = generic_ci_d_hash, > + .d_compare = generic_ci_d_compare, > }; > #endif Can you make the structure generic since it is

Re: [PATCH v9 2/4] fs: Add standard casefolding support

2020-06-23 Thread Eric Biggers
On Tue, Jun 23, 2020 at 09:33:39PM -0700, Daniel Rosenberg wrote: > This adds general supporting functions for filesystems that use > utf8 casefolding. It provides standard dentry_operations and adds the > necessary structures in struct super_block to allow this standardization. > > Ext4 and F2fs

Re: [PATCH v9 1/4] unicode: Add utf8_casefold_hash

2020-06-23 Thread Eric Biggers
On Tue, Jun 23, 2020 at 09:33:38PM -0700, Daniel Rosenberg wrote: > This adds a case insensitive hash function to allow taking the hash > without needing to allocate a casefolded copy of the string. It would be helpful to add a few more details in this commit message. Somewhat along the lines of:

Re: process '/usr/bin/rsync' started with executable stack

2020-06-23 Thread Christian Kujau
On Tue, 23 Jun 2020, Kees Cook wrote: > > If you run something with exec stack after the message > > you shouldn't get it second time. > > If you want to reset this flag, you can do: > # echo 1 > /sys/kernel/debug/clear_warn_once Thanks. Although, I tend to not mount

Re: [PATCH v9 0/4] Prepare for upcoming Casefolding/Encryption patches

2020-06-23 Thread Eric Biggers
On Tue, Jun 23, 2020 at 09:33:37PM -0700, Daniel Rosenberg wrote: > This lays the ground work for enabling casefolding and encryption at the > same time for ext4 and f2fs. A future set of patches will enable that > functionality. These unify the highly similar dentry_operations that ext4 > and

[tip:ras/core] BUILD SUCCESS bb2de0adca217a114ce023489426e24152e4bfcf

2020-06-23 Thread kernel test robot
powerpc allnoconfig i386 randconfig-a006-20200623 i386 randconfig-a002-20200623 i386 randconfig-a003-20200623 i386 randconfig-a001-20200623 i386 randconfig-a005-20200623 i386

Re: [PATCH v9 2/4] fs: Add standard casefolding support

2020-06-23 Thread Gabriel Krisman Bertazi
Daniel Rosenberg writes: > This adds general supporting functions for filesystems that use > utf8 casefolding. It provides standard dentry_operations and adds the > necessary structures in struct super_block to allow this standardization. > > Ext4 and F2fs will switch to these common

[PATCH 2/9] soundwire: intel: revisit SHIM programming sequences.

2020-06-23 Thread Bard Liao
From: Pierre-Louis Bossart Somehow the existing code is not aligned with the steps described in the documentation, refactor code and make sure the register programming sequences are correct. Also add missing power-up, power-down and wake capabilities (the last two are used in follow-up patches

Re: [PATCH v5 1/2] remoteproc: qcom: Add per subsystem SSR notification

2020-06-23 Thread Bjorn Andersson
On Tue 23 Jun 18:41 PDT 2020, risha...@codeaurora.org wrote: > On 2020-06-23 14:45, Alex Elder wrote: > > On 6/22/20 8:04 PM, Rishabh Bhatnagar wrote: > > > Currently there is a single notification chain which is called > > > whenever any > > > remoteproc shuts down. This leads to all the

[PATCH 6/9] soundwire: intel_init: use EXPORT_SYMBOL_NS

2020-06-23 Thread Bard Liao
From: Pierre-Louis Bossart Make sure all symbols in this soundwire-intel-init module are exported with a namespace. The MODULE_IMPORT_NS will be used in Intel/SOF HDaudio modules to be posted in a separate series. Namespaces are only introduced for the Intel parts of the SoundWire code at this

[PATCH 5/9] soundwire: intel_init: add implementation of sdw_intel_enable_irq()

2020-06-23 Thread Bard Liao
From: Pierre-Louis Bossart This function is required to enable all interrupts across all links. Signed-off-by: Pierre-Louis Bossart Signed-off-by: Bard Liao --- drivers/soundwire/intel_init.c | 24 1 file changed, 24 insertions(+) diff --git

[PATCH 8/9] soundwire: intel: add wake interrupt support

2020-06-23 Thread Bard Liao
From: Rander Wang When system is suspended in clock stop mode on intel platforms, both master and slave are in clock stop mode and soundwire bus is taken over by a glue hardware. The bus message for jack event is processed by this glue hardware, which will trigger an interrupt to resume audio

[PATCH 1/9] soundwire: intel: reuse code for wait loops to set/clear bits

2020-06-23 Thread Bard Liao
From: Pierre-Louis Bossart Refactor code and use same routines on set/clear Signed-off-by: Pierre-Louis Bossart Signed-off-by: Bard Liao --- drivers/soundwire/intel.c | 45 +-- 1 file changed, 19 insertions(+), 26 deletions(-) diff --git

[PATCH 7/9] soundwire: intel/cadence: merge Soundwire interrupt handlers/threads

2020-06-23 Thread Bard Liao
The existing code uses one pair of interrupt handler/thread per link but at the hardware level the interrupt is shared. This works fine for legacy PCI interrupts, but leads to timeouts in MSI (Message-Signaled Interrupt) mode, likely due to edges being lost. This patch unifies interrupt handling

[PATCH 9/9] Soundwire: intel_init: save Slave(s) _ADR info in sdw_intel_ctx

2020-06-23 Thread Bard Liao
Save ACPI information in context so that we can match machine driver with sdw _ADR matching tables. Suggested-by: Guennadi Liakhovetski Signed-off-by: Bard Liao Signed-off-by: Pierre-Louis Bossart --- drivers/soundwire/intel_init.c | 24 1 file changed, 24

[PATCH 3/9] soundwire: intel: introduce a helper to arm link synchronization

2020-06-23 Thread Bard Liao
From: Pierre-Louis Bossart Move code from pre_bank_switch to dedicated helper, will be used in follow-up patches as recommended by programming flows. Signed-off-by: Pierre-Louis Bossart Signed-off-by: Bard Liao --- drivers/soundwire/intel.c | 26 -- 1 file changed, 16

[PATCH 0/9] soundwire: intel: revisit SHIM programming

2020-06-23 Thread Bard Liao
This series does some cleanup, revisits SHIM programming sequences, and merges Soundwire interrupt handlers/threads. Bard Liao (2): soundwire: intel/cadence: merge Soundwire interrupt handlers/threads Soundwire: intel_init: save Slave(s) _ADR info in sdw_intel_ctx Pierre-Louis Bossart (6):

[PATCH 4/9] soundwire: intel: introduce helper for link synchronization

2020-06-23 Thread Bard Liao
From: Pierre-Louis Bossart After arming the synchronization, the SYNCGO field controls the hardware-based synchronization between links. Move the programming and wait for clear of SYNCGO to dedicated helper. Signed-off-by: Pierre-Louis Bossart Signed-off-by: Bard Liao ---

Re: [PATCH v2 9/9] docs: staging: use small font for literal includes

2020-06-23 Thread Mauro Carvalho Chehab
Em Tue, 23 Jun 2020 16:12:04 -0700 Joe Perches escreveu: > On Tue, 2020-06-23 at 11:53 +0200, Mauro Carvalho Chehab wrote: > > The normal font is too big to display 80 columns, causing extra > > breaks to be added at weird places. > > > > change to the footnotesize, as this would fit a little

Re: [dm-devel] [dm-crypt] [RFC PATCH 1/1] Add DM_CRYPT_FORCE_INLINE flag to dm-crypt target

2020-06-23 Thread Eric Biggers
On Wed, Jun 24, 2020 at 05:21:24AM +, Damien Le Moal wrote: > >> @@ -1458,13 +1459,18 @@ static void crypt_alloc_req_skcipher(struct > >> crypt_config *cc, > >> > >>skcipher_request_set_tfm(ctx->r.req, cc->cipher_tfm.tfms[key_index]); > >> > >> - /* > >> - * Use REQ_MAY_BACKLOG so

Re: [RFC PATCH 0/1] dm-crypt excessive overhead

2020-06-23 Thread Mike Snitzer
On Wed, Jun 24 2020 at 12:54am -0400, Damien Le Moal wrote: > On 2020/06/24 0:23, Mike Snitzer wrote: > > On Tue, Jun 23 2020 at 11:07am -0400, > > Ignat Korchagin wrote: > > > >> Do you think it may be better to break it in two flags: one for read > >> path and one for write? So, depending on

Re: [dm-devel] [dm-crypt] [RFC PATCH 1/1] Add DM_CRYPT_FORCE_INLINE flag to dm-crypt target

2020-06-23 Thread Damien Le Moal
On 2020/06/24 14:05, Eric Biggers wrote: > On Fri, Jun 19, 2020 at 05:41:32PM +0100, Ignat Korchagin wrote: >> Sometimes extra thread offloading imposed by dm-crypt hurts IO latency. This >> is >> especially visible on busy systems with many processes/threads. Moreover, >> most >> Crypto API

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

2020-06-23 Thread Sedat Dilek
On Tue, Jun 23, 2020 at 8:44 PM Nick Desaulniers wrote: > > On Mon, Jun 22, 2020 at 7:56 PM Sedat Dilek wrote: > > > > When building with LLVM_IAS=1 means using Clang's Integrated Assembly (IAS) > > from LLVM/Clang >= v10.0.1-rc1+ instead of GNU/as from GNU/binutils > > I see the following

Re: [PATCH] libnvdimm/security: Fix key lookup permissions

2020-06-23 Thread Ira Weiny
On Tue, Jun 23, 2020 at 09:35:26PM -0700, Dan Williams wrote: > As of commit 8c0637e950d6 ("keys: Make the KEY_NEED_* perms an enum rather > than a mask") lookup_user_key() needs an explicit declaration of what it > wants to do with the key. Add KEY_NEED_SEARCH to fix a warning with the > below

[PATCH 5.7 v4] x86/crypto: aesni: Fix build with LLVM_IAS=1

2020-06-23 Thread Sedat Dilek
When building with LLVM_IAS=1 means using Clang's Integrated Assembly (IAS) from LLVM/Clang >= v10.0.1-rc1+ instead of GNU/as from GNU/binutils I see the following breakage in Debian/testing AMD64: :15:74: error: too many positional arguments PRECOMPUTE 8*3+8(%rsp), %xmm1, %xmm2, %xmm3, %xmm4,

Re: [dm-devel] [RFC PATCH 1/1] Add DM_CRYPT_FORCE_INLINE flag to dm-crypt target

2020-06-23 Thread Damien Le Moal
On 2020/06/22 17:47, Ignat Korchagin wrote: > Sometimes extra thread offloading imposed by dm-crypt hurts IO latency. This > is > especially visible on busy systems with many processes/threads. Moreover, most > Crypto API implementaions are async, that is they offload crypto operations on > their

Re: [PATCH v9 1/4] unicode: Add utf8_casefold_hash

2020-06-23 Thread Gabriel Krisman Bertazi
Daniel Rosenberg writes: > This adds a case insensitive hash function to allow taking the hash > without needing to allocate a casefolded copy of the string. > > Signed-off-by: Daniel Rosenberg > --- > fs/unicode/utf8-core.c | 23 ++- > include/linux/unicode.h | 3 +++ >

Re: [PATCH 5.7 000/477] 5.7.6-rc1 review

2020-06-23 Thread Guenter Roeck
On 6/23/20 12:49 PM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 5.7.6 release. > There are 477 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses

Re: [PATCH 5.4 000/314] 5.4.49-rc1 review

2020-06-23 Thread Guenter Roeck
On 6/23/20 12:53 PM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 5.4.49 release. > There are 314 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses

Re: [PATCH 5.7 318/477] pinctrl: freescale: imx: Use devm_of_iomap() to avoid a resource leak in case of error in imx_pinctrl_probe()

2020-06-23 Thread Marion & Christophe JAILLET
Hi, This one must NOT be included. It generates a regression. This should be removed from 5.4 as well. See 13f2d25b951f139064ec2dd53c0c7ebdf8d8007e. There is also a thread on ML about it. I couldn't find it right away, but I'm sure that Dan will be quicker than me for finding it, if needed

Re: [dm-crypt] [RFC PATCH 1/1] Add DM_CRYPT_FORCE_INLINE flag to dm-crypt target

2020-06-23 Thread Eric Biggers
On Fri, Jun 19, 2020 at 05:41:32PM +0100, Ignat Korchagin wrote: > Sometimes extra thread offloading imposed by dm-crypt hurts IO latency. This > is > especially visible on busy systems with many processes/threads. Moreover, most > Crypto API implementaions are async, that is they offload crypto

Re: Subject: [PATCH v3 1/2] USB: serial: cp210x: Enable usb generic throttle/unthrottle

2020-06-23 Thread gre...@linuxfoundation.org
On Wed, Jun 24, 2020 at 04:11:21AM +, Phu Luu wrote: > Assign the .throttle and .unthrottle functions to be generic function > in the driver structure to prevent data loss that can otherwise occur > if the host does not enable USB throttling. > > Signed-off-by: Phu Luu An > Signed-off-by:

Re: [dm-devel] [RFC PATCH 0/1] dm-crypt excessive overhead

2020-06-23 Thread Damien Le Moal
On 2020/06/24 0:23, Mike Snitzer wrote: > On Tue, Jun 23 2020 at 11:07am -0400, > Ignat Korchagin wrote: > >> Do you think it may be better to break it in two flags: one for read >> path and one for write? So, depending on the needs and workflow these >> could be enabled independently? > > If

Re: [PATCH v3 05/10] x86: Make sure _etext includes function sections

2020-06-23 Thread Kees Cook
On Tue, Jun 23, 2020 at 10:23:22AM -0700, Kristen Carlson Accardi wrote: > When using -ffunction-sections to place each function in > it's own text section so it can be randomized at load time, the > linker considers these .text.* sections "orphaned sections", and > will place them after the first

[PATCH] libnvdimm/security: Fix key lookup permissions

2020-06-23 Thread Dan Williams
As of commit 8c0637e950d6 ("keys: Make the KEY_NEED_* perms an enum rather than a mask") lookup_user_key() needs an explicit declaration of what it wants to do with the key. Add KEY_NEED_SEARCH to fix a warning with the below signature, and fixes the inability to retrieve a key. WARNING: CPU:

Re: [PATCH 5.4 000/314] 5.4.49-rc1 review

2020-06-23 Thread Guenter Roeck
On 6/23/20 12:53 PM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 5.4.49 release. > There are 314 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses

Re: [PATCH v3 3/9] efi/libstub: Remove .note.gnu.property

2020-06-23 Thread Kees Cook
On Tue, Jun 23, 2020 at 08:31:42PM -0700, 'Fangrui Song' via Clang Built Linux wrote: > On 2020-06-23, Kees Cook wrote: > > In preparation for adding --orphan-handling=warn to more architectures, > > make sure unwanted sections don't end up appearing under the .init > > section prefix that

Re: [PATCH] soc: qcom: rpmh-rsc: Don't use ktime for timeout in write_tcs_reg_sync()

2020-06-23 Thread Maulik Shah
Reviewed-by: Maulik Shah Thanks, Maulik On 5/28/2020 8:18 PM, Douglas Anderson wrote: The write_tcs_reg_sync() may be called after timekeeping is suspended so it's not OK to use ktime. The readl_poll_timeout_atomic() macro implicitly uses ktime. This was causing a warning at suspend time.

Re: [PATCH 05/15] mm: allow read-ahead with IOCB_NOWAIT set

2020-06-23 Thread Dave Chinner
On Thu, Jun 18, 2020 at 08:43:45AM -0600, Jens Axboe wrote: > The read-ahead shouldn't block, so allow it to be done even if > IOCB_NOWAIT is set in the kiocb. > > Acked-by: Johannes Weiner > Signed-off-by: Jens Axboe BTW, Jens, in case nobody had mentioned it, the Reply-To field for the

[PATCH v9 1/4] unicode: Add utf8_casefold_hash

2020-06-23 Thread Daniel Rosenberg
This adds a case insensitive hash function to allow taking the hash without needing to allocate a casefolded copy of the string. Signed-off-by: Daniel Rosenberg --- fs/unicode/utf8-core.c | 23 ++- include/linux/unicode.h | 3 +++ 2 files changed, 25 insertions(+), 1

[PATCH v9 3/4] f2fs: Use generic casefolding support

2020-06-23 Thread Daniel Rosenberg
This switches f2fs over to the generic support provided in commit 5f829feca774 ("fs: Add standard casefolding support") Signed-off-by: Daniel Rosenberg --- fs/f2fs/dir.c | 84 + fs/f2fs/f2fs.h | 4 -- fs/f2fs/super.c | 10 ++---

[PATCH v9 4/4] ext4: Use generic casefolding support

2020-06-23 Thread Daniel Rosenberg
This switches ext4 over to the generic support provided in commit 5f829feca774 ("fs: Add standard casefolding support") Signed-off-by: Daniel Rosenberg --- fs/ext4/dir.c | 64 ++--- fs/ext4/ext4.h | 12 -- fs/ext4/hash.c | 2 +-

[PATCH v9 0/4] Prepare for upcoming Casefolding/Encryption patches

2020-06-23 Thread Daniel Rosenberg
This lays the ground work for enabling casefolding and encryption at the same time for ext4 and f2fs. A future set of patches will enable that functionality. These unify the highly similar dentry_operations that ext4 and f2fs both use for casefolding. Daniel Rosenberg (4): unicode: Add

[PATCH v9 2/4] fs: Add standard casefolding support

2020-06-23 Thread Daniel Rosenberg
This adds general supporting functions for filesystems that use utf8 casefolding. It provides standard dentry_operations and adds the necessary structures in struct super_block to allow this standardization. Ext4 and F2fs will switch to these common implementations. Signed-off-by: Daniel

Re: [PATCH -next] scsi: ufs: allow exynos ufs driver to build as module

2020-06-23 Thread Martin K. Petersen
On Sat, 20 Jun 2020 23:02:32 +0530, Alim Akhtar wrote: > Allow Exynos UFS driver to build as a module. > This patch fix the below build issue reported by > kernel build robot. > > drivers/scsi/ufs/ufs-exynos.o: in function `exynos_ufs_probe': > drivers/scsi/ufs/ufs-exynos.c:1231: undefined

Re: Switching dmaengine tree to kernel.org

2020-06-23 Thread Vinod Koul
Hi Stephen, On 24-06-20, 08:09, Stephen Rothwell wrote: > Hi Vinod, > > On Tue, 23 Jun 2020 20:13:13 +0530 Vinod Koul wrote: > > > > I have switched dmaengine tree to kernel.org [1], please update your > > database to new tree which can be found at [2] > > > > [1]: > >

Re: memcg missing charge when setting BPF

2020-06-23 Thread Roman Gushchin
Hello Xie! It's actually not a surprise, it's a known limitation/exception. Partially it was so because historically there was no way to account percpu memory, and some bpf maps can are using it quite extensively. Fortunately, it changed recently, and 5.9 will likely get an ability to account

Re: [PATCH] scsi: lpfc: Avoid another null dereference in lpfc_sli4_hba_unset()

2020-06-23 Thread Martin K. Petersen
On Tue, 23 Jun 2020 10:41:22 +0200, SeongJae Park wrote: > Commit cdb42becdd40 ("scsi: lpfc: Replace io_channels for nvme and fcp > with general hdw_queues per cpu") has introduced static checker warnings > for potential null dereferences in 'lpfc_sli4_hba_unset()' and > commit 1ffdd2c0440d

Re: [PATCH] qla2xxx: Set NVME status code for failed NVME FCP request

2020-06-23 Thread Martin K. Petersen
On Thu, 4 Jun 2020 12:07:45 +0200, Daniel Wagner wrote: > The qla2xxx driver knows when request was processed successfully or > not. But it always sets the NVME status code to 0/NVME_SC_SUCCESS. The > upper layer needs to figure out from the rcv_rsplen and > transferred_length variables if the

Re: [RFC PATCH 0/1] dm-crypt excessive overhead

2020-06-23 Thread Damien Le Moal
On 2020/06/24 0:01, Mike Snitzer wrote: > On Sun, Jun 21 2020 at 8:45pm -0400, > Damien Le Moal wrote: > >> On 2020/06/20 1:56, Mike Snitzer wrote: >>> On Fri, Jun 19 2020 at 12:41pm -0400, >>> Ignat Korchagin wrote: >>> This is a follow up from the long-forgotten [1], but with some more

Re: [PATCH v6 0/2] Renovate memcpy_mcsafe with copy_mc_to_{user, kernel}

2020-06-23 Thread Luck, Tony
On Wed, Jun 17, 2020 at 05:31:58PM -0700, Dan Williams wrote: > No changes since v5 [1], just rebased to v5.8-rc1. No comments since > that posting back at the end of May either, will continue to re-post > weekly, I am otherwise at a loss for what else to do to move this > forward. Should it go

arch/s390/include/asm/atomic.h:51:35: sparse: sparse: context imbalance in 'btrfs_set_lock_blocking_read' - unexpected unlock

2020-06-23 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 3e08a95294a4fb3702bb3d35ed08028433c37fe6 commit: d6156218bec93965b6a43ba2686ad962ce77c854 btrfs: make locking assertion helpers static inline date: 7 months ago config: s390-randconfig-s031-20200623

Re: [PATCH 5.4 000/314] 5.4.49-rc1 review

2020-06-23 Thread Nathan Chancellor
On Tue, Jun 23, 2020 at 09:53:15PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 5.4.49 release. > There are 314 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me

Subject: [PATCH v3 1/2] USB: serial: cp210x: Enable usb generic throttle/unthrottle

2020-06-23 Thread Phu Luu
Assign the .throttle and .unthrottle functions to be generic function in the driver structure to prevent data loss that can otherwise occur if the host does not enable USB throttling. Signed-off-by: Phu Luu An Signed-off-by: Brant Merryman --- 06/09/2020: Patch v3 1/2 Modified based on feedback

Re: linux-next: manual merge of the rcu tree with the tip tree

2020-06-23 Thread Paul E. McKenney
On Wed, Jun 24, 2020 at 01:04:50PM +1000, Stephen Rothwell wrote: > Hi all, > > Today's linux-next merge of the rcu tree got a conflict in: > > kernel/sched/core.c > > between commit: > > 964ed98b0752 ("sched/core: Fix ttwu() race") > > from the tip tree and commit: > > 3c88d09bfb1b

Re: [PATCH 2/2] Documentation/litmus-tests: Add note on herd7 7.56 in atomic litmus test

2020-06-23 Thread Paul E. McKenney
On Wed, Jun 24, 2020 at 01:24:25AM +0200, Andrea Parri wrote: > On Wed, Jun 24, 2020 at 07:09:01AM +0900, Akira Yokosawa wrote: > > From f808c371075d2f92b955da1a83ecb3828db1972e Mon Sep 17 00:00:00 2001 > > From: Akira Yokosawa > > Date: Wed, 24 Jun 2020 06:59:26 +0900 > > Subject: [PATCH 2/2]

Re: reset-brcmstb-rescal.c:undefined reference to `devm_ioremap_resource'

2020-06-23 Thread Florian Fainelli
On 6/23/2020 6:26 PM, kernel test robot wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > master > head: 3e08a95294a4fb3702bb3d35ed08028433c37fe6 > commit: 4cf176e52397853e4a4dd37e917c5eafb47ba8d1 reset: Add Broadcom STB > RESCAL reset controller > date:

[PATCH] powerpc/boot: Use address-of operator on section symbols

2020-06-23 Thread Nathan Chancellor
Clang warns: arch/powerpc/boot/main.c:107:18: warning: array comparison always evaluates to a constant [-Wtautological-compare] if (_initrd_end > _initrd_start) { ^ arch/powerpc/boot/main.c:155:20: warning: array comparison always evaluates to a constant

Re: [PATCH] mm/spase: never partially remove memmap for early section

2020-06-23 Thread Wei Yang
On Wed, Jun 24, 2020 at 11:52:36AM +0800, Baoquan He wrote: >On 06/24/20 at 11:46am, Wei Yang wrote: >> On Wed, Jun 24, 2020 at 09:47:37AM +0800, Baoquan He wrote: >> >On 06/23/20 at 05:21pm, Dan Williams wrote: >> >> On Tue, Jun 23, 2020 at 2:43 AM Wei Yang >> >> wrote: >> >> > >> >> > For early

Re: [PATCH] mm/spase: never partially remove memmap for early section

2020-06-23 Thread Baoquan He
On 06/24/20 at 11:46am, Wei Yang wrote: > On Wed, Jun 24, 2020 at 09:47:37AM +0800, Baoquan He wrote: > >On 06/23/20 at 05:21pm, Dan Williams wrote: > >> On Tue, Jun 23, 2020 at 2:43 AM Wei Yang > >> wrote: > >> > > >> > For early sections, we assumes its memmap will never be partially > >> >

Re: Add MediaTek MT6873 devapc driver

2020-06-23 Thread Neal Liu
On Tue, 2020-06-09 at 11:32 -0600, Rob Herring wrote: > On Tue, Jun 09, 2020 at 06:24:19PM +0800, Neal Liu wrote: > > These patch series introduce a MediaTek MT6873 devapc driver. > > > > MT6873 bus frabric provides TrustZone security support and data > > protection to prevent slaves from being

RE: [PATCH 1/1] power_supply: wilco_ec: Add permanent long life charging mode

2020-06-23 Thread Crag.Wang
> > > Since this is normally only done in the factory context, can you > > > please confirm does something need to be artificially done to block > > > userland from trying to set the battery charging to this mode? Or > > > will the EC already > > handle > > > blocking it directly? > > > > This is

Re: [PATCH 1/2] selftests/lkdtm: Don't clear dmesg when running tests

2020-06-23 Thread Joe Lawrence
On 6/22/20 4:51 AM, Naresh Kamboju wrote: On Fri, 8 May 2020 at 12:23, Michael Ellerman wrote: It is Very Rude to clear dmesg in test scripts. That's because the script may be part of a larger test run, and clearing dmesg potentially destroys the output of other tests. We can avoid using

Re: [PATCH] mm/spase: never partially remove memmap for early section

2020-06-23 Thread Wei Yang
On Wed, Jun 24, 2020 at 09:47:37AM +0800, Baoquan He wrote: >On 06/23/20 at 05:21pm, Dan Williams wrote: >> On Tue, Jun 23, 2020 at 2:43 AM Wei Yang >> wrote: >> > >> > For early sections, we assumes its memmap will never be partially >> > removed. But current behavior breaks this. >> >> Where

RE: [EXT] Re: [PATCH v1] ARM: imx6plus: enable internal routing of clk_enet_ref where possible

2020-06-23 Thread Andy Duan
From: Sven Van Asbroeck Sent: Wednesday, June 24, 2020 10:56 AM > Hi Andy, > > On Tue, Jun 23, 2020 at 9:40 PM Andy Duan wrote: > > > > The patch looks good. > > Reviewed-by: Fugang Duan > > Thank you ! > > To check we're on a plus, the patch uses: > cpu_is_imx6q() && imx_get_soc_revision()

ERROR: "min_low_pfn" undefined!

2020-06-23 Thread kernel test robot
months ago config: microblaze-randconfig-c021-20200623 (attached as .config) compiler: microblaze-linux-gcc (GCC) 9.3.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All errors (new ones prefixed by >>): ERROR: "min_low_pfn" [net/m

Re: [PATCH RESEND] net/cisco: Fix a sleep-in-atomic-context bug in enic_init_affinity_hint()

2020-06-23 Thread Kaige Li
On 06/24/2020 11:23 AM, David Miller wrote: From: Kaige Li Date: Wed, 24 Jun 2020 09:56:47 +0800 On 06/24/2020 06:26 AM, David Miller wrote: From: David Miller Date: Tue, 23 Jun 2020 14:33:11 -0700 (PDT) Calling a NIC driver open function from a context holding a spinlock is very much

[for-linus][PATCH 4/4] tracing/boottime: Fix kprobe multiple events

2020-06-23 Thread Steven Rostedt
From: Sascha Ortmann Fix boottime kprobe events to report and abort after each failure when adding probes. As an example, when we try to set multiprobe kprobe events in bootconfig like this: ftrace.event.kprobes.vfsevents { probes = "vfs_read $arg1 $arg2,, !error! not

[for-linus][PATCH 0/4] tracing: Fixes for 5.8-rc

2020-06-23 Thread Steven Rostedt
Masami Hiramatsu (2): tracing/boot: Fix config dependency for synthedic event tracing: Fix event trigger to accept redundant spaces Sascha Ortmann (1): tracing/boottime: Fix kprobe multiple events Steven Rostedt (VMware) (1): ring-buffer: Zero out time extend if it is

[for-linus][PATCH 1/4] ring-buffer: Zero out time extend if it is nested and not absolute

2020-06-23 Thread Steven Rostedt
From: "Steven Rostedt (VMware)" Currently the ring buffer makes events that happen in interrupts that preempt another event have a delta of zero. (Hopefully we can change this soon). But this is to deal with the races of updating a global counter with lockless and nesting functions updating

[for-linus][PATCH 2/4] tracing/boot: Fix config dependency for synthedic event

2020-06-23 Thread Steven Rostedt
From: Masami Hiramatsu Since commit 726721a51838 ("tracing: Move synthetic events to a separate file") decoupled synthetic event from histogram, boot-time tracing also has to check CONFIG_SYNTH_EVENT instead of CONFIG_HIST_TRIGGERS. Link:

[for-linus][PATCH 3/4] tracing: Fix event trigger to accept redundant spaces

2020-06-23 Thread Steven Rostedt
From: Masami Hiramatsu Fix the event trigger to accept redundant spaces in the trigger input. For example, these return -EINVAL echo " traceon" > events/ftrace/print/trigger echo "traceon if common_pid == 0" > events/ftrace/print/trigger echo "disable_event:kmem:kmalloc " >

Re: [PATCH v3 2/2] net: phy: call phy_disable_interrupts() in phy_init_hw()

2020-06-23 Thread Florian Fainelli
Le 2020-06-23 à 20:26, Jisheng Zhang a écrit : > Call phy_disable_interrupts() in phy_init_hw() to "have a defined init > state as we don't know in which state the PHY is if the PHY driver is > loaded. We shouldn't assume that it's the chip power-on defaults, BIOS > or boot loader could have

Re: [PATCH v1 0/5] ethernet: dec: tulip: use generic power management

2020-06-23 Thread David Miller
From: Vaibhav Gupta Date: Mon, 22 Jun 2020 17:12:23 +0530 > Linux Kernel Mentee: Remove Legacy Power Management. > > The purpose of this patch series is to remove legacy power management > callbacks and invocation of PCI helper functions, from tulip ethernet drivers. > > With legacy PM,

Re: [PATCH v2 0/3] ethernet: amd: Convert to generic power management

2020-06-23 Thread David Miller
From: Vaibhav Gupta Date: Mon, 22 Jun 2020 16:43:57 +0530 > Linux Kernel Mentee: Remove Legacy Power Management. > > The purpose of this patch series is to remove legacy power management > callbacks > from amd ethernet drivers. > > The callbacks performing suspend() and resume() operations

Re: [PATCH -V2] swap: Reduce lock contention on swap cache from swap slots allocation

2020-06-23 Thread Huang, Ying
"Huang, Ying" writes: > Andrew Morton writes: > >> On Wed, 20 May 2020 11:15:02 +0800 Huang Ying wrote: >> >>> In some swap scalability test, it is found that there are heavy lock >>> contention on swap cache even if we have split one swap cache radix >>> tree per swap device to one swap cache

[PATCH] [drivers/x86] fix bound check in pmc_core_mphy_pg_show

2020-06-23 Thread Gaurav Singh
Check bounds before accessing map[]. Signed-off-by: Gaurav Singh --- drivers/platform/x86/intel_pmc_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/platform/x86/intel_pmc_core.c b/drivers/platform/x86/intel_pmc_core.c index 7c8bdab078cf..f571f9cf7217 100644

Re: [PATCH v3 3/9] efi/libstub: Remove .note.gnu.property

2020-06-23 Thread Fangrui Song
On 2020-06-23, Kees Cook wrote: In preparation for adding --orphan-handling=warn to more architectures, make sure unwanted sections don't end up appearing under the .init section prefix that libstub adds to itself during objcopy. Signed-off-by: Kees Cook ---

Re: [PATCH] [net] dcb_doit: remove redundant skb check

2020-06-23 Thread David Miller
From: Gaurav Singh Date: Mon, 22 Jun 2020 22:50:39 -0400 > skb cannot be NULL here since its already being accessed > before: sock_net(skb->sk). Remove the redundant null check. > > Signed-off-by: Gaurav Singh Applied.

Re: [PATCH] [net/decnet] dn_route_rcv: remove redundant dev null check

2020-06-23 Thread David Miller
From: Gaurav Singh Date: Mon, 22 Jun 2020 23:41:19 -0400 > dev cannot be NULL here since its already being accessed > before. Remove the redundant null check. > > Signed-off-by: Gaurav Singh Applied.

Re: [PATCH][next] net: ipv6: Use struct_size() helper and kcalloc()

2020-06-23 Thread David Miller
From: "Gustavo A. R. Silva" Date: Mon, 22 Jun 2020 18:07:41 -0500 > Make use of the struct_size() helper instead of an open-coded version > in order to avoid any potential type mistakes. Also, remove unnecessary > function ipv6_rpl_srh_alloc_size() and replace kzalloc() with kcalloc(), > which

[PATCH] mm/page_alloc: fix documentation error and remove magic numbers

2020-06-23 Thread Joel Savitz
When I increased the upper bound of the min_free_kbytes value in ee8eb9a5fe863, I forgot to tweak the above comment to reflect the new value. This patch fixes that mistake. In addition, this patch replaces the magic number bounds with symbolic constants to clarify the logic. Suggested-by: John

[PATCH v3 1/2] net: phy: make phy_disable_interrupts() non-static

2020-06-23 Thread Jisheng Zhang
We face an issue with rtl8211f, a pin is shared between INTB and PMEB, and the PHY Register Accessible Interrupt is enabled by default, so the INTB/PMEB pin is always active in polling mode case. As Heiner pointed out "I was thinking about calling phy_disable_interrupts() in phy_init_hw(), to

[PATCH v3 0/2] net: phy: call phy_disable_interrupts() in phy_init_hw()

2020-06-23 Thread Jisheng Zhang
We face an issue with rtl8211f, a pin is shared between INTB and PMEB, and the PHY Register Accessible Interrupt is enabled by default, so the INTB/PMEB pin is always active in polling mode case. As Heiner pointed out "I was thinking about calling phy_disable_interrupts() in phy_init_hw(), to

[PATCH v3 2/2] net: phy: call phy_disable_interrupts() in phy_init_hw()

2020-06-23 Thread Jisheng Zhang
Call phy_disable_interrupts() in phy_init_hw() to "have a defined init state as we don't know in which state the PHY is if the PHY driver is loaded. We shouldn't assume that it's the chip power-on defaults, BIOS or boot loader could have changed this. Or in case of dual-boot systems the other OS

Re: [PATCH] net: qrtr: free flow in __qrtr_node_release

2020-06-23 Thread David Miller
From: Carl Huang Date: Tue, 23 Jun 2020 11:22:03 +0800 > @@ -168,6 +168,7 @@ static void __qrtr_node_release(struct kref *kref) > struct radix_tree_iter iter; > unsigned long flags; > void __rcu **slot; > + struct qrtr_tx_flow *flow; Please retain the reverse christmas

Re: [PATCH] fs/read_write.c: Fix memory leak in read_write.c

2020-06-23 Thread Matthew Wilcox
On Wed, Jun 24, 2020 at 11:07:03AM +0800, Peng Fan wrote: > kmemleak report: > unreferenced object 0x9802bb591d00 (size 256): > comm "ftest03", pid 24778, jiffies 4301603810 (age 490.665s) > hex dump (first 32 bytes): > 00 01 04 20 01 00 00 00 80 00 00 00 00 00 00 00 ...

Re: [PATCH RESEND] net/cisco: Fix a sleep-in-atomic-context bug in enic_init_affinity_hint()

2020-06-23 Thread David Miller
From: Kaige Li Date: Wed, 24 Jun 2020 10:07:16 +0800 > You are right. Should I do spin_unlock before the enic_open, or remove > spin_lock in enic_reset? You need to learn how this driver's locking works and design a correct adjustment.

  1   2   3   4   5   6   7   8   9   10   >