On Thu, Apr 25, 2024 at 09:08:13PM -0700, Andrew Morton wrote: > The quilt patch titled > Subject: s390/netiucv: remove function pointer cast > has been removed from the -mm tree. Its filename was > s390-netiucv-remove-function-pointer-cast.patch > > This patch was dropped because it was merged into the mm-nonmm-stable branch > of git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm > > ------------------------------------------------------ > From: Nathan Chancellor <nathan@xxxxxxxxxx> > Subject: s390/netiucv: remove function pointer cast > Date: Wed, 17 Apr 2024 11:24:37 -0700 > > Clang warns (or errors with CONFIG_WERROR) after enabling > -Wcast-function-type-strict by default: > > drivers/s390/net/netiucv.c:1716:18: error: cast from 'void (*)(const void *)' to 'void (*)(struct device *)' converts to incompatible function type [-Werror,-Wcast-function-type-strict] > 1716 | dev->release = (void (*)(struct device *))kfree; > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > 1 error generated. > > Add a standalone function to fix the warning properly, which addresses > the root of the warning that these casts are not safe for kCFI. The > comment is not really relevant after this change, so remove it. > > Link: https://lkml.kernel.org/r/20240417-s390-drivers-fix-cast-function-type-v1-3-fd048c9903b0@xxxxxxxxxx > Signed-off-by: Nathan Chancellor <nathan@xxxxxxxxxx> > Reviewed-by: Arnd Bergmann <arnd@xxxxxxxx> > Cc: Alexander Gordeev <agordeev@xxxxxxxxxxxxx> > Cc: Alexandra Winter <wintera@xxxxxxxxxxxxx> > Cc: Christian Borntraeger <borntraeger@xxxxxxxxxxxxx> > Cc: Heiko Carstens <hca@xxxxxxxxxxxxx> > Cc: Sven Schnelle <svens@xxxxxxxxxxxxx> > Cc: Thorsten Winkler <twinkler@xxxxxxxxxxxxx> > Cc: Vasily Gorbik <gor@xxxxxxxxxxxxx> > Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> > --- > > drivers/s390/net/netiucv.c | 14 ++++++-------- > 1 file changed, 6 insertions(+), 8 deletions(-) And this one should be dropped as well.
