Re: [PATCH 00/23] i915/display: split and constify vtable (v3)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi, 

> -----Original Message-----
> From: Intel-gfx <intel-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of Dave Airlie
> Sent: torstai 9. syyskuuta 2021 4.53
> To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> Cc: jani.nikula@xxxxxxxxxxxxxxx
> Subject:  [PATCH 00/23] i915/display: split and constify vtable (v3)
> 
> (v3 just adds some missing ,)
> 
>       Details below, I've taken all the review feedback (thanks Jani).
> I added 3 patches moving to wrappers before refactoring, and one other patch is
> unreviewed (07) but the main comment was wanting the wrappers.
> 
> Jani if you are happy with the final 4 patches can you land this series, I don't think I
> have drm-intel commit rights.
Not right Jani to answer but in my view CI in not healthy at all if looking from: 
https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20995/index.html? 
See taints in the boot. 

> 
> v1:
> This is orthogonal to my display ptr refactoring and should probably be applied first.
> 
> The display funcs vtable was a bit of mess, lots of intermixing of internal display
> functionality and interfaces to watermarks/irqs.
> 
> It's also considered not great security practice to leave writeable function pointers
> around for exploits to get into.
> 
> This series attempts to address both problems, first there are a few cleanups, then it
> splits the function table into multiple pieces.
> Some of the splits might be bikesheds but I think we should apply first and merge
> things later if there is good reason.
> 
> The second half converts all the vtables to static const structs, I've used macros in
> some of them to make it less messy, the cdclk one is probably the worst one.
> 
> v2:
> Added some patches adding wrappers around things before refactoring them as
> suggested by Jani.
> Fixed up all struct names as suggested by Jani.
> Added s-o-b lines
> Added commit msgs.
> 
> v3:
> added missing , (Jani)
> 
> Dave.
> 





[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux