An attribute's datatype can be an enumeration or CDATA, but it can't be both. You should remove the CDATA
:
<!ATTLIST city
district CDATA #REQUIRED
province (Southern|Western|Nothern|Eastern) "One Sri Lanka">
Source: http://www.xmlfiles.com/dtd/dtd_attributes.asp
However there's still a problem here. The value "One Sri Lanka" does not match any of the four values you have listed there. What is your thinking behind that?