From: Jason Gunthorpe <jgg@ziepe.ca>
To: Dan Carpenter <dan.carpenter@oracle.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>,
syzbot <syzbot+015dd7cdbbbc2c180c65@syzkaller.appspotmail.com>,
akpm@linux-foundation.org, bp@alien8.de, daniel.vetter@ffwll.ch,
daniel.vetter@intel.com, hpa@zytor.com, jmattson@google.com,
jmorris@namei.org, joro@8bytes.org, kvm@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-media@vger.kernel.org,
linux-mm@kvack.org, linux-security-module@vger.kernel.org,
m.szyprowski@samsung.com, mchehab@kernel.org, mingo@redhat.com,
seanjc@google.com, serge@hallyn.com,
syzkaller-bugs@googlegroups.com, tfiga@chromium.org,
tglx@linutronix.de, vkuznets@redhat.com, wanpengli@tencent.com,
x86@kernel.org
Subject: Re: [syzbot] WARNING in unsafe_follow_pfn
Date: Thu, 1 Apr 2021 09:19:33 -0300 [thread overview]
Message-ID: <20210401121933.GA2710221@ziepe.ca> (raw)
In-Reply-To: <20210331042922.GE2065@kadam>
On Wed, Mar 31, 2021 at 07:29:22AM +0300, Dan Carpenter wrote:
> On Tue, Mar 30, 2021 at 07:04:30PM +0200, Paolo Bonzini wrote:
> > On 30/03/21 17:26, syzbot wrote:
> > > Hello,
> > >
> > > syzbot found the following issue on:
> > >
> > > HEAD commit: 93129492 Add linux-next specific files for 20210326
> > > git tree: linux-next
> > > console output: https://syzkaller.appspot.com/x/log.txt?x=169ab21ad00000
> > > kernel config: https://syzkaller.appspot.com/x/.config?x=6f2f73285ea94c45
> > > dashboard link: https://syzkaller.appspot.com/bug?extid=015dd7cdbbbc2c180c65
> > > syz repro: https://syzkaller.appspot.com/x/repro.syz?x=119b8d06d00000
> > > C reproducer: https://syzkaller.appspot.com/x/repro.c?x=112e978ad00000
> > >
> > > The issue was bisected to:
> > >
> > > commit d40b9fdee6dc819d8fc35f70c345cbe0394cde4c
> > > Author: Daniel Vetter <daniel.vetter@ffwll.ch>
> > > Date: Tue Mar 16 15:33:01 2021 +0000
> > >
> > > mm: Add unsafe_follow_pfn
> > >
> > > bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=122d2016d00000
> > > final oops: https://syzkaller.appspot.com/x/report.txt?x=112d2016d00000
> > > console output: https://syzkaller.appspot.com/x/log.txt?x=162d2016d00000
> > >
> > > IMPORTANT: if you fix the issue, please add the following tag to the commit:
> > > Reported-by: syzbot+015dd7cdbbbc2c180c65@syzkaller.appspotmail.com
> > > Fixes: d40b9fdee6dc ("mm: Add unsafe_follow_pfn")
> >
> > This is basically intentional because get_vaddr_frames is broken, isn't it?
> > I think it needs to be ignored in syzkaller.
>
> What?
>
> The bisect is wrong (because it's blaming the commit which added the
> warning instead of the commit which added the buggy caller) but the
> warning is correct.
>
> Plus users are going to be seeing this as well. According to the commit
> message for 69bacee7f9ad ("mm: Add unsafe_follow_pfn") "Unfortunately
> there's some users where this is not fixable (like v4l userptr of iomem
> mappings)". It sort of seems crazy to dump this giant splat and then
> tell users to ignore it forever because it can't be fixed... 0_0
I think the discussion conclusion was that this interface should not
be used by userspace anymore, it is obsolete by some new interface?
It should be protected by some kconfig and the kconfig should be
turned off for syzkaller runs.
Jason
next prev parent reply other threads:[~2021-04-01 17:42 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-30 15:26 [syzbot] WARNING in unsafe_follow_pfn syzbot
2021-03-30 17:04 ` Paolo Bonzini
2021-03-31 4:29 ` Dan Carpenter
2021-04-01 12:19 ` Jason Gunthorpe [this message]
2021-04-13 17:20 ` Dmitry Vyukov
2021-04-13 18:11 ` Jason Gunthorpe
2021-04-13 18:27 ` Dmitry Vyukov
2021-04-14 4:37 ` Dan Carpenter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210401121933.GA2710221@ziepe.ca \
--to=jgg@ziepe.ca \
--cc=akpm@linux-foundation.org \
--cc=bp@alien8.de \
--cc=dan.carpenter@oracle.com \
--cc=daniel.vetter@ffwll.ch \
--cc=daniel.vetter@intel.com \
--cc=hpa@zytor.com \
--cc=jmattson@google.com \
--cc=jmorris@namei.org \
--cc=joro@8bytes.org \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=linux-security-module@vger.kernel.org \
--cc=m.szyprowski@samsung.com \
--cc=mchehab@kernel.org \
--cc=mingo@redhat.com \
--cc=pbonzini@redhat.com \
--cc=seanjc@google.com \
--cc=serge@hallyn.com \
--cc=syzbot+015dd7cdbbbc2c180c65@syzkaller.appspotmail.com \
--cc=syzkaller-bugs@googlegroups.com \
--cc=tfiga@chromium.org \
--cc=tglx@linutronix.de \
--cc=vkuznets@redhat.com \
--cc=wanpengli@tencent.com \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).