Re: [edk2-devel] GSoC 2021 (MinPlatform, Ext2, ACPICA, etc)

["Andrew Fish" <afish@apple.com>]

[-- Attachment #1: Type: text/plain, Size: 1925 bytes --]



> On Apr 4, 2021, at 3:50 AM, Pedro Falcato <pedro.falcato@gmail.com> wrote:
> 
> Hi,
> 
> Sounds great! You may be right about that, although I believe I read somewhere that you need to set a milestone for each evaluation? If so, I'm out of ideas as to what can be tangible enough to be a milestone; if not, specifying a read-only driver as project's objective and having write support as a stretch goal is certainly the way to go.
> 

Pedro,

Just an idea but but you could have a milestone to feature complete  of the driver, and have a milestone to develop a test strategy, and test suite to validate the driver. Documentation could also be a milestone. For example I’ve worked on some proprietary EFI File System drivers (HFS+, apfs) and we chose not to implement certain features like compressed files. So you would want to have good documentation about that and tests to make sure the driver gracefully fails in these cases. The test cases don’t only need to be tests as you could create a file system disk image that you could mount in OVMF that contains a lot of file system end case constructs (max size file, links, etc.) to make testing easier. It is not just about leaving a driver behind, but a driver that is easy to maintain and modify over the years. For bonus points that would probably look good on your CV. 

Historical note: EFI did not start with the FAT32 file system as it required a license. EFI started with a made-up file system. The Microsoft file system team rejected this as it turns out testing file systems for all the possible end cases, across the range of produces and consumers is very very complicated. Microsoft ended up contributing FAT32 to EFI so that another new file system would not end up in the world that needed to be validated. So that is probably a good argument to invest in testing this new driver. 

Thanks,

Andrew Fish

> Thanks,
> Pedro
> 


[-- Attachment #2: Type: text/html, Size: 2644 bytes --]