All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jason Gunthorpe <jgg@nvidia.com>
To: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Christoph Hellwig <hch@lst.de>,
	Jani Nikula <jani.nikula@linux.intel.com>,
	Rodrigo Vivi <rodrigo.vivi@intel.com>,
	Zhenyu Wang <zhenyuw@linux.intel.com>,
	Zhi Wang <zhi.a.wang@intel.com>,
	intel-gfx@lists.freedesktop.org,
	intel-gvt-dev@lists.freedesktop.org,
	dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 02/29] drm/i915/gvt: integrate into the main Makefile
Date: Thu, 4 Nov 2021 09:32:25 -0300	[thread overview]
Message-ID: <20211104123225.GV2744544@nvidia.com> (raw)
In-Reply-To: <163602902009.4807.3745093259631583283@jlahtine-mobl.ger.corp.intel.com>

On Thu, Nov 04, 2021 at 02:30:20PM +0200, Joonas Lahtinen wrote:
> Quoting Christoph Hellwig (2021-11-02 09:05:34)
> > Remove the separately included Makefile and just use the relative
> > reference from the main i915 Makefile as for source files in other
> > subdirectories.
> 
> The thinking behind the split is to avoid any merge conflicts as the
> gvt/ subdirectory is handled through separate pull request flow and
> are note part of drm-tip.

Oh? In that case can we eventually move the VFIO mdev driver to
drivers/vfio/mdev/intel_gvt/ please?

Jason

WARNING: multiple messages have this Message-ID (diff)
From: Jason Gunthorpe <jgg@nvidia.com>
To: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org,
	linux-kernel@vger.kernel.org,
	Rodrigo Vivi <rodrigo.vivi@intel.com>,
	intel-gvt-dev@lists.freedesktop.org,
	Christoph Hellwig <hch@lst.de>, Zhi Wang <zhi.a.wang@intel.com>
Subject: Re: [PATCH 02/29] drm/i915/gvt: integrate into the main Makefile
Date: Thu, 4 Nov 2021 09:32:25 -0300	[thread overview]
Message-ID: <20211104123225.GV2744544@nvidia.com> (raw)
In-Reply-To: <163602902009.4807.3745093259631583283@jlahtine-mobl.ger.corp.intel.com>

On Thu, Nov 04, 2021 at 02:30:20PM +0200, Joonas Lahtinen wrote:
> Quoting Christoph Hellwig (2021-11-02 09:05:34)
> > Remove the separately included Makefile and just use the relative
> > reference from the main i915 Makefile as for source files in other
> > subdirectories.
> 
> The thinking behind the split is to avoid any merge conflicts as the
> gvt/ subdirectory is handled through separate pull request flow and
> are note part of drm-tip.

Oh? In that case can we eventually move the VFIO mdev driver to
drivers/vfio/mdev/intel_gvt/ please?

Jason

WARNING: multiple messages have this Message-ID (diff)
From: Jason Gunthorpe <jgg@nvidia.com>
To: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org,
	linux-kernel@vger.kernel.org,
	intel-gvt-dev@lists.freedesktop.org,
	Christoph Hellwig <hch@lst.de>
Subject: Re: [Intel-gfx] [PATCH 02/29] drm/i915/gvt: integrate into the main Makefile
Date: Thu, 4 Nov 2021 09:32:25 -0300	[thread overview]
Message-ID: <20211104123225.GV2744544@nvidia.com> (raw)
In-Reply-To: <163602902009.4807.3745093259631583283@jlahtine-mobl.ger.corp.intel.com>

On Thu, Nov 04, 2021 at 02:30:20PM +0200, Joonas Lahtinen wrote:
> Quoting Christoph Hellwig (2021-11-02 09:05:34)
> > Remove the separately included Makefile and just use the relative
> > reference from the main i915 Makefile as for source files in other
> > subdirectories.
> 
> The thinking behind the split is to avoid any merge conflicts as the
> gvt/ subdirectory is handled through separate pull request flow and
> are note part of drm-tip.

