]> Sergey Matveev's repositories - public-inbox.git/log
public-inbox.git
5 years agot/nntpd*.t: skip TLS tests for old Net::NNTP
Eric Wong [Sun, 30 Jun 2019 00:01:55 +0000 (00:01 +0000)]
t/nntpd*.t: skip TLS tests for old Net::NNTP

Perl prior to 5.22 did not bundle a Net::NNTP (or libnet)
capable of handling TLS.

5 years agodskqxs: more closely match epoll semantics
Eric Wong [Sat, 29 Jun 2019 20:19:29 +0000 (20:19 +0000)]
dskqxs: more closely match epoll semantics

EV_DISPATCH is actually a better match for EPOLLONESHOT
semantics than EV_ONESHOT in that it doesn't require EV_ADD
for every mod operation.

Blindly using EV_ADD everywhere forces the FreeBSD kernel to
do extra allocations up front, so it's best avoided.

5 years agohttp: use bigger, but shorter-lived buffers for pipes
Eric Wong [Fri, 28 Jun 2019 19:26:56 +0000 (19:26 +0000)]
http: use bigger, but shorter-lived buffers for pipes

Linux pipes default to 65536 bytes in size, and we want to read
external processes as fast as possible now that we don't use
Danga::Socket or buffer to heap.

However, drop the buffer ASAP if we need to wait on anything;
since idle buffers can be idle for eons.  This lets other
execution contexts can reuse that memory right away.

5 years agohttpd/async: switch to buffering-as-fast-as-possible
Eric Wong [Fri, 28 Jun 2019 18:58:36 +0000 (18:58 +0000)]
httpd/async: switch to buffering-as-fast-as-possible

With DS buffering to a temporary file nowadays, applying
backpressure to git-http-backend(1) hurts overall memory
usage of the system.  Instead, try to get git-http-backend(1)
to finish as quickly as possible and use edge-triggered
notifications to reduce wakeups on our end.

5 years agoparentpipe: make the ->close call more obvious
Eric Wong [Fri, 28 Jun 2019 18:55:09 +0000 (18:55 +0000)]
parentpipe: make the ->close call more obvious

We can close directly in event_step without bad side effects,
and then we also don't need to take a reason arg from worker_quit,
since we weren't logging it anywhere.

5 years agoparentpipe: document and use one-shot wakeups
Eric Wong [Fri, 28 Jun 2019 18:46:10 +0000 (18:46 +0000)]
parentpipe: document and use one-shot wakeups

The master process only dies once and we close ourselves right
away.  So it doesn't matter if it's level-triggered or
edge-triggered, actually, but one-shot is most consistent with
our use and keeps the kernel from doing extra work.

5 years agohttp: support HTTPS (kinda)
Eric Wong [Fri, 28 Jun 2019 08:18:51 +0000 (08:18 +0000)]
http: support HTTPS (kinda)

It's barely any effort at all to support HTTPS now that we have
NNTPS support and can share all the code for writing daemons.

However, we still depend on Varnish to avoid hug-of-death
situations, so supporting reverse-proxying will be required.

5 years agods: consolidate IO::Socket::SSL checks
Eric Wong [Fri, 28 Jun 2019 07:37:26 +0000 (07:37 +0000)]
ds: consolidate IO::Socket::SSL checks

We need to be careful about handling EAGAIN on write(2)
failures deal with SSL_WANT_READ vs SSL_WANT_WRITE as
appropriate.

5 years agods: handle deferred DS->close after timers
Eric Wong [Fri, 28 Jun 2019 05:25:40 +0000 (05:25 +0000)]
ds: handle deferred DS->close after timers

Our hacks in EvCleanup::next_tick and EvCleanup::asap were due
to the fact "closed" sockets were deferred and could not wake
up the event loop, causing certain actions to be delayed until
an event fired.

Instead, ensure we don't sleep if there are pending sockets to
close.

We can then remove most of the EvCleanup stuff

While we're at it, split out immediate timer handling into a
separate array so we don't need to deal with time calculations
for the event loop.

5 years agolistener: use edge-triggered notifications
Eric Wong [Thu, 27 Jun 2019 21:21:05 +0000 (21:21 +0000)]
listener: use edge-triggered notifications

We don't need extra wakeups from the kernel when we know a
listener is already active.

5 years agohttp: use requeue instead of watch_in1
Eric Wong [Thu, 27 Jun 2019 21:21:04 +0000 (21:21 +0000)]
http: use requeue instead of watch_in1

Don't use epoll or kqueue to watch for anything unless we hit
EAGAIN, since we don't know if a socket is SSL or not.

5 years agods: move requeue logic over from NNTP
Eric Wong [Thu, 27 Jun 2019 21:21:03 +0000 (21:21 +0000)]
ds: move requeue logic over from NNTP

We'll be reusing requeue in other places to reduce trips to
the kernel to retrieve "hot" descriptors.

5 years agods: share lazy rbuf handling between HTTP and NNTP
Eric Wong [Wed, 26 Jun 2019 08:11:10 +0000 (08:11 +0000)]
ds: share lazy rbuf handling between HTTP and NNTP

Doing this for HTTP cuts the memory usage of 10K
idle-after-one-request HTTP clients from 92 MB to 47 MB.

The savings over the equivalent NNTP change in commit
6f173864f5acac89769a67739b8c377510711d49,
("nntp: lazily allocate and stash rbuf") seems down to the
size of HTTP requests and the fact HTTP is a client-sends-first
protocol where as NNTP is server-sends-first.

5 years agot/ds-leak: fix race
Eric Wong [Sat, 29 Jun 2019 06:34:40 +0000 (06:34 +0000)]
t/ds-leak: fix race

We need to ensure we run lsof on the sleep(1) process, and not
the fork of ourselves before execve(2).  This race applies when
we're using the default pure-Perl spawn() implementation.

5 years agowatchmaildir: show the current path on spamcheck failures
Eric Wong [Wed, 26 Jun 2019 09:00:43 +0000 (09:00 +0000)]
watchmaildir: show the current path on spamcheck failures

Knowing which message failed a spam check is tough when I have
many Maildirs and don't have a search indexing tool setup for
spam mail.

5 years agocerts/create-certs: create certs in 'certs/' directory
Eric Wong [Mon, 24 Jun 2019 08:16:43 +0000 (08:16 +0000)]
certs/create-certs: create certs in 'certs/' directory

If running in our top-level source tree, use the 'certs/'
directory as  the prefix so we can just invoke `./certs/create-certs.perl'
instead of `(cd certs && ./create-certs.perl)'

5 years agods: cleanup poll test and avoid clobbering imports
Eric Wong [Wed, 26 Jun 2019 07:58:15 +0000 (07:58 +0000)]
ds: cleanup poll test and avoid clobbering imports

On Linux systems with epoll support, we don't want to be
clobbering defined subs in the t/ds-poll.t test; so use
OO ->method dispatch instead and require users to explicitly
import subs via EXPORT_OK.

5 years agoMerge remote-tracking branch 'origin/nntp-tls'
Eric Wong [Wed, 26 Jun 2019 06:36:27 +0000 (06:36 +0000)]
Merge remote-tracking branch 'origin/nntp-tls'

* origin/nntp-tls: (59 commits)
  ds: ->write must not clobber empty wbuf array
  Makefile: skip DSKQXS in global syntax check
  ds: reduce overhead of tempfile creation
  Revert "ci: require IO::KQueue on FreeBSD, for now"
  ds: reimplement IO::Poll support to look like epoll
  ds: split out IO::KQueue-specific code
  daemon: use FreeBSD accept filters on non-NNTP
  daemon: set TCP_DEFER_ACCEPT on everything but NNTP
  nntp: send greeting immediately for plain sockets
  ci: require IO::KQueue on FreeBSD, for now
  nntp: lazily allocate and stash rbuf
  ds: flush_write runs ->write callbacks even if closed
  nntp: simplify long response logic and fix nesting
  ds: always use EV_ADD with EV_SET
  nntp: reduce allocations for greeting
  ds: allow ->write callbacks to syswrite directly
  daemon: use SSL_MODE_RELEASE_BUFFERS
  t/nntpd-tls: slow client connection test
  nntp: call SSL_shutdown in normal cases
  ds|nntp: use CORE::close on socket
  ...

5 years agosearchview: avoid displaying full paths on errors
Eric Wong [Tue, 25 Jun 2019 04:08:14 +0000 (04:08 +0000)]
searchview: avoid displaying full paths on errors

Displaying full path names of installed modules could expose
unnecessary information about user home directory names or other
potentially sensitive information.  However, displaying a module
name could still be useful for diagnosing problems, so map full
paths to the relevant part of the path name which is relevant to
the package name.

Reported-by: Ali Alnubani <alialnu@mellanox.com>
  https://public-inbox.org/meta/20190611193815.c4uovtlp574bid6x@dcvr/

5 years agomsgmap: mid_insert: use plain "INSERT" to detect duplicates
Eric Wong [Mon, 24 Jun 2019 23:29:37 +0000 (23:29 +0000)]
msgmap: mid_insert: use plain "INSERT" to detect duplicates

"INSERT OR IGNORE" still bumps the auto-increment counter in
SQLite, which causes gaps to appear in NNTP article numbering.

This bug appeared in v2 repos where V2Writable may call ->add
repeatedly on the same message.  This bug is apparent with
public-inbox-watch and work-in-progress IMAP watchers which may
rescan and (attempt to) reinsert the same message on mailbox
changes.

Most uses of public-inbox-mda were not affected, unless the
same message is actually delivered multiple times to the mda.
v1 is not affected, either, since deduplication is only based
on Message-ID and msgmap never sees the duplicate.

Reported-by: "Eric W. Biederman" <ebiederm@xmission.com>
5 years agods: ->write must not clobber empty wbuf array
Eric Wong [Mon, 24 Jun 2019 18:18:18 +0000 (18:18 +0000)]
ds: ->write must not clobber empty wbuf array

We need to account for ->write(CODE) calls doing ->write(SCALARREF),
otherwise flush_write may see the wrong ->{wbuf} field.

5 years agoMakefile: skip DSKQXS in global syntax check
Eric Wong [Mon, 24 Jun 2019 04:06:42 +0000 (04:06 +0000)]
Makefile: skip DSKQXS in global syntax check

IO::KQueue isn't easily installable on Linux systems.

5 years agods: reduce overhead of tempfile creation
Eric Wong [Mon, 24 Jun 2019 02:52:58 +0000 (02:52 +0000)]
ds: reduce overhead of tempfile creation

We end up buffering giant things to the FS sometimes, and open()
is not a cheap syscall; so being forced to do it twice to get a
file description with O_APPEND is gross when we can just use
O_EXCL ourselves and loop on EEXIST.

5 years agoRevert "ci: require IO::KQueue on FreeBSD, for now"
Eric Wong [Mon, 24 Jun 2019 02:52:57 +0000 (02:52 +0000)]
Revert "ci: require IO::KQueue on FreeBSD, for now"

Now that we support IO::Poll once again, we can remove
the IO::KQueue requirement.

5 years agods: reimplement IO::Poll support to look like epoll
Eric Wong [Mon, 24 Jun 2019 02:52:56 +0000 (02:52 +0000)]
ds: reimplement IO::Poll support to look like epoll

At least the subset of epoll we use.  EPOLLET might be
difficult to emulate if we end up using it.

5 years agods: split out IO::KQueue-specific code
Eric Wong [Mon, 24 Jun 2019 02:52:55 +0000 (02:52 +0000)]
ds: split out IO::KQueue-specific code

We don't need to code multiple event loops or have branches in
watch() if we can easily make the IO::KQueue-based interface
look like our lower-level epoll_* API.

5 years agodaemon: use FreeBSD accept filters on non-NNTP
Eric Wong [Mon, 24 Jun 2019 02:52:54 +0000 (02:52 +0000)]
daemon: use FreeBSD accept filters on non-NNTP

Similar to TCP_DEFER_ACCEPT on Linux, FreeBSD has a 'dataready'
accept filter which we can use to reduce wakeups when doing
TLS negotiation or plain HTTP.  There's also a 'httpready'
which we can use for plain HTTP connections.

5 years agodaemon: set TCP_DEFER_ACCEPT on everything but NNTP
Eric Wong [Mon, 24 Jun 2019 02:52:53 +0000 (02:52 +0000)]
daemon: set TCP_DEFER_ACCEPT on everything but NNTP

This Linux-specific option can save us some wakeups during
the TLS negotiation phase, and it can help with ordinary HTTP,
too.

Plain NNTP (and in the future, POP3) are the only things which
require the server send messages, first.

5 years agonntp: send greeting immediately for plain sockets
Eric Wong [Mon, 24 Jun 2019 02:52:52 +0000 (02:52 +0000)]
nntp: send greeting immediately for plain sockets

A tiny write() for the greeting on a just accept()-ed TCP socket
won't fail with EAGAIN, so we can avoid the extra epoll syscall
traffic with plain sockets.

5 years agoci: require IO::KQueue on FreeBSD, for now
Eric Wong [Mon, 24 Jun 2019 02:52:51 +0000 (02:52 +0000)]
ci: require IO::KQueue on FreeBSD, for now

We'll likely replace IO::KQueue (at least on FreeBSD) using
a pure-Perl syscall()-based version since syscall numbers are
consistent across architectures on FreeBSD and easy to maintain.

IO::KQueue->EV_SET is also shockingly inefficient in that it
calls kqueue() as much as epoll_ctl.

5 years agonntp: lazily allocate and stash rbuf
Eric Wong [Mon, 24 Jun 2019 02:52:50 +0000 (02:52 +0000)]
nntp: lazily allocate and stash rbuf

Allocating a per-client buffer up front is unnecessary and
wastes a hash slot.  For the majority of (non-malicious)
clients, we won't need to store rbuf in a long-lived object
associated with a client socket at all.

This saves around 10M on 64-bit with 20K connected-but-idle
clients.

5 years agods: flush_write runs ->write callbacks even if closed
Eric Wong [Mon, 24 Jun 2019 02:52:49 +0000 (02:52 +0000)]
ds: flush_write runs ->write callbacks even if closed

We may need to rely on cleanup code running in enqueued
callbacks, so ensure we call it when flush_write happens.

5 years agonntp: simplify long response logic and fix nesting
Eric Wong [Mon, 24 Jun 2019 02:52:48 +0000 (02:52 +0000)]
nntp: simplify long response logic and fix nesting

We can get rid of the {long_res} field and reuse the write
buffer ordering logic to prevent nesting of responses from
requeue.

On FreeBSD, this fixes a problem of callbacks firing twice
because kqueue as event_step is now our only callback entry
point.

