es muss auf jeden fall heißen am anfang:

volatile uint8_t x;
volatile uint8_t i; //Das = 0 kannst du weglassen. das macht der compiler automatisch.

und:
ISR (SIG_OVERFLOW0) ist eine mischung der alten und neuen schreibweise. normalerweise gehts glaub ich. aber beser du nimmst nur die alte schreibweise SIGNAL (SIG_OVERFLOW0) oder nur die neue ISR (OVERFLOW0_vect), bin mir bei der neuen abe rnicht ganz sicher.

abe rich denk das erste müsste dein problem sicher lösen. denn der compiler optimiert den code so dass in den funktionen nu rmehr kopien verwendet werden, die alt bleiben. und durch volatile verhinderst du das.