Received-SPF: pass (google.com: domain of linux-kernel+bounces-37512-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Wed, 24 Jan 2024 12:27:56 -0600
From: Lucas De Marchi <lucas.demarchi@intel.com>
To: David Gow <davidgow@google.com>
CC: Rae Moar <rmoar@google.com>, <linux-kernel@vger.kernel.org>, "Brendan
 Higgins" <brendan.higgins@linux.dev>, <linux-kselftest@vger.kernel.org>,
	"intel-xe@lists.freedesktop.org" <intel-xe@lists.freedesktop.org>,
	<kunit-dev@googlegroups.com>
Subject: Re: Re: Re: [PATCH v2] kunit: Mark filter* params as rw
Message-ID: <tmrstqs2okfzttk2tqedjphjcaampmmltehwjfhxeicfarknw7@idsjipdtsjbq>
References: <20240122171408.3328510-1-lucas.demarchi@intel.com>
 <CABVgOSmjyCf5DSh+LJtaLubHN3F_mrFo=ZRk23u2PEmmkLBUjA@mail.gmail.com>
 <gi2oosfohy3jjjvuazlg76se6m4w76ys5gxtmj5ngquxwqzz46@s7vgxulxzhao>
 <CABVgOSmm8Arj2RvHcsQYMssrK_GKq-wNixjFtmB6AAxhs2hgMg@mail.gmail.com>
Content-Type: text/plain; charset="us-ascii"; format=flowed
Content-Disposition: inline
In-Reply-To: <CABVgOSmm8Arj2RvHcsQYMssrK_GKq-wNixjFtmB6AAxhs2hgMg@mail.gmail.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?cSwa7zOpHTr3tiBRQV+owt+13/PESX6y5JWetr23TmHVBGoT1A3rsq4/Oyb/?=
 =?us-ascii?Q?Eqsz7tm+c6NZK4mblzpfKxbOX0MSg0Tyk9sM/J2XeES6FOlbpjz5zk3HfSAU?=
 =?us-ascii?Q?o9EBlophvc1qcPw1lb0cZiSAUW0lqbdWXNJunw72Jj+wtBAGE8D0qCcbtEUw?=
 =?us-ascii?Q?UGx69tSwSV+AB8xHUwgBWjI+9Ih8jg8DHlBDnaUqwxVw3H9LjMrTzS5UmX2z?=
 =?us-ascii?Q?mLBJZsXrx248lSE4qWxVUjT1YaNtMyVJNNJGvfG4lUPzc9MRTmZvOROkR3mX?=
 =?us-ascii?Q?QIjDAfJDN5vy6bwk4vw4jOHjjgwqDPuBTF5xY+a4Tq4l+J47yDOkNz3zt3J+?=
 =?us-ascii?Q?9Yp2P5bhUoeT1e235qy3AN6DdVozkC00ZfY6oTmA44VsVZoSQMHXt0Zx3ZHS?=
 =?us-ascii?Q?nXLT65q3Yxbax79PhSvziTpH9RTJAL8nGoLVCwl2nJ6dtkaiPB3L7E0uzUkc?=
 =?us-ascii?Q?xCkt5+J9Sior4j2jwvrrNCtSO0nnvwED1r5yyaYTKj5nDTvKZzp6JFqfUo7v?=
 =?us-ascii?Q?3tYkc1jbGFRkksEiiSbnvN7Gvbw2E0TnV95GBZftT3o3Fv7FL5P1OFHcO5uH?=
 =?us-ascii?Q?gVTHOg/WS86Af5QfM9IFprOq6yu7hlnHoOCT33WdkauWFKdWn8ZX0JU0Ggec?=
 =?us-ascii?Q?4nRZLcjXaf/YZj3lt10cxSCssU6tY6lzuRqmUARdbvX9KDTTyZsiRXcnWTIM?=
 =?us-ascii?Q?AeD6Gb6P4uFwIf4J5d3Os60CviXrdU3AookfWfmpg/PpmPu2lZ4Gfah7jugs?=
 =?us-ascii?Q?QoW2ZhF69Fw0SGorGGdmNfob3A2O6k0SZwYXBff7rTcMZkVy3iPZZ7TOif09?=
 =?us-ascii?Q?77cCxCgkjKttGtxVJCx+ESf5j/+JC6ZRJc/b4HQNAVQPlgY9mT8brjeMoK3f?=
 =?us-ascii?Q?2XziQyculWAQU9Yw1j3+403HpTahgheH3ebYdnU32BvhnmaIad1Gk13dNp7a?=
 =?us-ascii?Q?wOd+Zdsa2y2p5CNBsSUDJKMkw/Axwb0gxgqaD4YxdX+0s7rocAGVK+soUwOk?=
 =?us-ascii?Q?jjHWJN2RM2E7x6cN/hfNCdGRd9/i8fns0ATAjDMhhNrIOI4sX7spRGNaGhdo?=
 =?us-ascii?Q?XJVw1uofbEOO/n6rcI61JPtwC7WlrhlNy1DX2R7P9D8m5d/MSYcswwAPBuQu?=
 =?us-ascii?Q?TMnBo9EJDxZoBsobtRrRKG+rtnkLeAWOMXgQ0D5/sblc+Agty69LcNS+a5H/?=
 =?us-ascii?Q?xjMbfu61s3slg1vCtk/fbKkk9I9G2n5KsRolwaw3sJJcEjEFebJjnyfz7YZi?=
 =?us-ascii?Q?sSkwCyRIjT3FSGxZJcuLEHOgHnJS3AWnDZL47ik2EmBi5jUjw5YPfaYJzQl6?=
 =?us-ascii?Q?I1/NNvQSVFEzAhidw8n2UYBhXZLq3W18Ie2uIDXv1A8h6G5nNKBR27wGltj/?=
 =?us-ascii?Q?hAZPBnWsBPhxHYpCwIOmFVKzWm3gx4xxBuQZbTpBlerDW4TsiCk53e0ID188?=
 =?us-ascii?Q?Jv7H68D22YNAGANBAB0Gz+N0nHASbAUbkQvhX8UPb5sKeBUNFBQGYPM0IX64?=
 =?us-ascii?Q?1yDQf3S0/FZK5K3xOVCq1Gp1GYE4tQR2in3SLVqTW8v9IFma6O70tASUyM+q?=
 =?us-ascii?Q?exUlS3CpsV3oRc/zUS6sU8lWL3uojJ3GZoGeEl7c6BdvMslXXRLG3nJE768Z?=
 =?us-ascii?Q?PQ=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 8788b0d1-653e-43cd-05ae-08dc1d0a30df
X-MS-Exchange-CrossTenant-AuthSource: CY5PR11MB6139.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Jan 2024 18:27:58.7985
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: jaPAI8YoHjXX3E4m/nrAlC3tyAjKcSgBU9qSgc5dBTBrGCH9dN7+PtPbw6vuU+uC0kBqyg45B1TcZsx9rWuFOJF3VdrNCr8EyEXoGgcNnA0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR11MB7918
X-OriginatorOrg: intel.com

On Wed, Jan 24, 2024 at 11:49:43AM +0800, David Gow wrote:
>On Wed, 24 Jan 2024 at 00:31, Lucas De Marchi <lucas.demarchi@intel.com> wrote:
>>
>> On Tue, Jan 23, 2024 at 04:01:49PM +0800, David Gow wrote:
>> >On Tue, 23 Jan 2024 at 01:14, Lucas De Marchi <lucas.demarchi@intel.com> wrote:
>> >>
>> >> By allowing the filter_glob parameter to be written to, it's possible to
>> >> tweak the testsuites that will be executed on new module loads. This
>> >> makes it easier to run specific tests without having to reload kunit and
>> >> provides a way to filter tests on real HW even if kunit is builtin.
>> >> Example for xe driver:
>> >>
>> >> 1) Run just 1 test
>> >>         # echo -n xe_bo > /sys/module/kunit/parameters/filter_glob
>> >>         # modprobe -r xe_live_test
>> >>         # modprobe xe_live_test
>> >>         # ls /sys/kernel/debug/kunit/
>> >>         xe_bo
>> >>
>> >> 2) Run all tests
>> >>         # echo \* > /sys/module/kunit/parameters/filter_glob
>> >>         # modprobe -r xe_live_test
>> >>         # modprobe xe_live_test
>> >>         # ls /sys/kernel/debug/kunit/
>> >>         xe_bo  xe_dma_buf  xe_migrate  xe_mocs
>> >>
>> >> For completeness and to cover other use cases, also change filter and
>> >> filter_action to rw.
>> >>
>> >> Link: https://lore.kernel.org/intel-xe/dzacvbdditbneiu3e3fmstjmttcbne44yspumpkd6sjn56jqpk@vxu7sksbqrp6/
>> >> Reviewed-by: Rae Moar <rmoar@google.com>
>> >> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
>> >> ---
>> >
>> >This looks good to me, and works here.
>> >
>> >Reviewed-by: David Gow <davidgow@google.com>
>>
>> are you going to merge this through kunit tree or should I carry it in
>> drm ?
>>
>
>I think we'll merge it via KUnit for 6.9, but if you need it sooner, I
>doubt it'd cause problems if it went in via DRM instead.

I think it's fine to have it on your tree.  For our CI, we can just
apply it on the side to have it working before the 6.9 merge window.

thanks
Lucas De Marchi