On 9/11/2024 12:18 AM, Jens Axboe wrote: > On 9/10/24 9:01 AM, Kanchan Joshi wrote: >> +static inline bool rw_placement_hint_valid(u64 val) >> +{ >> + if (val <= MAX_PLACEMENT_HINT_VAL) >> + return true; >> + >> + return false; >> +} > Nit, why not just: > > static inline bool rw_placement_hint_valid(u64 val) > { > return val <= MAX_PLACEMENT_HINT_VAL; > } >
Right, concise. I can fold in both the changes in next respin. _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel