diff options
| author | Oleg Nesterov | 2012-08-06 14:49:56 +0200 |
|---|---|---|
| committer | Oleg Nesterov | 2012-08-28 18:21:17 +0200 |
| commit | 5e5be71ab3fd8bd2076606923791ece1634c199c (patch) | |
| tree | 3bfa62096792ec812d8b4e3aa3caa6b5662384da /mm | |
| parent | uprobes: Kill dup_mmap()->uprobe_mmap(), simplify uprobe_mmap/munmap (diff) | |
| download | kernel-qcow2-linux-5e5be71ab3fd8bd2076606923791ece1634c199c.tar.gz kernel-qcow2-linux-5e5be71ab3fd8bd2076606923791ece1634c199c.tar.xz kernel-qcow2-linux-5e5be71ab3fd8bd2076606923791ece1634c199c.zip | |
uprobes: Change uprobe_mmap() to ignore the errors but check fatal_signal_pending()
Once install_breakpoint() fails uprobe_mmap() "ignores" all other
uprobes and returns the error.
It was never really needed to to stop after the first error, and
in fact it was always wrong at least in -ENOTSUPP case.
Change uprobe_mmap() to ignore the errors and always return 0.
This is not what we want in the long term, but until we teach
the callers to handle the failure it would be better to remove
the pointless complications. And this doesn't look too bad, the
only "reasonable" error is ENOMEM but in this case the caller
should be oom-killed in the likely case or the system has more
serious problems.
However it makes sense to stop if fatal_signal_pending() == T.
In particular this helps if the task was oom-killed.
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Acked-by: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
Diffstat (limited to 'mm')
0 files changed, 0 insertions, 0 deletions