> + /* We pin the BO to system memory so it won't be moved during > + * the update and doesn't waste video ram. If the BO is already > + * located in VRAM, the pin operation will simply increment the > + * pin count. > */ s/located/pinned/ I guess? Maybe we should update drm_gem_vram_pin() to skip the drm_gem_vram_placement() call in case pl_flag is zero, so you can pin the bo where it happens to be at the moment. This will avoid moving around the bo for no good reason. We don't care where the bo is, we only want make sure it doesn't move while we access it. (same goes for the cursor patch, and I guess the mga patches too but I havn't looked yet). cheers, Gerd _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
