On Wed, Dec 08 2021 at 11:58, Jason Gunthorpe wrote:
> On Mon, Dec 06, 2021 at 11:39:26PM +0100, Thomas Gleixner wrote:
>> Store the properties which are interesting for various places so the MSI
>> descriptor fiddling can be removed.
>>
>> Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
>> ---
>> V2: Use the setter function
>> ---
>> drivers/pci/msi/msi.c | 8 ++++++++
>> 1 file changed, 8 insertions(+)
>
> I took more time to look at this, to summarize my remarks on the other
> patches
>
> I think we don't need properties. The info in the msi_desc can come
> from the pci_dev which we have easy access to. This seems overall
> clearer
I fixed that now.
> The notable one is the sysfs, but that is probably better handled by
> storing a
>
> const char *sysfs_label
>
> in the dev->msi and emitting that instead of computing it.
I just compute is for now via is_pci_dev() and
to_pci_dev()->msi_enabled.
We are still debating to remove the whole thing completely.
Thanks,
tglx
