RE: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Matias Bjorling
Alistair Francis ; Philippe > Mathieu-Daudé > Subject: Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and > Zoned Namespace Command Set > > On Sep 29 18:17, Matias Bjorling wrote: > > > > > > > -Original Message- > > > From: Klaus Je

RE: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Matias Bjorling
t; ; Philippe Mathieu-Daudé ; > Matias Bjorling > Subject: Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and > Zoned Namespace Command Set > > On Sep 29 10:29, Keith Busch wrote: > > On Tue, Sep 29, 2020 at 12:46:33PM +0200, Klaus Jensen wrote: > > > It is

RE: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Dmitry Fomichev
Alistair Francis ; Philippe > Mathieu-Daudé ; Matias Bjorling > > Subject: Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types > and Zoned Namespace Command Set > > All, > > Let's de-escalate this, please. There's no reason to doubt Klaus wants > to see

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Keith Busch
All, Let's de-escalate this, please. There's no reason to doubt Klaus wants to see this to work well, just as everyone else does. We unfortunately have conflicting proposals posted, and everyone is passionate enough about their work, but please simmer down. As I mentioned earlier, I'd like to ref

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Klaus Jensen
ieu-Daudé ; > > Maxim Levitsky ; Fam Zheng ; > > Niklas Cassel ; qemu-bl...@nongnu.org; qemu- > > de...@nongnu.org; Alistair Francis ; Matias > > Bjorling > > Subject: Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types > > and Zoned Namespace Command Set

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Klaus Jensen
g; Niklas Cassel ; Klaus Jensen > > ; qemu-devel@nongnu.org; Alistair Francis > > ; Philippe Mathieu-Daudé ; > > Matias Bjorling > > Subject: Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and > > Zoned Namespace Command Set > > > > On Sep 29 10

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Klaus Jensen
On Sep 29 11:15, Keith Busch wrote: > On Tue, Sep 29, 2020 at 08:00:04PM +0200, Klaus Jensen wrote: > > On Sep 29 10:29, Keith Busch wrote: > > > On Tue, Sep 29, 2020 at 12:46:33PM +0200, Klaus Jensen wrote: > > > > It is unmistakably clear that you are invalidating my arguments about > > > > porta

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Keith Busch
On Tue, Sep 29, 2020 at 08:00:04PM +0200, Klaus Jensen wrote: > On Sep 29 10:29, Keith Busch wrote: > > On Tue, Sep 29, 2020 at 12:46:33PM +0200, Klaus Jensen wrote: > > > It is unmistakably clear that you are invalidating my arguments about > > > portability and endianness issues by suggesting tha

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Klaus Jensen
On Sep 29 10:29, Keith Busch wrote: > On Tue, Sep 29, 2020 at 12:46:33PM +0200, Klaus Jensen wrote: > > It is unmistakably clear that you are invalidating my arguments about > > portability and endianness issues by suggesting that we just remove > > persistent state and deal with it later, but pers

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Keith Busch
On Tue, Sep 29, 2020 at 11:13:51AM +, Damien Le Moal wrote: > OK. Then let's move the persistence implementation as the last patch in the > series. This way, if it is still controversial, it will not block the rest. > > Here is what I propose: > Dmitry: remove persistence stuff from your patch

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Keith Busch
On Tue, Sep 29, 2020 at 12:46:33PM +0200, Klaus Jensen wrote: > It is unmistakably clear that you are invalidating my arguments about > portability and endianness issues by suggesting that we just remove > persistent state and deal with it later, but persistence is the killer > feature that sets th

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Klaus Jensen
; Kevin Wolf ; qemu- > > bl...@nongnu.org; Niklas Cassel ; Klaus Jensen > > ; qemu-devel@nongnu.org; Alistair Francis > > ; Keith Busch ; Philippe > > Mathieu-Daudé ; Matias Bjorling > > > > Subject: Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Type

RE: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Dmitry Fomichev
gt; ; qemu-devel@nongnu.org; Alistair Francis > ; Keith Busch ; Philippe > Mathieu-Daudé ; Matias Bjorling > > Subject: Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types > and Zoned Namespace Command Set > > On Sep 28 22:54, Damien Le Moal wrote: > > On 20

RE: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Dmitry Fomichev
l...@nongnu.org; qemu- > de...@nongnu.org; Alistair Francis ; Matias > Bjorling > Subject: Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types > and Zoned Namespace Command Set > > On Sep 28 02:33, Dmitry Fomichev wrote: > > > -Original Message- > >

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Damien Le Moal
On 2020/09/29 19:46, Klaus Jensen wrote: > On Sep 28 22:54, Damien Le Moal wrote: >> On 2020/09/29 6:25, Keith Busch wrote: >>> On Mon, Sep 28, 2020 at 08:36:48AM +0200, Klaus Jensen wrote: On Sep 28 02:33, Dmitry Fomichev wrote: > You are making it sound like the entire WDC series relies

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Klaus Jensen
On Sep 28 22:54, Damien Le Moal wrote: > On 2020/09/29 6:25, Keith Busch wrote: > > On Mon, Sep 28, 2020 at 08:36:48AM +0200, Klaus Jensen wrote: > >> On Sep 28 02:33, Dmitry Fomichev wrote: > >>> You are making it sound like the entire WDC series relies on this > >>> approach. > >>> Actually, the

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-28 Thread Damien Le Moal
On 2020/09/29 6:25, Keith Busch wrote: > On Mon, Sep 28, 2020 at 08:36:48AM +0200, Klaus Jensen wrote: >> On Sep 28 02:33, Dmitry Fomichev wrote: >>> You are making it sound like the entire WDC series relies on this approach. >>> Actually, the persistency is introduced in the second to last patch i

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-28 Thread Keith Busch
On Mon, Sep 28, 2020 at 08:36:48AM +0200, Klaus Jensen wrote: > On Sep 28 02:33, Dmitry Fomichev wrote: > > You are making it sound like the entire WDC series relies on this approach. > > Actually, the persistency is introduced in the second to last patch in the > > series and it only adds a couple

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-27 Thread Klaus Jensen
On Sep 28 02:33, Dmitry Fomichev wrote: > > -Original Message- > > From: Klaus Jensen > > > > If it really needs to be memory mapped, then I think a hostmem-based > > approach similar to what Andrzej did for PMR is needed (I think that > > will get rid of the CONFIG_POSIX ifdef at least, b

RE: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-27 Thread Dmitry Fomichev
u-bl...@nongnu.org; qemu-devel@nongnu.org; Alistair Francis > ; Matias Bjorling > Subject: Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types > and Zoned Namespace Command Set > > On Sep 24 03:20, Dmitry Fomichev wrote: > > v3 -> v4 > > > > - Fix bugs i

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-24 Thread Klaus Jensen
On Sep 24 03:20, Dmitry Fomichev wrote: > v3 -> v4 > > - Fix bugs introduced in v2/v3 for QD > 1 operation. Now, all writes >to a zone happen at the new write pointer variable, zone->w_ptr, >that is advanced right after submitting the backend i/o. The existing >zone->d.wp variable is

[PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-23 Thread Dmitry Fomichev
v3 -> v4 - Fix bugs introduced in v2/v3 for QD > 1 operation. Now, all writes to a zone happen at the new write pointer variable, zone->w_ptr, that is advanced right after submitting the backend i/o. The existing zone->d.wp variable is updated upon the successful write completion and