linux.git/drivers/vfio/platform, branch v4.4-rc2

VFIO: platform: reset: AMD xgbe reset module

2015-11-03T19:55:21+00:00

This patch introduces a module that registers and implements a low-level
reset function for the AMD XGBE device.

it performs the following actions:
- reset the PHY
- disable auto-negotiation
- disable & clear auto-negotiation IRQ
- soft-reset the MAC

Those tiny pieces of code are inherited from the native xgbe driver.

Signed-off-by: Eric Auger 
Reviewed-by: Arnd Bergmann 
Signed-off-by: Alex Williamson

vfio: platform: reset: calxedaxgmac: fix ioaddr leak

2015-11-03T19:55:06+00:00

In the current code the vfio_platform_region is copied on the stack.
As a consequence the ioaddr address is not iounmapped in the vfio
platform driver (vfio_platform_regions_cleanup). The patch uses the
pointer to the region instead.

Signed-off-by: Eric Auger 
Signed-off-by: Alex Williamson

vfio: platform: add dev_info on device reset

2015-11-03T19:55:05+00:00

It might be helpful for the end-user to check the device reset
function was found by the vfio platform reset framework.

Lets store a pointer to the struct device in vfio_platform_device
and trace when the reset function is called or not found.

Signed-off-by: Eric Auger 
Signed-off-by: Alex Williamson

vfio: platform: use list of registered reset function

2015-11-03T19:55:03+00:00

Remove the static lookup table and use the dynamic list of registered
reset functions instead. Also load the reset module through its alias.
The reset struct module pointer is stored in vfio_platform_device.

We also remove the useless struct device pointer parameter in
vfio_platform_get_reset.

This patch fixes the issue related to the usage of __symbol_get, which
besides from being moot, prevented compilation with CONFIG_MODULES
disabled.

Also usage of MODULE_ALIAS makes possible to add a new reset module
without needing to update the framework. This was suggested by Arnd.

Signed-off-by: Eric Auger 
Reported-by: Arnd Bergmann 
Reviewed-by: Arnd Bergmann 
Signed-off-by: Alex Williamson

vfio: platform: add compat in vfio_platform_device

2015-11-03T19:55:02+00:00

Let's retrieve the compatibility string on probe and store it
in the vfio_platform_device struct

Signed-off-by: Eric Auger 
Signed-off-by: Alex Williamson

vfio: platform: reset: calxedaxgmac: add reset function registration

2015-11-03T19:55:00+00:00

This patch adds the reset function registration/unregistration.
This is handled through the module_vfio_reset_handler macro. This
latter also defines a MODULE_ALIAS which simplifies the load from
vfio-platform.

Signed-off-by: Eric Auger 
Reviewed-by: Arnd Bergmann 
Signed-off-by: Alex Williamson

vfio: platform: introduce module_vfio_reset_handler macro

2015-11-03T19:54:59+00:00

The module_vfio_reset_handler macro
- define a module alias
- implement module init/exit function which respectively registers
  and unregisters the reset function.

Signed-off-by: Eric Auger 
Reviewed-by: Arnd Bergmann 
Signed-off-by: Alex Williamson

vfio: platform: add capability to register a reset function

2015-11-03T19:54:57+00:00

In preparation for subsequent changes in reset function lookup,
lets introduce a dynamic list of reset combos (compat string,
reset module, reset function). The list can be populated/voided with
vfio_platform_register/unregister_reset. Those are not yet used in
this patch.

Signed-off-by: Eric Auger 
Reviewed-by: Arnd Bergmann 
Signed-off-by: Alex Williamson

vfio: platform: introduce vfio-platform-base module

2015-11-03T19:54:56+00:00

To prepare for vfio platform reset rework let's build
vfio_platform_common.c and vfio_platform_irq.c in a separate
module from vfio-platform and vfio-amba. This makes possible
to have separate module inits and works around a race between
platform driver init and vfio reset module init: that way we
make sure symbols exported by base are available when vfio-platform
driver gets probed.

The open/release being implemented in the base module, the ref
count is applied to the parent module instead.

Signed-off-by: Eric Auger 
Suggested-by: Arnd Bergmann 
Reviewed-by: Arnd Bergmann 
Signed-off-by: Alex Williamson

vfio/platform: store mapped memory in region, instead of an on-stack copy

2015-11-03T19:54:00+00:00

vfio_platform_{read,write}_mmio() call ioremap_nocache() to map
a region of io memory, which they store in struct vfio_platform_region to
be eventually re-used, or unmapped by vfio_platform_regions_cleanup().

These functions receive a copy of their struct vfio_platform_region
argument on the stack - so these mapped areas are always allocated, and
always leaked.

Pass this argument as a pointer instead.

Fixes: 6e3f26456009 "vfio/platform: read and write support for the device fd"
Signed-off-by: James Morse 
Acked-by: Baptiste Reynal 
Tested-by: Baptiste Reynal 
Signed-off-by: Alex Williamson