re: list_lru: per-node list infrastructure

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Dave,

The patch a8739514fa91: "list_lru: per-node list infrastructure" in -mm
has a signedness bug.

include/linux/list_lru.h
   116  static inline unsigned long
   117  list_lru_walk(struct list_lru *lru, list_lru_walk_cb isolate,
   118                void *cb_arg, unsigned long nr_to_walk)
   119  {
   120          long isolated = 0;
   121          int nid;
   122  
   123          for_each_node_mask(nid, lru->active_nodes) {
   124                  isolated += list_lru_walk_node(lru, nid, isolate,
   125                                                 cb_arg, &nr_to_walk);
   126                  if (nr_to_walk <= 0)
                            ^^^^^^^^^^^^^^^
nr_to_walk is unsigned so the timeout value from list_lru_walk_node() is
ULONG_MAX (it's not zero).

   127                          break;
   128          }
   129          return isolated;
   130  }

regards,
dan carpenter

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]