From patchwork Mon Aug 6 05:07:49 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jinhui Huang X-Patchwork-Id: 953698 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (mailfrom) smtp.mailfrom=lists.linux.it (client-ip=213.254.12.146; helo=picard.linux.it; envelope-from=ltp-bounces+incoming=patchwork.ozlabs.org@lists.linux.it; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=cn.fujitsu.com Received: from picard.linux.it (picard.linux.it [213.254.12.146]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 41kQhm0VXPz9s0R for ; Mon, 6 Aug 2018 15:10:45 +1000 (AEST) Received: from picard.linux.it (localhost [IPv6:::1]) by picard.linux.it (Postfix) with ESMTP id 4D3253E658B for ; Mon, 6 Aug 2018 07:10:38 +0200 (CEST) X-Original-To: ltp@lists.linux.it Delivered-To: ltp@picard.linux.it Received: from in-5.smtp.seeweb.it (in-5.smtp.seeweb.it [217.194.8.5]) by picard.linux.it (Postfix) with ESMTP id 50B433E6570 for ; Mon, 6 Aug 2018 07:10:35 +0200 (CEST) Received: from heian.cn.fujitsu.com (mail.cn.fujitsu.com [183.91.158.132]) by in-5.smtp.seeweb.it (Postfix) with ESMTP id 9BBC9600712 for ; Mon, 6 Aug 2018 07:10:32 +0200 (CEST) X-IronPort-AV: E=Sophos;i="5.43,368,1503331200"; d="scan'208";a="43105869" Received: from unknown (HELO cn.fujitsu.com) ([10.167.33.5]) by heian.cn.fujitsu.com with ESMTP; 06 Aug 2018 13:10:29 +0800 Received: from G08CNEXCHPEKD02.g08.fujitsu.local (unknown [10.167.33.83]) by cn.fujitsu.com (Postfix) with ESMTP id 99D714B66A09; Mon, 6 Aug 2018 13:10:24 +0800 (CST) Received: from localhost.localdomain.localdomain (10.167.220.196) by G08CNEXCHPEKD02.g08.fujitsu.local (10.167.33.89) with Microsoft SMTP Server (TLS) id 14.3.399.0; Mon, 6 Aug 2018 13:10:58 +0800 From: Jinhui huang To: Date: Mon, 6 Aug 2018 13:07:49 +0800 Message-ID: <1533532069-7458-1-git-send-email-huangjh.jy@cn.fujitsu.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1739695693.37582576.1533212220162.JavaMail.zimbra@redhat.com> References: <1739695693.37582576.1533212220162.JavaMail.zimbra@redhat.com> MIME-Version: 1.0 X-Originating-IP: [10.167.220.196] X-yoursite-MailScanner-ID: 99D714B66A09.AE55F X-yoursite-MailScanner: Found to be clean X-yoursite-MailScanner-From: huangjh.jy@cn.fujitsu.com X-Spam-Status: No, score=0.0 required=7.0 tests=none autolearn=disabled version=3.4.0 X-Virus-Scanned: clamav-milter 0.99.2 at in-5.smtp.seeweb.it X-Virus-Status: Clean X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on in-5.smtp.seeweb.it Cc: ltp@lists.linux.it Subject: [LTP] [PATCH v2 4/5] syscalls/flock04: Rewrite to new library X-BeenThere: ltp@lists.linux.it X-Mailman-Version: 2.1.18 Precedence: list List-Id: Linux Test Project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: ltp-bounces+incoming=patchwork.ozlabs.org@lists.linux.it Sender: "ltp" 1) Avoid locking closed fd when running flock04 in loops 2) Merge flock05 into flock04 Signed-off-by: Jinhui huang --- runtest/ltplite | 1 - runtest/stress.part3 | 1 - runtest/syscalls | 1 - testcases/kernel/syscalls/flock/.gitignore | 1 - testcases/kernel/syscalls/flock/flock04.c | 224 +++++++++-------------------- testcases/kernel/syscalls/flock/flock05.c | 207 -------------------------- 6 files changed, 69 insertions(+), 366 deletions(-) delete mode 100644 testcases/kernel/syscalls/flock/flock05.c diff --git a/runtest/ltplite b/runtest/ltplite index 0840564..9ca6c42 100644 --- a/runtest/ltplite +++ b/runtest/ltplite @@ -239,7 +239,6 @@ flock01 flock01 flock02 flock02 flock03 flock03 flock04 flock04 -flock05 flock05 flock06 flock06 fmtmsg01 fmtmsg01 diff --git a/runtest/stress.part3 b/runtest/stress.part3 index fea4ccb..ec18dcf 100644 --- a/runtest/stress.part3 +++ b/runtest/stress.part3 @@ -178,7 +178,6 @@ flock01 flock01 flock02 flock02 flock03 flock03 flock04 flock04 -flock05 flock05 flock06 flock06 fmtmsg01 fmtmsg01 diff --git a/runtest/syscalls b/runtest/syscalls index 420ff45..6f16317 100644 --- a/runtest/syscalls +++ b/runtest/syscalls @@ -286,7 +286,6 @@ flock01 flock01 flock02 flock02 flock03 flock03 flock04 flock04 -flock05 flock05 flock06 flock06 fmtmsg01 fmtmsg01 diff --git a/testcases/kernel/syscalls/flock/.gitignore b/testcases/kernel/syscalls/flock/.gitignore index 93cce0a..c8cb0fc 100644 --- a/testcases/kernel/syscalls/flock/.gitignore +++ b/testcases/kernel/syscalls/flock/.gitignore @@ -2,5 +2,4 @@ /flock02 /flock03 /flock04 -/flock05 /flock06 diff --git a/testcases/kernel/syscalls/flock/flock04.c b/testcases/kernel/syscalls/flock/flock04.c index 8a282b4..93fa43a 100644 --- a/testcases/kernel/syscalls/flock/flock04.c +++ b/testcases/kernel/syscalls/flock/flock04.c @@ -1,178 +1,92 @@ -/* - * Copyright (c) Wipro Technologies Ltd, 2002. All Rights Reserved. - * - * This program is free software; you can redistribute it and/or modify it - * under the terms of version 2 of the GNU General Public License as - * published by the Free Software Foundation. - * - * This program is distributed in the hope that it would be useful, but - * WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. - * - * You should have received a copy of the GNU General Public License along - * with this program; if not, write the Free Software Foundation, Inc., - * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. +// SPDX-License-Identifier: GPL-2.0-or-later +/* Copyright (c) Wipro Technologies Ltd, 2002. All Rights Reserved. + * Author: Vatsal Avasthi * + * Test Description: + * This test verifies that flock() behavior with different locking + * combinations along with LOCK_SH and LOCK_EX: + * 1) flock() succeeded in acquiring shared lock on shared lock file. + * 2) flock() failed to acquire exclusive lock on shared lock file. + * 3) flock() failed to acquire shared lock on exclusive lock file. + * 4) flock() failed to acquire exclusive lock on exclusive lock file. */ -/********************************************************** - * - * TEST IDENTIFIER : flock04 - * - * EXECUTED BY : anyone - * - * TEST TITLE : Testing different locks on flock(2) - * - * TEST CASE TOTAL : 2 - * - * AUTHOR : Vatsal Avasthi - * - * SIGNALS - * Uses SIGUSR1 to pause before test if option set. - * (See the parse_opts(3) man page). - * - * DESCRIPTION - * Tests to verify flock(2) behavior with different locking combinations along - * with LOCK_SH. - * $ - * Setup: - * Setup signal handling. - * Pause for SIGUSR1 if option specified. - * Create a temporary directory and chdir to it. - * Create a temporary file - * - * Test: - * Loop if proper options are given. - * Parent flocks(2) a file - * fork() a child process - * Child tries to flock() the already flocked file with different types of locks - * Check return code, if system call failed (return == -1) - * Log the error number and issue a FAIL message - * otherwise issue a PASS message - * - * Cleanup: - * Print errno log and/or timing stats if options given - * Deletes temporary directory. - * - * USAGE: - * flock04 [-c n] [-e] [-i n] [-I x] [-P x] [-t] [-h] [-f] [-p] - * where, -c n : Run n copies concurrently. - * -f : Turn off functional testing - * -e : Turn on errno logging. - * -h : Show help screen $ - * -i n : Execute test n times. - * -I x : Execute test for x seconds. - * -p : Pause for SIGUSR1 before starting - * -P x : Pause for x seconds between iterations. - * -t : Turn on syscall timing. - * - ****************************************************************/ #include -#include -#include -#include -#include #include -#include -#include -#include "test.h" -#include "safe_macros.h" +#include -void setup(void); -void cleanup(void); +#include "tst_test.h" -char *TCID = "flock04"; -int TST_TOTAL = 2; -char filename[100]; -int fd, fd1, status; +static struct tcase { + int operation; + char *f_lock; +} tcases[] = { + {LOCK_SH, "shared lock"}, + {LOCK_EX, "exclusive lock"}, +}; -int main(int argc, char **argv) +static void child(int opt, int should_pass, char *lock) { - int lc, retval; - pid_t pid; - - tst_parse_opts(argc, argv, NULL, NULL); - - setup(); - - for (lc = 0; TEST_LOOPING(lc); lc++) { - - tst_count = 0; - - TEST(flock(fd, LOCK_SH)); - if (TEST_RETURN == 0) { - - pid = FORK_OR_VFORK(); - if (pid == -1) - tst_brkm(TBROK | TERRNO, cleanup, - "fork failed"); - if (pid == 0) { - fd1 = open(filename, O_RDONLY); - retval = flock(fd1, LOCK_SH | LOCK_NB); - if (retval == -1) - tst_resm(TFAIL, - "flock() FAILED to acquire shared lock on existing " - "Share Locked file"); - else - tst_resm(TPASS, - "flock() PASSED in acquiring shared lock on " - "Share Locked file"); - exit(0); - } else { - SAFE_WAIT(cleanup, &status); - } - - pid = FORK_OR_VFORK(); - if (pid == -1) - tst_brkm(TBROK | TERRNO, cleanup, - "fork failed"); - - if (pid == 0) { - fd1 = open(filename, O_RDWR); - retval = flock(fd1, LOCK_EX | LOCK_NB); - if (retval == -1) { - tst_resm(TPASS, - "flock() failed to acquire exclusive lock on existing " - "share locked file as expected"); - } else { - tst_resm(TFAIL, - "flock() unexpectedly passed in acquiring exclusive lock on " - "Share Locked file"); - } - exit(0); - } else if (wait(&status) == -1) - tst_resm(TBROK | TERRNO, "wait failed"); - TEST(flock(fd, LOCK_UN)); - } else - tst_resm(TFAIL | TERRNO, "flock failed"); - - close(fd); - close(fd1); + int retval, fd1; + + fd1 = SAFE_OPEN("testfile", O_RDWR); + retval = flock(fd1, opt); + if (should_pass) { + tst_res(retval == -1 ? TFAIL : TPASS, + " Child acquiring %s got %d", lock, retval); + } else { + tst_res(retval == -1 ? TPASS : TFAIL, + " Child acquiring %s got %d", lock, retval); } - cleanup(); - tst_exit(); + SAFE_CLOSE(fd1); + exit(0); } -void setup(void) +static void verify_flock(unsigned n) { + int fd2; + pid_t pid; + struct tcase *tc = &tcases[n]; + + fd2 = SAFE_OPEN("testfile", O_RDWR); + TEST(flock(fd2, tc->operation)); + if (TST_RET != 0) { + tst_res(TFAIL | TERRNO, "flock() failed to acquire %s", + tc->f_lock); + SAFE_CLOSE(fd2); + return; + } - tst_sig(FORK, DEF_HANDLER, cleanup); - - TEST_PAUSE; + tst_res(TPASS, "Parent had %s", tc->f_lock); - tst_tmpdir(); + pid = SAFE_FORK(); + if (pid == 0) + child(LOCK_SH | LOCK_NB, tc->operation & LOCK_SH, tc->f_lock); + else + tst_reap_children(); - sprintf(filename, "flock04.%d", getpid()); + pid = SAFE_FORK(); + if (pid == 0) + child(LOCK_EX | LOCK_NB, 0, tc->f_lock); + else + tst_reap_children(); - fd = open(filename, O_CREAT | O_TRUNC | O_RDWR, 0666); - if (fd == -1) - tst_brkm(TFAIL, cleanup, "creating a new file failed"); + SAFE_CLOSE(fd2); } -void cleanup(void) +static void setup(void) { - unlink(filename); + int fd; - tst_rmdir(); + fd = SAFE_OPEN("testfile", O_CREAT | O_TRUNC | O_RDWR, 0644); + SAFE_CLOSE(fd); } + +static struct tst_test test = { + .tcnt = ARRAY_SIZE(tcases), + .test = verify_flock, + .needs_tmpdir = 1, + .setup = setup, + .forks_child = 1, +}; diff --git a/testcases/kernel/syscalls/flock/flock05.c b/testcases/kernel/syscalls/flock/flock05.c deleted file mode 100644 index 13ae2f6..0000000 --- a/testcases/kernel/syscalls/flock/flock05.c +++ /dev/null @@ -1,207 +0,0 @@ -/* - * Copyright (c) Wipro Technologies Ltd, 2002. All Rights Reserved. - * - * This program is free software; you can redistribute it and/or modify it - * under the terms of version 2 of the GNU General Public License as - * published by the Free Software Foundation. - * - * This program is distributed in the hope that it would be useful, but - * WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. - * - * You should have received a copy of the GNU General Public License along - * with this program; if not, write the Free Software Foundation, Inc., - * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. - * - */ -/********************************************************** - * - * TEST IDENTIFIER : flock05 - * - * EXECUTED BY : anyone - * - * TEST TITLE : Testing different locks on flock(2) - * - * TEST CASE TOTAL : 2 - * - * AUTHOR : Vatsal Avasthi - * - * SIGNALS - * Uses SIGUSR1 to pause before test if option set. - * (See the parse_opts(3) man page). - * - * DESCRIPTION - * Tests to verify flock(2) behavior with different locking combinations along - * with LOCK_EX. - * $ - * Setup: - * Setup signal handling. - * Pause for SIGUSR1 if option specified. - * Create a temporary directory and chdir to it. - * Create a temporary file - * - * Test: - * Loop if proper options are given. - * Parent flocks(2) a file - * fork() a child process - * Child tries to flock() the already flocked file with different types of locks - * Check return code, if system call failed (return == -1) - * Log the error number and issue a FAIL message - * otherwise issue a PASS message - * - * Cleanup: - * Print errno log and/or timing stats if options given - * Deletes temporary directory. - * - * USAGE: - * flock05 [-c n] [-e] [-i n] [-I x] [-P x] [-t] [-h] [-f] [-p] - * where, -c n : Run n copies concurrently. - * -f : Turn off functional testing - * -e : Turn on errno logging. - * -h : Show help screen $ - * -i n : Execute test n times. - * -I x : Execute test for x seconds. - * -p : Pause for SIGUSR1 before starting - * -P x : Pause for x seconds between iterations. - * -t : Turn on syscall timing. - * - ****************************************************************/ - -#include -#include -#include -#include -#include -#include -#include -#include -#include "test.h" - -void setup(void); -void cleanup(void); - -char *TCID = "flock05"; -int TST_TOTAL = 2; -char filename[100]; -int fd, fd1, status; - -int main(int argc, char **argv) -{ - int lc, retval; - pid_t pid; - - tst_parse_opts(argc, argv, NULL, NULL); - - /* global setup */ - setup(); - - /* The following loop checks looping state if -i option given */ - - for (lc = 0; TEST_LOOPING(lc); lc++) { - - /* reset tst_count in case we are looping */ - tst_count = 0; - - /* Testing Shared lock on Exclusive Locked file */ - TEST(flock(fd, LOCK_EX)); - if (TEST_RETURN == 0) { - - pid = FORK_OR_VFORK(); - if (pid == 0) { - fd1 = open(filename, O_RDWR); - retval = flock(fd1, LOCK_SH | LOCK_NB); - if (retval == -1) { - tst_resm(TPASS, - "flock() failed to acquire shared lock on an already" - "exclusive locked file as expected"); - } else { - tst_resm(TFAIL, - "flock() unexpectedly PASSED in acquiring shared lock on " - "an already exclusive locked file"); - } - exit(0); - } else { - /* parent waiting */ - wait(&status); - } - - /* Testing Exclusive lock on a Exclusive Locked file */ - pid = FORK_OR_VFORK(); - - if (pid == 0) { - fd1 = open(filename, O_RDWR); - retval = flock(fd1, LOCK_EX | LOCK_NB); - if (retval == -1) { - tst_resm(TPASS, - "flock() failed to acquire exclusive lock on existing " - " exclusive locked file as expected"); - } else { - tst_resm(TFAIL, - "flock() unexpectedly passed in acquiring exclusive lock on " - "an exclusive locked file"); - } - exit(0); - } else { - /* parent waiting */ - wait(&status); - } - TEST(flock(fd, LOCK_UN)); - } else { - tst_resm(TFAIL, - "flock() failed to acquire exclusive lock"); - } - - } - - close(fd); - close(fd1); - cleanup(); - tst_exit(); - -} - -/* - * setup() - * performs all ONE TIME setup for this test - */ -void setup(void) -{ - - tst_sig(FORK, DEF_HANDLER, cleanup); - - /* Pause if that option was specified - * TEST_PAUSE contains the code to fork the test with the -i option. - * You want to make sure you do this before you create your temporary - * directory. - */ - TEST_PAUSE; - - /* Create a unique temporary directory and chdir() to it. */ - tst_tmpdir(); - - sprintf(filename, "flock05.%d", getpid()); - - /* creating temporary file */ - fd = open(filename, O_CREAT | O_TRUNC | O_RDWR, 0666); - if (fd == -1) { - tst_resm(TFAIL, "creating a new file failed"); - - /* Removing temp dir */ - tst_rmdir(); - - } -} - -/* - * cleanup() - * performs all ONE TIME cleanup for this test at - * completion or premature exit - */ -void cleanup(void) -{ - - unlink(filename); - - tst_rmdir(); - -} From patchwork Mon Aug 6 05:08:55 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jinhui Huang X-Patchwork-Id: 953699 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (mailfrom) smtp.mailfrom=lists.linux.it (client-ip=2001:1418:10:5::2; helo=picard.linux.it; envelope-from=ltp-bounces+incoming=patchwork.ozlabs.org@lists.linux.it; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=cn.fujitsu.com Received: from picard.linux.it (picard.linux.it [IPv6:2001:1418:10:5::2]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 41kQjl5v6Fz9s0R for ; Mon, 6 Aug 2018 15:11:39 +1000 (AEST) Received: from picard.linux.it (localhost [IPv6:::1]) by picard.linux.it (Postfix) with ESMTP id 9D0C43E658B for ; Mon, 6 Aug 2018 07:11:34 +0200 (CEST) X-Original-To: ltp@lists.linux.it Delivered-To: ltp@picard.linux.it Received: from in-3.smtp.seeweb.it (in-3.smtp.seeweb.it [IPv6:2001:4b78:1:20::3]) by picard.linux.it (Postfix) with ESMTP id E17B53E6570 for ; Mon, 6 Aug 2018 07:11:32 +0200 (CEST) Received: from heian.cn.fujitsu.com (mail.cn.fujitsu.com [183.91.158.132]) by in-3.smtp.seeweb.it (Postfix) with ESMTP id 38EBB1A00E62 for ; Mon, 6 Aug 2018 07:11:30 +0200 (CEST) X-IronPort-AV: E=Sophos;i="5.43,368,1503331200"; d="scan'208";a="43105923" Received: from unknown (HELO cn.fujitsu.com) ([10.167.33.5]) by heian.cn.fujitsu.com with ESMTP; 06 Aug 2018 13:11:30 +0800 Received: from G08CNEXCHPEKD02.g08.fujitsu.local (unknown [10.167.33.83]) by cn.fujitsu.com (Postfix) with ESMTP id CC3504B66A09; Mon, 6 Aug 2018 13:11:29 +0800 (CST) Received: from localhost.localdomain.localdomain (10.167.220.196) by G08CNEXCHPEKD02.g08.fujitsu.local (10.167.33.89) with Microsoft SMTP Server (TLS) id 14.3.399.0; Mon, 6 Aug 2018 13:12:03 +0800 From: Jinhui huang To: Date: Mon, 6 Aug 2018 13:08:55 +0800 Message-ID: <1533532135-7496-1-git-send-email-huangjh.jy@cn.fujitsu.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1649896669.37585897.1533212964490.JavaMail.zimbra@redhat.com> References: <1649896669.37585897.1533212964490.JavaMail.zimbra@redhat.com> MIME-Version: 1.0 X-Originating-IP: [10.167.220.196] X-yoursite-MailScanner-ID: CC3504B66A09.AEB0C X-yoursite-MailScanner: Found to be clean X-yoursite-MailScanner-From: huangjh.jy@cn.fujitsu.com X-Spam-Status: No, score=0.0 required=7.0 tests=none autolearn=disabled version=3.4.0 X-Virus-Scanned: clamav-milter 0.99.2 at in-3.smtp.seeweb.it X-Virus-Status: Clean X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on in-3.smtp.seeweb.it Cc: ltp@lists.linux.it Subject: [LTP] [PATCH v2 5/5] syscalls/flock06: Rewrite to new library X-BeenThere: ltp@lists.linux.it X-Mailman-Version: 2.1.18 Precedence: list List-Id: Linux Test Project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: ltp-bounces+incoming=patchwork.ozlabs.org@lists.linux.it Sender: "ltp" Signed-off-by: Jinhui huang --- testcases/kernel/syscalls/flock/flock06.c | 213 ++++++++---------------------- 1 file changed, 52 insertions(+), 161 deletions(-) diff --git a/testcases/kernel/syscalls/flock/flock06.c b/testcases/kernel/syscalls/flock/flock06.c index 617eddf..9ef1280 100644 --- a/testcases/kernel/syscalls/flock/flock06.c +++ b/testcases/kernel/syscalls/flock/flock06.c @@ -1,177 +1,68 @@ -/* +// SPDX-License-Identifier: GPL-2.0-or-later +/* Copyright (c) Matthew Wilcox for Hewlett Packard 2003 + * Author: Matthew Wilcox * - * Copyright (c) Matthew Wilcox for Hewlett Packard 2003 + * Test Description: + * This test verifies that flock locks held on one fd conflict with flock + * locks held on a different fd. * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation; either version 2 of the License, or - * (at your option) any later version. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See - * the GNU General Public License for more details. - * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software - * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA + * Test Steps: + * The process opens two file descriptors on the same file. It acquires + * an exclusive flock on the first descriptor, checks that attempting to + * acquire an flock on the second descriptor fails. Then it removes the + * first descriptor's lock and attempts to acquire an exclusive lock on + * the second descriptor. */ -/********************************************************** - * - * TEST IDENTIFIER : flock06 - * - * EXECUTED BY : anyone - * - * TEST TITLE : Error condition test for flock(2) - * - * TEST CASE TOTAL : 1 - * - * AUTHOR : Matthew Wilcox - * - * SIGNALS - * Uses SIGUSR1 to pause before test if option set. - * (See the parse_opts(3) man page). - * - * DESCRIPTION - * This test verifies that flock locks held on one fd conflict with - * flock locks held on a different fd. - * - * Test: - * The process opens two file descriptors on the same file. - * It acquires an exclusive flock on the first descriptor, - * checks that attempting to acquire an flock on the second - * descriptor fails. Then it removes the first descriptor's - * lock and attempts to acquire an exclusive lock on the - * second descriptor. - * - * USAGE: - * flock06 [-c n] [-e] [-i n] [-I x] [-P x] [-t] [-h] [-f] [-p] - * where, -c n : Run n copies concurrently - * -f : Turn off functional testing - * -e : Turn on errno logging - * -h : Show help screen - * -i n : Execute test n times - * -I x : Execute test for x seconds - * -p : Pause for SIGUSR1 before starting - * -P x : Pause for x seconds between iterations - * -t : Turn on syscall timing - * - ****************************************************************/ - -#include #include -#include -#include -#include #include -#include -#include "test.h" -#include "safe_macros.h" -void setup(void); -void cleanup(void); +#include "tst_test.h" -char *TCID = "flock06"; -int TST_TOTAL = 3; -char filename[100]; - -int main(int argc, char **argv) +static void verify_flock(void) { - int lc; - - tst_parse_opts(argc, argv, NULL, NULL); - - setup(); - - /* The following loop checks looping state if -i option given */ - - for (lc = 0; TEST_LOOPING(lc); lc++) { - int fd1, fd2; - - /* reset tst_count in case we are looping */ - tst_count = 0; - - fd1 = open(filename, O_RDWR); - if (fd1 == -1) - tst_brkm(TFAIL | TERRNO, cleanup, - "failed to open the file"); - - TEST(flock(fd1, LOCK_EX | LOCK_NB)); - if (TEST_RETURN != 0) - tst_resm(TFAIL | TTERRNO, - "First attempt to flock() failed"); - else - tst_resm(TPASS, "First attempt to flock() passed"); - - fd2 = open(filename, O_RDWR); - if (fd2 == -1) - tst_brkm(TFAIL | TERRNO, cleanup, - "failed to open the file"); - - TEST(flock(fd2, LOCK_EX | LOCK_NB)); - if (TEST_RETURN == -1) - tst_resm(TPASS, "Second attempt to flock() denied"); - else - tst_resm(TFAIL, "Second attempt to flock() succeeded!"); - - TEST(flock(fd1, LOCK_UN)); - if (TEST_RETURN == -1) - tst_resm(TFAIL | TTERRNO, "Failed to unlock fd1"); - else - tst_resm(TPASS, "Unlocked fd1"); - - TEST(flock(fd2, LOCK_EX | LOCK_NB)); - if (TEST_RETURN == -1) - tst_resm(TFAIL, "Third attempt to flock() denied!"); - else - tst_resm(TPASS, "Third attempt to flock() succeeded"); - close(fd1); - close(fd2); - - } - - cleanup(); - tst_exit(); - + int fd1, fd2; + + fd1 = SAFE_OPEN("testfile", O_RDWR); + TEST(flock(fd1, LOCK_EX | LOCK_NB)); + if (TST_RET != 0) + tst_res(TFAIL | TTERRNO, "First attempt to flock() failed"); + else + tst_res(TPASS, "First attempt to flock() passed"); + + fd2 = SAFE_OPEN("testfile", O_RDWR); + TEST(flock(fd2, LOCK_EX | LOCK_NB)); + if (TST_RET == -1) + tst_res(TPASS | TTERRNO, "Second attempt to flock() denied"); + else + tst_res(TFAIL, "Second attempt to flock() succeeded!"); + + TEST(flock(fd1, LOCK_UN)); + if (TST_RET == -1) + tst_res(TFAIL | TTERRNO, "Failed to unlock fd1"); + else + tst_res(TPASS, "Unlocked fd1"); + + TEST(flock(fd2, LOCK_EX | LOCK_NB)); + if (TST_RET == -1) + tst_res(TFAIL | TTERRNO, "Third attempt to flock() denied!"); + else + tst_res(TPASS, "Third attempt to flock() succeeded"); + + SAFE_CLOSE(fd1); + SAFE_CLOSE(fd2); } -/* - * setup() - * performs all ONE TIME setup for this test - */ -void setup(void) +static void setup(void) { int fd; - tst_sig(FORK, DEF_HANDLER, cleanup); - - /* Pause if that option was specified - * TEST_PAUSE contains the code to fork the test with the -i option. - * You want to make sure you do this before you create your temporary - * directory. - */ - TEST_PAUSE; - - /* Create a unique temporary directory and chdir() to it. */ - tst_tmpdir(); - - sprintf(filename, "flock06.%d", getpid()); - - /* creating temporary file */ - fd = SAFE_OPEN(tst_rmdir, filename, O_CREAT | O_TRUNC | O_RDWR, 0666); - close(fd); + fd = SAFE_OPEN("testfile", O_CREAT | O_TRUNC | O_RDWR, 0666); + SAFE_CLOSE(fd); } -/* - * cleanup() - * performs all ONE TIME cleanup for this test at - * completion or premature exit - */ -void cleanup(void) -{ - - unlink(filename); - tst_rmdir(); - -} +static struct tst_test test = { + .test_all = verify_flock, + .needs_tmpdir = 1, + .setup = setup, +};