casaroli commented on PR #19536: URL: https://github.com/apache/nuttx/pull/19536#issuecomment-5097687159
> > Great, I tested xipfs locally and it seems to work the same way with PIC ELF. We have `mps3-an547:picostest `in our repository; can we use that as a test case for xipfs? Theoretically, compiling ostest with PIC ELF and then running it in xipfs without using NXflat would also work. @casaroli > > yes, @casaroli could you create a new config which enable xipfs and elf pic which should fix all issues you hit on nxflat? Thank you for your comments. We could add the ELF examples now, but honestly, xipfs does not bring any benefit to ELF PIC as the modules will need to be copied to RAM anyway. This would only be noise. Spoiler alert: My next PR in this series will propose adding real FDPIC ELF (idea stolen from uclinux) that will provide real XIP ELF binaries. But since I am doing this in stages, the only thing that can actualy benefit from xipfs is NXFLAT. So once you merge it, the bomb will drop 💣 I hope you will like it. 😅 My suggestin is to try to get this merged as is -- after addressing the style comments -- it brings real benefits to NXFLAT and almost no difference to ELF PIC. And wait for the FDPIC ELF real XIP. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
