Comment 4 for bug 606811

Revision history for this message
Vadim Tkachenko (vadim-tk) wrote : Re: [Bug 606811] Re: Make BUF_READ_AHEAD_AREA a constant

Yasufumi,

The patch is to avoid expensive call
ut_min(64, ut_2_power_up(buf_pool->curr_size / 32))

function ut_2_power_up is CPU time consuming and not needed if we work
with big BP

On Fri, Jul 23, 2010 at 10:17 AM, Yasufumi Kinoshita
<email address hidden> wrote:
> Vadim,
>
> The patch is not
> "Make BUF_READ_AHEAD_AREA a constant".
>
> But "Don't start up when small BP for which 64 page BUF_READ_AHEAD_AREA is too large".
>
> So. I don't understand "working with default BP" meaning.
> The patch is for "don't start up when small BP such as default".
>
> And for small BP, BUF_READ_AHEAD_AREA should not be 64. it is too large.
> (e.g. BP = 8MB, read ahed 1MB? why do you think it is needed?)
>

--
Vadim Tkachenko, CTO, Percona Inc.
Phone +1-888-401-3403,  Skype: vadimtk153
Schedule meeting: http://tungle.me/VadimTkachenko

Percona Training Workshops
http://www.percona.com/training/