remove useless lock, the simplified user struct will never ever change itself, only the data in it