On Sun, Mar 3, 2019 at 6:36 PM Al Viro wrote:
>
> OK, having dug through the archives, the reasons were not strong.
> So that part is OK...
I've committed the patch.
However, I didn't actually do the separate and independent cleanups:
> > @@ -1060,6 +1071,8 @@ static inline void iocb_put(struc
On Sun, Mar 3, 2019 at 4:19 PM Al Viro wrote:
>
> On Sun, Mar 03, 2019 at 01:55:02PM +, Al Viro wrote:
>
> > Maybe unrelated to this bug, but... What's to prevent a wakeup
> > that happens just after we'd been added to a waitqueue by ->poll()
> > triggering aio_poll_wake(), which gets to aio_
On Sun, Mar 03, 2019 at 02:23:33PM -0800, Linus Torvalds wrote:
OK, having dug through the archives, the reasons were not strong.
So that part is OK...
> @@ -1060,6 +1071,8 @@ static inline void iocb_put(struct aio_kiocb *iocb)
> {
> if (refcount_read(&iocb->ki_refcnt) == 0 ||
>
On Sun, Mar 3, 2019 at 12:30 PM Al Viro wrote:
>
> On Sun, Mar 03, 2019 at 11:44:33AM -0800, Linus Torvalds wrote:
> >
> > I'm assuming you're talking about the second vfs_poll() in
> > aio_poll_complete_work()? The one we call before we check for
> > "rew->cancelled" properly under the spinlock?
On Sun, Mar 03, 2019 at 11:44:33AM -0800, Linus Torvalds wrote:
> On Sun, Mar 3, 2019 at 7:18 AM Al Viro wrote:
> >
> > > Maybe unrelated to this bug, but... What's to prevent a wakeup
> > > that happens just after we'd been added to a waitqueue by ->poll()
> > > triggering aio_poll_wake(), which
On Sun, Mar 3, 2019 at 11:44 AM Linus Torvalds
wrote:
>
> But doesn't it look nice to see
>
> 2 files changed, 41 insertions(+), 50 deletions(-)
>
> with actual code reduction, and a fundamental simplification in
> handling of the file pointer?
A coupl,e of the changes are "useless", and do the
On Sun, Mar 3, 2019 at 7:18 AM Al Viro wrote:
>
> > Maybe unrelated to this bug, but... What's to prevent a wakeup
> > that happens just after we'd been added to a waitqueue by ->poll()
> > triggering aio_poll_wake(), which gets to aio_poll_complete()
> > with its fput() *before* we'd reached the
On 03/03/2019 07:18 AM, Al Viro wrote:
> Fixes: bfe4037e722ec
> Cc: sta...@vger.kernel.org
> Signed-off-by: Al Viro
> ---
> diff --git a/fs/aio.c b/fs/aio.c
> index 3083180a54c8..7e88bfabdac2 100644
> --- a/fs/aio.c
> +++ b/fs/aio.c
> @@ -1767,6 +1767,7 @@ static ssize_t aio_poll(struct aio_ki
On Sun, Mar 03, 2019 at 01:55:02PM +, Al Viro wrote:
> Maybe unrelated to this bug, but... What's to prevent a wakeup
> that happens just after we'd been added to a waitqueue by ->poll()
> triggering aio_poll_wake(), which gets to aio_poll_complete()
> with its fput() *before* we'd reached th
On Sun, Mar 03, 2019 at 02:22:04AM -0800, syzbot wrote:
> Hello,
>
> syzbot found the following crash on:
>
> HEAD commit:7d762d69145a afs: Fix manually set volume location server ..
> git tree: upstream
> console output: https://syzkaller.appspot.com/x/log.txt?x=131d832ac0
> kernel
Hello,
syzbot found the following crash on:
HEAD commit:7d762d69145a afs: Fix manually set volume location server ..
git tree: upstream
console output: https://syzkaller.appspot.com/x/log.txt?x=131d832ac0
kernel config: https://syzkaller.appspot.com/x/.config?x=b76ec970784287c
das
11 matches
Mail list logo
