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: <2deec417-b1f6-5831-23f2-91fda2c802e5@amd.com>
Date:   Mon, 12 Jun 2023 12:31:37 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.11.0
Subject: Re: [PATCH v2 0/2] PCI: pciehp: Add support for native AER and DPC
 handling on async remove
Content-Language: en-US
To:     Bjorn Helgaas <helgaas@kernel.org>
Cc:     linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org,
        Bjorn Helgaas <bhelgaas@google.com>, oohall@gmail.com,
        Mahesh J Salgaonkar <mahesh@linux.ibm.com>,
        Lukas Wunner <lukas@wunner.de>,
        Kuppuswamy Sathyanarayanan 
        <sathyanarayanan.kuppuswamy@linux.intel.com>,
        Yazen Ghannam <yazen.ghannam@amd.com>,
        Fontenot Nathan <Nathan.Fontenot@amd.com>
References: <20230612175457.GA1342613@bhelgaas>
From:   Smita Koralahalli <Smita.KoralahalliChannabasappa@amd.com>
In-Reply-To: <20230612175457.GA1342613@bhelgaas>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?enFvb3kweWY4ZXUwNEQwbHdodmh0ekE1eG9PTTc0K3ovVU9lZ05VWUl0SElh?=
 =?utf-8?B?Y2VBNTBQUnR6bnhCSE55Zk5tMXdRd244R0NtSUplRDhiKzdnUHF3K0d1T2Fq?=
 =?utf-8?B?Vk9ucmRzNklnRHdpS0Q0TnEvdGV5cTlaYzRhcEtxRXB2aXBEQzlQeFV2WHlQ?=
 =?utf-8?B?bU15d1FwMXphSFNYL3BCaWhsYUtlbGN1T2VvNWlnenI4MitGdUFaOW9jZFNC?=
 =?utf-8?B?aDZBakM1U0Y0Wmc5aU9Ba05iMmZJQUdvNUVvMDhXUFNWNHVLZnIzWXdhc1Ns?=
 =?utf-8?B?TFE4UDcwdG5ITW5heW5iaUlpWE5KcjhDSThLSEV2Z3JqaFhpNVplVHlOVnZp?=
 =?utf-8?B?bzNaQ0FLWEdaWDZKUzNjRURFdWVCZkhZWGs3N3k1eENxRmdxN1JCWUhtejZa?=
 =?utf-8?B?MGIwdkdGRjNNMnJWQ2NGRThEdFFiTzdnRmUyWkdXMHBoTi9wcmVoUjVYc2Jw?=
 =?utf-8?B?WmVOWERVdDJTQmRrM0w3ZmpXcjZuTmtlWHMrMXF5RWtQWXF2REtLL2pjb0xY?=
 =?utf-8?B?SG1SVTlUdWh5TFEzMVNXL0FMaHc1WjlBQjN2eDZvaGNrTUV3b3c3YlRxQVNV?=
 =?utf-8?B?dkJIQWFiTUQzclhyd2JVNks5aUJ4Y2tOUTNnd2lDZUZsU1lJaVBLaExNNFVx?=
 =?utf-8?B?RE9zTE1TQk12K1JvME1ZcTFhN1V1SHNwK2hOY2UrcXd4bE9JU2tUckpJeUJo?=
 =?utf-8?B?S1lPSXJmWVV3WnAwWWY3OXhLeWFxNHJveEJqM2gyVC9GV3A3SUQ2WW5DZlhz?=
 =?utf-8?B?R3RWZnZadjduaEluWWR5c1VEYnJIWVFwaUZOTmF4cnVBQzJwSHFqWlA5Y0t3?=
 =?utf-8?B?TnAvOWVWRzFFem9UNjJiblZLRDVqd01tWTJhV01lVkpwVFdLUVBNeHY2aloz?=
 =?utf-8?B?R05NV21ENU1VNVR2a0tpMU1GY1VNRmlCbUZpQ1JTZG1OajVBc2VDNDRKeG8x?=
 =?utf-8?B?S1hDNmtYYS9jTmkxdFhvVll6TUExQzRyVlNDOWxXQldhbUJWQW93a2xUR1Ro?=
 =?utf-8?B?T3NTRTFiU2s1YUZDZWdHcW9weExtUzQ1VHIvb1lWM2w1cnR5L2wyTXhOTC9p?=
 =?utf-8?B?a1g0TkQzR1lxZTlpMHF6K1VDZ3VhZWl4RkRXaU1rckZKYUlta0RRZEVVMWFW?=
 =?utf-8?B?T3pPMFRaZi9iVHRRT3JKMzdvTDFYVitERWxaTmtUSjFUOU9NZDh3UkVlL0tN?=
 =?utf-8?B?dEE0QVJBa20rYWl4TE1SNDdGZnIrS0k1Tnh3QzFXRTY0bjQrdnlrZml6TDNu?=
 =?utf-8?B?OVVUdld6a0NqOTkydVR3VFRXdzJnM3hGTEVyK3RxZHBudzNyM1dTNGd2QWp2?=
 =?utf-8?B?cG1vczd6UWFMU2kzVDEwNTE5MzJKckNjMStMMDg0ejV4aHkyVlF5aVRYVDRL?=
 =?utf-8?B?YWdmSlU0Y0hvb01IeFY3SUlwWGR2SS91RVJUTkRyYWZVYUxpZHV1aTNLaTYw?=
 =?utf-8?B?ZnBmVUNPaEVoQkFrdEUxRzAzRnBnVjkwN1diUUlyR2dVOUFEM2w5N1NmVzU2?=
 =?utf-8?B?T2RvTktpVDZac1J4RjJndFUxellkaFNpSmhiRzRLOE9wMGZWUGRjbC94VDdB?=
 =?utf-8?B?dUpjeWdYUkdjaFpVQ0d6Z0c1VjQxTWRtSXhSVFVnbk92VHhUV3pZa0JSamJG?=
 =?utf-8?B?WHNmT2RrcGxOalVsSTNBd3pNMStRa1J5TzRKanhXYURkVkZZYWZuMlc2U2tO?=
 =?utf-8?B?ZklDQi9RNVFBR2RyL01BVVNjNnV2K3c0ZG1iVHdXUHNkRTNad29GTENDODAw?=
 =?utf-8?B?TnBMRDMrRkpjV01JU0taSXZqUktsU042ODd0RFdHeFJtdlh5Y2poTm5ZOXpO?=
 =?utf-8?B?MGFZbXczcVd1VWpnUUZtL2xhSVMyOElORlBJUVRvWTVLdDVSOWp0WHQyTUo4?=
 =?utf-8?B?dEJjNFZ5Uld4c1VVWHY1Sjd0ejFoQTY2NVBUSjBxZy9LTlNRRTVTaklpTytT?=
 =?utf-8?B?WkR0NDhpRmYwbWtjS2hBQWVZSjNBMXpXTkYyRVMrL0dEOGJIempCNkNzT1dE?=
 =?utf-8?B?TnZyajAwUjhJSm42aG85ZG1NZmJpbkJJVUV2Vk1RM0wzOENWT0JqKzZEUTBM?=
 =?utf-8?B?MzRFMUxzbHRtekFkRlh5bnBrelVuOE9YRjRoeDFhSlVnZmQyS0NKRDBvQ0Yx?=
 =?utf-8?B?Mlh5d1BjcW9LL2FtelI2azg3Vkk0bURHTmtnMEFpK0RLVXpXTzZ2OTBoZkVp?=
 =?utf-8?Q?Pmj41aeLyCJLksMrp6KvF5v3KYHzev71+pbCg6R9QxrH?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 7228ed78-47bc-4713-d114-08db6b7ba51a
X-MS-Exchange-CrossTenant-AuthSource: BYAPR12MB2869.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Jun 2023 19:31:39.9713
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: F4Pg3adsVafGNZgHnggMkOmpPgX7fQ0F7gEQN5XT4LC0Bka99Zsq67Y1hbBi//JmQ2ebj4Xct4FTQOER24hJOg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN9PR12MB5321
X-Spam-Status: No, score=-1.2 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,NICE_REPLY_A,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,
        T_SCC_BODY_TEXT_LINE autolearn=no 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

On 6/12/2023 10:54 AM, Bjorn Helgaas wrote:
> On Tue, May 09, 2023 at 01:58:37PM -0700, Smita Koralahalli wrote:
>> Hi,
>>
>> Could I please get review comments on my patch set? I had re-based
>> it on latest tree in v2 as I did not hear back on my questions in
>> v1.
> 
> Thanks for persevering.  It looks like there may still be some ongoing
> discussion with Lukas?  And a few of his comments that you plan to
> address?  Thanks for looking into the 10-bit tag issue.  I know those
> tag widths are non-trivial to handle correctly and it would be good to
> clean them up.
> 
> Bjorn

Yeah. Since we didn't come to a conclusion on how should we attempt 
clearing 10-bit tags and Atomic Op Requestor, I held on preparing v3.

https://lore.kernel.org/all/cc36bb5b-6a4a-258b-6707-4d019154e019@amd.com/

But I can go ahead and send v3 for just the first patch, 
https://lore.kernel.org/all/8ab986f2-6aa5-401a-aa21-e8b21f68eaad@amd.com/ and 
wait on Lukas for 10-bit tag issue.

Thanks,
Smita