[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: SV: Re: Fwd: [PATCH] Fix authentication cache lookup
From:       David Faure <faure () kde ! org>
Date:       2006-07-26 16:19:49
Message-ID: 200607261819.50498.faure () kde ! org
[Download RAW message or body]

On Monday 24 July 2006 10:54, Anders Bergström wrote:
> > Leo Savernik <l.savernik@aon.at> skrev:
> > 
> > Do you imply that loading the document from the web is faster than looking it 
> > up in the cache? In this case, the patch only fixes a symptom and not the 
> > real cause.
> > 
> > mfg
> > Leo

> No, I don't. This cache-lookup is just for the authentication in kpasswdserver and \
> as you know most pages doesn't use any authentication hence it is totally waste of \
> time and when you have a page with 100s of objects(images) it will do a lookup for \
> each and every one. As I said this makes kded take 20-40% of my CPU (400MHz) while \
> loading a page(This goes to 0%  with the patch). The time is spent i kpasswdserver \
> and if I remember correctly most of that time was in the kwallet-backend which \
> probably could be faster.

This makes sense to me. I saw that we always ask kpasswdserver "is there a password \
for this url" even on sites that really don't need a password. So I agree with the \
general idea of the patch, but unfortunately I don't know the http.cc code enough to \
comment on the actual code change. Dawit would be the best person to comment on it I \
guess?

-- 
David Faure, faure@kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
 

 
> > Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


[prev in list] [next in list] [prev in thread] [next in thread]