Re: [PATCH 00/47] RADOS Block Device: Fine-tuning for several function implementations

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> Date: Mon, 12 Sep 2016 20:35:20 +0200
> 
> Several update suggestions were taken into account
> from static source code analysis.
> 
> Markus Elfring (47):
>   Use kmalloc_array() in rbd_header_from_disk()
>   Less function calls in rbd_header_from_disk() after error detection
>   Adjust the position of a jump label in rbd_header_from_disk()
>   Refactor two calls for memory allocations in rbd_dev_image_id()
>   One function call less in rbd_dev_image_id() after error detection
>   Rename jump labels in rbd_add_parse_args()
>   Rename a jump label in rbd_dev_v2_snap_name()
>   Rename jump labels in rbd_dev_v2_snap_context()
>   Rename a jump label in rbd_spec_fill_names()
>   One function call less in rbd_dev_image_name() after error detection
>   Delete three unnecessary initialisations in rbd_dev_image_name()
>   One function call less in rbd_dev_v2_parent_info() after error detection
>   Delete an unnecessary initialisation in rbd_dev_v2_parent_info()
>   Rename a jump label in rbd_dev_v2_object_prefix()
>   Rename jump labels in rbd_dev_create()
>   Rename jump labels in rbd_dev_v1_header_info()
>   Rename jump labels in rbd_init_disk()
>   Fix jump targets in rbd_queue_workfn()
>   Rename a jump label in rbd_reregister_watch()
>   Rename a jump label in rbd_register_watch()
>   Rename jump labels in rbd_try_lock()
>   Rename a jump label in find_watcher()
>   Rename jump labels in get_lock_owner_info()
>   Rename jump labels in rbd_request_lock()
>   Fix jump targets in rbd_img_parent_read()
>   Rename a jump label in rbd_img_parent_read_callback()
>   Rename a jump label in rbd_img_request_submit()
>   Refactor a jump target in rbd_img_obj_exists_submit()
>   Delete an unnecessary initialisation in rbd_img_obj_exists_submit()
>   Refactor a jump target in rbd_img_obj_exists_callback()
>   Fix three jump targets in rbd_img_obj_parent_read_full()
>   Rename a jump label in rbd_img_obj_parent_read_full_callback()
>   Adjust the position of a jump label in rbd_img_request_fill()
>   Rename a jump label in rbd_img_obj_callback()
>   Rename jump labels in rbd_osd_req_create_copyup()
>   Rename jump labels in rbd_osd_req_create()
>   Rename a jump label in bio_chain_clone_range()
>   Rename jump labels in rbd_client_create()
>   Rename a jump label in rbd_ioctl_set_ro()
>   One function call less in rbd_dev_probe_parent() after error detection
>   Rename jump labels in rbd_dev_device_setup()
>   Rename jump labels in rbd_dev_image_probe()
>   Rename jump labels in do_rbd_add()
>   Delete an unnecessary initialisation in do_rbd_add()
>   Rename a jump label in rbd_slab_init()
>   Rename jump labels in rbd_init()
>   Delete unwanted spaces behind usages of the sizeof operator
> 
>  drivers/block/rbd.c | 544 ++++++++++++++++++++++++++--------------------------
>  1 file changed, 272 insertions(+), 272 deletions(-)

How can the clarification be continued for the shown change possibilities?
(Can a bit more development discussion become helpful?)
https://lkml.org/lkml/2016/9/12/932
https://lkml.kernel.org/r/<57806eb1-10ed-582e-72d0-2975e1ee967a@xxxxxxxxxxxxxxxxxxxxx>

https://patchwork.kernel.org/patch/9327545/

Regards,
Markus
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux