Re: [PATCH v2 0/11] limit memory allocations for v2 servers

From: Taylor Blau <me@ttaylorr.com>
Date: Sep 15, 2021, 04:17 UTC
Message-ID: <YUFz6RqVaBunxJ7y@nand.local>
In-Reply-To: <YUE1alo58cGyTw6/@coredump.intra.peff.net>

On Tue, Sep 14, 2021 at 07:51:06PM -0400, Jeff King wrote:

Show 10 quoted lines

> Here's a re-roll of my series to limit the memory a v2 server is willing
> to allocate on behalf of a client. See v1:
>
>   https://lore.kernel.org/git/YUC%2F6n1hhUbMJiLw@coredump.intra.peff.net/
>
> for an overview. The existing patches are mostly small fixups pointed
> out by reviewers (thanks!), but I did take Martin's TOO_MANY_PREFIXES
> suggestion in patch 7 (without the change to the name of the constant it
> seemed too clever to me, but with it it seems just the right amount of
> clever).

Thanks for this. The two new patches look good to me, and I took a light skim over the ones which were modified since v1. Everything I saw seemed very reasonable to me.

    Reviewed-by: Taylor Blau <me@ttaylorr.com>

Thanks, Taylor