On Wed, 24 Aug 2022 at 17:48, Quentin Monnet <quentin@xxxxxxxxxxxxx> wrote: > > On 24/08/2022 17:02, Jakub Wilk wrote: > > The latest bpf-helpers(7) man page update introduced a few misspellings: > > > >> -EFAULT on failrue to parse the existing header options. > > > > failrue -> failure > > > >> this helper enforces the key must be an task_struct > > > > an -> a > > > >> Returns the stored IMA hash of the inode (if it's avaialable) > > > > avaialable -> available > > > >> Providing an len_diff adjustment that is larger than the actual packet > >> size (resulting in negative packet size) will in principle not exceed > >> the MTU, why it is not considered a failure. > > > > an -> a > > > > But I can't parse the subordinate clause. I think it should be: > > > > ..., which is why it is not considered a failure. > > > >> Other BPF-helpers are needed for performing the planned size change, > >> why the responsability for catch a negative packet size belong in > >> those helpers. > > > > I can't parse this subordinate clause either. I guess it was meant to say: > > > > ..., which is why the responsibility for catching a negative packet size > > belongs in those helpers. > > > > Any volunteers to fix the misspellings on the kernel side? > > > > I'll have a look at these and send a fix. It's likely that some of those > are my mistakes anyway. Out of curiosity, did you spot these mistakes by reading the page, or did you run a tool on the document? Quentin
