powerpc/vas: Take reference to PID and mm for user space windows

When process opens a window, its pid and tgid will be saved in the
vas_window struct. This window will be closed when the process exits.
The kernel handles NX faults by updating CSB or send SEGV signal to pid
of the process if the userspace csb addr is invalid.

In multi-thread applications, a window can be opened by a child thread,
but it will not be closed when this thread exits. It is expected that
the parent will clean up all resources including NX windows opened by
child threads. A child thread can send NX requests using this window
and could be killed before completion is reported. If the pid assigned
to this thread is reused while requests are pending, a failure SEGV
would be directed to the wrong place.

To prevent reusing the pid, take references to pid and mm when the window
is opened and release them when when the window is closed. Then if child
thread is not running, SEGV signal will be sent to thread group leader
(tgid).

Signed-off-by: Haren Myneni <haren@linux.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/1587016936.2275.1057.camel@hbabu-laptop

diff --git a/arch/powerpc/platforms/powernv/vas-debug.c b/arch/powerpc/platforms/powernv/vas-debug.c
index 44035a3d6414d7b0637a98ad01ec3f7c1da755d9..41fa90d2f4abdfecbfde352f1d4fb5253eed83e4 100644 (file)
--- a/arch/powerpc/platforms/powernv/vas-debug.c
+++ b/arch/powerpc/platforms/powernv/vas-debug.c
@@ -38,7 +38,7 @@ static int info_show(struct seq_file *s, void *private)
 
        seq_printf(s, "Type: %s, %s\n", cop_to_str(window->cop),
                                        window->tx_win ? "Send" : "Receive");
-       seq_printf(s, "Pid : %d\n", window->pid);
+       seq_printf(s, "Pid : %d\n", vas_window_pid(window));
 
 unlock:
        mutex_unlock(&vas_mutex);

diff --git a/arch/powerpc/platforms/powernv/vas-window.c b/arch/powerpc/platforms/powernv/vas-window.c
index dc46bf6d61494afb3486f3c6e18159352b576ea2..063cda2b540f05848b4cc0ea95257b0d767abb94 100644 (file)
--- a/arch/powerpc/platforms/powernv/vas-window.c
+++ b/arch/powerpc/platforms/powernv/vas-window.c
@@ -12,6 +12,8 @@
 #include <linux/log2.h>
 #include <linux/rcupdate.h>
 #include <linux/cred.h>
+#include <linux/sched/mm.h>
+#include <linux/mmu_context.h>
 #include <asm/switch_to.h>
 #include <asm/ppc-opcode.h>
 #include "vas.h"
@@ -876,8 +878,6 @@ struct vas_window *vas_rx_win_open(int vasid, enum vas_cop_type cop,
        rxwin->user_win = rxattr->user_win;
        rxwin->cop = cop;
        rxwin->wcreds_max = rxattr->wcreds_max ?: VAS_WCREDS_DEFAULT;
-       if (rxattr->user_win)
-               rxwin->pid = task_pid_vnr(current);
 
        init_winctx_for_rxwin(rxwin, rxattr, &winctx);
        init_winctx_regs(rxwin, &winctx);
@@ -1027,7 +1027,6 @@ struct vas_window *vas_tx_win_open(int vasid, enum vas_cop_type cop,
        txwin->tx_win = 1;
        txwin->rxwin = rxwin;
        txwin->nx_win = txwin->rxwin->nx_win;
-       txwin->pid = attr->pid;
        txwin->user_win = attr->user_win;
        txwin->wcreds_max = attr->wcreds_max ?: VAS_WCREDS_DEFAULT;
 
@@ -1057,6 +1056,40 @@ struct vas_window *vas_tx_win_open(int vasid, enum vas_cop_type cop,
                rc = set_thread_uses_vas();
                if (rc)
                        goto free_window;
+
+               /*
+                * Window opened by a child thread may not be closed when
+                * it exits. So take reference to its pid and release it
+                * when the window is free by parent thread.
+                * Acquire a reference to the task's pid to make sure
+                * pid will not be re-used - needed only for multithread
+                * applications.
+                */
+               txwin->pid = get_task_pid(current, PIDTYPE_PID);
+               /*
+                * Acquire a reference to the task's mm.
+                */
+               txwin->mm = get_task_mm(current);
+
+               if (!txwin->mm) {
+                       put_pid(txwin->pid);
+                       pr_err("VAS: pid(%d): mm_struct is not found\n",
+                                       current->pid);
+                       rc = -EPERM;
+                       goto free_window;
+               }
+
+               mmgrab(txwin->mm);
+               mmput(txwin->mm);
+               mm_context_add_copro(txwin->mm);
+               /*
+                * Process closes window during exit. In the case of
+                * multithread application, the child thread can open
+                * window and can exit without closing it. Expects parent
+                * thread to use and close the window. So do not need
+                * to take pid reference for parent thread.
+                */
+               txwin->tgid = find_get_pid(task_tgid_vnr(current));
        }
 
        set_vinst_win(vinst, txwin);
@@ -1257,8 +1290,17 @@ int vas_win_close(struct vas_window *window)
        poll_window_castout(window);
 
        /* if send window, drop reference to matching receive window */
-       if (window->tx_win)
+       if (window->tx_win) {
+               if (window->user_win) {
+                       /* Drop references to pid and mm */
+                       put_pid(window->pid);
+                       if (window->mm) {
+                               mm_context_remove_copro(window->mm);
+                               mmdrop(window->mm);
+                       }
+               }
                put_rx_win(window->rxwin);
+       }
 
        vas_window_free(window);
 

diff --git a/arch/powerpc/platforms/powernv/vas.h b/arch/powerpc/platforms/powernv/vas.h
index 88d084d3bfd9b182d20bd4f0c963dab825fa161a..2a040722dffa6e7ed60dcdf14c12074acb87a60d 100644 (file)
--- a/arch/powerpc/platforms/powernv/vas.h
+++ b/arch/powerpc/platforms/powernv/vas.h
@@ -355,7 +355,9 @@ struct vas_window {
        bool user_win;          /* True if user space window */
        void *hvwc_map;         /* HV window context */
        void *uwc_map;          /* OS/User window context */
-       pid_t pid;              /* Linux process id of owner */
+       struct pid *pid;        /* Linux process id of owner */
+       struct pid *tgid;       /* Thread group ID of owner */
+       struct mm_struct *mm;   /* Linux process mm_struct */
        int wcreds_max;         /* Window credits */
 
        char *dbgname;
@@ -430,6 +432,11 @@ extern void vas_window_init_dbgdir(struct vas_window *win);
 extern void vas_window_free_dbgdir(struct vas_window *win);
 extern int vas_setup_fault_window(struct vas_instance *vinst);
 
+static inline int vas_window_pid(struct vas_window *window)
+{
+       return pid_vnr(window->pid);
+}
+
 static inline void vas_log_write(struct vas_window *win, char *name,
                        void *regptr, u64 val)
 {