This patch adds enumeration constant VFIO_AP_CFG_CHG_IRQ_INDEX to specify
an IRQ index for signaling that a change has been made to the guest's AP
configuration. This is a placeholder for QEMU patches that use this value
since it is a linux-headers update which includes changes that aren't
merged into the kernel. Linux-headers patches should be generated using
scripts/update-linux-headers.sh.

Signed-off-by: Rorie Reyes <rreyes@linux.ibm.com>
---
 linux-headers/linux/vfio.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/linux-headers/linux/vfio.h b/linux-headers/linux/vfio.h
index 1b5e254d6a..d0426b5ec0 100644
--- a/linux-headers/linux/vfio.h
+++ b/linux-headers/linux/vfio.h
@@ -671,6 +671,7 @@ enum {
  */
 enum {
 	VFIO_AP_REQ_IRQ_INDEX,
+	VFIO_AP_CFG_CHG_IRQ_INDEX,
 	VFIO_AP_NUM_IRQS
 };
 
-- 
2.39.5 (Apple Git-154)

Hello Rorie,

On 1/7/25 19:43, Rorie Reyes wrote:
> This patch adds enumeration constant VFIO_AP_CFG_CHG_IRQ_INDEX to specify
> an IRQ index for signaling that a change has been made to the guest's AP
> configuration. This is a placeholder for QEMU patches that use this value
> since it is a linux-headers update which includes changes that aren't
> merged into the kernel.

Is there an upstream proposal for this change that we could look at ?
It is nice to mention related series in the cover letter.

Thanks,

C.



> Linux-headers patches should be generated using
> scripts/update-linux-headers.sh.
> 
> Signed-off-by: Rorie Reyes <rreyes@linux.ibm.com>
> ---
>   linux-headers/linux/vfio.h | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/linux-headers/linux/vfio.h b/linux-headers/linux/vfio.h
> index 1b5e254d6a..d0426b5ec0 100644
> --- a/linux-headers/linux/vfio.h
> +++ b/linux-headers/linux/vfio.h
> @@ -671,6 +671,7 @@ enum {
>    */
>   enum {
>   	VFIO_AP_REQ_IRQ_INDEX,
> +	VFIO_AP_CFG_CHG_IRQ_INDEX,
>   	VFIO_AP_NUM_IRQS
>   };
>

On 1/8/25 2:29 AM, Cédric Le Goater wrote:
> Hello Rorie,
>
> On 1/7/25 19:43, Rorie Reyes wrote:
>> This patch adds enumeration constant VFIO_AP_CFG_CHG_IRQ_INDEX to 
>> specify
>> an IRQ index for signaling that a change has been made to the guest's AP
>> configuration. This is a placeholder for QEMU patches that use this 
>> value
>> since it is a linux-headers update which includes changes that aren't
>> merged into the kernel.
>
> Is there an upstream proposal for this change that we could look at ?
> It is nice to mention related series in the cover letter.
>
> Thanks,
>
> C.
>
>
Hey Cedric,

There is no upstream proposal for this. It is an s390 internal line item.

Thanks,

Rorie

On Tue, 2025-01-14 at 13:51 -0500, Rorie Reyes wrote:
> On 1/8/25 2:29 AM, Cédric Le Goater wrote:
> > Hello Rorie,
> > 
> > On 1/7/25 19:43, Rorie Reyes wrote:
> > > This patch adds enumeration constant VFIO_AP_CFG_CHG_IRQ_INDEX to 
> > > specify
> > > an IRQ index for signaling that a change has been made to the guest's AP
> > > configuration. This is a placeholder for QEMU patches that use this 
> > > value
> > > since it is a linux-headers update which includes changes that aren't
> > > merged into the kernel.
> > 
> > Is there an upstream proposal for this change that we could look at ?
> > It is nice to mention related series in the cover letter.
> > 
> > Thanks,
> > 
> > C.
> > 
> > 
> Hey Cedric,
> 
> There is no upstream proposal for this. It is an s390 internal line item.

Rorie,

I think Cedric was looking for the matching kernel code you proposed, which this QEMU series depends
on:

https://lore.kernel.org/r/20250107183645.90082-1-rreyes@linux.ibm.com/

Thanks,
Eric

On 1/14/25 20:55, Eric Farman wrote:
> On Tue, 2025-01-14 at 13:51 -0500, Rorie Reyes wrote:
>> On 1/8/25 2:29 AM, Cédric Le Goater wrote:
>>> Hello Rorie,
>>>
>>> On 1/7/25 19:43, Rorie Reyes wrote:
>>>> This patch adds enumeration constant VFIO_AP_CFG_CHG_IRQ_INDEX to
>>>> specify
>>>> an IRQ index for signaling that a change has been made to the guest's AP
>>>> configuration. This is a placeholder for QEMU patches that use this
>>>> value
>>>> since it is a linux-headers update which includes changes that aren't
>>>> merged into the kernel.
>>>
>>> Is there an upstream proposal for this change that we could look at ?
>>> It is nice to mention related series in the cover letter.
>>>
>>> Thanks,
>>>
>>> C.
>>>
>>>
>> Hey Cedric,
>>
>> There is no upstream proposal for this. It is an s390 internal line item.
> 
> Rorie,
> 
> I think Cedric was looking for the matching kernel code you proposed, which this QEMU series depends
> on:
> 
> https://lore.kernel.org/r/20250107183645.90082-1-rreyes@linux.ibm.com/

There is a v2 in the works, right? is that 6.14 material ?

Thanks,

C.


> 
> Thanks,
> Eric
> 

On Fri, 2025-01-24 at 09:49 +0100, Cédric Le Goater wrote:
> On 1/14/25 20:55, Eric Farman wrote:
> > On Tue, 2025-01-14 at 13:51 -0500, Rorie Reyes wrote:
> > > On 1/8/25 2:29 AM, Cédric Le Goater wrote:
> > > > Hello Rorie,
> > > > 
> > > > On 1/7/25 19:43, Rorie Reyes wrote:
> > > > > This patch adds enumeration constant VFIO_AP_CFG_CHG_IRQ_INDEX to
> > > > > specify
> > > > > an IRQ index for signaling that a change has been made to the guest's AP
> > > > > configuration. This is a placeholder for QEMU patches that use this
> > > > > value
> > > > > since it is a linux-headers update which includes changes that aren't
> > > > > merged into the kernel.
> > > > 
> > > > Is there an upstream proposal for this change that we could look at ?
> > > > It is nice to mention related series in the cover letter.
> > > > 
> > > > Thanks,
> > > > 
> > > > C.
> > > > 
> > > > 
> > > Hey Cedric,
> > > 
> > > There is no upstream proposal for this. It is an s390 internal line item.
> > 
> > Rorie,
> > 
> > I think Cedric was looking for the matching kernel code you proposed, which this QEMU series depends
> > on:
> > 
> > https://lore.kernel.org/r/20250107183645.90082-1-rreyes@linux.ibm.com/
> 
> There is a v2 in the works, right? is that 6.14 material ?

Yes, but will probably miss the current merge window. :(

Thanks,
Eric