Mo Chen

Results 11 comments of Mo Chen

My text editor trimmed trailing whitespace from the gold file. It looks like the other gold file to match ATS log output also failed. I'm investigating.

Autest failure should be fixed with latest push. Thanks @bneradt

> Couple of things to do before we can merge: > > 1. This needs to be discussed on the mailing list, as is the normal process for adding new...

I'm still reading through all the changes. One thing I've noticed is that there doesn't seem to be bounds check for the memory mapped I/O. This may introduce crashes or...

There is an documented way in LeakSanitizer to ignore leaks: https://github.com/google/sanitizers/wiki/AddressSanitizerLeakSanitizer#suppressions. We should use that method if we want to ignore leaks. If we want to change the code to...

This has been in draft for a while. Is it ready for review?

Is it possible to go all the way, remove `thr_info`, and use `this` instead?

> Is the problem you are seeing different from this one? https://the-asf.slack.com/archives/CHQ1FJ9EG/p1706825958698989 I think they might be the same issue.

I'm getting some failures in on Ubuntu 23.04 running in UTM. [results2.txt](https://github.com/apache/trafficserver/files/14103788/results2.txt) ATS is built as follows: ``` CC=clang-17 CXX=clang++-17 cmake -GNinja -DCMAKE_INSTALL_PREFIX:PATH=/opt/ats2 \ -DCMAKE_BUILD_TYPE=Debug \ -DENABLE_CCACHE=ON \ -DENABLE_ASAN=OFF \...