control: tags -1 patch pending

Hello Alastair, please take this patch and commit on git. I couldn't do it 
because the latest changes were not pushed

diff -Nru ucx-1.18.1+ds/debian/changelog ucx-1.18.1+ds/debian/changelog
--- ucx-1.18.1+ds/debian/changelog      2025-03-04 13:40:04.000000000 +0100
+++ ucx-1.18.1+ds/debian/changelog      2025-03-11 14:17:30.000000000 +0100
@@ -1,3 +1,11 @@
+ucx (1.18.1+ds-2) unstable; urgency=medium
+
+  * Team Upload
+  [ Dandan Zhang <zhangdan...@loongson.cn> ]
+  * Update loongarch64 patch for new release (Closes: #1099636)
+
+ -- Gianfranco Costamagna <locutusofb...@debian.org>  Tue, 11 Mar 2025 
14:17:30 +0100
+
 ucx (1.18.1+ds-1) unstable; urgency=medium
* New upstream release
diff -Nru ucx-1.18.1+ds/debian/patches/ucx-add-loongarch64-support.patch 
ucx-1.18.1+ds/debian/patches/ucx-add-loongarch64-support.patch
--- ucx-1.18.1+ds/debian/patches/ucx-add-loongarch64-support.patch      
2025-03-04 13:40:04.000000000 +0100
+++ ucx-1.18.1+ds/debian/patches/ucx-add-loongarch64-support.patch      
2025-03-11 14:15:59.000000000 +0100
@@ -438,7 +438,7 @@
 ===================================================================
 --- /dev/null
 +++ ucx-1.18.0+ds~rc3/src/ucs/arch/loongarch64/cpu.h
-@@ -0,0 +1,113 @@
+@@ -0,0 +1,115 @@
 +/**
 +* Copyright (C) Tactical Computing Labs, LLC. 2022. ALL RIGHTS RESERVED.
 +* Copyright (C) Rivos Inc. 2023
@@ -537,7 +537,9 @@
 +}
 +#endif
 +
-+static inline void *ucs_memcpy_relaxed(void *dst, const void *src, size_t len)
++static inline void *ucs_memcpy_relaxed(void *dst, const void *src, size_t len,
++                                       ucs_arch_memcpy_hint_t hint,
++                                       size_t total_len)
 +{
 +    return memcpy(dst, src, len);
 +}

On Thu, 6 Mar 2025 11:45:47 +0800 zhangdandan <zhangdan...@loongson.cn> wrote:
Source: ucx
Version: 1.18.1+ds-1
Severity: normal
Tags: FTBFS patch
User: debian-loonga...@lists.debian.org
Usertags: loong64

Dear maintainers,

Compiling the ucx failed for loong64 in the Debian Package Auto-Building environment.
The error log is as follows,
```
......
sys/iovec.c: In function 'ucs_iov_copy':
sys/iovec.c:47:13: error: too many arguments to function 'ucs_memcpy_relaxed'    47 |             ucs_memcpy_relaxed(iov_buf, UCS_PTR_BYTE_OFFSET(buf, copied), len,
       |             ^~~~~~~~~~~~~~~~~~
.......
```
The full build log can be found at https://buildd.debian.org/status/fetch.php?pkg=ucx&arch=loong64&ver=1.18.1%2Bds-1&stamp=1741101442&raw=0.

Please consider the patch I attached.
According to the function calling in ./src/ucs/sys/iovec.c file, I need to update the parameters of function ucs_memcpy_relaxed();. Based on attached patch, I have built ucx (1.18.1+ds-1+loong64) successfully on locally.
```
......
     dpkg-deb --root-owner-group --build debian/.debhelper/libucx0/dbgsym-root .. dpkg-deb: building package 'libucx-dev' in '../libucx-dev_1.18.1+ds-1+loong64_loong64.deb'. dpkg-deb: building package 'ucx-utils-dbgsym' in '../ucx-utils-dbgsym_1.18.1+ds-1+loong64_loong64.deb'. dpkg-deb: building package 'libucx0' in '../libucx0_1.18.1+ds-1+loong64_loong64.deb'. dpkg-deb: building package 'ucx-utils' in '../ucx-utils_1.18.1+ds-1+loong64_loong64.deb'. dpkg-deb: building package 'libucx0-dbgsym' in '../libucx0-dbgsym_1.18.1+ds-1+loong64_loong64.deb'.
  dpkg-genbuildinfo -O../ucx_1.18.1+ds-1+loong64_loong64.buildinfo
  dpkg-genchanges -O../ucx_1.18.1+ds-1+loong64_loong64.changes
```

Your opinions are welcome.

Best regards,
Dandan Zhang


Reply via email to