On Sat, 2016-07-23 at 03:10 -0400, Ray Satiro wrote:
> Yes, we are referring to a git bisect of the curl repository https://
> github.com/curl/curl.git

Thank you for the clarification and detailed instructions. I took the
original request to do a bisect in the general sense (hadn't heard of
git bisect).

I performed the bisect and here is the offending commit that causes
performance issue and eventual dowload timeout:
# curl -V
790d6de485154440d4f8822a54641ee00a0d09f3 is the first bad commit
commit 790d6de485154440d4f8822a54641ee00a0d09f3
Author: Daniel Stenberg <[email protected]>
Date:   Sun Sep 27 20:48:35 2015 +0200

    readwrite_data: set a max number of loops

    ... as otherwise a really fast pipe can "lock" one transfer for
some
    protocols, like with HTTP/2.

:040000 040000 be90c05ef52f4836905b3d25a7a22f0f2a690ef0
8829a22fc115f125e3d9d0282b85da987518d80a M  lib

Offending version:
curl 7.45.0-DEV (powerpc-pw_uniran-linux-gnuspe) libcurl/7.45.0-DEV
zlib/1.2.7 libssh2/1.4.3
Protocols: dict file ftp gopher http imap pop3 rtsp scp sftp smtp
telnet tftp 
Features: Largefile libz UnixSockets 

