Re: Patch: Improve HTTP time by sending Connection:close

2016-11-17 Thread Konrad Rzeszutek Wilk
On Thu, Nov 17, 2016 at 07:32:16PM +0100, Daniel Kiper wrote: > On Tue, Nov 15, 2016 at 01:43:16PM -0800, Walter Huf wrote: > > GRUB has a bug where it waits a minimum of 400ms for every file it fetches > > over HTTP, unless the server serves it with Transfer-Encoding:chunked or > > the file just h

Re: Patch: Improve HTTP time by sending Connection:close

2016-11-17 Thread Daniel Kiper
On Tue, Nov 15, 2016 at 01:43:16PM -0800, Walter Huf wrote: > GRUB has a bug where it waits a minimum of 400ms for every file it fetches > over HTTP, unless the server serves it with Transfer-Encoding:chunked or > the file just happens to be split into 20 TCP packets. When using > pxeboot.img built

Re: Patch: Improve HTTP time by sending Connection:close

2016-11-15 Thread Andrei Borzenkov
16.11.2016 00:43, Walter Huf пишет: > The SOB (statement of benefit?) of this patch It is "Signed-off-by: xxx" line that are required by some projects (first appeared in Linux kernel I think). OTOH some projects actively reject it. ___ Grub-devel mailin

Re: Patch: Improve HTTP time by sending Connection:close

2016-11-15 Thread Walter Huf
GRUB has a bug where it waits a minimum of 400ms for every file it fetches over HTTP, unless the server serves it with Transfer-Encoding:chunked or the file just happens to be split into 20 TCP packets. When using pxeboot.img built with just pxe and http module (following instructions from https://

Re: Patch: Improve HTTP time by sending Connection:close

2016-11-15 Thread Daniel Kiper
On Tue, Nov 15, 2016 at 05:30:28PM +0300, Andrei Borzenkov wrote: > On Tue, Nov 15, 2016 at 4:21 PM, Daniel Kiper wrote: > > On Sat, Nov 12, 2016 at 09:26:17AM -0800, Walter Huf wrote: > >> GRUB's HTTP module declares support for HTTP/1.1, which defaults to > >> Connection:keepalive. At the end of

Re: Patch: Improve HTTP time by sending Connection:close

2016-11-15 Thread Andrei Borzenkov
On Tue, Nov 15, 2016 at 4:21 PM, Daniel Kiper wrote: > On Sat, Nov 12, 2016 at 09:26:17AM -0800, Walter Huf wrote: >> GRUB's HTTP module declares support for HTTP/1.1, which defaults to >> Connection:keepalive. At the end of the content, the server holds the TCP >> connection open waiting for the

Re: Patch: Improve HTTP time by sending Connection:close

2016-11-15 Thread Daniel Kiper
On Sat, Nov 12, 2016 at 09:26:17AM -0800, Walter Huf wrote: > GRUB's HTTP module declares support for HTTP/1.1, which defaults to > Connection:keepalive. At the end of the content, the server holds the TCP > connection open waiting for the next request. > It seems that grub_net_poll_cards() is watc

Patch: Improve HTTP time by sending Connection:close

2016-11-12 Thread Walter Huf
GRUB's HTTP module declares support for HTTP/1.1, which defaults to Connection:keepalive. At the end of the content, the server holds the TCP connection open waiting for the next request. It seems that grub_net_poll_cards() is watching for the HTTP module to set net->stall, and otherwise waits the