Re: [BTT PATCH] Clean up all memory leaks

["Alan D. Brunelle" <alan.brunelle@hp.com>]

[-- Attachment #1: Type: text/plain, Size: 166 bytes --]

Alan D. Brunelle wrote:
> This has a "small" bug in it - for some reason the .dat file 
> containing "ranges" now contains every IO.
>
> Looking into it...
>
> Alan


[-- Attachment #2: fix-ranges --]
[-- Type: text/plain, Size: 606 bytes --]

From: Alan D. Brunelle <Alan.Brunelle@hp.com>

Fix range check: use proper data type for comparison

Signed-off-by: Alan D. Brunelle <Alan.Brunelle@hp.com>
---

 btt/inlines.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/btt/inlines.h b/btt/inlines.h
index 3c8deac..edb2182 100644
--- a/btt/inlines.h
+++ b/btt/inlines.h
@@ -52,7 +52,7 @@ static inline void update_range(struct list_head *head_p, __u64 time)
 		if (time < rip->end)
 			return;
 
-		if ((time - rip->end) < range_delta) {
+		if (BIT_TIME(time - rip->end) < range_delta) {
 			rip->end = time;
 			return;
 		}