httpd: remove redundant search and check return value
The server foreach-loop serves the same purpose as "serverconfig_byid"
did previously.
OK kirill@
httpd: do NOT send location fcgiparams twice
Only emit the params when the current receiver is the server child.
When sending a location-config from the parent, the FCGI params IMSG
was emitted once per receiver iteration and always targeted the server
child. With both the server and the logger process needing the server
config, the server child received IMSG_CFG_FCGI twice per location and
appended the same params on top of the existing list. The FastCGI backends
then saw every param duplicated.
Input Fabien Romano, OK kirill@
sys/powerpc64: encode PTCR PATS correctly
The PTCR PATS field stores log2(partition table size) - 12. Since
PATMEMSZ is a power of two, ffs(PATMEMSZ) is log2(PATMEMSZ) + 1; the
old expression wrote 5 for the 64 KiB table, describing 128 KiB to
Book3S v3 hardware.
Use ffs(PATMEMSZ) - 1 - 12 to express the intended log2 calculation
and encode the 64 KiB partition table as PATS == 4.
Discussed with kettenis@
Tested and OK: gkoehler@
properly handle qwx VIF DMA map allocation failures at attach time
qwx_attach() runs as a config_mountroot hook, which means errors returned
from this function will be ignored. Since returning errors is pointless,
change this function's return value type to void.
There is a theoretical risk that allocating the virtual interface's DMA
maps will fail. Allocate these maps earlier and avoid attaching the driver
if the maps cannot be allocated.
replace qwx virtual interface list with a single interface structure
Fixes use of stale pointers on the vif list which caused crashes.
We do not support multiple interfaces per driver in our net80211 stack,
so this list inherited from Linux was pointless for us anyway.
Tested by ajacoutot, phessler, kettenis, gnezdo, krw, and myself.
ok phessler@ kettenis@ gnezdo@