From mboxrd@z Thu Jan 1 00:00:00 1970 Received: by 10.25.208.211 with SMTP id h202csp1150125lfg; Fri, 19 Feb 2016 05:17:20 -0800 (PST) X-Received: by 10.28.224.87 with SMTP id x84mr9645722wmg.32.1455887840598; Fri, 19 Feb 2016 05:17:20 -0800 (PST) Return-Path: Received: from mail-wm0-x22e.google.com (mail-wm0-x22e.google.com. [2a00:1450:400c:c09::22e]) by mx.google.com with ESMTPS id z5si12732152wmg.38.2016.02.19.05.17.20 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 19 Feb 2016 05:17:20 -0800 (PST) Received-SPF: pass (google.com: domain of eric.auger@linaro.org designates 2a00:1450:400c:c09::22e as permitted sender) client-ip=2a00:1450:400c:c09::22e; Authentication-Results: mx.google.com; spf=pass (google.com: domain of eric.auger@linaro.org designates 2a00:1450:400c:c09::22e as permitted sender) smtp.mailfrom=eric.auger@linaro.org; dkim=pass header.i=@linaro.org Received: by mail-wm0-x22e.google.com with SMTP id b205so67781666wmb.1 for ; Fri, 19 Feb 2016 05:17:20 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=5BgbfLAp2lwH3W2ZHI6QvOGoj1A043G8JzOi5p5EehY=; b=MDpucVNowxQbTXS5xCxmhPLS9N/WevwpAMJdte6nybzyzOe8YVSKnvlF8CJlIoiGGg Yo0sRfOSP/eq0BClXiP4Ly0kwXdUvVPkNCsuliN3zMMKg8zM3sFRV7CskOV5X2AHXTXU a7zjHcZX2VaT6ufoJ1xXu+SMY/kTXNHTgSNJY= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=5BgbfLAp2lwH3W2ZHI6QvOGoj1A043G8JzOi5p5EehY=; b=cSaXt4oODy7ND2PU8dtFnDO0QBGX/L3ccIZWejia0QRz336obGRVn7FeyLeEi5DsqV tw9llH4R5CLUoKb2UfNto071iRI7dwQTVDOssOy112Ne+CjnW9JALHI3YKE9/Vgvq+Bg l6Bqr5BPNXkSlXZi9FefOWP2cJ7Kos6fb2G7vJBj0rm8Vrv2TdFk69BdRT5+BFWZiBAj MiCGBU73+SYpRjWPQ4pa3PGUCAJ/dMR+r14AqESpacGY96/FCKfC8C1B2ojlPV/yIWQc R7AXz0cs45FLxMp5zaC5eapDNcdw0q7rRyBSpjJKIWJ7tj28tleCk6ayvp7XLjfIlYf1 stRg== X-Gm-Message-State: AG10YORs9wgFTtzVjG83K+fbq5XiTbGoLaDWr1crIPTW4ZNE3cgmrBjJZfEx65znoPixL04SNcc= X-Received: by 10.28.132.17 with SMTP id g17mr9654357wmd.63.1455887840168; Fri, 19 Feb 2016 05:17:20 -0800 (PST) Return-Path: Received: from midway01-04-00.lavalab ([81.128.185.50]) by smtp.gmail.com with ESMTPSA id bg1sm11229846wjc.27.2016.02.19.05.17.18 (version=TLS1_1 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Fri, 19 Feb 2016 05:17:18 -0800 (PST) From: Eric Auger To: eric.auger@st.com, eric.auger@linaro.org, qemu-devel@nongnu.org, qemu-arm@nongnu.org, peter.maydell@linaro.org, alex.williamson@redhat.com, david@gibson.dropbear.id.au Cc: christoffer.dall@linaro.org, suravee.suthikulpanit@amd.com, alex.bennee@linaro.org, thuth@redhat.com, crosthwaitepeter@gmail.com, patches@linaro.org, pbonzini@redhat.com, b.reynal@virtualopensystems.com, thomas.lendacky@amd.com Subject: [PATCH v8 0/8] AMD XGBE KVM platform passthrough Date: Fri, 19 Feb 2016 13:17:05 +0000 Message-Id: <1455887833-7306-1-git-send-email-eric.auger@linaro.org> X-Mailer: git-send-email 1.8.3.2 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-TUID: I6AxYBE/K94M This series allows to set up AMD XGBE passthrough. This was tested on AMD Seattle. The first upstreamed device supporting KVM platform passthrough was the Calxeda Midway XGMAC. Compared to this latter, the XGBE XGMAC exposes a much more complex device tree node. - First There are 2 device tree node formats: one where XGBE and PHY are described in separate nodes and another one that combines both description in a single node (only supported by 4.2 onwards kernels). Only the combined description is supported for passthrough, meaning the host must be >= 4.2 and must feature a device tree with a combined description. The guest will also be exposed with a combined description, meaning only >= 4.2 guest are supported. It is not planned to support separate node representation since assignment of the PHY is less straigtforward. - the XGMAC/PHY node depends on 2 clock nodes (DMA and PTP). The code checks those clocks are fixed to make sure they cannot be switched off at some point after the native driver gets unbound. - there are many property values to populate on guest side. Most of them cannot be hardcoded. That series implements host device tree blob extraction from the host /proc/device-tree (inspired from dtc implementation) and retrieve host property values to populate guest dtb. - the case where the host uses ACPI is not yet covered since there is no usable ACPI description for this HW yet. The patches can be found at https://git.linaro.org/people/eric.auger/qemu.git/shortlog/refs/heads/v2.5.0-xgbe-v8 Previous versions can be found at https://git.linaro.org/people/eric.auger/qemu.git/shortlog/refs/heads/v2.5.0-xgbe-v HISTORY: v7 -> v8: - don't use g_slist_free_full introduced in 2.28 v6 -> v7: - rebase on "all: Clean up include" - add Peter's Reviewed-by v5 -> v6: - main functional modification is in qemu_fdt_node_path where I free all allocated resources in case of blob parsing failure and return NULL - fix error handling in sysfs_to_dt_name and take into account possible NULL returned value from qemu_fdt_node_path - remove any reference to self-asserts - fix style/spelling issues reported by Peter - add Peter's R-b on "hw/arm/sysbus-fdt: helpers for clock node generation" v4 -> v5: - add Peter's R-b on qemu_fdt_getprop patches + remove comment on error_fatal - renamed inherit_properties* into copied_properties - qemu_fdt_node_path now returns a char **: it supports the case where several nodes match the name/compat - add_amd_xgbe_fdt_node now checks a single node is found v3 -> v4: - explicitly return 0 in node creation functions v2 -> v3: - added "device_tree: qemu_fdt_getprop_cell converted to use the error API" v1 -> v2: - take into account Peter's comments: - add CONFIG_LINUX protection - improve error handling and messages - no fixed size buffer anymore - fix read_fstree error handling - use /proc/device-tree symlink instead of /sys/firmware/devicetree/base - added hw/arm/sysbus-fdt: remove qemu_fdt_setprop returned value check - see individual commits for full details RFC -> v1: - no dependency anymore on dtc binary: load_device_tree_from_sysfs is self-contained and implements something similar to dtc read_fstree. - take into account Alex' comments - remove qemu_fdt_getprop_optional and use error API instead Best Regards Eric Eric Auger (8): hw/vfio/platform: amd-xgbe device device_tree: introduce load_device_tree_from_sysfs device_tree: introduce qemu_fdt_node_path device_tree: qemu_fdt_getprop converted to use the error API device_tree: qemu_fdt_getprop_cell converted to use the error API hw/arm/sysbus-fdt: helpers for clock node generation hw/arm/sysbus-fdt: enable amd-xgbe dynamic instantiation hw/arm/sysbus-fdt: remove qemu_fdt_setprop returned value check device_tree.c | 182 +++++++++++++++++++++-- hw/arm/boot.c | 6 +- hw/arm/sysbus-fdt.c | 319 ++++++++++++++++++++++++++++++++++++++-- hw/arm/vexpress.c | 6 +- hw/vfio/Makefile.objs | 1 + hw/vfio/amd-xgbe.c | 55 +++++++ include/hw/vfio/vfio-amd-xgbe.h | 51 +++++++ include/sysemu/device_tree.h | 53 ++++++- 8 files changed, 644 insertions(+), 29 deletions(-) create mode 100644 hw/vfio/amd-xgbe.c create mode 100644 include/hw/vfio/vfio-amd-xgbe.h -- 1.8.3.2