Re: [PATCH v3] nvme-pci: Support shared tags across queues for Apple 2018 controllers

2019-07-30 Thread Keith Busch
On Fri, Jul 19, 2019 at 03:31:02PM +1000, Benjamin Herrenschmidt wrote: > From 8dcba2ef5b1466b023b88b4eca463b30de78d9eb Mon Sep 17 00:00:00 2001 > From: Benjamin Herrenschmidt > Date: Fri, 19 Jul 2019 15:03:06 +1000 > Subject: > > Another issue with the Apple T2 based 2018 controllers seem to be

Re: [Regression] Commit "nvme/pci: Use host managed power state for suspend" has problems

2019-07-30 Thread Keith Busch
On Wed, Jul 31, 2019 at 02:50:01AM +0800, Kai-Heng Feng wrote: > > Just did a quick test, this patch regress SK Hynix BC501, the SoC stays at > PC3 once the patch is applied. Okay, I'm afraid device/platform quirks may be required unless there are any other ideas out there. I'm not a big fan of

Re: [Regression] Commit "nvme/pci: Use host managed power state for suspend" has problems

2019-07-30 Thread Keith Busch
On Tue, Jul 30, 2019 at 09:05:22PM +, mario.limoncie...@dell.com wrote: > > -Original Message- > > From: Keith Busch > > Sent: Tuesday, July 30, 2019 2:20 PM > > To: Kai-Heng Feng > > Cc: Limonciello, Mario; r...@rjwysocki.net; keith.bu...@inte

Re: [PATCH v3] nvme-pci: Support shared tags across queues for Apple 2018 controllers

2019-08-05 Thread Keith Busch
then I'll implement the "proper" way > sometimes this week, adding a way to shrink the AQ down to something > like 3 (one admin request, one async event (AEN), and the empty slot) > by making a bunch of the constants involved variables instead. I don't feel too strongly about it. I think your patch is fine, so Acked-by: Keith Busch

[PATCH 2/3] hmat: Register attributes for memory hot add

2019-08-05 Thread Keith Busch
siter a memory notifier callback and register the memory attributes the first time its node is brought online if it wasn't registered. Signed-off-by: Keith Busch --- drivers/acpi/hmat/hmat.c | 75 1 file changed, 57 insertions(+), 18 deletion

[PATCH 1/3] hmat: Register memory-side cache after parsing

2019-08-05 Thread Keith Busch
Instead of registering the hmat cache attributes in line with parsing the table, save the attributes in the memory target and register them after parsing completes. This will make it easier to register the attributes later when hot add is supported. Tested-by: Brice Goglin Signed-off-by: Keith

[PATCH 0/3] HMAT node online fixes

2019-08-05 Thread Keith Busch
From: Keith Busch Hi Rafael, These are just some fixes from a while ago to work correctly with memory node onlining, but haven't been merged in yet. I've included a fix from Dan, but had to modify it slightly for conflicts. I think it makes most sense for this to go through the acpi

[PATCH 3/3] acpi/hmat: Skip publishing target info for nodes with no online memory

2019-08-05 Thread Keith Busch
: Invalid table" ...result for HMAT parsing. Reviewed-by: Dave Hansen Reviewed-by: Keith Busch Acked-by: Rafael J. Wysocki Signed-off-by: Dan Williams --- drivers/acpi/hmat/hmat.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/drivers/acpi/hmat/hmat.c

Re: [PATCH v3] nvme-pci: Support shared tags across queues for Apple 2018 controllers

2019-08-05 Thread Keith Busch
o something > > > like 3 (one admin request, one async event (AEN), and the empty slot) > > > by making a bunch of the constants involved variables instead. > > > > I don't feel too strongly about it. I think your patch is fine, so > > > > Acked-by: Kei

Re: [PATCHv2 1/2] hmat: Register memory-side cache after parsing

2019-07-24 Thread Keith Busch
On Thu, Jun 13, 2019 at 01:27:05PM -0700, Rafael J. Wysocki wrote: > On Wednesday, May 15, 2019 11:54:43 PM CEST Keith Busch wrote: > > Instead of registering the hmat cache attributes in line with parsing > > the table, save the attributes in the memory target and register them &g

Re:

2019-03-22 Thread Keith Busch
On Fri, Mar 22, 2019 at 07:54:50AM +, Felipe Franciosi wrote: > > > > Note though that SPDK doesn't support sharing the device between host and > > the > > guests, it takes over the nvme device, thus it makes the kernel nvme driver > > unbind from it. > > That is absolutely true. However, I

Re: [PATCH 06/10] mm: vmscan: demote anon DRAM pages to PMEM node

2019-03-24 Thread Keith Busch
On Sat, Mar 23, 2019 at 12:44:31PM +0800, Yang Shi wrote: > /* > + * Demote DRAM pages regardless the mempolicy. > + * Demot anonymous pages only for now and skip MADV_FREE > + * pages. > + */ > + if (PageAnon(page) && !P

Re: [PATCH] nvme: fix NULL pointer dereference

2020-09-16 Thread Keith Busch
On Wed, Sep 16, 2020 at 11:36:49AM -0400, Tong Zhang wrote: > @@ -960,6 +960,8 @@ static inline void nvme_handle_cqe(struct nvme_queue > *nvmeq, u16 idx) > } > > req = blk_mq_tag_to_rq(nvme_queue_tagset(nvmeq), cqe->command_id); > + if (!req) > + return; As I mention

Re: [PATCH] nvme: fix NULL pointer dereference

2020-09-18 Thread Keith Busch
On Thu, Sep 17, 2020 at 11:32:12PM -0400, Tong Zhang wrote: > Please correct me if I am wrong. > After a bit more digging I found out that it is indeed command_id got > corrupted is causing this problem. Although the tag and command_id > range is checked like you said, the elements in rqs cannot be

Re: [PATCH] [v2] nvme: use correct upper limit for tag in nvme_handle_cqe()

2020-09-18 Thread Keith Busch
On Fri, Sep 18, 2020 at 06:44:20PM +0800, Xianting Tian wrote: > @@ -940,7 +940,9 @@ static inline void nvme_handle_cqe(struct nvme_queue > *nvmeq, u16 idx) > struct nvme_completion *cqe = &nvmeq->cqes[idx]; > struct request *req; > > - if (unlikely(cqe->command_id >= nvmeq->q_de

Re: [PATCH] nvme: replace meaningless judgement by checking whether req is null

2020-09-21 Thread Keith Busch
On Mon, Sep 21, 2020 at 10:10:52AM +0800, Xianting Tian wrote: > @@ -940,13 +940,6 @@ static inline void nvme_handle_cqe(struct nvme_queue > *nvmeq, u16 idx) > struct nvme_completion *cqe = &nvmeq->cqes[idx]; > struct request *req; > > - if (unlikely(cqe->command_id >= nvmeq->q_d

Re: [PATCH] nvme: replace meaningless judgement by checking whether req is null

2020-09-21 Thread Keith Busch
s queue_depth is a valid point to mention as well. The dirver's current indirect check is not necessarily in sync with the actual tagset. > Thanks > > -Original Message- > From: Keith Busch [mailto:kbu...@kernel.org] > Sent: Monday, September 21, 2020 11:0

Re: [PATCH] nvme: fix doulbe irq free

2020-09-17 Thread Keith Busch
On Thu, Sep 17, 2020 at 11:22:54AM -0400, Tong Zhang wrote: > On Thu, Sep 17, 2020 at 4:30 AM Christoph Hellwig wrote: > > > > On Wed, Sep 16, 2020 at 11:37:00AM -0400, Tong Zhang wrote: > > > the irq might already been released before reset work can run > > > > If it is we have a problem with the

Re: [PATCH] nvme: fix NULL pointer dereference

2020-09-17 Thread Keith Busch
On Thu, Sep 17, 2020 at 12:56:59PM -0400, Tong Zhang wrote: > The command_id in CQE is writable by NVMe controller, driver should > check its sanity before using it. We already do that.

Re: [PATCH] [v2] nvme: replace meaningless judgement by checking whether req is null

2020-09-22 Thread Keith Busch
The commit subject is a too long. We should really try to keep these to 50 characters or less. nvme-pci: fix NULL req in completion handler Otherwise, looks fine. Reviewed-by: Keith Busch

Re: [PATCH AUTOSEL 5.8 11/42] nvme: skip noiob for zoned devices

2020-08-31 Thread Keith Busch
On Mon, Aug 31, 2020 at 11:29:03AM -0400, Sasha Levin wrote: > From: Keith Busch > > [ Upstream commit c41ad98bebb8f4f0335b3c50dbb7583a6149dce4 ] > > Zoned block devices reuse the chunk_sectors queue limit to define zone > boundaries. If a such a device happens to also

Re: [PATCH V3 1/1] nvme: Add quirk for LiteON CL1 devices running FW 220TQ,22001

2020-10-28 Thread Keith Busch
On Thu, Oct 29, 2020 at 02:20:27AM +, Gloria Tsai wrote: > Corrected the description of this bug that SSD will not do GC after receiving > shutdown cmd. > Do GC before shutdown -> delete IO Q -> shutdown from host -> breakup GC -> > D3hot -> enter PS4 -> have a chance swap block -> use wrong

Re: [PATCH V3 1/1] nvme: Add quirk for LiteON CL1 devices running FW 220TQ,22001

2020-10-29 Thread Keith Busch
On Thu, Oct 29, 2020 at 11:33:06AM +0900, Keith Busch wrote: > On Thu, Oct 29, 2020 at 02:20:27AM +, Gloria Tsai wrote: > > Corrected the description of this bug that SSD will not do GC after > > receiving shutdown cmd. > > Do GC before shutdown -> delete IO Q -> sh

Re: [PATCH 1/3] nvme: Add Arbitration Burst support

2020-06-23 Thread Keith Busch
On Tue, Jun 23, 2020 at 09:24:32PM +0800, Baolin Wang wrote: > From the NVMe spec, "In order to make efficient use of the non-volatile > memory, it is often advantageous to execute multiple commands from a > Submission Queue in parallel. For Submission Queues that are using > weighted round robin w

Re: [PATCH 1/3] nvme: Add Arbitration Burst support

2020-06-23 Thread Keith Busch
On Tue, Jun 23, 2020 at 10:39:01AM -0700, Sagi Grimberg wrote: > > > > From the NVMe spec, "In order to make efficient use of the non-volatile > > > memory, it is often advantageous to execute multiple commands from a > > > Submission Queue in parallel. For Submission Queues that are using > > >

[tip:irq/urgent] genirq: Fix memory leak when calling irq_free_hwirqs()

2014-07-05 Thread tip-bot for Keith Busch
Commit-ID: 8844aad89ed61545b4db6a3467e1b21ca1c49460 Gitweb: http://git.kernel.org/tip/8844aad89ed61545b4db6a3467e1b21ca1c49460 Author: Keith Busch AuthorDate: Mon, 30 Jun 2014 16:24:44 -0600 Committer: Thomas Gleixner CommitDate: Sat, 5 Jul 2014 21:42:08 +0200 genirq: Fix memory leak

[tip:irq/urgent] genirq/affinity: Fix calculating vectors to assign

2017-04-20 Thread tip-bot for Keith Busch
Commit-ID: b72f8051f34b8164a62391e3676edc34523c5952 Gitweb: http://git.kernel.org/tip/b72f8051f34b8164a62391e3676edc34523c5952 Author: Keith Busch AuthorDate: Wed, 19 Apr 2017 19:51:10 -0400 Committer: Thomas Gleixner CommitDate: Thu, 20 Apr 2017 16:03:09 +0200 genirq/affinity: Fix

[tip:irq/urgent] irq/affinity: Fix extra vecs calculation

2017-04-13 Thread tip-bot for Keith Busch
Commit-ID: 3412386b531244f24a27c79ee003506a52a00848 Gitweb: http://git.kernel.org/tip/3412386b531244f24a27c79ee003506a52a00848 Author: Keith Busch AuthorDate: Thu, 13 Apr 2017 13:28:12 -0400 Committer: Thomas Gleixner CommitDate: Thu, 13 Apr 2017 23:41:00 +0200 irq/affinity: Fix extra

[tip:irq/urgent] irq/affinity: Fix CPU spread for unbalanced nodes

2017-04-04 Thread tip-bot for Keith Busch
Commit-ID: 7bf8222b9bd0ba867e18b7f4537b61ef2e92eee8 Gitweb: http://git.kernel.org/tip/7bf8222b9bd0ba867e18b7f4537b61ef2e92eee8 Author: Keith Busch AuthorDate: Mon, 3 Apr 2017 15:25:53 -0400 Committer: Thomas Gleixner CommitDate: Tue, 4 Apr 2017 11:57:28 +0200 irq/affinity: Fix CPU

[tip:irq/core] genirq: Add untracked irq handler

2016-06-18 Thread tip-bot for Keith Busch
Commit-ID: edd14cfebc4404698544d407ecf8eda6e19aa19e Gitweb: http://git.kernel.org/tip/edd14cfebc4404698544d407ecf8eda6e19aa19e Author: Keith Busch AuthorDate: Fri, 17 Jun 2016 16:00:20 -0600 Committer: Thomas Gleixner CommitDate: Sat, 18 Jun 2016 10:00:55 +0200 genirq: Add untracked

[tip:irq/core] x86/PCI/VMD: Use untracked irq handler

2016-06-18 Thread tip-bot for Keith Busch
Commit-ID: 30ce0350381351646ef86b64e6d3840b3869833b Gitweb: http://git.kernel.org/tip/30ce0350381351646ef86b64e6d3840b3869833b Author: Keith Busch AuthorDate: Fri, 17 Jun 2016 16:00:21 -0600 Committer: Thomas Gleixner CommitDate: Sat, 18 Jun 2016 10:00:55 +0200 x86/PCI/VMD: Use

<    5   6   7   8   9   10