On Thu, 15 Mar 2018 13:25:25 -0400
Tony Krowiak <akrowiak@xxxxxxxxxxxxxxxxxx> wrote:
On 03/15/2018 09:25 AM, Pierre Morel wrote:OTOH, nobody else seems to do a select on these symbols so far.
On 14/03/2018 19:25, Tony Krowiak wrote:Connie also asked about this in the last review, so I will go ahead
+config VFIO_APnot sure it must be module by default.
+ def_tristate m
I would not set it by default.
and change it.
I was thinking the same just yesterday and I agree, this makes sense.+ prompt "VFIO support for AP devices"VFIO_MDEV_DEVICE is a general feature *needed* by VFIO_AP
+ depends on ZCRYPT && VFIO_MDEV_DEVICE
and has no use case by its own. If it is set it is obviously because some
mediated device drivers needs it.
while ZCRYPT is a Z feature which may be set without VFIO_AP.
So you need:
config VFIO_AP
def_tristate n
prompt "VFIO support for AP devices"
depends on ZCRYPT
select VFIO_MDEV
select VFIO_MDEV_DEVICE
...
If you decide to go that route, you'll also need to depend on VFIO
(otherwise you could end up selecting symbols with unmet dependencies).Why do you prefer this approach?
All in all, I prefer the 'depends' approach.