Re: cvs commit: src/sys/geom/eli g_eli.c
- From: John Baldwin <jhb@xxxxxxxxxxx>
- Date: Tue, 30 Jan 2007 16:36:36 -0500
On Monday 29 January 2007 15:19, Pawel Jakub Dawidek wrote:
I fully agree that there should be a clean KPI for this. What you
proposed if fine. Because of lack of such KPI geli has to handle HTT
CPUs which are turned off by default in releases also by abusing
scheduler internals. KPI you proposed would allow me to remove those
hacks. And I'm really all for it.
What you and Scott are missing is that when I implement a GEOM class,
I'm using what is available to do my work. I'm not going to educate
myself how schedulers work, implement nice and clean KPI to use it in
my class. I'm not saying it wouldn't be great to be able to do so, but I
don't have time for everything, unfortunately, and you guys should
understand that very well.
Something you seem to be missing though is that in general it's nice to not
just endulge in an endless series of hacks in your specific area of the
system. IWBN to at some point devote some time to help fix more general
problems that will benefit the system as a whole rather than adding hacks
that just benefit yourself and require someone else to clean up when they
eventually add the more-architected solution. We are all very busy, but if
everyone just did hacks and never put any effort into improving the general
infrastructure, we'd have a royal mess.
--
John Baldwin
_______________________________________________
freebsd-arch@xxxxxxxxxxx mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-arch
To unsubscribe, send any mail to "freebsd-arch-unsubscribe@xxxxxxxxxxx"
- Follow-Ups:
- Re: cvs commit: src/sys/geom/eli g_eli.c
- From: Poul-Henning Kamp
- Re: cvs commit: src/sys/geom/eli g_eli.c
- From: Pawel Jakub Dawidek
- Re: cvs commit: src/sys/geom/eli g_eli.c
- References:
- Re: cvs commit: src/sys/geom/eli g_eli.c
- From: Pawel Jakub Dawidek
- Re: cvs commit: src/sys/geom/eli g_eli.c
- Prev by Date: vimage and modules
- Next by Date: Re: cvs commit: src/sys/geom/eli g_eli.c
- Previous by thread: Re: cvs commit: src/sys/geom/eli g_eli.c
- Next by thread: Re: cvs commit: src/sys/geom/eli g_eli.c
- Index(es):