throttle: Test burst limits lower than the normal limits
This checks that making FOO_max lower than FOO is not allowed. We could also forbid having FOO_max == FOO, but that doesn't have any odd side effects and it would require us to update several other tests, so let's keep it simple. Signed-off-by: Alberto Garcia <berto@igalia.com> Reviewed-by: Eric Blake <eblake@redhat.com> Message-id: 2f90f9ee58aa14b7bd985f67c5996b06e0ab6c19.1469693110.git.berto@igalia.com Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
This commit is contained in:
		
							parent
							
								
									aaa1e77ffa
								
							
						
					
					
						commit
						5fc8c052ce
					
				@ -394,6 +394,14 @@ static void test_max_is_missing_limit(void)
 | 
				
			|||||||
        cfg.buckets[i].max = 0;
 | 
					        cfg.buckets[i].max = 0;
 | 
				
			||||||
        cfg.buckets[i].avg = 100;
 | 
					        cfg.buckets[i].avg = 100;
 | 
				
			||||||
        g_assert(throttle_is_valid(&cfg, NULL));
 | 
					        g_assert(throttle_is_valid(&cfg, NULL));
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					        cfg.buckets[i].max = 30;
 | 
				
			||||||
 | 
					        cfg.buckets[i].avg = 100;
 | 
				
			||||||
 | 
					        g_assert(!throttle_is_valid(&cfg, NULL));
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					        cfg.buckets[i].max = 100;
 | 
				
			||||||
 | 
					        cfg.buckets[i].avg = 100;
 | 
				
			||||||
 | 
					        g_assert(throttle_is_valid(&cfg, NULL));
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user