All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCHv3 0/6] make vendor specific subdirectory inclusion conditional
@ 2025-05-12 13:07 Rolf Eike Beer
  2025-05-12 13:10 ` [PATCH 1/6] iommu: remove duplicate selection of DMAR_TABLE Rolf Eike Beer
                   ` (6 more replies)
  0 siblings, 7 replies; 14+ messages in thread
From: Rolf Eike Beer @ 2025-05-12 13:07 UTC (permalink / raw)
  To: Joerg Roedel, Will Deacon
  Cc: iommu, linux-kernel, Bert Karwatzki, Lu Baolu, Jerry Snitselaar,
	Suravee Suthikulpanit, Tomasz Jeznach, Nick Kossifidis,
	Sebastien Boeuf, Palmer Dabbelt, Paul Walmsley, Albert Ou,
	Alexandre Ghiti, Robin Murphy, linux-arm-kernel, Jason Gunthorpe,
	Kevin Tian

[-- Attachment #1: Type: text/plain, Size: 1119 bytes --]

I found it strange that the vendor specific subdirectories were unconditionally 
added. Everything in these directories obviously depends on the specific Kconfig 
option. This series cleans this up by making the directory itself depend on 
the respective option which looks more natural to me.

I must confess that I am too lazy to find a good solution for the "arm/" 
directory, so I left it as is.

Regards,

Eike

---
Changes in v3:
- added Reviewed-by: Lu Baolu <baolu.lu@linux.intel.com>
- rebased on top of 2e9b2ee2ba403cbe270a8256b8794ef5ad19b38d

Changes in v2:
- clean up subdirectory Makefiles to use obj-y now
- add arm/arm-smmu-v3 and iommufd patches
-- 
Rolf Eike Beer

emlix GmbH
Headquarters: Berliner Str. 12, 37073 Göttingen, Germany
Phone +49 (0)551 30664-0, e-mail info@emlix.com
District Court of Göttingen, Registry Number HR B 3160
Managing Directors: Heike Jordan, Dr. Uwe Kracke
VAT ID No. DE 205 198 055
Office Berlin: Panoramastr. 1, 10178 Berlin, Germany
Office Bonn: Bachstr. 6, 53115 Bonn, Germany
http://www.emlix.com

emlix - your embedded Linux partner

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 313 bytes --]

^ permalink raw reply	[flat|nested] 14+ messages in thread
* Re: [PATCH 6/6] iommu: make inclusion of iommufd directory conditional
@ 2025-05-13  2:50 kernel test robot
  0 siblings, 0 replies; 14+ messages in thread
From: kernel test robot @ 2025-05-13  2:50 UTC (permalink / raw)
  To: oe-kbuild; +Cc: lkp

:::::: 
:::::: Manual check reason: "only suspicious fbc files changed"
:::::: 

BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
In-Reply-To: <5568089.Sb9uPGUboI@devpool92.emlix.com>
References: <5568089.Sb9uPGUboI@devpool92.emlix.com>
TO: Rolf Eike Beer <eb@emlix.com>
TO: Joerg Roedel <joro@8bytes.org>
TO: Will Deacon <will@kernel.org>
CC: iommu@lists.linux.dev
CC: linux-kernel@vger.kernel.org
CC: Robin Murphy <robin.murphy@arm.com>
CC: Jason Gunthorpe <jgg@ziepe.ca>

Hi Rolf,

kernel test robot noticed the following build errors:

[auto build test ERROR on next-20250512]
[also build test ERROR on v6.15-rc6]
[cannot apply to arm-perf/for-next/perf linus/master v6.15-rc6 v6.15-rc5 v6.15-rc4]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Rolf-Eike-Beer/iommu-remove-duplicate-selection-of-DMAR_TABLE/20250512-212423
base:   next-20250512
patch link:    https://lore.kernel.org/r/5568089.Sb9uPGUboI%40devpool92.emlix.com
patch subject: [PATCH 6/6] iommu: make inclusion of iommufd directory conditional
:::::: branch date: 13 hours ago
:::::: commit date: 13 hours ago
config: arm64-randconfig-003-20250513 (https://download.01.org/0day-ci/archive/20250513/202505131001.hEIJFgMF-lkp@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 8.5.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250513/202505131001.hEIJFgMF-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/r/202505131001.hEIJFgMF-lkp@intel.com/

All errors (new ones prefixed by >>):

   aarch64-linux-ld: Unexpected GOT/PLT entries detected!
   aarch64-linux-ld: Unexpected run-time procedure linkages detected!
   aarch64-linux-ld: drivers/vfio/vfio_main.o: in function `vfio_ioctl_device_feature':
>> vfio_main.c:(.text+0x34fc): undefined reference to `iova_bitmap_alloc'
>> aarch64-linux-ld: vfio_main.c:(.text+0x351c): undefined reference to `iova_bitmap_for_each'
>> aarch64-linux-ld: vfio_main.c:(.text+0x3528): undefined reference to `iova_bitmap_free'
   aarch64-linux-ld: drivers/vfio/pci/pds/dirty.o: in function `pds_vfio_dma_logging_report':
>> dirty.c:(.text+0x13c4): undefined reference to `iova_bitmap_set'

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

^ permalink raw reply	[flat|nested] 14+ messages in thread
* [PATCHv2 0/6] make vendor specific subdirectory inclusion conditional
@ 2025-03-20  9:01 Rolf Eike Beer
  2025-03-20  9:22 ` [PATCH 6/6] iommu: make inclusion of iommufd directory conditional Rolf Eike Beer
  0 siblings, 1 reply; 14+ messages in thread
From: Rolf Eike Beer @ 2025-03-20  9:01 UTC (permalink / raw)
  To: Joerg Roedel, Will Deacon
  Cc: iommu, linux-kernel, Bert Karwatzki, Lu Baolu, Jerry Snitselaar,
	Suravee Suthikulpanit, Tomasz Jeznach, Nick Kossifidis,
	Sebastien Boeuf, Palmer Dabbelt, Paul Walmsley, Albert Ou,
	Alexandre Ghiti, Robin Murphy, linux-arm-kernel, Jason Gunthorpe,
	Kevin Tian

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

I found it strange that the vendor specific subdirectories were unconditionally 
added. Everything in these directories obviously depends on the specific Kconfig 
option. This series cleans this up by making the directory itself depend on 
the respective option which looks more natural to me.

I must confess that I am too lazy to find a good solution for the "arm/" 
directory, so I left it as is.

Regards,

Eike

---
Changes in v2:
- clean up subdirectory Makefiles to use obj-y now
- add arm/arm-smmu-v3 and iommufd patches
-- 
Rolf Eike Beer

emlix GmbH
Headquarters: Berliner Str. 12, 37073 Göttingen, Germany
Phone +49 (0)551 30664-0, e-mail info@emlix.com
District Court of Göttingen, Registry Number HR B 3160
Managing Directors: Heike Jordan, Dr. Uwe Kracke
VAT ID No. DE 205 198 055
Office Berlin: Panoramastr. 1, 10178 Berlin, Germany
Office Bonn: Bachstr. 6, 53115 Bonn, Germany
http://www.emlix.com

emlix - your embedded Linux partner

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 313 bytes --]

^ permalink raw reply	[flat|nested] 14+ messages in thread
end of thread, other threads:[~2025-05-26  5:59 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-05-12 13:07 [PATCHv3 0/6] make vendor specific subdirectory inclusion conditional Rolf Eike Beer
2025-05-12 13:10 ` [PATCH 1/6] iommu: remove duplicate selection of DMAR_TABLE Rolf Eike Beer
2025-05-12 13:11 ` [PATCH 2/6] iommu: make inclusion of intel directory conditional Rolf Eike Beer
2025-05-12 13:12 ` [PATCH 3/6] iommu: make inclusion of amd " Rolf Eike Beer
2025-05-12 13:13 ` [PATCH 4/6] iommu: make inclusion of riscv " Rolf Eike Beer
2025-05-12 13:14 ` [PATCH 5/6] iommu: make inclusion of arm/arm-smmu-v3 " Rolf Eike Beer
2025-05-12 13:15 ` [PATCH 6/6] iommu: make inclusion of iommufd " Rolf Eike Beer
2025-05-16 12:52   ` Joerg Roedel
2025-05-19  6:16     ` Rolf Eike Beer
2025-05-26  5:59     ` Rolf Eike Beer
2025-05-16  6:47 ` [PATCHv3 0/6] make vendor specific subdirectory inclusion conditional Joerg Roedel
  -- strict thread matches above, loose matches on Subject: below --
2025-05-13  2:50 [PATCH 6/6] iommu: make inclusion of iommufd directory conditional kernel test robot
2025-03-20  9:01 [PATCHv2 0/6] make vendor specific subdirectory inclusion conditional Rolf Eike Beer
2025-03-20  9:22 ` [PATCH 6/6] iommu: make inclusion of iommufd directory conditional Rolf Eike Beer
2025-03-21  2:14   ` Baolu Lu

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.