From: "Theodore Ts'o" <tytso@mit.edu>
To: Dmitry Vyukov <dvyukov@google.com>
Cc: Tudor Ambarus <tudor.ambarus@linaro.org>,
Herbert Xu <herbert@gondor.apana.org.au>,
syzbot <syzbot+726dc8c62c3536431ceb@syzkaller.appspotmail.com>,
davem@davemloft.net, linux-crypto@vger.kernel.org,
linux-kernel@vger.kernel.org, olivia@selenic.com,
syzkaller-bugs@googlegroups.com, Jason Wang <jasowang@redhat.com>,
"Michael S. Tsirkin" <mst@redhat.com>,
Laurent Vivier <lvivier@redhat.com>,
Rusty Russell <rusty@rustcorp.com.au>,
Aleksandr Nogikh <nogikh@google.com>,
syzkaller <syzkaller@googlegroups.com>
Subject: Re: [PATCH] hwrng: virtio - Fix race on data_avail and actual data
Date: Mon, 8 May 2023 04:55:49 -0400 [thread overview]
Message-ID: <ZFi5FfjVpxLEk48A@mit.edu> (raw)
In-Reply-To: <CACT4Y+ZnTRf5BocMZZCkUva+VddOMXYGu13iWo6+3sopZzh5hQ@mail.gmail.com>
On Mon, May 08, 2023 at 07:33:39AM +0200, Dmitry Vyukov wrote:
> A link like this may work for syzbot instead of the Reported-by tag
> (may work out of the box, but need to double check if we start to use
> this):
>
> Link: https://syzkaller.appspot.com/bug?extid=726dc8c62c3536431ceb
>
> Or similarly this may work:
>
> Reported-by: https://syzkaller.appspot.com/bug?extid=726dc8c62c3536431ceb
> I think the parsing code mostly looks for the hash.
>
> This was proposed, but people said that they need links to lore and
> don't want links to syzkaller dashboard. So this was rejected at the
> time.
I think the "Reported-by: " line should continue to contain an e-mail,
since that way "git send-email" will automatically include a Cc: to
the mailing list address so that the syzbot page for the report will
contain a link to the page.
What *would* be useful would be a search box on the top-level
https://syzkaller.appspot.com where you could either enter an e-mail
address like:
syzbot+726dc8c62c3536431ceb@syzkaller.appspotmail.com
or the syzbot report title e.g.:
KCSAN: data-race in random_recv_done / virtio_read (3)
or just a function name:
sys_quotactl_fd
The search box could just push the text to google.com with
"site:syzkaller.appspot.com", which should mostly do the right thing.
Also, it would also be nice if all of the URL links on the
syzkaller.appspot.com used the id form of the URL. That is, to use
https://syzkaller.appspot.com/bug?extid=6c73bd34311ee489dbf5
instead of:
https://syzkaller.appspot.com/bug?id=32c54626e170a6b327ca2c8ae4c1aea666a8c20b
The extid form of the URL is shorter, and having a consistency so that
the primary URL is the extid would reduce confusion. The web site
will need to continue to support the id form of the URL since there
are quite a few of those URL's in mailing list archives and git commit
descriptions.
It also would be useful if there was a way to translate from the extid
hash to the id hash, so that it's possible to search for the extid and
id forms of the URL --- since the URL aliasing means that for a
developer trying to do code archeology and web searches, that we need
to search for both URL forms for past syzbot reports. (But if we can
avoid the aliasing confusion moving forward, that would be **really**
nice.)
Cheers,
- Ted
next prev parent reply other threads:[~2023-05-08 9:06 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-04-21 14:35 [syzbot] [crypto?] KCSAN: data-race in random_recv_done / virtio_read (3) syzbot
2023-04-21 14:52 ` Dmitry Vyukov
2023-05-03 10:22 ` Herbert Xu
2023-05-03 10:54 ` [PATCH] hwrng: virtio - Fix race on data_avail and actual data Herbert Xu
2023-05-03 11:19 ` Tudor Ambarus
2023-05-04 3:55 ` Herbert Xu
2023-05-04 8:10 ` Tudor Ambarus
2023-05-05 4:01 ` Theodore Ts'o
2023-05-08 5:33 ` Dmitry Vyukov
2023-05-08 8:55 ` Theodore Ts'o [this message]
2023-05-11 15:11 ` Aleksandr Nogikh
2023-05-03 11:37 ` Michael S. Tsirkin
2023-05-04 3:59 ` [v2 PATCH] " Herbert Xu
2023-05-04 5:28 ` Michael S. Tsirkin
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=ZFi5FfjVpxLEk48A@mit.edu \
--to=tytso@mit.edu \
--cc=davem@davemloft.net \
--cc=dvyukov@google.com \
--cc=herbert@gondor.apana.org.au \
--cc=jasowang@redhat.com \
--cc=linux-crypto@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lvivier@redhat.com \
--cc=mst@redhat.com \
--cc=nogikh@google.com \
--cc=olivia@selenic.com \
--cc=rusty@rustcorp.com.au \
--cc=syzbot+726dc8c62c3536431ceb@syzkaller.appspotmail.com \
--cc=syzkaller-bugs@googlegroups.com \
--cc=syzkaller@googlegroups.com \
--cc=tudor.ambarus@linaro.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.