mirror of
				https://git.proxmox.com/git/mirror_zfs.git
				synced 2025-10-26 18:05:04 +03:00 
			
		
		
		
	ZTS: limit mmapwrite file size
commitee6bf97c77upstream 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 of2a493a4c71, which checks the return value of lseek. Signed-off-by: Antonio Russo <aerusso@aerusso.net> Closes #14277 Closes #14345
This commit is contained in:
		
							parent
							
								
									75fbe7eb99
								
							
						
					
					
						commit
						a75af541cf
					
				| @ -52,6 +52,7 @@ | |||||||
|  */ |  */ | ||||||
| 
 | 
 | ||||||
| #define	NORMAL_WRITE_TH_NUM	2 | #define	NORMAL_WRITE_TH_NUM	2 | ||||||
|  | #define	MAX_WRITE_BYTES	262144000 | ||||||
| 
 | 
 | ||||||
| static void * | static void * | ||||||
| normal_writer(void *filename) | normal_writer(void *filename) | ||||||
| @ -67,18 +68,25 @@ normal_writer(void *filename) | |||||||
| 	} | 	} | ||||||
| 
 | 
 | ||||||
| 	char *buf = malloc(1); | 	char *buf = malloc(1); | ||||||
| 	while (1) { | 	off_t bytes_written = 0; | ||||||
|  | 
 | ||||||
|  | 	while (bytes_written < MAX_WRITE_BYTES) { | ||||||
| 		write_num = write(fd, buf, 1); | 		write_num = write(fd, buf, 1); | ||||||
| 		if (write_num == 0) { | 		if (write_num == 0) { | ||||||
| 			err(1, "write failed!"); | 			err(1, "write failed!"); | ||||||
| 			break; | 			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) { | 	if (buf) { | ||||||
| 		free(buf); | 		free(buf); | ||||||
| 	} | 	} | ||||||
|  | 	return (NULL); | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| static void * | static void * | ||||||
|  | |||||||
		Loading…
	
		Reference in New Issue
	
	Block a user
	 Antonio Russo
						Antonio Russo