I've also attached the bisect log in case you're interested.
git bisect start
# bad: [cf93a7b364a70b56150cf6ea77492b799ec02a45] RELEASE-NOTES: 7.49.1
git bisect bad cf93a7b364a70b56150cf6ea77492b799ec02a45
# good: [1a7f66a3de2625d10f65415e6eb3e56067dc0555] RELEASE-NOTES: synced with c75a1e775061
git bisect good 1a7f66a3de2625d10f65415e6eb3e56067dc0555
# bad: [078753c40d05fc4a07ee20a18f60fc325592b31c] gtls: fix for builds lacking encrypted key file support
git bisect bad 078753c40d05fc4a07ee20a18f60fc325592b31c
# skip: [0755308a6a210fdb886ea127dfa6b6d2359d7f9c] openssl: remove 0.9.3 check
git bisect skip 0755308a6a210fdb886ea127dfa6b6d2359d7f9c
# skip: [1ea3a7d5e4a9d273351b68ad3799aec18bb0ccea] RELEASE-NOTES: synced with cba5bc585410
git bisect skip 1ea3a7d5e4a9d273351b68ad3799aec18bb0ccea
# skip: [53d1e42c41e15171387a17bd5ad178c7d249007f] FAQ: Update to point to GitHub
git bisect skip 53d1e42c41e15171387a17bd5ad178c7d249007f
# skip: [60c86297bf81235c18eec68f7ddf1a30f21aa1b7] tftp tests: verify sent options too
git bisect skip 60c86297bf81235c18eec68f7ddf1a30f21aa1b7
# skip: [35083ca60ed035a4a097c003a339d92b69f3d87c] openssl: improved error detection/reporting
git bisect skip 35083ca60ed035a4a097c003a339d92b69f3d87c
# skip: [23c4090fd37ad46a1183c79fbdf45c45aff545ff] imap: Quote other 'atom-specials' and not just the space character
git bisect skip 23c4090fd37ad46a1183c79fbdf45c45aff545ff
# good: [cdba82e1c29b6d372c3c72ef091e336cba98debe] opts: 19 more CURLINFO_* options made into stand-alone man pages
git bisect good cdba82e1c29b6d372c3c72ef091e336cba98debe
# skip: [98acebb5261eb3b272dd74145081f729d032d691] TODO: TCP Fast Open
git bisect skip 98acebb5261eb3b272dd74145081f729d032d691
# skip: [9e093f0e74f4a83f14a809a573b5e592f304f8e1] test 1326: fix getting stuck on Windows due to incomplete request
git bisect skip 9e093f0e74f4a83f14a809a573b5e592f304f8e1
# skip: [716302c2cd59f96ecd96f949db92576d204cabae] mbedtls: fix ALPN usage segfault
git bisect skip 716302c2cd59f96ecd96f949db92576d204cabae
# skip: [a7c4bcd97109b42a665fcef0243173dc7db63f89] runtests: more compact "System characteristics" output
git bisect skip a7c4bcd97109b42a665fcef0243173dc7db63f89
# bad: [18c735e790e47a1199f9dd71a01aa9847d6474b1] cookies: allow spaces in cookie names, cut of trailing spaces
git bisect bad 18c735e790e47a1199f9dd71a01aa9847d6474b1
# bad: [41efdadf094d1dede5d3f8d6c9be7b8cf98dd894] pop3: Differentiate between success and continuation responses
git bisect bad 41efdadf094d1dede5d3f8d6c9be7b8cf98dd894
# bad: [529f9310b1febcd3626aefbc598032572efb248a] ftp: allow CURLOPT_IGNORE_CONTENT_LENGTH to ignore size
git bisect bad 529f9310b1febcd3626aefbc598032572efb248a
# bad: [2eb4f5efe9998b9eb0eda94beb713e89d20fb342] tool_setopt: fix c_escape truncated octal
git bisect bad 2eb4f5efe9998b9eb0eda94beb713e89d20fb342
# good: [0583ed3e48be3fcbccf77dc5ac571ba442d203cb] CURLOPT_PROXY.3: A proxy given as env variable gets no special treatment
git bisect good 0583ed3e48be3fcbccf77dc5ac571ba442d203cb
# good: [119037325de02579f5c58256ca2ed2a0aa592c86] build: fix failures with -Wcast-align and -Werror
git bisect good 119037325de02579f5c58256ca2ed2a0aa592c86
# bad: [963b7bd4f76fd0e19beaa202bf6f5239eb6153a5] docs: fix unescaped '\n' in man pages
git bisect bad 963b7bd4f76fd0e19beaa202bf6f5239eb6153a5
# bad: [790d6de485154440d4f8822a54641ee00a0d09f3] readwrite_data: set a max number of loops
git bisect bad 790d6de485154440d4f8822a54641ee00a0d09f3
# good: [32fc638ddf046d501abc2a58c91c3e934caf303a] FTP: fix uploading ASCII with unknown size
git bisect good 32fc638ddf046d501abc2a58c91c3e934caf303a
# good: [4f037367ed69b77e14953fe88d8c0497782c0aab] CI: Added AppVeyor-CI for curl
git bisect good 4f037367ed69b77e14953fe88d8c0497782c0aab
# first bad commit: [790d6de485154440d4f8822a54641ee00a0d09f3] readwrite_data: set a max number of loops
# good: [4f037367ed69b77e14953fe88d8c0497782c0aab] CI: Added AppVeyor-CI for curl
git bisect good 4f037367ed69b77e14953fe88d8c0497782c0aab
# first bad commit: [790d6de485154440d4f8822a54641ee00a0d09f3] readwrite_data: set a max number of loops
# good: [4f037367ed69b77e14953fe88d8c0497782c0aab] CI: Added AppVeyor-CI for curl
git bisect good 4f037367ed69b77e14953fe88d8c0497782c0aab
# first bad commit: [790d6de485154440d4f8822a54641ee00a0d09f3] readwrite_data: set a max number of loops
# good: [4f037367ed69b77e14953fe88d8c0497782c0aab] CI: Added AppVeyor-CI for curl
git bisect good 4f037367ed69b77e14953fe88d8c0497782c0aab
# first bad commit: [790d6de485154440d4f8822a54641ee00a0d09f3] readwrite_data: set a max number of loops
-------------------------------------------------------------------
List admin: https://cool.haxx.se/list/listinfo/curl-library
Etiquette:  https://curl.haxx.se/mail/etiquette.html

Reply via email to