[PATCH V2 1/2] scsi: ufs: Fix interrupt error message for shared interrupts

2020-08-11 Thread Adrian Hunter
The interrupt might be shared, in which case it is not an error for the interrupt handler to be called when the interrupt status is zero, so don't print the message unless there was enabled interrupt status. Signed-off-by: Adrian Hunter Fixes: 9333d77573485 ("scsi: ufs: Fix irq return code") ---

[PATCH V2 2/2] scsi: ufs: Improve interrupt handling for shared interrupts

2020-08-11 Thread Adrian Hunter
For shared interrupts, the interrupt status might be zero, so check that first. Signed-off-by: Adrian Hunter --- drivers/scsi/ufs/ufshcd.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c index bb2543010af9..c315420e6

[PATCHv1 0/2] add Intel SoCFPGA crypto service driver

2020-08-11 Thread richard . gong
From: Richard Gong I followed the process to register or request a valid IOCTL number/letter, but I got the delivery failure status notification. Cypto service driver and service layer driver patches have been reviewed internally by colleagues at Intel. Intel SoCFPGA is composed of a 64 bit qua

Re: [RFC PATCH] printk: Change timestamp to triplet as mono, boot and real

2020-08-11 Thread Prarit Bhargava
On 8/11/20 9:02 AM, Petr Mladek wrote: > On Tue 2020-08-11 14:05:12, Thomas Gleixner wrote: >> Petr Mladek writes: >>> At least "crash" tool would need an update anyway. AFAIK, it checks >>> the size of struct printk_log and refuses to read it when it changes. >>> >>> It means that the hack wit

[PATCH] block: don't read block device if it's invalid

2020-08-11 Thread Xianting Tian
We found several processes in 'D' state after nvme device hot-removed, The call trace as below, we can see process 848 got lock 'bdev->bd_mutex' in blkdev_reread_part(), but scheduled out due to wait for IO done. But the IO won't be completed as the device is hot-removed. Then it caused the lock 'b

Re: [RFC v4 1/3] usb: dwc3: Resize TX FIFOs to meet EP bursting requirements

