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: <dcef3eff-47c0-25ba-a09a-c0fe3e4404ba@amd.com>
Date:   Mon, 5 Jun 2023 20:59:02 +0530
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.11.2
Subject: Re: [RESEND PATCH] cdx: fix driver managed dma support
Content-Language: en-US
To:     Greg KH <gregkh@linuxfoundation.org>
Cc:     alex.williamson@redhat.com, jgg@ziepe.ca,
        linux-kernel@vger.kernel.org, git@amd.com, harpreet.anand@amd.com,
        pieter.jansen-van-vuuren@amd.com, nikhil.agarwal@amd.com,
        michal.simek@amd.com
References: <20230605131009.6869-1-nipun.gupta@amd.com>
 <2023060533-reseal-glider-11e7@gregkh>
From:   Nipun Gupta <nipun.gupta@amd.com>
In-Reply-To: <2023060533-reseal-glider-11e7@gregkh>
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?cURxNnRMTnBVWVhwTGJxUmsvTjkrRzFhb3NVN3BSUVArbFYrRmJkdWxUWUta?=
 =?utf-8?B?c3VEcEZpRTUzeHhJYzk2cTFWWFFTNU5zQ284azNRTXZBVlV6QXlvQzd1eWRB?=
 =?utf-8?B?U2ZFRHE2RklFZVBRZ1JNeW9ScEc2SlFUSmFFNkJHVjBoWFl6eFd2MU12bHE2?=
 =?utf-8?B?dXFXOHpUajZFcUIyQnlXQVFxNmlwcHBPZ05TemUwYUxlZVZBWUpXYitWK1lT?=
 =?utf-8?B?Q2hxbjdFUHVER1RtU3lZbURtT0grL3pFNk10cmhNa2o5MmNyb2pQTWczRWQ3?=
 =?utf-8?B?MEk5N0ZXU01UdGhQVlAzZ1RhemlYVlJ2RkdGZmU2akdTZEp5VHAyWDdKMEM2?=
 =?utf-8?B?V2JpalpPK3VhUmVFQS9wUEN6cURpOExTKzVZWGpjQ1JMdnZJYUpPV05qejhK?=
 =?utf-8?B?d1hmNDk2cFVWdVVvVzhKZXpncjU3RlEyRFVtZkxhc3l3dEF6Mmo0bWx3dnlS?=
 =?utf-8?B?ejgvdk96RE1xYm5qN05WdEl6VkRjR20wcWhTSTc0ZlBneDVUMWdGamV5Q0hj?=
 =?utf-8?B?T0dnTytiVWpyYXRIQm9xTmNTbGwyT0xac0dKL3NrbXNiRyt0Wk1xQmF3OXJh?=
 =?utf-8?B?RVgvL2xnQ3NLK2dlcUtPVU5XVGlwWTNEV2h4UitGaFI3cHNDVUtHZmtMK2hN?=
 =?utf-8?B?dmt3M3hqOHhwUWllY2JlREpvbURiNVZrZHQ5VEhPMll6eTNKanFPZlU3Nndn?=
 =?utf-8?B?Y1JBSGsyVG1zYldqd3VwRmU0RmdLeDF0Ly9ITk1ZREJjMTZsTWpYZzd2M09G?=
 =?utf-8?B?c0U3ejAwdno2YTZnVzcyRHpNdmJ6L0pvbFRjb0Z6enZlMDhTaEptcUlaSVhH?=
 =?utf-8?B?T3R5Z3lTVWFCek96dlV2dVhScklpbDlhTXVET0UwSXVQODdmU2NlUXhxcGt6?=
 =?utf-8?B?M0FSSnRIN3FndVU5UFkzQXpsSklOMXVEdCtBMlFabVVHQnhYVFNpcXY1SSt1?=
 =?utf-8?B?NkszUi9yTzZYZGNFaGJNTGk1cmVQN3UzbGk1NXJvQ1VUZXA4cDRiMXhGV084?=
 =?utf-8?B?WFk2am9IeFk3MVNSS0kzTm83SzRWaEp2SFYxTlpVUGVEelBRdXlQbEdFZlVs?=
 =?utf-8?B?MXpRWXVUZzU0MnMvNW9nRExwc01KWS82dERWVzFrclAwazhueURGa3hTaEJF?=
 =?utf-8?B?VEdsOUpNZGdZWWxYcUl5TGoxSTBXRHV6aXFPZHNnb2g5SXRZUUh0NkRTL2lK?=
 =?utf-8?B?N3VCMzFWYkhvWXQrWDN6R2JDYVlISGxjeGpxNW9HSTNIdmxaaS9NNGh4T3Rx?=
 =?utf-8?B?KzExa1V1ZEw5b05JbkpaR3FVOXVwanA5enlsNXFQNU1RamV4RGVsYVAvNVRP?=
 =?utf-8?B?b3pyb1VUN3pUd3hnRUpqYk44a1UxSGdZRFN4OWgyUm1QRVlxNTduZVBrZUNH?=
 =?utf-8?B?Mkk5R0MwaDZkdzQrTDVUL0oyNHNoV1FRZEdWQ1NrbjlrcHV1MzZMVzh1RmtH?=
 =?utf-8?B?SEZubjZBK3pncmRHVmFUYWIzM1N3UUg4aGRuZWkyYythQ3NMVWhLMi9DYnFQ?=
 =?utf-8?B?dlVFdzQ1SDhZbHVnUWlnWlVudVhDVFRKWFV3UXJJK2hNU3FSV2R1MFcxUDVi?=
 =?utf-8?B?bUZtNUpwSU9HN3hPeHdsdUZzczd2YlNuckR4cit4aUkxYVhLZmwzS2psQUJU?=
 =?utf-8?B?am9SdW9sbk80Sm5tVjBUMEh3c2MrQnFSR3pYWmIyVGl0ZnNiNGxPbWZ6ZTZ5?=
 =?utf-8?B?RUFPb2tkdDVUMUJoVWdDTTVRTCtFOFl4anF1VnZQdlNPemhTc2xkMlNHc29D?=
 =?utf-8?B?bG9hUFNvNVBmbXp3VTBNbWIrdHRkK0lHVFVrdkluZldPUVB6UEQxK2hSdEt6?=
 =?utf-8?B?YkxYbWF4aXh5WTdGb2VUR2dYMEdCNjFaZzJLcmFycTd2N2ROWnVXdW1zTnJa?=
 =?utf-8?B?SmhQZVJnKzliYWFwREpwQnNGZGI0Y3NRQ284dFMvelFsNnM2K1RzOFloVmRN?=
 =?utf-8?B?QjVVODQwMFpXcGE2MkVuZ0lpRjBtL1ZIU1lGUHRXaEhLc1lRdEw0cll1dmw1?=
 =?utf-8?B?UGRENGsrN01nVnpuaENQWDlzMmR0WERIQmpTNlNBcmordXUySzVIUnAxZ3VB?=
 =?utf-8?B?SGRHOXY1OTl5TWpKTm5rVnYxS1BXU0drbjBqdG1GVCs0OXdYSmo4b0lvOEx6?=
 =?utf-8?Q?K0G8=3D?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6a360a2b-e134-4af2-969d-08db65d99ea0
X-MS-Exchange-CrossTenant-AuthSource: CH3PR12MB8308.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Jun 2023 15:29:14.8427
 (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: 7IbpRidD2cGIICdWyVeJyNI4JVU9l3PCzZ8LmKpizLA18Ne6KPGuB3fRr3fT8G77
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR12MB7016
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/5/2023 8:52 PM, Greg KH wrote:
> 
> On Mon, Jun 05, 2023 at 06:40:09PM +0530, Nipun Gupta wrote:
>> The devices on cdx could be bound to drivers with the device
>> DMA managed by kernel drivers or user-space applications.
>> As multiple devices can be placed in the same IOMMU group, the
>> DMA on these devices must either be entirely under kernel control
>> or userspace control. Fix the CDX bus driver to acknowlege the
>> driver_managed_dma flag and call the appropriate iommu APIs.
>>
>> Fixes: 2959ab247061 ("cdx: add the cdx bus driver")
>> Signed-off-by: Nipun Gupta <nipun.gupta@amd.com>
>> Reported-by: Alex Williamson <alex.williamson@redhat.com>
>> Closes: https://lore.kernel.org/lkml/20230524134831.28dc97e2.alex.williamson@redhat.com/
>> ---
>>
>> I have resend this patch to include fixes, closes and
>> reported by tags. We request that this fix be incorporated
>> into the 6.4 release, along with the inclusion of CDX bus
>> driver support.
> 
> What do you mean by "inclusion of CDX bus driver support"?  What patches
> exactly?

Sorry for the confusion. I mean that CDX bus support would be coming up 
in 6.4 release and this patch (cdx: fix driver managed dma support) is a 
fix for the driver managed dma support in the bus. So if this patch can 
be part of 6.4 release (i.e. added in upcoming rc release/s for 6.4)?

Thanks,
Nipun

> 
> confused,
> 
> greg k-h