kernel 3.04:这段代码是不是有点儿小问题？

jiabo2011

static void build_zonelists(pg_data_t *pgdat)
{
        int node, local_node;
        enum zone_type j;
        struct zonelist *zonelist;

        local_node = pgdat->node_id;

        zonelist = &pgdat->node_zonelists[0];
        j = build_zonelists_node(pgdat, zonelist, 0, MAX_NR_ZONES - 1);

        /*
         * Now we build the zonelist so that it contains the zones
         * of all the other nodes.
         * We don't want to pressure a particular node, so when
         * building the zones for node N, we make sure that the
         * zones coming right after the local ones are those from
         * node N+1 (modulo N)
         */
        for (node = local_node + 1; node < MAX_NUMNODES; node++) {
                if (!node_online(node))
                        continue;
                j = build_zonelists_node(NODE_DATA(node), zonelist, j,
                                                        MAX_NR_ZONES - 1);
        }
        for (node = 0; node < local_node; node++) {
                if (!node_online(node))
                        continue;
                j = build_zonelists_node(NODE_DATA(node), zonelist, j,
                                                        MAX_NR_ZONES - 1);
        }

        zonelist->_zonerefs[j].zone = NULL;
        zonelist->_zonerefs[j].zone_idx = 0;
}

里面的enum zone_type j 是不是应该是int j ??

liujunwei1234

回复 1# jiabo2011


    int和enum型没看出有啥本质区别，个人感觉enum使得代码更具可读性。

jiabo2011

int 和 enum在实现上是一样的，但是这里我觉得j和zone_type没有关系啊，j就是zonelist中zonerefs的数量而已。用了zone_type反而没有可读行啊～

lengyuex

int 和 enum在实现上是一样的，但是这里我觉得j和zone_type没有关系啊，j就是zonelist中zonerefs的数量而已 ...
jiabo2011 发表于 2011-11-14 09:05

    这样可以限制j的范围

jiabo2011

做不到吧，zone_type的范围是MAX_NR_ZONES，而j的范围是MAX_ZONELISTS。

zylthinking

回复  jiabo2011


    int和enum型没看出有啥本质区别，个人感觉enum使得代码更具可读性。
liujunwei1234 发表于 2011-11-13 20:45

靠， 这个头像很眼熟， 想了想原来是csdn blog上的， 那个 blog 将 zx_wing 的中断的文章转载了， 然后标注成 原创