Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Message-ID: <4947c9e9-ec52-c6be-9f81-e341ddac4e54@amd.com>
Date:   Fri, 13 May 2022 12:42:02 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.7.0
Subject: Re: [PATCH V3] dma-buf: ensure unique directory name for dmabuf stats
Content-Language: en-US
To:     Charan Teja Kalla <quic_charante@quicinc.com>,
        Greg KH <gregkh@linuxfoundation.org>
Cc:     sumit.semwal@linaro.org, hridya@google.com, daniel.vetter@ffwll.ch,
        tjmercier@google.com, linux-media@vger.kernel.org,
        dri-devel@lists.freedesktop.org, linaro-mm-sig@lists.linaro.org,
        linux-kernel@vger.kernel.org
References: <1652434689-6203-1-git-send-email-quic_charante@quicinc.com>
 <Yn4u0AG8iC33S3jO@kroah.com>
 <2a45f1e1-39d5-76b3-8fd3-c1f8b288afac@quicinc.com>
 <0071e117-1d91-b086-7cb1-976b2a1c3498@amd.com>
 <f5988884-ba0a-643c-e920-6ae626bb67a1@quicinc.com>
From:   =?UTF-8?Q?Christian_K=c3=b6nig?= <christian.koenig@amd.com>
In-Reply-To: <f5988884-ba0a-643c-e920-6ae626bb67a1@quicinc.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 2
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?dmJpejFVYmZSSlYvUDBEM3lnQU50TjErYzc3S1kwNmViSUVFcHlUWVRqT09L?=
 =?utf-8?B?ZnJocjJLNHc5WDhTb1A2UUVhZmhKK00yaVVMb2w1VkFQV2x6Um16ZERqR0p3?=
 =?utf-8?B?cnh6ZWlkSWRObnVPS01jTHNVRlhycGp2djhvelpRZU40NG1sQ1QzelJmbE9h?=
 =?utf-8?B?SnZxMkZwcDE3dnFDck1TejJGaHFlRGM3dmhQZ052dDcwODRZN2psa0tTU2Jy?=
 =?utf-8?B?Z2RhQXgwb0VNdm9YS0E0eE1FaHZqaHFxaC93NE1tSjMvemlwdkd2ak9RbWZn?=
 =?utf-8?B?SHd4emlyaTdhMFg1NEVJSDUzWW1nQ3pVeW51T0RSQWtHNkxSb3FtRGdUTlJL?=
 =?utf-8?B?SXZmQ1FhWFdHWkgrc0hRR3plT1FhS2JsWkRPaVNtYjMxd0dabSs5Q0VmZ1BO?=
 =?utf-8?B?YVFZYVA4UFJsckc3WFZTZTk4V0FDUzJ4VEYrMFNZa1hycnZnRWh3UVNZMUZ0?=
 =?utf-8?B?WGNjbnFKSHRiS2g3bXgwS2hkQURCZStTVkdVdnk2L1Vvd2VaVGsyL0ZTSzRW?=
 =?utf-8?B?cjg5eVl1aFVtaHArb251Nm9ldzNUYU9oSFRYYmhjTlQvRjFPa1IyNXdPWUFp?=
 =?utf-8?B?bDMwOUQwbWNBYUMrdEZVTThEcHdiS2VQQUEyMlNIQmo1M2l1Q1o4VGZOM3hV?=
 =?utf-8?B?akREcGVacmxwZnMrMVJPcEdxblZGRTM1MmUrbVdnZlp2L2t5Umhvc005WWdl?=
 =?utf-8?B?RkpIcFRmMGlHV0pva1lCRHordUloZ2FrTU5pei9NbjhwNlloSTM5UEtxOXZt?=
 =?utf-8?B?ZmQ1dkI3WE9GQUFKRWF5SGlRSU8yRzl4bUk1c2xOVHlsY1FYSnZDaEdrOERG?=
 =?utf-8?B?L3dHZThKZmk3M3FQT3gyMmIzU1o4YXNHNGlack9EWGVoNVlmNEg5eFVEUjJr?=
 =?utf-8?B?VndEandvRG54VWtKWHZaYlVDNU90N3Jidjh2V1BITmtTc0dwclZFek04bWJG?=
 =?utf-8?B?alZGZEhwTy95U0NxdmEwVzFFN3VBWFo0TEtoS3UrNDlxczhHdGlrZjMxVTVl?=
 =?utf-8?B?MnRscUR6RUl3ZGxWem1KaG56NUdEemVNN3VocHpUaXZiODBRNUJnMUx4NVV4?=
 =?utf-8?B?dTdVZDJFaEdwa2dnT3lQL0pMeFVkS1BPdDhMWTVDVDR4RHoxc3ZPUUxmUTlv?=
 =?utf-8?B?WHpQaldWMzgwck9OWG1jdkMwLzNlZkpjUTNtanBLVGlMY0czdTFmdk0yaC9s?=
 =?utf-8?B?TW5paWhWTVNPWjFjMGtWbW9kZGZrekJsTU5BTVY2aWpVRUZXY2dmZGtIa0t0?=
 =?utf-8?B?akFpdWJ3WjgxaGIwWE5iL1AzMDlGaHQxY0F3WWdmVmRKZ0tSWklGRm1TcENa?=
 =?utf-8?B?NU9vTTExeTJHWGZzOERuS3dBeTFpY3NlYjF2MXJTbHBJVThmQUZwWU41WXhO?=
 =?utf-8?B?MmV3Njd1QnROWVV5U09sWHFLbklrNlB0NVhOWWJmczhYRGFSSFFzQjlUM0VC?=
 =?utf-8?B?dHJndUdtdVY5c0JFK2cwdDl4QVNYVmM4NEFmMENxb25EQU14b1kzSGp1NlNS?=
 =?utf-8?B?LzVNalFpcmYxMVRuejJpZjVGaGpXM1BIaDlPNUdLR1dHR29mWEN3Uzg1SnBY?=
 =?utf-8?B?Z1lYL0VQUngrTnlqWmdqRUtkLzM0aXBaSWt1aEw4UlVWdDNsQmlTdThsN0JV?=
 =?utf-8?B?K1E4bkYyZVYxU0NrVkZQdjNPdzcySTQxT1RQdUo3eXh5ZVNEeHUwV2dydVB4?=
 =?utf-8?B?Rk5aVXlMaHRQc3phL3VIYTlBem5kWTRjeXhleTB1NFNPMzR4R3J6aStvQmlM?=
 =?utf-8?B?T3B4T3hHZ1VYQTNKNmxHT3Q4WGhzaGJSNStjWnUzbW81SVByb0NKN1piRTha?=
 =?utf-8?B?TmdxQ2ttb1IwdWxhenNlQWM3SGlwbXpVRDhaKzZDRGtQZWJzZ2o2ZkdnMG5z?=
 =?utf-8?B?blduTE5DQldndVNZZHVQQ1ppR3dtUkM2TlpjTFQ4bEd2bC9Ra3VvU04zcVVa?=
 =?utf-8?B?T0Nsc01yd0FhRE92djlIWmhnVmZ3Si9MRzExTE1EMWdSZG1LVWJqcW5QVGhM?=
 =?utf-8?B?SXp1K25EamczV0hVMWQwWUJYTFBUeWhSOHR4L0JmdE5Cd3BJRy92WTMwMkhE?=
 =?utf-8?B?S0J4MVkvQTBic3RzT3JJcVFlY0VOdkJIOTBVeFdHTzdpTGUzTjM4b0lyeG5S?=
 =?utf-8?B?TUxzMkpiUGJsNmRPN1I4Ui9Qc29YeVhUblVycS9ORkdvWW52bFFHeVozUzAz?=
 =?utf-8?B?V24xSW5vK2xOQ3R5alF6RnEyeUVBN09KUEJJejlIdUlsd0swWGFZVFE3QlhQ?=
 =?utf-8?B?K3pCdVlvaXFtS2J4L0NEc015NTVQN1N3SlhtQ1RrSXo3V1VaRXJrK3E3T2t4?=
 =?utf-8?B?UFRsN2l0b1lBTzY0SW1LbXRPWW9yZWY2SFlIUW13Q3h5STU1a090RVQ1NXdz?=
 =?utf-8?Q?Zu36Mv4wkaYCn5aH6BcZe71f6PmuSINm4TpPx/iEImujC?=
