Skip to content

Commit b05ce92

Browse files
abaireabaire
committed
nv2a: Download overlapping VGA framebuffer surfaces
1 parent 518208e commit b05ce92

File tree

10 files changed

+320
-88
lines changed

10 files changed

+320
-88
lines changed

‎hw/xbox/nv2a/pgraph/gl/renderer.c‎

Lines changed: 87 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,7 @@ static void pgraph_gl_process_pending(NV2AState *d)
114114

115115
if (qatomic_read(&r->downloads_pending) ||
116116
qatomic_read(&r->download_dirty_surfaces_pending) ||
117+
qatomic_read(&r->download_dirty_surfaces_in_range_pending) ||
117118
qatomic_read(&d->pgraph.sync_pending) ||
118119
qatomic_read(&d->pgraph.flush_pending) ||
119120
qatomic_read(&r->shader_cache_writeback_pending)) {
@@ -125,6 +126,13 @@ static void pgraph_gl_process_pending(NV2AState *d)
125126
if (qatomic_read(&r->download_dirty_surfaces_pending)) {
126127
pgraph_gl_download_dirty_surfaces(d);
127128
}
129+
if (qatomic_read(&r->download_dirty_surfaces_in_range_pending)) {
130+
pgraph_gl_download_surfaces_in_range_if_dirty(
131+
d, r->download_dirty_surfaces_in_range_start,
132+
r->download_dirty_surfaces_in_range_size);
133+
qatomic_set(&r->download_dirty_surfaces_in_range_pending, false);
134+
qemu_event_set(&r->dirty_surfaces_download_complete);
135+
}
128136
if (qatomic_read(&d->pgraph.sync_pending)) {
129137
pgraph_gl_sync(d);
130138
}
@@ -156,6 +164,46 @@ static void pgraph_gl_pre_savevm_wait(NV2AState *d)
156164
qemu_event_wait(&r->dirty_surfaces_download_complete);
157165
}
158166

167+
static bool pgraph_gl_have_overlapping_dirty_surfaces(NV2AState *d,
168+
hwaddr start, hwaddr size)
169+
{
170+
PGRAPHState *pg = &d->pgraph;
171+
PGRAPHGLState *r = pg->gl_renderer_state;
172+
173+
SurfaceBinding *surface;
174+
175+
hwaddr end = start + size - 1;
176+
177+
QTAILQ_FOREACH (surface, &r->surfaces, entry) {
178+
hwaddr surf_end = surface->vram_addr + surface->size - 1;
179+
bool overlapping = !(surface->vram_addr >= end || start >= surf_end);
180+
if (overlapping && surface->draw_dirty) {
181+
return true;
182+
}
183+
}
184+
185+
return false;
186+
}
187+
188+
static void pgraph_gl_download_overlapping_surfaces_trigger(NV2AState *d,
189+
hwaddr start,
190+
hwaddr size)
191+
{
192+
PGRAPHState *pg = &d->pgraph;
193+
PGRAPHGLState *r = pg->gl_renderer_state;
194+
195+
r->download_dirty_surfaces_in_range_start = start;
196+
r->download_dirty_surfaces_in_range_size = size;
197+
qemu_event_reset(&r->dirty_surfaces_download_complete);
198+
qatomic_set(&r->download_dirty_surfaces_in_range_pending, true);
199+
}
200+
201+
static void pgraph_gl_download_overlapping_surfaces_wait(NV2AState *d)
202+
{
203+
qemu_event_wait(
204+
&d->pgraph.gl_renderer_state->dirty_surfaces_download_complete);
205+
}
206+
159207
static void pgraph_gl_pre_shutdown_trigger(NV2AState *d)
160208
{
161209
PGRAPHState *pg = &d->pgraph;
@@ -173,33 +221,45 @@ static void pgraph_gl_pre_shutdown_wait(NV2AState *d)
173221
qemu_event_wait(&r->shader_cache_writeback_complete);
174222
}
175223

