so_ver: increase all major versions after SPDK 24.01To allow SO_MINOR updates on LTS for the whole year it is supported,the major version for all components needs to be increased.This is to preve
so_ver: increase all major versions after SPDK 24.01To allow SO_MINOR updates on LTS for the whole year it is supported,the major version for all components needs to be increased.This is to prevent scenario where two versions exists with matchingversions, but conflicting ABI.Ex. Next SPDK release adds an API call increasing the minor version,then LTS needs just a subset of those additions.Increasing major so version after LTS, allows the future releasesto update versions as needed. Yet allowing LTS to increase minorversion separately.This patch:- increases SO_VER by 1 for all components- resets SO_MINOR to 0 for all componentsDue to patch below being introduced after v24.01 code freeze,bdev lib version should not be increased:61623c5ca9 bdev: add spdk_bdev_io padding in place of io_submit_chShort reference to how the versions were changed:MAX=$(git grep "SO_VER := " | cut -d" " -f 3 | sort -ubnr | head -1)for((i=$MAX;i>0;i-=1)); do find . -name "Makefile" -exec \ sed -i -e "s/SO_VER := $i\$/SO_VER := $(($i+1))/g" {} +; donefind . -name "Makefile" -exec \ sed -i -e "s/SO_MINOR := .*/SO_MINOR := 0/g" {} +Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>Change-Id: I0166d6be6c17bbbf985d48b5dfcb36f1d4af1b48Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21582Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
so_ver: increase all major versionsTo allow SO_MINOR updates on LTS for the whole year it is supported,the major version for all components needs to be increased.This is to prevent scenario where
so_ver: increase all major versionsTo allow SO_MINOR updates on LTS for the whole year it is supported,the major version for all components needs to be increased.This is to prevent scenario where two versions exists with matchingversions, but conflicting ABI.Ex. Next SPDK release adds an API call increasing the minor version,then LTS needs just a subset of those additions.Increasing major so version after LTS, allows the future releasesto update versions as needed. Yet allowing LTS to increase minorversion separately.Disabled test for increasing SO version without ABI change, asthat is goal of this patch. This check shall be removed with SPDK 23.05release.Looks like this was left over from prior LTS, to avoid thatmake sure it is only skipped when running against v23.01.x as latestrelease.This patch:- increases SO_VER by 1 for all components- resets SO_MINOR to 0 for all components- removes suppressions for ABI testsShort reference to how the versions were changed:MAX=$(git grep "SO_VER := " | cut -d" " -f 3 | sort -ubnr | head -1)for((i=$MAX;i>0;i-=1)); do find . -name "Makefile" -exec \ sed -i -e "s/SO_VER := $i\$/SO_VER := $(($i+1))/g" {} +; donefind . -name "Makefile" -exec \ sed -i -e "s/SO_MINOR := .*/SO_MINOR := 0/g" {} +Change-Id: I3e5681802c0a5ac6d7d652a18896997cd07cc8bfSigned-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16419Reviewed-by: Ben Walker <benjamin.walker@intel.com>Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
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
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>
so_ver: increase all major versionsTo allow SO_MINOR updates on LTS for the whole year it is supported,the major version for all components needs to be increased.This is to prevent scenario where two versions exists with matchingversions, but conflicting ABI.Ex. Next SPDK release adds an API call increasing the minor version,then LTS needs just a subset of those additions.Increasing major so version after LTS, allows the future releasesto update versions as needed. Yet allowing LTS to increase minorversion separately.Disabled test for increasing SO version without ABI change, asthat is goal of this patch. This check shall be removed with SPDK 22.05release.This patch:- increases SO_VER by 1 for all components- resets SO_MINOR to 0 for all components- removes suppressions for ABI testsSigned-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>Change-Id: Id1a5358882dc496faa5b0b5c9a63b326c378c551Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/11361Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>Reviewed-by: Jim Harris <james.r.harris@intel.com>Reviewed-by: Ben Walker <benjamin.walker@intel.com>
event: Shift subsystem initialization code to a separate libraryThis is useful for applications even if they elect not to use the SPDKevent framework.This doesn't shift everything in one go - ju
event: Shift subsystem initialization code to a separate libraryThis is useful for applications even if they elect not to use the SPDKevent framework.This doesn't shift everything in one go - just the subsysteminitialization logic. Configuration file loading also needs to movein a separate patch later.Change-Id: Id419df1045442d416650ed90e5ee78adfdd623d7Signed-off-by: Ben Walker <benjamin.walker@intel.com>Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/6641Community-CI: Broadcom CICommunity-CI: Mellanox Build BotTested-by: SPDK CI Jenkins <sys_sgci@intel.com>Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com>Reviewed-by: Jim Harris <james.r.harris@intel.com>Reviewed-by: Paul Luse <paul.e.luse@intel.com>Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
so_ver: increase all major versionsTo allow SO_MINOR updates on LTS for the whole year it is supported,the major version for all components needs to be increased.This is to prevent scenario where two versions exists with matchingversions, but conflicting ABI.Ex. Next SPDK release adds an API call increasing the minor version,then LTS needs just a subset of those additions.Increasing major so version after LTS, allows the quarterly releasesto update versions as needed. Yet allowing LTS to increase minorversion separately.Disabled test for increasing SO version without ABI change, asthat is goal of this patch. This check shall be removed with SPDK 21.04release.This patch:- increases SO_VER by 1 for all components- resets SO_MINOR to 0 for all components- removes suppressions for ABI testsSigned-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>Change-Id: I44d01154430a074103bd21c7084f44932e81fe72Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/6167Community-CI: Broadcom CICommunity-CI: Mellanox Build BotTested-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>
nbd: Use async manner to stop nbd device.Purpose: In order to free all the allocated resources.Our current code uses sync behaviour, and it will not wait for allthe resources are freed if there
nbd: Use async manner to stop nbd device.Purpose: In order to free all the allocated resources.Our current code uses sync behaviour, and it will not wait for allthe resources are freed if there is active I/Os, and thus we willnot free some resources, e.g., some fds will not be closed.Signed-off-by: Ziye Yang <ziye.yang@intel.com>Change-Id: Iaf9a606da2049ffd0096860c46d89d094038a5ffReviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/5601Community-CI: Broadcom CITested-by: SPDK CI Jenkins <sys_sgci@intel.com>Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>Reviewed-by: Jim Harris <james.r.harris@intel.com>Reviewed-by: Sun Zhenyuan <sunzhenyuan@baidu.com>Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>Reviewed-by: <dongx.yi@intel.com>
event: remove subsystem legacy config printThis patch removes callback function for subsystems topresent their options for legacy config.Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com
event: remove subsystem legacy config printThis patch removes callback function for subsystems topresent their options for legacy config.Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>Change-Id: I63076fc03eff45da5d57fab03501602922a20e3eReviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/4749Reviewed-by: Jim Harris <james.r.harris@intel.com>Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
mk/lib: add a check that major and minor version is set for libs.Also, while we are here, consolidate setting SO_SUFFIX to one spot.Previously, it was possible for a library to slip throughwitho
mk/lib: add a check that major and minor version is set for libs.Also, while we are here, consolidate setting SO_SUFFIX to one spot.Previously, it was possible for a library to slip throughwithout an SO version.Signed-off-by: Seth Howell <seth.howell@intel.com>Change-Id: I4db5fa5839502d266c6259892e5719b05134518cReviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/2361Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>Community-CI: Mellanox Build BotCommunity-CI: Broadcom CIReviewed-by: Paul Luse <paul.e.luse@intel.com>Reviewed-by: Ben Walker <benjamin.walker@intel.com>Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com>Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
module/event_nbd: add a map file and remove spdk prefix.Signed-off-by: Seth Howell <seth.howell@intel.com>Change-Id: Id2920389b7cf762cf80006da5dbe582dfdc81438Reviewed-on: https://review.spdk.io/g
module/event_nbd: add a map file and remove spdk prefix.Signed-off-by: Seth Howell <seth.howell@intel.com>Change-Id: Id2920389b7cf762cf80006da5dbe582dfdc81438Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/2350Community-CI: Mellanox Build BotCommunity-CI: Broadcom CITested-by: SPDK CI Jenkins <sys_sgci@intel.com>Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com>Reviewed-by: Jim Harris <james.r.harris@intel.com>
make: rev SO versions individually for libraries.This will allow us to keep track of compatibility issues on aper-library basis.Change-Id: Ib0c796adb1efe1570212a503ed660bef6f142b6eSigned-off-by
make: rev SO versions individually for libraries.This will allow us to keep track of compatibility issues on aper-library basis.Change-Id: Ib0c796adb1efe1570212a503ed660bef6f142b6eSigned-off-by: Seth Howell <seth.howell@intel.com>Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/1067Tested-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>
lib/event: move rpc and subsystems dirs to moduleThese directories fit in with the module concept we are forming insideof SPDK. Essentially modules are derivative or specialized libraries thatrel
lib/event: move rpc and subsystems dirs to moduleThese directories fit in with the module concept we are forming insideof SPDK. Essentially modules are derivative or specialized libraries thatrely on a general or core library.Change-Id: Ib40f05422f144ff8fd579f47a3867ef4412b3372Signed-off-by: Seth Howell <seth.howell@intel.com>Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/465455Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>Reviewed-by: Jim Harris <james.r.harris@intel.com>Reviewed-by: Ben Walker <benjamin.walker@intel.com>Reviewed-by: Broadcom SPDK FC-NVMe CI <spdk-ci.pdl@broadcom.com>Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>