kernel: Enable virtio-fs in kata kernel#484
kernel: Enable virtio-fs in kata kernel#484grahamwhaley merged 1 commit intokata-containers:masterfrom
Conversation
|
/test |
10b04e9 to
4f448f6
Compare
chavafg
left a comment
There was a problem hiding this comment.
I think you'll need to bump the kernel version at https://github.com/kata-containers/packaging/blob/master/kernel/kata_config_version
| @@ -1,7 +1,7 @@ | |||
| From 074a6a9d83a4e790f892ef0fc91cdabbfbf26202 Mon Sep 17 00:00:00 2001 | |||
There was a problem hiding this comment.
with this patch, now we have 3 patches that start with 0001, would you mind changing their name?
There was a problem hiding this comment.
I believe this change actually renames all the patches to have a proper order. Do you think i have missed it?
This patches adds virtio-fs capability to the kata kernel along with config changes to enable the same on kata by default. The system will only be exercised when `shared_fs` is set to `virtio-fs` in the kata configuration file. the default still remains to be 9p Fixes: kata-containers#387 Depends-on: github.com/kata-containers/runtime#1016 Signed-off-by: Ganesh Maharaj Mahalingam <ganesh.mahalingam@intel.com>
4f448f6 to
88d93ad
Compare
|
/retest |
Add configuration option to use virtio-fs. We will currently use nemu for testing the virtio-fs support. Depends-on: github.com/kata-containers/runtime#1016 Depends-on: github.com/kata-containers/packaging#484 Fixes: kata-containers#1536. Signed-off-by: Salvador Fuentes <salvador.fuentes@intel.com>
|
|
||
| # | ||
| # Compiler: gcc (Ubuntu 5.4.0-6ubuntu1~16.04.11) 5.4.0 20160609 | ||
| # Compiler: gcc (Ubuntu 7.4.0-1ubuntu1~16.04~ppa1) 7.4.0 |
There was a problem hiding this comment.
OOI, why are you using a ppa (non-standard) compiler version?
There was a problem hiding this comment.
My guess - only way to get a recent gcc on bunty...
|
We'd better get Mr Fragment^w^w @grahamwhaley to ack this one too... 😄 |
| # CONFIG_FUSE_FS is not set | ||
| CONFIG_FUSE_FS=y | ||
| # CONFIG_CUSE is not set | ||
| CONFIG_VIRTIO_FS=y |
There was a problem hiding this comment.
As these two CONFIG items are only present with the patches in this PR (that is, they are not upstream yet afaict), we can't apply them to the fragments PR until this PR has landed.
They look find in the context of this PR though.
Add configuration option to use virtio-fs. We will currently use nemu for testing the virtio-fs support. Depends-on: github.com/kata-containers/runtime#1016 Depends-on: github.com/kata-containers/packaging#484 Fixes: kata-containers#1536. Signed-off-by: Salvador Fuentes <salvador.fuentes@intel.com>
|
@grahamwhaley @ganeshmaharaj The config options look good. |
|
Great, let's merge this puppy... |
This patches adds virtio-fs capability to the kata kernel along with
config changes to enable the same on kata by default. The system will
only be exercised when
shared_fsis set tovirtio-fsin the kataconfiguration file. the default still remains to be 9p
Fixes: #387
Signed-off-by: Ganesh Maharaj Mahalingam ganesh.mahalingam@intel.com