Working on a series of tidy up patches for gfx_v9 and found this block in gfx_v9_0_sw_init(). for (i = 0; i < adev->gfx.num_gfx_rings; i++) { ring = &adev->gfx.gfx_ring[i]; ring->ring_obj = NULL; sprintf(ring->name, "gfx"); Won't that make a series of rings all with the same name? Tom