Received: by 2002:a05:6a10:8c0a:0:0:0:0 with SMTP id go10csp1788815pxb; Mon, 8 Mar 2021 06:30:03 -0800 (PST) X-Google-Smtp-Source: ABdhPJzV30b6gUUIXvZXrFeWpNeceZQmghiZOp8IS15iJxcXceICqLCuLtQveF9XdUN8YErfsIFu X-Received: by 2002:a17:906:dfcc:: with SMTP id jt12mr15426723ejc.31.1615213803722; Mon, 08 Mar 2021 06:30:03 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1615213803; cv=none; d=google.com; s=arc-20160816; b=qx4IWwPCILJ8tx7gIWA8sJCM++uA+eIbkYINUDnJiRY9DhANf5ETGwYCmxOkxqWPcY zytLxn2havSp3841HNZw+DfTkgxVH5wLgZhPvI8aI4HCAACGM4fXL/A+3bdP45OOfc6Q PzgvpcLXo0ZMLIfCgtbTVB1Wz33+uXz0aLGNS5znBXoSxVHVq6w48LJNv6mIMk8yuuDL pWyLVns3i9cD0ZHilS0qNA830imVRSx822hSbt5qE1c8evyOpgHy+60riLWSSdBDmGvq 5Zv3VwnOSBT4dNAjKbnNXNBI8nQtMlCmCXt9GN+e7RHfhhz38wpTOed4waN9JdDq7iQ9 RdNg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject:dkim-signature; bh=5Tx7iM5/yDvqIEFiuaAtchQiAX0fZCuVvfKdSztb6qc=; b=KQlF1X09pgDZj8tKNfKv4UK2shGDJhw3ymbxbPF9cQdZjJCxlt9MCB827ocawqI+8R mTjOTXpfOqMqyGXvS8LjZWfpEartM2q1V2F0ENinVzMJMDnZyhF4H39K9e6PkeLkVPI6 ob9INYGps0c2AOQRsFxAzj/xVKlk5J5IbPh78ktqG7vyChucQYyYnBnv+Uo9hIsqAOPG nXX6W64b+LMtHQEey6IuKjhvDqHO+1/IR42bh3rpZoXXIa3IE/cNb4feJxIL/fCtpN79 1nOcN9Iyk5MPYjgwxtlxrZgsgwrc01/osOERXr9TCWZh8MFEOoT2hjb9OMiDyvkRiHvZ 6I2A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=TDYBDrBC; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id w2si6736808edr.159.2021.03.08.06.29.19; Mon, 08 Mar 2021 06:30:03 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=TDYBDrBC; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229690AbhCHO2P (ORCPT + 99 others); Mon, 8 Mar 2021 09:28:15 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56626 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229528AbhCHO2L (ORCPT ); Mon, 8 Mar 2021 09:28:11 -0500 Received: from mail-lj1-x230.google.com (mail-lj1-x230.google.com [IPv6:2a00:1450:4864:20::230]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4B6BFC06174A; Mon, 8 Mar 2021 06:28:11 -0800 (PST) Received: by mail-lj1-x230.google.com with SMTP id m11so16279539lji.10; Mon, 08 Mar 2021 06:28:11 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=5Tx7iM5/yDvqIEFiuaAtchQiAX0fZCuVvfKdSztb6qc=; b=TDYBDrBCriR9cjYkY3F4y+0jYt0cu6kj/EMJye/kwXLUZ3arB3XzCW43yePDA4YfYc tB4d7JKketl6MELigit3fT+egHHKb/F+tOUoTiGVz7y83lh+QzwlZM0aD6Jed4dlqI6g 7nSO04IbNA9ZnOreHWEa4JY3WdS8DOHrSi2KJ/V7Gf7YNH6yzFpNmCs5ooikoWCcu+WP OCcE6RHv9ClT/Xy/lENFD+nv5M7A47+WrA47hvedADDA8QeIlj+6hWzLnLCyAGgicYlW 8G6a9Gz1tEwqIC+ryDCFcvkRsm3LppD15ynL9Z2rhTEsvd+8bqez8o8AY8EVSMVFnkNJ Cayw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=5Tx7iM5/yDvqIEFiuaAtchQiAX0fZCuVvfKdSztb6qc=; b=O2S5bJwydobGcv3a5PjSXHNbtqUvMdDEWxfdSD8PiVWtYiUpDGbA9sfFR2DvBk2073 Bbg73wSIM4vNruQfKdlg5f9fnlE7b/aObtWlibatiJSbPUBCFo/w99Mpr02dzQ6pqbOb N1/YCnXrpxBVaV1OtUiDCn/vFb5QOw1SgEIBedEmdi7vCK61VuxFLW2W/L5JiU03kP8D iuAYN/RzDXVySX9CTYEgfqa+sOscOixg7rWzurrZ7wHlz5Kq0xS8dju/Lm/f76oMwGoA GrdYbCCJGiWyu2sZfZxYa0bJYbsrhryYu39UcXOyo8dH2GYqrj3MPoKDaasd+YtP9JUl 6OzA== X-Gm-Message-State: AOAM532E649GGzSmlJV0W41MKpmL9EqQmsTEnuXxF3Aj4Q6E0XA9i0eg xSHXwRLlX1equiouP/rRhMs= X-Received: by 2002:a2e:b55a:: with SMTP id a26mr12150911ljn.297.1615213689870; Mon, 08 Mar 2021 06:28:09 -0800 (PST) Received: from ?IPv6:2001:14ba:efe:51f0:513f:8184:2b7b:4a52? (db7-gd8cvj0y4km77llly-3.rev.dnainternet.fi. [2001:14ba:efe:51f0:513f:8184:2b7b:4a52]) by smtp.gmail.com with ESMTPSA id d8sm1420576lfm.160.2021.03.08.06.28.08 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 08 Mar 2021 06:28:09 -0800 (PST) Subject: Re: [PATCH v2] PCI: quirk for preventing bus reset on TI C667X To: =?UTF-8?Q?Krzysztof_Wilczy=c5=84ski?= Cc: helgaas@kernel.org, alex.williamson@redhat.com, bhelgaas@google.com, kishon@ti.com, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, m-karicheri2@ti.com References: <20210217211817.GA914074@bjorn-Precision-5520> <20210228135311.668-1-antti.jarvinen@gmail.com> From: =?UTF-8?Q?Antti_J=c3=a4rvinen?= Message-ID: <461e12f3-fb0a-9615-09c6-91efcfd8aaa9@gmail.com> Date: Mon, 8 Mar 2021 16:28:09 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.6.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 7.3.2021 2.22, Krzysztof WilczyƄski wrote: > Hi Antti, > > A few nitpicks, so feel free to ignore these, of course. > > If possible, capitalise the subject line. Also, perhaps "Add quirk to > prevent bus (...)" might read better. > >> Some TI keystone C667X devices do no support bus/hot reset. Its PCIESS > [...] > > It would be KeyStone in the above sentence. > > [...] >> With this change device can be assigned to VMs with VFIO, but it will >> leak state between VMs. > > Following-up on Bojrn's question about the state leak, see: > https://lore.kernel.org/linux-pci/20210129234946.GA124037@bjorn-Precision-5520/ > > Would there be anything else that has to be done? > To my understanding this is all that needs to be done. At least on other similar case, adding quirk was the only change https://lore.kernel.org/patchwork/patch/1086083/