bug-bash
[Top][All Lists]
Advanced

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

Re: incremental history i/o? (was Re: A Feature Request for History)


From: Chet Ramey
Subject: Re: incremental history i/o? (was Re: A Feature Request for History)
Date: Fri, 24 Jun 2011 12:27:59 -0400
User-agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.2.17) Gecko/20110414 Lightning/1.0b2 Thunderbird/3.1.10

On 6/16/11 3:55 PM, Bradley M. Kuhn wrote:

> Actually, this relates to a thing I'd been looking into recently.  My
> bash history is 11MB now, and on some machines I have a noticeable load
> time as it reads the history.  I'd thought about adding support for
> incremental read to bash history/readline code.  Basically, it would
> load only the parts of the history it needed based on the history
> requested.  Obviously running "history" would read it all, but if
> reverse-search was requested, it could perhaps be read incrementally
> somehow.
> 
> Given that this would be a big change (esp. to make it seamless to
> existing readline API users), and would provide a feature clearly that
> isn't universally desired (ability to have really big history files),
> I'm asking, albeit with some trepidation, if such a rewrite of the
> history reading/writing code would likely be accepted, and if so what it
> would need to look like to be an acceptable patch.

It wouldn't be that big a change, given the existence of
read_history_range -- the substantive changes would be, at least,
localized.  The question is as you stated: is the work to put
it in, and the potential change to users who are satisfied with the
current behavior, worth the benefit to the admittedly small number of
users who would use it?  How many bash users have multi-megabyte
history files?

> 
> I noticed someone previously attempted to implement mmap() in the
> history code, but it's #ifdef'd out (IIRC from my investigations a few
> weeks ago).  I theorized that it was #ifdef'd out because implementing
> mmap() didn't help anything, since the history reading code immediately
> goes through the whole array of history anyway, so the file will be
> immediately read in to RAM anyway the way the code currently operates,
> even if you mmap() it.  In other words, just slapping mmap() in place
> wouldn't work (in fact, it's seem to have been tried and abandoned);
> more in-depth changes would be made.

It works, there's just not that big a performance difference between mmap
and reading an entire file into a huge buffer and going through it.

I'm just skeptical of the cost-benefit here.

-- 
``The lyf so short, the craft so long to lerne.'' - Chaucer
                 ``Ars longa, vita brevis'' - Hippocrates
Chet Ramey, ITS, CWRU    address@hidden    http://cnswww.cns.cwru.edu/~chet/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]