Re: [PATCH 1/4] fanotify: Shrink struct fanotify_event_metadata by 32 bits

From: Alexey Zaytsev
Date: Fri Nov 26 2010 - 02:01:37 EST


Hi, Eric. Sorry for the noise, but could you please comment at this
one? Because if it goes, it needs to go soon, before the .37 release,
or we won't be able change the ABI.

The other patches in the series are clearly not .37 material, so no
problem if you can't look at them right now.

On Mon, Nov 22, 2010 at 03:33, Alexey Zaytsev <alexey.zaytsev@xxxxxxxxx> wrote:
> And this saves additional 32 bits in the future
>
> Signed-off-by: Alexey Zaytsev <alexey.zaytsev@xxxxxxxxx>
> ---
> Âinclude/linux/fanotify.h | Â Â7 ++++---
> Â1 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/include/linux/fanotify.h b/include/linux/fanotify.h
> index 0f01214..9a7986f 100644
> --- a/include/linux/fanotify.h
> +++ b/include/linux/fanotify.h
> @@ -86,10 +86,11 @@
> Â#define FANOTIFY_METADATA_VERSION Â Â Â2
>
> Âstruct fanotify_event_metadata {
> - Â Â Â __u32 event_len;
> - Â Â Â __u32 vers;
> - Â Â Â __aligned_u64 mask;
> + Â Â Â __u16 event_len;
> + Â Â Â __u8 vers;
> + Â Â Â __u8 reserved;
> Â Â Â Â__s32 fd;
> + Â Â Â __aligned_u64 mask;
> Â Â Â Â__s32 pid;
> Â};
>
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/