dm raid: round region_size to power of two
If the user does not supply a bitmap region_size to the dm raid target, a reasonable size is computed automatically. If this is not a power of 2, the md code will report an error later. This patch catches the problem early and rounds the region_size to the next power of two. Signed-off-by: Jonathan Brassow <jbrassow@redhat.com> Signed-off-by: Alasdair G Kergon <agk@redhat.com>
This commit is contained in:
		
							parent
							
								
									2aab38502d
								
							
						
					
					
						commit
						3a0f9aaee0
					
				| @ -295,9 +295,11 @@ static int validate_region_size(struct raid_set *rs, unsigned long region_size) | |||||||
| 		 * Choose a reasonable default.  All figures in sectors. | 		 * Choose a reasonable default.  All figures in sectors. | ||||||
| 		 */ | 		 */ | ||||||
| 		if (min_region_size > (1 << 13)) { | 		if (min_region_size > (1 << 13)) { | ||||||
|  | 			/* If not a power of 2, make it the next power of 2 */ | ||||||
|  | 			if (min_region_size & (min_region_size - 1)) | ||||||
|  | 				region_size = 1 << fls(region_size); | ||||||
| 			DMINFO("Choosing default region size of %lu sectors", | 			DMINFO("Choosing default region size of %lu sectors", | ||||||
| 			       region_size); | 			       region_size); | ||||||
| 			region_size = min_region_size; |  | ||||||
| 		} else { | 		} else { | ||||||
| 			DMINFO("Choosing default region size of 4MiB"); | 			DMINFO("Choosing default region size of 4MiB"); | ||||||
| 			region_size = 1 << 13; /* sectors */ | 			region_size = 1 << 13; /* sectors */ | ||||||
|  | |||||||
		Loading…
	
		Reference in New Issue
	
	Block a user