access idmap cache directly from smbd
Gerald (Jerry) Carter
jerry at samba.org
Mon Aug 18 23:18:59 GMT 2008
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Jeremy Allison wrote:
>
> The only complaint that makes sense to me is that
> if winbindd isn't running or the lookup fails for
> some reason smbd has to call the legacy_convert_XX_to_YY
> functions and do the conversion itself.
>
> Currently smbd caches these in memory for reasons
> mentioned above, Volker's patch also puts them into
> the tdb cache (from my reading of it, corrections
> welcome). We might keep a memory cache for those
> conversions so we don't write into the winbindd
> cache if winbindd isn't running.
>
> But if we're returning these mappings to the
> client anyway we've already committed to them,
> so what harm storing them in the cache ?
I don't care as long as it's graceful failure over
if the entry is not in the cache. smbd should not
be writing to the cache.
For example,
smbd_check_cache();
if (!in_cache)
ask_winbindd();
do_legacy_mapping()
But in order to solve Volker's pdb_ldap.c issue, just make
smbd in memory cache a tdb separate from winbind's cache
and let smbd write all it wants.
It's you guys' call. I'm just expressing my thoughts.
The fact is that pdb_ldap should have had it's own cache
to begin with. That would have made more sense IMO.
We knew that doing ldap_search() all the time was a bad
idea.
I'm bowing out this thread as I don't think I have much
more to add nuless it's actual code.
cheers, jerry
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFIqgNjIR7qMdg1EfYRAhIbAKC8I+4pB+c7o5Wo2pq6NDqDMxAesQCeNnEw
wEE/Utx/MQTNfQpf0QPEqCM=
=z4H9
-----END PGP SIGNATURE-----
More information about the samba-technical
mailing list