Re: [PATCH v4 1/3] fanotify: add support for a variable length permission event

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jul 23, 2025 at 9:43 AM Jan Kara <jack@xxxxxxx> wrote:
>
> On Tue 22-07-25 17:13:07, Amir Goldstein wrote:
> > On Tue, Jul 22, 2025 at 4:01 PM Jan Kara <jack@xxxxxxx> wrote:
> > > Sorry for a bit delayed reply I was busy with other work...
> > >
> > > On Fri 11-07-25 11:30:59, Ibrahim Jirdeh wrote:
> > > > From: Amir Goldstein <amir73il@xxxxxxxxx>
> > > >
> > > > In preparation for pre-content events that report fid info + name,
> > > > we need a new event type that is both variable length and can be
> > > > put on a user response wait list.
> > > >
> > > > Create an event type FANOTIFY_EVENT_TYPE_FID_NAME_PERM with is a
> > > > combination of the variable length fanotify_name_event prefixed
> > > > with a fix length fanotify_perm_event and they share the common
> > > > fanotify_event memeber.
> > > >
> > > > Signed-off-by: Amir Goldstein <amir73il@xxxxxxxxx>
> > >
> > > As a procedural note, this patch should have your Signed-off-by as well
> > > Ibrahim, when you resend it as part of your patch set.
> >
> > Right, but I don't think Ibrahim needs those patches anymore,
> > because as you said FAN_RESTARTABLE_EVENTS do not require
> > a response id.
>
> Right, I've lost track a bit of who needs what :). So nobody is currently
> striving to get the response ID changes merged? Just that I don't waste
> time reviewing changes nobody is interested in at this point...

Yeh, it's hard to keep track, but IIUC, no need for respose_id patch
or my prep patch for FAN_CLASS_PRE_CONTENT_FID for now.
If you have time, it would be great if you could take a high-level peek at
https://github.com/amir73il/linux/commits/fan_pre_dir_access/
but as I wrote, I won't be getting back to that before end of August and
when I do, it is likely that I will pick up Ibrahim's response_id patch
and make those events fd-less.

Thanks,
Amir.





[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [NTFS 3]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [NTFS 3]     [Samba]     [Device Mapper]     [CEPH Development]

  Powered by Linux