[PATCH 05/19] drm: Allow the caller to pass in the format info to drm_helper_mode_fill_fb_struct()

2025-04-10 Thread Ville Syrjala
From: Ville Syrjälä Soon all drivers should have the format info already available in the places where they call drm_helper_mode_fill_fb_struct(). Allow it to be passed along into drm_helper_mode_fill_fb_struct() instead of doing yet another redundant lookup. Start by always passing in NULL and

[PATCH 04/19] drm: Pass the format info to .fb_create()

2025-04-10 Thread Ville Syrjala
From: Ville Syrjälä Pass long the format information from the top to .fb_create() so that we can avoid redundant (and somewhat expensive) lookups in the drivers. Done with cocci (with some manual fixups): @@ identifier func =~ ".*create.*"; identifier dev, file, mode_cmd; @@ struct drm_framebuff

Re: [PATCH v4 3/5] iommu/virtio: Move to domain_alloc_paging()

2025-04-10 Thread Jason Gunthorpe
On Thu, Apr 10, 2025 at 05:55:50AM +, Tian, Kevin wrote: > > From: Jason Gunthorpe > > Sent: Wednesday, April 9, 2025 12:36 AM > > > > +static struct iommu_domain *viommu_domain_alloc_identity(struct device > > *dev) > > +{ > > + struct viommu_endpoint *vdev = dev_iommu_priv_get(dev); > > +

RE: [PATCH v4 3/5] iommu/virtio: Move to domain_alloc_paging()

2025-04-10 Thread Tian, Kevin
> From: Jason Gunthorpe > Sent: Wednesday, April 9, 2025 12:36 AM > > +static struct iommu_domain *viommu_domain_alloc_identity(struct device > *dev) > +{ > + struct viommu_endpoint *vdev = dev_iommu_priv_get(dev); > + struct iommu_domain *domain; > + int ret; > + > + if (virtio_h

Re: [PATCH 1/2] dma-fence: Rename dma_fence_is_signaled()

2025-04-10 Thread Christian König
Am 09.04.25 um 17:04 schrieb Philipp Stanner: > On Wed, 2025-04-09 at 16:10 +0200, Christian König wrote: >>> I only see improvement by making things more obvious. >>> >>> In any case, how would you call a wrapper that just does >>> test_bit(IS_SIGNALED, …) ? >> Broken, that was very intentionally

Re: [PATCH 04/19] drm: Pass the format info to .fb_create()

2025-04-10 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Thu, Apr 10, 2025 at 07:32:03PM +0300, Ville Syrjala wrote: > From: Ville Syrjälä > > Pass long the format information from the top to .fb_create() > so that we can avoid redundant (and somewhat expensive) lookups > in the drivers. > > Done with cocci (wit

Re: [PATCH 05/19] drm: Allow the caller to pass in the format info to drm_helper_mode_fill_fb_struct()

2025-04-10 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Thu, Apr 10, 2025 at 07:32:04PM +0300, Ville Syrjala wrote: > From: Ville Syrjälä > > Soon all drivers should have the format info already available in the > places where they call drm_helper_mode_fill_fb_struct(). Allow it to > be passed along into drm_hel

Re: [PATCH 04/19] drm: Pass the format info to .fb_create()

2025-04-10 Thread Geert Uytterhoeven
On Thu, 10 Apr 2025 at 18:33, Ville Syrjala wrote: > From: Ville Syrjälä > > Pass long the format information from the top to .fb_create() s/long/along/ > so that we can avoid redundant (and somewhat expensive) lookups > in the drivers. [...] > Signed-off-by: Ville Syrjälä > drivers/gpu/dr

RE: [PATCH v4 2/5] iommu: Add domain_alloc_identity()

2025-04-10 Thread Tian, Kevin
> From: Jason Gunthorpe > Sent: Wednesday, April 9, 2025 12:36 AM > > + * @domain_alloc_identity: allocate an IDENTITY domain. Drivers should > prefer to > + * use identity_domain instead. This should only be > used > + * if dynamic logic is necess

[PATCH v4 1/5] iommu/virtio: Break out bypass identity support into a global static

2025-04-10 Thread Jason Gunthorpe
To make way for a domain_alloc_paging conversion add the typical global static IDENTITY domain. This supports VMMs that have a VIRTIO_IOMMU_F_BYPASS_CONFIG config. If the VMM does not have support then the domain_alloc path is still used, which creates an IDENTITY domain out of a paging domain. R

Re: [PATCH v4 5/5] iommu: Hide ops.domain_alloc behind CONFIG_FSL_PAMU

2025-04-10 Thread Jason Gunthorpe
On Thu, Apr 10, 2025 at 05:57:44AM +, Tian, Kevin wrote: > > From: Jason Gunthorpe > > Sent: Wednesday, April 9, 2025 12:36 AM > > > > +#if IS_ENABLED(CONFIG_FSL_PAMU) > > struct iommu_domain *(*domain_alloc)(unsigned > > iommu_domain_type); > > +#endif > > what about directly calling it

Re: [PATCH v2 0/3] arm-smmu: select suitable IOVA

2025-04-10 Thread Jason Gunthorpe
On Thu, Apr 10, 2025 at 03:50:27PM -0700, Shyam Saini wrote: > > Hi, > > Currently, the MSI_IOVA_BASE address is hard-coded to 0x8000, > assuming that all platforms have this address available for MSI IOVA > reservation. However, this is not always the case, as some platforms > reserve this a

Re: [PATCH 1/2] dma-fence: Rename dma_fence_is_signaled()

2025-04-10 Thread Christian König
Am 09.04.25 um 14:56 schrieb Philipp Stanner: > On Wed, 2025-04-09 at 14:51 +0200, Philipp Stanner wrote: >> On Wed, 2025-04-09 at 14:39 +0200, Boris Brezillon wrote: >>> Hi Philipp, >>> >>> On Wed,  9 Apr 2025 14:06:37 +0200 >>> Philipp Stanner wrote: >>> dma_fence_is_signaled()'s name stron

[PATCH v2 3/3] arm-smmu: select suitable MSI IOVA

2025-04-10 Thread Shyam Saini
Currently ARM SMMU drivers hardcode PCI MSI IOVA address. Not all the platform have same memory mappings and some platform could have this address already being mapped for something else. This can lead to collision and as a consequence the MSI IOVA addr range is never reserved. Fix this by adding

[PATCH v2 1/3] arm-smmu: move MSI_IOVA macro definitions

2025-04-10 Thread Shyam Saini
MSI_IOVA* are common among different iommu/smu drivers, so move them to common iommu.h header file. Suggested-by: Jacob Pan Signed-off-by: Shyam Saini --- drivers/iommu/arm/arm-smmu-v3/arm-smmu-v3.h | 3 --- drivers/iommu/arm/arm-smmu/arm-smmu.c | 3 --- drivers/iommu/virtio-iommu.c

[PATCH v2 2/3] dt-bindings: iommu: add "arm,smmu-faulty-msi-iova" property

2025-04-10 Thread Shyam Saini
By default ARM SMMU drivers use MSI_IOVA_BASE macro to reserve PCI MSI IOVA memory range, this assumes that all the platforms have MSI_IOVA_BASE address available for MSI reservation. However, this is not always the case, as some platforms may have the default address reserved for some other purpos

[PATCH v2 0/3] arm-smmu: select suitable IOVA

2025-04-10 Thread Shyam Saini
Hi, Currently, the MSI_IOVA_BASE address is hard-coded to 0x8000, assuming that all platforms have this address available for MSI IOVA reservation. However, this is not always the case, as some platforms reserve this address for other purposes. Consequently, these platforms cannot reserve th

Re: [PATCH 1/2] dma-fence: Rename dma_fence_is_signaled()

2025-04-10 Thread Boris Brezillon
Hi Philipp, On Wed, 9 Apr 2025 14:06:37 +0200 Philipp Stanner wrote: > dma_fence_is_signaled()'s name strongly reads as if this function were > intended for checking whether a fence is already signaled. Also the > boolean it returns hints at that. > > The function's behavior, however, is more

[PATCH net-next 2/2] vsock/test: Expand linger test to ensure close() does not misbehave

2025-04-10 Thread Michal Luczaj
Add a check to alert on close() lingering when it should not. Signed-off-by: Michal Luczaj --- tools/testing/vsock/vsock_test.c | 30 +++--- 1 file changed, 27 insertions(+), 3 deletions(-) diff --git a/tools/testing/vsock/vsock_test.c b/tools/testing/vsock/vsock_test.c