[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 7/10] qemu-binfmt-conf.sh: add option --reset
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-devel] [PATCH v3 7/10] qemu-binfmt-conf.sh: add option --reset <ARCHS> |
Date: |
Sun, 10 Mar 2019 18:15:16 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 |
On 06/03/2019 05:52, Unai Martinez-Corral wrote:
> This is a partial implementation.
>
> Allows to remove a single or a list of already registered binfmt
> interpreters. If <ARCHS> is a list, it must be comma-separated.
I think ARCHS and CPUS are the same thing, so use the same word.
> Valid values are those in qemu_target_list. If <ARCHS> is 'ALL', all
> the existing 'qemu-*' interpreters are removed.
>
> This is partial because 'debian' and 'systemd' configurations are not
> removed. If option 'reset' is provided before any of those, reset is
> executed first and the configuration proceeds. However, if 'reset' is
> provided after any of them, the script will exit with error 'option
> reset not implemented for this mode yet'.
>
> Removal is done by printing '-1' as explained at:
> https://www.kernel.org/doc/Documentation/admin-guide/binfmt-misc.rst
>
> Signed-off-by: Unai Martinez-Corral <address@hidden>
> ---
> scripts/qemu-binfmt-conf.sh | 36 +++++++++++++++++++++++++++++++++---
> 1 file changed, 33 insertions(+), 3 deletions(-)
>
> diff --git a/scripts/qemu-binfmt-conf.sh b/scripts/qemu-binfmt-conf.sh
> index 2751363089..824e3c4c34 100755
> --- a/scripts/qemu-binfmt-conf.sh
> +++ b/scripts/qemu-binfmt-conf.sh
> @@ -197,8 +197,8 @@ qemu_get_family() {
>
> usage() {
> cat <<EOF
> -Usage: qemu-binfmt-conf.sh [--path PATH][--debian][--systemd]
> - [--help][--credential][--exportdir PATH]
> +Usage: qemu-binfmt-conf.sh [--help][--path PATH][--debian][--systemd]
> + [--reset ARCHS][--credential][--exportdir PATH]
> [--persistent][--suffix SUFFIX][CPUS]
I don't like the idea of being able to set and reset different
interpretes with same command line call.
Make "--reset" exclusive without parameter to be able to use the common
cpu list at the end of the command line (this way, I think you would be
able to share more common code between setting an resetting).
You could have a look to Alex Bennée's patch:
https://patchwork.ozlabs.org/patch/938796/
I think this is the good way to do this and it should merge easily with
your series.
Thanks,
Laurent
- Re: [Qemu-devel] [PATCH v3 3/10] qemu-binfmt-conf.sh: add QEMU_CREDENTIAL and QEMU_PERSISTENT, (continued)
- [Qemu-devel] [PATCH v3 4/10] qemu-binfmt-conf.sh: remove 'qemu' prefix from cli options, Unai Martinez-Corral, 2019/03/05
- [Qemu-devel] [PATCH v3 5/10] qemu-binfmt-conf.sh: honour QEMU_PATH and/or QEMU_SUFFIX, Unai Martinez-Corral, 2019/03/05
- [Qemu-devel] [PATCH v3 6/10] qemu-binfmt-conf.sh: generalize <CPU> to positional <CPUS>, Unai Martinez-Corral, 2019/03/05
- [Qemu-devel] [PATCH v3 7/10] qemu-binfmt-conf.sh: add option --reset <ARCHS>, Unai Martinez-Corral, 2019/03/05
- Re: [Qemu-devel] [PATCH v3 7/10] qemu-binfmt-conf.sh: add option --reset <ARCHS>,
Laurent Vivier <=
- [Qemu-devel] [PATCH v3 8/10] qemu-binfmt-conf.sh: refactor usage(), Unai Martinez-Corral, 2019/03/05
- [Qemu-devel] [PATCH v3 9/10] qemu-binfmt-conf.sh: update usage(), Unai Martinez-Corral, 2019/03/05
- [Qemu-devel] [PATCH v3 10/10] qemu-binfmt-conf.sh: support QEMU_TARGETS, Unai Martinez-Corral, 2019/03/05
- Re: [Qemu-devel] [PATCH v3 0/10] qemu-binfmt-conf.sh, no-reply, 2019/03/05
- Re: [Qemu-devel] [PATCH v3 0/10] qemu-binfmt-conf.sh, no-reply, 2019/03/09
- Re: [Qemu-devel] [PATCH v3 0/10] qemu-binfmt-conf.sh, no-reply, 2019/03/09