There's a slight change in the stdout "logging" format, in
that we can no longer distinguish between writes blocked
due to slow clients or deferred long responses.  Not sure
if this affects anybody parsing logs or not, but preserving
the old format could prove expensive and not worth the
effort.

5 years agods: always use EV_ADD with EV_SET
Eric Wong [Mon, 24 Jun 2019 02:52:47 +0000 (02:52 +0000)]
ds: always use EV_ADD with EV_SET

kqueue EV_ONESHOT semantics are different than epoll
EPOLLONESHOT.  epoll only disables watches for that event while
keeping the item in the rbtree for future EPOLL_CTL_MOD.  kqueue
removes the watch from the filter set entirely, necessitating
the use of EV_ADD for future modifications.

5 years agonntp: reduce allocations for greeting
Eric Wong [Mon, 24 Jun 2019 02:52:46 +0000 (02:52 +0000)]
nntp: reduce allocations for greeting

No need to allocate a new PerlIO::scalar filehandle for every
client, instead we can now pass the same CODE reference which
calls DS->write on a reused string reference.

5 years agods: allow ->write callbacks to syswrite directly
Eric Wong [Mon, 24 Jun 2019 02:52:45 +0000 (02:52 +0000)]
ds: allow ->write callbacks to syswrite directly

We can bypass buffering when wbuf is empty when it's called
from a CODE reference passed to ->write.

5 years agodaemon: use SSL_MODE_RELEASE_BUFFERS
Eric Wong [Mon, 24 Jun 2019 02:52:44 +0000 (02:52 +0000)]
daemon: use SSL_MODE_RELEASE_BUFFERS

34K per idle connection adds up to large amounts of memory;
especially with the speed of malloc nowadays compared to the
cost of cache misses or worse, swapping.

5 years agot/nntpd-tls: slow client connection test
Eric Wong [Mon, 24 Jun 2019 02:52:43 +0000 (02:52 +0000)]
t/nntpd-tls: slow client connection test

We need to ensure slowly negotiating TLS clients don't block
the event loop.  This is why I added the size check of
{wbuf} before and after calling the CODE ref in DS::flush_write.

5 years agonntp: call SSL_shutdown in normal cases
Eric Wong [Mon, 24 Jun 2019 02:52:42 +0000 (02:52 +0000)]
nntp: call SSL_shutdown in normal cases

This is in accordance with TLS standards and will be needed
to support session caching/reuse in the future.  However, we
don't issue shutdown(2) since we know not to inadvertantly
share our sockets with other processes.

5 years agods|nntp: use CORE::close on socket
Eric Wong [Mon, 24 Jun 2019 02:52:41 +0000 (02:52 +0000)]
ds|nntp: use CORE::close on socket

IO::Socket::SSL will try to re-bless back to the original class
on TLS negotiation failure.  Unfortunately, the original class
is 'GLOB', and re-blessing to 'GLOB' takes away all the IO::Handle
methods, because Filehandle/IO are a special case in Perl5.
Anyways, since we already use syswrite() and sysread() as functions
on our socket, we might as well use CORE::close(), as well (and
it plays nicely with tied classes).

5 years agodaemon: map inherited sockets to well-known schemes
Eric Wong [Mon, 24 Jun 2019 02:52:40 +0000 (02:52 +0000)]
daemon: map inherited sockets to well-known schemes

