为什么zswap不直接用swp_entry_t做索引而要用pgoff_t?

内核编译和嵌入式产品的设计与开发
回复
科学之子
帖子: 2284
注册时间: 2013-05-26 6:58
系统: Debian 9

为什么zswap不直接用swp_entry_t做索引而要用pgoff_t?

#1

帖子 科学之子 » 2016-05-27 16:50

为什么zswap不直接用swp_entry_t做索引而要用pgoff_t?
pgoff_t好像是从swp_entry_t里面提取出来的
但pgoff_t和swp_entry_t都是几乎等于unsigned long类型
难道是为了防止以后类型的实现发生变化?
用swp_entry_t做索引的理由是"zswap_duplicate_entry"
感觉直接用swp_entry_t做索引,重复概率会更低.
或者是因为swap_duplicate_entry的发生概率很低(zswap.c的注释说罕见重复),所以为了避免Bug风险(上述说的"类型实现"改变),仅使用pgoff_t作为索引?
mm_types.h:

代码: 全选

typedef struct {
	unsigned long val;
} swp_entry_t;
swapops.h

代码: 全选

static inline swp_entry_t swp_entry(unsigned long type, pgoff_t offset)
{
	swp_entry_t ret;

	ret.val = (type << SWP_TYPE_SHIFT(ret)) |
			(offset & SWP_OFFSET_MASK(ret));
	return ret;
}
rosynirvana
帖子: 893
注册时间: 2011-02-14 17:46

Re: 为什么zswap不直接用swp_entry_t做索引而要用pgoff_t?

#2

帖子 rosynirvana » 2016-05-27 20:32

这是很自然的吧

一个swp_entry_t包含4项信息,其中的type用来指一个swap area,offset用来指在swap area中的位置(从代码的角度来说,指swap_info中swap_map项的index)
在zswap中,上游传过来的type用来对应rbtree root的index,相关代码在

代码: 全选

static struct zswap_tree *zswap_trees[MAX_SWAPFILES];
static int zswap_frontswap_store(unsigned type, pgoff_t offset,
                                 struct page *page)
{
         struct zswap_tree *tree = zswap_trees[type];
offset用来对应在树里面做索引,还是上面的函数中

代码: 全选

 entry->offset = offset;
这样把一个swp_entry_t对应到zswap中的数据去的
科学之子
帖子: 2284
注册时间: 2013-05-26 6:58
系统: Debian 9

Re: 为什么zswap不直接用swp_entry_t做索引而要用pgoff_t?

#3

帖子 科学之子 » 2016-05-28 8:38

rosynirvana 写了:这是很自然的吧

一个swp_entry_t包含4项信息,其中的type用来指一个swap area,offset用来指在swap area中的位置(从代码的角度来说,指swap_info中swap_map项的index)
在zswap中,上游传过来的type用来对应rbtree root的index,相关代码在

代码: 全选

static struct zswap_tree *zswap_trees[MAX_SWAPFILES];
static int zswap_frontswap_store(unsigned type, pgoff_t offset,
                                 struct page *page)
{
         struct zswap_tree *tree = zswap_trees[type];
offset用来对应在树里面做索引,还是上面的函数中

代码: 全选

 entry->offset = offset;
这样把一个swp_entry_t对应到zswap中的数据去的
4个?都有哪些?好像只有两个吧?
http://blog.sina.com.cn/s/blog_72a2ef140100pdeg.html
回复