Page MenuHomeFreeBSD

[PowerPC] De-giant powermac_nvram, update documentation
ClosedPublic

Authored by bdragon on May 11 2020, 8:55 PM.
Referenced Files
Unknown Object (File)
Sat, Apr 13, 3:07 AM
Unknown Object (File)
Fri, Apr 12, 10:50 PM
Unknown Object (File)
Fri, Apr 12, 9:50 PM
Unknown Object (File)
Mon, Mar 18, 9:41 AM
Unknown Object (File)
Feb 23 2024, 10:51 AM
Unknown Object (File)
Jan 12 2024, 5:22 PM
Unknown Object (File)
Dec 23 2023, 12:31 AM
Unknown Object (File)
Dec 14 2023, 4:32 PM

Details

Summary
  • Remove the giant lock requirement from powermac_nvram.
  • Update manual pages to reflect current state.

(Currently tested on G4 powerbook)

Diff Detail

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

Event Timeline

sys/dev/powermac_nvram/powermac_nvram.c
277 ↗(On Diff #71657)

I'm not really a fan of the way this just ignores the requested position. If we were to honor uio_offset instead, we could avoid locking here.

I guess in practice this driver is exclusive access to start with in any case, as it stores a copy of the nvram in the sc while the device is open, so it doesn't *really* matter.

sys/dev/powermac_nvram/powermac_nvramvar.h
52 ↗(On Diff #71657)

I originally tried this with a mutex, but it turns out that uiomove() can go to sleep.

usr.sbin/nvram/nvram.8
101 ↗(On Diff #71657)

There are a couple of other manpages that reference the eeprom tool, which is gone ever since the sparc64 removal.

bcr added a subscriber: bcr.

OK from manpages.

This revision is now accepted and ready to land.Jun 15 2020, 2:22 AM