Messages in this thread | | | From | Ramji Jiyani <> | Date | Tue, 5 Oct 2021 12:46:36 -0700 | Subject | Re: [RESEND PATCH] aio: Add support for the POLLFREE |
| |
Hi Jeff:
On Tue, Oct 5, 2021 at 12:33 PM Jeff Moyer <jmoyer@redhat.com> wrote: > > Hi, Ramji, > > Thanks for the explanation of the use after free. I went ahead and > ran the patch through the libaio test suite and it passed. >
Thanks for taking time to test and providing feedback.
> > -#define POLLFREE (__force __poll_t)0x4000 /* currently only for epoll */ > > +#define POLLFREE ((__force __poll_t)0x4000) > > You added parenthesis, here, and I'm not sure if that's a necessary part > of this patch.
I added parenthesis to silence the checkpatch script. Should I just ignore it? I'll send v2 with the change, if it is required.
> > Other than that: > > Reviewed-by: Jeff Moyer <jmoyer@redhat.com> > Thanks, Ramji
| |