Panagiotis Apostolou

Results 47 comments of Panagiotis Apostolou

Some thoughts on `vkFrameBoundaryANDROID` and virtual swapchains: `VulkanReplayConsumerBase::WriteScreenshots` takes the image information from the `SwapchainKHRInfo` which should take care of virtual swapchains. On the other hand `vkFrameBoundaryANDROID` takes as a...

> I'm not sure if the image on `vkFrameBoundaryANDROID` is from `vkGetSwapchainImagesKHR`. My option is quit simple that it could have a function, like `VulkanReplayConsumerBase::WriteScreenshots(ImageInfo* image_info)`. Yes, I agree with...

Hi @ishitatsuyuki Thank you for taking the time to suggest possible improvements. From what I understand you are referring to ~~`UFFDIO_WRITEPROTECT_MODE_WP`~~ `UFFD_FEATURE_PAGEFAULT_FLAG_WP` feature flag. We are aware of that but...

To be honest we haven't tried skipping shadow memory completely with userfaultfd (or maybe we did but that was some time ago and I can't remember exactly). We have tried...

I am afraid that this is not possible. If I understand the uffd documentation correctly, both `UFFDIO_REGISTER_MODE_MISSING` and `UFFDIO_REGISTER_MODE_WP` must be applied on private anonymous memory regions. The memory regions...

Interesting. This doesn't agree with the documentation found online. I only tried applying `UFFDIO_REGISTER_MODE_MISSING` that is currently used. I'll look into it a bit more

No it's still failing. I'm initializing with `UFFD_FEATURE_PAGEFAULT_FLAG_WP`, registering only with `UFFDIO_REGISTER_MODE_WP` and registrations fail with `EINVAL`. Tried this on both android and desktop. Maybe I am missing something? Have...

I recently installed Ubuntu 24.04 which has 6.8 kernel with support for the `UFFD_FEATURE_WP_ASYNC` feature and did a quick prototype to see if it would work with GFXR. Unfortunately it...

> I've also noticed similar problems with multiple drivers (intel, nvidia and mali, from memory). From talking to driver people, this is apparently not trivial to fix on linux due...

Hi Per I'll attempt to resurrect this PR in order to get it merged. We agree with you that including binaries in the repo is not a nice solution. However...