Skip to content

Added NUC10 recommendation#182

Merged
rocodes merged 4 commits intomainfrom
nuc10-support
Apr 1, 2021
Merged

Added NUC10 recommendation#182
rocodes merged 4 commits intomainfrom
nuc10-support

Conversation

@eloquence
Copy link
Contributor

@eloquence eloquence commented Mar 17, 2021

Status

Ready for review

Description of Changes

Checklist

  • Doc linting (make docs-lint) passed locally
  • Doc link linting (make docs-linkcheck) passed
  • You have previewed (make docs) docs at http://localhost:8000

@rmol rmol force-pushed the nuc10-support branch 2 times, most recently from 8f87c52 to 37963a3 Compare March 17, 2021 13:42
rocodes
rocodes previously approved these changes Mar 22, 2021
Copy link
Contributor

@rocodes rocodes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks both

@eloquence
Copy link
Contributor Author

eloquence commented Mar 25, 2021

The NUC10i5FNH uses an Aptio V UEFI Firmware Core and the BIOS files are called recovery capsules with the .cap file extension, so the BIOS update guidelines had to be updated to make them a bit more generic.

The Aptio BIOS menu structure is also a bit different than what we have documented via the NUC8 screenshots; since the Aptio core does not seem to offer a native screenshot facility, I haven't retaken those yet, but can use the HDMI capture method to do so.

That said, I am a bit skeptical about maintaining the docs at this level of detail, especially given the frequency of NUC releases and the fact that it's a visual BIOS that's fairly easy to navigate.

Intel's own BIOS docs are pretty solid, so we may want to streamline things a bit further:

  • remove the dedicated BIOS update guidelines and just recommend using the F7 method and verifying the MD5 sum
  • replace screenshots consistently with text-based navigational guidelines that we can more efficiently update for each NUC generation

Thoughts?

@rocodes
Copy link
Contributor

rocodes commented Mar 25, 2021

I look very favourably on cutting down our documentation, I think it's entirely reasonable to link out to other docs and/or expect people installing SecureDrop to find their own way w.r.t BIOS updates.

Erik Moeller and others added 4 commits March 25, 2021 14:31
The visual BIOS provides a clear navigational structure which
we can reference in text; it's not clear that the screenshots
provide significant help when adjusting these settings. This
reduces the burden of updates for each new NUC generation.

Note also that the NUC10 Aptio BIOS does not appear to have a
built-in screenshot facility (not a blocker if we really
need them, but again, plain text is IMO sufficient here).

This commit also cross-references the BIOS settings from
the server guide.

- Under **Advanced ▸ Onboard Devices**, disable all onboard devices
other than LAN: HD audio, microphone, Thunderbolt, WLAN, Bluetooth,
SD card controller, and enhanced consumer infrared.
Copy link
Contributor Author

@eloquence eloquence Mar 25, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know if the infrared option exists for older NUCs; it seems to be intended for remote controls, and worth enumerating. The settings I have not enumerated here are:

  • Gaussian Mixture Models and Neutral Networks Accelerator (enabled by default)
  • HDMI CEC Control (enabled by default)
  • Auto Turn on TV (set to "From S3/S4/S5 boot")
  • Auto Turn OFF TV (disabled by default)
  • Wake On TV (disabled by default)
  • Standby by TV (disabled by default)

If any of those settings should be explicitly changed, I can enumerate them here as well. Note that this settings page scrolls vertically, and the above settings are not immediately visible.

|NUC10 leads|

Before the initial OS installation, boot into the BIOS by pressing **F2** at
startup and adjust the system configuration:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that while it's branded to look very similar, the Aptio BIOS uses a slightly different menu structure than the Visual BIOS for other NUC models.

.. |NUC8 leads| image:: images/hardware/nuc8_leads.jpg
.. |NUC8 VisualBIOS1| image:: images/hardware/nuc8_visualbios1.png
.. |NUC8 VisualBIOS2| image:: images/hardware/nuc8_visualbios2.png
.. |NUC8 VisualBIOS SecureBoot| image:: images/hardware/nuc8_visualbios_secureboot.png
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As noted in the PR summary, I'm removing these screenshots to increase maintainability of the docs, given that the menu structure is generally straightforward to navigate.


- Under **Advanced ▸ Devices ▸ Onboard Devices**, disable all onboard devices
other than LAN: audio, audio DSP, microphone, Thunderbolt, WLAN, Bluetooth,
and SD card.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't have a NUC7 at hand, but I noticed that these BIOS instructions don't include the recommendations we specify for the other NUC models:

  • Under Advanced ▸ Security, disable SGX support, which is not used by SecureDrop and may be targeted by active CPU exploits.
  • Under Advanced ▸ Boot ▸ Secure Boot, uncheck the Secure Boot checkbox.

If these recommendations are applicable on NUC7 as well, and someone can validate that these are the correct menus, I can add them here as well.

Other 7th-generation NUCs have also been reported to work, although we have not
tested them. For example, the `NUC7i5DNHE <https://www.intel.com/content/www/us/en/products/boards-kits/nuc/kits/nuc7i5dnhe.html>`__ uses the same Ethernet chipset as the NUC7i5BNH,
and also has a removable wireless card, simplifying the server setup process.
However, it may be harder to find a retail source for this model.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given that the NUC7 was discontinued last year and we do not recommend installing on NUC7s for new installations at this point, I figured it was best to remove this outdated guidance.


#. A clean USB device to download the ``.bio`` file
#. An internet-connected workstation, such as the *Admin Workstation*
#. A clean USB device to download the BIOS file
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As noted in #187, I would recommend phasing out this chapter of the documentation; I can do that in a separate follow-up PR. For now, the changes in this document should cover us for the supported NUC models.

@eloquence eloquence marked this pull request as ready for review March 25, 2021 22:45
Copy link
Contributor

@rocodes rocodes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Merging

@rocodes rocodes merged commit 28ffac9 into main Apr 1, 2021
@rocodes rocodes deleted the nuc10-support branch April 1, 2021 17:53
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.

Expand docs and hardware support for recent Intel NUCs

3 participants