[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v7 00/12] PCI devices passthrough on Arm, part 3


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
  • Date: Wed, 21 Jun 2023 22:11:32 +0000
  • Accept-language: en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=epam.com; dmarc=pass action=none header.from=epam.com; dkim=pass header.d=epam.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=A6NkDdED+KCPwQTnWsRpLa6snzAHE0PLji6rM/7j8oA=; b=H4PMKImk1Vk5WGZVYM6lMrSC7uP0EBfewkqsfUk9wCZsl8WheDJWdPNdfky0UeWByC06ZNtiTDx1OegWTC81eN9bLOwtecOPw+4sE16sJsDWU+JCiiWrZtI7AwgcODOyvTJC6WXcIeIKbDhe8OIARYFNmn/KrMuT/2+dx9QIFolEHg1pPDCWppcp2mTiM031NgmWGSPNbFSr2O5ZyBcY/2ZFXBAlLHNWBUBFP7h+HV07kB3Gcbgg1y/M335ym8jHKS62FuBUBFAcBfBf7fOR/+5J+/VRVPUVXn/gxlEnrPNN+Vdy9dY6e9KN1D8f+Ckce1OtcbwOwkes90yPDAC/yw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=iRShdsX+nLVa31+9OcEPtbiB5Qrpe//Mc5CWczfXXKv/R8KxgfcL+PWU4iez7Oa3D8PVq5vX3IDER16UdHkyS1tFxPjjW3CGnq2gVj0vYD5Rjf2lN7+j6jUY3jTU7xgwlhpnZ0Vu6L+CNBtgh7PCV36YA9GvXQ8FNl5X4P6EkQHeEBV6/CmlzfLhZSIxbAboxGtD9ghGrsqJOqo8ogAgnLIaT9S/6vp0hiOGFWLMearLxfYmrBZ1RrH9Dho1zf0JeeQu61hnt8seGgBxWz0o+SeQqaeaq4RDIwailY6YuV7hliigr5lKADJfi+H6VDODV8/wJH7oJFHiHTQiubKzNQ==
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Paul Durrant <paul@xxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>, Stewart Hildebrand <stewart.hildebrand@xxxxxxx>
  • Delivery-date: Wed, 21 Jun 2023 22:13:40 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHZneJYJeyWMREaaUSf/027X0QuCq+LHuyAgAB+64CAACzJAIAKEyeA
  • Thread-topic: [PATCH v7 00/12] PCI devices passthrough on Arm, part 3

Hi Jan,

Jan Beulich <jbeulich@xxxxxxxx> writes:

> On 15.06.2023 11:39, Volodymyr Babchuk wrote:
>> Stewart Hildebrand <stewart.hildebrand@xxxxxxx> writes:
>>> On 6/13/23 06:32, Volodymyr Babchuk wrote:
>>>> Hello,
>>>>
>>>> This is another another version of vPCI rework (previous one can be
>>>> found at [1]). The biggest change is how vPCI locking is done. This
>>>> series uses per-domain vPCI rwlock.
>>>>
>>>> Note that this series does not include my work on reference counting
>>>> for PCI devices because this counting does not resolve isses we are
>>>> having for vPCI. While it is (maybe) nice to have PCI refcounting, it
>>>> does not moves us towards PCI on ARM.
>>>>
>>>>
>>>> [1]
>>>> https://urldefense.com/v3/__https://lore.kernel.org/all/20220204063459.680961-1-andr2000@xxxxxxxxx/__;!!GF_29dbcQIUBPA!0BUqPos1zFKUoPwbKLLwKItNgBVPaBgxmH1Y6zXpms2bngrlWrzB-qMNvIaiAy2WSWMa93UrlvRi0ijYP8X4Ymx07GXYPO1W$
>>>> [lore[.]kernel[.]org]
>>>
>>> Thanks for sending this!
>>>
>>> Should this be v8? I see v7 at [2].
>> 
>> Oops, my bad. 
>> 
>>> I had to rewind my xen.git back to 67c28bfc5245 for this series to apply 
>>> cleanly (just before ee045f3a4a6d "vpci/header: cope with devices not 
>>> having vpci allocated").
>> 
>> I rebased this series onto staging about two weeks ago. Looks like
>> there was new changes into the PCI code after that.
>> 
>> Should I send a new, real v8 which is rebased onto current staging, or
>> we'll wait for review for the current set of patches?
>
> Please send a version which, at least at the time of posting, actually
> applies. Taking into account Stewart's observation on the version
> number makes it even more desirable to have a re-post.

I am terribly sorry about version mishmash. But Roger made valuable
comments for the first patch already.

So I'll post the updated version with an additional lock and other
fixes. Should it be v8 or v9 in that case?

-- 
WBR, Volodymyr


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.