Re: [PATCH 5/6] x86/paravirt: Switch MSR access pv_ops functions to instruction interfaces

[Juergen Gross <jgross@suse.com>]

[-- Attachment #1.1.1: Type: text/plain, Size: 1363 bytes --]

On 26.08.25 12:39, Jürgen Groß wrote:
> On 25.08.25 03:54, Xin Li wrote:
>> On 6/11/2025 5:58 AM, Juergen Gross wrote:
>>> I'm just doing a V2 of my series, but this time including the additional
>>> support of the non-serializing and immediate forms. Lets see how this will
>>> look like. I will drop using the EAX_EDX_* macros, but due to the reason
>>> mentioned above I won't switch to your variant completely.
>>
>> Hi Juergen,
>>
>> Do you have any update on this?
> 
> I've been very busy with other stuff (downstream, security, ...).
> 
> In between I've been working on the series. I hope to post it some time in
> September.

I have been working on this the last week.

Turns out things are a little bit complicated for adding them into the
paravirt framework, especially regarding the exception fixups.

I first thought that Peter's patch "x86/extable: Implement EX_TYPE_FUNC_REWIND"
would help, but I'm seeing problems with his approach in case of shadow stack
being enabled. This case would at least needed to be handled in his patch, as
otherwise shadow stack and normal stack could get out of sync.

For this reason your patch series won't work easily, too.

OTOH using your basic idea it seems to be possible to solve the fixup problem
without needing Peter's patch. I'm working on that approach now.


Juergen

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 3743 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]