From c77ca79e899d9cff96f90a2c5b6a866cf02ed47e Mon Sep 17 00:00:00 2001 From: notaz Date: Sat, 6 Oct 2007 12:20:44 +0000 Subject: [PATCH] giz dblbuff, scanline mode git-svn-id: file:///home/notaz/opt/svn/PicoDrive@270 be3aeb3a-fb24-0410-a615-afba39da0efa --- platform/base_readme.txt | 17 ++++++----- platform/common/emu.h | 2 +- platform/common/menu.h | 1 + platform/gizmondo/emu.c | 63 ++++++++++++++++++++++++++-------------- platform/gizmondo/emu.h | 2 ++ platform/gizmondo/giz.c | 2 +- platform/gizmondo/menu.c | 10 ++++--- 7 files changed, 63 insertions(+), 34 deletions(-) diff --git a/platform/base_readme.txt b/platform/base_readme.txt index 5656e20b..078c3267 100644 --- a/platform/base_readme.txt +++ b/platform/base_readme.txt @@ -132,11 +132,10 @@ graphics problems for some games, so it's best to use 16bit one. #endif #ifdef GIZ -@@0. "Interlaced rendering" +@@0. "Scanline mode" This option was designed to work around slow framebuffer access (the Gizmondo's -main bottleneck) by drawing every other line (odd numbered lines during odd -numbered frames and even numbered lines during even frames). This improves -performance greatly, but introduces artifacts for fast scrolling games. +main bottleneck) by drawing every other line (even nummbered lines only). +This improves performance greatly, but looses detail. #endif #ifdef GP2X @@ -272,10 +271,14 @@ game to get sound. This is because most games initialize sound chips on startup, and this data is lost when sound chips are being enabled/disabled. #ifdef GIZ +@@1. "Double buffering" +Draws the display to offscreen buffer, and flips it with visible one when done. +Unfortunately this causes serious tearing, unless v-sync is used (next option). + @@1. "Wait for V-sync" -Waits for vertical sync before drawing. This option doesn't eliminate tearing -problems, because full framebuffer update takes much more time then the blanking -period lasts on Gizmondo.. +Waits for vertical sync before drawing (or flipping buffers, if previous option +is enabled). Emulation is stopped while waiting, so this causes large performance +hit. #endif @@1. "gzip savestates" diff --git a/platform/common/emu.h b/platform/common/emu.h index 96cd5501..f8261575 100644 --- a/platform/common/emu.h +++ b/platform/common/emu.h @@ -8,7 +8,7 @@ typedef struct { int EmuOpt; // LSb->MSb: use_sram, show_fps, enable_sound, gzip_saves, // squidgehack, no_save_cfg_on_exit, , 16_bit_mode // craigix_ram, confirm_save, show_cd_leds, confirm_load - // A_SNs_gamma, perfect_vsync, interlace + // A_SNs_gamma, perfect_vsync, giz_scanlines, giz_dblbuff int PicoOpt; // used for config saving only, see Pico.h int PsndRate; // ditto int PicoRegion; // ditto diff --git a/platform/common/menu.h b/platform/common/menu.h index 8a9f424d..cbf0d856 100644 --- a/platform/common/menu.h +++ b/platform/common/menu.h @@ -53,6 +53,7 @@ typedef enum MA_OPT_INTERLACED, /* giz */ MA_OPT2_GAMMA, MA_OPT2_A_SN_GAMMA, + MA_OPT2_DBLBUFF, /* giz */ MA_OPT2_VSYNC, MA_OPT2_ENABLE_Z80, MA_OPT2_ENABLE_YM2612, diff --git a/platform/gizmondo/emu.c b/platform/gizmondo/emu.c index e4fdab2c..440a8de3 100644 --- a/platform/gizmondo/emu.c +++ b/platform/gizmondo/emu.c @@ -52,8 +52,8 @@ void emu_getMainDir(char *dst, int len) static void emu_msg_cb(const char *msg) { - if (giz_screen == NULL) - giz_screen = Framework2D_LockBuffer(); + if (giz_screen != NULL) Framework2D_UnlockBuffer(); + giz_screen = Framework2D_LockBuffer(1); memset32((int *)((char *)giz_screen + 321*232*2), 0, 321*8*2/4); emu_textOut16(4, 232, msg); @@ -61,12 +61,15 @@ static void emu_msg_cb(const char *msg) /* assumption: emu_msg_cb gets called only when something slow is about to happen */ reset_timing = 1; + + Framework2D_UnlockBuffer(); + giz_screen = Framework2D_LockBuffer((currentConfig.EmuOpt&0x8000) ? 0 : 1); } -static void emu_state_cb(const char *str) +void emu_stateCb(const char *str) { - if (giz_screen == NULL) - giz_screen = Framework2D_LockBuffer(); + if (giz_screen != NULL) Framework2D_UnlockBuffer(); + giz_screen = Framework2D_LockBuffer(1); clearArea(0); blit("", str); @@ -157,7 +160,7 @@ static int EmuScan16(unsigned int num, void *sdata) if (!(Pico.video.reg[1]&8)) num += 8; DrawLineDest = (unsigned short *) giz_screen + 321*(num+1); - if ((currentConfig.EmuOpt&0x4000) && (num&1) == (Pico.m.frame_count&1)) + if ((currentConfig.EmuOpt&0x4000) && (num&1) == 0) // (Pico.m.frame_count&1)) return 1; // skip next line return 0; @@ -224,7 +227,7 @@ static void blit(const char *fps, const char *notice) } if (!(Pico.video.reg[12]&1)) lines_flags|=0x10000; if (currentConfig.EmuOpt&0x4000) - lines_flags|=(Pico.m.frame_count&1)?0x20000:0x40000; + lines_flags|=0x40000; // (Pico.m.frame_count&1)?0x20000:0x40000; vidCpy8to16((unsigned short *)giz_screen+321*8, PicoDraw2FB+328*8, localPal, lines_flags); } else if (!(emu_opt&0x80)) @@ -252,7 +255,7 @@ static void blit(const char *fps, const char *notice) lines_flags = (Pico.video.reg[1]&8) ? 240 : 224; if (!(Pico.video.reg[12]&1)) lines_flags|=0x10000; if (currentConfig.EmuOpt&0x4000) - lines_flags|=(Pico.m.frame_count&1)?0x20000:0x40000; + lines_flags|=0x40000; // (Pico.m.frame_count&1)?0x20000:0x40000; vidCpy8to16((unsigned short *)giz_screen+321*8, PicoDraw2FB+328*8, localPal, lines_flags); } @@ -270,14 +273,21 @@ static void blit(const char *fps, const char *notice) static void clearArea(int full) { if (giz_screen == NULL) - giz_screen = Framework2D_LockBuffer(); + giz_screen = Framework2D_LockBuffer(1); if (full) memset32(giz_screen, 0, 320*240*2/4); else memset32((int *)((char *)giz_screen + 321*232*2), 0, 321*8*2/4); + + if (currentConfig.EmuOpt&0x8000) { + Framework2D_UnlockBuffer(); + giz_screen = Framework2D_LockBuffer(0); + if (full) memset32(giz_screen, 0, 320*240*2/4); + else memset32((int *)((char *)giz_screen + 321*232*2), 0, 321*8*2/4); + } } static void vidResetMode(void) { - giz_screen = Framework2D_LockBuffer(); + giz_screen = Framework2D_LockBuffer(1); if (PicoOpt&0x10) { } else if (currentConfig.EmuOpt&0x80) { @@ -297,6 +307,11 @@ static void vidResetMode(void) Pico.m.dirtyPal = 1; memset32(giz_screen, 0, 321*240*2/4); + if (currentConfig.EmuOpt&0x8000) { + Framework2D_UnlockBuffer(); + giz_screen = Framework2D_LockBuffer(0); + memset32(giz_screen, 0, 321*240*2/4); + } Framework2D_UnlockBuffer(); giz_screen = NULL; } @@ -339,6 +354,7 @@ static void SkipFrame(void) PicoSkipFrame=0; } +/* forced frame to front buffer */ void emu_forcedFrame(void) { int po_old = PicoOpt; @@ -349,7 +365,7 @@ void emu_forcedFrame(void) currentConfig.EmuOpt |= 0x80; if (giz_screen == NULL) - giz_screen = Framework2D_LockBuffer(); + giz_screen = Framework2D_LockBuffer(1); PicoDrawSetColorFormat(1); PicoScan = EmuScan16; @@ -374,7 +390,7 @@ static void RunEvents(unsigned int which) if (PsndOut != NULL) FrameworkAudio_SetPause(1); if (giz_screen == NULL) - giz_screen = Framework2D_LockBuffer(); + giz_screen = Framework2D_LockBuffer(1); if ( emu_checkSaveFile(state_slot) && (( (which & 0x1000) && (currentConfig.EmuOpt & 0x800)) || // load (!(which & 0x1000) && (currentConfig.EmuOpt & 0x200))) ) // save @@ -392,7 +408,7 @@ static void RunEvents(unsigned int which) if (do_it) { osd_text(4, 232, (which & 0x1000) ? "LOADING GAME" : "SAVING GAME"); - PicoStateProgressCB = emu_state_cb; + PicoStateProgressCB = emu_stateCb; emu_SaveLoadGame((which & 0x1000) >> 12, 0); PicoStateProgressCB = NULL; Sleep(0); @@ -487,7 +503,7 @@ static void updateKeys(void) if (vol > 0) vol--; } FrameworkAudio_SetVolume(vol, vol); - sprintf(noticeMsg, "VOL: %02i", vol); + sprintf(noticeMsg, "VOL: %02i ", vol); noticeMsgTime = GetTickCount(); currentConfig.volume = vol; } @@ -735,18 +751,17 @@ void emu_Loop(void) updateKeys(); - if (giz_screen == NULL && (currentConfig.EmuOpt&0x80)) - giz_screen = Framework2D_LockBuffer(); + if (currentConfig.EmuOpt&0x80) + /* be sure correct framebuffer is locked */ + giz_screen = Framework2D_LockBuffer((currentConfig.EmuOpt&0x8000) ? 0 : 1); + if (!(PicoOpt&0x10)) PicoScan((unsigned) -1, NULL); PicoFrame(); - if (currentConfig.EmuOpt&0x2000) - Framework2D_WaitVSync(); - if (giz_screen == NULL) - giz_screen = Framework2D_LockBuffer(); + giz_screen = Framework2D_LockBuffer((currentConfig.EmuOpt&0x8000) ? 0 : 1); blit(fpsbuff, notice); @@ -755,6 +770,12 @@ void emu_Loop(void) giz_screen = NULL; } + if (currentConfig.EmuOpt&0x2000) + Framework2D_WaitVSync(); + + if (currentConfig.EmuOpt&0x8000) + Framework2D_Flip(); + // check time tval = GetTickCount(); tval_diff = (int)(tval - tval_thissec) << 8; @@ -784,7 +805,7 @@ void emu_Loop(void) // save SRAM if ((currentConfig.EmuOpt & 1) && SRam.changed) { - emu_state_cb("Writing SRAM/BRAM.."); + emu_stateCb("Writing SRAM/BRAM.."); emu_SaveLoadGame(0, 1); SRam.changed = 0; } diff --git a/platform/gizmondo/emu.h b/platform/gizmondo/emu.h index 86ab426b..581672ab 100644 --- a/platform/gizmondo/emu.h +++ b/platform/gizmondo/emu.h @@ -27,3 +27,5 @@ void emu_Loop(void); void emu_ResetGame(void); void emu_forcedFrame(void); +void emu_stateCb(const char *str); + diff --git a/platform/gizmondo/giz.c b/platform/gizmondo/giz.c index 25f5d58e..640fde54 100644 --- a/platform/gizmondo/giz.c +++ b/platform/gizmondo/giz.c @@ -60,7 +60,7 @@ void giz_init(HINSTANCE hInstance, HINSTANCE hPrevInstance) } // test screen - giz_screen = Framework2D_LockBuffer(); + giz_screen = Framework2D_LockBuffer(1); if (giz_screen == NULL) { lprintf_al("Framework2D_LockBuffer() failed\n"); diff --git a/platform/gizmondo/menu.c b/platform/gizmondo/menu.c index bf56c32e..eb50f9c8 100644 --- a/platform/gizmondo/menu.c +++ b/platform/gizmondo/menu.c @@ -92,8 +92,7 @@ static void menu_draw_begin(int use_bgbuff) static void menu_draw_end(void) { - Framework2D_WaitVSync(); - giz_screen = Framework2D_LockBuffer(); + giz_screen = Framework2D_LockBuffer(0); if (giz_screen == NULL) { lprintf_al("%s: Framework2D_LockBuffer() returned NULL\n", __FUNCTION__); @@ -102,6 +101,7 @@ static void menu_draw_end(void) memcpy32(giz_screen, (int *)menu_screen, 321*240*2/4); Framework2D_UnlockBuffer(); giz_screen = NULL; + Framework2D_Flip(1); } @@ -635,6 +635,7 @@ static int savestate_menu_loop(int is_loading) if(inp & BTN_PLAY) { // save/load if (menu_sel < 10) { state_slot = menu_sel; + PicoStateProgressCB = emu_stateCb; /* also suitable for menu */ if (emu_SaveLoadGame(is_loading, 0)) { strcpy(menuErrorMsg, is_loading ? "Load failed" : "Save failed"); return 1; @@ -991,6 +992,7 @@ menu_entry opt2_entries[] = { "Emulate Z80", MB_ONOFF, MA_OPT2_ENABLE_Z80, ¤tConfig.PicoOpt,0x0004, 0, 0, 1 }, { "Emulate YM2612 (FM)", MB_ONOFF, MA_OPT2_ENABLE_YM2612, ¤tConfig.PicoOpt,0x0001, 0, 0, 1 }, { "Emulate SN76496 (PSG)", MB_ONOFF, MA_OPT2_ENABLE_SN76496,¤tConfig.PicoOpt,0x0002, 0, 0, 1 }, + { "Double buffering", MB_ONOFF, MA_OPT2_DBLBUFF, ¤tConfig.EmuOpt, 0x8000, 0, 0, 1 }, { "Wait for V-sync (slow)", MB_ONOFF, MA_OPT2_VSYNC, ¤tConfig.EmuOpt, 0x2000, 0, 0, 1 }, { "gzip savestates", MB_ONOFF, MA_OPT2_GZIP_STATES, ¤tConfig.EmuOpt, 0x0008, 0, 0, 1 }, { "Don't save last used ROM", MB_ONOFF, MA_OPT2_NO_LAST_ROM, ¤tConfig.EmuOpt, 0x0020, 0, 0, 1 }, @@ -1057,7 +1059,7 @@ static void amenu_loop_options(void) menu_entry opt_entries[] = { { NULL, MB_NONE, MA_OPT_RENDERER, NULL, 0, 0, 0, 1 }, - { "Interlaced rend. (faster)", MB_ONOFF, MA_OPT_INTERLACED, ¤tConfig.EmuOpt, 0x4000, 0, 0, 1 }, + { "Scanline mode (faster)", MB_ONOFF, MA_OPT_INTERLACED, ¤tConfig.EmuOpt, 0x4000, 0, 0, 1 }, { "Scale low res mode", MB_ONOFF, MA_OPT_SCALING, ¤tConfig.scaling, 0x0001, 0, 3, 1 }, { "Accurate timing (slower)", MB_ONOFF, MA_OPT_ACC_TIMING, ¤tConfig.PicoOpt, 0x0040, 0, 0, 1 }, { "Accurate sprites (slower)", MB_ONOFF, MA_OPT_ACC_SPRITES, ¤tConfig.PicoOpt, 0x0080, 0, 0, 1 }, @@ -1562,7 +1564,7 @@ static void menu_prepare_bg(int use_game_bg) { // darken the active framebuffer if (giz_screen == NULL) - giz_screen = Framework2D_LockBuffer(); + giz_screen = Framework2D_LockBuffer(1); memset(bg_buffer, 0, 321*8*2); menu_darken_bg(bg_buffer + 321*8*2, (char *)giz_screen + 321*8*2, 321*224, 1); memset(bg_buffer + 321*232*2, 0, 321*8*2); -- 2.39.5