Skip to content

Conversation

@nbriggs
Copy link
Contributor

@nbriggs nbriggs commented Dec 17, 2025

In the case that the ethernet is unavailable the \ETHEREVENTFN should still call \SETETHERFLAGS, \SETLOCALNSNUMBERS, and \FLUSHNDBS, or a newly started image will retain information from the system on which it was built (such as ethernet MAC address in the NS address). This will also ensure that the NS address is not set to 0, instead initializing with a fake address based on the system serial number (calls \SERIALNUMBER).

In the case that the ethernet is unavailable the \ETHEREVENTFN should
still call \SETETHERFLAGS, \SETLOCALNSNUMBERS, and \FLUSHNDBS, or
a newly started image will retain information from the system on which
it was built (such as ethernet MAC address in the NS address).  This
will also ensure that the NS address is not set to 0, instead initializing
with a fake  address based on the system serial number (calls \SERIALNUMBER).
@nbriggs
Copy link
Contributor Author

nbriggs commented Dec 17, 2025

Fixes issue #2413

@pamoroso
Copy link
Contributor

I'm running this branch on Linux Mint 22.1 Cinnamon and so far I haven't noticed anything unusual.

@nbriggs
Copy link
Contributor Author

nbriggs commented Dec 18, 2025

Check \MY.NSADDRESS with the new and old Maiko in fresh sysouts, they should be different.

@pamoroso
Copy link
Contributor

What new Maiko? Isn't this change limited to Medley?

@nbriggs
Copy link
Contributor Author

nbriggs commented Dec 18, 2025

Sorry, yeah, I haven't pushed that. You should still check \MY.NSADDRESS with the new Lisp code though.

@pamoroso
Copy link
Contributor

Under Medley/nhb-fix-ethereventfn and Maiko/master:

4_ \MY.NSADDRESS
0#0.12345.401#

@nbriggs
Copy link
Contributor Author

nbriggs commented Dec 18, 2025

Perfect, thanks. That's what I'd hoped we'd see. The .12345. is constant for a non-ethernet Medley, the 401 is coming from the "host serial number" (low 16 bits of the result of gethostid()).

@nbriggs
Copy link
Contributor Author

nbriggs commented Dec 20, 2025

This PR should wait for an additional change -- \FIND.LOCALPUPHOSTNUMBER should not call \LOOKUPPUPNUMBER if there's no ethernet available. That is, starting with

          [COND
             [(AND (\ETHER-AVAILABLE)
                   (SETQ NEWNUMBER (\LOOKUPPUPNUMBER \MY.NSHOSTNUMBER NDB)))
          ...

I'd just make the change to the branch, but I'm having trouble getting a clean update to PUP.

@MattHeffron
Copy link
Contributor

This PR should wait for an additional change

Probably should convert this PR to "Draft" until it's ready...

@nbriggs nbriggs marked this pull request as draft December 20, 2025 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants