Bugfix for binder, page_owner and slub

Signed-off-by: quic_chahuan <[email protected]>

Author: quic-chahuan

USAGE.md

@@ -253,6 +253,21 @@ PID      Vss        Rss        Pss        Uss        Swap       Comm
 2865     1021.14MB  34.44MB    1.68MB     0B         25.35MB    com.qti.diagservices
 2292     1017.32MB  33.75MB    1.64MB     0B         25.34MB    com.qualcomm.qti.aonutility
 ```
+
+### procrank -c
+```
+crash> mod -s zram ./lib/zram.ko
+     MODULE       NAME                             BASE            SIZE  OBJECT FILE
+ffffffd4d3758440  zram                       ffffffd4d3752000     49152  ./lib/zram.ko
+crash> mod -s zsmalloc ./lib/zsmalloc.ko
+     MODULE       NAME                             BASE            SIZE  OBJECT FILE
+ffffffd4d34c6200  zsmalloc                   ffffffd4d34bf000     57344  ./lib/zsmalloc.ko
+
+crash> procrank -c
+PID      Comm                 Cmdline
+1        init                 /system/bin/init
+```
+
 ## vmalloc
 This command dumps the vmalloc info.
 

memory/slub.cpp

@@ -351,16 +351,18 @@ void Slub::parser_slab_caches(){
     cache_list.clear();
     if (csymbol_exists("depot_index")){
         depot_index = read_int(csymbol_value("depot_index"),"depot_index");
-    }else if (csymbol_exists("pool_index")){
+    } else if (csymbol_exists("pool_index")){
         depot_index = read_int(csymbol_value("pool_index"),"pool_index");
+    } else if(csymbol_exists("pools_num")){ /* 6.12 and later */
+        depot_index = read_int(csymbol_value("pools_num"), "pools_num");
     }
     if (!depot_index){
         fprintf(fp, "cannot get depot_index\n");
         return;
     }
     if (csymbol_exists("stack_slabs")){
         stack_slabs = csymbol_value("stack_slabs");
-    }else if (csymbol_exists("stack_pools")){/* 6.3 and later */
+    } else if (csymbol_exists("stack_pools")){/* 6.3 and later */
         stack_slabs = csymbol_value("stack_pools");
     }
     if (!stack_slabs){

pageowner/pageowner.cpp

@@ -75,9 +75,10 @@ bool Pageowner::is_enable_pageowner(){
         return false;
     }
     int inited;
+    // see set_page_owner in page_owner.h
     try_get_symbol_data(TO_CONST_STRING("page_owner_inited"), sizeof(int), &inited);
     if (inited != 1){
-        fprintf(fp, "cannot find page_owner_inited\n");
+        fprintf(fp, "page_owner is disabled, pls check the page_owner=on in cmdline\n");
         return false;
     }
     return true;
@@ -667,6 +668,8 @@ void Pageowner::parser_all_pageowners(){
         depot_index = read_int(csymbol_value("depot_index"),"depot_index");
     }else if (csymbol_exists("pool_index")){
         depot_index = read_int(csymbol_value("pool_index"),"pool_index");
+    } else if(csymbol_exists("pools_num")){
+        depot_index = read_int(csymbol_value("pools_num"), "pools_num");
     }
     if (csymbol_exists("stack_slabs")){
         stack_slabs = csymbol_value("stack_slabs");

plugin.cpp

@@ -261,7 +261,6 @@ std::vector<ulong> PaserPlugin::for_each_rbtree(ulong rb_root,int offset){
     retrieve_list(treeList, cnt);
     for (int i = 0; i < cnt; ++i) {
         if (!is_kvaddr(treeList[i]))continue;
-        // LOGI("node addr:%lx\n",treeList[i]);
         treeList[i] -= td.node_member_offset;
         res.push_back(treeList[i]);
     }
@@ -284,6 +283,12 @@ std::vector<ulong> PaserPlugin::for_each_list(ulong list_head,int offset){
     struct list_data ld;
     BZERO(&ld, sizeof(struct list_data));
     ld.flags |= LIST_ALLOCATE;
+    /*
+    case : invalid list entry: 4000000000000000
+    readflag = ld->flags & RETURN_ON_LIST_ERROR ? (RETURN_ON_ERROR|QUIET) : FAULT_ON_ERROR; in tools.c
+    Even if the list is incomplete, we should ensure that the existing elements can be used normally.
+    */
+    // ld.flags |= RETURN_ON_LIST_ERROR;
     readmem(list_head, KVADDR, &ld.start,sizeof(ulong), TO_CONST_STRING("for_each_list list_head"), FAULT_ON_ERROR);
     ld.end = list_head;
     // ld.member_offset = offset;