Я всё-таки рекомендую побольше почитать на тему многопоточных приложений, ибо lock и volatile имеют совершенно разное назначение. Оператор lock обеспечивает, что внутри соответствующего блока может находиться не более одного потока одновременно. Ключевое слово volatile этого не обеспечивает, его основное назначение - предотвратить буферизацию значения переменной в контексте одного потока, при которой один поток может не получить изменения, выполненного в другом потоке. В данном случае volatile не сильно критично, его неиспользование может привести лишь к тому, что отрисовка выполнится по устаревшим данным, т. е. проигнорируются сколько-то обновлений. Ну и ещё одним косвенным последствием неиспользования volatile в этом случае станет дополнительный расход памяти на лишнюю копию коллекции, поскольку неактуальное значение указателя в некотором потоке будет препятствовать удалению этой коллекции из памяти garbage collector'ом.