Coder Social home page Coder Social logo

Comments (7)

gl-yziquel avatar gl-yziquel commented on June 20, 2024 2

The test is conclusive: the fix you proposed works on such a virtualised setting. You should feel free to commit it to the main branch.

All clear.

from gitoxide.

gl-yziquel avatar gl-yziquel commented on June 20, 2024 1

Just wait for the test for some 24 to 48 hours.

from gitoxide.

Byron avatar Byron commented on June 20, 2024

Thanks a lot for letting me know!

gitoxide crates use the mmap2 crate which hides these details entirely. That's great as it means there is no deeper reason behind the MAP_SHARED mode, and it shouldn't be a problem to switch to MAP_PRIVATE.

Since I probably won't be able to reproduce the issue I wonder if you'd be willing to validate a version from a PR that I could create rather swiftly?

Thanks for your help.

from gitoxide.

gl-yziquel avatar gl-yziquel commented on June 20, 2024

Thanks a lot for letting me know!

gitoxide crates use the mmap2 crate which hides these details entirely. That's great as it means there is no deeper reason behind the MAP_SHARED mode, and it shouldn't be a problem to switch to MAP_PRIVATE.

Since I probably won't be able to reproduce the issue I wonder if you'd be willing to validate a version from a PR that I could create rather swiftly?

Thanks for your help.

I'll have to build the PR myself on that guest. Which will be a problem since I cannot run "cargo build" (for likely the same reason, which indeed led me down here first with gix). So it will take some amount of tweaking to get it right. Moreover, I have to go get some food. So I won't reply in a quick feedback loop. Do not expect a reply before sunday or monday...

from gitoxide.

Byron avatar Byron commented on June 20, 2024

Thanks for the quick response and for your help.

I am sorry to hear cargo also bails out in that setup. In case, for some reason, it's not due to memmap but to C toolchains or such, you should be able to build gix with cargo build --bin gix --no-default-features --features max-pure, which requires only a Rust compiler and a linker and Cargo.

from gitoxide.

gl-yziquel avatar gl-yziquel commented on June 20, 2024

Just looked at the proposed fix. Yes, the memmap2 crate has a way to enforce MAP_PRIVATE. I hope this will fix it (as it will allow me to narrow down the behaviour of virtiofs). Going to get some food. Then I'll try it out.

Thank you for being so unbelievably responsive.

from gitoxide.

Byron avatar Byron commented on June 20, 2024

Bugs get the priority lane :).

from gitoxide.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.