summaryrefslogtreecommitdiffstats
path: root/src/include/kernel
diff options
context:
space:
mode:
authorPatrick Williams <iawillia@us.ibm.com>2011-08-11 00:17:29 -0500
committerA. Patrick Williams III <iawillia@us.ibm.com>2011-08-16 08:11:55 -0500
commit82fa7a749fbd1d8f17891dbd97b17a3bdae36c53 (patch)
tree5e4857f1fc30e6b9506ac4973f597f17c8b16738 /src/include/kernel
parentab9e15e2e44eb52d4d0aa3602498a62db0cc0c37 (diff)
downloadtalos-hostboot-82fa7a749fbd1d8f17891dbd97b17a3bdae36c53.tar.gz
talos-hostboot-82fa7a749fbd1d8f17891dbd97b17a3bdae36c53.zip
Implement Kernel->User-space message bridge.
Change-Id: Icf6fc9e10b1c39e981dddf180607b710c597112b Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/249 Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
Diffstat (limited to 'src/include/kernel')
-rw-r--r--src/include/kernel/msg.H1
-rw-r--r--src/include/kernel/msghandler.H179
-rw-r--r--src/include/kernel/task.H19
-rw-r--r--src/include/kernel/vmmmgr.H10
4 files changed, 206 insertions, 3 deletions
diff --git a/src/include/kernel/msg.H b/src/include/kernel/msg.H
index 7a32197f5..954f4ab39 100644
--- a/src/include/kernel/msg.H
+++ b/src/include/kernel/msg.H
@@ -3,6 +3,7 @@
#include <kernel/types.h>
#include <sys/msg.h>
+#include <kernel/spinlock.H>
#include <util/locked/list.H>
#include <util/locked/queue.H>
diff --git a/src/include/kernel/msghandler.H b/src/include/kernel/msghandler.H
new file mode 100644
index 000000000..522ffd757
--- /dev/null
+++ b/src/include/kernel/msghandler.H
@@ -0,0 +1,179 @@
+/** @file msghandler.H
+ * @brief Interfaces for messaging from kernel-space to user-space.
+ */
+#ifndef __KERNEL_MSGHANDLER_H
+#define __KERNEL_MSGHANDLER_H
+
+#include <stdint.h>
+#include <kernel/types.h>
+#include <kernel/msg.H>
+#include <kernel/spinlock.H>
+#include <util/locked/list.H>
+
+// Forward declaration.
+namespace Systemcalls { void MsgRespond(task_t*); };
+
+/** @brief Struct used for holding pending message information. */
+struct MessageHandler_Pending
+{
+ typedef void* key_type;
+ /** Key used for duplicative association. */
+ key_type key;
+ /** (optional) Task deferred by this message. */
+ task_t* task;
+
+ // Linked list pointers used by Util::Locked::List.
+ MessageHandler_Pending* prev;
+ MessageHandler_Pending* next;
+};
+
+/** @class MessageHandler
+ * @brief Class which facilitiates bridging messages between kernel-space and
+ * user space.
+ *
+ * This class provides a 'send message' interface to kernel code to send a
+ * message into user-space. The message is a <type, key, data> tuple sent
+ * to the message queue associated with this object. The expected response
+ * from userspace is a <type, key, rc> tuple.
+ *
+ * The typical use-case of these interfaces is to send a <va, page> pair to a
+ * user-space resource provider to request a page of memory be read or written.
+ * A parameter to the 'send message' interface is an optional task which is to
+ * be deferred until the response is fully processed.
+ *
+ * The 'send message' interface will attempt to combine duplicate messages
+ * by associating message requests with the same key and only sending a
+ * single message to user-space for the key (while that key is outstanding).
+ * This behavior aids the typical use-case of these interfaces by combining
+ * multiple tasks page-faulting on a single page into a single request to the
+ * resource provider.
+ *
+ * When a response is received, via the user-space msg_respond system call,
+ * the 'recv message' interface will be called to handle the response. This
+ * interface will trigger an overrideable 'handle response' interface for
+ * each outstanding <key, task> pair. The default behavior is to resume the
+ * task on successful responses and kill the task on error responses. This
+ * behavior can be changed by overriding the 'handle response' interface.
+ *
+ * SMP safety:
+ * The constructor of this class requests the spinlock for the kernel
+ * subsystem associated with this message handling instance. Since the
+ * 'recv message' interface is the entry point into this subsystem, the
+ * spinlock will be obtained prior to handling the message receipt. When
+ * the 'send message' interface is called, the typical use-case is that
+ * the subsystem spinlock is already obtained (since the subsystem is
+ * requesting a message to be sent), so this object does not obtain the
+ * spinlock. This spinlock must be obtained prior to calling 'send
+ * message' in order for the combining of duplicate messages to be
+ * SMP-safe.
+ */
+class MessageHandler
+{
+ public:
+ /** @brief Results of the 'handle response' interface.
+ *
+ * These are used to communicate back to the 'recv message' interface
+ * what to do with the deferred thread.
+ *
+ * A result of UNHANDLED_RC will cause the default behavior of
+ * resuming the task on success and killing the task on error.
+ */
+ enum HandleResult
+ {
+ /** Response rc (if non-zero) has been successfully handled. */
+ SUCCESS,
+ /** Do not resume task yet, no matter the rc. */
+ CONTINUE_DEFER,
+ /** Nothing has been specifically handled for the rc. */
+ UNHANDLED_RC,
+ };
+
+ /** @brief Constructor.
+ * @param[in] i_lock - Subsystem lock for this message handler, such
+ * as VmmManager::lock.
+ * @param[in] i_msgq - Queue used to send messages into userspace.
+ *
+ * @note The ownership of these parameter instances is maintained by
+ * the caller and not cleaned up by this object upon destruction.
+ */
+ MessageHandler(Spinlock* i_lock, MessageQueue* i_msgq)
+ : iv_lock(i_lock), iv_msgq(i_msgq) {};
+
+ /** @brief Destructor.
+ * No behavior required since ownership is maintained elsewhere.
+ */
+ virtual ~MessageHandler() {};
+
+ /** @brief 'Send message' interface.
+ * Used to send a message into userspace.
+ *
+ * @param[in] i_type - Message type (from sys/msg.h).
+ * @param[in] i_key - Key (msg->data[0]) for the message.
+ * @param[in] i_data - Data (msg->data[1]) for the message.
+ * @param[in] i_task - Optional task being deferred due to this
+ * message.
+ *
+ * The result of this message is that a message will be created and
+ * inserted onto a user-space message queue, awaking the waiter if
+ * blocked. The task passed as a parameter, if not NULL, will be
+ * deferred.
+ */
+ void sendMessage(msg_sys_types_t i_type, void* i_key,
+ void* i_data, task_t* i_task);
+
+ /** @brief 'Handle response' interface.
+ * Overrideable handling of the response to the 'send message'.
+ *
+ * This is called once per previous 'send message' invocation for
+ * the same key even if duplicate keys were combined into a single
+ * user-space message.
+ *
+ * This class, and specifically this method, should be inherited /
+ * overridden if advanced handling is required for the response
+ * above resuming the deferred task.
+ *
+ * @param[in] i_type - The message type previously sent.
+ * @param[in] i_key - The key value for the received message.
+ * @param[in] i_task - The deferred task.
+ * @param[in] i_rc - The response rc from userspace.
+ *
+ * @return HandleResult - The desired behavior on the 'recv message'
+ * interface for this <key, task> pair.
+ */
+ virtual HandleResult handleResponse(msg_sys_types_t i_type, void* i_key,
+ task_t* i_task, int i_rc);
+
+ protected:
+ /** @brief 'Recv message' interface.
+ * Called by the msg_respond sys-call handler to relay the response
+ * from userspace.
+ *
+ * This interface will call handleResponse for each of the deferred
+ * <key, task> pairs and then, based on the return of handleResponse,
+ * resume or kill the deferred task.
+ *
+ * @param[in] i_msg - The message responded from user-space.
+ *
+ * @return Return code for msg_respond.
+ */
+ int recvMessage(msg_t* i_msg);
+
+ private:
+ /** Pointer to the subsystem lock. */
+ Spinlock* const iv_lock;
+ /** Message queue to relay messages to. */
+ MessageQueue* const iv_msgq;
+ /** Queue of pending user-space responses. */
+ Util::Locked::List<MessageHandler_Pending,
+ MessageHandler_Pending::key_type> iv_pending;
+
+ // Prevent copies.
+ MessageHandler(const MessageHandler&);
+ MessageHandler& operator=(const MessageHandler&);
+
+ public:
+ // Allow msg_respond handler to call recvMessage.
+ friend void Systemcalls::MsgRespond(task_t* t);
+};
+
+#endif
diff --git a/src/include/kernel/task.H b/src/include/kernel/task.H
index 6cd49028d..b72322d83 100644
--- a/src/include/kernel/task.H
+++ b/src/include/kernel/task.H
@@ -29,17 +29,17 @@ struct task_t
{
/** Pointer to the CPU this task is assigned to. */
cpu_t* cpu;
- /** Context information. This MUST stay here due to
+ /** Context information. This MUST stay here due to
* save-restore asm code. */
context_t context;
-
+
/** Task ID */
tid_t tid;
/** Determines if user-space would like this task pinned to a CPU.
* This value is considered a count of the number of times the pinned
* as been requested, so pinning can be used recursively. */
uint64_t affinity_pinned;
-
+
// Pointers for queue containers.
task_t* prev;
task_t* next;
@@ -47,4 +47,17 @@ struct task_t
enum { TASK_DEFAULT_STACK_SIZE = 4 };
+// Macros for manipulating task's saved contexts.
+#define TASK_GETARGN(t, n) (t->context.gprs[n+4])
+#define TASK_GETARG0(t) (TASK_GETARGN(t,0))
+#define TASK_GETARG1(t) (TASK_GETARGN(t,1))
+#define TASK_GETARG2(t) (TASK_GETARGN(t,2))
+#define TASK_GETARG3(t) (TASK_GETARGN(t,3))
+#define TASK_GETARG4(t) (TASK_GETARGN(t,4))
+#define TASK_GETARG5(t) (TASK_GETARGN(t,5))
+#define TASK_GETARG6(t) (TASK_GETARGN(t,6))
+#define TASK_GETARG7(t) (TASK_GETARGN(t,7))
+#define TASK_SETRTN(t, n) (t->context.gprs[3] = (n))
+
+
#endif
diff --git a/src/include/kernel/vmmmgr.H b/src/include/kernel/vmmmgr.H
index 7c8c63c22..4fc112f09 100644
--- a/src/include/kernel/vmmmgr.H
+++ b/src/include/kernel/vmmmgr.H
@@ -84,6 +84,13 @@ class VmmManager
VmmManager();
~VmmManager() {};
+ /** @brief Get spinlock for memory subsystem.
+ * This is useful for passing to a deferred user-space message
+ * handler so that the subsystem code is SMP-safe when the message
+ * response is obtained.
+ */
+ static Spinlock* getLock();
+
private:
Spinlock lock;
@@ -92,6 +99,9 @@ class VmmManager
bool _pteMiss(task_t*, uint64_t);
+ public:
+ friend class Block;
+
};
#endif
OpenPOWER on IntegriCloud