I don't want to specify "--listen" in my systemd .service files,
so map 563 to NNTPS automatically (and 443 to HTTPS, but HTTPS
support doesn't work, yet).

5 years agocerts/create-certs.perl: fix cert validity on 32-bit
Eric Wong [Mon, 24 Jun 2019 02:52:39 +0000 (02:52 +0000)]
certs/create-certs.perl: fix cert validity on 32-bit

If I'm still alive, I won't be coding after 2038 :<

5 years agonntp: NNTPS and NNTP+STARTTLS working
Eric Wong [Mon, 24 Jun 2019 02:52:38 +0000 (02:52 +0000)]
nntp: NNTPS and NNTP+STARTTLS working

It kinda, barely works, and I'm most happy I got it working
without any modifications to the main NNTP::event_step callback
thanks to the DS->write(CODE) support we inherited from
Danga::Socket.

5 years agonntp: wait for writability before sending greeting
Eric Wong [Mon, 24 Jun 2019 02:52:37 +0000 (02:52 +0000)]
nntp: wait for writability before sending greeting

This will be needed for NNTPS support, since we need
to negotiate the TLS connection before writing the
greeting and we can reuse the existing buffer layer
to enqueue writes.

5 years agods: deal better with FS-related errors IO buffers
Eric Wong [Mon, 24 Jun 2019 02:52:36 +0000 (02:52 +0000)]
ds: deal better with FS-related errors IO buffers

Instead of ENOMEM (or fragmentation/swap storms), using tempfile
buffers opens us up to filesystem and storage-related errors
(e.g. ENOSPC, EFBIG, EIO, EROFS).  Log these errors, drop the
particular client, and try to limp by with whateve we have left.

5 years agoallow use of PerlIO layers for filesystem writes
Eric Wong [Mon, 24 Jun 2019 02:52:35 +0000 (02:52 +0000)]
allow use of PerlIO layers for filesystem writes

It may make sense to use PerlIO::mmap or PerlIO::scalar for
DS write buffering with IO::Socket::SSL or similar (since we can't
use MSG_MORE), so that means we need to go through buffering
in userspace for the common case; while still being easily
compatible with slow clients.

And it also simplifies GitHTTPBackend slightly.

Maybe it can make sense for HTTP input buffering, too...

5 years agonntp: simplify re-arming/requeue logic
Eric Wong [Mon, 24 Jun 2019 02:52:34 +0000 (02:52 +0000)]
nntp: simplify re-arming/requeue logic

We can be smarter about requeuing clients to run and avoid
excessive epoll_ctl calls since we can trust event_step to do
the right thing depending on the state of the client.

5 years agods: hoist out do_read from NNTP and HTTP
Eric Wong [Mon, 24 Jun 2019 02:52:33 +0000 (02:52 +0000)]
ds: hoist out do_read from NNTP and HTTP

Both NNTP and HTTP have common needs and we can factor
out some common code to make dealing with IO::Socket::SSL
easier.

5 years agohttp|nntp: be explicit about bytes::length on rbuf
Eric Wong [Mon, 24 Jun 2019 02:52:32 +0000 (02:52 +0000)]
http|nntp: be explicit about bytes::length on rbuf

It should not matter because our rbuf is always from
a socket without encoding layers, but this makes things
easier to follow.

5 years agods: remove pointless exit calls
Eric Wong [Mon, 24 Jun 2019 02:52:31 +0000 (02:52 +0000)]
ds: remove pointless exit calls

They're never called; the only way to break out of that loop
is the PostEventLoop callback.

5 years agoevcleanup: replace _run_asap with `event_step' callback
Eric Wong [Mon, 24 Jun 2019 02:52:30 +0000 (02:52 +0000)]
evcleanup: replace _run_asap with `event_step' callback

No point in keeping a one-line wrapper sub around.

5 years agods: pass $self to code references
Eric Wong [Mon, 24 Jun 2019 02:52:29 +0000 (02:52 +0000)]
ds: pass $self to code references

We can reduce the amount of short-lived anonymous subs we
create by passing $self to code references.

5 years agohttp: don't pass extra args to PublicInbox::DS::close
Eric Wong [Mon, 24 Jun 2019 02:52:28 +0000 (02:52 +0000)]
http: don't pass extra args to PublicInbox::DS::close

YAGNI

Followup-to: commit 30ab5cf82b9d47242640f748a0f9a088ca783e32
("ds: reduce Errno imports and drop ->close reason")

5 years agods: favor `delete' over assigning fields to `undef'
Eric Wong [Mon, 24 Jun 2019 02:52:27 +0000 (02:52 +0000)]
ds: favor `delete' over assigning fields to `undef'

This is cleaner in most cases and may allow Perl to reuse memory
from unused fields.

We can do this now that we no longer support Perl 5.8; since
Danga::Socket was written with struct-like pseudo-hash support
in mind, and Perl 5.9+ dropped support for pseudo-hashes over
a decade ago.

5 years agohttp|nntp: favor "$! == EFOO" over $!{EFOO} checks
Eric Wong [Mon, 24 Jun 2019 02:52:26 +0000 (02:52 +0000)]
http|nntp: favor "$! == EFOO" over $!{EFOO} checks

Integer comparisions of "$!" are faster than hash lookups.

See commit 6fa2b29fcd0477d126ebb7db7f97b334f74bbcbc
("ds: cleanup Errno imports and favor constant comparisons")
for benchmarks.

5 years agoqspawn: describe where `$rpipe' come from
Eric Wong [Mon, 24 Jun 2019 02:52:25 +0000 (02:52 +0000)]
qspawn: describe where `$rpipe' come from

It wasn't immediately obvious to me after several months of
not looking at this code.

5 years agospawn: remove `Blocking' flag handling
Eric Wong [Mon, 24 Jun 2019 02:52:24 +0000 (02:52 +0000)]
spawn: remove `Blocking' flag handling

Instead, the O_NONBLOCK flag is set by PublicInbox::HTTPD::Async;
and we won't be setting it elsewhere.

5 years agohttpd/async: remove EINTR check
Eric Wong [Mon, 24 Jun 2019 02:52:23 +0000 (02:52 +0000)]
httpd/async: remove EINTR check

This pipe is always non-blocking when run under public-inbox-httpd
and it won't fail with EINTR in that case

5 years agods: get rid of event_watch field
Eric Wong [Mon, 24 Jun 2019 02:52:22 +0000 (02:52 +0000)]
ds: get rid of event_watch field

We don't need to keep track of that field since we always
know what events we're interested in when using one-shot
wakeups.

5 years agods: remove IO::Poll support (for now)
Eric Wong [Mon, 24 Jun 2019 02:52:21 +0000 (02:52 +0000)]
ds: remove IO::Poll support (for now)

It may be reinstated at a later time if there's interest; but I
want to be able to use one-shot notifications for certain events
while retaining level-triggered notifications others.

OTOH, I intend to fully support kqueue; via IO::KQueue for now,
but via syscall() eventually to take advantage of the syscall
reduction kevent(2) can provide over (current) epoll APIs.

5 years agods: share watch_chg between watch_read/watch_write
Eric Wong [Mon, 24 Jun 2019 02:52:20 +0000 (02:52 +0000)]
ds: share watch_chg between watch_read/watch_write

There was much duplicate logic between watch_read and
watch_write.  Share that logic, and give us room to enable
edge-triggered or one-shot notifications in the future.

5 years agods: import IO::KQueue namespace
Eric Wong [Mon, 24 Jun 2019 02:52:19 +0000 (02:52 +0000)]
ds: import IO::KQueue namespace

Make the rest of our IO::KQueue-using code less verbose and
closer to the C equivalent.

5 years agods: set event flags directly at initialization
Eric Wong [Mon, 24 Jun 2019 02:52:18 +0000 (02:52 +0000)]
ds: set event flags directly at initialization

We can avoid the EPOLL_CTL_ADD && EPOLL_CTL_MOD sequence with
a single EPOLL_CTL_ADD.

5 years agosyscall: get rid of unnecessary uname local vars
Eric Wong [Mon, 24 Jun 2019 02:52:17 +0000 (02:52 +0000)]
syscall: get rid of unnecessary uname local vars

We don't need to keep information from uname(2) around outside
of startup.

5 years agosyscall: get rid of unused EPOLL* constants
Eric Wong [Mon, 24 Jun 2019 02:52:16 +0000 (02:52 +0000)]
syscall: get rid of unused EPOLL* constants

EPOLLRDBAND is used for DECnet; and I'm pretty sure I won't be
updating any of our code to work with DECnet.

I've never found use for EPOLLHUP or EPOLLERR, either; so
disable those for now and add comments for things I might
actually use: EPOLLET and EPOLLONESHOT.

5 years agods: get rid of redundant and unnecessary POLL* constants
Eric Wong [Mon, 24 Jun 2019 02:52:15 +0000 (02:52 +0000)]
ds: get rid of redundant and unnecessary POLL* constants

EPOLL* constants already match their POLL* counterparts and
there's no way Linux can ever diverge or change the values
of those constants.  So we'll favor the EPOLL* ones since we
use EPOLLEXCLUSIVE, already.

For weird stuff like kqueue, we'd need to keep maintaining
the mapping, anyways.

5 years agods: switch write buffering to use a tempfile
Eric Wong [Mon, 24 Jun 2019 02:52:14 +0000 (02:52 +0000)]
ds: switch write buffering to use a tempfile

Data which can't fit into a generously-sized socket buffer,
has no business being stored in heap.

5 years agods: share send(..., MSG_MORE) logic
Eric Wong [Mon, 24 Jun 2019 02:52:13 +0000 (02:52 +0000)]
ds: share send(..., MSG_MORE) logic

No sense in having similar Linux-specific functionality in
both our NNTP.pm and HTTP.pm

5 years agohttp: favor DS->write(strref) when reasonable
Eric Wong [Mon, 24 Jun 2019 02:52:12 +0000 (02:52 +0000)]
http: favor DS->write(strref) when reasonable

This can avoid large memory copies when strings can't be
copy-on-write and saves us the trouble of creating new
refs in the code.

5 years agods: remove support for DS->write(undef)
Eric Wong [Mon, 24 Jun 2019 02:52:11 +0000 (02:52 +0000)]
ds: remove support for DS->write(undef)

We call ->flush_write directly, now; so we can eliminate a
needless check.

5 years agods: don't pass `events' arg to EPOLL_CTL_DEL
Eric Wong [Mon, 24 Jun 2019 02:52:10 +0000 (02:52 +0000)]
ds: don't pass `events' arg to EPOLL_CTL_DEL

There's no point in passing a mask of interesting events
when removing an item from the epoll watch set.

5 years agods: lazy-initialize wbuf
Eric Wong [Mon, 24 Jun 2019 02:52:09 +0000 (02:52 +0000)]
ds: lazy-initialize wbuf

We don't need write buffering unless we encounter slow clients
requesting large responses.  So don't waste a hash slot or
(empty) arrayref for it.

5 years agods: split out from ->flush_write and ->write
Eric Wong [Mon, 24 Jun 2019 02:52:08 +0000 (02:52 +0000)]
ds: split out from ->flush_write and ->write

Get rid of the confusing $need_queue variable and all
the associated documentation for it.  Instead, make it
obvious that we're either skipping the write buffer or
flushing the write buffer by splitting the sub in two.

5 years agods: lazy initialize wbuf_off
Eric Wong [Mon, 24 Jun 2019 02:52:07 +0000 (02:52 +0000)]
ds: lazy initialize wbuf_off

Since Perl 5.10+, "fields" makes a restricted hash; not a
compile-time-defined array (struct) with fixed offsets as
it did in Perl <= 5.8.

Thus in-use fields cost memory, and since the write buffer
offset is rarely needed; stop relying on it.

5 years agods: get rid of on_incomplete_write wrapper
Eric Wong [Mon, 24 Jun 2019 02:52:06 +0000 (02:52 +0000)]
ds: get rid of on_incomplete_write wrapper

Wrong place to be wrapping this method.

5 years agoAddTimer: avoid clock_gettime for the '0' case
Eric Wong [Mon, 24 Jun 2019 02:52:05 +0000 (02:52 +0000)]
AddTimer: avoid clock_gettime for the '0' case

We rely on immediate timers often, so we can avoid the overhead
of an extra subroutine call to retrieve the monotonic time (and
a sometimes-system call on some platforms).

5 years agods: use and export monotonic now()
Eric Wong [Mon, 24 Jun 2019 02:52:04 +0000 (02:52 +0000)]
ds: use and export monotonic now()

All of our internal timing code should use monotonic clocks
for consistency against system clock adjustments.

This can be shared by our Daemon and NNTP packages.

5 years agods: get rid of more unused debug instance methods
Eric Wong [Mon, 24 Jun 2019 02:52:03 +0000 (02:52 +0000)]
ds: get rid of more unused debug instance methods

Over a decade of using Danga::Socket and I never found the
built-in debug functionality useful.

5 years agods: get rid of {closed} field
Eric Wong [Mon, 24 Jun 2019 02:52:02 +0000 (02:52 +0000)]
ds: get rid of {closed} field

Merely checking the presence of the {sock} field is
enough, and having multiple sources of truth increases
confusion and the likelyhood of bugs.

5 years agomanifest: v2 epoch descriptions based on inbox->description
Eric Wong [Sun, 23 Jun 2019 17:42:05 +0000 (17:42 +0000)]
manifest: v2 epoch descriptions based on inbox->description

The default $GIT_DIR/description (provided by git.git templates)
isn't very useful for v2 epochs, so use the inbox description
and suffix it with the epoch number if it's otherwise unnamed.

Requested-by: Konstantin Ryabitsev <konstantin@linuxfoundation.org>
  https://public-inbox.org/meta/20190620190017.GA27175@chatter.i7.local/

5 years agot/httpd-corner: ensure chunk payload read doesn't overreach
Eric Wong [Thu, 20 Jun 2019 01:03:47 +0000 (01:03 +0000)]
t/httpd-corner: ensure chunk payload read doesn't overreach

It never has, AFAIK, but I'm making some changes to this code in
another branch and nearly introduced a bug where it would be
overreading and discarding the pipelined request.

5 years agot/httpd-corner.t: fix braino :x
Eric Wong [Thu, 20 Jun 2019 00:16:17 +0000 (00:16 +0000)]
t/httpd-corner.t: fix braino :x

Plack is for Perl, Rack is for Ruby; this a Perl project :x

5 years agods: stop distinguishing event read and write callbacks
Eric Wong [Sun, 16 Jun 2019 06:11:28 +0000 (06:11 +0000)]
ds: stop distinguishing event read and write callbacks

Having separate read/write callbacks in every class is too
confusing to my easily-confused mind.  Instead, give every class
an "event_step" callback which is easier to wrap my head around.

This will make future code to support IO::Socket::SSL-wrapped
sockets easier-to-digest, since SSL_write() can require waiting
on POLLIN events, and SSL_read() can require waiting on POLLOUT
events.

5 years agot/replace.t: fix SKIP label for testing w/o Xapian
Eric Wong [Sun, 16 Jun 2019 03:22:25 +0000 (03:22 +0000)]
t/replace.t: fix SKIP label for testing w/o Xapian

5 years agoxcpdb: don't warn on --jobs != --reshard
Eric Wong [Sun, 16 Jun 2019 01:32:01 +0000 (01:32 +0000)]
xcpdb: don't warn on --jobs != --reshard

It's slightly confusing since we dedicate one job
to dealing with fast-import + SQLite indexing; and
it's not worth complaining about when it happens.

5 years agoMerge remote-tracking branch 'origin/newspeak' into xcpdb
Eric Wong [Sun, 16 Jun 2019 01:04:28 +0000 (01:04 +0000)]
Merge remote-tracking branch 'origin/newspeak' into xcpdb

* origin/newspeak:
  comments: replace "partition" with "shard"
  t/xcpdb-reshard: use 'shard' term in local variables
  xapcmd: favor 'shard' over 'part' in local variables
  search: use "shard" for local variable
  v2writable: use "epoch" consistently when referring to git repos
  adminedit: "part" => "shard" for local variables
  v2writable: rename local vars to match Xapian terminology
  v2writable: avoid "part" in internal subs and fields
  search*: rename {partition} => {shard}
  xapcmd: update comments referencing "partitions"
  v2: rename SearchIdxPart => SearchIdxShard
  inboxwritable: s/partitions/shards/ in local var
  tests: change messages to use "shard" instead of partition
  v2writable: rename {partitions} field to {shards}
  v2writable: count_partitions => count_shards
  searchidxpart: start using "shard" in user-visible places
  rename reference to git epochs as "partitions"
  admin|xapcmd: user-facing messages say "shard"
  v2writable: update comments regarding xcpdb --reshard
  doc: rename our Xapian "partitions" to "shards"

5 years agot/psgi_search.t: use higher-level APIs
Eric Wong [Sun, 16 Jun 2019 00:57:36 +0000 (00:57 +0000)]
t/psgi_search.t: use higher-level APIs

No point in using lower-level APIs for a PSGI test.

5 years agosearchview: add link at bottom to reverse results
Eric Wong [Sat, 15 Jun 2019 20:23:42 +0000 (20:23 +0000)]
searchview: add link at bottom to reverse results

I could not find a place to put the link the top without
making navigation too cluttered.  Putting it at the bottom
of the page seems reasonable...

5 years agosearchview: support negative offsets to reverse ordering
Eric Wong [Sat, 15 Jun 2019 20:23:41 +0000 (20:23 +0000)]
searchview: support negative offsets to reverse ordering

Taking a hint from Perl array access, we'll allow negative
offsets for the 'o' parameter and to reverse the sort order.

5 years agoMerge remote-tracking branch 'origin/ds'
Eric Wong [Sat, 15 Jun 2019 17:38:42 +0000 (17:38 +0000)]
Merge remote-tracking branch 'origin/ds'

* origin/ds:
  ds: stop caring about event flags set by epoll/poll/kqueue
  ds: do not distinguish between POLLHUP and POLLERR
  ds: remove read method, here, too
  nntp: use sysread to append to existing buffer
  ds: remove steal_socket method
  ds: remove {fd} field
  ds: reduce Errno imports and drop ->close reason
  ds: cleanup Errno imports and favor constant comparisons
  ds: simplify write buffer accounting

5 years agot/git-http-backend: explain purpose of test
Eric Wong [Fri, 14 Jun 2019 10:09:13 +0000 (10:09 +0000)]
t/git-http-backend: explain purpose of test

I found myself tempted to switch to HTTP::Tiny, here, since
it's distributed with Perl since 5.14, unlike Net::HTTP
(which AFAIK was never a part of Perl proper).

But we really want to use Net::HTTP, here, since it's
lower-level and allows us to trigger server-side buffering
by not reading the entity body.

5 years agocomments: replace "partition" with "shard"
Eric Wong [Sat, 15 Jun 2019 08:25:44 +0000 (08:25 +0000)]
comments: replace "partition" with "shard"

Now that the code matches Xapian terminology, ensure
our comments match, too.

5 years agot/xcpdb-reshard: use 'shard' term in local variables
Eric Wong [Fri, 14 Jun 2019 21:30:31 +0000 (21:30 +0000)]
t/xcpdb-reshard: use 'shard' term in local variables

Another step in maintaining consistency with Xapian docs.

5 years agoxapcmd: favor 'shard' over 'part' in local variables
Eric Wong [Fri, 14 Jun 2019 21:29:37 +0000 (21:29 +0000)]
xapcmd: favor 'shard' over 'part' in local variables

Yet another step to keeping our naming consistent with Xapian
terminology.

5 years agosearch: use "shard" for local variable
Eric Wong [Fri, 14 Jun 2019 18:18:06 +0000 (18:18 +0000)]
search: use "shard" for local variable

Another small step towards terminology consistency with Xapian.

5 years agov2writable: use "epoch" consistently when referring to git repos
Eric Wong [Fri, 14 Jun 2019 18:16:09 +0000 (18:16 +0000)]
v2writable: use "epoch" consistently when referring to git repos

Be consistent with our own terminology and use "epoch" for
[0-9]+\.git repos.  The term "partition" is going away entirely.

5 years agoadminedit: "part" => "shard" for local variables
Eric Wong [Fri, 14 Jun 2019 18:14:42 +0000 (18:14 +0000)]
adminedit: "part" => "shard" for local variables

5 years agov2writable: rename local vars to match Xapian terminology
Eric Wong [Fri, 14 Jun 2019 18:00:20 +0000 (18:00 +0000)]
v2writable: rename local vars to match Xapian terminology

5 years agov2writable: avoid "part" in internal subs and fields
Eric Wong [Fri, 14 Jun 2019 17:50:45 +0000 (17:50 +0000)]
v2writable: avoid "part" in internal subs and fields

We'll be using the term "shard" from now on to be consistent
with Xapian terminology.

5 years agosearch*: rename {partition} => {shard}
Eric Wong [Fri, 14 Jun 2019 17:42:57 +0000 (17:42 +0000)]
search*: rename {partition} => {shard}

Another step towards keeping our internal data structures
consistent with Xapian naming.