guix-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

01/05: ssh: Work around 'get-bytevector-some' bug.


From: Ludovic Courtès
Subject: 01/05: ssh: Work around 'get-bytevector-some' bug.
Date: Wed, 10 Jan 2018 18:01:26 -0500 (EST)

civodul pushed a commit to branch master
in repository guix.

commit 17af5d51de7c40756a4a39d336f81681de2ba447
Author: Ludovic Courtès <address@hidden>
Date:   Wed Jan 10 17:52:23 2018 +0100

    ssh: Work around 'get-bytevector-some' bug.
    
    This works around <https://bugs.gnu.org/30066> and noticeably improves
    performance when using GUIX_DAEMON_SOCKET=ssh://HOST (the redirect code
    was transferring data to guix-daemon one byte at a time!).
    
    * guix/ssh.scm (remote-daemon-channel)[redirect]: Define 'read!' and use
    it instead of 'get-bytevector-some'.
---
 guix/ssh.scm | 33 +++++++++++++++++++++++----------
 1 file changed, 23 insertions(+), 10 deletions(-)

diff --git a/guix/ssh.scm b/guix/ssh.scm
index 469f4fa..96e4af9 100644
--- a/guix/ssh.scm
+++ b/guix/ssh.scm
@@ -101,11 +101,24 @@ Throw an error on failure."
     ;; Unix-domain sockets but libssh doesn't have an API for that, hence this
     ;; hack.
     `(begin
-       (use-modules (ice-9 match) (rnrs io ports))
+       (use-modules (ice-9 match) (rnrs io ports)
+                    (rnrs bytevectors) (system foreign))
+
+       (define read!
+         ;; XXX: We would use 'get-bytevector-some' but it always returns a
+         ;; single byte in Guile <= 2.2.3---see <https://bugs.gnu.org/30066>.
+         ;; This procedure works around it.
+         (let ((proc (pointer->procedure int
+                                         (dynamic-func "read" (dynamic-link))
+                                         (list int '* size_t))))
+           (lambda (port bv)
+             (proc (fileno port) (bytevector->pointer bv)
+                   (bytevector-length bv)))))
 
        (let ((sock   (socket AF_UNIX SOCK_STREAM 0))
              (stdin  (current-input-port))
-             (stdout (current-output-port)))
+             (stdout (current-output-port))
+             (buffer (make-bytevector 65536)))
          (setvbuf stdin _IONBF)
          (setvbuf stdout _IONBF)
          (connect sock AF_UNIX ,socket-name)
@@ -114,17 +127,17 @@ Throw an error on failure."
            (match (select (list stdin sock) '() (list stdin stdout sock))
              ((reads writes ())
               (when (memq stdin reads)
-                (match (get-bytevector-some stdin)
-                  ((? eof-object?)
+                (match (read! stdin buffer)
+                  ((? zero?)                      ;EOF
                    (primitive-exit 0))
-                  (bv
-                   (put-bytevector sock bv))))
+                  (count
+                   (put-bytevector sock buffer 0 count))))
               (when (memq sock reads)
-                (match (get-bytevector-some sock)
-                  ((? eof-object?)
+                (match (read! sock buffer)
+                  ((? zero?)                      ;EOF
                    (primitive-exit 0))
-                  (bv
-                   (put-bytevector stdout bv))))
+                  (count
+                   (put-bytevector stdout buffer 0 count))))
               (loop))
              (_
               (primitive-exit 1)))))))



reply via email to

[Prev in Thread] Current Thread [Next in Thread]