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: <3d51a79e-faa3-5a7e-6ce6-372684e0f14a@axentia.se>
Date:   Wed, 24 May 2023 09:45:27 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.0
Subject: Re: PWM regression causing failures with the pwm-atmel driver
Content-Language: sv-SE, en-US
To:     =?UTF-8?Q?Uwe_Kleine-K=c3=b6nig?= <u.kleine-koenig@pengutronix.de>
Cc:     LKML <linux-kernel@vger.kernel.org>, linux-pwm@vger.kernel.org,
        Thorsten Leemhuis <regressions@leemhuis.info>,
        Thierry Reding <thierry.reding@gmail.com>,
        Conor Dooley <conor.dooley@microchip.com>,
        Claudiu Beznea <claudiu.beznea@microchip.com>
From:   Peter Rosin <peda@axentia.se>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?TVdYUVZGNmdESmVERngzaDBHZTk5bE9KWitYcU50L2M5UU5hRThTWTNBZEI1?=
 =?utf-8?B?ZlY1YUF0WkJMUC8wZk44NDRsdFpDQjg5NitNMmNNM1RJV0NXVzVwY0NOU2xu?=
 =?utf-8?B?bnRDcU51eFJzYUl1UmZZVkQ5R2ZadkNybEFXamJZWFJMcUU0V1Y2V2hBb3R3?=
 =?utf-8?B?YXp5bUNFSHk4VnlzeHFOamY5VGhQOHllRHRHVDBMdkpUbFllTWYxQVAxZkEy?=
 =?utf-8?B?akd4T01jcVpxZjJlOHg2T2ZVQXJwN1EvTTdRcXIwWTd4d1NEaFYwRFh5U2l3?=
 =?utf-8?B?Vy9rQmhuTDJBQnFxZzVLaWhFRmh0V0drcXhSTEY5VlYvaDAxMmNuS29BL0lr?=
 =?utf-8?B?MTVuNys0UWV4ZnlsdTJDQkZZQ01RYm43MW5CMEh6Q1N1TnhrUjRtY3VhY0ky?=
 =?utf-8?B?ODY1Q3M5VER4T1BST0N5VklJNmdYTGc4ZVVIQTZRNXRCWFhpNzY0eDNtR1ZF?=
 =?utf-8?B?bml6bDFDRkdWMkw2WGNDNHFETCtIcHJ6Y2VKS0RhOUF0Yitmd1B3SndOTHRv?=
 =?utf-8?B?SDJVUHlQblhlN3puRlp2V0xxckJkV3UyTkVJZXh2V2JKTmRRcDF4MVIrZWd5?=
 =?utf-8?B?ZUlJSEg5WG93TU9GWGtqa1ZOV2RwUXdJT3N2bXlGU3BZT3o2dm50dFBCNEFJ?=
 =?utf-8?B?Ym9QY1hQYnd1Y0hFSjRYQWdydmY1OTU5Z29DTVpLZWoyKzhqbHhJQWRTU1Bz?=
 =?utf-8?B?Njd4eFBxdWZtVzNCR3BwT1ZYdnhrYzRTQ3crYXhaclAvZkFLK0VCUnZDY0ZE?=
 =?utf-8?B?UzlubElOSG9HYmFWbThqN0hjalVjc1c4WTFIbys5dFVyYktpS1hrbjhVTk9l?=
 =?utf-8?B?TjU2VnBNRmthTnE3NENsRStzQWVLRTR1MUZjaUJUZ0dmTldBa1VickZsN1RX?=
 =?utf-8?B?SEJTbWJsSjUwTTJ2YUpzQSt4RWkzcmlzZEhBTlAxTDZ6RmY3NldmMk9IUXc5?=
 =?utf-8?B?Z3FHT2pCaHJ6SWd5UkRUcW5VaGlmUmdhY3ZWRE4weU9aZmtZaWFsWnBTZlM1?=
 =?utf-8?B?MU1lU1FJOFNvMnpXZ0pTWkpyZEVUOFdmcVJDaUcwK2NLV3E0VmVBcFU0d2J3?=
 =?utf-8?B?M0ZtYk5BMzdUcHNqQ20zTUFxbllJREQweGZ0OXlpcEZiUitYVnNlSHpiRXpo?=
 =?utf-8?B?N0IvdDRLUVoxUlZCU29kMEVCU2NTSVZWOUJQSDJ5Q3VWK0QzZzA1N3FrR1o4?=
 =?utf-8?B?NHJLamFZUUNVdy9RNWJDRjZMdUxGaS9pNkFTWlFBWEVZZWY2K1NKT2dxMHU1?=
 =?utf-8?B?VTdQamtzWURnUVRYTk8xL1Y1Y1ovRXpHMTFzcnBaN1NaN3RBY0REelg0U0dy?=
 =?utf-8?B?N01leVVOM241RnEySUlzRkZxZkU2MFFSWXVKTklnckNBdWlIQ0hwMnBFSGd2?=
 =?utf-8?B?Skc0OExGbVFVNGNVb3Fia2YvN2pGeFNCa2JrdElDN0dxaUhEWTRCS3c0RmZR?=
 =?utf-8?B?ZkozLzVITGdvWVFhTTA3Zkd1ZFlSdk9MNk5kcXhxMXhLdUE1a21OcUpjOWF1?=
 =?utf-8?B?MWdCM0FmZG1HM3RTWjRTZ0c2M2l6KzNXQ0dQWlNJd0hzUEZwTUJBZjBMQ002?=
 =?utf-8?B?dTc3QVhlNHdYejBxbVQ0NVNTYitZYzBvVDJnWkpPVjE5MHcwUGRWMkU1Umh4?=
 =?utf-8?B?U3Bvdy92dVJhb1dEWE5rZnA5TFUwcEpON1VlTkRTUHVjU1luWHROQ3NHSkxv?=
 =?utf-8?B?MU90cEtrV0NOTlkydzNHTStwaWdjcXZiVFFBMFVXWlJOUjlvQk80T1pEajYw?=
 =?utf-8?B?cXBlejc5ZmJaVXp1MmVZVE5hM3NCcDBXYlJBVlBnQW9FWlNaQUdWeUNCSVlv?=
 =?utf-8?B?RFRYeUhxak80U0Q5Tk14eU1Mb0tXZUhQci93UVBpNDUvdzZOakFwTnJnWHkz?=
 =?utf-8?B?WnZaODk5RjRyaFEyVE96ZEJjSWNYSzdNWitQSkJzbGkvdmpnQzhjckFweUNI?=
 =?utf-8?B?MUh3TlZWUmhmMGlQdnBoOVBRN3M3ZnZrVmxVR0J6azh0OTUzVHFjZWE0bW5C?=
 =?utf-8?B?U05WdVZUTG9hWS91RFlTZ3BzTTc3Uzl6RnB0V09FYXNseHMrcVJSMlB3K05n?=
 =?utf-8?B?aXpYbnN3RG10bjkvSktpMTVMR2dCdWQ2NDU5UGgvVGZET3BHelM2NzQ5OTha?=
 =?utf-8?Q?BO5TpggigqqJp5Hn2iUlR66ov?=
