All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mary Strodl <mstrodl@freedom.csh.rit.edu>
To: Rudolf Marek <r.marek@assembler.cz>
Cc: Christian Gmeiner <christian.gmeiner@gmail.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	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
Subject: Re: [PATCH 1/3] mm: vmalloc: export __vmalloc_node_range
Date: Mon, 22 Jul 2024 10:54:34 -0400	[thread overview]
Message-ID: <Zp5yqoeH752kli8J@freedom.csh.rit.edu> (raw)
In-Reply-To: <b3d6d20d-1b14-447f-a6b2-aa7712df1156@assembler.cz>

On Fri, Jul 19, 2024 at 09:59:37PM +0200, Rudolf Marek wrote:
> I would suggest to simply run the BIOS code of this interface in usermode. Sort of similar to VM86 VESA stuff.
> Last time I looked into this it used STI/CLI/RDMSR/WRMSR and couple of I/O ports and cf8/cfc for PCI.

I took a look at uvesafb (which appears to be what you were talking about) and
it looks like it starts a separate executable and uses some IPC to talk to it.
Is that the best way to do it?

I guess it would look something like:
- driver gets loaded
- driver spawns /sbin/cgeb-helper
- driver uses cn_netlink_send to send the `high_desc` to helper

Then the calls to `board->entry` in the driver get replaced with
`cn_netlink_send` with a `cgeb_fps`.

When the userspace helper gets the message with the `cgeb_fps`, it calls into
the bios code and replies to the driver with send() and passes back cgeb_fps.

From there, a callback registered with cn_add_callback will pick up the message
and call `wake_up_interruptible` to send the message back to the `cgeb_call`
caller.

Is this what you were imagining? Or is there a simpler way to do it?

Where should the code for the userspace helper live? The vesa stuff appeared to
be out of tree.

  reply	other threads:[~2024-07-22 14:54 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
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 [this message]
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=Zp5yqoeH752kli8J@freedom.csh.rit.edu \
    --to=mstrodl@freedom.csh.rit.edu \
    --cc=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=r.marek@assembler.cz \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.