aboutsummaryrefslogtreecommitdiff
path: root/src/ppusim
diff options
context:
space:
mode:
authorlonkaars <loek@pipeframe.xyz>2023-02-25 14:59:20 +0100
committerlonkaars <loek@pipeframe.xyz>2023-02-25 14:59:20 +0100
commit0e6f7148ab0987574c45e78cb8fd6ff72fdf103a (patch)
tree2ccc32a6708e079db94cab4911eb10f912756af3 /src/ppusim
parent5850f4ab766256791f72301349e30d4cd304c675 (diff)
optimize ppusim with threads
Diffstat (limited to 'src/ppusim')
-rw-r--r--src/ppusim/pixel.c19
-rw-r--r--src/ppusim/pixel.h4
-rw-r--r--src/ppusim/sim.c8
-rw-r--r--src/ppusim/work.c38
-rw-r--r--src/ppusim/work.h14
5 files changed, 62 insertions, 21 deletions
diff --git a/src/ppusim/pixel.c b/src/ppusim/pixel.c
index 2163c10..6e915e6 100644
--- a/src/ppusim/pixel.c
+++ b/src/ppusim/pixel.c
@@ -1,21 +1,11 @@
#include <SDL2/SDL.h>
#include <stdbool.h>
+#include "ppusim/work.h"
#include "ppu/internals.h"
#include "ppusim/mem.h"
-#include "ppusim/sim.h"
#include "ppu/consts.h"
-static void hh_ppusim_draw(SDL_Renderer* r, unsigned px, unsigned py, unsigned cr, unsigned cg, unsigned cb) {
- SDL_SetRenderDrawColor(r, cr, cg, cb, 255);
- SDL_RenderFillRect(r, &(SDL_Rect) {
- .x = px * HH_PPUSIM_UPSCALE_FACTOR,
- .y = py * HH_PPUSIM_UPSCALE_FACTOR,
- .w = HH_PPUSIM_UPSCALE_FACTOR,
- .h = HH_PPUSIM_UPSCALE_FACTOR
- });
-}
-
/* transform xy if tile is flipped */
static uint16_t hh_ppusim_apply_transform(unsigned x, unsigned y, bool fliph, bool flipv) {
unsigned tx = fliph ? HH_PPU_SPRITE_WIDTH - x - 1 : x;
@@ -77,7 +67,7 @@ static uint8_t hh_ppusim_fg_pixel(unsigned x, unsigned y) {
return cidx;
}
-void hh_ppusim_pixel(SDL_Renderer* r, unsigned x, unsigned y) {
+void hh_ppusim_pixel(uint8_t* s, unsigned x, unsigned y) {
uint8_t bg_cidx = hh_ppusim_bg_pixel(x, y);
uint8_t fg_cidx = hh_ppusim_fg_pixel(x, y);
uint8_t cidx = (fg_cidx & HH_MASK(3)) == 0 ? bg_cidx : fg_cidx;
@@ -87,6 +77,9 @@ void hh_ppusim_pixel(SDL_Renderer* r, unsigned x, unsigned y) {
HH_RESIZE(pal_rgb, 7, 4),
HH_RESIZE(pal_rgb, 3, 0)
};
- hh_ppusim_draw(r, x, y, rgb[0] << 4, rgb[1] << 4, rgb[2] << 4);
+
+ s[0] = rgb[0] << 4;
+ s[1] = rgb[1] << 4;
+ s[2] = rgb[2] << 4;
}
diff --git a/src/ppusim/pixel.h b/src/ppusim/pixel.h
index 8044560..475fe11 100644
--- a/src/ppusim/pixel.h
+++ b/src/ppusim/pixel.h
@@ -1,6 +1,6 @@
#pragma once
-#include <SDL2/SDL.h>
+#include "ppusim/work.h"
-void hh_ppusim_pixel(SDL_Renderer*, unsigned x, unsigned y);
+void hh_ppusim_pixel(uint8_t* s, unsigned x, unsigned y);
diff --git a/src/ppusim/sim.c b/src/ppusim/sim.c
index 8f01359..4226a9b 100644
--- a/src/ppusim/sim.c
+++ b/src/ppusim/sim.c
@@ -7,7 +7,7 @@
#include "ppu/ppu.h"
#include "ppusim/mem.h"
#include "ppusim/sim.h"
-#include "ppusim/pixel.h"
+#include "ppusim/work.h"
SDL_Window *g_hh_window = NULL;
SDL_Renderer *g_hh_renderer = NULL;
@@ -38,11 +38,7 @@ void hh_loop() {
hh_ppu_vblank_interrupt();
SDL_RenderClear(g_hh_renderer);
-
- for (unsigned x = 0; x < HH_PPU_SCREEN_WIDTH; x++)
- for (unsigned y = 0; y < HH_PPU_SCREEN_HEIGHT; y++)
- hh_ppusim_pixel(g_hh_renderer, x, y);
-
+ hh_ppusim_draw_frame(g_hh_renderer);
SDL_SetRenderDrawColor(g_hh_renderer, 0, 0, 0, 255);
SDL_RenderPresent(g_hh_renderer);
diff --git a/src/ppusim/work.c b/src/ppusim/work.c
new file mode 100644
index 0000000..2073907
--- /dev/null
+++ b/src/ppusim/work.c
@@ -0,0 +1,38 @@
+#include <SDL2/SDL.h>
+#include <pthread.h>
+#include <stdio.h>
+
+#include "ppusim/work.h"
+#include "ppusim/sim.h"
+#include "ppu/consts.h"
+#include "ppusim/pixel.h"
+
+pthread_t g_hh_ppusim_scanline_threads[HH_PPU_SCREEN_HEIGHT];
+hh_s_ppusim_screen g_hh_ppusim_screen;
+
+void* hh_ppusim_draw_scanline(void* scanline_context) {
+ unsigned scanline = (unsigned long) scanline_context;
+ for (unsigned x = 0; x < HH_PPU_SCREEN_WIDTH; x++)
+ hh_ppusim_pixel(g_hh_ppusim_screen[scanline][x], x, scanline);
+ return NULL;
+}
+
+void hh_ppusim_draw_frame(SDL_Renderer* renderer) {
+ for (unsigned y = 0; y < HH_PPU_SCREEN_HEIGHT; y++)
+ pthread_create(&g_hh_ppusim_scanline_threads[y], NULL, hh_ppusim_draw_scanline, (void*)(unsigned long)y);
+ for (unsigned y = 0; y < HH_PPU_SCREEN_HEIGHT; y++)
+ pthread_join(g_hh_ppusim_scanline_threads[y], NULL);
+
+ for (unsigned x = 0; x < HH_PPU_SCREEN_WIDTH; x++) {
+ for (unsigned y = 0; y < HH_PPU_SCREEN_HEIGHT; y++) {
+ SDL_SetRenderDrawColor(renderer, g_hh_ppusim_screen[y][x][0], g_hh_ppusim_screen[y][x][1], g_hh_ppusim_screen[y][x][2], 255);
+ SDL_RenderFillRect(renderer, &(SDL_Rect) {
+ .x = x * HH_PPUSIM_UPSCALE_FACTOR,
+ .y = y * HH_PPUSIM_UPSCALE_FACTOR,
+ .w = HH_PPUSIM_UPSCALE_FACTOR,
+ .h = HH_PPUSIM_UPSCALE_FACTOR,
+ });
+ }
+ }
+}
+
diff --git a/src/ppusim/work.h b/src/ppusim/work.h
new file mode 100644
index 0000000..1171a22
--- /dev/null
+++ b/src/ppusim/work.h
@@ -0,0 +1,14 @@
+#pragma once
+
+#include <SDL2/SDL.h>
+
+#include "ppu/consts.h"
+
+typedef uint8_t hh_s_ppusim_color[3];
+typedef hh_s_ppusim_color hh_s_ppusim_scanline[HH_PPU_SCREEN_WIDTH];
+typedef hh_s_ppusim_scanline hh_s_ppusim_screen[HH_PPU_SCREEN_HEIGHT];
+
+void* hh_ppusim_draw_scanline(void*);
+
+void hh_ppusim_draw_frame(SDL_Renderer*);
+