X-MS-Exchange-AntiSpam-MessageData-1: kiJX5s3c5jokzg==
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6569a305-f697-48e3-bc89-08da34cd39ed
X-MS-Exchange-CrossTenant-AuthSource: BN8PR12MB3587.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 May 2022 10:42:07.4449
 (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: p0TXpPI8KrapUGKRXGxFE/x1VmyZ4SXowZY/qwFOFjeHBV8lpnDwsJidBy4vJ6md
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR12MB4320
X-Spam-Status: No, score=-4.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,NICE_REPLY_A,RDNS_NONE,SPF_HELO_NONE,
	T_SCC_BODY_TEXT_LINE autolearn=unavailable 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


Am 13.05.22 um 12:38 schrieb Charan Teja Kalla:
> On 5/13/2022 3:59 PM, Christian König wrote:
>> Am 13.05.22 um 12:18 schrieb Charan Teja Kalla:
>>> On 5/13/2022 3:41 PM, Greg KH wrote:
>>>>> Reported-by: kernel test robot <lkp@intel.com>
>>>> The trest robot did not say that the dmabuf stat name was being
>>>> duplicated, did it?
>>>>
>>> It reported a printk warning on V2[1]. Should we remove this on V3?
>> We only add the kernel test robot is report when it found the underlying
>> problem and not just noted some warning on an intermediate patch version.
> Noted. Thanks!!
>>> @Christian: Could you please drop this tag while merging?
>> Sure, I don't have much on my plate at the moment. But don't let it
>> become a habit.
>>
> Sure. I am also thinking If it is worth to add stable tag? Though it is
> not crashing the kernel but definitely making the dma_buf_export to fail
> for no reason.
>
> If yes, I can resend the patch with all these tags.

Yeah, sure.

Christian.

>
>> Going to push it upstream through drm-misc-fixes now.