Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rzSQu-0048hs-Ub for pgsql-hackers@arkaria.postgresql.org; Wed, 24 Apr 2024 02:32:29 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.94.2) (envelope-from ) id 1rzSQt-00BFX9-3O for pgsql-hackers@arkaria.postgresql.org; Wed, 24 Apr 2024 02:32:27 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rzSQs-00BFUv-Pd for pgsql-hackers@lists.postgresql.org; Wed, 24 Apr 2024 02:32:26 +0000 Received: from mail-lf1-x12d.google.com ([2a00:1450:4864:20::12d]) by makus.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 (Exim 4.94.2) (envelope-from ) id 1rzSQq-004LGD-8R for pgsql-hackers@postgresql.org; Wed, 24 Apr 2024 02:32:25 +0000 Received: by mail-lf1-x12d.google.com with SMTP id 2adb3069b0e04-518a56cdbcfso9827495e87.2 for ; Tue, 23 Apr 2024 19:32:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1713925941; x=1714530741; darn=postgresql.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=YJQk0i/hGgD4EQz733zPzj+yN/y3++Azh9didQ1HX5w=; b=k5PQalhVHDSFd6FVv9UIbMHfpRpeTjERy+AtM0F1KDC2gkXu4IU/GinsN7bSBEfTOI re2KlNaOjPwW1ArrT0V2/NqVzNpHE2b7Jh90Qf55bOSMDDHxwee5IrjAjRtMXRQ/59M4 mmUyVFUyRh8dhxqwpWOWBC1WQlATBVQxCqKAtpFRXY9GT/jOb69ygqIVcKYs8/rL2Q9e p+tDoKqH9KWrXICNN2/zmZqs+5EQgJJBGLydfYU4cIfwhbyebSXN7T6wJ2HR6QHechhP tRWG0oRfFYzUnxRKLxX6r6I2JYUQTCuzQ8+guzTAwzHDrx2mydBoNzOxiaMjQEOMGwKx eP5w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1713925941; x=1714530741; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=YJQk0i/hGgD4EQz733zPzj+yN/y3++Azh9didQ1HX5w=; b=ilk3cKdLqvkpMZ6p2j1C0wRygfAfHj4Q66uuuTIo1EQqn2dTuf/r0g7whHNwz9Epsy pJ/5YeLzB/MLdbrEL+x4kTdPrJDt+mSZgAD2sjTerehpejR17cXJ59jaYuWbvJcQLYPm TH89HzCC+HYObnIs1KzYZHZzMmGUGzJOcjTTBKKrQ5vFMxy72Nhac5zIM+9JO/xdTSkY kTv9nLOuanf2Wgqp1UBzc7QRDNaI84m+fLX5HPiGXLclpAh02N/hWFlPzCPLL9ysBZW7 s3/EM9AQjxRIHsWTvpspkjZbJ1jRGP7GLyLTHgUCZWBJrMAu4rsGHRxN1dM27o4k/rJO gS8w== X-Forwarded-Encrypted: i=1; AJvYcCV1zP2ff3wxlLoJU82WMglcrd3OmSVQGpBTirKQ1MJLYjuWHXn+7/LI8NaBKavyxPDo1+9/ZxbChBD6fUxI/pVkML+2BEKGRHHAq3gu X-Gm-Message-State: AOJu0Ywi4wBuXibNZt3XkEjvWbiTvrG5U6JwM6xUFDI1PJW8BLOUHhBI nCmFjTgkiqc2XNPuoNRi836l2emAzJXIAWeGKijNAUNJgsl8qB5RxYTgr6Y6rKcPshpOvPSttzc 14FlayiTx2p59+Y9TS3w7lPQpQC08ceGA X-Google-Smtp-Source: AGHT+IEtl3QVez1w/9SH56dZ2w39BJLboyGADOVw1yj160y588Ed/xmLP+QkKeBCuRjfhuBS5coiJiAkVeIeoK42SUI= X-Received: by 2002:ac2:58d9:0:b0:518:c8e1:478 with SMTP id u25-20020ac258d9000000b00518c8e10478mr945034lfo.58.1713925941417; Tue, 23 Apr 2024 19:32:21 -0700 (PDT) MIME-Version: 1.0 References: <56bc4ac4-9a9e-4f28-af1e-b6d537bd9a5e@iki.fi> <289a1c0e-8444-4009-a8c2-c2d77ced6f07@iki.fi> <98f01395-fc60-4d50-89c9-fc3011fa2a45@iki.fi> In-Reply-To: From: David Rowley Date: Wed, 24 Apr 2024 14:32:09 +1200 Message-ID: Subject: Re: Streaming I/O, vectored I/O (WIP) To: Nazir Bilal Yavuz Cc: Thomas Munro , Heikki Linnakangas , Dilip Kumar , Robert Haas , pgsql-hackers , Andres Freund , Melanie Plageman Content-Type: multipart/mixed; boundary="000000000000e8f0f10616ce7a2e" List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk --000000000000e8f0f10616ce7a2e Content-Type: text/plain; charset="UTF-8" I've attached a patch with a few typo fixes and what looks like an incorrect type for max_ios. It's an int16 and I think it needs to be an int. Doing "max_ios = Min(max_ios, PG_INT16_MAX);" doesn't do anything when max_ios is int16. David --000000000000e8f0f10616ce7a2e Content-Type: text/plain; charset="US-ASCII"; name="read_stream_fixes.patch" Content-Disposition: attachment; filename="read_stream_fixes.patch" Content-Transfer-Encoding: base64 Content-ID: X-Attachment-Id: f_lvd778h70 ZGlmZiAtLWdpdCBhL3NyYy9iYWNrZW5kL3N0b3JhZ2UvYWlvL3JlYWRfc3RyZWFtLmMgYi9zcmMv YmFja2VuZC9zdG9yYWdlL2Fpby9yZWFkX3N0cmVhbS5jCmluZGV4IDYzNGNmNGYwZDEuLjc0Yjli YWU2MzEgMTAwNjQ0Ci0tLSBhL3NyYy9iYWNrZW5kL3N0b3JhZ2UvYWlvL3JlYWRfc3RyZWFtLmMK KysrIGIvc3JjL2JhY2tlbmQvc3RvcmFnZS9haW8vcmVhZF9zdHJlYW0uYwpAQCAtMjYsMTIgKzI2 LDEyIEBACiAgKgogICogQikgSS9PIGlzIG5lY2Vzc2FyeSwgYnV0IGZhZHZpc2UgaXMgdW5kZXNp cmFibGUgYmVjYXVzZSB0aGUgYWNjZXNzIGlzCiAgKiBzZXF1ZW50aWFsLCBvciBpbXBvc3NpYmxl IGJlY2F1c2UgZGlyZWN0IEkvTyBpcyBlbmFibGVkIG9yIHRoZSBzeXN0ZW0KLSAqIGRvZXNuJ3Qg c3VwcG9ydCBhZHZpY2UuICBUaGVyZSBpcyBubyBiZW5lZml0IGluIGxvb2tpbmcgYWhlYWQgbW9y ZSB0aGFuCi0gKiBpb19jb21iaW5lX2xpbWl0LCBiZWNhdXNlIGluIHRoaXMgY2FzZSBvbmx5IGdv YWwgaXMgbGFyZ2VyIHJlYWQgc3lzdGVtCisgKiBkb2Vzbid0IHN1cHBvcnQgZmFkdmlzZS4gIFRo ZXJlIGlzIG5vIGJlbmVmaXQgaW4gbG9va2luZyBhaGVhZCBtb3JlIHRoYW4KKyAqIGlvX2NvbWJp bmVfbGltaXQsIGJlY2F1c2UgaW4gdGhpcyBjYXNlIHRoZSBvbmx5IGdvYWwgaXMgbGFyZ2VyIHJl YWQgc3lzdGVtCiAgKiBjYWxscy4gIExvb2tpbmcgZnVydGhlciBhaGVhZCB3b3VsZCBwaW4gbWFu eSBidWZmZXJzIGFuZCBwZXJmb3JtCiAgKiBzcGVjdWxhdGl2ZSB3b3JrIGxvb2tpbmcgYWhlYWQg Zm9yIG5vIGJlbmVmaXQuCiAgKgotICogQykgSS9PIGlzIG5lY2Vzc3NhcnksIGl0IGFwcGVhcnMg cmFuZG9tLCBhbmQgdGhpcyBzeXN0ZW0gc3VwcG9ydHMgZmFkdmlzZS4KKyAqIEMpIEkvTyBpcyBu ZWNlc3NhcnksIGl0IGFwcGVhcnMgcmFuZG9tLCBhbmQgdGhpcyBzeXN0ZW0gc3VwcG9ydHMgZmFk dmlzZS4KICAqIFdlJ2xsIGxvb2sgZnVydGhlciBhaGVhZCBpbiBvcmRlciB0byByZWFjaCB0aGUg Y29uZmlndXJlZCBsZXZlbCBvZiBJL08KICAqIGNvbmN1cnJlbmN5LgogICoKQEAgLTQxOCw3ICs0 MTgsNyBAQCByZWFkX3N0cmVhbV9iZWdpbl9yZWxhdGlvbihpbnQgZmxhZ3MsCiAJUmVhZFN0cmVh bSAqc3RyZWFtOwogCXNpemVfdAkJc2l6ZTsKIAlpbnQxNgkJcXVldWVfc2l6ZTsKLQlpbnQxNgkJ bWF4X2lvczsKKwlpbnQJCQltYXhfaW9zOwogCWludAkJCXN0cmF0ZWd5X3Bpbl9saW1pdDsKIAl1 aW50MzIJCW1heF9waW5uZWRfYnVmZmVyczsKIAlPaWQJCQl0YWJsZXNwYWNlX2lkOwpAQCAtNDQ3 LDYgKzQ0Nyw4IEBAIHJlYWRfc3RyZWFtX2JlZ2luX3JlbGF0aW9uKGludCBmbGFncywKIAkJbWF4 X2lvcyA9IGdldF90YWJsZXNwYWNlX21haW50ZW5hbmNlX2lvX2NvbmN1cnJlbmN5KHRhYmxlc3Bh Y2VfaWQpOwogCWVsc2UKIAkJbWF4X2lvcyA9IGdldF90YWJsZXNwYWNlX2lvX2NvbmN1cnJlbmN5 KHRhYmxlc3BhY2VfaWQpOworCisJLyogQ2FwIHRvIElOVDE2X01BWCB0byBhdm9pZCBvdmVyZmxv d2luZyBiZWxvdyAqLwogCW1heF9pb3MgPSBNaW4obWF4X2lvcywgUEdfSU5UMTZfTUFYKTsKIAog CS8qCg== --000000000000e8f0f10616ce7a2e--