All of lore.kernel.org
 help / color / mirror / Atom feed
From: Joe Damato <jdamato@fastly.com>
To: Alejandro Colomar <alx@kernel.org>
Cc: linux-man@vger.kernel.org
Subject: Re: [PATCH 0/1] ioctl_epoll.2: Add epoll ioctl documentation
Date: Thu, 6 Jun 2024 19:08:06 -0700	[thread overview]
Message-ID: <ZmJrhtfUHA6ZVoz5@LQ3V64L9R2> (raw)
In-Reply-To: <jqnuggiy2od5xc4xi67oemdhavh2aqityedevpxchypeqeal43@xp75h2cplbj7>

On Fri, Jun 07, 2024 at 12:25:05AM +0200, Alejandro Colomar wrote:
> Hi!
> 
> On Thu, Jun 06, 2024 at 03:14:33PM GMT, Joe Damato wrote:
> > On Thu, Jun 06, 2024 at 11:44:10PM +0200, Alejandro Colomar wrote:
> > > Sure.  I used a different pattern with prctl(2)s, so let's use a mix.
> > > Do the ioctl_*.2 tradition, but also add link pages with the names of
> > > the actual operations (which will allow to search directly for the man
> > > pages of the individual operations).
> > 
> > Thanks for your careful review. I will make the changes you
> > suggested for the v2.
> 
> Thank you.
> 
> > 
> > > > Based on the current status of glibc, I would assume that this change will
> > > > be part of glibc 2.40 (it is listed under 2.40 in the NEWS section), which
> > > > may be released in a few months [3].
> > > 
> > > If you're certain that this will be part of glibc 2.40, I'm fine merging
> > > it already.  We can always patch it if something changes.
> > 
> > I have no idea if I can be certain of that. I am not a maintainer
> > nor do I have commit access to glibc, so I truly have no idea.
> > 
> > I suppose it is possible that they may decide to cut glibc 2.40 from
> > before my patch went in? It does not seem there is any 2.40 tag yet,
> > AFAICT.
> > 
> > How about I proceed by making all the changes you've requested and
> > get the patch into a shape where it can be merged. Hopefully,
> > that'll only take one (or two ;) more revisions.
> > 
> > Once the patch is in good shape, then we can decide whether to merge
> > or wait for glibc 2.40? If they are releasing it in 8/2024, it's not
> > too long to wait.
> > 
> > Does that seem OK to you?
> 
> Yep, that's fine.  If you know who merged your patch, maybe you can CC
> him/her?

I had an existing email thread with them on another topic; they
mentioned that my change should be included in 2.40 as there has
been no announcement of a release freeze yet.

As I mentioned above, if you'd like to be cautious, I'm happy to
just get the patch into a pristine state for merging and then
waiting to merge until glibc 2.40 is shipped.

Totally up to you!

  reply	other threads:[~2024-06-07  2:08 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-06-04 18:17 [PATCH 0/1] ioctl_epoll.2: Add epoll ioctl documentation Joe Damato
2024-06-04 18:17 ` [PATCH 1/1] ioctl_epoll.2: New page describing ioctl(2) operations for epoll fds Joe Damato
2024-06-06 21:39   ` Alejandro Colomar
2024-06-06 21:46     ` Alejandro Colomar
2024-06-07 21:53       ` Joe Damato
2024-06-09 17:04         ` Alejandro Colomar
2024-06-10 17:29           ` Joe Damato
2024-06-10 18:24             ` Alejandro Colomar
2024-06-10 20:16               ` Joe Damato
2024-06-10 22:21                 ` Alejandro Colomar
2024-06-11 12:39                   ` G. Branden Robinson
2024-06-11 14:34                     ` Alejandro Colomar
2024-06-11 16:06                       ` G. Branden Robinson
2024-06-11 16:42                         ` Alejandro Colomar
2024-06-07  2:06     ` Joe Damato
2024-06-07 10:29       ` Alejandro Colomar
2024-06-07 21:56         ` Joe Damato
2024-06-06 21:44 ` [PATCH 0/1] ioctl_epoll.2: Add epoll ioctl documentation Alejandro Colomar
2024-06-06 22:14   ` Joe Damato
2024-06-06 22:25     ` Alejandro Colomar
2024-06-07  2:08       ` Joe Damato [this message]
2024-06-07 10:44         ` Alejandro Colomar

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ZmJrhtfUHA6ZVoz5@LQ3V64L9R2 \
    --to=jdamato@fastly.com \
    --cc=alx@kernel.org \
    --cc=linux-man@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.