Re: [PATCH v5 0/2] rbind mounts for /dev and /sys

[Henning Schild <henning.schild@siemens.com>]

Looking at how many places we have to change to get the mounts right,
we most likely want all that logic in a function. I recently had issues
with an imager class in a layer, that also carries mounting logic.

But i guess that would be a cleanup step on top.

Henning

Am Sat, 17 Nov 2018 15:56:57 +0100
schrieb Cedric Hombourger <Cedric_Hombourger@mentor.com>:

> Some of the packages I have been rebuilding within Isar require /sys
> mounted. For instance, golang-google-grpc runs some package tests
> after it builds itself and reads properties from /sys/class/dmi.
> 
> The initial version of this patch series only added a mount of /sys.
> Henning then suggested that we follow Gentoo's guidelines for
> chroots. As such, this patch series is now mounting /dev and /sys as
> rbind mounts but leaves /proc as a regular procfs mount (like Gentoo
> does).
> 
> ChangeLog:
> 
> v5: cover letter added as suggested by Jan (private e-mail)
> v4: limit rbind mounts to /dev and /sys, leave /proc as a regular
> mount v3: rbind mounts for /dev, /proc and /sys, add explicit umount
> calls v2: create /sys directory
> v1: initial submission, simply mount /sys
> 
> Cedric Hombourger (2):
>   buildchroot+bootstrap: rbind mount for /dev
>   buildchroot: add sysfs mount since expected by some packages
> 
>  meta/classes/buildchroot.bbclass                    | 5 ++++-
>  meta/classes/image.bbclass                          | 3 ++-
>  meta/classes/isar-bootstrap-helper.bbclass          | 6 +++++-
>  meta/classes/isar-image.bbclass                     | 3 ++-
>  meta/recipes-core/isar-bootstrap/isar-bootstrap.inc | 8 ++++++--
>  5 files changed, 19 insertions(+), 6 deletions(-)
>