lkml.org 
[lkml]   [2013]   [Jul]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH] mm: for shm_open()/mmap() with OVERCOMMIT_NEVER, return -1 if no memory avail
Date
Otherwize if there is no left space on shmem device, there will be
"Bus error" when application will try to write to address space that was
returned by mmap(2)

This patch also preserve old behaviour if MAP_NORESERVE/VM_NORESERVE
isset.

So, with this patch, you will get next:

a)
$ echo 2 >| /proc/sys/vm/overcommit_memory
....
mmap() = MAP_FAILED;
....

b)
....
mmap(0, length, PROT_READ | PROT_WRITE, MAP_SHARED | MAP_NORESERVE) = !MAP_FAILED;
write()
killed by SIGBUS
....

c)
$ echo 0 >| /proc/sys/vm/overcommit_memory
....
mmap() = !MAP_FAILED;
write()
killed by SIGBUS
....

Signed-off-by: Azat Khuzhin <a3at.mail@gmail.com>
---
mm/shmem.c | 16 ++++++++++++++++
1 file changed, 16 insertions(+)

diff --git a/mm/shmem.c b/mm/shmem.c
index a87990c..965f4ba 100644
--- a/mm/shmem.c
+++ b/mm/shmem.c
@@ -32,6 +32,8 @@
#include <linux/export.h>
#include <linux/swap.h>
#include <linux/aio.h>
+#include <linux/statfs.h>
+#include <linux/path.h>

static struct vfsmount *shm_mnt;

@@ -1356,6 +1358,20 @@ out_nomem:

static int shmem_mmap(struct file *file, struct vm_area_struct *vma)
{
+ if (!(vma->vm_flags & VM_NORESERVE) &&
+ sysctl_overcommit_memory == OVERCOMMIT_NEVER) {
+ struct inode *inode = file_inode(file);
+ struct kstatfs sbuf;
+ u64 size;
+
+ inode->i_sb->s_op->statfs(file->f_dentry, &sbuf);
+ size = sbuf.f_bfree * sbuf.f_bsize;
+
+ if (size < inode->i_size) {
+ return -ENOMEM;
+ }
+ }
+
file_accessed(file);
vma->vm_ops = &shmem_vm_ops;
return 0;
--
1.7.10.4


\
 
 \ /
  Last update: 2013-07-30 22:21    [W:0.067 / U:0.048 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site