2020-08-11 Thread Alan Stern
On Tue, Aug 11, 2020 at 10:12:41AM +0300, Felipe Balbi wrote: > One possibility around this was suggested above, something along the > lines of: > > usb_gadget_ep_enable_bulk(struct usb_gadget *, struct > usb_ep_alloc_desc *alloc_desc) > > (please think of better names, I'm ho

Re: [PATCH AUTOSEL 4.4 03/16] fs/btrfs: Add cond_resched() for try_release_extent_mapping() stalls

2020-08-11 Thread Paul E. McKenney
On Tue, Aug 11, 2020 at 09:57:20AM +0200, David Sterba wrote: > On Mon, Aug 10, 2020 at 03:14:30PM -0400, Sasha Levin wrote: > > From: "Paul E. McKenney" > > > > [ Upstream commit 9f47eb5461aaeb6cb8696f9d11503ae90e4d5cb0 ] > > > > Very large I/Os can cause the following RCU CPU stall warning: >

Re: [PATCH] scsi: sd: add runtime pm to open / release

2020-08-11 Thread Alan Stern
On Tue, Aug 11, 2020 at 09:55:54AM +0200, Martin Kepplinger wrote: > On 10.08.20 16:13, Alan Stern wrote: > > This may not matter... but it's worth pointing out that > > expecting_media_change doesn't get cleared if ++scmd->retries > > > scmd->allowed. > > absolutely worth pointing out and I'm

Re: [RFC PATCH v1] power: don't manage floating point regs when no FPU

2020-08-11 Thread Christophe Leroy
Le 11/08/2020 à 14:07, Michael Ellerman a écrit : Christophe Leroy writes: There is no point in copying floating point regs when there is no FPU and MATH_EMULATION is not selected. Yeah I guess you're right. I've never touched a system with neither, but if such a thing exists then it does

RE: [PATCH V2 1/2] scsi: ufs: Fix interrupt error message for shared interrupts

2020-08-11 Thread Avri Altman
> > The interrupt might be shared, in which case it is not an error for the > interrupt handler to be called when the interrupt status is zero, so > don't print the message unless there was enabled interrupt status. > > Signed-off-by: Adrian Hunter > Fixes: 9333d77573485 ("scsi: ufs: Fix irq r

Re: [RFC 0/5] Introduce NMI aware serial drivers

2020-08-11 Thread Sumit Garg
On Tue, 21 Jul 2020 at 17:40, Sumit Garg wrote: > > Make it possible for UARTs to trigger magic sysrq from an NMI. With the > advent of pseudo NMIs on arm64 it became quite generic to request serial > device interrupt as an NMI rather than IRQ. And having NMI driven serial > RX will allow us to tr

[PATCHv1 1/2] firmware: stratix10-svc: extend svc to support new crypto features

2020-08-11 Thread richard . gong
From: Richard Gong Extend Intel service layer driver to support new crypto services on Intel SoCFPGA platforms. The crypto services include security certificate, image boot validation, security key cancellation, get provision data, random number generation, advance encrtption standard (AES) encr

[PATCH] leds: we don't want people to use LED subsystem for vibrations

2020-08-11 Thread Pavel Machek
Hi! This will eventually go in, unless I hear some good reasons not to. Best regards, Pavel === leds: we don't want people to use LED subsystem for vibrations Remove notes about vibrations, as that is better done using

Re: KASAN: use-after-free Read in io_async_task_func

2020-08-11 Thread Jens Axboe
On 8/11/20 12:47 AM, syzbot wrote: > Hello, > > syzbot found the following issue on: > > HEAD commit:fc80c51f Merge tag 'kbuild-v5.9' of git://git.kernel.org/p.. > git tree: upstream > console output: https://syzkaller.appspot.com/x/log.txt?x=17601ab290 > kernel config: https://syz

Re: file metadata via fs API (was: [GIT PULL] Filesystem Information)

2020-08-11 Thread Miklos Szeredi
On Wed, Aug 05, 2020 at 10:24:23AM +0200, Miklos Szeredi wrote: > On Tue, Aug 4, 2020 at 4:36 PM Miklos Szeredi wrote: > > > I think we already lost that with the xattr API, that should have been > > done in a way that fits this philosophy. But given that we have "/" > > as the only special pur

Re: [PATCH v7 2/4] SFH: PCIe driver to add support of AMD sensor fusion hub

2020-08-11 Thread Marco Felsch
Hi Sandeep, On 20-08-10 21:30, Sandeep Singh wrote: > From: Sandeep Singh > > AMD SFH uses HID over PCIe bus.SFH fw is part of MP2 processor > (MP2 which is an ARM® Cortex-M4 core based co-processor to x86) and > it runs on MP2 where in driver resides on X86. This part of module > will communica

Re: [Resend PATCH 2/6] mm/memcg: remove useless check on page->mem_cgroup

2020-08-11 Thread Michal Hocko
On Tue 11-08-20 20:54:18, Alex Shi wrote: > >From beeac61119ab39b1869c520c0f272fb8bab93765 Mon Sep 17 00:00:00 2001 > From: Alex Shi > Date: Wed, 5 Aug 2020 21:02:30 +0800 > Subject: [PATCH 2/6] memcg: bail out early from swap accounting when memcg is > disabled > > If we disabled memcg by cgrou

memory leak in io_submit_sqes

2020-08-11 Thread syzbot
Hello, syzbot found the following issue on: HEAD commit:d6efb3ac Merge tag 'tty-5.9-rc1' of git://git.kernel.org/p.. git tree: upstream console output: https://syzkaller.appspot.com/x/log.txt?x=13cb076290 kernel config: https://syzkaller.appspot.com/x/.config?x=42163327839348a9 das

[PATCHv1 2/2] crypto: add Intel SoCFPGA crypto service driver

2020-08-11 Thread richard . gong
From: Richard Gong Add Intel FPGA crypto service (FCS) driver to support new crypto services on Intel SoCFPGA platforms. The crypto services include security certificate, image boot validation, security key cancellation, get provision data, random number generation, advance encrtption standard (

Re: [RFC 0/5] Introduce NMI aware serial drivers

2020-08-11 Thread Greg Kroah-Hartman
On Tue, Aug 11, 2020 at 07:20:26PM +0530, Sumit Garg wrote: > On Tue, 21 Jul 2020 at 17:40, Sumit Garg wrote: > > > > Make it possible for UARTs to trigger magic sysrq from an NMI. With the > > advent of pseudo NMIs on arm64 it became quite generic to request serial > > device interrupt as an NMI

Re: [PATCH v2 00/17] arm/arm64: Turning IPIs into normal interrupts

2020-08-11 Thread Marc Zyngier
Hi Sumit, On 2020-08-11 14:15, Sumit Garg wrote: Hi Marc, On Thu, 25 Jun 2020 at 01:28, Marc Zyngier wrote: For as long as SMP ARM has existed, IPIs have been handled as something special. The arch code and the interrupt controller exchange a couple of hooks (one to generate an IPI, another

Re: [PATCH v7 0/7] Add support for O_MAYEXEC

2020-08-11 Thread Mimi Zohar
On Tue, 2020-08-11 at 10:48 +0200, Mickaël Salaün wrote: > On 11/08/2020 01:03, Jann Horn wrote: > > On Tue, Aug 11, 2020 at 12:43 AM Mickaël Salaün wrote: > > > On 10/08/2020 22:21, Al Viro wrote: > > > > On Mon, Aug 10, 2020 at 10:11:53PM +0200, Mickaël Salaün wrote: > > > > > It seems that ther

Re: possible deadlock in __io_queue_deferred

2020-08-11 Thread Stefano Garzarella
On Mon, Aug 10, 2020 at 09:55:17AM -0600, Jens Axboe wrote: > On 8/10/20 9:36 AM, syzbot wrote: > > Hello, > > > > syzbot found the following issue on: > > > > HEAD commit:449dc8c9 Merge tag 'for-v5.9' of git://git.kernel.org/pub/.. > > git tree: upstream > > console output: https://syz

Zdravstvujte Vas interesujut bazy dannyh kontaktov kompanij dlja prodazhi vashih tovarov i uslug?

2020-08-11 Thread prodawez
Zdravstvujte Vas interesujut bazy dannyh kontaktov kompanij dlja prodazhi vashih tovarov i uslug?

Re: [PATCH v7 0/7] Add support for O_MAYEXEC

2020-08-11 Thread Matthew Wilcox
On Tue, Aug 11, 2020 at 09:56:50AM -0400, Mimi Zohar wrote: > On Tue, 2020-08-11 at 10:48 +0200, Mickaël Salaün wrote: > > On 11/08/2020 01:03, Jann Horn wrote: > > > On Tue, Aug 11, 2020 at 12:43 AM Mickaël Salaün wrote: > > > > On 10/08/2020 22:21, Al Viro wrote: > > > > > On Mon, Aug 10, 2020 a

Re: power-off delay/hang due to commit 6d25be57 (mainline)

2020-08-11 Thread Rafael J. Wysocki
On Tue, Aug 11, 2020 at 12:27 PM Sebastian Andrzej Siewior wrote: > > On 2020-07-14 17:53:15 [+0200], Rafael J. Wysocki wrote: > > acpi_evaluate_integer() doesn't show up in the trace, though, AFAICS. > > > > > I assumed acpi_ex_opcode_2A_0T_0R() since the other > > > candidate was acpi_ev_asynch_

[PATCH] rndis_wlan: tighten check of rndis_query_oid return

2020-08-11 Thread trix
From: Tom Rix clang static analysis reports this problem rndis_wlan.c:3147:25: warning: Assigned value is garbage or undefined wiphy->max_num_pmkids = le32_to_cpu(caps.num_pmkids); ^ The setting of caps happens h

Re: btrfs crash in kobject_del while running xfstest

2020-08-11 Thread David Sterba
On Tue, Aug 11, 2020 at 04:19:47AM -0700, John Hubbard wrote: > Somehow the copy-paste of Chris Mason's name failed (user error > on my end), sorry about that Chris! > > On 8/11/20 4:17 AM, John Hubbard wrote: > > Hi, > > > > Here's an early warning of a possible problem. > > > > I'm seeing a ne

[PATCH v2] arm64: defconfig: Enable R-Car PCIe endpoint driver

2020-08-11 Thread Lad Prabhakar
Enable R-Car PCIe endpoint driver on RZ/G2E board, including enabling endpoint configurations CONFIG_PCI_ENDPOINT, CONFIG_PCI_ENDPOINT_CONFIGFS, CONFIG_PCI_EPF_TEST and CONFIG_PCI_ENDPOINT_TEST required to use and test the driver. Signed-off-by: Lad Prabhakar --- v1->v2 * Made CONFIG_PCI_EPF_TEST

Re: [RFC PATCH v1] power: don't manage floating point regs when no FPU

2020-08-11 Thread Christophe Leroy
Le 11/08/2020 à 14:07, Michael Ellerman a écrit : Christophe Leroy writes: 10 files changed, 44 insertions(+), 1 deletion(-) In general this looks fine. It's a bit #ifdef heavy. Maybe some of those can be cleaned up a bit with some wrapper inlines? Looking at it once more, looks like

Re: [PATCH] nfs: Fix getxattr kernel panic and memory overflow

2020-08-11 Thread Trond Myklebust
On Wed, 2020-08-05 at 12:23 -0500, Jeffrey Mitchell wrote: > Move the buffer size check to decode_attr_security_label() before > memcpy() > Only call memcpy() if the buffer is large enough > > Signed-off-by: Jeffrey Mitchell > --- > fs/nfs/nfs4proc.c | 2 -- > fs/nfs/nfs4xdr.c | 5 - > 2 fi

Re: file metadata via fs API (was: [GIT PULL] Filesystem Information)

2020-08-11 Thread Al Viro
On Tue, Aug 11, 2020 at 03:54:19PM +0200, Miklos Szeredi wrote: > On Wed, Aug 05, 2020 at 10:24:23AM +0200, Miklos Szeredi wrote: > > On Tue, Aug 4, 2020 at 4:36 PM Miklos Szeredi wrote: > > > > > I think we already lost that with the xattr API, that should have been > > > done in a way that fits

Re: [RFC][PATCH v3 11/11] misc: hisi_hikey_usb: Driver to support usb functionality of Hikey960

2020-08-11 Thread Mauro Carvalho Chehab
Em Tue, 11 Aug 2020 14:21:06 +0200 Mauro Carvalho Chehab escreveu: > I was able to drop it, but I had to add this at dwc3 settings: > > regulator-on-in-suspend; > > As otherwise the device seems to stop a few seconds after the dwc3 > driver gets started. > > I suspect it could be related

[PATCH v2 1/2] scsi: ufs: Cleanup completed request without interrupt notification

2020-08-11 Thread Bean Huo
From: Stanley Chu If somehow no interrupt notification is raised for a completed request and its doorbell bit is cleared by host, UFS driver needs to cleanup its outstanding bit in ufshcd_abort(). Otherwise, system may behave abnormally by below flow: After ufshcd_abort() returns, this request w

[PATCH v2 2/2] scsi: ufs: no need to send one Abort Task TM in case the task in DB was cleared

2020-08-11 Thread Bean Huo
From: Bean Huo If the bit corresponds to a task in the Doorbell register has been cleared, no need to poll the status of the task on the device side and to send an Abort Task TM. Instead, let it directly goto cleanup. Meanwhile, to keep original debug print, move this goto below the debug print.

[PATCH v2 0/2] scsi: ufs: two fixups of ufshcd_abort()

2020-08-11 Thread Bean Huo
From: Bean Huo Changelog: v1 - v2: 1. add patch [1/2], which is from Stanley Chu 2. change goto command in patch [2/2], let it goto cleanup flow Bean Huo (1): scsi: ufs: no need to send one Abort Task TM in case the task in DB was cleared Stanley Chu (1): scsi: ufs: Cleanup co

Re: possible deadlock in __io_queue_deferred

2020-08-11 Thread Jens Axboe
On 8/11/20 8:00 AM, Stefano Garzarella wrote: > On Mon, Aug 10, 2020 at 09:55:17AM -0600, Jens Axboe wrote: >> On 8/10/20 9:36 AM, syzbot wrote: >>> Hello, >>> >>> syzbot found the following issue on: >>> >>> HEAD commit:449dc8c9 Merge tag 'for-v5.9' of git://git.kernel.org/pub/.. >>> git tree:

Re: [PATCH 5.7 00/79] 5.7.15-rc1 review

2020-08-11 Thread Guenter Roeck
On Mon, Aug 10, 2020 at 05:20:19PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 5.7.15 release. > There are 79 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.

Re: [PATCH 5.4 00/67] 5.4.58-rc1 review

2020-08-11 Thread Guenter Roeck
On Mon, Aug 10, 2020 at 05:20:47PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 5.4.58 release. > There are 67 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.

Re: file metadata via fs API (was: [GIT PULL] Filesystem Information)

2020-08-11 Thread Miklos Szeredi
On Tue, Aug 11, 2020 at 4:08 PM Al Viro wrote: > > On Tue, Aug 11, 2020 at 03:54:19PM +0200, Miklos Szeredi wrote: > > On Wed, Aug 05, 2020 at 10:24:23AM +0200, Miklos Szeredi wrote: > > > On Tue, Aug 4, 2020 at 4:36 PM Miklos Szeredi wrote: > > > > > > > I think we already lost that with the xat

Re: [PATCH] power: supply: Add dependency to lego-ev3-battery Kconfig options

2020-08-11 Thread David Lechner
On 8/9/20 1:54 PM, Alex Dewar wrote: This battery appears only to be used by a single board (DA850), so it makes sense to add this to the Kconfig file so that users don't build the module unnecessarily. It currently seems to be built for the x86 Arch Linux kernel where it's probably not doing muc

drivers/scsi/mpt3sas/mpt3sas_ctl.c:1900 _ctl_diag_register() warn: potential spectre issue 'ioc->diag_buffer_status'

2020-08-11 Thread kernel test robot
-20200811 (attached as .config) compiler: hppa-linux-gcc (GCC) 9.3.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot New smatch warnings: drivers/scsi/mpt3sas/mpt3sas_ctl.c:1900 _ctl_diag_register() warn: potential spectre issue 'ioc->diag_buffer_st

Re: [PATCH 4.19 00/48] 4.19.139-rc1 review

2020-08-11 Thread Guenter Roeck
On Mon, Aug 10, 2020 at 05:21:22PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.19.139 release. > There are 48 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 kno

Re: [PATCH 5.8 00/38] 5.8.1-rc1 review

2020-08-11 Thread Guenter Roeck
On Mon, Aug 10, 2020 at 05:18:50PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 5.8.1 release. > There are 38 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.

[PATCH] task_work: only grab task signal lock when needed

2020-08-11 Thread Jens Axboe
If JOBCTL_TASK_WORK is already set on the targeted task, then we need not go through {lock,unlock}_task_sighand() to set it again and queue a signal wakeup. This is safe as we're checking it _after adding the new task_work with cmpxchg(). Signed-off-by: Jens Axboe --- Tested this with an intens

Re: file metadata via fs API (was: [GIT PULL] Filesystem Information)

2020-08-11 Thread Al Viro
On Tue, Aug 11, 2020 at 04:22:19PM +0200, Miklos Szeredi wrote: > On Tue, Aug 11, 2020 at 4:08 PM Al Viro wrote: > > > > On Tue, Aug 11, 2020 at 03:54:19PM +0200, Miklos Szeredi wrote: > > > On Wed, Aug 05, 2020 at 10:24:23AM +0200, Miklos Szeredi wrote: > > > > On Tue, Aug 4, 2020 at 4:36 PM Mikl

Re: [PATCH RESEND 1/9] crypto: caam/jr - add fallback for XTS with more than 8B IV

2020-08-11 Thread Horia Geantă
On 8/6/2020 7:36 PM, Andrei Botila (OSS) wrote: > @@ -3344,12 +3382,30 @@ static int caam_cra_init(struct crypto_skcipher *tfm) > struct caam_skcipher_alg *caam_alg = > container_of(alg, typeof(*caam_alg), skcipher); > struct caam_ctx *ctx = crypto_skcipher_ctx(tfm); > +

Re: [RFC 0/5] Introduce NMI aware serial drivers

2020-08-11 Thread Sumit Garg
Hi Greg, Thanks for your comments. On Tue, 11 Aug 2020 at 19:27, Greg Kroah-Hartman wrote: > > On Tue, Aug 11, 2020 at 07:20:26PM +0530, Sumit Garg wrote: > > On Tue, 21 Jul 2020 at 17:40, Sumit Garg wrote: > > > > > > Make it possible for UARTs to trigger magic sysrq from an NMI. With the > >

Re: [PATCH v7 0/7] Add support for O_MAYEXEC

2020-08-11 Thread Mimi Zohar
On Tue, 2020-08-11 at 15:02 +0100, Matthew Wilcox wrote: > On Tue, Aug 11, 2020 at 09:56:50AM -0400, Mimi Zohar wrote: > > On Tue, 2020-08-11 at 10:48 +0200, Mickaël Salaün wrote: > > > On 11/08/2020 01:03, Jann Horn wrote: > > > > On Tue, Aug 11, 2020 at 12:43 AM Mickaël Salaün > > > > wrote: >

Re: [PATCH] ext4: flag as supporting buffered async reads

2020-08-11 Thread Jens Axboe
On 8/3/20 5:02 PM, Jens Axboe wrote: > ext4 uses generic_file_read_iter(), which already supports this. Ping... -- Jens Axboe

Re: power-off delay/hang due to commit 6d25be57 (mainline)

2020-08-11 Thread Rafael J. Wysocki
On Tue, Aug 11, 2020 at 3:29 PM Sebastian Andrzej Siewior wrote: > > On 2020-08-11 13:58:39 [+0200], Stephen Berman wrote: > > him about your workaround of adding 'thermal.tzp=300' to the kernel > > commandline, and he replied that this works for him too. And it turns > > out we have similar moth

Re: [PATCH v2] module: Harden STRICT_MODULE_RWX

2020-08-11 Thread Mauro Carvalho Chehab
Hi Jessica, Em Mon, 10 Aug 2020 17:06:50 +0200 Jessica Yu escreveu: > +++ Jessica Yu [10/08/20 11:25 +0200]: > >+++ Mauro Carvalho Chehab [08/08/20 10:12 +0200]: > >[snip] > >>Right now, what happens is: > >> > >># modprobe wlcore > >>modprobe: ERROR: could not insert 'wlcore': Exec fo

Re: [PATCH 1/2] perf: Add closing sibling events' file descriptors

2020-08-11 Thread Andi Kleen
On Tue, Aug 11, 2020 at 12:47:24PM +0300, Alexander Shishkin wrote: > Andi Kleen writes: > > >> It didn't. I can't figure out what to charge on the locked memory, as > >> all that memory is in kernel-side objects. It also needs to make sense > > > > I don't see how that makes a difference for the

How to Setup TWO DMZ Zones (DMZ1 and DMZ2) on the Cisco ASA 5506-X Firewall

2020-08-11 Thread Turritopsis Dohrnii Teo En Ming
Subject: How to Setup TWO DMZ Zones (DMZ1 and DMZ2) on the Cisco ASA 5506-X Firewall Author: Mr. Turritopsis Dohrnii Teo En Ming (Targeted Individual) Country: Singapore, Singapore, Singapore Date: 11 August 2020 Tuesday Singapore Time Type of Publication: Plain Text Cisco ASA firewall applianc

Re: file metadata via fs API (was: [GIT PULL] Filesystem Information)

2020-08-11 Thread Al Viro
On Tue, Aug 11, 2020 at 10:33:59AM -0400, Tang Jiye wrote: > anyone knows how to post a question? Generally the way you just have, except that you generally put it *after* the relevant parts of the quoted text (and removes the irrelevant ones).

Re: file metadata via fs API (was: [GIT PULL] Filesystem Information)

2020-08-11 Thread Miklos Szeredi
On Tue, Aug 11, 2020 at 4:31 PM Al Viro wrote: > > On Tue, Aug 11, 2020 at 04:22:19PM +0200, Miklos Szeredi wrote: > > On Tue, Aug 11, 2020 at 4:08 PM Al Viro wrote: > > > > > > On Tue, Aug 11, 2020 at 03:54:19PM +0200, Miklos Szeredi wrote: > > > > On Wed, Aug 05, 2020 at 10:24:23AM +0200, Miklo

Re: [PATCH RESEND 4/9] crypto: caam/jr - add support for more XTS key lengths

2020-08-11 Thread Horia Geantă
On 8/6/2020 7:36 PM, Andrei Botila (OSS) wrote: > @@ -1790,7 +1792,9 @@ static inline int skcipher_crypt(struct > skcipher_request *req, bool encrypt) > if (!req->cryptlen) > return 0; > > - if (ctx->fallback && xts_skcipher_ivsize(req)) { > + if (ctx->fallback && (xt

Re: [PATCH RESEND] KVM: MIPS/VZ: Fix build error caused by 'kvm_run' cleanup

2020-08-11 Thread Jiaxun Yang
在 2020/8/11 下午5:31, Xingxing Su 写道: Commit c34b26b98caca48ec9ee9 ("KVM: MIPS: clean up redundant 'kvm_run' parameters") remove the 'kvm_run' parameter in kvm_vz_gpsi_lwc2. The following build error: arch/mips/kvm/vz.c: In function ‘kvm_trap_vz_handle_gpsi’: arch/mips/kvm/vz.c:1243:43: error:

Re: [PATCH RESEND] KVM: MIPS/VZ: Fix build error caused by 'kvm_run' cleanup

2020-08-11 Thread Jiaxun Yang
在 2020/8/11 下午10:37, Jiaxun Yang 写道: 在 2020/8/11 下午5:31, Xingxing Su 写道: Commit c34b26b98caca48ec9ee9 ("KVM: MIPS: clean up redundant 'kvm_run' parameters") remove the 'kvm_run' parameter in kvm_vz_gpsi_lwc2. The following build error: arch/mips/kvm/vz.c: In function ‘kvm_trap_vz_handle_g

Re: [PATCH 1/2] perf: Add closing sibling events' file descriptors

2020-08-11 Thread Andi Kleen
On Mon, Aug 10, 2020 at 10:36:32PM +0200, Peter Zijlstra wrote: > On Mon, Aug 10, 2020 at 07:45:18AM -0700, Andi Kleen wrote: > > > Unfortunately we're kind of stuck with the old NFILE=1024 default > > even though it makes little sense on modern servers. > > Why can't that be changed? It seems to

Re: file metadata via fs API (was: [GIT PULL] Filesystem Information)

2020-08-11 Thread Al Viro
On Tue, Aug 11, 2020 at 04:36:32PM +0200, Miklos Szeredi wrote: > > > - strip off trailing part after first instance of /// > > > - perform path lookup as normal > > > - resolve meta path after /// on result of normal lookup > > > > ... and interpolation of relative symlink body into the pathna

Re: [PATCH] fs: autofs: delete repeated words in comments

2020-08-11 Thread Randy Dunlap
On 8/11/20 1:36 AM, Ian Kent wrote: > On Mon, 2020-08-10 at 19:18 -0700, Randy Dunlap wrote: >> Drop duplicated words {the, at} in comments. >> >> Signed-off-by: Randy Dunlap >> Cc: Ian Kent >> Cc: aut...@vger.kernel.org > > Acked-by: Ian Kent Hi Ian, Since you are the listed maintainer of th

Re: [RFC-PATCH 1/2] mm: Add __GFP_NO_LOCKS flag

2020-08-11 Thread Thomas Gleixner
Michal Hocko writes: > On Mon 10-08-20 18:07:39, Uladzislau Rezki wrote: >> > On Sun 09-08-20 22:43:53, Uladzislau Rezki (Sony) wrote: >> > Is there any fundamental problem to make zone raw_spin_lock? >> > >> Good point. Converting a regular spinlock to the raw_* variant can solve >> an issue an

Re: possible deadlock in __io_queue_deferred

2020-08-11 Thread Stefano Garzarella
On Tue, Aug 11, 2020 at 08:21:12AM -0600, Jens Axboe wrote: > On 8/11/20 8:00 AM, Stefano Garzarella wrote: > > On Mon, Aug 10, 2020 at 09:55:17AM -0600, Jens Axboe wrote: > >> On 8/10/20 9:36 AM, syzbot wrote: > >>> Hello, > >>> > >>> syzbot found the following issue on: > >>> > >>> HEAD commit:

Re: possible deadlock in __io_queue_deferred

2020-08-11 Thread Jens Axboe
On 8/11/20 8:44 AM, Stefano Garzarella wrote: > On Tue, Aug 11, 2020 at 08:21:12AM -0600, Jens Axboe wrote: >> On 8/11/20 8:00 AM, Stefano Garzarella wrote: >>> On Mon, Aug 10, 2020 at 09:55:17AM -0600, Jens Axboe wrote: On 8/10/20 9:36 AM, syzbot wrote: > Hello, > > syzbot found t

Re: [PATCH v3 4/5] mm: memcg: charge memcg percpu memory to the parent cgroup

2020-08-11 Thread Michal Koutný
On Thu, Aug 06, 2020 at 09:37:17PM -0700, Roman Gushchin wrote: > In general, yes. But in this case I think it wouldn't be a good idea: > most often cgroups are created by a centralized daemon (systemd), > which is usually located in the root cgroup. Even if it's located not in > the root cgroup,

Re: file metadata via fs API (was: [GIT PULL] Filesystem Information)

2020-08-11 Thread Miklos Szeredi
On Tue, Aug 11, 2020 at 4:42 PM Al Viro wrote: > > On Tue, Aug 11, 2020 at 04:36:32PM +0200, Miklos Szeredi wrote: > > > > > - strip off trailing part after first instance of /// > > > > - perform path lookup as normal > > > > - resolve meta path after /// on result of normal lookup > > > > > >

RE: [PATCH 1/2] perf: Add closing sibling events' file descriptors

2020-08-11 Thread David Laight
From: Andi Kleen > On Mon, Aug 10, 2020 at 10:36:32PM +0200, Peter Zijlstra wrote: > > On Mon, Aug 10, 2020 at 07:45:18AM -0700, Andi Kleen wrote: > > > > > Unfortunately we're kind of stuck with the old NFILE=1024 default > > > even though it makes little sense on modern servers. > > > > Why can't

Re: [PATCH v17 14/21] mm/compaction: do page isolation first in compaction

2020-08-11 Thread Alexander Duyck
On Tue, Aug 11, 2020 at 1:23 AM Alex Shi wrote: > > > > 在 2020/8/10 下午10:41, Alexander Duyck 写道: > > On Mon, Aug 10, 2020 at 6:10 AM Alex Shi wrote: > >> > >> > >> > >> 在 2020/8/7 下午10:51, Alexander Duyck 写道: > >>> I wonder if this entire section shouldn't be restructured. This is the > >>> only

Re: [dm-devel] [RFC PATCH v5 00/11] Integrity Policy Enforcement LSM (IPE)

2020-08-11 Thread Chuck Lever
> On Aug 11, 2020, at 1:43 AM, James Bottomley > wrote: > > On Mon, 2020-08-10 at 19:36 -0400, Chuck Lever wrote: >>> On Aug 10, 2020, at 11:35 AM, James Bottomley >>> wrote: >>> On Sun, 2020-08-09 at 13:16 -0400, Mimi Zohar wrote: On Sat, 2020-08-08 at 13:47 -0400, Chuck Lever wrote: >

Re: [PATCH V5 0/3] Intel Platform Monitoring Technology

2020-08-11 Thread David E. Box
On Tue, 2020-08-11 at 09:04 +0100, Lee Jones wrote: > On Mon, 10 Aug 2020, David E. Box wrote: > > > Friendly ping. > > Don't do that. Sending contentless pings is seldom helpful. > > If you think your set has been dropped please just send a [RESEND]. > > This is probably worth doing anyway, s

[PATCH] Bug fix to ELF Loader which rejects valid ELFs

2020-08-11 Thread Burrow, Ryan - 0553 - MITLL
Check was incorrectly being applied to size of elf phdrs, instead of the number. The ELF standard allows for up to 65535 headers, but the check was being compared to the number of headers multiplied by the size of a program header. --- fs/binfmt_elf.c | 6 +++--- 1 file changed, 3 insertions(+),

Re: [PATCH v3 2/2] ASoC: Intel: Add period size constraint on strago board

2020-08-11 Thread Mark Brown
On Tue, Aug 11, 2020 at 05:35:45PM +0800, Yu-Hsuan Hsu wrote: > Takashi Iwai 於 2020年8月11日 週二 下午4:39寫道: > > ... Why only 240? That's the next logical question. > > If you have a clarification for it, it may be the rigid reason to > > introduce such a hw constraint. > According to Brent, the DSP

Re: [PATCH v36 21/24] x86/vdso: Implement a vDSO for Intel SGX enclave call

2020-08-11 Thread Sean Christopherson
On Tue, Aug 11, 2020 at 09:16:28AM +0200, Jethro Beekman wrote: > On 2020-08-11 00:23, Sean Christopherson wrote: > > Another thought would be to wrap sgx_enclave_exception in a struct to give > > room for supporting additional exit information (if such a thing ever pops > > up) and to allow the ca

Re: [RFC 0/5] Introduce NMI aware serial drivers

2020-08-11 Thread Greg Kroah-Hartman
On Tue, Aug 11, 2020 at 07:59:24PM +0530, Sumit Garg wrote: > Hi Greg, > > Thanks for your comments. > > On Tue, 11 Aug 2020 at 19:27, Greg Kroah-Hartman > wrote: > > > > On Tue, Aug 11, 2020 at 07:20:26PM +0530, Sumit Garg wrote: > > > On Tue, 21 Jul 2020 at 17:40, Sumit Garg wrote: > > > > >

Re: memory leak in io_submit_sqes

2020-08-11 Thread Jens Axboe
On 8/11/20 7:57 AM, syzbot wrote: > Hello, > > syzbot found the following issue on: > > HEAD commit:d6efb3ac Merge tag 'tty-5.9-rc1' of git://git.kernel.org/p.. > git tree: upstream > console output: https://syzkaller.appspot.com/x/log.txt?x=13cb076290 > kernel config: https://syzk

Re: [PATCH v2] module: Harden STRICT_MODULE_RWX

2020-08-11 Thread peterz
On Tue, Aug 11, 2020 at 04:34:27PM +0200, Mauro Carvalho Chehab wrote: > [33] .plt PROGBITS 0340 00035c80 >0001 WAX 0 0 1 > [34] .init.plt NOBITS 0341 00035c81 >0

[PATCH v3 2/4] MIPS: BCM63xx: remove EHCI from BCM6348 boards

2020-08-11 Thread Álvaro Fernández Rojas
There's no EHCI controller on BCM6348. Signed-off-by: Álvaro Fernández Rojas Acked-by: Florian Fainelli --- v3: no changes. v2: no changes. arch/mips/bcm63xx/boards/board_bcm963xx.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/arch/mips/bcm63xx/boards/board_bcm963xx.c b/arch/mips/b

[PATCH v3 3/4] MIPS: BCM63xx: enable EHCI for DWV-S0 board

2020-08-11 Thread Álvaro Fernández Rojas
BCM6358 SoCs have OHCI and EHCI controllers that share the same USB ports. Therefore, the board should also have EHCI enabled. Signed-off-by: Álvaro Fernández Rojas Acked-by: Florian Fainelli --- v3: Reword commit description to avoid possible confusions. v2: no changes. arch/mips/bcm63xx/bo

[PATCH v3 0/4] MIPS: BCM63xx: board improvements

2020-08-11 Thread Álvaro Fernández Rojas
Theses patches improve BCM63xx board declarations and source code. v3: Reword DWV-S0 board commit description to avoid possible confusions. v2: switch to SPDX license identifier. Álvaro Fernández Rojas (4): MIPS: BCM63xx: remove duplicated new lines MIPS: BCM63xx: remove EHCI from BCM6348 boa

[PATCH v3 1/4] MIPS: BCM63xx: remove duplicated new lines

2020-08-11 Thread Álvaro Fernández Rojas
There are 3 duplicated new lines, let's remove them. Signed-off-by: Álvaro Fernández Rojas Acked-by: Florian Fainelli Reviewed-by: Philippe Mathieu-Daudé --- v3: no changes. v2: no changes. arch/mips/bcm63xx/boards/board_bcm963xx.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/arch/

[PATCH] debugfs: Fix module state check condition

2020-08-11 Thread Vladis Dronov
The '#ifdef MODULE' check in the original commit does not work as intended. The code under the check is not built at all if CONFIG_DEBUG_FS=y. Fix this by using a correct check. Fixes: 275678e7a9be ("debugfs: Check module state before warning in {full/open}_proxy_open()") Signed-off-by: Vladis Dr

[PATCH v3 4/4] MIPS: BCM63xx: refactor board declarations

2020-08-11 Thread Álvaro Fernández Rojas
Current board declarations are a mess. Let's put some order and make them follow the same structure. Also board declarations tabs. Switch to SPDX license identifier. Signed-off-by: Álvaro Fernández Rojas Acked-by: Florian Fainelli --- v3: no changes. v2: switch to SPDX license identifier. ar

Re: [PATCH] vt: defer kfree() of vc_screenbuf in vc_do_resize()

2020-08-11 Thread Tetsuo Handa
On 2020/08/04 21:58, Greg Kroah-Hartman wrote: > On Tue, Aug 04, 2020 at 08:15:43PM +0900, Tetsuo Handa wrote: >> Do you think this approach is acceptable? Or, do we need to modify >> set_origin() ? >> > I think what you have here is fine, as cleaning up set_orgin() might be > hard to do at this p

Re: [PATCH v2] mm/gup: Allow real explicit breaking of COW

2020-08-11 Thread Linus Torvalds
On Mon, Aug 10, 2020 at 4:39 PM Jann Horn wrote: > > The way I understand Peter, he doesn't want to avoid doing COW; he > wants to decouple userfaultfd-WP's fault handling from COW, so that > userfaultfd-wp notifies only when a previously-write-protected page is > actually written to. In other wor

Re: [PATCH] Bug fix to ELF Loader which rejects valid ELFs

2020-08-11 Thread Matthew Wilcox
On Tue, Aug 11, 2020 at 02:44:08PM +, Burrow, Ryan - 0553 - MITLL wrote: > /* Sanity check the number of program headers... */ > - /* ...and their total size. */ > - size = sizeof(struct elf_phdr) * elf_ex->e_phnum; > - if (size == 0 || size > 65536 || size > ELF_MIN_ALIGN) >

Re: [PATCH v3 3/5] mm: memcg/percpu: per-memcg percpu memory statistics

2020-08-11 Thread Johannes Weiner
On Tue, Jun 23, 2020 at 11:45:13AM -0700, Roman Gushchin wrote: > Percpu memory can represent a noticeable chunk of the total memory > consumption, especially on big machines with many CPUs. Let's track > percpu memory usage for each memcg and display it in memory.stat. > > A percpu allocation is

Re: [PATCH] driver core: Let dev_err_probe() use the symbolic error code

2020-08-11 Thread Andrzej Hajda
Hi Uwe, On 11.08.2020 09:20, Uwe Kleine-König wrote: > This makes the error message: > > error -EIO: ... > > instead of > > error -5: ... > > Signed-off-by: Uwe Kleine-König > --- > drivers/base/core.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/dri

Re: [PATCH] vt: defer kfree() of vc_screenbuf in vc_do_resize()

2020-08-11 Thread Greg Kroah-Hartman
On Wed, Aug 12, 2020 at 12:02:03AM +0900, Tetsuo Handa wrote: > On 2020/08/04 21:58, Greg Kroah-Hartman wrote: > > On Tue, Aug 04, 2020 at 08:15:43PM +0900, Tetsuo Handa wrote: > >> Do you think this approach is acceptable? Or, do we need to modify > >> set_origin() ? > >> > > I think what you hav

[PATCH v2 1/2] clk: samsung: exynos5420: Add definition of clock ID for mout_sw_aclk_g3d

2020-08-11 Thread Sylwester Nawrocki
This patch adds ID for the mout_sw_aclk_g3d (SW_CLKMUX_ACLK_G3D) clock, mostly for internal use in the CMU driver. It will allow to avoid the __clk_lookup() call when setting up the clock during the clock provider initialization. Signed-off-by: Sylwester Nawrocki --- include/dt-bindings/clock/ex

[PATCH v2 2/2] clk: samsung: exynos5420: Avoid __clk_lookup() calls when enabling clocks

2020-08-11 Thread Sylwester Nawrocki
This patch adds a clk ID to the mout_sw_aclk_g3d clk definition so related clk pointer gets cached in the driver's private data and can be used later instead of a __clk_lookup() call. With that we have all clocks used in the clk_prepare_enable() calls in the clk provider init callback cached in cl

[PATCH] USB: realtek_cr: fix return check for dma functions

2020-08-11 Thread trix
From: Tom Rix clang static analysis reports this representative problem realtek_cr.c:639:3: warning: The left expression of the compound assignment is an uninitialized value. The computed value will also be garbage SET_BIT(value, 2); ^ value is set by a successful ca

Re: [PATCH v36 21/24] x86/vdso: Implement a vDSO for Intel SGX enclave call

2020-08-11 Thread Andy Lutomirski
> On Aug 10, 2020, at 5:52 PM, Andy Lutomirski wrote: > >  >>> On Aug 10, 2020, at 4:48 PM, Sean Christopherson >>> wrote: >>> >>> On Mon, Aug 10, 2020 at 04:08:46PM -0700, Andy Lutomirski wrote: >>> What am I missing? I still don't really understand why we are >>> supporting this mechani

Re: file metadata via fs API (was: [GIT PULL] Filesystem Information)

2020-08-11 Thread Linus Torvalds
[ I missed the beginning of this discussion, so maybe this was already suggested ] On Tue, Aug 11, 2020 at 6:54 AM Miklos Szeredi wrote: > > > > > E.g. > > openat(AT_FDCWD, "foo/bar//mnt/info", O_RDONLY | O_ALT); > > Proof of concept patch and test program below. I don't think this works for t

Re: [RFC-PATCH 1/2] mm: Add __GFP_NO_LOCKS flag

2020-08-11 Thread Thomas Gleixner
Thomas Gleixner writes: > Michal Hocko writes: >> zone->lock should be held for a very limited amount of time. > > Emphasis on should. free_pcppages_bulk() can hold it for quite some time > when a large amount of pages are purged. We surely would have converted > it to a raw lock long time ago ot

Re: [PATCH] task_work: only grab task signal lock when needed

2020-08-11 Thread Oleg Nesterov
On 08/11, Jens Axboe wrote: > > --- a/kernel/task_work.c > +++ b/kernel/task_work.c > @@ -42,7 +42,8 @@ task_work_add(struct task_struct *task, struct > callback_head *work, int notify) > set_notify_resume(task); > break; > case TWA_SIGNAL: > - if (loc

Re: memory leak in io_submit_sqes

2020-08-11 Thread Jens Axboe
On 8/11/20 8:59 AM, Jens Axboe wrote: > On 8/11/20 7:57 AM, syzbot wrote: >> Hello, >> >> syzbot found the following issue on: >> >> HEAD commit:d6efb3ac Merge tag 'tty-5.9-rc1' of git://git.kernel.org/p.. >> git tree: upstream >> console output: https://syzkaller.appspot.com/x/log.txt?x=

Re: [PATCH for 5.9 v2 1/4] futex: introduce FUTEX_SWAP operation

2020-08-11 Thread Rik van Riel
On Tue, 2020-08-04 at 14:31 +0200, pet...@infradead.org wrote: > On Mon, Aug 03, 2020 at 03:15:07PM -0700, Peter Oskolkov wrote: > > A simplified/idealized use case: imagine a multi-user service > > application > > (e.g. a DBMS) that has to implement the following user CPU quota > > policy: > > So

Re: [PATCH] [v2] tools/objtool: Fix unnecessary jumps

2020-08-11 Thread Kamalesh Babulal
On 11/08/20 9:48 am, Youling Tang wrote: > There is no need to jump to the "out" tag when "ret < 0", just return > directly to "ret". > > Signed-off-by: Youling Tang the patch looks good to me, the commit message doesn't explain the reason for the change. It can be written like: "Previously cle

Re: power-off delay/hang due to commit 6d25be57 (mainline)

2020-08-11 Thread Sebastian Andrzej Siewior
On 2020-08-11 16:34:09 [+0200], Rafael J. Wysocki wrote: > On Tue, Aug 11, 2020 at 3:29 PM Sebastian Andrzej Siewior > wrote: > > > > On 2020-08-11 13:58:39 [+0200], Stephen Berman wrote: > > > him about your workaround of adding 'thermal.tzp=300' to the kernel > > > commandline, and he replied th

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