Skip to content
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

Jump to bottom

fio_plugin.c:50:25: fatal error: config-host.h: No such file or directory #29

Closed
cooljiansir opened this issue Jul 25, 2016 · 2 comments
Closed

fio_plugin.c:50:25: fatal error: config-host.h: No such file or directory #29

cooljiansir opened this issue Jul 25, 2016 · 2 comments

Comments

@cooljiansir
Copy link

@cooljiansir cooljiansir commented Jul 25, 2016

I followed the introduction spdk/examples/nvme/fio_plugin/README, but meet the following ERROR:

fio_plugin.c:50:25: fatal error: config-host.h: No such file or directory
 #include "config-host.h"
                         ^
compilation terminated.
make[3]: *** [fio_plugin.o] Error 1
make[2]: *** [fio_plugin] Error 2
make[1]: *** [nvme] Error 2
make: *** [examples] Error 2
@ghost
Copy link

@ghost ghost commented Jul 25, 2016

Make sure you have set SPDK's CONFIG_FIO_PLUGIN to the correct location, and verify that the FIO source has been configured and built (config-host.h is generated by the FIO configure script).

@cooljiansir
Copy link
Author

@cooljiansir cooljiansir commented Jul 26, 2016

Thanks! It works after running configure script.

@benlwalker benlwalker closed this Jul 29, 2016
ghost pushed a commit that referenced this issue Oct 5, 2017
@peluse @danielverkamp
blob: fix issue with blobid handling
721695e 
Fixes github issue #29.

Because of how we handle the blobid and pagenum in blobstore,
it was possible to have blobstore inadvertently open the wrong
blob if open is provided a blobid where the lower 32 bits match
an existing blob but the upper 32 are clear.

Patch does the following:
- removes assert() that caught this on MD load and replace with
an error given that this condition can be induced via the API
- cleanup of pagenum and blobid conversion/handling to make it
clearer how they're related and converted
- new UTs that would have failed w/o the new check in place

Change-Id: I2b49b237922b3b8cfc4df296f5bc20195e41dc41
Signed-off-by: Paul Luse <paul.e.luse@intel.com>
Reviewed-on: https://review.gerrithub.io/380872
Tested-by: SPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: Daniel Verkamp <daniel.verkamp@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
@pcybulski pcybulski mentioned this issue Feb 5, 2019
Closed
@x2c3z4 x2c3z4 mentioned this issue Jun 20, 2019
Closed
@yidong0635 yidong0635 mentioned this issue Aug 23, 2019
Closed
@optimistyzy optimistyzy mentioned this issue Mar 25, 2020
Closed
tmakatos added a commit to tmakatos/spdk that referenced this issue Jun 4, 2020
@tmakatos
add test to repro issue spdk#29
dcb0422 
Signed-off-by: Thanos Makatos <thanos.makatos@nutanix.com>
tmakatos added a commit to tmakatos/spdk that referenced this issue Jun 4, 2020
@tmakatos
mmap fd instead of grabbing pages when receiving DMA region registration
ea1c32f 
This enables huge pages to work. Also, it avoids requiring having to pin memory.

The solution is not 100% correct because since we no longer call vfio_pin_pages,
VFIO doesn't send the DMA registration event and we end up getting duplicate
regions which we reject (this is because of our internal DMA bookkeeping
implementation). We need to work with the community to find a solution.

fixes spdk#28
fixes spdk#29
fixes spdk#38

Signed-off-by: Thanos Makatos <thanos.makatos@nutanix.com>
@toan2013 toan2013 mentioned this issue Jan 11, 2021
Open
@optimistyzy optimistyzy mentioned this issue Apr 27, 2021
Closed
@suruifeng suruifeng mentioned this issue Aug 19, 2021
Closed
@changpe1 changpe1 mentioned this issue Mar 31, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cooljiansir @benlwalker