All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Yongji Xie <xieyongji@bytedance.com>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	Christoph Hellwig <hch@lst.de>,
	virtualization <virtualization@lists.linux-foundation.org>
Subject: Re: [PATCH v2 01/11] genirq/affinity:: Export irq_create_affinity_masks()
Date: Fri, 27 Jan 2023 03:22:16 -0500	[thread overview]
Message-ID: <20230127032039-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <CACycT3utDJFZtWzqCUXJaqRjkCXPMTAi+VJd3g6dw25vWqaduw@mail.gmail.com>

On Mon, Dec 19, 2022 at 05:36:02PM +0800, Yongji Xie wrote:
> On Mon, Dec 19, 2022 at 3:33 PM Michael S. Tsirkin <mst@redhat.com> wrote:
> >
> > On Mon, Dec 05, 2022 at 04:41:17PM +0800, Xie Yongji wrote:
> > > Export irq_create_affinity_masks() so that some modules
> > > can make use of it to implement interrupt affinity
> > > spreading mechanism.
> > >
> > > Signed-off-by: Xie Yongji <xieyongji@bytedance.com>
> >
> > So this got nacked, what's the plan now?
> >
> 
> I‘d like to check with Christoph again first.
> 
> Hi Christoph,
> 
> Jason will post some patches to get rid of the DMA API for vDPA
> simulators. And the irq affinity algorithm is independent of the IRQ
> subsystem IIUC. So could you allow this patch so that we can reuse the
> algorithm to select the best CPU (per-cpu affinity if possible, or at
> least per-node) to run the virtqueue's irq callback.
> 
> Thanks,
> Yongji

I think you need to explain why you are building low level
affinity masks.  what's the plan now?

