[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 12/15] xen/xsm: remove .xsm_initcall.init section


  • To: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
  • From: Daniel De Graaf <dgdegra@xxxxxxxxxxxxx>
  • Date: Fri, 17 Jun 2016 19:17:23 -0400
  • Cc: xen-devel@xxxxxxxxxxxxx
  • Delivery-date: Fri, 17 Jun 2016 23:17:55 +0000
  • Ironport-phdr: 9a23:zzo+HxbUhxmbWi3i6xMA8VH/LSx+4OfEezUN459isYplN5qZpMy/bnLW6fgltlLVR4KTs6sC0LqH9fG5EjVeqb+681k8M7V0HycfjssXmwFySOWkMmbcaMDQUiohAc5ZX0Vk9XzoeWJcGcL5ekGA6ibqtW1aJBzzOEJPK/jvHcaK1oLsh7H0qsSYOlsArQH+SI0xBS3+lR/WuMgSjNkqAYcK4TyNnEF1ff9Lz3hjP1OZkkW0zM6x+Jl+73YY4Kp5pIZoGJ/3dKUgTLFeEC9ucyVsvJWq5iTZSUOzwldUEiBMylsbSzTCuTL9WI34vmPApOdmyjSGMM6+crEwWnz27KBtUhbkzjgGMz0R93vegch9yqlcpUTyiQZ4xtv4aYeUOf42Uq6VUskTTGQJCspeWyFOGIqUc5oED+1HO/1R6Yb6uQ1d/lOFGQCwCba3mXdzjXjs0Ph/ir55HA==
  • List-id: Xen developer discussion <xen-devel.lists.xen.org>

On 06/17/2016 01:21 PM, Konrad Rzeszutek Wilk wrote:
On Fri, Jun 17, 2016 at 01:18:47PM -0400, Daniel De Graaf wrote:
On 06/17/2016 01:14 PM, Konrad Rzeszutek Wilk wrote:
On Fri, Jun 17, 2016 at 01:04:01PM -0400, Daniel De Graaf wrote:
On 06/17/2016 11:50 AM, Konrad Rzeszutek Wilk wrote:
On Thu, Jun 09, 2016 at 10:47:15AM -0400, Daniel De Graaf wrote:
Since FLASK is the only implementation of XSM hooks in Xen, using an
iterated initcall dispatch for setup is overly complex.  Change this to

As such, should the Kconfig file enable the FLASK by default?
Or make the XSM entry have the configuration for FLASK?

Or perhaps make the FLASK be the visible one and select XSM?

XSM has previously been the configuration option to enable.  If XSM is
enabled (by choice), FLASK will then be enabled by default.

Ah, OK. I need to check, but could you disable FLASK and still have XSM enabled?

Yes, but it won't do anything except slow down the hypervisor a bit.
It's the same behavior as enabling flask and passing "flask=disabled" on
the hypervisor command line.

Right. Would it make sense to squash XSM and FLASK together in the Kconfig?

When making the Kconifg changes, I originally tried making a choice option for
the XSM provider (like for scheduler), with FLASK and dummy being the two
choices.  I discarded that because it didn't add anything useful over the
binary option.

However, thinking about it, if it doesn't make sense to disable FLASK then
hiding the option would simplify the configuration. It removes the grouping of
flask-only options, but since there's only one that isn't all that important.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.