Page MenuHomeFreeBSD
Differential D3470

Fix panic with Linux apps and RACCT/RCTL.
ClosedPublic
Actions

Authored by trasz on Aug 24 2015, 2:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 3 2024, 5:48 AM
Unknown Object (File)
Nov 3 2024, 5:37 AM
Unknown Object (File)
Nov 3 2024, 5:25 AM
Unknown Object (File)
Oct 5 2024, 8:50 AM
Unknown Object (File)
Oct 4 2024, 10:58 AM
Unknown Object (File)
Oct 2 2024, 5:49 AM
Unknown Object (File)
Sep 30 2024, 1:33 PM
Unknown Object (File)
Sep 8 2024, 1:11 PM
View All 89 Files
Subscribers
ed
emaste
imp
ler
ngie

Details

Reviewers
ngie
ed
Group Reviewers
Linux Emulation
Commits
rS287395: Fixes a panic triggered by threaded Linux applications when running
Summary

The patch below fixes a panic triggered by threaded Linux applications
when running with RACCT/RCTL enabled. The crash, as reported, looked
like this:

panic: racct_sub: freeing 1 of resource 11, which is more than allocated 0 for
wcgrid_fahv_vina_pr (pid 1140)

GNU gdb 6.1.1 [FreeBSD]
Copyright 2004 Free Software Foundation, Inc.
GDB is free software, covered by the GNU General Public License, and you are
welcome to change it and/or distribute copies of it under certain conditions.
Type "show copying" to see the conditions.
There is absolutely no warranty for GDB. Type "show warranty" for details.
This GDB was configured as "amd64-marcel-freebsd"...

Unread portion of the kernel message buffer:
panic: racct_sub: freeing 1 of resource 11, which is more than allocated 0 for
wcgrid_fahv_vina_pr (pid 1140)
cpuid = 3
KDB: stack backtrace:
db_trace_self_wrapper() at db_trace_self_wrapper+0x2b/frame 0xfffffe2eb3859920
vpanic() at vpanic+0x189/frame 0xfffffe2eb38599a0
kassert_panic() at kassert_panic+0x132/frame 0xfffffe2eb3859a10
racct_sub() at racct_sub+0x13e/frame 0xfffffe2eb3859a50
exit1() at exit1+0xd4/frame 0xfffffe2eb3859ad0
linux_exit_group() at linux_exit_group+0xd/frame 0xfffffe2eb3859ae0
ia32_syscall() at ia32_syscall+0x28b/frame 0xfffffe2eb3859bf0
Xint0x80_syscall() at Xint0x80_syscall+0x95/frame 0xfffffe2eb3859bf0

  • syscall (252, Linux ELF32, linux_exit_group), rip = 0x817a9d7, rsp =

0xffffca3c, rbp = 0xffffca58 ---

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

trasz updated this revision to Diff 8165.Aug 24 2015, 2:08 PM
trasz retitled this revision from to Fix panic with Linux apps and RACCT/RCTL..
trasz updated this object.
trasz edited the test plan for this revision. (Show Details)
Herald added a subscriber: imp. · View Herald TranscriptAug 24 2015, 2:08 PM
trasz added a reviewer: Linux Emulation.Aug 24 2015, 2:09 PM
emaste added a subscriber: emaste.Aug 25 2015, 10:37 AM
emaste added inline comments.
sys/compat/linux/linux_fork.c
301 ↗(On Diff #8165)

I'm curious why you don't do the racct_sub here, in this if. As a hint to someone who may modify this file in the future, adding another error return case?

trasz added inline comments.Aug 25 2015, 11:47 AM
sys/compat/linux/linux_fork.c
301 ↗(On Diff #8165)

Exactly. And also to avoid diverging from a very similar code in sys/kern/kern_thr.c.

ngie added a subscriber: ngie.Aug 29 2015, 7:17 PM

It seems like the relevant calls in kern_thr.c should at least be refactored into 2 helper functions and called from here and from there, and potentially applied to the cloudabi stuff ed's been working on.

ngie added a subscriber: ed.Aug 29 2015, 7:17 PM
ngie accepted this revision.Aug 29 2015, 7:20 PM
ngie added a reviewer: ngie.

As for right now though, this looks like a good solution to unbreak Larry though (it matches what's done in kern_thr.c:thread_create). I would commit it to unbreak LINUX + RACCT, then work on a proper solution.

I'd get signoff from someone else who did linux compat though, like dchagin.

This revision is now accepted and ready to land.Aug 29 2015, 7:20 PM
ed accepted this revision.Aug 29 2015, 9:53 PM
ed added a reviewer: ed.
In D3470#72335, @ngie wrote:

[...] and potentially applied to the cloudabi stuff ed's been working on.

On the CloudABI side it's all good. CloudABI's thread creation system call just calls into thread_create() to do all of its hard work. :-)

Closed by commit rS287395: Fixes a panic triggered by threaded Linux applications when running (authored by trasz). · Explain WhySep 2 2015, 2:04 PM
This revision was automatically updated to reflect the committed changes.
ler added a subscriber: ler.Dec 1 2015, 9:29 PM
This comment was removed by ler.

Revision Contents
Changeset List

PathSize
head/
sys/
compat/
linux/
22 lines

Diff 8420

View Options

head/sys/compat/linux/linux_fork.c

Loading...