[WIP][PATCH] Improve tdb add/modify/delete loop perf by almost 20%
Stefan Metzmacher
metze at samba.org
Tue Jul 26 19:29:26 UTC 2016
Am 26.07.2016 um 21:23 schrieb Stefan Metzmacher:
> Hi Andrew,
>
> I guess you have a logic bug in the following:
>
>> + if (ctx->unpack_flags & LDB_UNPACK_DATA_FLAG_NO_DATA_ALLOC) {
>> + struct ldb_val data_talloc = {
>> + .data = talloc_memdup(ctx->msg,
>> + data.dptr,
>> + data.dsize),
>> + .length = data.dsize
>> + };
>
>
> I changed the logic a bit to make it more clear why we do a memdup()
> on LDB_UNPACK_DATA_FLAG_NO_DATA_ALLOC.
>
> See
> https://git.samba.org/?p=metze/samba/wip.git;a=shortlog;h=refs/heads/master-tmp
>
> It has the version bumps for tdb, talloc, tevent and ldb ready to go
> for 4.5.0rc1.
>
> Please push if you're happy with the result.
BTW: I temporary added ldb_unpack_data_only_attr_list_flags() to
lib/ldb/ABI/ldb-1.1.26.sigs and removed it with the version bump
to 1.1.27 in order to get a nice "ldb: version 1.1.27" commit.
metze
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20160726/ee3db76c/signature.sig>
More information about the samba-technical
mailing list