Skip to content

rbd: have rbd_add_parse_args() only parse args

Alex Elder ceph-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest8 patches 14 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/8rbd: get rid of snap_name_lendrivers/block/rbd.c+0 -01 reply
2/8rbd: remove options args from rbd_add_parse_args()drivers/block/rbd.c+0 -01 reply
3/8rbd: remove snap_name arg from rbd_add_parse_args()drivers/block/rbd.c+0 -01 reply
4/8rbd: pass and populate rbd_options structuredrivers/block/rbd.c+0 -01 reply
5/8rbd: have rbd_add_parse_args() return errordrivers/block/rbd.c+0 -01 reply
6/8rbd: define image specification structuredrivers/block/rbd.c+23 -93 replies
7/8rbd: add reference counting to rbd_specdrivers/block/rbd.c+42 -103 replies
8/8rbd: fill rbd_spec in rbd_add_parse_args()drivers/block/rbd.c+0 -03 replies

Reviews trailers per patch

7 of 8 patches have a Reviewed-by or Acked-by.

PatchJosh Durgin
1/8R-b
2/8R-b
3/8R-b
4/8R-b
5/8R-b
6/8R-b
7/8R-b
8/8

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/8] rbd: have rbd_add_parse_args() only parse args

rbd_add_parse_args() parses options written to the "add" entry in /sys/bus/rbd/, but it does a mix of other things as well. This series rearranges things such that the only thing that function does is parse arguments, and encode the information found into some data structures that are used later for configuration and setting up new mapped rbd images.

-Alex

[PATCH 1/8] rbd: get rid of snap_name_len
[PATCH 2/8] rbd: remove options args from rbd_add_parse_args() [PATCH 3/8] rbd: remove snap_name arg from rbd_add_parse_args() [PATCH 4/8] rbd: pass and populate rbd_options structure [PATCH 5/8] rbd: have rbd_add_parse_args() return error [PATCH 6/8] rbd: define image specification structure [PATCH 7/8] rbd: add reference counting to rbd_spec [PATCH 8/8] rbd: fill rbd_spec in rbd_add_parse_args()