Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • BIND BIND
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 633
    • Issues 633
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 87
    • Merge requests 87
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Artifacts
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • ISC Open Source ProjectsISC Open Source Projects
  • BINDBIND
  • Merge requests
  • !7281

TLS: use isc_buffer_t for send requests, reuse isc_buffer_t during send requests

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Artem Boldariev requested to merge artem-tls-use-isc-buffer into main Dec 30, 2022
  • Overview 11
  • Commits 1
  • Pipelines 6
  • Changes 2

This merge replaces ad-hoc code for send requests buffer management within TLS with the one based on isc_buffer_t.

To be precise, this merge request brings the idea of send requests object reuse further, by allowing the related dynamically allocated buffer to be reused as well.

Previous version of the code was trying to use pre-allocated small buffers to avoid extra allocations. The code would allocate a larger dynamic buffer when needed. There is no need to have ad-hoc code for this, as isc_buffer_t now provides this functionality internally.

Additionally to the above, the old version of the code lacked any logic to reuse the dynamically allocated buffers. Now, as we do not manage memory buffers, but isc_buffer_t objects, we can implement this strategy. It can be in particular helpful for longer lasting connections, as in this case the buffer will adjust itself to the size of the messages being transferred. That is, it is in particular useful for XoT, as Stream DNS happen to order send requests in such a way that the send request object will get reused.

Edited Dec 30, 2022 by Artem Boldariev
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: artem-tls-use-isc-buffer