[PATCH v2 0/3] xenconsole: Add connection flag

Jason Andryuk posted 3 patches 2 months, 1 week ago
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20250822213946.245307-1-jason.andryuk@amd.com
tools/console/daemon/io.c                |  4 +++
tools/include/xenguest.h                 |  4 +++
tools/libs/guest/xg_dom_arm.c            |  2 +-
tools/libs/guest/xg_dom_boot.c           | 36 ++++++++++++++++++++++++
tools/libs/guest/xg_dom_x86.c            |  6 ++--
tools/libs/guest/xg_sr_restore_x86_hvm.c |  2 +-
tools/libs/guest/xg_sr_restore_x86_pv.c  |  1 +
xen/include/public/io/console.h          | 13 +++++++++
8 files changed, 63 insertions(+), 5 deletions(-)
[PATCH v2 0/3] xenconsole: Add connection flag
Posted by Jason Andryuk 2 months, 1 week ago
Add a connection flag to the console interface page so a domain can tell
if it is connected or not.  This became a series in v2 to add flag
setting to libxenguest.

Jason Andryuk (3):
  xenconsole: Add connection flag
  libs/guest: Set console page to disconnected
  libs/guest: Set console as disconnected on resume

 tools/console/daemon/io.c                |  4 +++
 tools/include/xenguest.h                 |  4 +++
 tools/libs/guest/xg_dom_arm.c            |  2 +-
 tools/libs/guest/xg_dom_boot.c           | 36 ++++++++++++++++++++++++
 tools/libs/guest/xg_dom_x86.c            |  6 ++--
 tools/libs/guest/xg_sr_restore_x86_hvm.c |  2 +-
 tools/libs/guest/xg_sr_restore_x86_pv.c  |  1 +
 xen/include/public/io/console.h          | 13 +++++++++
 8 files changed, 63 insertions(+), 5 deletions(-)

-- 
2.50.1
Re: [PATCH v2 0/3] xenconsole: Add connection flag
Posted by Jürgen Groß 1 month, 3 weeks ago
On 22.08.25 23:39, Jason Andryuk wrote:
> Add a connection flag to the console interface page so a domain can tell
> if it is connected or not.  This became a series in v2 to add flag
> setting to libxenguest.
> 
> Jason Andryuk (3):
>    xenconsole: Add connection flag
>    libs/guest: Set console page to disconnected
>    libs/guest: Set console as disconnected on resume
> 
>   tools/console/daemon/io.c                |  4 +++
>   tools/include/xenguest.h                 |  4 +++
>   tools/libs/guest/xg_dom_arm.c            |  2 +-
>   tools/libs/guest/xg_dom_boot.c           | 36 ++++++++++++++++++++++++
>   tools/libs/guest/xg_dom_x86.c            |  6 ++--
>   tools/libs/guest/xg_sr_restore_x86_hvm.c |  2 +-
>   tools/libs/guest/xg_sr_restore_x86_pv.c  |  1 +
>   xen/include/public/io/console.h          | 13 +++++++++
>   8 files changed, 63 insertions(+), 5 deletions(-)
> 

For the series:

Reviewed-by: Juergen Gross <jgross@suse.com>


Juergen
Re: [PATCH v2 0/3] xenconsole: Add connection flag
Posted by Anthony PERARD 1 month ago
On Tue, Sep 09, 2025 at 12:18:26PM +0200, Jürgen Groß wrote:
> On 22.08.25 23:39, Jason Andryuk wrote:
> > Add a connection flag to the console interface page so a domain can tell
> > if it is connected or not.  This became a series in v2 to add flag
> > setting to libxenguest.
> > 
> > Jason Andryuk (3):
> >    xenconsole: Add connection flag
> >    libs/guest: Set console page to disconnected
> >    libs/guest: Set console as disconnected on resume
> > 
> >   tools/console/daemon/io.c                |  4 +++
> >   tools/include/xenguest.h                 |  4 +++
> >   tools/libs/guest/xg_dom_arm.c            |  2 +-
> >   tools/libs/guest/xg_dom_boot.c           | 36 ++++++++++++++++++++++++
> >   tools/libs/guest/xg_dom_x86.c            |  6 ++--
> >   tools/libs/guest/xg_sr_restore_x86_hvm.c |  2 +-
> >   tools/libs/guest/xg_sr_restore_x86_pv.c  |  1 +
> >   xen/include/public/io/console.h          | 13 +++++++++
> >   8 files changed, 63 insertions(+), 5 deletions(-)
> > 
> 
> For the series:
> 
> Reviewed-by: Juergen Gross <jgross@suse.com>

For the series:
Acked-by: Anthony PERARD <anthony.perard@vates.tech>

Hi Oleksii,
I think this series needs your "release-ack" tag.

Thanks,

-- 
Anthony PERARD
Re: [PATCH v2 0/3] xenconsole: Add connection flag
Posted by Oleksii Kurochko 1 month ago
On 9/29/25 3:21 PM, Anthony PERARD wrote:
> On Tue, Sep 09, 2025 at 12:18:26PM +0200, Jürgen Groß wrote:
>> On 22.08.25 23:39, Jason Andryuk wrote:
>>> Add a connection flag to the console interface page so a domain can tell
>>> if it is connected or not.  This became a series in v2 to add flag
>>> setting to libxenguest.
>>>
>>> Jason Andryuk (3):
>>>     xenconsole: Add connection flag
>>>     libs/guest: Set console page to disconnected
>>>     libs/guest: Set console as disconnected on resume
>>>
>>>    tools/console/daemon/io.c                |  4 +++
>>>    tools/include/xenguest.h                 |  4 +++
>>>    tools/libs/guest/xg_dom_arm.c            |  2 +-
>>>    tools/libs/guest/xg_dom_boot.c           | 36 ++++++++++++++++++++++++
>>>    tools/libs/guest/xg_dom_x86.c            |  6 ++--
>>>    tools/libs/guest/xg_sr_restore_x86_hvm.c |  2 +-
>>>    tools/libs/guest/xg_sr_restore_x86_pv.c  |  1 +
>>>    xen/include/public/io/console.h          | 13 +++++++++
>>>    8 files changed, 63 insertions(+), 5 deletions(-)
>>>
>> For the series:
>>
>> Reviewed-by: Juergen Gross<jgross@suse.com>
> For the series:
> Acked-by: Anthony PERARD<anthony.perard@vates.tech>
>
> Hi Oleksii,
> I think this series needs your "release-ack" tag.\

It is a little bit too late. But considering that this patch should increase
boot performance (right?) and it is pretty straightforward, I think (IIUC regarding
boot performance) we can consider these patch series to be in 4.21:
  Release-Acked-By: Oleksii Kurochko<oleksii.kurochko@gmail.com>

~ Oleksii