mbox series

[0/4] cleanup for blk-wbt and blk-throttle

Message ID 20200508220015.11528-1-guoqing.jiang@cloud.ionos.com (mailing list archive)
Headers show
Series cleanup for blk-wbt and blk-throttle | expand

Message

Guoqing Jiang May 8, 2020, 10 p.m. UTC 
Hi,

Find some functions can be removed since there is no caller of them when
read the code.

Thanks,
Guoqing   

Guoqing Jiang (4):
  blk-throttle: remove blk_throtl_drain
  blk-throttle: remove tg_drain_bios
  blk-wbt: remove wbt_update_limits
  blk-wbt: rename __wbt_update_limits to wbt_update_limits

 block/blk-throttle.c | 63 --------------------------------------------
 block/blk-wbt.c      | 16 +++--------
 block/blk-wbt.h      |  4 ---
 block/blk.h          |  2 --
 4 files changed, 4 insertions(+), 81 deletions(-)

Comments

Guoqing Jiang May 29, 2020, 9:55 p.m. UTC | #1 
Gently ping.

On 5/9/20 12:00 AM, Guoqing Jiang wrote:
> Hi,
>
> Find some functions can be removed since there is no caller of them when
> read the code.
>
> Thanks,
> Guoqing
>
> Guoqing Jiang (4):
>    blk-throttle: remove blk_throtl_drain
>    blk-throttle: remove tg_drain_bios
>    blk-wbt: remove wbt_update_limits
>    blk-wbt: rename __wbt_update_limits to wbt_update_limits
>
>   block/blk-throttle.c | 63 --------------------------------------------
>   block/blk-wbt.c      | 16 +++--------
>   block/blk-wbt.h      |  4 ---
>   block/blk.h          |  2 --
>   4 files changed, 4 insertions(+), 81 deletions(-)
>
Jens Axboe May 29, 2020, 10:31 p.m. UTC | #2 
On 5/8/20 4:00 PM, Guoqing Jiang wrote:
> Hi,
> 
> Find some functions can be removed since there is no caller of them when
> read the code.
> 
> Thanks,
> Guoqing   
> 
> Guoqing Jiang (4):
>   blk-throttle: remove blk_throtl_drain
>   blk-throttle: remove tg_drain_bios
>   blk-wbt: remove wbt_update_limits
>   blk-wbt: rename __wbt_update_limits to wbt_update_limits
> 
>  block/blk-throttle.c | 63 --------------------------------------------
>  block/blk-wbt.c      | 16 +++--------
>  block/blk-wbt.h      |  4 ---
>  block/blk.h          |  2 --
>  4 files changed, 4 insertions(+), 81 deletions(-)

Thanks applied, missed this originally. Nice diffstat!