176-
static PGRAPHRenderer pgraph_gl_renderer = {
177-
.type = CONFIG_DISPLAY_RENDERER_OPENGL,
178-
.name = "OpenGL",
179-
.ops = {
180-
.init = pgraph_gl_init,
181-
.early_context_init = early_context_init,
182-
.finalize = pgraph_gl_finalize,
183-
.clear_report_value = pgraph_gl_clear_report_value,
184-
.clear_surface = pgraph_gl_clear_surface,
185-
.draw_begin = pgraph_gl_draw_begin,
186-
.draw_end = pgraph_gl_draw_end,
187-
.flip_stall = pgraph_gl_flip_stall,
188-
.flush_draw = pgraph_gl_flush_draw,
189-
.get_report = pgraph_gl_get_report,
190-
.image_blit = pgraph_gl_image_blit,
191-
.pre_savevm_trigger = pgraph_gl_pre_savevm_trigger,
192-
.pre_savevm_wait = pgraph_gl_pre_savevm_wait,
193-
.pre_shutdown_trigger = pgraph_gl_pre_shutdown_trigger,
194-
.pre_shutdown_wait = pgraph_gl_pre_shutdown_wait,
195-
.process_pending = pgraph_gl_process_pending,
196-
.process_pending_reports = pgraph_gl_process_pending_reports,
197-
.surface_update = pgraph_gl_surface_update,
198-
.set_surface_scale_factor = pgraph_gl_set_surface_scale_factor,
199-
.get_surface_scale_factor = pgraph_gl_get_surface_scale_factor,
200-
.get_framebuffer_surface = pgraph_gl_get_framebuffer_surface,
201-
}
202-
};
224+
static PGRAPHRenderer
225+
pgraph_gl_renderer = { .type = CONFIG_DISPLAY_RENDERER_OPENGL,
226+
.name = "OpenGL",
227+
.ops = {
228+
.init = pgraph_gl_init,
229+
.early_context_init = early_context_init,
230+
.finalize = pgraph_gl_finalize,
231+
.clear_report_value =
232+
pgraph_gl_clear_report_value,
233+
.clear_surface = pgraph_gl_clear_surface,
234+
.draw_begin = pgraph_gl_draw_begin,
235+
.draw_end = pgraph_gl_draw_end,
236+
.flip_stall = pgraph_gl_flip_stall,
237+
.flush_draw = pgraph_gl_flush_draw,
238+
.get_report = pgraph_gl_get_report,
239+
.image_blit = pgraph_gl_image_blit,
240+
.pre_savevm_trigger =
241+
pgraph_gl_pre_savevm_trigger,
242+
.pre_savevm_wait = pgraph_gl_pre_savevm_wait,
243+
.pre_shutdown_trigger =
244+
pgraph_gl_pre_shutdown_trigger,
245+
.pre_shutdown_wait = pgraph_gl_pre_shutdown_wait,
246+
.process_pending = pgraph_gl_process_pending,
247+
.process_pending_reports =
248+
pgraph_gl_process_pending_reports,
249+
.surface_update = pgraph_gl_surface_update,
250+
.have_overlapping_dirty_surfaces =
251+
pgraph_gl_have_overlapping_dirty_surfaces,
252+
.download_overlapping_surfaces_trigger =
253+
pgraph_gl_download_overlapping_surfaces_trigger,
254+
.download_overlapping_surfaces_wait =
255+
pgraph_gl_download_overlapping_surfaces_wait,
256+
.set_surface_scale_factor =
257+
pgraph_gl_set_surface_scale_factor,
258+
.get_surface_scale_factor =
259+
pgraph_gl_get_surface_scale_factor,
260+
.get_framebuffer_surface =
261+
pgraph_gl_get_framebuffer_surface,
262+
} };
203263

