On Fri, Jan 21, 2022 at 9:42 AM Kenny Yu <kennyyu@xxxxxx> wrote: > > Thanks for the suggestions! I'll go with the all-or-nothing approach to > be consistent with `bpf_copy_from_user` and will make the following changes: > > * Return value: returns 0 on success, or negative error on failure. > * If we had a partial read, we will memset the read bytes to 0 and return > -EFAULT Not read bytes, but all bytes. copy_from_user zeros leftover for us, but access_process_vm doesn't do this on partial read.