Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <11fede06-0481-511a-f9d0-7393339b05d1@intel.com>
Date:   Mon, 5 Dec 2022 09:19:46 -0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Firefox/102.0 Thunderbird/102.5.1
Subject: Re: [patch 33/33] irqchip: Add IDXD Interrupt Message Store driver
To:     Thomas Gleixner <tglx@linutronix.de>,
        LKML <linux-kernel@vger.kernel.org>
CC:     <x86@kernel.org>, Joerg Roedel <joro@8bytes.org>,
        Will Deacon <will@kernel.org>, <linux-pci@vger.kernel.org>,
        Bjorn Helgaas <bhelgaas@google.com>,
        Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
        "Marc Zyngier" <maz@kernel.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Jason Gunthorpe <jgg@mellanox.com>,
        Dave Jiang <dave.jiang@intel.com>,
        "Alex Williamson" <alex.williamson@redhat.com>,
        Kevin Tian <kevin.tian@intel.com>,
        Dan Williams <dan.j.williams@intel.com>,
        Logan Gunthorpe <logang@deltatee.com>,
        Ashok Raj <ashok.raj@intel.com>, Jon Mason <jdmason@kudzu.us>,
        Allen Hubbe <allenbh@gmail.com>,
        "Ahmed S. Darwish" <darwi@linutronix.de>
References: <20221111133158.196269823@linutronix.de>
 <20221111135207.141746268@linutronix.de>
 <4a15c569-0545-20ac-e74c-ae17f7eb067d@intel.com> <87sfhxoa7i.ffs@tglx>
 <87v8mpg9mo.ffs@tglx>
Content-Language: en-US
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <87v8mpg9mo.ffs@tglx>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Ym5Zb3pBMXdCUm1vQ1M1OEp2QllUMFRDTi9ZS2pJNTVGam9FRmsweld0bnFP?=
 =?utf-8?B?Y2lEa3lucXd3bG9YanV0VzdKZ3JjWFNDMThRYW1rdUhrNlVvWVVYUnFRTUQ5?=
 =?utf-8?B?NVRNOHZ5aWhBRVZpTWhibFJyeUE3N3FSb0piMEJpN05vVlZhUmI2ODdtbG1F?=
 =?utf-8?B?VElrdVViVWxIYllSYTVoc3Uxd0J3NWZWdmhGSTdabU9MVWtQWFplOHc4TjNp?=
 =?utf-8?B?OFhIRDRac05KaVFYamRMVzR6aWdJMUxvUlEzdkNWUkdzUGVXbzZuM1Z6akFn?=
 =?utf-8?B?M2tUMXBJMDdZQTNmOFNLNDNBMW5tUm9VeUVPU3VpMThXbU9xNGQ5Y2UvM0xa?=
 =?utf-8?B?aDNuOUNGWnpISGhWZ1ZlazhBbUdnMm1IU1lEU2g5NXZ2QkIyKzBVTGo1S1hY?=
 =?utf-8?B?OEtvR1AyMlA0dkNsd2lWdmVVYVZMcUVxdDFvZVlrTWZXSDYzMjg5UXhjL3BC?=
 =?utf-8?B?bFpLNXh4WDF1UWhScDRwYU1WaG5qK0IwTTRuTWs1L1ZzdTREbW9iV2EvdHht?=
 =?utf-8?B?S1JxSjJ5ZXNTTFdwSW5MQ2FVYzZMcjMwWW5ubTEwbzFacVdpZ2h2QVJoVHdF?=
 =?utf-8?B?REplUUkwa2lVZlVmb09IUFM0ZDREbWJLbkZWUEJKYmlZZlozRmxPaGJxdVlo?=
 =?utf-8?B?MFFNdUUvQXkwWUtvODBhaEJVdEdHeWdOcUVLQVY1STFkNitSUEtZaEhUNmR3?=
 =?utf-8?B?WnBqdnY5R2FOY3BXUjBEVkxZaUZwRGx6bjMxZHBjd2NRdzJvQWIwanVxanBr?=
 =?utf-8?B?V1NKUU1YMjBuNTZRZzY4RWxITzhGSlBaazZsVXlXOUZJM1NHbmRsVU9xRzIw?=
 =?utf-8?B?TjRITWFVVHArOTREWndqcnRadHVtOWRFTXJMZ0VGa21idVp5M2JzdlV2QWZ6?=
 =?utf-8?B?eWVnUTMreEpBa2l3VkdDRWxPMWtOQTJpSzIyRXlPMmFjYTYzMGhoNEI2WU9k?=
 =?utf-8?B?VkdCTE1XZEF6a1pvckptT3lpQzIrVDlHeFAxcFdWOFd4RG5UNTkySnc2c3Zs?=
 =?utf-8?B?aWRHWWhTUlN3UnFXRFBvKzV3QTlsYlgyY2VQSDgwRm81TkNsMDU4bWJIK3Yx?=
 =?utf-8?B?elVRSFJMdE1rOWNId1pHb3VzeDNFb3Q0ZFFMRmtKaE8wSUVtcTJFT0VyNWR5?=
 =?utf-8?B?bCtEcU9EUmExamdXTlR1cGVPQWgweStUZGtUWVpqM2x3MklSUDVIOHRpM3Az?=
 =?utf-8?B?SWFRRmJxWDdEak1QQTJndnJKMXFUckhUS3VVVnhpdXFmRXljcGtnUEtYdDlx?=
 =?utf-8?B?cnZXWGJQTVNnSm8yU2VtcTIxdjhmT1RTcFpMVzMvR1ZtbTY2dkxQR2dQMEVG?=
 =?utf-8?B?Q0ttU1VQaW9lV3dQcy9RY2JCNDdzRk5SbHVCS1ZoclI0eUt5aWFlaVpRVXMz?=
 =?utf-8?B?Z1hjMU1oQ0hJQjRVaDRsRXZqdjhhVWgyM0pjb05ja25qYVFEbk56R3c5cW9r?=
 =?utf-8?B?SmZRNlYwRFBXUld4Wk8wdmZBblE4QnpPNU1XamUyZUY1akV4emNjOW8zOWtt?=
 =?utf-8?B?c0pPaGl6ZlpDY1c1Z2FRSmV2eFRjR213RGR4OTB0M1JQMWdEOFJxd2E2ZFQr?=
 =?utf-8?B?di93cE9uU3FpeHNSWitWMkZZeVhKdkJxQ2ltaXB5SkVMempRYzFrTDJ0QWJv?=
 =?utf-8?B?cU9ZZytxK3Zaa1dXWWhNTGsxZkJIODZBQ3p4dmtTdThXTHRVU2hzdTkvNXZM?=
 =?utf-8?B?V3VxaXJOaklNbTkxbzExTlk1YWV2dkVQZmlwUVNXUFpaZjFUSE9NQWM5OGVK?=
 =?utf-8?B?QTRucFlad2JNZUxTeFZlMWhuRHdLWVVpZ1lZMlhITThTVUtEb0RCZEFsR2xC?=
 =?utf-8?B?Tmg5VU9vTVQxZ1BPVWJsMFNmWWx2ZXlkcVJlbmVlcWVneHBJVm9CM2pkYXVN?=
 =?utf-8?B?RjBFd2xqRjd6bXRFbmpkZERWeE5uTGsxR2xvYWN0a0M5TGlyamxUQkZ5R0w5?=
 =?utf-8?B?ZzAyU3ZGbXJZdEwwMXVRSnMzMUExblRlaG1UOERnR25CeURuM3MyVWw5Qktj?=
 =?utf-8?B?R2diNFp6MFFtNG8wY003VnFOWFRZVURJQ2lzaEYxNWhsKzBrUkFFWGlTSC83?=
 =?utf-8?B?T1BEM2MySCtMZnBzY3FwWTVsZkdSWnRSdkhWTmhDelViKytMdnpucTZESkxv?=
 =?utf-8?B?anZ2TkJ0UE5xdEFTSi9kT3NhL3FySnBlZ3B5cFlsaU9mOVhvNkdJV0RHRUY1?=
 =?utf-8?B?dmc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: e99ab476-2fe4-42fa-984f-08dad6e4eaac
