All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/2] Use REQ_OP_WRITE_SAME to implement WRITE SAME
@ 2019-07-02 15:30 Bart Van Assche
  2019-07-02 16:26 ` Christoph Hellwig
  2019-07-02 17:15 ` Bart Van Assche
  0 siblings, 2 replies; 3+ messages in thread
From: Bart Van Assche @ 2019-07-02 15:30 UTC (permalink / raw)
  To: target-devel

Hi Martin,

The SCSI target iblock backend uses regular writes to implement the SCSI WRITE
SAME command. That is suboptimal for at least SCSI and NVMe SSDs. This patch
series makes the iblock backend use REQ_OP_WRITE_SAME instead. Please consider
these patches for inclusion in the upstream kernel.

Thanks,

Bart.

Bart Van Assche (2):
  block: Add blkdev_submit_write_same()
  target/core: Use REQ_OP_WRITE_SAME to implement WRITE SAME

 block/blk-lib.c                     | 37 ++++++++++++++++++++++++++++-
 drivers/target/target_core_iblock.c | 17 +++++++++++++
 include/linux/blkdev.h              |  3 +++
 3 files changed, 56 insertions(+), 1 deletion(-)

-- 
2.22.0.410.gd8fdbe21b5-goog

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 0/2] Use REQ_OP_WRITE_SAME to implement WRITE SAME
  2019-07-02 15:30 [PATCH 0/2] Use REQ_OP_WRITE_SAME to implement WRITE SAME Bart Van Assche
@ 2019-07-02 16:26 ` Christoph Hellwig
  2019-07-02 17:15 ` Bart Van Assche
  1 sibling, 0 replies; 3+ messages in thread
From: Christoph Hellwig @ 2019-07-02 16:26 UTC (permalink / raw)
  To: target-devel

I actually rather wanted to get rid of REQ_OP_WRITE_SAME entirely.
Do you have a use case for this?

Note that it isn't going to help with nvme either, as there is no
write same command in nvme.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 0/2] Use REQ_OP_WRITE_SAME to implement WRITE SAME
  2019-07-02 15:30 [PATCH 0/2] Use REQ_OP_WRITE_SAME to implement WRITE SAME Bart Van Assche
  2019-07-02 16:26 ` Christoph Hellwig
@ 2019-07-02 17:15 ` Bart Van Assche
  1 sibling, 0 replies; 3+ messages in thread
From: Bart Van Assche @ 2019-07-02 17:15 UTC (permalink / raw)
  To: target-devel

On 7/2/19 9:26 AM, Christoph Hellwig wrote:
> I actually rather wanted to get rid of REQ_OP_WRITE_SAME entirely.
> Do you have a use case for this?

Hi Christoph,

I don't have a use case for this myself. I posted these patches because 
Roman Bolshakov reported a bug in the SCSI target WRITE SAME 
implementation. I think that means that he uses this functionality.

Bart.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-07-02 17:15 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-07-02 15:30 [PATCH 0/2] Use REQ_OP_WRITE_SAME to implement WRITE SAME Bart Van Assche
2019-07-02 16:26 ` Christoph Hellwig
2019-07-02 17:15 ` Bart Van Assche

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.