Hello Maziar,

I think that the declaration in unit2.c should look as follows:

extern xdata char str1;

This is a completely different from
extern char str1 --> of "dummy" idata type

as fetching xdata and idata (dummy type) variables involve a completely different addressing instructions.

Regards,
Zbyszek