test: disable coverage for cpp_headersThese object files have no functions in them, causing newerlcov to complain. So just disable coverage for this directory, there'snothing to cover here anyway
test: disable coverage for cpp_headersThese object files have no functions in them, causing newerlcov to complain. So just disable coverage for this directory, there'snothing to cover here anyways.Signed-off-by: Jim Harris <jim.harris@samsung.com>Change-Id: Ib6d687fdedc140b67189ab34ae795c2fd03abfb4Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24835Reviewed-by: Michal Berger <michal.berger@intel.com>Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
show more ...
update Intel copyright noticesper Intel policy to include file commit date using git cmdbelow. The policy does not apply to non-Intel (C) notices.git log --follow -C90% --format=%ad --date defa
update Intel copyright noticesper Intel policy to include file commit date using git cmdbelow. The policy does not apply to non-Intel (C) notices.git log --follow -C90% --format=%ad --date default <file> | tail -1and then pull just the 4 digit year from the result.Intel copyrights were not added to files where Intel either hadno contribution ot the contribution lacked substance (ie licenseheader updates, formatting changes, etc). Contribution date used"--follow -C95%" to get the most accurate date.Note that several files in this patch didn't end the license/(c)block with a blank comment line so these were added as the vastmajority of files do have this last blank line. Simply there forconsistency.Signed-off-by: paul luse <paul.e.luse@intel.com>Change-Id: Id5b7ce4f658fe87132f14139ead58d6e285c04d4Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15192Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>Reviewed-by: Jim Harris <james.r.harris@intel.com>Reviewed-by: Ben Walker <benjamin.walker@intel.com>Community-CI: Mellanox Build Bot
lib/vfu_tgt: add library for PCI device emulationPreviously SPDK use libvfio-user library to provide emulated NVMedevices to VM, but it's limited to NVMe device type only. Here weadd SPDK vfu_ta
lib/vfu_tgt: add library for PCI device emulationPreviously SPDK use libvfio-user library to provide emulated NVMedevices to VM, but it's limited to NVMe device type only. Here weadd SPDK vfu_target library abstraction based on libvfio-user whichsupports more PCI device types.We will add virtio-blk and virtio-scsi devices emulation based onvfu_tgt library in following patches, actually this library cansupport NVMe emulation too, due to the fact that the NVMe emulationis already exist, so we will keep the NVMe emulation which based onlibvfio-user directly as it is.Change-Id: Ib0ead6c6118fa62308355fe432003dd928a2fae9Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/12597Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>Community-CI: Mellanox Build BotReviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>Reviewed-by: Ben Walker <benjamin.walker@intel.com>
Replace most BSD 3-clause license text with SPDX identifier.Many open source projects have moved to using SPDX identifiersto specify license information, reducing the amount ofboilerplate code in
Replace most BSD 3-clause license text with SPDX identifier.Many open source projects have moved to using SPDX identifiersto specify license information, reducing the amount ofboilerplate code in every source file. This patch replacesthe bulk of SPDK .c, .cpp and Makefiles with the BSD-3-Clauseidentifier.Almost all of these files share the exact same license text,and this patch only modifies the files that contain themost common license text. There can be slight variationsbecause the third clause contains company names - most say"Intel Corporation", but there are instances for Nvidia,Samsung, Eideticom and even "the copyright holder".Used a bash script to automate replacement of the license textwith SPDX identifier which is checked into scripts/spdx.sh.Signed-off-by: Jim Harris <james.r.harris@intel.com>Change-Id: Iaa88ab5e92ea471691dc298cfe41ebfb5d169780Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/12904Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>Community-CI: Mellanox Build BotTested-by: SPDK CI Jenkins <sys_sgci@intel.com>Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>Reviewed-by: Dong Yi <dongx.yi@intel.com>Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>Reviewed-by: Paul Luse <paul.e.luse@intel.com>Reviewed-by: <qun.wan@intel.com>
queue_extras: use SPDK_CONTAINEROFA couple of the functions inside of the queue_extras file were relyingon an undefined __containerof function. Replace that withSPDK_CONTAINEROF. Then add an extr
queue_extras: use SPDK_CONTAINEROFA couple of the functions inside of the queue_extras file were relyingon an undefined __containerof function. Replace that withSPDK_CONTAINEROF. Then add an extra check so that when someone includesthis file on FreeBSD, it doesn't throw compiler warnings for theredefinitions (see our cpp headers test for example).The argument order for both functions is the same, namely pointer, type,member.Change-Id: I1bd0497ee14df8b37f4de1046e271e5fe144ca82Signed-off-by: Seth Howell <seth.howell@intel.com>Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/470191Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>Reviewed-by: Jim Harris <james.r.harris@intel.com>Reviewed-by: Ben Walker <benjamin.walker@intel.com>Community-CI: Broadcom SPDK FC-NVMe CI <spdk-ci.pdl@broadcom.com>
Makefile: Add possibility to uninstall spdk.Add uninstall target to makefiles to be able to performreverse of install target.Fixes #464This patch adds 'uninstall' target to makefile.'make unin
Makefile: Add possibility to uninstall spdk.Add uninstall target to makefiles to be able to performreverse of install target.Fixes #464This patch adds 'uninstall' target to makefile.'make uninstall' will remove spdk_tgt app, headers, librariesand shared libraries from system directories defined by $DESTDIR.Additionaly, if there will be any empty directories left afterthis operation, they will be removed as well.Signed-off-by: Piotr Pelplinski <piotr.pelplinski@intel.com>Change-Id: I7b07fb4b81081d3914ff09165991fbe3a26b9067Signed-off-by: Pawel Kaminski <pawelx.kaminski@intel.com>Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/431471Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>Reviewed-by: Ben Walker <benjamin.walker@intel.com>Reviewed-by: Jim Harris <james.r.harris@intel.com>
build: don't include spdk.app.mk in test/cpp_headersThis Makefile doesn't depend on anything in spdk.app.mk,and provides its own rules that are different than anyof the other applications (includ
build: don't include spdk.app.mk in test/cpp_headersThis Makefile doesn't depend on anything in spdk.app.mk,and provides its own rules that are different than anyof the other applications (including unit tests).The exception here is the simple "install : app" rule.Just add that rule to the cpp_headers Makefile insteadof including spdk.app.mk just for that rule.Signed-off-by: Jim Harris <james.r.harris@intel.com>Change-Id: I684dfba9108dfa7f06d7eb4acecd6806e5797312Reviewed-on: https://review.gerrithub.io/434282Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com>Reviewed-by: Darek Stojaczyk <dariusz.stojaczyk@intel.com>Reviewed-by: Ben Walker <benjamin.walker@intel.com>
build: add @: to remaining 'all' targetsIt is not ideal to have to add this in a bunch ofdifferent Makefiles, but further consolidation ofMakefiles is going to be a more arduous process.With th
build: add @: to remaining 'all' targetsIt is not ideal to have to add this in a bunch ofdifferent Makefiles, but further consolidation ofMakefiles is going to be a more arduous process.With this change, rebuilding SPDK after no changeswill result in no output - all of the "Nothing tobe done for 'all'" messages have been removed. Notethat DPDK build output still remains - this can besuppressed by either using an out-of-tree DPDK, orusing SKIP_DPDK_BUILD=1 when using the in-tree DPDKsubmodule.Signed-off-by: Jim Harris <james.r.harris@intel.com>Change-Id: Ib1d45432f2baaa1d9c1b9509906ee1e77966d3caReviewed-on: https://review.gerrithub.io/399918Reviewed-by: Daniel Verkamp <daniel.verkamp@intel.com>Tested-by: SPDK Automated Test System <sys_sgsw@intel.com>
build: include spdk.app.mk in all app MakefilesThis will be used to include common app-specific rules in upcomingpatches (it currently makes no difference).Change-Id: Ia828ba01c94fc9f70e15db3fc1
build: include spdk.app.mk in all app MakefilesThis will be used to include common app-specific rules in upcomingpatches (it currently makes no difference).Change-Id: Ia828ba01c94fc9f70e15db3fc1d9cafda387e912Signed-off-by: Daniel Verkamp <daniel.verkamp@intel.com>Reviewed-on: https://review.gerrithub.io/387629Tested-by: SPDK Automated Test System <sys_sgsw@intel.com>Reviewed-by: Jim Harris <james.r.harris@intel.com>Reviewed-by: Ben Walker <benjamin.walker@intel.com>
test/cpp_headers: autogenerate header test filesRather than requiring changes to a static list of header files, use theGNU Make wildcard function to generate a .cpp file per header.This also tes
test/cpp_headers: autogenerate header test filesRather than requiring changes to a static list of header files, use theGNU Make wildcard function to generate a .cpp file per header.This also tests whether each header includes all of the system headersfor the types it uses.Change-Id: I05b82510b194533672568019e138d7d1aad2e86bSigned-off-by: Daniel Verkamp <daniel.verkamp@intel.com>
test: validate that all headers compile as C++Change-Id: I743a2694a58725f51a0f3d6e1fe130df3d5cd3c5Signed-off-by: Daniel Verkamp <daniel.verkamp@intel.com>