Skip to content

convert st and sg to always use scatterlists take 2

Mike Christie linux-scsi 1 version In review

Get series with b4
b4 am 1126845571.9276.40.camel@max

Versions newest first

  1. v1 Latest5 patches 32 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5export block layer functions and add a bio helperdrivers/block/ll_rw_blk.c fs/bio.c include/linux/bio.h +1 more+55 -10 replies
2/5add scsi helpers to send async requests and map pagesdrivers/scsi/scsi_error.c drivers/scsi/scsi_lib.c drivers/scsi/scsi_priv.h +1 more+164 -730 replies
3/5make sg always send scatterlists take 2drivers/scsi/sg.c+255 -4133 replies
4/5convert st to use scsi_execte_asyncdrivers/scsi/st.c drivers/scsi/st.h+116 -14126 replies
5/5kill scsi_wait_reqdrivers/scsi/cpqfcTSinit.c drivers/scsi/scsi_lib.c include/scsi/scsi_request.h+4 -522 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/5] convert st and sg to always use scatterlists take 2

The following patches are a second (actually third if you count when I tried to fixup all their duplicated code but broke a lot of features the first time) try at converting sg and st to not use scsi_requests and always use scatterlists.

It is not the grand cleanups I had hoped to do. It does allow us to finish converting scsi-ml and LLDs without breaking anything though. It is also a nice bit of cleanup for sg. I do hope to still move the the ULDs to use the block layer blk_rq_map_user* functions for DIO and copy_to/from_user cases so we can kill a lot of their code. And we should be able to consolidate their buffer allocation code soon.

If these patches are ok, we just have to convert gdth, osst and cpqfc.