From: "José Pekkarinen" <jose.pekkarinen@unikie.com>
To: Alyssa Ross <hi@alyssa.is>
Cc: devel@spectrum-os.org
Subject: Re: [PATCH] Add image configuration option
Date: Fri, 16 Sep 2022 07:59:32 +0300 [thread overview]
Message-ID: <CAJPV9Mp4d8P4Cp6SUJC-Hntsgiy=-nNn2F4o7eNe3e=uR0vtoA@mail.gmail.com> (raw)
In-Reply-To: <87zgf0vklc.fsf@alyssa.is>
[-- Attachment #1: Type: text/plain, Size: 2225 bytes --]
On Thu, Sep 15, 2022 at 4:22 PM Alyssa Ross <hi@alyssa.is> wrote:
> José Pekkarinen <jose.pekkarinen@unikie.com> writes:
>
> > On Thu, Sep 15, 2022 at 2:31 PM Alyssa Ross <hi@alyssa.is> wrote:
>
> [...]
>
> You mean you'd like to manually provide a Kconfig file, rathen than
> using Nixpkgs' (not very good) structured config mechanism, right?
> That should be possible with an overlay, but maybe some documentation
> with an example would be a good idea?
>
Yes, but, for example, if I provide the overlay that uses that
that Kconfig, the Kconfig should be present in your system, as some
sort of default configuration for the developer to consume if they want
to use the overlay in question, otherwise, the developer needs to fetch
spectrum sources, and then fetch out the default configuration somewhere
else, put them together and test. The goal would be to upstream the overlay
so that one can take spectrum source code, make a config.nix to select
the overlay, and build, without extra steps to fetch other artifacts.
[...]
>
> Well, it's not the size of the change that's important, but whether it
> can be demonstrated that the change solves a problem. A big change to
> fix a clear problem is fine!
>
It is offering a way to template configurations for the cases
we were commenting before. So it solves a problem, the problem is
that currently the source code doesn't ship default configurations for
developers to test, so I can create a config.nix file downstream that
makes the overlay for the hardened kernel use case, and now, instead
of upstreaming and shipping it with any spectrum checkout, I have
to publish it somewhere else, and document how to put the puzzle
together so that a developer can test, use, and develop further. For
now, it doesn't let combine configuration files, so these templates
may be fat, because you can only make one template per case,
and choose it. In the future it would be good if they are small snippets
that do a particular purpose, and we list all the snippets we want
to make the full use case the user want(for ex. making a cross compiled
build from x86_64 of arm64 which includes security plus debugging).
José.
[-- Attachment #2: Type: text/html, Size: 3681 bytes --]
next prev parent reply other threads:[~2022-09-16 5:00 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-15 7:35 [PATCH] Add image configuration option José Pekkarinen
2022-09-15 8:21 ` Alyssa Ross
2022-09-15 10:42 ` José Pekkarinen
2022-09-15 11:31 ` Alyssa Ross
2022-09-15 12:31 ` José Pekkarinen
2022-09-15 13:22 ` Alyssa Ross
2022-09-15 13:48 ` Ville Ilvonen
2022-09-15 14:00 ` Alyssa Ross
2022-09-15 14:09 ` Ville Ilvonen
2022-09-15 14:47 ` Integrating Spectrum and platform firmware Alyssa Ross
2022-09-16 5:29 ` Ville Ilvonen
2022-09-16 4:59 ` José Pekkarinen [this message]
2022-09-16 7:25 ` [PATCH] Add image configuration option Alyssa Ross
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAJPV9Mp4d8P4Cp6SUJC-Hntsgiy=-nNn2F4o7eNe3e=uR0vtoA@mail.gmail.com' \
--to=jose.pekkarinen@unikie.com \
--cc=devel@spectrum-os.org \
--cc=hi@alyssa.is \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://spectrum-os.org/git/crosvm
https://spectrum-os.org/git/doc
https://spectrum-os.org/git/mktuntap
https://spectrum-os.org/git/nixpkgs
https://spectrum-os.org/git/spectrum
https://spectrum-os.org/git/ucspi-vsock
https://spectrum-os.org/git/www
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).