| Message ID | 20200930220414.562527-1-kbusch@kernel.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | nvme qemu cleanups and fixes | expand |
On Sep 30 15:04, Keith Busch wrote: > After going through the zns enabling, I notice the controller enabling > is not correct. Then I just continued maked more stuff. The series, I > think, contains some of the less controversial patches from the two > conflicting zns series, preceeded by some cleanups and fixes from me. > > If this is all fine, I took the liberty of porting the zns enabling to > it and made a public branch for consideration here: > > http://git.infradead.org/qemu-nvme.git/shortlog/refs/heads/kb-zns > I rebased my patches on top of Keith's fixups as well. And in compliance with the concensus, I removed persistence. https://irrelevant.dk/g/pci-nvme.git/log/?h=zns
On Sep 30 15:04, Keith Busch wrote: > After going through the zns enabling, I notice the controller enabling > is not correct. Then I just continued maked more stuff. The series, I > think, contains some of the less controversial patches from the two > conflicting zns series, preceeded by some cleanups and fixes from me. > > If this is all fine, I took the liberty of porting the zns enabling to > it and made a public branch for consideration here: > > http://git.infradead.org/qemu-nvme.git/shortlog/refs/heads/kb-zns > > Dmitry Fomichev (1): > hw/block/nvme: report actual LBA data shift in LBAF > > Keith Busch (5): > hw/block/nvme: remove pointless rw indirection > hw/block/nvme: fix log page offset check > hw/block/nvme: support per-namespace smart log > hw/block/nvme: validate command set selected > hw/block/nvme: support for admin-only command set > > Klaus Jensen (3): > hw/block/nvme: reject io commands if only admin command set selected > hw/block/nvme: add nsid to get/setfeat trace events > hw/block/nvme: add trace event for requests with non-zero status code > > hw/block/nvme-ns.c | 5 ++ > hw/block/nvme.c | 194 ++++++++++++++++++++---------------------- > hw/block/trace-events | 6 +- > include/block/nvme.h | 11 +++ > 4 files changed, 114 insertions(+), 102 deletions(-) > > -- > 2.24.1 > > These fixes all look good to me apart from the odd fixes that has been mentioned in the reviews. Since soft freeze is only two weeks away (Oct 27th), it would be nice to get this staged on nvme-next so we can get a pull sent off to Peter.
On Tue, Oct 13, 2020 at 11:04:01AM +0200, Klaus Jensen wrote: > On Sep 30 15:04, Keith Busch wrote: > > After going through the zns enabling, I notice the controller enabling > > is not correct. Then I just continued maked more stuff. The series, I > > think, contains some of the less controversial patches from the two > > conflicting zns series, preceeded by some cleanups and fixes from me. > > > > If this is all fine, I took the liberty of porting the zns enabling to > > it and made a public branch for consideration here: > > > > http://git.infradead.org/qemu-nvme.git/shortlog/refs/heads/kb-zns > > > > Dmitry Fomichev (1): > > hw/block/nvme: report actual LBA data shift in LBAF > > > > Keith Busch (5): > > hw/block/nvme: remove pointless rw indirection > > hw/block/nvme: fix log page offset check > > hw/block/nvme: support per-namespace smart log > > hw/block/nvme: validate command set selected > > hw/block/nvme: support for admin-only command set > > > > Klaus Jensen (3): > > hw/block/nvme: reject io commands if only admin command set selected > > hw/block/nvme: add nsid to get/setfeat trace events > > hw/block/nvme: add trace event for requests with non-zero status code > > > > hw/block/nvme-ns.c | 5 ++ > > hw/block/nvme.c | 194 ++++++++++++++++++++---------------------- > > hw/block/trace-events | 6 +- > > include/block/nvme.h | 11 +++ > > 4 files changed, 114 insertions(+), 102 deletions(-) > > > > -- > > 2.24.1 > > > > > > These fixes all look good to me apart from the odd fixes that has been > mentioned in the reviews. Since soft freeze is only two weeks away (Oct > 27th), it would be nice to get this staged on nvme-next so we can get a > pull sent off to Peter. I've fixed up the comments mentioned and added the received reviews. Since it was pretty trivial fixups and passes my basic santify tests, I went ahead and pushed to nvme-next.
On Oct 13 10:48, Keith Busch wrote: > On Tue, Oct 13, 2020 at 11:04:01AM +0200, Klaus Jensen wrote: > > On Sep 30 15:04, Keith Busch wrote: > > > After going through the zns enabling, I notice the controller enabling > > > is not correct. Then I just continued maked more stuff. The series, I > > > think, contains some of the less controversial patches from the two > > > conflicting zns series, preceeded by some cleanups and fixes from me. > > > > > > If this is all fine, I took the liberty of porting the zns enabling to > > > it and made a public branch for consideration here: > > > > > > http://git.infradead.org/qemu-nvme.git/shortlog/refs/heads/kb-zns > > > > > > Dmitry Fomichev (1): > > > hw/block/nvme: report actual LBA data shift in LBAF > > > > > > Keith Busch (5): > > > hw/block/nvme: remove pointless rw indirection > > > hw/block/nvme: fix log page offset check > > > hw/block/nvme: support per-namespace smart log > > > hw/block/nvme: validate command set selected > > > hw/block/nvme: support for admin-only command set > > > > > > Klaus Jensen (3): > > > hw/block/nvme: reject io commands if only admin command set selected > > > hw/block/nvme: add nsid to get/setfeat trace events > > > hw/block/nvme: add trace event for requests with non-zero status code > > > > > > hw/block/nvme-ns.c | 5 ++ > > > hw/block/nvme.c | 194 ++++++++++++++++++++---------------------- > > > hw/block/trace-events | 6 +- > > > include/block/nvme.h | 11 +++ > > > 4 files changed, 114 insertions(+), 102 deletions(-) > > > > > > -- > > > 2.24.1 > > > > > > > > > > These fixes all look good to me apart from the odd fixes that has been > > mentioned in the reviews. Since soft freeze is only two weeks away (Oct > > 27th), it would be nice to get this staged on nvme-next so we can get a > > pull sent off to Peter. > > I've fixed up the comments mentioned and added the received reviews. > Since it was pretty trivial fixups and passes my basic santify tests, I > went ahead and pushed to nvme-next. > Looks good, thanks!