204264
static void __attribute__((constructor)) register_renderer(void)
205265
{

‎hw/xbox/nv2a/pgraph/gl/renderer.h‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,11 @@ typedef struct PGRAPHGLState {
187187
SurfaceBinding *color_binding, *zeta_binding;
188188
bool downloads_pending;
189189
QemuEvent downloads_complete;
190+
190191
bool download_dirty_surfaces_pending;
192+
bool download_dirty_surfaces_in_range_pending;
193+
hwaddr download_dirty_surfaces_in_range_start;
194+
hwaddr download_dirty_surfaces_in_range_size;
191195
QemuEvent dirty_surfaces_download_complete; // common
192196

193197
TextureBinding *texture_binding[NV2A_MAX_TEXTURES];
@@ -280,6 +284,7 @@ void pgraph_gl_reload_surface_scale_factor(PGRAPHState *pg);
280284
void pgraph_gl_render_surface_to_texture(NV2AState *d, SurfaceBinding *surface, TextureBinding *texture, TextureShape *texture_shape, int texture_unit);
281285
void pgraph_gl_set_surface_dirty(PGRAPHState *pg, bool color, bool zeta);
282286
void pgraph_gl_surface_download_if_dirty(NV2AState *d, SurfaceBinding *surface);
287+
void pgraph_gl_download_surfaces_in_range_if_dirty(NV2AState *d, hwaddr start, hwaddr size);
283288
SurfaceBinding *pgraph_gl_surface_get(NV2AState *d, hwaddr addr);
284289
SurfaceBinding *pgraph_gl_surface_get_within(NV2AState *d, hwaddr addr);
285290
void pgraph_gl_surface_invalidate(NV2AState *d, SurfaceBinding *e);

‎hw/xbox/nv2a/pgraph/gl/surface.c‎

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -628,6 +628,24 @@ void pgraph_gl_surface_download_if_dirty(NV2AState *d,
628628
}
629629
}
630630

631+
void pgraph_gl_download_surfaces_in_range_if_dirty(NV2AState *d, hwaddr start, hwaddr size)
632+
{
633+
PGRAPHState *pg = &d->pgraph;
634+
PGRAPHGLState *r = pg->gl_renderer_state;
635+
636+
SurfaceBinding *surface;
637+
638+
hwaddr end = start + size - 1;
639+
640+
QTAILQ_FOREACH(surface, &r->surfaces, entry) {
641+
hwaddr surf_end = surface->vram_addr + surface->size - 1;
642+
bool overlapping = !(surface->vram_addr >= end || start >= surf_end);
643+
if (overlapping) {
644+
pgraph_gl_surface_download_if_dirty(d, surface);
645+
}
646+
}
647+
}
648+
631649
static void bind_current_surface(NV2AState *d)
632650
{
633651
PGRAPHState *pg = &d->pgraph;

‎hw/xbox/nv2a/pgraph/gl/texture.c‎

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -302,15 +302,8 @@ void pgraph_gl_bind_textures(NV2AState *d)
302302
// FIXME: Restructure to support rendering surfaces to cubemap faces
303303

304304
// Writeback any surfaces which this texture may index
305-
hwaddr tex_vram_end = texture_vram_offset + length - 1;
306-
QTAILQ_FOREACH(surface, &r->surfaces, entry) {
307-
hwaddr surf_vram_end = surface->vram_addr + surface->size - 1;
308-
bool overlapping = !(surface->vram_addr >= tex_vram_end
309-
|| texture_vram_offset >= surf_vram_end);
310-
if (overlapping) {
311-
pgraph_gl_surface_download_if_dirty(d, surface);
312-
}
313-
}
305+
pgraph_gl_download_surfaces_in_range_if_dirty(
306+
d, texture_vram_offset, length);
314307
}
315308

316309
TextureKey key;

‎hw/xbox/nv2a/pgraph/null/renderer.c‎

Lines changed: 49 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -111,34 +111,61 @@ static void pgraph_null_surface_update(NV2AState *d, bool upload,
111111
{
112112
}
113113

114+
static bool pgraph_null_have_overlapping_dirty_surfaces(NV2AState *d,
115+
hwaddr start,
116+
hwaddr size)
117+
{
118+
return false;
119+
}
120+
121+
static void pgraph_null_download_overlapping_surfaces_trigger(NV2AState *d,
122+
hwaddr start,
123+
hwaddr size)
124+
{
125+
}
126+
127+
static void pgraph_null_download_overlapping_surfaces_wait(NV2AState *d)
128+
{
129+
}
130+
114131
static void pgraph_null_init(NV2AState *d, Error **errp)
115132
{
116133
PGRAPHState *pg = &d->pgraph;
117134
pg->null_renderer_state = NULL;
118135
}
119136

120-
static PGRAPHRenderer pgraph_null_renderer = {
121-
.type = CONFIG_DISPLAY_RENDERER_NULL,
122-
.name = "Null",
123-
.ops = {
124-
.init = pgraph_null_init,
125-
.clear_report_value = pgraph_null_clear_report_value,
126-
.clear_surface = pgraph_null_clear_surface,
127-
.draw_begin = pgraph_null_draw_begin,
128-
.draw_end = pgraph_null_draw_end,
129-
.flip_stall = pgraph_null_flip_stall,
130-
.flush_draw = pgraph_null_flush_draw,
131-
.get_report = pgraph_null_get_report,
132-
.image_blit = pgraph_null_image_blit,
133-
.pre_savevm_trigger = pgraph_null_pre_savevm_trigger,
134-
.pre_savevm_wait = pgraph_null_pre_savevm_wait,
135-
.pre_shutdown_trigger = pgraph_null_pre_shutdown_trigger,
136-
.pre_shutdown_wait = pgraph_null_pre_shutdown_wait,
137-
.process_pending = pgraph_null_process_pending,
138-
.process_pending_reports = pgraph_null_process_pending_reports,
139-
.surface_update = pgraph_null_surface_update,
140-
}
141-
};
137+
static PGRAPHRenderer
138+
pgraph_null_renderer = { .type = CONFIG_DISPLAY_RENDERER_NULL,
139+
.name = "Null",
140+
.ops = {
141+
.init = pgraph_null_init,
142+
.clear_report_value =
143+
pgraph_null_clear_report_value,
144+
.clear_surface = pgraph_null_clear_surface,
145+
.draw_begin = pgraph_null_draw_begin,
146+
.draw_end = pgraph_null_draw_end,
147+
.flip_stall = pgraph_null_flip_stall,
148+
.flush_draw = pgraph_null_flush_draw,
149+
.get_report = pgraph_null_get_report,
150+
.image_blit = pgraph_null_image_blit,
151+
.pre_savevm_trigger =
152+
pgraph_null_pre_savevm_trigger,
153+
.pre_savevm_wait = pgraph_null_pre_savevm_wait,
154+
.pre_shutdown_trigger =
155+
pgraph_null_pre_shutdown_trigger,
156+
.pre_shutdown_wait =
157+
pgraph_null_pre_shutdown_wait,
158+
.process_pending = pgraph_null_process_pending,
159+
.process_pending_reports =
160+
pgraph_null_process_pending_reports,
161+
.surface_update = pgraph_null_surface_update,
162+
.have_overlapping_dirty_surfaces =
163+
pgraph_null_have_overlapping_dirty_surfaces,
164+
.download_overlapping_surfaces_trigger =
165+
pgraph_null_download_overlapping_surfaces_trigger,
166+
.download_overlapping_surfaces_wait =
167+
pgraph_null_download_overlapping_surfaces_wait,
168+
} };
142169

143170
static void __attribute__((constructor)) register_renderer(void)
144171
{

‎hw/xbox/nv2a/pgraph/pgraph.c‎

Lines changed: 61 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -354,6 +354,55 @@ void pgraph_destroy(PGRAPHState *pg)
354354
qemu_mutex_destroy(&pg->lock);
355355
}
356356

357+
static void pgraph_download_vga_surfaces(NV2AState *d)
358+
{
359+
VGADisplayParams vga_display_params;
360+
d->vga.get_params(&d->vga, &vga_display_params);
361+
362+
hwaddr framebuffer_start = d->pcrtc.start + vga_display_params.line_offset;
363+
if (!framebuffer_start) {
364+
// Early exit during guest startup.
365+
return;
366+
}
367+
368+
PGRAPHState *pg = &d->pgraph;
369+
{
370+
DisplaySurface *display_surface = qemu_console_surface(d->vga.con);
371+
hwaddr framebuffer_size =
372+
surface_height(display_surface) * surface_stride(display_surface);
373+
374+
if (!pg->renderer->ops.have_overlapping_dirty_surfaces(
375+
d, framebuffer_start, framebuffer_size)) {
376+
return;
377+
}
378+
379+
trace_nv2a_pgraph_surface_download_vga_overlapping(framebuffer_start,
380+
framebuffer_size);
381+
bql_lock();
382+
qemu_mutex_lock(&d->pfifo.lock);
383+
qatomic_set(&d->pfifo.halt, true);
384+
385+
pg->renderer->ops.download_overlapping_surfaces_trigger(
386+
d, framebuffer_start, framebuffer_size);
387+
388+
pfifo_kick(d);
389+
qemu_mutex_unlock(&d->pfifo.lock);
390+
391+
bql_unlock();
392+
}
393+
394+
pg->renderer->ops.download_overlapping_surfaces_wait(d);
395+
396+
{
397+
bql_lock();
398+
qemu_mutex_lock(&d->pfifo.lock);
399+
qatomic_set(&d->pfifo.halt, false);
400+
pfifo_kick(d);
401+
qemu_mutex_unlock(&d->pfifo.lock);
402+
bql_unlock();
403+
}
404+
}
405+
357406
int nv2a_get_framebuffer_surface(void)
358407
{
359408
NV2AState *d = g_nv2a;
@@ -363,9 +412,21 @@ int nv2a_get_framebuffer_surface(void)
363412
qemu_mutex_lock(&pg->renderer_lock);
364413
assert(!pg->framebuffer_in_use);
365414
pg->framebuffer_in_use = true;
415+
416+
// Presumably in the case that there is no get_framebuffer_surface it's
417+
// impossible for there to be an overlapping modified GL/VK surface and we
418+
// could just bail early?
366419
if (pg->renderer->ops.get_framebuffer_surface) {
367420
s = pg->renderer->ops.get_framebuffer_surface(d);
368421
}
422+
423+
if (!s) {
424+
// If there is no framebuffer surface, it is possible that there are
425+
// some GL surfaces that overlap the VGA region. Such surfaces must be
426+
// downloaded before the default VGA texture is created.
427+
pgraph_download_vga_surfaces(d);
428+
}
429+
369430
qemu_mutex_unlock(&pg->renderer_lock);
370431

371432
return s;

‎hw/xbox/nv2a/pgraph/pgraph.h‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -119,6 +119,12 @@ typedef struct PGRAPHRenderer {
119119
void (*process_pending_reports)(NV2AState *d);
120120
void (*surface_flush)(NV2AState *d);
121121
void (*surface_update)(NV2AState *d, bool upload, bool color_write, bool zeta_write);
122+
bool (*have_overlapping_dirty_surfaces)(NV2AState *d, hwaddr start,
123+
hwaddr size);
124+
void (*download_overlapping_surfaces_trigger)(NV2AState *d,
125+
hwaddr start,
126+
hwaddr size);
127+
void (*download_overlapping_surfaces_wait)(NV2AState *d);
122128
void (*set_surface_scale_factor)(NV2AState *d, unsigned int scale);
123129
unsigned int (*get_surface_scale_factor)(NV2AState *d);
124130
int (*get_framebuffer_surface)(NV2AState *d);

0 commit comments

Comments
 (0)