David Dillow wrote:
> On Tue, 2011-01-18 at 09:16 +0200, Or Gerlitz wrote:
>> David Dillow wrote:

> For SCSI commands, it is called in scsi_init_sgtable(), which is called
> by scsi_init_io(), called in turn by scsi_setup_blk_pc_cmnd() and
> scsi_setup_fs_cmnd(). Those are called by various function in the sd
> driver, notably sd_prep_fn() which is the block queue prep function
> called for each request

funny, if I understand this correct, assuming each scsi_host has a dma device
associated with it, sg_dma_map/unmap can be taken out from the SCSI LLD code
and be done in higher levels, just have someone scan everyone queuecommand and
command response code flows to clean that out, anyway

> Most likely, but there's no point in using an FMR for a individual S/G
> entry larger than the FMR size -- it's just extra work and consumes
> unneeded resources. That's not an issue in the current code, but is a
> small optimization in my new mapping code, which I hope to post later today.

I'm still trying to understand the bigger picture with your patch set and what
role the mlx4/mthca patch has in it, here you mention "small optimization" but
the srp patchset has up to 50% improvement in some flows, so what's the actual
part of this patch? e.g what patch/es in the srp patch series take advantage of
this fix?

Or.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to