[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 2/2] nvme: allow cmb and pmr to be enabled on same device
From: |
Klaus Jensen |
Subject: |
Re: [PATCH v6 2/2] nvme: allow cmb and pmr to be enabled on same device |
Date: |
Tue, 22 Sep 2020 11:04:25 +0200 |
On Aug 17 08:29, Klaus Jensen wrote:
> On Jul 30 00:50, Klaus Jensen wrote:
> > On Jul 29 15:01, Andrzej Jakowski wrote:
> > > So far it was not possible to have CMB and PMR emulated on the same
> > > device, because BAR2 was used exclusively either of PMR or CMB. This
> > > patch places CMB at BAR4 offset so it not conflicts with MSI-X vectors.
> > >
> > > Signed-off-by: Andrzej Jakowski <andrzej.jakowski@linux.intel.com>
> > > ---
> >
> > Well, I'm certainly happy now. LGTM!
> >
> > Reviewed-by: Klaus Jensen <k.jensen@samsung.com>
> >
>
> Are anyone willing to chip in with another review on this?
>
I think this patch is ready (and have been for some time) for inclusion,
but would really like an additional review on this; preferably from
Keith, since he is the one that originally mentioned that we could do
something like this.
I've mentioned it before, but I would prefer that the MSI-X stuff was in
BAR0 instead of mixing it with the CMB, but that's bikeshedding and my
R-b still holds of course.
signature.asc
Description: PGP signature
- Re: [PATCH v6 2/2] nvme: allow cmb and pmr to be enabled on same device,
Klaus Jensen <=