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

Re: [PATCH v1 09/14] xen/arm: Add cmdline boot option "pci=on"


  • To: Julien Grall <julien@xxxxxxx>
  • From: Rahul Singh <Rahul.Singh@xxxxxxx>
  • Date: Fri, 20 Aug 2021 12:19:27 +0000
  • Accept-language: en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.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-SenderADCheck; bh=4jDM9W5p0EmNb8fTDvb8b8rIFFo/atxGhDoAPxeuvsE=; b=Hgy2tiThF3Haa2JruTszRO8FoCP/8saVQ1ZK/3Rzx/+cHz8IB4c7EkC4d+Qubp2Zqm4Mp+5UBF+daD0zjZRIJvrgzpxpagrwbBfQFFajnxfXDbGrZqEVmOwmZ55t25q9PFhQTai+5ayeYOMQbOnJdkFgYPaAc0Y58zMxxFUwTRL+SXr7cmujU3iEWND65hqMpLcaYiYwX2fnoh6W0yRWqkEZGIL0AhJOERaBWHZAR6/9cgXIe3a4F57s43cwoB8bMvyWxMcPTC6v7jZhkdQZhdzKjZImQeVpNPl/MDv+2iagGHruQKhQSoDQnOJPOda2MsU3Xnz9VgYFfJS/I4V31A==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Foq9z1k4q7asC9HodmsV9N6ydIeT8OlDuAm5lp2OUl9AfoMJfu510KqU3e366DNus6bdC7HXJt0R3ScMsP8odFdmDfIryUi1E0YYHALx4u02Xs4SOQltpqV+7+bRi54uVoMiFh47wttOKCGn7ogSyIGX1K1YbRUMke4EEgHJhwhSqByes4sbAI5nvudG4S0GjfPlaB/bjQEKM0/QaSoEIZGGNquSEZpPj/wfcCIXsrkzjrWx1qzi7g6YwheZ4knpOWx8I0lm77jGiql302WCljXvnadwU03I7xGfiMZu21vqd2UzG5CxA2pQgfcufhNmG8s/hPSsWpVYBudUDpdjRg==
  • Authentication-results-original: xen.org; dkim=none (message not signed) header.d=none;xen.org; dmarc=none action=none header.from=arm.com;
  • Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
  • Delivery-date: Fri, 20 Aug 2021 12:19:45 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: xen.org; dkim=none (message not signed) header.d=none;xen.org; dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHXlPLhtufH+Y7V4UGOnj5QL/fazat6wkEAgAGPFAA=
  • Thread-topic: [PATCH v1 09/14] xen/arm: Add cmdline boot option "pci=on"

Hi Julien,

> On 19 Aug 2021, at 1:31 pm, Julien Grall <julien@xxxxxxx> wrote:
> 
> Hi Rahul,
> 
> On 19/08/2021 13:02, Rahul Singh wrote:
>> Add cmdline boot option "pci=on" to enable/disable the PCI init during
>> boot.
> 
> I read this as "PCI" will be either disabled/enabled for the platform. 
> Whereas, I think it will be used to decide whether Xen discover PCI and PCI 
> passthrough is supported or not.

Yes. I will modify the option to "pci-passthrough== <boolean>"
> 
> Can you also clarify why a user would want to select "pci=off"?

As pci-passthrough support emulate the PCI devices for DOM0 also, I thought if 
someone want to 
boot the DOM0 without emulating the PCI device in XEN and wants to have direct 
access to device.

I am ok to drop this patch if you feel adding the option is not required at all.
 
Regards,
Rahul


 


Rackspace

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