On 12/7/22 20:35, Christoph Hellwig wrote:
On Wed, Dec 07, 2022 at 12:29:56PM -0600, Mike Christie wrote:
I think you need to make this configurable.
If you use loop with pscsi, then the sgl that loop now gets might be too
big for the backend device so we now fail in:
pscsi_map_sg -> blk_rq_append_bio -> ll_back_merge_fn
So some users might be relying on the smaller limit.
Note that this could happen even now, you just need sufficiently
horrible hardware to pass through for it. But yes, for pscsi
this needs to look at the underlying device, and increasing the
limit might be a good point to do that. I'm not sure it's worth
to add user configuration, though.
Thanks for the feedback.
What should I do? Change pscsi to look at the underlying device limits?
Nikos