lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <7aa99a5f-306b-4c9b-88d2-9ea93c013eda@gmx.de>
Date: Fri, 30 Aug 2024 12:19:08 +0200
From: Helge Deller <deller@....de>
To: Thomas Weißschuh <linux@...ssschuh.net>,
 Daniel Vetter <daniel@...ll.ch>
Cc: linux-fbdev@...r.kernel.org, dri-devel@...ts.freedesktop.org,
 linux-kernel@...r.kernel.org, Bert Karwatzki <spasswolf@....de>,
 "Borah, Chaitanya Kumar" <chaitanya.kumar.borah@...el.com>,
 intel-gfx@...ts.freedesktop.org,
 "Kurmi, Suresh Kumar" <suresh.kumar.kurmi@...el.com>,
 "Saarinen, Jani" <jani.saarinen@...el.com>
Subject: Re: [PATCH] fbdev: Introduce devm_register_framebuffer()

On 8/30/24 12:16, Helge Deller wrote:
> On 8/30/24 11:45, Thomas Weißschuh wrote:
>> Introduce a device-managed variant of register_framebuffer() which
>> automatically unregisters the framebuffer on device destruction.
>> This can simplify the error handling and resource management in drivers.
>>
>> Signed-off-by: Thomas Weißschuh <linux@...ssschuh.net>
>> ---
>> This is a fixed resend of [0], which was broken.
>> Thanks to Bert [1], and Chaitanya Kumar [2]
>> for reporting the issue.
>>
>> [0] https://lore.kernel.org/lkml/20240827-efifb-sysfs-v1-3-c9cc3e052180@weissschuh.net/
>> [1] https://lore.kernel.org/lkml/20240829224124.2978-1-spasswolf@web.de/
>> [2] https://lore.kernel.org/lkml/SJ1PR11MB612925C1C533C09F8F62F7CBB9972@SJ1PR11MB6129.namprd11.prod.outlook.com/
>
> I've applied this patch to the fbdev git tree.
> Please double check at
> https://git.kernel.org/pub/scm/linux/kernel/git/deller/linux-fbdev.git/log/?h=for-next
>
> Can you please check if this fixes this new report too:
> https://marc.info/?l=linux-fbdev&m=172500784802901&w=2

Please ignore this ^^^.  You already mentioned this one above.
Helge


>> Helge, I didn't document the function devm_unregister_framebuffer() as
>> it is only an internal helper and will ever only used by one user,
>> similar to other helpers in fbmem.c.
>
> Ok.
>
> Helge


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