> > > ---
> > >  kernel/irq/affinity.c | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/kernel/irq/affinity.c b/kernel/irq/affinity.c
> > > index d9a5c1d65a79..f074a7707c6d 100644
> > > --- a/kernel/irq/affinity.c
> > > +++ b/kernel/irq/affinity.c
> > > @@ -487,6 +487,7 @@ irq_create_affinity_masks(unsigned int nvecs, struct irq_affinity *affd)
> > >
> > >       return masks;
> > >  }
> > > +EXPORT_SYMBOL_GPL(irq_create_affinity_masks);
> > >
> > >  /**
> > >   * irq_calc_affinity_vectors - Calculate the optimal number of vectors
> > > --
> > > 2.20.1
> >

_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization

WARNING: multiple messages have this Message-ID (diff)
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Yongji Xie <xieyongji@bytedance.com>
Cc: Christoph Hellwig <hch@lst.de>, Jason Wang <jasowang@redhat.com>,
	Thomas Gleixner <tglx@linutronix.de>,
	virtualization <virtualization@lists.linux-foundation.org>,
	linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2 01/11] genirq/affinity:: Export irq_create_affinity_masks()
Date: Fri, 27 Jan 2023 03:22:16 -0500	[thread overview]
Message-ID: <20230127032039-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <CACycT3utDJFZtWzqCUXJaqRjkCXPMTAi+VJd3g6dw25vWqaduw@mail.gmail.com>

On Mon, Dec 19, 2022 at 05:36:02PM +0800, Yongji Xie wrote:
> On Mon, Dec 19, 2022 at 3:33 PM Michael S. Tsirkin <mst@redhat.com> wrote:
> >
> > On Mon, Dec 05, 2022 at 04:41:17PM +0800, Xie Yongji wrote:
> > > Export irq_create_affinity_masks() so that some modules
> > > can make use of it to implement interrupt affinity
> > > spreading mechanism.
> > >
> > > Signed-off-by: Xie Yongji <xieyongji@bytedance.com>
> >
> > So this got nacked, what's the plan now?
> >
> 
> I‘d like to check with Christoph again first.
> 
> Hi Christoph,
> 
> Jason will post some patches to get rid of the DMA API for vDPA
> simulators. And the irq affinity algorithm is independent of the IRQ
> subsystem IIUC. So could you allow this patch so that we can reuse the
> algorithm to select the best CPU (per-cpu affinity if possible, or at
> least per-node) to run the virtqueue's irq callback.
> 
> Thanks,
> Yongji

I think you need to explain why you are building low level
affinity masks.  what's the plan now?

> > > ---
> > >  kernel/irq/affinity.c | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/kernel/irq/affinity.c b/kernel/irq/affinity.c
> > > index d9a5c1d65a79..f074a7707c6d 100644
> > > --- a/kernel/irq/affinity.c
> > > +++ b/kernel/irq/affinity.c
> > > @@ -487,6 +487,7 @@ irq_create_affinity_masks(unsigned int nvecs, struct irq_affinity *affd)
> > >
> > >       return masks;
> > >  }
> > > +EXPORT_SYMBOL_GPL(irq_create_affinity_masks);
> > >
> > >  /**
> > >   * irq_calc_affinity_vectors - Calculate the optimal number of vectors
> > > --
> > > 2.20.1
> >


  reply	other threads:[~2023-01-27  8:22 UTC|newest]

Thread overview: 74+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-05  8:41 [PATCH v2 00/11] VDUSE: Improve performance Xie Yongji
2022-12-05  8:41 ` [PATCH v2 01/11] genirq/affinity:: Export irq_create_affinity_masks() Xie Yongji
2022-12-06  8:18   ` Christoph Hellwig
2022-12-06  8:18     ` Christoph Hellwig
2022-12-06  8:40     ` Yongji Xie
2022-12-06  8:47       ` Christoph Hellwig
2022-12-06  8:47         ` Christoph Hellwig
2022-12-06  9:28         ` Yongji Xie
2022-12-07  5:56           ` Jason Wang
2022-12-07  5:56             ` Jason Wang
2022-12-19  7:33   ` Michael S. Tsirkin
2022-12-19  7:33     ` Michael S. Tsirkin
2022-12-19  9:36     ` Yongji Xie
2023-01-27  8:22       ` Michael S. Tsirkin [this message]
2023-01-27  8:22         ` Michael S. Tsirkin
2023-01-30 11:53         ` Yongji Xie
2023-02-13 11:59           ` Michael S. Tsirkin
2023-02-13 11:59             ` Michael S. Tsirkin
2023-02-13 14:50             ` Yongji Xie
2023-02-13 18:53               ` Thomas Gleixner
2023-02-13 18:53                 ` Thomas Gleixner
2023-02-14  2:38                 ` Yongji Xie
2022-12-05  8:41 ` [PATCH v2 02/11] vdpa: Add set/get_vq_affinity callbacks in vdpa_config_ops Xie Yongji
2022-12-16  3:52   ` Jason Wang
2022-12-16  3:52     ` Jason Wang
2022-12-05  8:41 ` [PATCH v2 03/11] vdpa: Add set_irq_affinity callback " Xie Yongji
2022-12-16  3:58   ` Jason Wang
2022-12-16  3:58     ` Jason Wang
2022-12-19  4:38     ` Yongji Xie
2022-12-19  6:06       ` Jason Wang
2022-12-19  6:06         ` Jason Wang
2022-12-19  7:12         ` Yongji Xie
2022-12-20  6:31           ` Jason Wang
2022-12-20  6:31             ` Jason Wang
2022-12-20 10:14             ` Yongji Xie
2022-12-21  3:20               ` Jason Wang
2022-12-21  3:20                 ` Jason Wang
2022-12-05  8:41 ` [PATCH v2 04/11] vduse: Refactor allocation for vduse virtqueues Xie Yongji
2022-12-16  3:59   ` Jason Wang
2022-12-16  3:59     ` Jason Wang
2022-12-05  8:41 ` [PATCH v2 05/11] vduse: Introduce bound workqueue for irq injection Xie Yongji
2022-12-16  4:02   ` Jason Wang
2022-12-16  4:02     ` Jason Wang
2022-12-19  5:04     ` Yongji Xie
2022-12-20  6:27       ` Jason Wang
2022-12-20  6:27         ` Jason Wang
2022-12-20 10:01         ` Yongji Xie
2022-12-21  3:19           ` Jason Wang
2022-12-21  3:19             ` Jason Wang
2022-12-05  8:58 ` [PATCH v2 06/11] vduse: Support automatic irq callback affinity Xie Yongji
2022-12-16  5:30   ` Jason Wang
2022-12-16  5:30     ` Jason Wang
2022-12-19  4:56     ` Yongji Xie
2022-12-20  6:32       ` Jason Wang
2022-12-20  6:32         ` Jason Wang
2022-12-20  8:21         ` Yongji Xie
2022-12-05  9:02 ` [PATCH v2 07/11] vduse: Support set/get_vq_affinity callbacks Xie Yongji
2022-12-05  9:02   ` [PATCH v2 08/11] vduse: Add sysfs interface for irq callback affinity Xie Yongji
2022-12-16  5:35     ` Jason Wang
2022-12-16  5:35       ` Jason Wang
2022-12-19  5:16       ` Yongji Xie
2022-12-20  6:29         ` Jason Wang
2022-12-20  6:29           ` Jason Wang
2022-12-20  8:24           ` Yongji Xie
2022-12-05  9:02   ` [PATCH v2 09/11] vduse: Add enable_irq_wq sysfs interface for virtqueues Xie Yongji
2022-12-16  5:43     ` Jason Wang
2022-12-16  5:43       ` Jason Wang
2022-12-19  6:39       ` Yongji Xie
2022-12-05  9:02   ` [PATCH v2 10/11] vduse: Delay iova domain creation Xie Yongji
2022-12-16  5:49     ` Jason Wang
2022-12-16  5:49       ` Jason Wang
2022-12-05  9:02   ` [PATCH v2 11/11] vduse: Support specifying bounce buffer size via sysfs Xie Yongji
2022-12-16  6:02     ` Jason Wang
2022-12-16  6:02       ` Jason Wang

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=20230127032039-mutt-send-email-mst@kernel.org \
    --to=mst@redhat.com \
    --cc=hch@lst.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tglx@linutronix.de \
    --cc=virtualization@lists.linux-foundation.org \
    --cc=xieyongji@bytedance.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.