19821f1d3SAlan Somers /*-
2*4d846d26SWarner Losh * SPDX-License-Identifier: BSD-2-Clause
39821f1d3SAlan Somers *
49821f1d3SAlan Somers * Copyright (c) 2019 The FreeBSD Foundation
59821f1d3SAlan Somers *
69821f1d3SAlan Somers * This software was developed by BFF Storage Systems, LLC under sponsorship
79821f1d3SAlan Somers * from the FreeBSD Foundation.
89821f1d3SAlan Somers *
99821f1d3SAlan Somers * Redistribution and use in source and binary forms, with or without
109821f1d3SAlan Somers * modification, are permitted provided that the following conditions
119821f1d3SAlan Somers * are met:
129821f1d3SAlan Somers * 1. Redistributions of source code must retain the above copyright
139821f1d3SAlan Somers * notice, this list of conditions and the following disclaimer.
149821f1d3SAlan Somers * 2. Redistributions in binary form must reproduce the above copyright
159821f1d3SAlan Somers * notice, this list of conditions and the following disclaimer in the
169821f1d3SAlan Somers * documentation and/or other materials provided with the distribution.
179821f1d3SAlan Somers *
189821f1d3SAlan Somers * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
199821f1d3SAlan Somers * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
209821f1d3SAlan Somers * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
219821f1d3SAlan Somers * ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
229821f1d3SAlan Somers * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
239821f1d3SAlan Somers * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
249821f1d3SAlan Somers * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
259821f1d3SAlan Somers * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
269821f1d3SAlan Somers * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
279821f1d3SAlan Somers * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
289821f1d3SAlan Somers * SUCH DAMAGE.
299821f1d3SAlan Somers */
309821f1d3SAlan Somers
319821f1d3SAlan Somers extern "C" {
329821f1d3SAlan Somers #include <aio.h>
339821f1d3SAlan Somers #include <fcntl.h>
349821f1d3SAlan Somers #include <unistd.h>
359821f1d3SAlan Somers }
369821f1d3SAlan Somers
379821f1d3SAlan Somers #include "mockfs.hh"
389821f1d3SAlan Somers #include "utils.hh"
399821f1d3SAlan Somers
409821f1d3SAlan Somers using namespace testing;
419821f1d3SAlan Somers
429821f1d3SAlan Somers /*
439821f1d3SAlan Somers * TODO: remove FUSE_FSYNC_FDATASYNC definition when upgrading to protocol 7.28.
449821f1d3SAlan Somers * This bit was actually part of kernel protocol version 5.2, but never
459821f1d3SAlan Somers * documented until after 7.28
469821f1d3SAlan Somers */
479821f1d3SAlan Somers #ifndef FUSE_FSYNC_FDATASYNC
489821f1d3SAlan Somers #define FUSE_FSYNC_FDATASYNC 1
499821f1d3SAlan Somers #endif
509821f1d3SAlan Somers
519821f1d3SAlan Somers class Fsync: public FuseTest {
529821f1d3SAlan Somers public:
expect_fsync(uint64_t ino,uint32_t flags,int error,int times=1)53d970778eSAlan Somers void expect_fsync(uint64_t ino, uint32_t flags, int error, int times = 1)
549821f1d3SAlan Somers {
559821f1d3SAlan Somers EXPECT_CALL(*m_mock, process(
569821f1d3SAlan Somers ResultOf([=](auto in) {
5729edc611SAlan Somers return (in.header.opcode == FUSE_FSYNC &&
5829edc611SAlan Somers in.header.nodeid == ino &&
5946c37cd0SAlan Somers /*
6046c37cd0SAlan Somers * TODO: reenable pid check after fixing
6146c37cd0SAlan Somers * bug 236379
6246c37cd0SAlan Somers */
6329edc611SAlan Somers //(pid_t)in.header.pid == getpid() &&
6429edc611SAlan Somers in.body.fsync.fh == FH &&
6529edc611SAlan Somers in.body.fsync.fsync_flags == flags);
669821f1d3SAlan Somers }, Eq(true)),
679821f1d3SAlan Somers _)
68d970778eSAlan Somers ).Times(times)
69d970778eSAlan Somers .WillRepeatedly(Invoke(ReturnErrno(error)));
709821f1d3SAlan Somers }
719821f1d3SAlan Somers
expect_lookup(const char * relpath,uint64_t ino,int times=1)72d970778eSAlan Somers void expect_lookup(const char *relpath, uint64_t ino, int times = 1)
739821f1d3SAlan Somers {
74d970778eSAlan Somers FuseTest::expect_lookup(relpath, ino, S_IFREG | 0644, 0, times);
759821f1d3SAlan Somers }
769821f1d3SAlan Somers
expect_write(uint64_t ino,uint64_t size,const void * contents)779821f1d3SAlan Somers void expect_write(uint64_t ino, uint64_t size, const void *contents)
789821f1d3SAlan Somers {
79bda39894SAlan Somers FuseTest::expect_write(ino, 0, size, size, 0, 0, contents);
809821f1d3SAlan Somers }
819821f1d3SAlan Somers
829821f1d3SAlan Somers };
839821f1d3SAlan Somers
84c2265ae7SAlan Somers class AioFsync: public Fsync {
SetUp()85c2265ae7SAlan Somers virtual void SetUp() {
86c2265ae7SAlan Somers if (!is_unsafe_aio_enabled())
87c2265ae7SAlan Somers GTEST_SKIP() <<
88c2265ae7SAlan Somers "vfs.aio.enable_unsafe must be set for this test";
89c2265ae7SAlan Somers FuseTest::SetUp();
90c2265ae7SAlan Somers }
91c2265ae7SAlan Somers };
92c2265ae7SAlan Somers
939821f1d3SAlan Somers /* https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=236379 */
TEST_F(AioFsync,aio_fsync)94c2265ae7SAlan Somers TEST_F(AioFsync, aio_fsync)
959821f1d3SAlan Somers {
969821f1d3SAlan Somers const char FULLPATH[] = "mountpoint/some_file.txt";
979821f1d3SAlan Somers const char RELPATH[] = "some_file.txt";
989821f1d3SAlan Somers const char *CONTENTS = "abcdefgh";
999821f1d3SAlan Somers ssize_t bufsize = strlen(CONTENTS);
1009821f1d3SAlan Somers uint64_t ino = 42;
1019821f1d3SAlan Somers struct aiocb iocb, *piocb;
1029821f1d3SAlan Somers int fd;
1039821f1d3SAlan Somers
1049821f1d3SAlan Somers expect_lookup(RELPATH, ino);
1059821f1d3SAlan Somers expect_open(ino, 0, 1);
1069821f1d3SAlan Somers expect_write(ino, bufsize, CONTENTS);
1079821f1d3SAlan Somers expect_fsync(ino, 0, 0);
1089821f1d3SAlan Somers
1099821f1d3SAlan Somers fd = open(FULLPATH, O_RDWR);
1109821f1d3SAlan Somers ASSERT_LE(0, fd) << strerror(errno);
1119821f1d3SAlan Somers ASSERT_EQ(bufsize, write(fd, CONTENTS, bufsize)) << strerror(errno);
1129821f1d3SAlan Somers
1139821f1d3SAlan Somers bzero(&iocb, sizeof(iocb));
1149821f1d3SAlan Somers iocb.aio_fildes = fd;
1159821f1d3SAlan Somers
1169821f1d3SAlan Somers ASSERT_EQ(0, aio_fsync(O_SYNC, &iocb)) << strerror(errno);
1179821f1d3SAlan Somers ASSERT_EQ(0, aio_waitcomplete(&piocb, NULL)) << strerror(errno);
1189821f1d3SAlan Somers
1197fc0921dSAlan Somers leak(fd);
1209821f1d3SAlan Somers }
1219821f1d3SAlan Somers
1229821f1d3SAlan Somers /*
1239821f1d3SAlan Somers * fuse(4) should NOT fsync during VOP_RELEASE or VOP_INACTIVE
1249821f1d3SAlan Somers *
1259821f1d3SAlan Somers * This test only really make sense in writeback caching mode, but it should
1269821f1d3SAlan Somers * still pass in any cache mode.
1279821f1d3SAlan Somers */
TEST_F(Fsync,close)1289821f1d3SAlan Somers TEST_F(Fsync, close)
1299821f1d3SAlan Somers {
1309821f1d3SAlan Somers const char FULLPATH[] = "mountpoint/some_file.txt";
1319821f1d3SAlan Somers const char RELPATH[] = "some_file.txt";
1329821f1d3SAlan Somers const char *CONTENTS = "abcdefgh";
1339821f1d3SAlan Somers ssize_t bufsize = strlen(CONTENTS);
1349821f1d3SAlan Somers uint64_t ino = 42;
1359821f1d3SAlan Somers int fd;
1369821f1d3SAlan Somers
1379821f1d3SAlan Somers expect_lookup(RELPATH, ino);
1389821f1d3SAlan Somers expect_open(ino, 0, 1);
1399821f1d3SAlan Somers expect_write(ino, bufsize, CONTENTS);
1409821f1d3SAlan Somers EXPECT_CALL(*m_mock, process(
1419821f1d3SAlan Somers ResultOf([=](auto in) {
14229edc611SAlan Somers return (in.header.opcode == FUSE_SETATTR);
1439821f1d3SAlan Somers }, Eq(true)),
1449821f1d3SAlan Somers _)
14529edc611SAlan Somers ).WillRepeatedly(Invoke(ReturnImmediate([=](auto i __unused, auto& out) {
1469821f1d3SAlan Somers SET_OUT_HEADER_LEN(out, attr);
14729edc611SAlan Somers out.body.attr.attr.ino = ino; // Must match nodeid
1489821f1d3SAlan Somers })));
1499821f1d3SAlan Somers EXPECT_CALL(*m_mock, process(
1509821f1d3SAlan Somers ResultOf([=](auto in) {
15129edc611SAlan Somers return (in.header.opcode == FUSE_FSYNC);
1529821f1d3SAlan Somers }, Eq(true)),
1539821f1d3SAlan Somers _)
1549821f1d3SAlan Somers ).Times(0);
1559f10f423SAlan Somers expect_flush(ino, 1, ReturnErrno(0));
15642d50d16SAlan Somers expect_release(ino, FH);
1579821f1d3SAlan Somers
1589821f1d3SAlan Somers fd = open(FULLPATH, O_RDWR);
1599821f1d3SAlan Somers ASSERT_LE(0, fd) << strerror(errno);
1609821f1d3SAlan Somers ASSERT_EQ(bufsize, write(fd, CONTENTS, bufsize)) << strerror(errno);
1619821f1d3SAlan Somers close(fd);
1629821f1d3SAlan Somers }
1639821f1d3SAlan Somers
TEST_F(Fsync,eio)164cc34f2f6SAlan Somers TEST_F(Fsync, eio)
1659821f1d3SAlan Somers {
1669821f1d3SAlan Somers const char FULLPATH[] = "mountpoint/some_file.txt";
1679821f1d3SAlan Somers const char RELPATH[] = "some_file.txt";
1689821f1d3SAlan Somers const char *CONTENTS = "abcdefgh";
1699821f1d3SAlan Somers ssize_t bufsize = strlen(CONTENTS);
1709821f1d3SAlan Somers uint64_t ino = 42;
1719821f1d3SAlan Somers int fd;
1729821f1d3SAlan Somers
1739821f1d3SAlan Somers expect_lookup(RELPATH, ino);
1749821f1d3SAlan Somers expect_open(ino, 0, 1);
1759821f1d3SAlan Somers expect_write(ino, bufsize, CONTENTS);
1769821f1d3SAlan Somers expect_fsync(ino, FUSE_FSYNC_FDATASYNC, EIO);
1779821f1d3SAlan Somers
1789821f1d3SAlan Somers fd = open(FULLPATH, O_RDWR);
1799821f1d3SAlan Somers ASSERT_LE(0, fd) << strerror(errno);
1809821f1d3SAlan Somers ASSERT_EQ(bufsize, write(fd, CONTENTS, bufsize)) << strerror(errno);
1819821f1d3SAlan Somers ASSERT_NE(0, fdatasync(fd));
1829821f1d3SAlan Somers ASSERT_EQ(EIO, errno);
1839821f1d3SAlan Somers
1847fc0921dSAlan Somers leak(fd);
1859821f1d3SAlan Somers }
1869821f1d3SAlan Somers
1879821f1d3SAlan Somers /*
1889821f1d3SAlan Somers * If the filesystem returns ENOSYS, it will be treated as success and
1899821f1d3SAlan Somers * subsequent calls to VOP_FSYNC will succeed automatically without being sent
1909821f1d3SAlan Somers * to the filesystem daemon
1919821f1d3SAlan Somers */
TEST_F(Fsync,enosys)1921f4a83f9SAlan Somers TEST_F(Fsync, enosys)
1939821f1d3SAlan Somers {
1949821f1d3SAlan Somers const char FULLPATH[] = "mountpoint/some_file.txt";
1959821f1d3SAlan Somers const char RELPATH[] = "some_file.txt";
1969821f1d3SAlan Somers const char *CONTENTS = "abcdefgh";
1979821f1d3SAlan Somers ssize_t bufsize = strlen(CONTENTS);
1989821f1d3SAlan Somers uint64_t ino = 42;
1999821f1d3SAlan Somers int fd;
2009821f1d3SAlan Somers
2019821f1d3SAlan Somers expect_lookup(RELPATH, ino);
2029821f1d3SAlan Somers expect_open(ino, 0, 1);
2039821f1d3SAlan Somers expect_write(ino, bufsize, CONTENTS);
2049821f1d3SAlan Somers expect_fsync(ino, FUSE_FSYNC_FDATASYNC, ENOSYS);
2059821f1d3SAlan Somers
2069821f1d3SAlan Somers fd = open(FULLPATH, O_RDWR);
2079821f1d3SAlan Somers ASSERT_LE(0, fd) << strerror(errno);
2089821f1d3SAlan Somers ASSERT_EQ(bufsize, write(fd, CONTENTS, bufsize)) << strerror(errno);
2099821f1d3SAlan Somers EXPECT_EQ(0, fdatasync(fd));
2109821f1d3SAlan Somers
2119821f1d3SAlan Somers /* Subsequent calls shouldn't query the daemon*/
2129821f1d3SAlan Somers EXPECT_EQ(0, fdatasync(fd));
2137fc0921dSAlan Somers leak(fd);
2149821f1d3SAlan Somers }
2159821f1d3SAlan Somers
2169821f1d3SAlan Somers
TEST_F(Fsync,fdatasync)217cc34f2f6SAlan Somers TEST_F(Fsync, fdatasync)
2189821f1d3SAlan Somers {
2199821f1d3SAlan Somers const char FULLPATH[] = "mountpoint/some_file.txt";
2209821f1d3SAlan Somers const char RELPATH[] = "some_file.txt";
2219821f1d3SAlan Somers const char *CONTENTS = "abcdefgh";
2229821f1d3SAlan Somers ssize_t bufsize = strlen(CONTENTS);
2239821f1d3SAlan Somers uint64_t ino = 42;
2249821f1d3SAlan Somers int fd;
2259821f1d3SAlan Somers
2269821f1d3SAlan Somers expect_lookup(RELPATH, ino);
2279821f1d3SAlan Somers expect_open(ino, 0, 1);
2289821f1d3SAlan Somers expect_write(ino, bufsize, CONTENTS);
2299821f1d3SAlan Somers expect_fsync(ino, FUSE_FSYNC_FDATASYNC, 0);
2309821f1d3SAlan Somers
2319821f1d3SAlan Somers fd = open(FULLPATH, O_RDWR);
2329821f1d3SAlan Somers ASSERT_LE(0, fd) << strerror(errno);
2339821f1d3SAlan Somers ASSERT_EQ(bufsize, write(fd, CONTENTS, bufsize)) << strerror(errno);
2349821f1d3SAlan Somers ASSERT_EQ(0, fdatasync(fd)) << strerror(errno);
2359821f1d3SAlan Somers
2367fc0921dSAlan Somers leak(fd);
2379821f1d3SAlan Somers }
2389821f1d3SAlan Somers
TEST_F(Fsync,fsync)239915012e0SAlan Somers TEST_F(Fsync, fsync)
2409821f1d3SAlan Somers {
2419821f1d3SAlan Somers const char FULLPATH[] = "mountpoint/some_file.txt";
2429821f1d3SAlan Somers const char RELPATH[] = "some_file.txt";
2439821f1d3SAlan Somers const char *CONTENTS = "abcdefgh";
2449821f1d3SAlan Somers ssize_t bufsize = strlen(CONTENTS);
2459821f1d3SAlan Somers uint64_t ino = 42;
2469821f1d3SAlan Somers int fd;
2479821f1d3SAlan Somers
2489821f1d3SAlan Somers expect_lookup(RELPATH, ino);
2499821f1d3SAlan Somers expect_open(ino, 0, 1);
2509821f1d3SAlan Somers expect_write(ino, bufsize, CONTENTS);
2519821f1d3SAlan Somers expect_fsync(ino, 0, 0);
2529821f1d3SAlan Somers
2539821f1d3SAlan Somers fd = open(FULLPATH, O_RDWR);
2549821f1d3SAlan Somers ASSERT_LE(0, fd) << strerror(errno);
2559821f1d3SAlan Somers ASSERT_EQ(bufsize, write(fd, CONTENTS, bufsize)) << strerror(errno);
2569821f1d3SAlan Somers ASSERT_EQ(0, fsync(fd)) << strerror(errno);
2579821f1d3SAlan Somers
2587fc0921dSAlan Somers leak(fd);
2599821f1d3SAlan Somers }
260d970778eSAlan Somers
261d970778eSAlan Somers /* If multiple FUSE file handles are active, we must fsync them all */
TEST_F(Fsync,two_handles)262d970778eSAlan Somers TEST_F(Fsync, two_handles)
263d970778eSAlan Somers {
264d970778eSAlan Somers const char FULLPATH[] = "mountpoint/some_file.txt";
265d970778eSAlan Somers const char RELPATH[] = "some_file.txt";
266d970778eSAlan Somers const char *CONTENTS = "abcdefgh";
267d970778eSAlan Somers ssize_t bufsize = strlen(CONTENTS);
268d970778eSAlan Somers uint64_t ino = 42;
269d970778eSAlan Somers int fd1, fd2;
270d970778eSAlan Somers
271d970778eSAlan Somers expect_lookup(RELPATH, ino, 2);
272d970778eSAlan Somers expect_open(ino, 0, 2);
273d970778eSAlan Somers expect_write(ino, bufsize, CONTENTS);
274d970778eSAlan Somers expect_fsync(ino, 0, 0, 2);
275d970778eSAlan Somers
276d970778eSAlan Somers fd1 = open(FULLPATH, O_WRONLY);
277d970778eSAlan Somers ASSERT_LE(0, fd1) << strerror(errno);
278d970778eSAlan Somers fd2 = open(FULLPATH, O_RDONLY);
279d970778eSAlan Somers ASSERT_LE(0, fd2) << strerror(errno);
280d970778eSAlan Somers ASSERT_EQ(bufsize, write(fd1, CONTENTS, bufsize)) << strerror(errno);
281d970778eSAlan Somers ASSERT_EQ(0, fsync(fd1)) << strerror(errno);
282d970778eSAlan Somers
283d970778eSAlan Somers leak(fd1);
284d970778eSAlan Somers leak(fd2);
285d970778eSAlan Somers }
286