Hans Petter Selasky wrote:
> On 8/11/18 9:44 AM, Roman Bogorodskiy wrote:
> >Hans Petter Selasky wrote:
> >
> >> On 08/06/18 21:43, Matthew Macy wrote:
> >>> The struct thread is typesafe. The problem is that the link is no longer
> >>> typesafe now that it’s not part of the thread. Thanks
On 8/11/18 9:44 AM, Roman Bogorodskiy wrote:
Hans Petter Selasky wrote:
On 08/06/18 21:43, Matthew Macy wrote:
The struct thread is typesafe. The problem is that the link is no longer
typesafe now that it’s not part of the thread. Thanks for pointing this
out. I’ll commit a fix later today.
Hans Petter Selasky wrote:
> On 08/06/18 21:43, Matthew Macy wrote:
> > The struct thread is typesafe. The problem is that the link is no longer
> > typesafe now that it’s not part of the thread. Thanks for pointing this
> > out. I’ll commit a fix later today.
> >
>
> Is there a patch yet?
>
On 08/06/18 21:43, Matthew Macy wrote:
The struct thread is typesafe. The problem is that the link is no longer
typesafe now that it’s not part of the thread. Thanks for pointing this
out. I’ll commit a fix later today.
Is there a patch yet?
--HPS
The struct thread is typesafe. The problem is that the link is no longer
typesafe now that it’s not part of the thread. Thanks for pointing this
out. I’ll commit a fix later today.
-M
On Mon, Aug 6, 2018 at 02:39 Hans Petter Selasky wrote:
> Hi Matthew,
>
> On 08/06/18 10:02, Hans Petter Sela
Hans Petter Selasky wrote:
> Hi Roman,
>
> Can you try the attached patch?
>
> --HPS
Thanks for the patch, works fine so far.
I'll give it more testing in the next few days.
Roman Bogorodskiy
signature.asc
Description: PGP signature
Hi Matthew,
On 08/06/18 10:02, Hans Petter Selasky wrote:
- if ((tdwait = TAILQ_FIRST(&record->er_tdlist)) != NULL &&
- TD_IS_RUNNING(tdwait->et_td)) {
At least the TD_IS_RUNNING() check is invalid. The "tdwait" structure is
in the control of the other CPU and
Hi Roman,
Can you try the attached patch?
--HPS
Index: sys/kern/subr_epoch.c
===
--- sys/kern/subr_epoch.c (revision 336962)
+++ sys/kern/subr_epoch.c (working copy)
@@ -232,33 +232,14 @@
struct epoch_thread *tdwait;
struct turn
Hi,
I think the problem is the thread pointed to by tdwait exited. I would
say it is not allowed to peek into the other records threads, because
they may change under the hood and are not protected by the current context.
if (record->er_cpuid != curcpu) {
This optimisation is inva
If you could give me a self-contained reproducer that would expedite a fix.
Thanks.
-M
On Sun, Aug 5, 2018 at 08:36 Roman Bogorodskiy wrote:
> Running -CURRENT r336863 on amd64. Get the following panic right after
> (or during) boot:
>
> Fatal trap 12: page fault while in kernel mode
> cpuid =
Running -CURRENT r336863 on amd64. Get the following panic right after
(or during) boot:
Fatal trap 12: page fault while in kernel mode
cpuid = 2; apic id = 04
fault virtual address = 0xdeadc2ff
fault code = supervisor read data, page not present
instruction pointer = 0x20:0xfff
11 matches
Mail list logo