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