Date: Tue, 5 Jan 2021 16:26:38 -0700 From: Alan Somers <asomers@freebsd.org> To: Brooks Davis <brooks@freebsd.org> Cc: Jessica Clarke <jrtc27@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org>, "dev-commits-src-all@freebsd.org" <dev-commits-src-all@freebsd.org>, "dev-commits-src-main@freebsd.org" <dev-commits-src-main@freebsd.org> Subject: Re: git: 022ca2fc7fe0 - main - Add aio_writev and aio_readv Message-ID: <CAOtMX2hBG4vNExS7bhCwzAQXnsZmG=BKOPgza_NeM4rmy=sLrw@mail.gmail.com> In-Reply-To: <20210105231100.GD94898@spindle.one-eyed-alien.net> References: <202101030259.1032x5hV006606@gitrepo.freebsd.org> <B37BE19B-1AB7-4B6A-BB32-539B3C9EDB17@freebsd.org> <CAOtMX2jqmYo%2Bzu8Q_EFiKpFOZ9ygRMpVB-bp2D1OYy90J1s-gA@mail.gmail.com> <20210105231100.GD94898@spindle.one-eyed-alien.net>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Jan 5, 2021 at 4:11 PM Brooks Davis <brooks@freebsd.org> wrote: > On Sat, Jan 02, 2021 at 10:09:04PM -0700, Alan Somers wrote: > > On Sat, Jan 2, 2021 at 9:39 PM Jessica Clarke <jrtc27@freebsd.org> > wrote: > > > > > On 3 Jan 2021, at 02:59, Alan Somers <asomers@FreeBSD.org> wrote: > > > > diff --git a/sys/kern/syscalls.master b/sys/kern/syscalls.master > > > > index b7ea5e939635..aaa0a1277461 100644 > > > > --- a/sys/kern/syscalls.master > > > > +++ b/sys/kern/syscalls.master > > > > @@ -1477,7 +1477,17 @@ > > > > _In_opt_ struct sigevent *sig > > > > ); > > > > } > > > > -258-271 AUE_NULL UNIMPL nosys > > > > +258 AUE_AIO_WRITEV STD { > > > > + int aio_writev( > > > > + _Inout_ struct aiocb *aiocbp > > > > + ); > > > > + } > > > > +259 AUE_AIO_READV STD { > > > > + int aio_readv( > > > > + _Inout_ struct aiocb *aiocbp > > > > + ); > > > > + } > > > > +260-271 AUE_NULL UNIMPL nosys > > > > 272 AUE_O_GETDENTS COMPAT11 { > > > > int getdents( > > > > int fd, > > > > > > Should these not be added to the end? > > > > > > Jess > > > > > > > Should they be? I'm not aware of any requirement to add new syscalls to > > the end. I put them here so they would be next to the other AIO > syscalls. > > Yes. I'm sorry I missed this in the review. It's vastly easier to audit > these files and address conflicts if they are append-only. We're also > using these syscall numbers internally specifically to avoid conflicts > with new syscalls. Please move them to the end. > > I do see that we only provide extremely outdated advice in the comments > of syscalls.master. I'll take a look at improving this and the wiki > page. > > Thanks, > Brooks > Ok, I'll move them. And could you please elaborate on how we're "using these syscall numbers internally"? -Alan
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAOtMX2hBG4vNExS7bhCwzAQXnsZmG=BKOPgza_NeM4rmy=sLrw>