Bug 464045 - Review Request: sreadahead - Super Read Ahead
Summary: Review Request: sreadahead - Super Read Ahead
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Dan Horák
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Keywords:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-09-26 01:53 UTC by Matt Domsch
Modified: 2012-12-12 14:06 UTC (History)
17 users (show)

(edit)
Clone Of:
(edit)
Last Closed: 2012-12-12 05:00:27 UTC
dan: fedora-review?


Attachments (Terms of Use)

Description Matt Domsch 2008-09-26 01:53:37 UTC
Spec URL: http://domsch.com/linux/fedora/sreadahead/sreadahead.spec
SRPM URL: http://domsch.com/linux/fedora/sreadahead/sreadahead-0.02-1.fc9.src.rpm
Description: sreadahead speeds up booting by pre-reading disk blocks used during
previous boots.

Part of the 5-second bootup project.

Comment 1 Dan Horák 2008-09-26 06:05:47 UTC
I have improved the original makefile a bit to support using custom CFLAGS and installing, so the autotools patch can be dropped. You can see the results (patch, new spec file and new srpm) at http://fedora.danny.cz/sreadahead/

Comment 2 Matt Domsch 2008-09-26 12:19:28 UTC
It doesn't matter to me if it uses autotools, or if it uses Dan's (nice and small) Makefile.  I had sent the autotools patch to Arjan, but whatever upstream wishes to use is fine by me.  Arjan, see the patch and spec Dan provides in a comment to this bug and decide for yourself.

Thanks,
Matt

Comment 3 Valent Turkovic 2008-10-07 16:19:23 UTC
when ever this comes in rawhide I volunteering to test it and provide feedback.

Comment 4 Dan Horák 2008-10-07 16:25:06 UTC
Few questions from fedora-devel that should be answered - https://www.redhat.com/archives/fedora-devel-list/2008-October/msg00273.html

Comment 5 Arjan van de Ven 2008-10-21 14:48:06 UTC
on how to generate the file list:

find / -type f > filelist.txt


on the kernel patch; that's still in upstream discussion.

Comment 6 Karel Zak 2008-10-21 19:38:51 UTC
(In reply to comment #5)
> on how to generate the file list: 
> find / -type f > filelist.txt

 shouldn't be better to explicitly define (somewhere in /etc/sysconfig/sreadahed) relevant directories (/bin, /sbin, /usr, ..)?

 "find /" could be real overkill for people with large full disks (I guess we needn't porn pictures and mp3 files during the boot... :-)

Comment 7 Harald Hoyer 2009-01-28 13:24:48 UTC
any progress on the kernel patch?

Comment 8 Bill Nottingham 2009-01-28 15:40:14 UTC
It's an entirely different kernel patch now.

Comment 9 Harald Hoyer 2009-01-28 16:25:40 UTC
This one: http://lkml.org/lkml/2009/1/27/297 
together with sreadahead-1.0 http://lkml.org/lkml/2009/1/27/296

Comment 10 Kyle McMartin 2009-01-29 00:56:55 UTC
http://kyle.fedorapeople.org/sreadahead/
|`- http://kyle.fedorapeople.org/sreadahead/sreadahead.spec
`-- http://kyle.fedorapeople.org/sreadahead/sreadahead-1.0-1.fc10.src.rpm

Please let me know if I made a hash of the specfile. I'd like us to get this and the hooks for it into F11.

I'm not willing to merge the patch for the open tracing yet, as we would effectively be "guaranteeing" a userspace interface (not that debugfs really provides these assurances.) and I'd like to wait and see what actually gets merged upstream...

Comment 11 Bernie Innocenti 2009-03-10 14:10:32 UTC
Ping?  Is this review stuck?

Comment 12 Bill Nottingham 2009-03-10 14:20:46 UTC
It still depends on non-upstream kernel changes.

Comment 13 Kyle McMartin 2009-06-02 17:07:54 UTC
I've ported this to use the utrace-ftrace support we currently have in Fedora... Is that acceptable for rawhide/F-12?

I'll submit a patched srpm when I've done a bit more testing to make it somewhat more bulletproof.

Thanks!
 Kyle

Comment 14 Dan Horák 2009-06-03 06:12:53 UTC
(In reply to comment #13)
> I've ported this to use the utrace-ftrace support we currently have in
> Fedora... Is that acceptable for rawhide/F-12?

If that means that instead of relying on kernel patch for the proper function, you have reworked readahead itself, then it is fine.

> I'll submit a patched srpm when I've done a bit more testing to make it
> somewhat more bulletproof.

OK

Comment 15 Alexey Torkhov 2010-06-20 07:37:02 UTC
(In reply to comment #13)
> I've ported this to use the utrace-ftrace support we currently have in
> Fedora... Is that acceptable for rawhide/F-12?

Is patch is available somewhere or this is an abandoned piece of software?

Comment 16 Miroslav Suchý 2012-12-11 23:06:58 UTC
Ping? Any progress here? Or we can close this review?

Comment 17 Matt Domsch 2012-12-12 05:00:27 UTC
I've got no interest anymore.  I think this was a good idea 5 years ago that never really came to fruition.  Closing - if I'm wrong, someone else can re-open and take it.

Thanks,
Matt

Comment 18 Arjan van de Ven 2012-12-12 14:06:48 UTC
This actually got folded into systemd, so as a separate item it is obsolete.


Note You need to log in before you can comment on or make changes to this bug.