The operation msk |= ~0ULL contains a redundant bit-wise or operation since all the bits are going to be set to 1, so replace this with an assignment since this is more optimal and probably clearer too. Signed-off-by: Colin Ian King <colin.i.king@xxxxxxxxx> --- kernel/bpf/inode.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/bpf/inode.c b/kernel/bpf/inode.c index 9aaf5124648b..fea07e12601f 100644 --- a/kernel/bpf/inode.c +++ b/kernel/bpf/inode.c @@ -914,7 +914,7 @@ static int bpf_parse_param(struct fs_context *fc, struct fs_parameter *param) str = param->string; while ((p = strsep(&str, ":"))) { if (strcmp(p, "any") == 0) { - msk |= ~0ULL; + msk = ~0ULL; } else if (find_btf_enum_const(info.btf, enum_t, enum_pfx, p, &val)) { msk |= 1ULL << val; } else { -- 2.39.5