From:   Shinichiro Kawasaki <shinichiro.kawasaki@wdc.com>
To:     Vincent Guittot <vincent.guittot@linaro.org>
CC:     Valentin Schneider <Valentin.Schneider@arm.com>,
        Vincent Donnefort <Vincent.Donnefort@arm.com>,
        "peterz@infradead.org" <peterz@infradead.org>,
        "mingo@redhat.com" <mingo@redhat.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "dietmar.eggemann@arm.com" <dietmar.eggemann@arm.com>,
        "jing-ting.wu@mediatek.com" <jing-ting.wu@mediatek.com>,
        Damien Le Moal <Damien.LeMoal@wdc.com>
Subject: Re: [PATCH] sched/core: Mitigate race
 cpus_share_cache()/update_top_cache_domain()
Thread-Topic: [PATCH] sched/core: Mitigate race
 cpus_share_cache()/update_top_cache_domain()
Thread-Index: AQHX2g37TPTJ8je3+U2WwmDdbyK6N6wE6pEAgADh9QCAAAvmgA==
Date:   Tue, 16 Nov 2021 08:45:12 +0000
Message-ID: <20211116084511.ocupfvgrkqrttghg@shindev>
References: <20211104175120.857087-1-vincent.donnefort@arm.com>
 <20211115104601.lwcwxoc67ftbf6uh@shindev> <87h7cdqne7.mognet@arm.com>
 <CAKfTPtAPaMU343=uRuo-LvoP6HnGwnLhTcEMyFVBisdjnT-Spg@mail.gmail.com>
In-Reply-To: <CAKfTPtAPaMU343=uRuo-LvoP6HnGwnLhTcEMyFVBisdjnT-Spg@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?ftbH12qaOXcsFVlyso8cjo3ejd72CzEwUjIHlnPP843JbJKqFjZc1PNTenyK?=
 =?us-ascii?Q?hAs5yMXtgzzTiXD4llfw6NNwqW9uTFrMJfBlUKq19IBcRHfjSPr+nDCUiNEe?=
 =?us-ascii?Q?no1RgcXAyF9pWIYflOfBRhrJj+rs8hVFc/pxX01PMouXhu4MTnX1NtD2UvuT?=
 =?us-ascii?Q?uttwKD+HqIoVHFzifgjOKTlqfiD0dm5nyU+dMEr/PdwHmNPlAQX5Id+W+3jb?=
 =?us-ascii?Q?5gHK0qSoXW8fTiXLclu1tHtcf0CVCQmCIgWwepDPF0ktIpksmFf7JI44JfHl?=
 =?us-ascii?Q?87kIGBLhMOowYyocPm67rkLQGXVxRbhmeVjfQZSyxg+rpoB6CDzAMCVninQf?=
 =?us-ascii?Q?6mJ89P+BSqgy/WcmQqBa8ZqHwjT5cYbpY7CzqwhsvZIMCFNjsA4lD9I/nwbZ?=
 =?us-ascii?Q?Ihwe4/InG9mweFQTE3RmhLe78cREdSPSSDM3IYtsEq68/des/86E/w6yu0Sx?=
 =?us-ascii?Q?dU6LnehqGQq1LldGB6YarAICVAJOl+0EWTBTpZOaNXo9Ppc3CN6rPYk7pTJ1?=
 =?us-ascii?Q?p0NluoFHJu2+Ar+W3DbT8RdVFfJ6CcDPfW5D7qyA5SxmDBpbYrT+COjjRa96?=
 =?us-ascii?Q?eLFbEQA/Psm9GAy1aYZBDCK9D0Jsv1HF/s3E2xjm82Z4QyDy3Pe+0w9uC1yy?=
 =?us-ascii?Q?U3nXU4Hu0m5/cEan3dtBYZmAkLg8rRc42ePrzim2GViCkXWbu1JNQdzONEo6?=
 =?us-ascii?Q?8wRxpzw7aIpJDkYA08vNjqXK7ut45gtN53DeMShBAFWBzlITrte+/bMp9s5n?=
 =?us-ascii?Q?KYXnCOXZsSWqcdGiWL8aPKSU5WKb68CsNPvLS6s2/INd7KhMWa6rRuStAwdV?=
 =?us-ascii?Q?yHaHD0gaU5Q/TR/QEKbKkI5FuAyw9uG8zvUjkWAydYwI96ZgPX/ZQJMezF98?=
 =?us-ascii?Q?f4ryvYNO1yoW05SehTtv4AmotoY3oNZoisHsmCWNa3+NIiGyLE5Gf4DkyBUm?=
 =?us-ascii?Q?dvfy2Ht5tjXKQfmSpvSaM5jsVuvhn5zXNIRABMZA9QejFTG7zctu9WLqDdko?=
 =?us-ascii?Q?eCniDL9JXiWDRQ9dqtpo+xquFIJtMk7lISs0yY1sWzTyEqOYifjCtN4ZHGRl?=
 =?us-ascii?Q?vnLXqpLduZUUmqzHBTzz4knpasmXgh/6Gw0eYvOpfwBzXsM7C5rMMC3lzMN3?=
 =?us-ascii?Q?poxS9pA5OXzvAtvIrF+IDdcbWplAaN+mFHOq8owl6CnnGIEftbWm7fYtDMac?=
 =?us-ascii?Q?LAWnsIRhXJ74ZgOG8hMN6KAp8FrBU72FbXiCxB5dvqkCis89RBjeCZChWxpl?=
 =?us-ascii?Q?/Vg5Gr5OW5PxpNfVUilOoAL3bUZkFYauvXLq5MrIOOvVRTyK/SHX9oimna91?=
 =?us-ascii?Q?OIBVrQYqDjfl5/u9CGplfwNGtU+CuvAsCW1WAW5f5PDqoieEGzMFljbsyl85?=
 =?us-ascii?Q?zx2L20cTD6LhqSKFbn7hfubNdJxn3cMNb0l9Bu8R/J9ugw16OBqxLWAvTvH1?=
 =?us-ascii?Q?zm0lQhzj8rFnVhgenKX4hbx1vJln5jdhPlWHmF2VmX03LuCE0ohpbx+PiAI5?=
 =?us-ascii?Q?K+a3sPgGRx2iORbB07C0vOxpy7IPI044y5HnxS2supVQ7BJTmO1GhJ0Cx1vw?=
 =?us-ascii?Q?8y4VkMTmVITdAXJSwMDR5cT/yoEuwfsjR2ppAbM2slYvVMd/o2F5XbzaKSRD?=
 =?us-ascii?Q?5yv/0WY4t9+IWJ+BJ9fKm1yP4pZTJpQhbR/VI9u5crg9T7CJ0Rn2aJAytM8A?=
 =?us-ascii?Q?3bziz+/KDmA0Kf/lqtiiUTmXq2c=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <EAB112DF5BC71C4DB1D624D48005CAD0@namprd04.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: wdc.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: SJ0PR04MB7184.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 731f316c-4582-4034-76a1-08d9a8dd6771
X-MS-Exchange-CrossTenant-originalarrivaltime: 16 Nov 2021 08:45:12.5119
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 5su4oxBlZ9KH9qFyn1Pdty75Zis2v0YrjHzbwW1CktQbr/3O5m+DJgrY30h3aoNNB/G///R4ROyvhdOdNfXnNmcmOpPWGbvDJIEthIa2VDU=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR04MB7727
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Nov 16, 2021 / 09:02, Vincent Guittot wrote:
> Hi Shinichiro
>=20
> On Mon, 15 Nov 2021 at 19:34, Valentin Schneider
> <valentin.schneider@arm.com> wrote:
> >
> > On 15/11/21 10:46, Shinichiro Kawasaki wrote:
> > > Oh, this is the exactly same fix as I posted before [1]. It is a litt=
le bit sad
> > > that my post did not get reviewed. Anyway, good to see the issue fixe=
d. Thanks.
> > >
> > > [1] https://lore.kernel.org/all/20211029005618.773579-1-shinichiro.ka=
wasaki@wdc.com/
> > >
> >
> > Oh, that sucks, sorry about that. I do try to keep an eye out for sched
> > stuff sent to LKML, but clearly there are some that fall through :(
>=20
> I would advise you to use get_maintainer.pl to make sure to not miss
> to cc someone.
> Like valentin, this patch has been lost in the lkml flow and i haven't
> noticed it

Hi Velentin, Vincent, thank you for the comments.

I have run get_maintainer.pl on may patch, and it listed 9 maintainers and
reviewers. But my patch had only 2 out of the 9 in its To/Cc list, which wa=
s
too small. For next time, I will use get_maintainer.pl and post with full
To/Cc list. Thanks!

--=20
Best Regards,
Shin'ichiro Kawasaki=