linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@linux-foundation.org>
To: Mary Strodl <mstrodl@freedom.csh.rit.edu>
Cc: Matthew Wilcox <willy@infradead.org>,
	Christoph Hellwig <hch@infradead.org>,
	Mary Strodl <mstrodl@csh.rit.edu>,
	linux-kernel@vger.kernel.org, urezki@gmail.com,
	linux-mm@kvack.org, lee@kernel.org, andi.shyti@kernel.org,
	linux-i2c@vger.kernel.org, s.hauer@pengutronix.de,
	christian.gmeiner@gmail.com
Subject: Re: [PATCH 1/3] mm: vmalloc: export __vmalloc_node_range
Date: Thu, 18 Jul 2024 14:31:03 -0700	[thread overview]
Message-ID: <20240718143103.82e33c556b2d1b6145ae43e0@linux-foundation.org> (raw)
In-Reply-To: <ZpkWj-iFiA-JHbbf@freedom.csh.rit.edu>

On Thu, 18 Jul 2024 09:20:15 -0400 Mary Strodl <mstrodl@freedom.csh.rit.edu> wrote:

> On Thu, Jul 18, 2024 at 01:53:23PM +0100, Matthew Wilcox wrote:
> > That does work, but I would assume that since this BIOS exists to
> > communicate with the hardware that it'd need various special privileges
> > and that running in ring 3 would not work.
> 
> Exactly.
>  
> > Ultimately, better off running the whole thing inside a VM and passing
> > the device through to the guest.  Or ignoring the BIOS entirely and
> > implementing direct access to the hardware.  But neither of these
> > approaches is "a way to do what I'm trying to do", they're entirely
> > different approaches to making this hardware work.
> 
> If I ran the whole thing inside a VM, I would still need support in the
> kernel, right?
> 
> As far as I know, there is no documentation on Congatec's side about the
> underlying interface. Obviously I could disassemble the blob in the BIOS
> and figure it out, but I suspect that will have much less hardware
> compatibility and be subject to random breakage if they make a BIOS
> update or something. Plus, I would probably run afoul of copyright if I
> wrote a driver after doing that.
> 
> I'm not really thrilled that this is their design either, but I'm not
> sure that there is a better answer...
> 

The hardware is weird, but we should try to support it in some fashion.
But without making dangerous functionality more widely available.  So
we're looking for some solution which can be fully contained within
that hardware's driver.

Dumb idea, there will be other ideas: is it practical to take that code
blob out of the BIOS, put it into a kernel module (as a .byte table in
a .s file and suitable C interfacing), compile that up and insmod that
module?  


  reply	other threads:[~2024-07-18 21:31 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-07-18  1:15 [PATCH 0/3] Add support for Congatec CGEB BIOS interface Mary Strodl
2024-07-18  1:15 ` [PATCH 1/3] mm: vmalloc: export __vmalloc_node_range Mary Strodl
2024-07-18  2:53   ` Andrew Morton
2024-07-18 12:29     ` Mary Strodl
2024-07-18  3:04   ` Christoph Hellwig
2024-07-18 12:40     ` Mary Strodl
2024-07-18 12:45       ` Matthew Wilcox
2024-07-18 12:49         ` Christoph Hellwig
2024-07-18 12:53           ` Matthew Wilcox
2024-07-18 13:20             ` Mary Strodl
2024-07-18 21:31               ` Andrew Morton [this message]
2024-07-18 21:35                 ` Matthew Wilcox
2024-07-18 21:39                   ` Andrew Morton
2024-07-19  6:41                     ` Christian Gmeiner
2024-07-19 11:58                       ` Mary Strodl
2024-07-19 12:42                         ` Matthew Wilcox
2024-07-24  0:00                           ` Andrew Morton
2024-07-24  0:16                             ` Matthew Wilcox
2024-07-24  1:36                               ` Christoph Hellwig
2024-07-19 19:59                         ` Rudolf Marek
2024-07-22 14:54                           ` Mary Strodl
2024-07-18  1:15 ` [PATCH 2/3] x86: Add basic support for the Congatec CGEB BIOS interface Mary Strodl
2024-07-18  3:56   ` kernel test robot
2024-07-18 14:01   ` kernel test robot
2024-07-18  1:15 ` [PATCH 3/3] i2c: Add Congatec CGEB I2C driver Mary Strodl

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=20240718143103.82e33c556b2d1b6145ae43e0@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=andi.shyti@kernel.org \
    --cc=christian.gmeiner@gmail.com \
    --cc=hch@infradead.org \
    --cc=lee@kernel.org \
    --cc=linux-i2c@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mstrodl@csh.rit.edu \
    --cc=mstrodl@freedom.csh.rit.edu \
    --cc=s.hauer@pengutronix.de \
    --cc=urezki@gmail.com \
    --cc=willy@infradead.org \
    /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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox