From a3b8e0aa0a52cfeebb9b7f31fa21c4480b61e606 Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Thu, 23 Feb 2017 02:36:22 +0800 Subject: [PATCH] ceph-dencoder: s/WITH_LIBAIO/HAVE_LIBAIO/ * s/WITH_LIBAIO/HAVE_LIBAIO/: as HAVE_LIBAIO is used to detect if libaio is installed and is exposed in the acconfig.h. * do not test bluestore_blob_t with ceph-dencoder, as it repurposes the "feature" parameter for struct_v. Signed-off-by: Willem Jan Withagen Signed-off-by: Kefu Chai --- src/test/encoding/types.h | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/test/encoding/types.h b/src/test/encoding/types.h index 3616f88233e3..a1891e3afdb8 100644 --- a/src/test/encoding/types.h +++ b/src/test/encoding/types.h @@ -116,14 +116,18 @@ TYPE(ObjectStore::Transaction) #include "os/filestore/SequencerPosition.h" TYPE(SequencerPosition) -#ifdef WITH_LIBAIO +#ifdef HAVE_LIBAIO #include "os/bluestore/bluestore_types.h" TYPE(bluestore_cnode_t) TYPE(bluestore_compression_header_t) TYPE(bluestore_extent_ref_map_t) TYPE(bluestore_pextent_t) -TYPE(bluestore_blob_t) -TYPE(bluestore_lextent_t) +// TODO: bluestore_blob_t repurposes the "feature" param of encode() for its +// struct_v. at a higher level, BlueStore::ExtendMap encodes the extends using +// a different interface than the normal ones. see +// BlueStore::ExtentMap::encode_some(). maybe we can test it using another +// approach. +// TYPE_FEATUREFUL(bluestore_blob_t) TYPE(bluestore_onode_t) TYPE(bluestore_wal_op_t) TYPE(bluestore_wal_transaction_t) -- 2.47.3