Skip to content

Commit

Permalink
ZTS: limit mmapwrite file size
Browse files Browse the repository at this point in the history
commit ee6bf97 upstream

mmapwrite spawns several threads, all of which perform writes on a file
for the purpose of testing the behavior of mmap(2)-ed files.  One
thread performs an mmap and a write to the beginning of that region,
while the others perform regular writes after lseek(2)-ing the end of
the file.

Because these regular writes are set in a while (1) loop, they will
write an unbounded amount of data to disk.  The mmap_write_001_pos test
script SIGKILLs them after 30 seconds, but on fast testbeds, this may
be enough time to exhaust the available space in the filesystem,
leading to spurious test failures.

Instead, limit the total file size by checking that the lseek return
value is no greater than 250 * 1024*1024 bytes, which is less than the
default minimum vdev size defined in includes/default.cfg .

This also includes part of 2a493a4,
which checks the return value of lseek.

Signed-off-by: Antonio Russo <aerusso@aerusso.net>
Closes openzfs#14277
Closes openzfs#14345
  • Loading branch information
aerusso committed Jan 7, 2023
1 parent 75fbe7e commit 92cbbab
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions tests/zfs-tests/cmd/mmapwrite/mmapwrite.c
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@
*/

#define NORMAL_WRITE_TH_NUM 2
#define MAX_WRITE_BYTES 262144000

static void *
normal_writer(void *filename)
Expand All @@ -67,18 +68,25 @@ normal_writer(void *filename)
}

char *buf = malloc(1);
while (1) {
off_t bytes_written = 0;

while (bytes_written < MAX_WRITE_BYTES) {
write_num = write(fd, buf, 1);
if (write_num == 0) {
err(1, "write failed!");
break;
}
lseek(fd, page_size, SEEK_CUR);
if ((bytes_written = lseek(fd, page_size, SEEK_CUR)) == -1) {
err(1, "lseek failed on %s: %s", file_path,
strerror(errno));
break;
}
}

if (buf) {
free(buf);
}
return (NULL);
}

static void *
Expand Down

0 comments on commit 92cbbab

Please sign in to comment.