libnxz / power-gzip

POWER NX zlib compliant library
23 stars 18 forks source link

Fixes for GCC 14 and zlib-ng compat usage #209

Closed tuliom closed 9 months ago

tuliom commented 10 months ago

These are trivial fixes.

I verified they compile, but I can't run the tests because I don't have access to systems with NX GZIP anymore. Could you run the tests please?

sachinmonga64 commented 10 months ago

@tuliom We need to run 3 tests ?

  1. test
  2. serial_test
  3. oct
tuliom commented 10 months ago

@tuliom We need to run 3 tests ?

1. test
2. serial_test
3. oct

@sachinmonga64 Ideally, yes. You can run them with make -j$(nproc) check, make -j$(nproc) selftest and make -j$(nproc) oct. The last one takes longer and is usually skipped. Considering these changes should not affect the output of the library, I believe it's OK to skip it.

sachinmonga64 commented 10 months ago

@tuliom Thanks for the information.

In addition to the changes committed for the id [f244b22] , I think this change may be required at one more location at line:144 in same file. I have done testing including this.

]$ less lib/nx_gzlib.c

...
135 gzFile __gzopen(const char* path, int fd, const char *mode, int force_nx)
136 {
137         gz_statep state;
138         gzFile file;
139         int err, strategy = Z_DEFAULT_STRATEGY, level = Z_DEFAULT_COMPRESSION;
140         char* digit;
141         void* stream;
142 
143         state = malloc(sizeof(struct gz_state));
144         if (state == NULL) {
145                 err=Z_MEM_ERROR;
146                 return Z_NULL;
147         }
...

Regarding testing, I have done the testing on RHEL8.9 and below is the result for tests directory.

...
]$ make check

SKIP: test_window_suspend
SKIP: test_shared_window_suspend
============================================================================
Testsuite summary for libnxz 0.64
============================================================================
# TOTAL: 3
# PASS:  1
# SKIP:  2
# XFAIL: 0
# FAIL:  0
# XPASS: 0
# ERROR: 0
============================================================================

Selftest is good.

For my information , can you please let me know in what situations, these 2 tests(test_window_suspend, test_shared_window_suspend) get SKIP ?

sachinmonga64 commented 9 months ago

@tuliom Can you please confirm the result shared in previous comment ?

tuliom commented 9 months ago

For my information , can you please let me know in what situations, these 2 tests(test_window_suspend, test_shared_window_suspend) get SKIP ?

@sachinmonga64 It has to be one of these 3 cases here: https://github.com/libnxz/power-gzip/blob/master/test/serial-tests/test_window_suspend.c#L65-L80

tuliom commented 9 months ago

In addition to the changes committed for the id [f244b22] , I think this change may be required at one more location at line:144 in same file. I have done testing including this.

Nice catch! I've just updated the PR including this change.

Thanks!

sachinmonga64 commented 9 months ago

Hi @tuliom

Libnxz testing is done from my side. Is the testing done on your side with the updated patch that you committed again ?

tuliom commented 9 months ago

Libnxz testing is done from my side. Is the testing done on your side with the updated patch that you committed again ?

@sachinmonga64 Yes.