[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <2024070407-pregnancy-hazing-add9@gregkh>
Date: Thu, 4 Jul 2024 11:22:32 +0200
From: Greg KH <gregkh@...uxfoundation.org>
To: Alice Ryhl <aliceryhl@...gle.com>
Cc: jfalempe@...hat.com, a.hindborg@...sung.com, airlied@...il.com,
alex.gaynor@...il.com, benno.lossin@...ton.me,
bjorn3_gh@...tonmail.com, boqun.feng@...il.com, dakr@...hat.com,
daniel@...ll.ch, dri-devel@...ts.freedesktop.org, gary@...yguo.net,
linux-kernel@...r.kernel.org, maarten.lankhorst@...ux.intel.com,
mripard@...nel.org, ojeda@...nel.org,
rust-for-linux@...r.kernel.org, tzimmermann@...e.de,
wedsonaf@...il.com
Subject: Re: [PATCH 4/4] drm/panic: Add a qr_code panic screen
On Thu, Jul 04, 2024 at 09:11:09AM +0000, Alice Ryhl wrote:
> As long as the above requirements are satisfied, calling
> `drm_panic_qr_generate` should never cause memory unsafety, so this is
> an appropriate list of safety requirements.
>
> (You also require that `data_len <= data_size`, but if this is violated
> you get a kernel panic which isn't a memory safety problem, so it does
> not need to be listed in the safety requirements.)
Panicing while trying to print out a panic message is not a good idea,
so enforcing this would be nice :)
thanks,
greg k-h
Powered by blists - more mailing lists