X-OriginatorOrg: axentia.se
X-MS-Exchange-CrossTenant-Network-Message-Id: 82c05466-d7a4-448e-4cac-08db5c2ad85d
X-MS-Exchange-CrossTenant-AuthSource: AM0PR02MB4436.eurprd02.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 May 2023 07:45:29.3707
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4ee68585-03e1-4785-942a-df9c1871a234
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: qRctRt/pE6Z9rj++i3KYi7NPbW7kD7EaPg7+KvT1nR0Cd8kQptNjIct2bcQv0T9K
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR02MB10265
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FAKE_REPLY_A1,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,
        T_SCC_BODY_TEXT_LINE 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!

2023-05-24 at 08:07, Uwe Kleine-König wrote:
> On Tue, May 23, 2023 at 10:42:34PM +0200, Peter Rosin wrote:
>> 2023-05-23 at 01:34, Peter Rosin wrote:
>>> So, I took a step back and can only conclude that there must be some
>>> another regression to find, and I was confused by that other regression.
>>> In short, I was on 6.1.<foo> and everything was fine, and then I bumped
>>> to 6.3 and a process crashed. I went to 6.2 and that same process also
>>> crashed. I then totally focused on v6.1..v6.2 to figure out the problem.
>>> I simply assumed v6.3 had the same problem because the symptom from
>>> 30.000ft was the same (that process died). I failed to go back to v6.3
>>> to confirm that it was indeed the same problem as I had found in the
>>> v6.1..v6.2 range.
>>>
>>> My bad, it seems I have another day of bisections lined up.
>>
>> For closure, I ended up with this:
>> https://lore.kernel.org/lkml/221d19e2-6b92-7f38-7d8a-a730f54c33ea@axentia.se/
>>
>> I.e. another v6.1..v6.2 regression that caused sound failures.
>> The two problems looked very similar to the suffering application.
>>
>> Anyway, sorry again for the noise.
> 
> OK. After your first mail I had the impression there is another PWM
> releated problem to be reported, but it seems this isn't the case.
> 
> Just to have that explicit: Did I understand you right?

The PWM regression was already solved by 1271a7b98e79 ("pwm: Zero-
initialize the pwm_state passed to driver's .get_state()") and I failed
to notice that. I was simply confused by that other totally unrelated
DMA problem that made me think the PWM problem persisted into the
present when in fact it didn't.

All is well.

Cheers,
Peter

#regzbot fixed-by: 1271a7b98e79