Re: [PATCH v8 5/9] hugetlb: disable region_add file_region coalescing

[Wenkuan Wang <wwk0817@xxxxxxxxx>]

On 10/30/19 9:36 AM, Mina Almasry wrote:

+} 
+ 
 /* Must be called with resv->lock held. Calling this with count_only == true 
  * will count the number of pages to be added but will not modify the linked 
- * list. 
+ * list. If regions_needed != NULL and count_only == true, then regions_needed 
+ * will indicate the number of file_regions needed in the cache to carry out to 
+ * add the regions for this range. 
  */ 
 static long add_reservation_in_range(struct resv_map *resv, long f, long t, 

Hi  Mina,

Would you please share which tree this patch set used? this patch 5/9 can't be applied with Linus's tree and add_reservation_in_range can't be found.

Thanks
Wenkuan

-                     bool count_only) 
+                     long *regions_needed, bool count_only) 
 { 
-    long chg = 0; 
+    long add = 0; 
     struct list_head *head = &resv->regions; 
+    long last_accounted_offset = f; 
     struct file_region *rg = NULL, *trg = NULL, *nrg = NULL; 
  
-    /* Locate the region we are before or in. */ 
-    list_for_each_entry (rg, head, link) 
-        if (f <= rg->to) 
-            break;