{"id":2231429,"url":"http://patchwork.ozlabs.org/api/patches/2231429/?format=json","web_url":"http://patchwork.ozlabs.org/project/ltp/patch/20260430-safe_splice-v1-2-d1d919ff2a3a@suse.com/","project":{"id":59,"url":"http://patchwork.ozlabs.org/api/projects/59/?format=json","name":"Linux Test Project development","link_name":"ltp","list_id":"ltp.lists.linux.it","list_email":"ltp@lists.linux.it","web_url":"","scm_url":"","webscm_url":"","list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20260430-safe_splice-v1-2-d1d919ff2a3a@suse.com>","list_archive_url":null,"date":"2026-04-30T17:25:30","name":"[2/2] tee01, vmsplice01, af_alg08: Use SAFE_SPLICE()","commit_ref":null,"pull_url":null,"state":"needs-review-ack","archived":false,"hash":"3be0678b7ac89d20cdffbd5a0da2a8e73fd09089","submitter":{"id":83220,"url":"http://patchwork.ozlabs.org/api/people/83220/?format=json","name":"Andrea Cervesato","email":"andrea.cervesato@suse.de"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/ltp/patch/20260430-safe_splice-v1-2-d1d919ff2a3a@suse.com/mbox/","series":[{"id":502349,"url":"http://patchwork.ozlabs.org/api/series/502349/?format=json","web_url":"http://patchwork.ozlabs.org/project/ltp/list/?series=502349","date":"2026-04-30T17:25:29","name":"Introduce SAFE_SPLICE()","version":1,"mbox":"http://patchwork.ozlabs.org/series/502349/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/2231429/comments/","check":"fail","checks":"http://patchwork.ozlabs.org/api/patches/2231429/checks/","tags":{},"related":[],"headers":{"Return-Path":"","X-Original-To":["incoming@patchwork.ozlabs.org","ltp@lists.linux.it"],"Delivered-To":["patchwork-incoming@legolas.ozlabs.org","ltp@picard.linux.it"],"Authentication-Results":["legolas.ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n unprotected) header.d=suse.de header.i=@suse.de header.a=rsa-sha256\n header.s=susede2_rsa header.b=U65SXpYb;\n\tdkim=fail reason=\"signature verification failed\" header.d=suse.de\n header.i=@suse.de header.a=ed25519-sha256 header.s=susede2_ed25519\n header.b=FGMs12Zj;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key)\n header.d=suse.de header.i=@suse.de header.a=rsa-sha256 header.s=susede2_rsa\n header.b=U65SXpYb;\n\tdkim=neutral header.d=suse.de header.i=@suse.de header.a=ed25519-sha256\n header.s=susede2_ed25519 header.b=FGMs12Zj;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=lists.linux.it\n (client-ip=213.254.12.146; helo=picard.linux.it;\n envelope-from=ltp-bounces+incoming=patchwork.ozlabs.org@lists.linux.it;\n receiver=patchwork.ozlabs.org)","smtp-out1.suse.de;\n\tnone"],"Received":["from picard.linux.it (picard.linux.it [213.254.12.146])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4g61L16M40z1yHZ\n\tfor ; Fri, 01 May 2026 03:26:17 +1000 (AEST)","from picard.linux.it (localhost [IPv6:::1])\n\tby picard.linux.it (Postfix) with ESMTP id AB72F3E4ED4\n\tfor ; Thu, 30 Apr 2026 19:26:15 +0200 (CEST)","from in-5.smtp.seeweb.it (in-5.smtp.seeweb.it\n [IPv6:2001:4b78:1:20::5])\n (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n key-exchange X25519 server-signature ECDSA (secp384r1))\n (No client certificate requested)\n by picard.linux.it (Postfix) with ESMTPS id EC40A3E6CA4\n for ; Thu, 30 Apr 2026 19:25:37 +0200 (CEST)","from smtp-out1.suse.de (smtp-out1.suse.de [195.135.223.130])\n (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest\n SHA256)\n (No client certificate requested)\n by in-5.smtp.seeweb.it (Postfix) with ESMTPS id 77991601403\n for ; Thu, 30 Apr 2026 19:25:37 +0200 (CEST)","from imap1.dmz-prg2.suse.org (unknown [10.150.64.97])\n (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest\n SHA256)\n (No client certificate requested)\n by smtp-out1.suse.de (Postfix) with ESMTPS id D37946A7DA;\n Thu, 30 Apr 2026 17:25:30 +0000 (UTC)","from imap1.dmz-prg2.suse.org (localhost [127.0.0.1])\n (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest\n SHA256)\n (No client certificate requested)\n by imap1.dmz-prg2.suse.org (Postfix) with ESMTPS id BADB4593B2;\n Thu, 30 Apr 2026 17:25:30 +0000 (UTC)","from dovecot-director2.suse.de ([2a07:de40:b281:106:10:150:64:167])\n by imap1.dmz-prg2.suse.org with ESMTPSA id wHXuK4qQ82kMIQAAD6G6ig\n (envelope-from ); Thu, 30 Apr 2026 17:25:30 +0000"],"DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de;\n s=susede2_rsa;\n t=1777569930;\n h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc:\n mime-version:mime-version:content-type:content-type:\n content-transfer-encoding:content-transfer-encoding:\n in-reply-to:in-reply-to:references:references;\n bh=se1LAEMQtEzNUPgG/AkcKNaW30iUlOUHrWY4GfiM/qk=;\n b=U65SXpYbI2dhPDuQsPDJN02u+LfRWIzAUObwe0YZYLvIBYpFZ46WL+AwZWjHRLt4BqI8gQ\n FmqqNRAYbS8Cr2ez8Fg6tLhYdImV4fXRQxkKXLbVL2Ju1wK3m7DnG0eR7vAd80ll86Vusv\n Fj+WQ8cjdiwjrTWL90J34bi42VU+bpo=","v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de;\n s=susede2_ed25519; t=1777569930;\n h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc:\n mime-version:mime-version:content-type:content-type:\n content-transfer-encoding:content-transfer-encoding:\n in-reply-to:in-reply-to:references:references;\n bh=se1LAEMQtEzNUPgG/AkcKNaW30iUlOUHrWY4GfiM/qk=;\n b=FGMs12Zj19gTRJzLc8/dYzVFc+ckMJasUxKRPerd0+h0otA9McxRGH2ona5bp9ejuQOP+m\n HPlLOfZWMZ9j14Aw==","v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de;\n s=susede2_rsa;\n t=1777569930;\n h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc:\n mime-version:mime-version:content-type:content-type:\n content-transfer-encoding:content-transfer-encoding:\n in-reply-to:in-reply-to:references:references;\n bh=se1LAEMQtEzNUPgG/AkcKNaW30iUlOUHrWY4GfiM/qk=;\n b=U65SXpYbI2dhPDuQsPDJN02u+LfRWIzAUObwe0YZYLvIBYpFZ46WL+AwZWjHRLt4BqI8gQ\n FmqqNRAYbS8Cr2ez8Fg6tLhYdImV4fXRQxkKXLbVL2Ju1wK3m7DnG0eR7vAd80ll86Vusv\n Fj+WQ8cjdiwjrTWL90J34bi42VU+bpo=","v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de;\n s=susede2_ed25519; t=1777569930;\n h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc:\n mime-version:mime-version:content-type:content-type:\n content-transfer-encoding:content-transfer-encoding:\n in-reply-to:in-reply-to:references:references;\n bh=se1LAEMQtEzNUPgG/AkcKNaW30iUlOUHrWY4GfiM/qk=;\n b=FGMs12Zj19gTRJzLc8/dYzVFc+ckMJasUxKRPerd0+h0otA9McxRGH2ona5bp9ejuQOP+m\n HPlLOfZWMZ9j14Aw=="],"From":"Andrea Cervesato ","Date":"Thu, 30 Apr 2026 19:25:30 +0200","MIME-Version":"1.0","Message-Id":"<20260430-safe_splice-v1-2-d1d919ff2a3a@suse.com>","References":"<20260430-safe_splice-v1-0-d1d919ff2a3a@suse.com>","In-Reply-To":"<20260430-safe_splice-v1-0-d1d919ff2a3a@suse.com>","To":"Linux Test Project ","X-Mailer":"b4 0.14.2","X-Developer-Signature":"v=1; a=ed25519-sha256; t=1777569930; l=3692;\n i=andrea.cervesato@suse.com; s=20251210; h=from:subject:message-id;\n bh=uDWFnnpZAVx7JYgujAotBUSjSCUQoumbX4pmce/tR0E=;\n b=GReNfeB3RDL1uMY3RDczhPMIuxXUILf+P7OnlJf1gIYCtNSoM1CcBIBDoKWcDI1ENOtmCkPxt\n klgDh8C1AXpDtvcTsql5QFcNXx52hj1w74mlCe4BNe85vc4M22Sekbt","X-Developer-Key":"i=andrea.cervesato@suse.com; a=ed25519;\n pk=zKY+6GCauOiuHNZ//d8PQ/UL4jFCTKbXrzXAOQSLevI=","X-Spamd-Result":"default: False [-4.30 / 50.00]; BAYES_HAM(-3.00)[100.00%];\n NEURAL_HAM_LONG(-1.00)[-1.000];\n NEURAL_HAM_SHORT(-0.20)[-0.999]; MIME_GOOD(-0.10)[text/plain];\n FUZZY_RATELIMITED(0.00)[rspamd.com];\n RCVD_VIA_SMTP_AUTH(0.00)[]; ARC_NA(0.00)[];\n RCPT_COUNT_TWO(0.00)[2]; RCVD_TLS_ALL(0.00)[];\n DKIM_SIGNED(0.00)[suse.de:s=susede2_rsa,suse.de:s=susede2_ed25519];\n TO_DN_ALL(0.00)[]; FROM_HAS_DN(0.00)[]; MIME_TRACE(0.00)[0:+];\n FROM_EQ_ENVFROM(0.00)[]; TO_MATCH_ENVRCPT_ALL(0.00)[];\n RCVD_COUNT_TWO(0.00)[2];\n DBL_BLOCKED_OPENRESOLVER(0.00)[imap1.dmz-prg2.suse.org:helo, suse.com:mid,\n suse.com:email]","X-Spam-Score":"-4.30","X-Spam-Level":"","X-Spam-Status":"No, score=0.1 required=7.0 tests=DKIM_SIGNED,DKIM_VALID,\n DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_PASS shortcircuit=no\n autolearn=disabled version=4.0.1","X-Spam-Checker-Version":"SpamAssassin 4.0.1 (2024-03-25) on in-5.smtp.seeweb.it","X-Virus-Scanned":"clamav-milter 1.0.9 at in-5.smtp.seeweb.it","X-Virus-Status":"Clean","Subject":"[LTP] [PATCH 2/2] tee01, vmsplice01, af_alg08: Use SAFE_SPLICE()","X-BeenThere":"ltp@lists.linux.it","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"Linux Test Project ","List-Unsubscribe":",\n ","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n ","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"ltp-bounces+incoming=patchwork.ozlabs.org@lists.linux.it","Sender":"\"ltp\" "},"content":"From: Andrea Cervesato \n\nReplace raw splice() calls with manual error checking by\nSAFE_SPLICE() in tests where splice is not the subject of\ntesting but just utility plumbing.\n\nSigned-off-by: Andrea Cervesato \n---\n testcases/kernel/crypto/af_alg08.c | 10 ++--------\n testcases/kernel/syscalls/tee/tee01.c | 9 ++-------\n testcases/kernel/syscalls/vmsplice/vmsplice01.c | 6 +-----\n 3 files changed, 5 insertions(+), 20 deletions(-)","diff":"diff --git a/testcases/kernel/crypto/af_alg08.c b/testcases/kernel/crypto/af_alg08.c\nindex 4f27f3c9da7cc277926c6fe3eee1a4649a483763..05f17eea4797b7575c97e194e34c0390c6d6be22 100644\n--- a/testcases/kernel/crypto/af_alg08.c\n+++ b/testcases/kernel/crypto/af_alg08.c\n@@ -27,7 +27,6 @@\n #include \"tst_test.h\"\n #include \"tst_af_alg.h\"\n #include \"lapi/socket.h\"\n-#include \"lapi/splice.h\"\n \n #define TESTFILE \"copy_fail\"\n #define OVERWRITE_SIZE 4\n@@ -92,13 +91,8 @@ static void try_corrupt(void)\n \n \tSAFE_PIPE(pipefd);\n \n-\tTEST(splice(file_fd, &off_in, pipefd[1], NULL, OVERWRITE_SIZE, 0));\n-\tif (TST_RET < 0)\n-\t\ttst_brk(TBROK | TTERRNO, \"splice(file -> pipe)\");\n-\n-\tTEST(splice(pipefd[0], NULL, reqfd, NULL, OVERWRITE_SIZE, 0));\n-\tif (TST_RET < 0)\n-\t\ttst_brk(TBROK | TTERRNO, \"splice(pipe -> AF_ALG)\");\n+\tSAFE_SPLICE(file_fd, &off_in, pipefd[1], NULL, OVERWRITE_SIZE, 0);\n+\tSAFE_SPLICE(pipefd[0], NULL, reqfd, NULL, OVERWRITE_SIZE, 0);\n \n \t/* Expected to fail (invalid ciphertext); triggers the scratch write */\n \tTST_EXP_FAIL_SILENT(recv(reqfd, recvbuf, sizeof(recvbuf), 0), EBADMSG);\ndiff --git a/testcases/kernel/syscalls/tee/tee01.c b/testcases/kernel/syscalls/tee/tee01.c\nindex d1489d0453f11faab41ab6e8e2dabbb372566f60..680793eb2b21cfd2609492d3be3bef10abb25acf 100644\n--- a/testcases/kernel/syscalls/tee/tee01.c\n+++ b/testcases/kernel/syscalls/tee/tee01.c\n@@ -15,7 +15,6 @@\n #include \"tst_test.h\"\n #include \"lapi/fcntl.h\"\n #include \"lapi/tee.h\"\n-#include \"lapi/splice.h\"\n \n #define TEST_BLOCK_SIZE 1024\n \n@@ -58,17 +57,13 @@ static void tee_test(void)\n \tSAFE_PIPE(pipe1);\n \tSAFE_PIPE(pipe2);\n \n-\tret = splice(fd_in, NULL, pipe1[1], NULL, TEST_BLOCK_SIZE, 0);\n-\tif (ret < 0)\n-\t\ttst_brk(TBROK | TERRNO, \"splice(fd_in, pipe1) failed\");\n+\tSAFE_SPLICE(fd_in, NULL, pipe1[1], NULL, TEST_BLOCK_SIZE, 0);\n \n \tret = tee(pipe1[0], pipe2[1], TEST_BLOCK_SIZE, SPLICE_F_NONBLOCK);\n \tif (ret < 0)\n \t\ttst_brk(TBROK | TERRNO, \"tee() failed\");\n \n-\tret = splice(pipe2[0], NULL, fd_out, NULL, TEST_BLOCK_SIZE, 0);\n-\tif (ret < 0)\n-\t\ttst_brk(TBROK | TERRNO, \"splice(pipe2, fd_out) failed\");\n+\tSAFE_SPLICE(pipe2[0], NULL, fd_out, NULL, TEST_BLOCK_SIZE, 0);\n \n \tSAFE_CLOSE(pipe2[0]);\n \tSAFE_CLOSE(pipe2[1]);\ndiff --git a/testcases/kernel/syscalls/vmsplice/vmsplice01.c b/testcases/kernel/syscalls/vmsplice/vmsplice01.c\nindex 17486179baa2a897c2efbf0d2b4d39273788aadc..7b55e66a373d2d72824fbb9146b2af353a5ff1c9 100644\n--- a/testcases/kernel/syscalls/vmsplice/vmsplice01.c\n+++ b/testcases/kernel/syscalls/vmsplice/vmsplice01.c\n@@ -15,7 +15,6 @@\n \n #include \"tst_test.h\"\n #include \"lapi/fcntl.h\"\n-#include \"lapi/splice.h\"\n #include \"lapi/vmsplice.h\"\n \n #define TEST_BLOCK_SIZE (1<<17)\t/* 128K */\n@@ -50,7 +49,6 @@ static void vmsplice_test(void)\n {\n \tint pipes[2];\n \tlong written;\n-\tint ret;\n \tint fd_out;\n \tstruct iovec v;\n \tloff_t offset;\n@@ -85,9 +83,7 @@ static void vmsplice_test(void)\n \t\t\t}\n \t\t}\n \n-\t\tret = splice(pipes[0], NULL, fd_out, &offset, written, 0);\n-\t\tif (ret < 0)\n-\t\t\ttst_brk(TBROK | TERRNO, \"splice() failed\");\n+\t\tSAFE_SPLICE(pipes[0], NULL, fd_out, &offset, written, 0);\n \t\t//printf(\"offset = %lld\\n\", (long long)offset);\n \t}\n \n","prefixes":["2/2"]}