Page MenuHomeFreeBSD
Differential D43570

Annotate 'rebooting' with __read_mostly
ClosedPublic
Actions

Authored by olce on Jan 24 2024, 1:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 8, 7:28 PM
Unknown Object (File)
Mar 10 2025, 2:28 PM
Unknown Object (File)
Feb 22 2025, 2:14 PM
Unknown Object (File)
Feb 22 2025, 1:30 PM
Unknown Object (File)
Feb 22 2025, 1:29 PM
Unknown Object (File)
Feb 15 2025, 4:54 PM
Unknown Object (File)
Feb 13 2025, 4:32 PM
Unknown Object (File)
Feb 13 2025, 1:10 PM
View All 32 Files
Subscribers
imp

Details

Reviewers
kib
emaste
mjg
markj
Commits
rG2c8244485795: Annotate 'rebooting' with __read_mostly
rG261f70ffc8a8: Annotate 'rebooting' with __read_mostly
rG12d6a032dff1: Annotate 'rebooting' with __read_mostly
Summary

While here, put such annotation after the variable for 'dumping', since
it concerns the variable and not the type.

MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
sys/
kern/
4 lines

Diff 133426

View Options

sys/kern/kern_shutdown.c

Show First 20 Lines • Show All 4,327 Lines • ▼ Show 20 Lines for dir in $$(${ALL-DEPENDS-LIST}); do \
_p=$$(cd $$dir; ${MAKE} -VPKGNAME); \ _p=$$(cd $$dir; ${MAKE} -VPKGNAME); \
if ! $$(${ECHO_CMD} $${_packages} | ${GREP} -q $${_p}); then \ if ! $$(${ECHO_CMD} $${_packages} | ${GREP} -q $${_p}); then \
${ECHO_CMD} $${_p}; \ ${ECHO_CMD} $${_p}; \
fi; \ fi; \
done done
# Install missing dependencies from package # Install missing dependencies from package
install-missing-packages: install-missing-packages:
@_dirs=$$(${MISSING-DEPENDS-LIST}); \ @_dirs=$$(${MISSING-DEPENDS-LIST}); \
if [ -n "$${_dirs}" ]; then \ ${ECHO_CMD} "$${_dirs}" | ${SED} "s%${PORTSDIR}/%%g" | ${SU_CMD} "${XARGS} -o ${PKG_BIN} install -A"
matUnsubmitted
Done
Inline Actions

I would use | ${XARGS} which will take care of that.

mat: I would use `| ${XARGS}` which will take care of that.
0mpAuthorUnsubmitted
Done
Inline Actions

Hmm, sure, although I think that tr is more explicit.

0mp: Hmm, sure, although I think that tr is more explicit.
tobikUnsubmitted
Done
Inline Actions

This should be

${SU_CMD} "${ECHO_CMD} \"$${_dirs}\" | ${SED} 's%${PORTSDIR}/%%g' | ${XARGS} ${PKG_BIN} install -A"

or even better push the su outwards as much as possible

${ECHO_CMD} "$${_dirs}" | ${SED} "s%${PORTSDIR}/%%g" | ${SU_CMD} "${XARGS} ${PKG_BIN} install -A"

tobik: This should be `${SU_CMD} "${ECHO_CMD} \"$${_dirs}\" | ${SED} 's%${PORTSDIR}/%%g' | ${XARGS}…
0mpAuthorUnsubmitted
Done
Inline Actions

We've tried that approach, @tobik.

The problem is that then you cannot have interactive prompts from pkg (stdin is taken by xargs).

0mp: We've tried that approach, @tobik. The problem is that then you cannot have interactive…
tobikUnsubmitted
Done
Inline Actions

I see. Luckily there seems to be an xargs(1) flag for exactly that, -o:

${ECHO_CMD} "$${_dirs}" | ${SED} "s%${PORTSDIR}/%%g" | ${SU_CMD} "${XARGS} -o ${PKG_BIN} install -A"
tobik: I see. Luckily there seems to be an xargs(1) flag for exactly that, `-o`: ``` ${ECHO_CMD}…
0mpAuthorUnsubmitted
Done
Inline Actions

Ha! I didn't know about this one! Thanks.

0mp: Ha! I didn't know about this one! Thanks.
${SU_CMD} "${PKG_BIN} install -A $$(${ECHO_CMD} "$${_dirs}" | ${SED} "s%${PORTSDIR}/%%g")"; \
fi
################################################################ ################################################################
# Everything after here are internal targets and really # Everything after here are internal targets and really
# shouldn't be touched by anybody but the release engineers. # shouldn't be touched by anybody but the release engineers.
################################################################ ################################################################
# This target generates an index entry suitable for aggregation into # This target generates an index entry suitable for aggregation into
# a large index. Format is: # a large index. Format is:
▲ Show 20 Lines • Show All 1,058 Lines • Show Last 20 Lines