r363881 mjg Aug. 5, 2020, 7:28 a.m.
r363880 lwhsu Aug. 5, 2020, 5:58 a.m.
Sponsored by:	The FreeBSD Foundation
ViewVC
r363875 mjg Aug. 4, 2020, 11:09 p.m.
This restores the pre-adaptive spinning state for SU which livelocks
otherwise. Note this is a bug in SU.

Reported by:	pho
ViewVC
r363874 mjg Aug. 4, 2020, 11:07 p.m.
Tested by:	pho
ViewVC
r363873 mjg Aug. 4, 2020, 11:07 p.m.
This allows making half-constructed entries visible to the lockless lookup,
which now can check for either "not yet fully constructed" and "no longer valid"
state.

This will be used for .. lookup.
ViewVC
r363872 mjg Aug. 4, 2020, 11:04 p.m.
cache_purge locklessly checks whether the vnode at hand has any namecache
entries. This can race with a concurrent purge which managed to remove
the last entry, but may not be done touching the vnode.

Make sure we observe the relevant vnode lock as not taken before proceeding
with vgone.

Paired with the fact that doomed vnodes cannnot receive entries this restores
the invariant that there are no namecache-related writing users past cache_purge
in vgone.

Reported by:	pho
ViewVC
r363871 mjg Aug. 4, 2020, 11 p.m.
r363870 kibab Aug. 4, 2020, 9:58 p.m.
* Downgrade some CAM debug messages from _INFO to _DEBUG level;
 * Add KASSERT for the case when we suspect incorrect CAM SIM initialization (using cam_sim_alloc() instead of cam_sim_alloc_dev());
 * Use waiting version of xpt_alloc_ccb(), we are not in hurry;
 * With the waiting version we cannot get NULL return, so remove the NULL check;
 * In some csses, the name of mmcprobe_done has been written as mmc_probedone();
 * Send AC_LOST_DEVICE if we, well, lost the device;
 * Misc style(9) fixes.

Reviewed by:	manu
Approved by:	imp (mentor)
Differential Revision:	https://reviews.freebsd.org/D25843
ViewVC
r363869 kevans Aug. 4, 2020, 9:49 p.m.
There were two separate issues here:

1.) #if/#else wasn't taken into account at all for maxsyscall figures, but
2.) We didn't validate contiguous syscall numbers anyways...

This kind of inconsistency is bad as we don't currently ensure explicit
indexing of, e.g., the sysent array if one syscall is unimplemented/missing.
This could be fixed and might be more robust, but it's also good to have the
"documentation" that comes from being explicit as to what the missing
syscalls are.

The new version looks much like the awk version; stash off the current
'last highest syscall seen' if we hit an #if, restore to that if we hit an
#else, and make sure that we're explicitly always defining the next syscall.

The logic at the tail end of process_syscall_def that moves maxsyscall has
been 'cleaned up' a little since we're now ensuring that it's monotonically
increasing earlier in the function. At the moment I think it's unlikely we'd
see range-definitions that are not UNIMPL, but there's no reason to
specifically handle that case for bumping maxsyscall there.

This change was provoked by reading the commit message for r363832 and
realizing that this validation hadn't been included in the initial rewrite
to lua.

Reviewed by:	brooks
Differential Revision:	https://reviews.freebsd.org/D25945
ViewVC
r363868 olivier Aug. 4, 2020, 9:34 p.m.
Approved by:	kp
Sponsored by:	Netflix
Differential Revision:	https://reviews.freebsd.org/D25935
ViewVC
r363867 vangyzen Aug. 4, 2020, 9:09 p.m.
Add man page links for all functions in devinfo(3).

Reported by:	vim
MFC after:	2 weeks
Sponsored by:	Dell EMC Isilon
ViewVC
r363866 vangyzen Aug. 4, 2020, 9:05 p.m.
Refactor to create devinfo_free_dev().  Call it to plug a memory leak
on two error paths in devinfo_init_devices().

Reported by:	Coverity
MFC after:	2 weeks
Sponsored by:	Dell EMC Isilon
ViewVC
r363865 br Aug. 4, 2020, 8:54 p.m.
Sponsored by:	DARPA, AFRL
ViewVC
r363864 br Aug. 4, 2020, 8:51 p.m.
and IOMMU unit, domain, ctx.

Reviewed by:	kib
Sponsored by:	DARPA, AFRL
Differential Revision:	https://reviews.freebsd.org/D25926
ViewVC
r363863 mjg Aug. 4, 2020, 8:31 p.m.
Reported by:	kib
ViewVC