-
Notifications
You must be signed in to change notification settings - Fork 79
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
Add Power ppc64le platform support #54
base: master
Are you sure you want to change the base?
Conversation
Handle the case of data equal to "".
Recompile gtest by obdevtools-gcc9 with -O2 -fPIC options
Add powerpc64 support
Add compile options for powerpc64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
编译用到的依赖文件,增加了 int *ret 参数
src/lib/oblog/ob_log.cpp
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
测试发现有data 为""时,虽不是NULL or nullptr ,但strlen是0,会导致logdata_vprintf函数中 vsnprintf调用出问题
deps/3rd/dep_create.sh
Outdated
case "$ID" in | ||
centos) | ||
version_ge "8.0" && compat_centos8 && return | ||
version_ge "7.0" && compat_centos7 && return |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ppc64现阶段就没打7的依赖,就先删掉吧,这里还是不支持比较好
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已删掉 7.0
rpm/obproxy-ce-VER.txt
Outdated
@@ -1 +1 @@ | |||
4.2.1.0 | |||
4.2.3.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里版本号为何更新了,是为了匹配observer的版本号吗
如果是的话,这不用动,让内部修改
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK
@@ -46,8 +46,8 @@ | |||
|
|||
extern "C" { | |||
struct _ParseNode; | |||
extern uint64_t parsenode_hash(const _ParseNode *node); | |||
extern bool parsenode_equal(const _ParseNode *node1, const _ParseNode *node2); | |||
extern uint64_t parsenode_hash(const _ParseNode *node, int *ret); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里我没有细看依赖的实现和obproxy的调用,插个眼,请proxy的同学帮忙看看
Summary
Add Power ppc64le platform support.
Solution Description
Patch to following files:
configure.ac
deps/3rd/dep_create.sh
deps/3rd/obproxy.el8.ppc64le.deps
deps/easy/src/include/easy_atomic.h
deps/easy/src/io/ev.c
deps/easy/src/util/easy_time.c
rpm/obproxy-ce-VER.txt
src/lib/atomic/ob_atomic.h
src/lib/cbtree/btree.h
src/lib/checksum/ob_crc64.cpp
src/lib/hash/ob_hashutils.h
src/lib/list/ob_atomic_list.h
src/lib/lock/ob_latch.cpp
src/lib/oblog/ob_log.cpp
src/lib/stat/ob_di_cache.cpp
src/obproxy/obutils/ob_config_server_processor.cpp
src/obproxy/opsql/parser/ob_proxy_parse_result.h
src/obproxy/utils/ob_proxy_table_define.h
unittest/obproxy/Makefile.am
unittest/obproxy/test_atomic_store.cpp