From: "Vladimir 'φ-coder/phcoder' Serbinenko" <phcoder@gmail.com>
To: grub-devel@gnu.org
Subject: Re: ReiserFS reading performance patch
Date: Tue, 01 May 2012 20:16:30 +0200 [thread overview]
Message-ID: <4FA0287E.7090208@gmail.com> (raw)
In-Reply-To: <4FA02495.7040405@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1425 bytes --]
On 01.05.2012 19:59, Vladimir 'φ-coder/phcoder' Serbinenko wrote:
> On 27.04.2012 16:05, Albert Zeyer wrote:
>> Hi,
>>
>> A while ago, I improved the grub-mount patches a bit. Mostly:
> You've based your work on abandonded branch. All grub-mount parts were
> merged back in trunk a long time ago. At least half of the work you've
> done was unnecessary with newer version.
>> * Write a simplified compile.py script.
> Custom compile scripts are likely to break often. If there were any
> problem in our compile scripts for your platform it's better to announce
> it here.
>> * Remove nested functions. They don't work on Mac because of stack
>> execution protection.
> As I said, your approach isn't just ugly it results in bugs.
>> * Improve ReiserFS reading performance.
> I wrote reiserfs caching a long time ago but it didn'tmake any sizeable
> difference. As for making offset search a logarithmic, here is a patch
> by few simple modifications of get_item.
Patch contained few unrelated items. Take just reiserfs.c part. With the
patch reading 5G takes around a minute. Few more optimisations are still
possible though like making get_item scan bnode using binary search but
their gain is much smaller and they have no chance of making it into
2.00. While the previously attached patch is more in a gray are and I'm
hesitating.
>
--
Regards
Vladimir 'φ-coder/phcoder' Serbinenko
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 294 bytes --]
prev parent reply other threads:[~2012-05-01 18:16 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-27 14:05 ReiserFS reading performance patch Albert Zeyer
2012-04-27 18:36 ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-04-27 20:31 ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-04-28 10:06 ` Bean
2012-04-28 11:08 ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-05-01 17:59 ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-05-01 18:16 ` Vladimir 'φ-coder/phcoder' Serbinenko [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4FA0287E.7090208@gmail.com \
--to=phcoder@gmail.com \
--cc=grub-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.