New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
virtiofs: Disable DAX #876
Conversation
virtiofs DAX support is not stable today, there are a few corner cases to make it default. Fixes: kata-containers#862 Fixes: kata-containers#875 Signed-off-by: Jose Carlos Venegas Munoz <jose.carlos.venegas.munoz@intel.com>
@@ -173,7 +173,8 @@ DEFSHAREDFS := virtio-9p | |||
DEFSHAREDFS_QEMU_VIRTIOFS := virtio-fs | |||
DEFVIRTIOFSDAEMON := $(VIRTIOFSDBINDIR)/virtiofsd | |||
# Default DAX mapping cache size in MiB | |||
DEFVIRTIOFSCACHESIZE := 1024 | |||
#if value is 0, DAX is not enabled | |||
DEFVIRTIOFSCACHESIZE := 0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense to me.
Though, for 2.0, should we change the default to virtio-fs w/ out DAX for the 'qemu' configuration? I would think 2.0 is the time to switch from 9p -> virtiofs, unless there are stability concerns.
If we do the above, then I'd consider being able to enable DAX still for the 'qemu-virtiofs' configuration of Kata, but disabled for 'qemu' and 'clh' configurations...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1. We should open another issue to discuss it though. The PR meant to forward port a 1.x commit.
/test-ubuntu |
/test-clh |
virtiofs DAX support is not stable today, there are
a few corner cases to make it default.
Fixes: #862
Fixes: #875
Signed-off-by: Jose Carlos Venegas Munoz jose.carlos.venegas.munoz@intel.com