Re: [PATCH 01/10] nvme.h: add RTD3R, RTD3E and OAES fields

2016-06-07 Thread Christoph Hellwig
On Tue, Jun 07, 2016 at 10:10:09AM +0200, Johannes Thumshirn wrote: > Just out of curiosity, why not CTRATT as well? ctratt is part of the fabrics spec and not the NVMe 1.2 spec, and we tried to keep those separate.

Re: [PATCH 01/10] nvme.h: add RTD3R, RTD3E and OAES fields

2016-06-07 Thread Johannes Thumshirn
On Tue, Jun 07, 2016 at 10:10:09AM +0200, Johannes Thumshirn wrote: > On Mon, Jun 06, 2016 at 11:20:43PM +0200, Christoph Hellwig wrote: > > These have been added in NVMe 1.2 and we'll need at least oaes for the > > NVMe target driver. > > > > Reviewed-by: Sagi Grimberg > > Reviewed-by: Jay Freye

Re: [PATCH 01/10] nvme.h: add RTD3R, RTD3E and OAES fields

2016-06-07 Thread Johannes Thumshirn
On Mon, Jun 06, 2016 at 11:20:43PM +0200, Christoph Hellwig wrote: > These have been added in NVMe 1.2 and we'll need at least oaes for the > NVMe target driver. > > Reviewed-by: Sagi Grimberg > Reviewed-by: Jay Freyensee > Signed-off-by: Christoph Hellwig > --- > include/linux/nvme.h | 5

[PATCH 01/10] nvme.h: add RTD3R, RTD3E and OAES fields

2016-06-06 Thread Christoph Hellwig
These have been added in NVMe 1.2 and we'll need at least oaes for the NVMe target driver. Reviewed-by: Sagi Grimberg Reviewed-by: Jay Freyensee Signed-off-by: Christoph Hellwig --- include/linux/nvme.h | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/include/linux/nvme.