Page MenuHomeFreeBSD
Differential D29108

linux: support AT_EMPTY_PATH flag in fchownat(2)
ClosedPublic
Actions

Authored by trasz on Mar 6 2021, 8:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 1, 11:36 PM
Unknown Object (File)
Thu, Mar 13, 7:31 PM
Unknown Object (File)
Feb 24 2025, 5:35 AM
Unknown Object (File)
Feb 21 2025, 12:38 PM
Unknown Object (File)
Feb 21 2025, 10:45 AM
Unknown Object (File)
Jan 26 2025, 12:24 PM
Unknown Object (File)
Jan 13 2025, 8:20 PM
Unknown Object (File)
Jan 1 2025, 2:34 AM
View All 82 Files
Subscribers
imp

Details

Reviewers
kib
Group Reviewers
Linux Emulation
Commits
rG1e44a4ea0d4a: linux: support AT_EMPTY_PATH flag in fchownat(2)
rGe47823b8312b: linux: support AT_EMPTY_PATH flag in fchownat(2)
Summary

This fixes rsyslog package installation scripts in Bionic.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 37649
Build 34538: arc lint + arc unit

Event Timeline

trasz created this revision.Mar 6 2021, 8:04 PM
Herald added a subscriber: imp. · View Herald TranscriptMar 6 2021, 8:04 PM
trasz requested review of this revision.Mar 6 2021, 8:04 PM
Harbormaster completed remote builds in B37642: Diff 85280.Mar 6 2021, 8:05 PM
trasz added reviewers: Linux Emulation, kib.Mar 6 2021, 8:06 PM
kib added a comment.Mar 6 2021, 8:32 PM

I think the flag should be implemented on the FreeBSD side.

Also, your patch does not handle AT_FDCWD case, that is described in the Linux man page.

sys/compat/linux/linux_file.c
1741–1742

You broke detection of the unknown flags.

trasz updated this revision to Diff 85291.Mar 7 2021, 11:59 AM

Fix unknown flag detection, detect AT_FDCWD.

Harbormaster completed remote builds in B37649: Diff 85291.Mar 7 2021, 12:00 PM
trasz added a comment.EditedMar 7 2021, 12:02 PM

I've considered doing it natively first - that would allow us to support AT_FDCWD with AT_EMPTY_PATH - but after a few hours I gave up. I'd strongly prefer committing this small change instead of making another attempt at understanding namei().

trasz updated this revision to Diff 85292.Mar 7 2021, 12:04 PM

Fix warning message.

Harbormaster completed remote builds in B37650: Diff 85292.Mar 7 2021, 12:04 PM
trasz added a comment.Mar 8 2021, 3:26 PM

(Tinderboxed.)

trasz updated this revision to Diff 87480.Apr 15 2021, 12:27 PM

Reimplement.

Harbormaster completed remote builds in B38589: Diff 87480.Apr 15 2021, 12:27 PM
kib accepted this revision.Apr 15 2021, 12:46 PM

BTW there are more Linux syscalls taking AT_EMPTY_PATH

sys/compat/linux/linux_file.c
1760

Extra ()

This revision is now accepted and ready to land.Apr 15 2021, 12:46 PM
trasz added inline comments.Apr 15 2021, 12:51 PM
sys/compat/linux/linux_file.c
1760

This is copy/pasted all over the tree; I'll fix it in a later sweep.

trasz added a comment.Apr 16 2021, 9:55 AM

(Tinderboxed.)

Closed by commit rGe47823b8312b: linux: support AT_EMPTY_PATH flag in fchownat(2) (authored by trasz). · Explain WhyApr 16 2021, 3:29 PM
This revision was automatically updated to reflect the committed changes.
trasz added a commit: rGe47823b8312b: linux: support AT_EMPTY_PATH flag in fchownat(2).
trasz added a commit: rG1e44a4ea0d4a: linux: support AT_EMPTY_PATH flag in fchownat(2).Feb 17 2022, 10:56 AM

Revision Contents
Changeset List

PathSize
sys/
compat/
linux/
12 lines
kern/
15 lines
sys/
1 line

Diff 85291

View Options

sys/compat/linux/linux_file.c

  • This file was added.
PropertyOld ValueNew Value
svn:eol-stylenullnative
\ No newline at end of property
svn:keywordsnullFreeBSD=%H
\ No newline at end of property
svn:mime-typenulltext/plain
\ No newline at end of property
# $FreeBSD$
PORTNAME= mping
DISTVERSION= 0.1.3
CATEGORIES= sysutils
MAINTAINER= dteske@FreeBSD.org
COMMENT= Ping multiple hosts
LICENSE= BSD2CLAUSE
LICENSE_FILE= ${WRKSRC}/LICENSE
USE_GITHUB= yes
GH_ACCOUNT= FrauBSD
NO_ARCH= yes
NO_BUILD= yes
PLIST_FILES= bin/mping
PORTDOCS= README.md
OPTIONS_DEFINE= DOCS
do-install:
${INSTALL_SCRIPT} ${WRKSRC}/mping ${STAGEDIR}${PREFIX}/bin
baptUnsubmitted
Done
Inline Actions

Why this line, I don't think it is actually needed

bapt: Why this line, I don't think it is actually needed
do-install-DOCS-on:
@${MKDIR} ${STAGEDIR}${DOCSDIR}
baptUnsubmitted
Done
Inline Actions

This is not needed as well, automatically created by the mtree file when creating the stage directory

bapt: This is not needed as well, automatically created by the mtree file when creating the stage…
${INSTALL_DATA} ${PORTDOCS:S,^,${WRKSRC}/,} ${STAGEDIR}${DOCSDIR}
.include <bsd.port.mk>
View Options

sys/kern/vfs_syscalls.c

  • This file was added.
PropertyOld ValueNew Value
svn:eol-stylenullnative
\ No newline at end of property
svn:keywordsnullFreeBSD=%H
\ No newline at end of property
svn:mime-typenulltext/plain
\ No newline at end of property
# $FreeBSD$
PORTNAME= mping
DISTVERSION= 0.1.3
CATEGORIES= sysutils
MAINTAINER= dteske@FreeBSD.org
COMMENT= Ping multiple hosts
LICENSE= BSD2CLAUSE
LICENSE_FILE= ${WRKSRC}/LICENSE
USE_GITHUB= yes
GH_ACCOUNT= FrauBSD
NO_ARCH= yes
NO_BUILD= yes
PLIST_FILES= bin/mping
PORTDOCS= README.md
OPTIONS_DEFINE= DOCS
do-install:
${INSTALL_SCRIPT} ${WRKSRC}/mping ${STAGEDIR}${PREFIX}/bin
baptUnsubmitted
Done
Inline Actions

Why this line, I don't think it is actually needed

bapt: Why this line, I don't think it is actually needed
do-install-DOCS-on:
@${MKDIR} ${STAGEDIR}${DOCSDIR}
baptUnsubmitted
Done
Inline Actions

This is not needed as well, automatically created by the mtree file when creating the stage directory

bapt: This is not needed as well, automatically created by the mtree file when creating the stage…
${INSTALL_DATA} ${PORTDOCS:S,^,${WRKSRC}/,} ${STAGEDIR}${DOCSDIR}
.include <bsd.port.mk>
View Options

sys/sys/syscallsubr.h

  • This file was added.
PropertyOld ValueNew Value
svn:eol-stylenullnative
\ No newline at end of property
svn:keywordsnullFreeBSD=%H
\ No newline at end of property
svn:mime-typenulltext/plain
\ No newline at end of property
# $FreeBSD$
PORTNAME= mping
DISTVERSION= 0.1.3
CATEGORIES= sysutils
MAINTAINER= dteske@FreeBSD.org
COMMENT= Ping multiple hosts
LICENSE= BSD2CLAUSE
LICENSE_FILE= ${WRKSRC}/LICENSE
USE_GITHUB= yes
GH_ACCOUNT= FrauBSD
NO_ARCH= yes
NO_BUILD= yes
PLIST_FILES= bin/mping
PORTDOCS= README.md
OPTIONS_DEFINE= DOCS
do-install:
${INSTALL_SCRIPT} ${WRKSRC}/mping ${STAGEDIR}${PREFIX}/bin
baptUnsubmitted
Done
Inline Actions

Why this line, I don't think it is actually needed

bapt: Why this line, I don't think it is actually needed
do-install-DOCS-on:
@${MKDIR} ${STAGEDIR}${DOCSDIR}
baptUnsubmitted
Done
Inline Actions

This is not needed as well, automatically created by the mtree file when creating the stage directory

bapt: This is not needed as well, automatically created by the mtree file when creating the stage…
${INSTALL_DATA} ${PORTDOCS:S,^,${WRKSRC}/,} ${STAGEDIR}${DOCSDIR}
.include <bsd.port.mk>