On Wed, 2022-08-31 at 08:33 -0700, Alexei Starovoitov wrote:

On Wed, Aug 31, 2022 at 2:24 AM Roberto Sassu
[off-list ref] wrote:

quoted

quoted

quoted

quoted

+#define KEY_LOOKUP_CREATE        0x01
+#define KEY_LOOKUP_PARTIAL       0x02
+#define KEY_LOOKUP_FLAGS_ALL     (KEY_LOOKUP_CREATE |
KEY_LOOKUP_PARTIAL)

IMHO this could be just KEY_LOOKUP_ALL.

Since this is supposed to be kernel internal flags
please make them enum, so that bpf progs can auto-adjust
(with the help of CORE) to changes in this enum.
With #define there is no way for bpf prog to know
when #define changed in the future kernels.

Ok, will add in the next version.

Thanks

Roberto