Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1239505

Re: [PATCH v3 0/4] SysFS driver for QEMU fw_cfg device

From Mark Rutland <mark.rutland@arm.com>
Newsgroups linux.kernel
Subject Re: [PATCH v3 0/4] SysFS driver for QEMU fw_cfg device
Date 2015-10-05 14:30 +0200
Message-ID <qgd3Y-7eX-17@gated-at.bofh.it> (permalink)
References <qfEIV-8ui-3@gated-at.bofh.it> <qgaSu-4bV-11@gated-at.bofh.it> <qgcrg-6gj-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Mon, Oct 05, 2015 at 01:48:52PM +0200, Paolo Bonzini wrote:
> 
> 
> On 05/10/2015 12:00, Mark Rutland wrote:
> > Some of the keys in the example look like they'd come from other sources
> > (e.g. the *-tables entries), while others look like kernel/bootloader
> > configuration options (e.g. etc/boot-fail-wait, bootorder) -- I'm
> > concerned about redundancy here.
> 
> The redundancy is because the firmware and the bootloader actually
> _consume_ these fw_cfg strings to produce the others (the ACPI tables,
> the kernel configuration options).
> 
> On the other hand, hiding some strings just because they ought to have
> been consumed already makes little sense.

Sure. However, I'm concerned that providing redundant interfaces for
those could lead to people grabbing information from here (because it's
convenient) rather than the existing canonical locations, which means we
get more software that works on fewer systems for no good reason.

What I couldn't figure out was what _additional_ information this
provided; it looked like a mixed bag of details we could already get
from disparate sources. If that's all it does, then it seems to me like
it doesn't add any benefit and potentially makes things worse.

So what do we get from this interface that we cannot get elsewhere, and
why is this the best way of exposing it?

Mark.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v3 0/4] SysFS driver for QEMU fw_cfg device "Gabriel L. Somlo" <somlo@cmu.edu> - 2015-10-04 01:50 +0200
  Re: [PATCH v3 0/4] SysFS driver for QEMU fw_cfg device Mark Rutland <mark.rutland@arm.com> - 2015-10-05 12:10 +0200
    Re: [PATCH v3 0/4] SysFS driver for QEMU fw_cfg device Paolo Bonzini <pbonzini@redhat.com> - 2015-10-05 13:50 +0200
      Re: [PATCH v3 0/4] SysFS driver for QEMU fw_cfg device Mark Rutland <mark.rutland@arm.com> - 2015-10-05 14:30 +0200
        Re: [PATCH v3 0/4] SysFS driver for QEMU fw_cfg device "Gabriel L. Somlo" <somlo@cmu.edu> - 2015-10-05 14:50 +0200
          Re: [PATCH v3 0/4] SysFS driver for QEMU fw_cfg device Mark Rutland <mark.rutland@arm.com> - 2015-10-05 15:00 +0200
            Re: [PATCH v3 0/4] SysFS driver for QEMU fw_cfg device "Gabriel L. Somlo" <somlo@cmu.edu> - 2015-10-05 15:30 +0200
    Re: [PATCH v3 0/4] SysFS driver for QEMU fw_cfg device "Gabriel L. Somlo" <somlo@cmu.edu> - 2015-10-05 14:50 +0200
      Re: [PATCH v3 0/4] SysFS driver for QEMU fw_cfg device Peter Maydell <peter.maydell@linaro.org> - 2015-10-05 15:00 +0200
        Re: [PATCH v3 0/4] SysFS driver for QEMU fw_cfg device "Gabriel L. Somlo" <somlo@cmu.edu> - 2015-10-05 15:20 +0200
        Re: [PATCH v3 0/4] SysFS driver for QEMU fw_cfg device Paolo Bonzini <pbonzini@redhat.com> - 2015-10-05 15:20 +0200
      Re: [PATCH v3 0/4] SysFS driver for QEMU fw_cfg device Mark Rutland <mark.rutland@arm.com> - 2015-10-05 15:10 +0200
        Re: [PATCH v3 0/4] SysFS driver for QEMU fw_cfg device Laszlo Ersek <lersek@redhat.com> - 2015-10-06 09:20 +0200

csiph-web