| |
ChristopherJam
Registered: Aug 2004
Posts: 1423 |
SEI considered harmful.
Once more for the people at the back:
Don't wrap your interrupt initialisation with SEI/CLI.
It's a bad way to avoid interrupts being dispatched while you're changing the IRQ pointer, because it then requires additional code to acknowledge any pending CIA IRQs that might trigger between the SEI being executed and you turning off the CIA timer or interrupt enable.
If you're coming from BASIC, just do this instead:
lda #$7f
sta $dc0d
..then change $01, set your IRQ pointer(s), and init your own raster and/or timer IRQ(s)
If you're coming from a previous part, ask your teammate to turn off any IRQs and acknowedge any requests before they pass the baton (cf 'IRQ recovery' link below) - then all you need to do is set up your own IRQ.
As previously flogged to death at
TIL: The instruction after SEI can be executed before a pending IRQ is handled
C-64 coding cargo cults
Best practice IRQ recovery |
|
... 36 posts hidden. Click here to view all posts....
|
| |
chatGPZ
Registered: Dec 2001
Posts: 11523 |
I tend to store $7f to $dd0d as well, knowing that it isnt really required at all. And indeed, once you link stuff together, those SEI/CLI things become a super annoying source of occasional bugs. Just don't. |
| |
JackAsser
Registered: Jun 2002
Posts: 2038 |
Exacly Groepaz. |
| |
Cruzer
Registered: Dec 2001
Posts: 1051 |
Quoting MagerValpThe only thing that's enabled at the basic prompt is the CIA1 timer irq. None of the cartridges I know of enable anything else, and if they did it'd break a shit ton of software. Nice to know, thanks! It's gonna be hard to change the habit of copy/pasting the same cargo cult code I've been using forever, but I'll try. |
| |
Burglar
Registered: Dec 2004
Posts: 1137 |
y'all make it sound too easy, there's music sync to worry about too, frameskips are fugly. |
| |
JackAsser
Registered: Jun 2002
Posts: 2038 |
Quote: y'all make it sound too easy, there's music sync to worry about too, frameskips are fugly.
That’s what you avoid by NOT doing SEI/CLI |
| |
Burglar
Registered: Dec 2004
Posts: 1137 |
Quote: That’s what you avoid by NOT doing SEI/CLI
yes, but it's not the full story, u dont know when (rasterline) u arrive, so u also dont know if music played this frame or not, there may be multiple irqs.
and u dont want to modify the rasterline the music is playing at too much (or at least not at once). |
| |
Cruzer
Registered: Dec 2001
Posts: 1051 |
Quoting Burglaryes, but it's not the full story, u dont know when (rasterline) u arrive, so u also dont know if music played this frame or not, there may be multiple irqs.
and u dont want to modify the rasterline the music is playing at too much (or at least not at once). Whatcha talking about? Not doing SEI/CLI fixes all this. Just remove SEI/CLI from your code and it automatically becomes a perfectly linked and sync'ed trackmo. :D |
| |
chatGPZ
Registered: Dec 2001
Posts: 11523 |
Just use a proper dubstep soundtrack |
| |
Krill
Registered: Apr 2002
Posts: 3098 |
Is it this time of the year again. But lemme try to add a twist to this very old discussion.
A more or less interesting detail is that when coming from BASIC, the VIC raster interrupt flag is ALWAYS set.
Meaning that when you don't clear the latch by writing a 1-bit to $D019 bit 0, the interrupt will trigger immediately after unmasking it via LDA #1 : STA $D01A.
Take note that a raster interrupt may be flagged in the 3 (or 4?) cycles between clearing the latch and unmasking the interrupt.
I'll stop here and let yous dwell on this for a bit as to what this might or might not mean for timing-sensitive routines like stable-raster clockslides. =) |
| |
Martin Piper
Registered: Nov 2007
Posts: 739 |
I think, in the C64 world, the SEI is there because the IRQ level interrupt can have a trigger that comes from more than one source, it's not just the CIA it's also the VIC, for example.
This means that technically it's not possible to completely stop all IRQ sources with just one store to $dc0d. It's not an atomic operation because you also need to stop the VIC trigger as well.
In theory it could be the case that even if the $dc0d source is disabled in the mainline (outside the IRQ), a raster IRQ could be triggered which then enables the $dc0d source before it returns, which then means a mainline VIC trigger disable would not guarantee the IRQs are really off.
The SEI then basically is an atomic way of guaranteeing that IRQs are stopped before then having to disable their triggers, and acknowledge those potential triggers, before continuing.
Of course this doesn't handle the case when an NMI could enable a raster or CIA IRQ trigger. However at least for NMI triggers then don't happen because of the VIC and their control is via $dd0d. |
| Previous - 1 | 2 | 3 | 4 | 5 - Next |
