id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
1377	Split parallelism is broken in VTM (8a621a2b)	adamjw		"In a recent version of VTM (8a621a2b), the split-parallelism is broken. After enabling the macro ENABLE_SPLIT_PARALLELISM, the encoder would first crash with check on consistency between CodingStructure and Partitioner tree/mode type. After fixing those by properly extending the Partitioner::copyState method, a more problematic issue arises. In EncCU.cpp (1449-1488) chroma search for local dual tree is performed. To start-off this search, the currently best luma candidate is stored in the picture-level CodingStructure (cs) instance. When using split parallelism, multiple threads concurrently perform the RD search for the same area in the picture, meaning, possibly creating a race condition when copying to the picture-level cs.
To solve this problem, one would need to adapt the chroma search to not assume the currently-best luma candidate is stored in the picture-level cs but rather in some cs instance that is local to the thread (i.e. below the split that activates the split-level parallelism). As this might be complicated and no-one noticed so far I would recommend removing the feature."	defect	closed	minor	VTM-12.1	VTM	VTM-10.0	fixed		ksuehring XiangLi fbossen jvet@…