Re: [Nbd] Proposal to merge WRITE_ZEROES extension into master
- To: Wouter Verhelst <w@...112...>
- Cc: "nbd-general@lists.sourceforge.net" <nbd-general@lists.sourceforge.net>
- Subject: Re: [Nbd] Proposal to merge WRITE_ZEROES extension into master
- From: Alex Bligh <alex@...872...>
- Date: Tue, 20 Dec 2016 09:20:58 +0000
- Message-id: <6C4590FF-C978-4CD3-B8AC-8C9A7729B06F@...872...>
- In-reply-to: <20161220082518.v27kqq63csmqhv4m@...3...>
- References: <BC48D5D1-550F-4EE3-BC84-623AE864FB3A@...872...> <20161214181434.ko22eeq6vnwpakh2@...3...> <75E8140C-3000-4500-89A4-71A26040ADE4@...872...> <0132c6df-dbcf-b55a-6b56-f3279c825f1a@...696...> <4888CDE5-8CAD-4F0F-ABBC-6C904EC96ED0@...872...> <71fc32d7-2e5b-28c8-ec45-893daf5710d7@...696...> <FE824089-E29F-4894-B9CC-6A3CBC421CD6@...872...> <20161215110942.6xepfmak53kgr4wc@...3...> <20161219214143.jhgno7krc7fpgmjy@...3...> <c085204a-0db4-fa6a-4c39-cbb98ab3a0d9@...696...> <20161220082518.v27kqq63csmqhv4m@...3...>
> On 20 Dec 2016, at 08:25, Wouter Verhelst <w@...112...> wrote:
>
> diff --git a/nbd-server.c b/nbd-server.c
> index cbe7e57..45cb591 100644
> --- a/nbd-server.c
> +++ b/nbd-server.c
> @@ -2025,7 +2025,7 @@ static void handle_write_zeroes(CLIENT* client, struct nbd_request* req) {
> // TODO: handle this far more efficiently with reference to the
> // actual backing driver
> pthread_mutex_lock(&(client->lock));
> - writeit(client->net, &rep, sizeof rep);
> + socket_write(client, &rep, sizeof rep);
> pthread_mutex_unlock(&(client->lock));
> }
>
> I'm going to have to do a 3.15.1 for that, otherwise WRITE_ZEROES
> handling will be broken with TLS enabled.
Sorry, that was me I think (possibly or possibly not in addition
to Eric). Must have come from a merge error / thinko.
--
Alex Bligh
Reply to: