On Tue, Nov 07, 2023 at 09:08:05AM +0000, Alice Ryhl wrote: > It's a bit confusing that the pair of methods binder_alloc_free_buf and > binder_free_buf_locked are inconsistent in whether they user the alloc_ > prefix. It might be worth it to change them to be consistent? Right, the prefix is concistent across the APIs but not really for the local function names. I wouldn't mind adding the "alloc" part, it just seemed easier to fix the comment and backport that. -- Carlos Llamas