Re: [PATCH v2 00/16] nvme: refactoring and cleanups

2020-04-22 Thread Klaus Birkelund Jensen
On Apr 21 19:24, Maxim Levitsky wrote: > Should I also review the V7 series or I should wait for V8 which will > not include these cleanups? Hi Maxim, Just wait for another series - I don't think I will post a v8, I will chop op the series into smaller ones instead. Most patches will hopefully

Re: [PATCH v2 00/16] nvme: refactoring and cleanups

2020-04-21 Thread Maxim Levitsky
On Wed, 2020-04-15 at 15:01 +0200, Klaus Jensen wrote: > From: Klaus Jensen > > Changes since v1 > > * nvme: fix pci doorbell size calculation > - added some defines and a better comment (Philippe) > > * nvme: rename trace events to pci_nvme > - changed the prefix from

Re: [PATCH v2 00/16] nvme: refactoring and cleanups

2020-04-21 Thread Kevin Wolf
Am 21.04.2020 um 08:38 hat Klaus Birkelund Jensen geschrieben: > On Apr 21 02:38, Keith Busch wrote: > > The series looks good to me. > > > > Reviewed-by: Keith Busch > > Thanks for the review Keith! > > Kevin, should I rebase this on block-next? I think it might have some > conflicts with the

Re: [PATCH v2 00/16] nvme: refactoring and cleanups

2020-04-21 Thread Klaus Birkelund Jensen
On Apr 21 02:38, Keith Busch wrote: > The series looks good to me. > > Reviewed-by: Keith Busch Thanks for the review Keith! Kevin, should I rebase this on block-next? I think it might have some conflicts with the PMR patch that went in previously. Philippe, then I can also change the *err to

Re: [PATCH v2 00/16] nvme: refactoring and cleanups

2020-04-20 Thread Keith Busch
The series looks good to me. Reviewed-by: Keith Busch

Re: [PATCH v2 00/16] nvme: refactoring and cleanups

2020-04-19 Thread Klaus Birkelund Jensen
On Apr 15 15:01, Klaus Jensen wrote: > From: Klaus Jensen > > Changes since v1 > > * nvme: fix pci doorbell size calculation > - added some defines and a better comment (Philippe) > > * nvme: rename trace events to pci_nvme > - changed the prefix from nvme_dev to pci_nvme

Re: [PATCH v2 00/16] nvme: refactoring and cleanups

2020-04-15 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200415130159.611361-1-...@irrelevant.dk/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [PATCH v2 00/16] nvme: refactoring and cleanups Message-id: 20200415130159.611361-1-...@irrelevant.dk Type

[PATCH v2 00/16] nvme: refactoring and cleanups

2020-04-15 Thread Klaus Jensen
From: Klaus Jensen Changes since v1 * nvme: fix pci doorbell size calculation - added some defines and a better comment (Philippe) * nvme: rename trace events to pci_nvme - changed the prefix from nvme_dev to pci_nvme (Philippe) * nvme: add max_ioqpairs device parameter