From 4ea28bf0a57ab88d3c5e3aec81c28de53c6ebc5b Mon Sep 17 00:00:00 2001
From: Andreas Kling <awesomekling@gmail.com>
Date: Sat, 16 Feb 2019 12:13:43 +0100
Subject: Kernel: Add a simple shared memory API for two processes only.

And use this to implement shared bitmaps between WindowServer and clients.
---
 WindowServer/WSAPITypes.h           |  2 +-
 WindowServer/WSClientConnection.cpp | 16 +++++++++-------
 WindowServer/WSClientConnection.h   |  2 ++
 3 files changed, 12 insertions(+), 8 deletions(-)

(limited to 'WindowServer')

diff --git a/WindowServer/WSAPITypes.h b/WindowServer/WSAPITypes.h
index ac55785b53..7c5b4e0541 100644
--- a/WindowServer/WSAPITypes.h
+++ b/WindowServer/WSAPITypes.h
@@ -118,7 +118,7 @@ struct WSAPI_ServerMessage {
             WSAPI_Size size;
             size_t bpp;
             size_t pitch;
-            RGBA32* pixels;
+            int shared_buffer_id;
         } backing;
     };
 };
diff --git a/WindowServer/WSClientConnection.cpp b/WindowServer/WSClientConnection.cpp
index ea134818a1..aadb48b7eb 100644
--- a/WindowServer/WSClientConnection.cpp
+++ b/WindowServer/WSClientConnection.cpp
@@ -42,13 +42,12 @@ WSClientConnection* WSClientConnection::ensure_for_client_id(int client_id)
 WSClientConnection::WSClientConnection(int fd)
     : m_fd(fd)
 {
-    pid_t pid;
-    int rc = WSMessageLoop::the().server_process().sys$ioctl(m_fd, 413, (int)&pid);
+    int rc = WSMessageLoop::the().server_process().sys$ioctl(m_fd, 413, (int)&m_pid);
     ASSERT(rc == 0);
 
     {
         InterruptDisabler disabler;
-        auto* process = Process::from_pid(pid);
+        auto* process = Process::from_pid(m_pid);
         ASSERT(process);
         m_process = process->make_weak_ptr();
         m_client_id = (int)process;
@@ -85,9 +84,12 @@ void WSClientConnection::post_message(const WSAPI_ServerMessage& message)
 
 RetainPtr<GraphicsBitmap> WSClientConnection::create_bitmap(const Size& size)
 {
-    if (!m_process)
-        return nullptr;
-    return GraphicsBitmap::create(*m_process, size);
+    RGBA32* buffer;
+    int shared_buffer_id = current->sys$create_shared_buffer(m_pid, size.area() * sizeof(RGBA32), (void**)&buffer);
+    ASSERT(shared_buffer_id >= 0);
+    ASSERT(buffer);
+    ASSERT(buffer != (void*)-1);
+    return GraphicsBitmap::create_with_shared_buffer(shared_buffer_id, size, buffer);
 }
 
 void WSClientConnection::on_message(WSMessage& message)
@@ -369,7 +371,7 @@ void WSClientConnection::handle_request(WSAPIGetWindowBackingStoreRequest& reque
     response.backing.bpp = sizeof(RGBA32);
     response.backing.pitch = backing_store->pitch();
     response.backing.size = backing_store->size();
-    response.backing.pixels = reinterpret_cast<RGBA32*>(backing_store->client_region()->laddr().as_ptr());
+    response.backing.shared_buffer_id = backing_store->shared_buffer_id();
     WSMessageLoop::the().post_message_to_client(request.client_id(), response);
 }
 
diff --git a/WindowServer/WSClientConnection.h b/WindowServer/WSClientConnection.h
index b8b09018e2..c835014d30 100644
--- a/WindowServer/WSClientConnection.h
+++ b/WindowServer/WSClientConnection.h
@@ -32,6 +32,7 @@ public:
     WSMenuBar* app_menubar() { return m_app_menubar.ptr(); }
 
     int fd() const { return m_fd; }
+    pid_t pid() const { return m_pid; }
 
 private:
     virtual void on_message(WSMessage&) override;
@@ -61,6 +62,7 @@ private:
 
     int m_client_id { 0 };
     int m_fd { -1 };
+    pid_t m_pid { 0 };
 
     HashMap<int, OwnPtr<WSWindow>> m_windows;
     HashMap<int, OwnPtr<WSMenuBar>> m_menubars;
-- 
cgit v1.2.3