On Jul 8 21:47, Dmitry Fomichev wrote: > > > -----Original Message----- > > From: Klaus Jensen <i...@irrelevant.dk> > > Sent: Wednesday, July 8, 2020 5:24 PM > > To: Dmitry Fomichev <dmitry.fomic...@wdc.com> > > Cc: qemu-block@nongnu.org; qemu-de...@nongnu.org; f...@euphon.net; > > javier.g...@samsung.com; kw...@redhat.com; mre...@redhat.com; > > mlevi...@redhat.com; phi...@redhat.com; kbu...@kernel.org; > > k.jen...@samsung.com > > Subject: Re: [PATCH v3 01/18] hw/block/nvme: bump spec data structures to > > v1.3 > > > > On Jul 8 19:19, Dmitry Fomichev wrote: > > > Looks good with a small nit (see below), > > > > > > Reviewed-by: Dmitry Fomichev <dmitry.fomic...@wdc.com> > > > > > > > > > > On Mon, 2020-07-06 at 08:12 +0200, Klaus Jensen wrote: > > > > +#define NVME_TEMP_TMPTH(temp) ((temp >> 0) & 0xffff) > > > > > > There is an extra space after temp >> > > > > > > > Good catch! I won't repost for this ;) - but I'll fix it and add it in > > the qemu-nvme tree. > > Yes, no need to repost :) Thanks for reviewing our ZNS series! I am working > on addressing your comments and I am also starting to review your > "AIO and address mapping refactoring" patchset. >
Since I think this patchset gets merged on nvme-next today, there is a v2 on the way for that set.