On Sun, 26 Sep 2010 16:29:23 -0700 (PDT), Hugh Dickins <hughd@xxxxxxxxxx> wrote: > If i915_get_object_get_pages() isn't doing its job, I think you need to > wonder why not - maybe somewhere doing a put_page or page_cache_release, > freeing one or all pages too soon? Thanks, that's a very useful bit of review. I surmised that after seeing corruption in a batch buffer after a swap storm that the pages were not as safe required. The fact that swapping was involved could have been a coincidence, except for the growing number of reports from other users reporting crashes in conjunction with swapping. So back to seeing whether i915_gem_object_[get/put]_pages is sufficient for our uses. -Chris -- Chris Wilson, Intel Open Source Technology Centre -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>