Re: [PATCHv3 0/3] change some function to be void function

From: zhong jiang
Date: Wed Aug 15 2018 - 09:07:03 EST


On 2018/8/15 20:53, David Sterba wrote:
> On Wed, Aug 08, 2018 at 09:47:52PM +0800, zhong jiang wrote:
>> v3->v2:
>> - Repost the patches with detailed changelog.
>> v1->v2:
>> - Fix v1 issue that forget to return after remove the variable "ret".
>>
>> zhong jiang (3):
>> fs/btrfs/disk-io: change btrfs_destroy_delayed_refs to be void
>> function
>> fs/btrfs/extent-tree: change btrfs_free_reserved_bytes to be void
>> funtion
>> fs/btrfs/tree-log: change btrfs_pin_log_trans to be void function
> The patches 2 and 3 can be applied, but please update che subject and
> put just "btrfs: " there, not the full file path. That we can find in
> the diff. The changelogs can be still improved to be clear about callers
> vs callees. Please update and resend.
>
> .
>
Thank you for review and suggestion. I will repost v3.

Sincerely,
zhong jiang