Skip to content

Feature Request: Expose llama.cpp --no-mmap option #37

Closed

Description

@TrajansRow

There was a performance regression in earlier versions of llama.cpp that I may be hitting with long running interactions. This was recently fixed with the addition of a --no-mmap option which forces the entire model to be loaded into ram, and I would like to also use it with koboldcpp.

ggml-org#801

Activity

  1. LostRuins commented on Apr 10, 2023

    @LostRuins
    Owner

    This should be fixed in the latest version, currently v1.3 at time of writing.
    In fact, I have made mmap disabled by default. To enable it, use the flag --usemmap
    https://github.com/LostRuins/koboldcpp/releases/latest

    Edit: for version 1.4 it is now mmap by default, you can toggle it off with --nommap instead.

  2. TrajansRow commented on Apr 10, 2023

    @TrajansRow
    Author

    Thanks for the update! I'm seeing much better memory utilization now, although not quite the same sort of performance improvement I saw in llama.cpp (maybe a generation speedup on the order of 10ms/token, running 30B/4-bit llama on an M1 Max).

    This is a great option to have in koboldcpp, but I don't think it should be enabled by default. For the majority of users, I don't expect the memory tradeoffs to provide meaningful benefit.

  3. LostRuins commented on Apr 11, 2023

    @LostRuins
    Owner

    Yeah it was quite a divided response, some people hate it others love it. In the end it is a toggle so everyone can just pick whichever option they prefer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions