Re: Why do flush page cache twice when change TT's cache attribute

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



typo error. abundant=>redundant

2012/3/19 Scott Fang <donnie.fang@xxxxxxxxx>
In function ttm_tt_set_caching
,,,,,,,

    if (ttm->caching_state == tt_cached)
        drm_clflush_pages(ttm->pages, ttm->num_pages);

    for (i = 0; i < ttm->num_pages; ++i) {
        cur_page = ttm->pages[i];
        if (likely(cur_page != NULL)) {
            ret = ttm_tt_set_page_caching(cur_page,
                              ttm->caching_state,
                              c_state);
            if (unlikely(ret != 0))
                goto out_err;
        }
    }

    ttm->caching_state = c_state;

    return 0;


drm_clflush_pages flush cache one time.
then in the following function
ttm_tt_set_page_caching->set_memory_wc->change_page_attr_set: in this function may flush page cache again.

Does the code do some redundant flush, or there is some trick to these codes?

Thanks for the answer in advance.

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux