On Tue, Mar 11, 2025 at 12:07 AM Song Liu <song@xxxxxxxxxx> wrote: > > + lskel = kfunc_call_test_lskel__open_and_load(); > > + if (!ASSERT_ERR_PTR(lskel, "lskel")) > > + goto close_prog; > > This goto is not necessary. But I don't think we need v8 just for this. I left goto as-is while applying. I felt all error handling being consistent this way is cleaner.