Page MenuHomeFreeBSD

loader: implement MEDIA_FILEPATH_DP support in efipart
ClosedPublic

Authored by tsoome on Feb 10 2017, 10:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 6, 6:49 AM
Unknown Object (File)
Jan 15 2024, 4:18 PM
Unknown Object (File)
Jan 12 2024, 3:26 AM
Unknown Object (File)
Jan 8 2024, 11:24 AM
Unknown Object (File)
Jan 8 2024, 11:24 AM
Unknown Object (File)
Jan 8 2024, 11:24 AM
Unknown Object (File)
Jan 8 2024, 11:11 AM
Unknown Object (File)
Jan 1 2024, 7:02 PM
Subscribers

Details

Summary

The efipart rework did break the ARM systems as the new code is
using more exact filters to sort the devices and we need to
add support for MEDIA_FILEPATH_DP device paths.

Test Plan

Need to verify on ARM platform.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tsoome retitled this revision from to loader: implement MEDIA_FILEPATH_DP support in efipart.
tsoome updated this object.
tsoome edited the test plan for this revision. (Show Details)
tsoome added reviewers: allanjude, imp.
allanjude edited edge metadata.

Once you get someone to verify this works on an ARM device, approved.

This revision is now accepted and ready to land.Feb 10 2017, 4:35 PM

Once you get someone to verify this works on an ARM device, approved.

It actually does need some more work - while device mapping appears to be OK, there is still something about mapping device path from image handle. Meaning, this snippet is not yet full fix.

I tested it on rpi3, it works with crochet-generated image

OK  lsdev
disk devices:
    disk0:    15564801 X 512 blocks (removable)
      disk0s1: DOS/Windows
      disk0s2: FreeBSD
        disk0s2a: FreeBSD UFS
net devices:
    net0:
In D9520#196965, @gonzo wrote:

I tested it on rpi3, it works with crochet-generated image

OK  lsdev
disk devices:
    disk0:    15564801 X 512 blocks (removable)
      disk0s1: DOS/Windows
      disk0s2: FreeBSD
        disk0s2a: FreeBSD UFS
net devices:
    net0:

does it actually boot properly in your case?

In D9520#196965, @gonzo wrote:

I tested it on rpi3, it works with crochet-generated image

... skipped ...

does it actually boot properly in your case?

Yes, it boots to login prompt

manu added a reviewer: manu.
manu added a subscriber: manu.

As I said on IRC on #bsdmips I don't know if u-boot is doing the right thing on disk (and by u-boot I mean me since I've done the patch for logical partition).
But it used to work before so if one commit broke it and this review restore the func I'm up for it.
Note that I didn't test the patch. It just looks sane for me.

This revision was automatically updated to reflect the committed changes.