Oh? In that case can we eventually move the VFIO mdev driver to
drivers/vfio/mdev/intel_gvt/ please?

Jason

  reply	other threads:[~2021-11-04 12:32 UTC|newest]

Thread overview: 172+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-11-02  7:05 refactor the i915 GVT support and move to the modern mdev API v2 Christoph Hellwig
2021-11-02  7:05 ` [Intel-gfx] " Christoph Hellwig
2021-11-02  7:05 ` [PATCH 01/29] drm/i915/gvt: undef TRACE_INCLUDE_FILE in i915_trace.h Christoph Hellwig
2021-11-02  7:05   ` [Intel-gfx] " Christoph Hellwig
2021-11-02  7:05 ` [PATCH 02/29] drm/i915/gvt: integrate into the main Makefile Christoph Hellwig
2021-11-02  7:05   ` [Intel-gfx] " Christoph Hellwig
2021-11-02 13:56   ` Jason Gunthorpe
2021-11-02 13:56     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02 13:56     ` Jason Gunthorpe
2021-11-02 14:55   ` Jani Nikula
2021-11-02 14:55     ` Jani Nikula
2021-11-02 14:55     ` [Intel-gfx] " Jani Nikula
2021-11-04 12:30   ` Joonas Lahtinen
2021-11-04 12:30     ` [Intel-gfx] " Joonas Lahtinen
2021-11-04 12:30     ` Joonas Lahtinen
2021-11-04 12:32     ` Jason Gunthorpe [this message]
2021-11-04 12:32       ` [Intel-gfx] " Jason Gunthorpe
2021-11-04 12:32       ` Jason Gunthorpe
2021-11-09  7:56       ` Christoph Hellwig
2021-11-09  7:56         ` [Intel-gfx] " Christoph Hellwig
2021-11-09  7:55     ` Christoph Hellwig
2021-11-09  7:55       ` [Intel-gfx] " Christoph Hellwig
2021-11-02  7:05 ` [PATCH 03/29] drm/i915/gvt: remove module refcounting in intel_gvt_{,un}register_hypervisor Christoph Hellwig
2021-11-02  7:05   ` [Intel-gfx] [PATCH 03/29] drm/i915/gvt: remove module refcounting in intel_gvt_{, un}register_hypervisor Christoph Hellwig
2021-11-02 15:36   ` [PATCH 03/29] drm/i915/gvt: remove module refcounting in intel_gvt_{,un}register_hypervisor Jason Gunthorpe
2021-11-02 15:36     ` [Intel-gfx] [PATCH 03/29] drm/i915/gvt: remove module refcounting in intel_gvt_{, un}register_hypervisor Jason Gunthorpe
2021-11-02 15:36     ` [PATCH 03/29] drm/i915/gvt: remove module refcounting in intel_gvt_{,un}register_hypervisor Jason Gunthorpe
2021-11-02  7:05 ` [PATCH 04/29] drm/i915/gvt: remove enum hypervisor_type Christoph Hellwig
2021-11-02  7:05   ` [Intel-gfx] " Christoph Hellwig
2021-11-02 15:37   ` Jason Gunthorpe
2021-11-02 15:37     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02 15:37     ` Jason Gunthorpe
2021-11-02  7:05 ` [Intel-gfx] [PATCH 05/29] drm/i915/gvt: rename intel_vgpu_ops to intel_vgpu_mdev_ops Christoph Hellwig
2021-11-02  7:05   ` Christoph Hellwig
2021-11-02 15:37   ` Jason Gunthorpe
2021-11-02 15:37     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02 15:37     ` Jason Gunthorpe
2021-11-02  7:05 ` [Intel-gfx] [PATCH 06/29] drm/i915/gvt: move the gvt code into kvmgt.ko Christoph Hellwig
2021-11-02  7:05   ` Christoph Hellwig
2021-11-04 12:54   ` Joonas Lahtinen
2021-11-04 12:54     ` [Intel-gfx] " Joonas Lahtinen
2021-11-02  7:05 ` [PATCH 07/29] drm/i915/gvt: remove intel_gvt_ops Christoph Hellwig
2021-11-02  7:05   ` [Intel-gfx] " Christoph Hellwig
2021-11-02 16:03   ` Jason Gunthorpe
2021-11-02 16:03     ` Jason Gunthorpe
2021-11-02 16:03     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02  7:05 ` [PATCH 08/29] drm/i915/gvt: remove the map_gfn_to_mfn and set_trap_area ops Christoph Hellwig
2021-11-02  7:05   ` [Intel-gfx] " Christoph Hellwig
2021-11-02 16:04   ` Jason Gunthorpe
2021-11-02 16:04     ` Jason Gunthorpe
2021-11-02 16:04     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02  7:05 ` [PATCH 09/29] drm/i915/gvt: remove the unused from_virt_to_mfn op Christoph Hellwig
2021-11-02  7:05   ` [Intel-gfx] " Christoph Hellwig
2021-11-02 16:04   ` Jason Gunthorpe
2021-11-02 16:04     ` Jason Gunthorpe
2021-11-02 16:04     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02  7:05 ` [PATCH 10/29] drm/i915/gvt: merge struct kvmgt_vdev into struct intel_vgpu Christoph Hellwig
2021-11-02  7:05   ` [Intel-gfx] " Christoph Hellwig
2021-11-02 16:05   ` Jason Gunthorpe
2021-11-02 16:05     ` Jason Gunthorpe
2021-11-02 16:05     ` Jason Gunthorpe
2021-11-02  7:05 ` [PATCH 11/29] drm/i915/gvt: merge struct kvmgt_guest_info into strut intel_vgpu Christoph Hellwig
2021-11-02  7:05   ` [Intel-gfx] " Christoph Hellwig
2021-11-02 16:06   ` Jason Gunthorpe
2021-11-02 16:06     ` Jason Gunthorpe
2021-11-02 16:06     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02  7:05 ` [PATCH 12/29] drm/i915/gvt: remove vgpu->handle Christoph Hellwig
2021-11-02  7:05   ` [Intel-gfx] " Christoph Hellwig
2021-11-02 16:07   ` Jason Gunthorpe
2021-11-02 16:07     ` Jason Gunthorpe
2021-11-02 16:07     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02  7:05 ` [PATCH 13/29] drm/i915/gvt: devirtualize ->{read,write}_gpa Christoph Hellwig
2021-11-02  7:05   ` [Intel-gfx] [PATCH 13/29] drm/i915/gvt: devirtualize ->{read, write}_gpa Christoph Hellwig
2021-11-02 16:08   ` [PATCH 13/29] drm/i915/gvt: devirtualize ->{read,write}_gpa Jason Gunthorpe
2021-11-02 16:08     ` Jason Gunthorpe
2021-11-02 16:08     ` [Intel-gfx] [PATCH 13/29] drm/i915/gvt: devirtualize ->{read, write}_gpa Jason Gunthorpe
2021-11-02  7:05 ` [Intel-gfx] [PATCH 14/29] drm/i915/gvt: devirtualize ->{get, put}_vfio_device Christoph Hellwig
2021-11-02  7:05   ` [PATCH 14/29] drm/i915/gvt: devirtualize ->{get,put}_vfio_device Christoph Hellwig
2021-11-02 16:12   ` Jason Gunthorpe
2021-11-02 16:12     ` Jason Gunthorpe
2021-11-02 16:12     ` [Intel-gfx] [PATCH 14/29] drm/i915/gvt: devirtualize ->{get, put}_vfio_device Jason Gunthorpe
2021-11-02  7:05 ` [Intel-gfx] [PATCH 15/29] drm/i915/gvt: devirtualize ->set_edid and ->set_opregion Christoph Hellwig
2021-11-02  7:05   ` Christoph Hellwig
2021-11-02 16:12   ` Jason Gunthorpe
2021-11-02 16:12     ` Jason Gunthorpe
2021-11-02 16:12     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02  7:05 ` [PATCH 16/29] drm/i915/gvt: devirtualize ->detach_vgpu Christoph Hellwig
2021-11-02  7:05   ` [Intel-gfx] " Christoph Hellwig
2021-11-02 16:13   ` Jason Gunthorpe
2021-11-02 16:13     ` Jason Gunthorpe
2021-11-02 16:13     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02  7:05 ` [PATCH 17/29] drm/i915/gvt: devirtualize ->inject_msi Christoph Hellwig
2021-11-02  7:05   ` [Intel-gfx] " Christoph Hellwig
2021-11-02 16:13   ` Jason Gunthorpe
2021-11-02 16:13     ` Jason Gunthorpe
2021-11-02 16:13     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02  7:05 ` [PATCH 18/29] drm/i915/gvt: devirtualize ->is_valid_gfn Christoph Hellwig
2021-11-02  7:05   ` [Intel-gfx] " Christoph Hellwig
2021-11-02 16:14   ` Jason Gunthorpe
2021-11-02 16:14     ` Jason Gunthorpe
2021-11-02 16:14     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02  7:05 ` [Intel-gfx] [PATCH 19/29] drm/i915/gvt: devirtualize ->gfn_to_mfn Christoph Hellwig
2021-11-02  7:05   ` Christoph Hellwig
2021-11-02 16:14   ` Jason Gunthorpe
2021-11-02 16:14     ` Jason Gunthorpe
2021-11-02 16:14     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02  7:05 ` [Intel-gfx] [PATCH 20/29] drm/i915/gvt: devirtualize ->{enable, disable}_page_track Christoph Hellwig
2021-11-02  7:05   ` [PATCH 20/29] drm/i915/gvt: devirtualize ->{enable,disable}_page_track Christoph Hellwig
2021-11-02 16:15   ` [Intel-gfx] [PATCH 20/29] drm/i915/gvt: devirtualize ->{enable, disable}_page_track Jason Gunthorpe
2021-11-02 16:15     ` [PATCH 20/29] drm/i915/gvt: devirtualize ->{enable,disable}_page_track Jason Gunthorpe
2021-11-02 16:15     ` Jason Gunthorpe
2021-11-02  7:05 ` [Intel-gfx] [PATCH 21/29] drm/i915/gvt: devirtualize ->dma_{, un}map_guest_page Christoph Hellwig
2021-11-02  7:05   ` [PATCH 21/29] drm/i915/gvt: devirtualize ->dma_{,un}map_guest_page Christoph Hellwig
2021-11-02 16:17   ` Jason Gunthorpe
2021-11-02 16:17     ` Jason Gunthorpe
2021-11-02 16:17     ` [Intel-gfx] [PATCH 21/29] drm/i915/gvt: devirtualize ->dma_{, un}map_guest_page Jason Gunthorpe
2021-11-02  7:05 ` [Intel-gfx] [PATCH 22/29] drm/i915/gvt: devirtualize dma_pin_guest_page Christoph Hellwig
2021-11-02  7:05   ` Christoph Hellwig
2021-11-02 16:17   ` Jason Gunthorpe
2021-11-02 16:17     ` Jason Gunthorpe
2021-11-02 16:17     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02  7:05 ` [Intel-gfx] [PATCH 23/29] drm/i915/gvt: remove struct intel_gvt_mpt Christoph Hellwig
2021-11-02  7:05   ` Christoph Hellwig
2021-11-02 16:18   ` Jason Gunthorpe
2021-11-02 16:18     ` Jason Gunthorpe
2021-11-02 16:18     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02  7:05 ` [Intel-gfx] [PATCH 24/29] drm/i915/gvt: remove the extra vfio_device refcounting for dmabufs Christoph Hellwig
2021-11-02  7:05   ` Christoph Hellwig
2021-11-02 16:19   ` Jason Gunthorpe
2021-11-02 16:19     ` Jason Gunthorpe
2021-11-02 16:19     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02  7:05 ` [Intel-gfx] [PATCH 25/29] drm/i915/gvt: streamline intel_vgpu_create Christoph Hellwig
2021-11-02  7:05   ` Christoph Hellwig
2021-11-02 16:19   ` Jason Gunthorpe
2021-11-02 16:19     ` Jason Gunthorpe
2021-11-02 16:19     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02  7:05 ` [Intel-gfx] [PATCH 26/29] drm/i915/gvt: pass a struct intel_vgpu to the vfio read/write helpers Christoph Hellwig
2021-11-02  7:05   ` Christoph Hellwig
2021-11-02 16:20   ` Jason Gunthorpe
2021-11-02 16:20     ` Jason Gunthorpe
2021-11-02 16:20     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02  7:05 ` [Intel-gfx] [PATCH 27/29] drm/i915/gvt: remove kvmgt_guest_{init, exit} Christoph Hellwig
2021-11-02  7:05   ` [PATCH 27/29] drm/i915/gvt: remove kvmgt_guest_{init,exit} Christoph Hellwig
2021-11-02 16:36   ` Jason Gunthorpe
2021-11-02 16:36     ` Jason Gunthorpe
2021-11-02 16:36     ` [Intel-gfx] [PATCH 27/29] drm/i915/gvt: remove kvmgt_guest_{init, exit} Jason Gunthorpe
2021-11-02  7:06 ` [PATCH 28/29] drm/i915/gvt: convert to use vfio_register_group_dev() Christoph Hellwig
2021-11-02  7:06   ` [Intel-gfx] " Christoph Hellwig
2021-11-02 16:41   ` Jason Gunthorpe
2021-11-02 16:41     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02 16:41     ` Jason Gunthorpe
2021-11-03  6:40     ` Christoph Hellwig
2021-11-03  6:40       ` [Intel-gfx] " Christoph Hellwig
2021-11-02  7:06 ` [PATCH 29/29] drm/i915/gvt: merge gvt.c into kvmgvt.c Christoph Hellwig
2021-11-02  7:06   ` [Intel-gfx] " Christoph Hellwig
2021-11-02 16:42   ` Jason Gunthorpe
2021-11-02 16:42     ` [Intel-gfx] " Jason Gunthorpe
2021-11-02 16:42     ` Jason Gunthorpe
2021-11-02  7:28 ` [Intel-gfx] ✗ Fi.CI.BUILD: failure for series starting with [01/29] drm/i915/gvt: undef TRACE_INCLUDE_FILE in i915_trace.h Patchwork
2021-11-04 12:59 ` refactor the i915 GVT support and move to the modern mdev API v2 Joonas Lahtinen
2021-11-04 12:59   ` Joonas Lahtinen
2021-11-04 12:59   ` [Intel-gfx] " Joonas Lahtinen
2021-11-04 14:51   ` Wang, Zhi A
2021-11-04 14:51     ` [Intel-gfx] " Wang, Zhi A
2021-11-04 14:51     ` Wang, Zhi A
2021-11-09  8:01     ` Christoph Hellwig
2021-11-09  8:01       ` [Intel-gfx] " Christoph Hellwig
2021-11-09  7:59   ` Christoph Hellwig
2021-11-09  7:59     ` [Intel-gfx] " Christoph Hellwig
2021-11-10 13:49     ` Joonas Lahtinen
2021-11-10 13:49       ` [Intel-gfx] " Joonas Lahtinen
2021-11-10 13:49       ` Joonas Lahtinen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20211104123225.GV2744544@nvidia.com \
    --to=jgg@nvidia.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=hch@lst.de \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=intel-gvt-dev@lists.freedesktop.org \
    --cc=jani.nikula@linux.intel.com \
    --cc=joonas.lahtinen@linux.intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rodrigo.vivi@intel.com \
    --cc=zhenyuw@linux.intel.com \
    --cc=zhi.a.wang@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.