2017-12-27 12:55:14 -06:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2016-06-15 08:32:18 -05:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2015-02-17 11:36:58 +09:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2018-10-04 20:30:40 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:29 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2016-11-01 14:14:22 -05:00
|
|
|
|
2018-04-26 13:00:30 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2016-11-01 14:14:22 -05:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2016-11-01 14:14:22 -05:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-04-26 13:00:30 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2016-11-01 14:14:22 -05:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2016-11-01 14:14:22 -05:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
2016-11-01 14:14:22 -05:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
2016-11-01 14:14:22 -05:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
2016-11-01 14:14:22 -05:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2016-11-01 14:14:22 -05:00
|
|
|
|
2017-10-19 14:18:27 -07:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2016-11-01 14:14:22 -05:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:24 -07:00
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:24 -07:00
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:24 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-10-04 20:32:04 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:21 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:21 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-16 17:54:32 +08:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2018-10-12 19:21:16 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2019-11-21 13:16:56 -06:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2018-10-04 20:32:04 -07:00
|
|
|
|
2018-10-12 19:21:16 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-10-04 20:32:04 -07:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-10-04 20:32:04 -07:00
|
|
|
|
|
|
|
|
|
2019-11-21 13:16:56 -06:00
|
|
|
|
2018-10-04 20:32:04 -07:00
|
|
|
|
|
|
|
|
|
2017-10-16 17:54:32 +08:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-16 17:54:32 +08:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-02-26 14:01:23 -08:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-10-12 19:21:16 -07:00
|
|
|
|
2015-03-03 10:04:45 -03:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2018-10-12 19:21:16 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-07-19 09:25:21 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
2018-10-04 20:29:01 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2018-10-12 19:21:16 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-10-04 20:24:17 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-10-04 20:25:13 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2017-10-17 16:36:28 -07:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:21 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2017-10-17 16:36:22 -07:00
|
|
|
|
2018-08-27 20:00:19 -05:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2015-10-22 11:02:48 +02:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
2018-10-04 20:33:35 -07:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
of: overlay: check prevents multiple fragments touching same property
Add test case of two fragments updating the same property. After
adding the test case, the system hangs at end of boot, after
after slub stack dumps from kfree() in crypto modprobe code.
Multiple overlay fragments adding, modifying, or deleting the same
property is not supported. Add check to detect the attempt and fail
the overlay apply.
Before this patch, the first fragment error would terminate
processing. Allow fragment checking to proceed and report all
of the fragment errors before terminating the overlay apply. This
is not a hot path, thus not a performance issue (the error is not
transient and requires fixing the overlay before attempting to
apply it again).
After applying this patch, the devicetree unittest messages will
include:
OF: overlay: ERROR: multiple fragments add, update, and/or delete property /testcase-data-2/substation@100/motor-1/rpm_avail
...
### dt-test ### end of unittest - 212 passed, 0 failed
The check to detect two fragments updating the same property is
folded into the patch that created the test case to maintain
bisectability.
Tested-by: Alan Tull <atull@kernel.org>
Signed-off-by: Frank Rowand <frank.rowand@sony.com>
2018-10-04 20:36:18 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-10-04 20:35:14 -07:00
|
|
|
|
of: overlay: check prevents multiple fragments touching same property
Add test case of two fragments updating the same property. After
adding the test case, the system hangs at end of boot, after
after slub stack dumps from kfree() in crypto modprobe code.
Multiple overlay fragments adding, modifying, or deleting the same
property is not supported. Add check to detect the attempt and fail
the overlay apply.
Before this patch, the first fragment error would terminate
processing. Allow fragment checking to proceed and report all
of the fragment errors before terminating the overlay apply. This
is not a hot path, thus not a performance issue (the error is not
transient and requires fixing the overlay before attempting to
apply it again).
After applying this patch, the devicetree unittest messages will
include:
OF: overlay: ERROR: multiple fragments add, update, and/or delete property /testcase-data-2/substation@100/motor-1/rpm_avail
...
### dt-test ### end of unittest - 212 passed, 0 failed
The check to detect two fragments updating the same property is
folded into the patch that created the test case to maintain
bisectability.
Tested-by: Alan Tull <atull@kernel.org>
Signed-off-by: Frank Rowand <frank.rowand@sony.com>
2018-10-04 20:36:18 -07:00
|
|
|
|
2018-10-04 20:35:14 -07:00
|
|
|
|
|
|
|
|
|
of: overlay: check prevents multiple fragments touching same property
Add test case of two fragments updating the same property. After
adding the test case, the system hangs at end of boot, after
after slub stack dumps from kfree() in crypto modprobe code.
Multiple overlay fragments adding, modifying, or deleting the same
property is not supported. Add check to detect the attempt and fail
the overlay apply.
Before this patch, the first fragment error would terminate
processing. Allow fragment checking to proceed and report all
of the fragment errors before terminating the overlay apply. This
is not a hot path, thus not a performance issue (the error is not
transient and requires fixing the overlay before attempting to
apply it again).
After applying this patch, the devicetree unittest messages will
include:
OF: overlay: ERROR: multiple fragments add, update, and/or delete property /testcase-data-2/substation@100/motor-1/rpm_avail
...
### dt-test ### end of unittest - 212 passed, 0 failed
The check to detect two fragments updating the same property is
folded into the patch that created the test case to maintain
bisectability.
Tested-by: Alan Tull <atull@kernel.org>
Signed-off-by: Frank Rowand <frank.rowand@sony.com>
2018-10-04 20:36:18 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-10-04 20:35:14 -07:00
|
|
|
|
of: overlay: check prevents multiple fragments touching same property
Add test case of two fragments updating the same property. After
adding the test case, the system hangs at end of boot, after
after slub stack dumps from kfree() in crypto modprobe code.
Multiple overlay fragments adding, modifying, or deleting the same
property is not supported. Add check to detect the attempt and fail
the overlay apply.
Before this patch, the first fragment error would terminate
processing. Allow fragment checking to proceed and report all
of the fragment errors before terminating the overlay apply. This
is not a hot path, thus not a performance issue (the error is not
transient and requires fixing the overlay before attempting to
apply it again).
After applying this patch, the devicetree unittest messages will
include:
OF: overlay: ERROR: multiple fragments add, update, and/or delete property /testcase-data-2/substation@100/motor-1/rpm_avail
...
### dt-test ### end of unittest - 212 passed, 0 failed
The check to detect two fragments updating the same property is
folded into the patch that created the test case to maintain
bisectability.
Tested-by: Alan Tull <atull@kernel.org>
Signed-off-by: Frank Rowand <frank.rowand@sony.com>
2018-10-04 20:36:18 -07:00
|
|
|
|
2018-10-04 20:35:14 -07:00
|
|
|
|
of: overlay: check prevents multiple fragments touching same property
Add test case of two fragments updating the same property. After
adding the test case, the system hangs at end of boot, after
after slub stack dumps from kfree() in crypto modprobe code.
Multiple overlay fragments adding, modifying, or deleting the same
property is not supported. Add check to detect the attempt and fail
the overlay apply.
Before this patch, the first fragment error would terminate
processing. Allow fragment checking to proceed and report all
of the fragment errors before terminating the overlay apply. This
is not a hot path, thus not a performance issue (the error is not
transient and requires fixing the overlay before attempting to
apply it again).
After applying this patch, the devicetree unittest messages will
include:
OF: overlay: ERROR: multiple fragments add, update, and/or delete property /testcase-data-2/substation@100/motor-1/rpm_avail
...
### dt-test ### end of unittest - 212 passed, 0 failed
The check to detect two fragments updating the same property is
folded into the patch that created the test case to maintain
bisectability.
Tested-by: Alan Tull <atull@kernel.org>
Signed-off-by: Frank Rowand <frank.rowand@sony.com>
2018-10-04 20:36:18 -07:00
|
|
|
|
2018-10-04 20:35:14 -07:00
|
|
|
|
of: overlay: check prevents multiple fragments touching same property
Add test case of two fragments updating the same property. After
adding the test case, the system hangs at end of boot, after
after slub stack dumps from kfree() in crypto modprobe code.
Multiple overlay fragments adding, modifying, or deleting the same
property is not supported. Add check to detect the attempt and fail
the overlay apply.
Before this patch, the first fragment error would terminate
processing. Allow fragment checking to proceed and report all
of the fragment errors before terminating the overlay apply. This
is not a hot path, thus not a performance issue (the error is not
transient and requires fixing the overlay before attempting to
apply it again).
After applying this patch, the devicetree unittest messages will
include:
OF: overlay: ERROR: multiple fragments add, update, and/or delete property /testcase-data-2/substation@100/motor-1/rpm_avail
...
### dt-test ### end of unittest - 212 passed, 0 failed
The check to detect two fragments updating the same property is
folded into the patch that created the test case to maintain
bisectability.
Tested-by: Alan Tull <atull@kernel.org>
Signed-off-by: Frank Rowand <frank.rowand@sony.com>
2018-10-04 20:36:18 -07:00
|
|
|
|
|
|
|
|
|
2018-10-04 20:35:14 -07:00
|
|
|
|
|
|
|
|
|
of: overlay: check prevents multiple fragments touching same property
Add test case of two fragments updating the same property. After
adding the test case, the system hangs at end of boot, after
after slub stack dumps from kfree() in crypto modprobe code.
Multiple overlay fragments adding, modifying, or deleting the same
property is not supported. Add check to detect the attempt and fail
the overlay apply.
Before this patch, the first fragment error would terminate
processing. Allow fragment checking to proceed and report all
of the fragment errors before terminating the overlay apply. This
is not a hot path, thus not a performance issue (the error is not
transient and requires fixing the overlay before attempting to
apply it again).
After applying this patch, the devicetree unittest messages will
include:
OF: overlay: ERROR: multiple fragments add, update, and/or delete property /testcase-data-2/substation@100/motor-1/rpm_avail
...
### dt-test ### end of unittest - 212 passed, 0 failed
The check to detect two fragments updating the same property is
folded into the patch that created the test case to maintain
bisectability.
Tested-by: Alan Tull <atull@kernel.org>
Signed-off-by: Frank Rowand <frank.rowand@sony.com>
2018-10-04 20:36:18 -07:00
|
|
|
|
|
|
|
|
|
2018-10-04 20:35:14 -07:00
|
|
|
|
|
|
|
|
|
of: overlay: check prevents multiple fragments touching same property
Add test case of two fragments updating the same property. After
adding the test case, the system hangs at end of boot, after
after slub stack dumps from kfree() in crypto modprobe code.
Multiple overlay fragments adding, modifying, or deleting the same
property is not supported. Add check to detect the attempt and fail
the overlay apply.
Before this patch, the first fragment error would terminate
processing. Allow fragment checking to proceed and report all
of the fragment errors before terminating the overlay apply. This
is not a hot path, thus not a performance issue (the error is not
transient and requires fixing the overlay before attempting to
apply it again).
After applying this patch, the devicetree unittest messages will
include:
OF: overlay: ERROR: multiple fragments add, update, and/or delete property /testcase-data-2/substation@100/motor-1/rpm_avail
...
### dt-test ### end of unittest - 212 passed, 0 failed
The check to detect two fragments updating the same property is
folded into the patch that created the test case to maintain
bisectability.
Tested-by: Alan Tull <atull@kernel.org>
Signed-off-by: Frank Rowand <frank.rowand@sony.com>
2018-10-04 20:36:18 -07:00
|
|
|
|
2018-10-04 20:35:14 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
|
|
|
|
|
2018-10-04 20:33:35 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-10-04 20:33:35 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
of: overlay: check prevents multiple fragments touching same property
Add test case of two fragments updating the same property. After
adding the test case, the system hangs at end of boot, after
after slub stack dumps from kfree() in crypto modprobe code.
Multiple overlay fragments adding, modifying, or deleting the same
property is not supported. Add check to detect the attempt and fail
the overlay apply.
Before this patch, the first fragment error would terminate
processing. Allow fragment checking to proceed and report all
of the fragment errors before terminating the overlay apply. This
is not a hot path, thus not a performance issue (the error is not
transient and requires fixing the overlay before attempting to
apply it again).
After applying this patch, the devicetree unittest messages will
include:
OF: overlay: ERROR: multiple fragments add, update, and/or delete property /testcase-data-2/substation@100/motor-1/rpm_avail
...
### dt-test ### end of unittest - 212 passed, 0 failed
The check to detect two fragments updating the same property is
folded into the patch that created the test case to maintain
bisectability.
Tested-by: Alan Tull <atull@kernel.org>
Signed-off-by: Frank Rowand <frank.rowand@sony.com>
2018-10-04 20:36:18 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:21 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:21 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2018-02-12 00:25:04 -08:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-02-12 00:25:04 -08:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2018-02-12 00:25:04 -08:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2018-02-12 00:25:04 -08:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
2017-12-05 16:27:02 +01:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
2017-10-17 16:36:31 -07:00
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-12-05 16:27:02 +01:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-12-08 14:13:02 +01:00
|
|
|
|
2017-12-08 14:13:03 +01:00
|
|
|
|
|
|
|
|
|
2017-11-28 09:26:33 +01:00
|
|
|
|
2017-12-08 14:13:03 +01:00
|
|
|
|
|
|
|
|
|
2018-10-04 20:28:08 -07:00
|
|
|
|
2017-12-08 14:13:03 +01:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
2017-12-08 14:13:03 +01:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:30 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-19 14:38:11 -07:00
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
|
|
|
|
|
2017-07-19 09:25:22 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:22 -07:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2017-12-05 16:27:02 +01:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-12-05 16:27:02 +01:00
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-12-05 16:27:02 +01:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
2018-04-26 13:00:30 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
2018-04-26 13:00:30 +02:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-11-28 09:25:23 +01:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-11-28 09:25:23 +01:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:29 -07:00
|
|
|
|
of: overlay: Fix (un)locking in of_overlay_apply()
The special overlay mutex is taken first, hence it should be released
last in the error path.
of_resolve_phandles() must be called with of_mutex held. Without it, a
node and new phandle could be added via of_attach_node(), making the max
phandle wrong.
free_overlay_changeset() must be called with of_mutex held, if any
non-trivial cleanup is to be done.
Hence move "mutex_lock(&of_mutex)" up, as suggested by Frank, and merge
the two tail statements of the success and error paths, now they became
identical.
Note that while the two mutexes are adjacent, we still need both:
__of_changeset_apply_notify(), which is called by __of_changeset_apply()
unlocks of_mutex, then does notifications then locks of_mutex. So the
mutex get released in the middle of of_overlay_apply()
Fixes: f948d6d8b792bb90 ("of: overlay: avoid race condition between applying multiple overlays")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Frank Rowand <frank.rowand@sony.com>
Signed-off-by: Rob Herring <robh@kernel.org>
2017-12-05 16:27:03 +01:00
|
|
|
|
2017-10-17 16:36:29 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
2016-11-01 14:14:22 -05:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
2016-06-15 08:32:18 -05:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2018-07-12 14:00:07 -07:00
|
|
|
|
|
|
|
|
|
2017-11-28 09:26:33 +01:00
|
|
|
|
|
|
|
|
|
2018-10-04 20:33:35 -07:00
|
|
|
|
2017-11-28 09:26:33 +01:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2016-11-01 14:14:22 -05:00
|
|
|
|
of: overlay: Fix (un)locking in of_overlay_apply()
The special overlay mutex is taken first, hence it should be released
last in the error path.
of_resolve_phandles() must be called with of_mutex held. Without it, a
node and new phandle could be added via of_attach_node(), making the max
phandle wrong.
free_overlay_changeset() must be called with of_mutex held, if any
non-trivial cleanup is to be done.
Hence move "mutex_lock(&of_mutex)" up, as suggested by Frank, and merge
the two tail statements of the success and error paths, now they became
identical.
Note that while the two mutexes are adjacent, we still need both:
__of_changeset_apply_notify(), which is called by __of_changeset_apply()
unlocks of_mutex, then does notifications then locks of_mutex. So the
mutex get released in the middle of of_overlay_apply()
Fixes: f948d6d8b792bb90 ("of: overlay: avoid race condition between applying multiple overlays")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Frank Rowand <frank.rowand@sony.com>
Signed-off-by: Rob Herring <robh@kernel.org>
2017-12-05 16:27:03 +01:00
|
|
|
|
2017-10-17 16:36:29 -07:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
of: overlay: Fix (un)locking in of_overlay_apply()
The special overlay mutex is taken first, hence it should be released
last in the error path.
of_resolve_phandles() must be called with of_mutex held. Without it, a
node and new phandle could be added via of_attach_node(), making the max
phandle wrong.
free_overlay_changeset() must be called with of_mutex held, if any
non-trivial cleanup is to be done.
Hence move "mutex_lock(&of_mutex)" up, as suggested by Frank, and merge
the two tail statements of the success and error paths, now they became
identical.
Note that while the two mutexes are adjacent, we still need both:
__of_changeset_apply_notify(), which is called by __of_changeset_apply()
unlocks of_mutex, then does notifications then locks of_mutex. So the
mutex get released in the middle of of_overlay_apply()
Fixes: f948d6d8b792bb90 ("of: overlay: avoid race condition between applying multiple overlays")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Frank Rowand <frank.rowand@sony.com>
Signed-off-by: Rob Herring <robh@kernel.org>
2017-12-05 16:27:03 +01:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
of: overlay: Fix (un)locking in of_overlay_apply()
The special overlay mutex is taken first, hence it should be released
last in the error path.
of_resolve_phandles() must be called with of_mutex held. Without it, a
node and new phandle could be added via of_attach_node(), making the max
phandle wrong.
free_overlay_changeset() must be called with of_mutex held, if any
non-trivial cleanup is to be done.
Hence move "mutex_lock(&of_mutex)" up, as suggested by Frank, and merge
the two tail statements of the success and error paths, now they became
identical.
Note that while the two mutexes are adjacent, we still need both:
__of_changeset_apply_notify(), which is called by __of_changeset_apply()
unlocks of_mutex, then does notifications then locks of_mutex. So the
mutex get released in the middle of of_overlay_apply()
Fixes: f948d6d8b792bb90 ("of: overlay: avoid race condition between applying multiple overlays")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Frank Rowand <frank.rowand@sony.com>
Signed-off-by: Rob Herring <robh@kernel.org>
2017-12-05 16:27:03 +01:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2018-02-12 00:19:42 -08:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:21 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:21 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2015-10-22 11:02:48 +02:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2015-10-22 11:02:48 +02:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:21 -07:00
|
|
|
|
2017-10-17 16:36:27 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:21 -07:00
|
|
|
|
2017-10-17 16:36:27 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:27 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:27 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:27 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
2018-03-09 11:44:47 +01:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-11-28 09:25:23 +01:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-11-28 09:25:23 +01:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
2018-07-12 14:00:07 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
2018-07-12 14:00:07 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
2017-11-28 09:26:33 +01:00
|
|
|
|
|
|
|
|
|
2018-10-04 20:33:35 -07:00
|
|
|
|
2017-11-28 09:26:33 +01:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2015-05-21 14:10:26 +02:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|
2017-10-17 16:36:26 -07:00
|
|
|
|
2017-10-17 16:36:25 -07:00
|
|
|
|
|
|
|
|
|
2014-10-28 22:35:58 +02:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
2017-10-17 16:36:23 -07:00
|
|
|
|