Messages in this thread | | | From | Song Liu <> | Date | Tue, 2 Jan 2024 09:25:24 -0800 | Subject | Re: [PATCH 0/5] bpf: Adjustments for four function implementations |
| |
On Mon, Jan 1, 2024 at 1:10 AM Markus Elfring <Markus.Elfring@web.de> wrote: > > >> A few update suggestions were taken into account > >> from static source code analysis. > > > > Auto Nack. > > Pls don't send such patches. You were told multiple > > times that such kfree usage is fine. > > Some implementation details are improvable. > Can you find an update step (like the following) helpful? > > [PATCH 2/5] bpf: Move an assignment for the variable “st_map” in bpf_struct_ops_link_create() > https://lore.kernel.org/bpf/ed2f5323-390f-4c9d-919d-df43ba1cad2b@web.de/
This change is not helpful at all. The use of "st_map" in current code as-is doesn't cause any confusion, i.e., it is always struct bpf_struct_ops_map *. OTOH, this patch will make it harder for folks who use git-blame. Therefore, it adds negative value to the code base.
Thanks, Song
| |