All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 00/12] Add MDESC & VIO support for VCC
@ 2017-06-23 18:58 Jag Raman
  2017-06-25 20:46 ` David Miller
  0 siblings, 1 reply; 2+ messages in thread
From: Jag Raman @ 2017-06-23 18:58 UTC (permalink / raw)
  To: sparclinux

This series of patches is part of an effort to add VCC (Virtual Console
Concentrator) support to Linux.

VCC enables the virtualization of serial console on SPARC processors. VCC 
provides access to the guest domain's serial console.

VCC depends on some core functionalities in the linux kernel for SPARC. The 
functionalities include LDC (Logical Domain Channels), MDESC (Machine
Descriptor) and VIO (Virtual IO protocol). In order for VCC to be enabled,
it requires that these core functionalities support them.

This series of patches adds MDESC & VIO support to enable VCC on Linux. It
is the second batch of changes to enable VCC.

This version of the series addresses the following changes
suggested by Dave Miller
Patch 4/5:
 - "name" field in vdev_port md_node_info is declared as "const char *"
 - Code has been modified to dynamically allocate & free "name" in
   vdev_port md_node_info
 - Parameters to mdesc_get_node(), mdesc_get_node_info() &
   mdesc_get_node_ops() have been updated to use "const char *"

Patch 6/5:
 - "node_name" parameter in vio_create_one() has been
   changed to "const char *" type from "char *"
 - Typecasts in vio_create_one() invocations to convert
   "const char *" to "char *" have been removed

Patch 11/5:
 - Invocations of mdesc_node_get() & mdesc_get_node_info()
   have been updated to use the prototypes defined in patch 4/5

Jag Raman (12):
  sparc64: ensure VIO operations are defined while being used
  sparc64: specify the device class in VIO version info. packet
  sparc64: skip handshake for LDC channels in RAW mode
  sparc64: expand MDESC interface
  sparc64: mdesc: use __GFP_REPEAT action modifier for VM allocation
  sparc64: add MDESC node name property to VIO device metadata
  sparc64: refactor code to obtain cfg_handle property from MDESC
  sparc64: remove restriction on VIO device name size
  sparc64: check if a client is allowed to register for MDESC
    notifications
  sparc64: enhance VIO device probing
  sparc64: Enhance search for VIO device in MDESC
  sparc64: add port_id to VIO device metadata

 arch/sparc/include/asm/mdesc.h |   24 +++-
 arch/sparc/include/asm/vio.h   |   13 ++-
 arch/sparc/kernel/ldc.c        |   10 ++
 arch/sparc/kernel/mdesc.c      |  331 ++++++++++++++++++++++++++++++++++------
 arch/sparc/kernel/vio.c        |  244 ++++++++++++++++--------------
 arch/sparc/kernel/viohs.c      |   24 +++-
 6 files changed, 479 insertions(+), 167 deletions(-)


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH v2 00/12] Add MDESC & VIO support for VCC
  2017-06-23 18:58 [PATCH v2 00/12] Add MDESC & VIO support for VCC Jag Raman
@ 2017-06-25 20:46 ` David Miller
  0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2017-06-25 20:46 UTC (permalink / raw)
  To: sparclinux

From: Jag Raman <jag.raman@oracle.com>
Date: Fri, 23 Jun 2017 14:58:28 -0400

> This series of patches is part of an effort to add VCC (Virtual Console
> Concentrator) support to Linux.
 ...

Series applied to sparc-next thanks.

I'm concerned about the MDESC allocation GFP flags change.

If we fail the MDESC update, that's it, we can end up never seeing the
new devices which were advertised or properly releasing the ones which
got removed.

If GFP_NOFAIL isn't the solution, then we need something that is.

A bulletproof solution would be possible if we knew the sizes of the
MDESC or an upper bounds ahead of time, but of course we do not.  That
way we could simply always have two buffers, and switch from one to
the other when an MDESC update occurs.

Any better ideas?

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-06-25 20:46 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-06-23 18:58 [PATCH v2 00/12] Add MDESC & VIO support for VCC Jag Raman
2017-06-25 20:46 ` David Miller

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.