Re: [PATCH] bugfix dax fault page should be no cache

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

 




The DAX means Direct Access for files,if the pages have cache,
the file is not direct access. But in normal condition,
we can not find the problem directly.

 In our case, we use ext2 filesystem with dax on the reserved mem.
we mmap the file and write some things ,the file is not changed immediately.

this is the wrong place to change that attribute:
	Do you mean
add
	ext4_file_mmap/ext2_file_mmap

	if (IS_DAX(file_inode(file))) {
		vma->vm_ops = &ext4_dax_vm_ops;
		vma->vm_flags |= VM_MIXEDMAP;
+	vma->vm_page_prot = pgprot_writecombine(vma->vm_page_prot);
	} else {
like this ???

On 2016/1/12 20:51, Matthew Wilcox wrote:
On Tue, Jan 12, 2016 at 07:43:45PM +0800, chenjie6@xxxxxxxxxx wrote:
From: chenjie <chenjie6@xxxxxxxxxx>

We show the pte:
	*pte=01600000159c8fd3
The page should be no_cache attr,it will be better.

I disagree.  What data do you have to show that the WC attribute would
be better than WB?

Even if we agree to that, this is the wrong place to change that
attribute, so the patch is NAKed.  But you need more than "it will
be better".

.


--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]