ADC maths gives wrong millivolts

On an AVR (16-bit int), this conversion gives nonsense for most readings. Why?

uint16_t raw = adc_read();          /* 0..1023 */
uint16_t mv  = raw * 5000u / 1023u;