[Webkit-unassigned] [Bug 77712] [JSC] Range of cache flush is not guaranteed by Linux kernel

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=77712





--- Comment #4 from hojong.han at samsung.com  2012-03-09 04:14:04 PST ---
(In reply to comment #3)
> (From update of attachment 130982 [details])
> Is size_t the right type to use for startPage and endPage?  It seems safer to use intptr_t or something.
> 
> Also it seems to me that you would be better off doing
> 
>   uintptr_t start = reinterpret_cast<uintptr_t>(code) & ~(pageSize() - 1);
> 
> and just adding pageSize() each round instead of doing the divide-and-multiply dance.  Dunno.
> 
> I'm not a reviewer, so please take these comments with a grain of salt.

Thank you. It's really necessary salt for me.
I'm going to put it into new patch.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.