Re: [PATCH v3] nvme/pci: remap BAR0 to cover admin CQ doorbell for large stride

2017-05-30 Thread Sagi Grimberg
Looks good to me, Reviewed-by: Sagi Grimberg

Re: [PATCH v3] nvme/pci: remap BAR0 to cover admin CQ doorbell for large stride

2017-05-24 Thread Christoph Hellwig
Looks good, Reviewed-by: Christoph Hellwig

[PATCH v3] nvme/pci: remap BAR0 to cover admin CQ doorbell for large stride

2017-05-24 Thread Xu Yu
The existing driver initially maps 8192 bytes of BAR0 which is intended to cover doorbells of admin SQ and CQ. However, if a large stride, e.g. 10, is used, the doorbell of admin CQ will be out of 8192 bytes. Consequently, a page fault will be raised when the admin CQ doorbell is accessed in nvme_c