From mboxrd@z Thu Jan 1 00:00:00 1970 Received: by 10.25.159.19 with SMTP id i19csp69005lfe; Mon, 1 Feb 2016 05:52:02 -0800 (PST) X-Received: by 10.28.20.83 with SMTP id 80mr12688785wmu.89.1454334722016; Mon, 01 Feb 2016 05:52:02 -0800 (PST) Return-Path: Received: from mail-wm0-x22a.google.com (mail-wm0-x22a.google.com. [2a00:1450:400c:c09::22a]) by mx.google.com with ESMTPS id 190si15093275wmt.70.2016.02.01.05.52.01 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 01 Feb 2016 05:52:01 -0800 (PST) Received-SPF: pass (google.com: domain of eric.auger@linaro.org designates 2a00:1450:400c:c09::22a as permitted sender) client-ip=2a00:1450:400c:c09::22a; Authentication-Results: mx.google.com; spf=pass (google.com: domain of eric.auger@linaro.org designates 2a00:1450:400c:c09::22a as permitted sender) smtp.mailfrom=eric.auger@linaro.org; dkim=pass header.i=@linaro.org Received: by mail-wm0-x22a.google.com with SMTP id l66so71266454wml.0 for ; Mon, 01 Feb 2016 05:52:01 -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-type :content-transfer-encoding; bh=U/JzPXN1JkpmwAug/45+uewTYqmbJl3lCBS17jpuRNU=; b=V0SQMsO+/dy8HqJ28IhQ5n1gkOHv6EOoCop9MfkJudi0M7DM6ijbK14sNwGwhcTYOq yz79MSie2f1VnR4XIAUhAI9SR4GJefoUpJTP/pHfhss0KHkEbHZomGy7ZxdoeXB+BOa6 9G63hmZyyBXTcP2Td8WjfViuKLvFPtPEBmn0Q= 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-type:content-transfer-encoding; bh=U/JzPXN1JkpmwAug/45+uewTYqmbJl3lCBS17jpuRNU=; b=J/+/h7zsHHBD7yl25bNpcw3w2cGPxessShDPh56ySCHCRnMfvhoj/mY9W/rva+95UO EmmlzCwavbQA17oNDvxQ36XONcjbJcM3ZBwziJ/+h+3Sw8bBcW0Ge2O5X8UmoLcoYj3k 4JjVZFOtYe/Re4dWyx6E6CAN1bqeUlePhJ24+zFlD36/F6nUmgQni+aiCjiIud4A+40D arxFImJYNfyKsi0jm8IrWebdiOwBAcyE/g717qMq94f7QRr9vQ5hFKkdIY0K8vNWm2jn r55lTFf96RpHQSdS/CLPl8nl7wDYtdgnCrPrnGz/QGjoK4YDexHsy9Is9GN2XMerke7H 0Pdg== X-Gm-Message-State: AG10YOT7OtkzbRBO/XYdgWOIxH7cOmxF48Tud5GGbMZ9Q7wGxxxX1ap80jlt72JmkWZY9aHrKuI= X-Received: by 10.28.61.138 with SMTP id k132mr1144327wma.36.1454334721804; Mon, 01 Feb 2016 05:52:01 -0800 (PST) Return-Path: Received: from new-host-13.home (LMontsouris-657-1-37-90.w80-11.abo.wanadoo.fr. [80.11.198.90]) by smtp.gmail.com with ESMTPSA id lw7sm7925488wjb.19.2016.02.01.05.51.58 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Mon, 01 Feb 2016 05:52:00 -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 v6 0/8] AMD XGBE KVM platform passthrough Date: Mon, 1 Feb 2016 13:51:44 +0000 Message-Id: <1454334712-12360-1-git-send-email-eric.auger@linaro.org> X-Mailer: git-send-email 1.9.1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-TUID: PrFmU30tXWP9 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-v6 Previous versions can be found at https://git.linaro.org/people/eric.auger/qemu.git/shortlog/refs/heads/v2.5.0-xgbe-v HISTORY: 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 | 178 ++++++++++++++++++++-- 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, 640 insertions(+), 29 deletions(-) create mode 100644 hw/vfio/amd-xgbe.c create mode 100644 include/hw/vfio/vfio-amd-xgbe.h -- 1.9.1