OS/2, eCS & ArcaOS - Technical > Programming

Qt6 Development

<< < (132/178) > >>

Andy Willis:

--- Quote from: Paul Smedley on March 22, 2023, 12:57:25 pm ---OK, we now have DLLs but it's not working yet:

--- Code: ---[109:1:0322/222035.678000:ERROR:shared_buffer_dispatcher.cc(298)] Failed to map shared memory region.
[109:8:0322/222037.341000:ERROR:shared_buffer_dispatcher.cc(298)] Failed to map shared memory region.
[109:1:0322/222037.875000:ERROR:system_network_context_manager.cpp(274)] Cannot use V8 Proxy resolver in single process mode.
[109:1:0322/222037.892000:ERROR:system_network_context_manager.cpp(274)] Cannot use V8 Proxy resolver in single process mode.

--- End code ---

I get the second pair with the current code.  I find that if I disable single process mode that I then have a better chance of loading pages but the system _will_ lock up much faster.

I'm not too worried about the second pair of messages, but the former sounds more serious. I'll have to go look at the code at https://github.com/psmedley/qt6-webengine-os2/blob/6.4.x/src/3rdparty/chromium/mojo/core/shared_buffer_dispatcher.cc#L298

Edit: Seems like due to my copy of posix code in https://github.com/psmedley/qt6-webengine-os2/commit/fe6e0638c422e146cb2379bf6a275d668b144edd#diff-0b0cccfdd90d1b8b3e0eced8a42e148326a514e6b73c3eea1435b488abe39e9b

--- End quote ---

Paul Smedley:
Yeah there is a reason dmitry make single process mode the default ;) I assume my current code you mean 6.3.2?

I may have found the problem causing the trap - the changes in https://github.com/psmedley/qt6-webengine-os2/commit/feb87f70dac13bb5b805fbd271e1ac98568ec062#diff-6d3616171ce7881cc8d3a757c2f9b476d645e4c61df77cdf47a5bf7a423bf56c - specifically the implementation of DecommitAndZeroSystemPagesInternal() is clearly wrong. I should have actually read the original code and the fix (I think) is pretty obvious and much simpler using   PA_CHECK(MyDosSetMem(reinterpret_cast<void*>(address), length, PAG_DECOMMIT));

Linking a new DLL now.

Paul Smedley:
This didn't fix it. I also remembered today about https://github.com/psmedley/qt6-webengine-os2/issues/9 - this may be the cause of issues with Qt >= 6.3 - will need to do some research on what the equivalent code on OS/2 might look like.

Paul Smedley:

--- Quote from: Paul Smedley on March 23, 2023, 09:42:05 am ---This didn't fix it. I also remembered today about https://github.com/psmedley/qt6-webengine-os2/issues/9 - this may be the cause of issues with Qt >= 6.3 - will need to do some research on what the equivalent code on OS/2 might look like.

--- End quote ---

Gotta love google - I searched for 'NtCurrentTeb())->StackBase)' (ie the windows code) and found https://github.com/psmedley/qt6-webengine-os2/blob/6.3.x/src/3rdparty/chromium/third_party/blink/renderer/platform/wtf/stack_util.cc#L163 - I'll steal the os2 code and add it to https://github.com/psmedley/qt6-webengine-os2/blob/6.3.x/src/3rdparty/chromium/base/allocator/partition_allocator/starscan/stack/stack.cc#L55

Paul Smedley:
OK, fixing that didn't change the Trap in QtWebEngine 6.3.x, rebuilding a new 6.4.x DLL to see if it makes any difference there.

Edit: Nope, it didn't, but at least it's one less thing to implement later.

Navigation

[0] Message Index

[#] Next page

[*] Previous page

Go to full version