Re: [PATCH v5 02/15] genpt: Add Documentation/ files

[Jason Gunthorpe <jgg@nvidia.com>]

On Thu, Sep 18, 2025 at 06:55:31AM +0000, Tian, Kevin wrote:
> > From: Jason Gunthorpe <jgg@nvidia.com>
> > Sent: Thursday, September 4, 2025 1:46 AM
> > 
> > +For this to work the .c file for each compilation unit will include both the
> > +format headers and the generic code for the implementation. For instance
> > in an
> > +implementation compilation unit the headers would normally be included
> > as
> > +follows::
> > +
> > +	#include <linux/generic_pt/common.h>
> > +	#include "fmt/defs_amdv1.h"
> > +	#include "pt_defs.h"
> > +	#include "fmt/amdv1.h"
> > +	#include "pt_common.h"
> > +	#include "pt_iter.h"
> > +	#include "IOMMUt_pt.h"  /* The IOMMU implementation */
> 
> "iommu_pt.h"
> 
> btw in reality the C file (e.g. iommu_amdv1.c) is put under fmt/. From that
> angle the path prefix above should be adjusted.

Better?

generic_pt/fmt/iommu_amdv1.c:
	#include <linux/generic_pt/common.h>
	#include "defs_amdv1.h"
	#include "../pt_defs.h"
	#include "amdv1.h"
	#include "../pt_common.h"
	#include "../pt_iter.h"
	#include "../iommu_pt.h"  /* The IOMMU implementation */

Thanks,
Jason