On Tue, Jul 09, 2024 at 12:04:02PM +0200, Jocelyn Falempe wrote: > > > On 09/07/2024 11:12, Greg KH wrote: > > On Tue, Jul 09, 2024 at 11:11:35AM +0200, Greg KH wrote: > > > On Tue, Jul 09, 2024 at 10:40:10AM +0200, Jocelyn Falempe wrote: > > > > +config DRM_PANIC_SCREEN_QR_CODE_URL > > > > + string "Base url of the QR code in the panic screen" > > > > + depends on DRM_PANIC_SCREEN_QR_CODE > > > > + help > > > > + This option sets the base url to report the kernel panic. If it's set > > > > + the qr code will contain the url and the kmsg compressed with zlib as > > > > + url parameter. If it's empty, the qr code will contain the kmsg as > > > > + uncompressed text only. > > > > > > meta-comment, should we by default do this on a kernel.org domain so > > > that no specific distro has to worry about hosing this type of web > > > service? > > > > Also, do you have the backend source for this to show how anyone can > > host it themselves as well? We can't add features to the kernel that no > > one but closed-source implementations will use for obvious reasons. > > I've made a proof of concept backend here: > https://github.com/kdj0c/panic_report/ > > And the javascript to decode the kmsg trace is here (under MIT licence): > https://github.com/kdj0c/panic_report/blob/main/docs/panic_report.js SPDX lines are your friend, you might want to look into that for this stuff :) > It uses the pako js library to uncompress the zlib data, which is also under > MIT/Zlib licence https://github.com/nodeca/pako/ Great, can you put that in the Kconfig help area for this option in your next version? > If kernel.org want to host a default service for that, that would be great. > It can be linked with https://bugzilla.kernel.org to easily create a bug, or > look for similar bugs. Someone should at least propose it if this is going to be an option that the kernel supports. thanks, greg k-h