X-MS-Exchange-CrossTenant-AuthSource: CY4PR11MB1862.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Dec 2022 17:19:51.7537
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: vXXJUPkMwBH3fk4H4C+QKlkgKee29Pf97kzlJiz2s/2Ha+RZeoIwfRz6Q0kc9rmqf5muYPIdm0a116o9kM7WMMN8OzcC8ZRz01V7DBQVbTU=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4629
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-7.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_HI,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,
        SPF_NONE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Thomas,

On 12/5/2022 7:20 AM, Thomas Gleixner wrote:
> On Fri, Dec 02 2022 at 20:51, Thomas Gleixner wrote:
>> On Fri, Dec 02 2022 at 09:55, Reinette Chatre wrote:
>>> With the first change I am able to test IMS on the host using devmsi-v2-part3
>>> of the development branch. I did try to update to the most recent development
>>> to confirm all is well but version devmsi-v3.1-part3 behaves differently
>>> in that pci_ims_alloc_irq() returns successfully but the returned
>>> virq is 0. This triggers a problem when request_threaded_irq() runs and
>>> reports:
>>> genirq: Flags mismatch irq 0. 00000000 (idxd-portal) vs. 00015a00 (timer)
>>
>> Bah. Let me figure out what I fat-fingered there.
> 
> tag devmsi-v3.2-part3 works again.

Thank you very much.

This tag is not yet available but I can confirm that the current tip of
devmsi, 6bd4ee6cb126 ("irqchip: Add IDXD Interrupt Message Store driver"),
combined with the earlier irqchip driver delta snippet passes the "dedicated
kernel work queue using host IMS" tests.

Reinette