packages/kernel
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

check-kabi: fix kabi check failed when no namespace

Browse Source 
Not every symbol has the namespace attribute. So use
starred expression mark namespace as optional.

Signed-off-by: Xie XiuQi <xiexiuqi@huawei.com>
This commit is contained in:
Xie XiuQi 2024-12-16 21:59:08 +08:00
parent 62a41c9506
commit a5ee4685ed
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
check-kabi
View File

@ -41,7 +41,7 @@ def load_symvers(symvers,filename):
break break
if in_line == "\n": if in_line == "\n":
continue continue
checksum,symbol,directory,type,namespace = in_line.split('\t') checksum,symbol,directory,type,*namespace = in_line.split('\t')
symvers[symbol] = in_line[0:-1] symvers[symbol] = in_line[0:-1]
@ -56,7 +56,7 @@ def load_kabi(kabi,filename):
break break
if in_line == "\n": if in_line == "\n":
continue continue
checksum,symbol,directory,type,namespace = in_line.split('\t') checksum,symbol,directory,type,*namespace = in_line.split('\t')
kabi[symbol] = in_line[0:-1] kabi[symbol] = in_line[0:-1]
@ -71,9 +71,9 @@ def check_kabi(symvers,kabi):
losted_symbols=[] losted_symbols=[]
for symbol in kabi: for symbol in kabi:
abi_hash,abi_sym,abi_dir,abi_type,abi_namespace = kabi[symbol].split('\t') abi_hash,abi_sym,abi_dir,abi_type,*abi_namespace = kabi[symbol].split('\t')
if symbol in symvers: if symbol in symvers:
sym_hash,sym_sym,sym_dir,sym_type,sym_namespace = symvers[symbol].split('\t') sym_hash,sym_sym,sym_dir,sym_type,*sym_namespace = symvers[symbol].split('\t')
if abi_hash != sym_hash: if abi_hash != sym_hash:
fail=1 fail=1
changed_symbols.append(symbol) changed_symbols.append(symbol)