Re: [PATCH 1/2] block: remove unnecessary argument from blk_execute_rq_nowait

2021-01-21 Thread Guoqing Jiang
On 1/21/21 18:02, Christoph Hellwig wrote: On Thu, Jan 21, 2021 at 03:29:04PM +0100, Guoqing Jiang wrote: The 'q' is not used since commit a1ce35fa4985 ("block: remove dead elevator code"), also update the comment of the function. And more importantly it never really was needed to start

[PATCH V7 04/13] sbitmap: move allocation hint into sbitmap

2021-01-21 Thread Ming Lei
Allocation hint should have belonged to sbitmap, also when sbitmap's depth is high and no need to use mulitple wakeup queues, user can benefit from percpu allocation hint too. So move allocation hint into sbitmap, then scsi device queue can benefit from allocation hint when converting to plain

[PATCH V7 02/13] sbitmap: maintain allocation round_robin in sbitmap

2021-01-21 Thread Ming Lei
Now allocation round_robin info is maintained by sbitmap_queue. Actually, bit allocation belongs to sbitmap. Also the following patch will move alloc_hint to sbitmap for users with high depth. So move round_robin to sbitmap. Cc: Omar Sandoval Cc: Kashyap Desai Cc: Sumanesh Samanta Cc: Ewan

Re: [PATCH v2 09/11] drm/atomic: Pass the full state to planes atomic disable and update

2021-01-21 Thread Laurent Pinchart
Hi Maxime, Thank you for the patch. On Thu, Jan 21, 2021 at 05:35:34PM +0100, Maxime Ripard wrote: > The current atomic helpers have either their object state being passed as > an argument or the full atomic state. > > The former is the pattern that was done at first, before switching to the >

Re: [PATCH 2/2] block: remove unnecessary argument from blk_execute_rq

2021-01-21 Thread Christoph Hellwig
On Thu, Jan 21, 2021 at 03:29:05PM +0100, Guoqing Jiang wrote: > We can remove 'q' from blk_execute_rq as well after the previous change > in blk_execute_rq_nowait. Same trivial nits as for the other one. Otherwise this looks good to me. ___

Re: [PATCH 1/2] block: remove unnecessary argument from blk_execute_rq_nowait

2021-01-21 Thread Christoph Hellwig
On Thu, Jan 21, 2021 at 03:29:04PM +0100, Guoqing Jiang wrote: > The 'q' is not used since commit a1ce35fa4985 ("block: remove dead > elevator code"), also update the comment of the function. And more importantly it never really was needed to start with given that we can triviall derive it from

Re: [PATCH bpf-next v3 3/3] xsk: build skb by page

2021-01-21 Thread Eric Dumazet
On 1/21/21 2:47 PM, Xuan Zhuo wrote: > This patch is used to construct skb based on page to save memory copy > overhead. > > This function is implemented based on IFF_TX_SKB_NO_LINEAR. Only the > network card priv_flags supports IFF_TX_SKB_NO_LINEAR will use page to > directly construct skb.

[PATCH 2/2] block: remove unnecessary argument from blk_execute_rq

2021-01-21 Thread Guoqing Jiang
We can remove 'q' from blk_execute_rq as well after the previous change in blk_execute_rq_nowait. Cc: linux-s...@vger.kernel.org Cc: virtualization@lists.linux-foundation.org Cc: linux-...@vger.kernel.org Cc: linux-...@vger.kernel.org Cc: linux-n...@lists.infradead.org Cc:

[PATCH 1/2] block: remove unnecessary argument from blk_execute_rq_nowait

2021-01-21 Thread Guoqing Jiang
The 'q' is not used since commit a1ce35fa4985 ("block: remove dead elevator code"), also update the comment of the function. Cc: target-de...@vger.kernel.org Cc: linux-s...@vger.kernel.org Cc: virtualization@lists.linux-foundation.org Cc: linux-...@vger.kernel.org Cc: linux-...@vger.kernel.org

[PATCH 0/2] remove unused argument from blk_execute_rq_nowait and blk_execute_rq

2021-01-21 Thread Guoqing Jiang
Hi Jens, This series remove unused 'q' from blk_execute_rq_nowait and blk_execute_rq. Also update the comment for blk_execute_rq_nowait. Thanks, Guoqing Guoqing Jiang (2): block: remove unnecessary argument from blk_execute_rq_nowait block: remove unnecessary argument from blk_execute_rq

Re: [PATCH v2] drm/virtio: Track total GPU memory for virtio driver

2021-01-21 Thread Gerd Hoffmann
On Wed, Jan 20, 2021 at 10:52:11AM -0800, Yiwei Zhang wrote: > On Wed, Jan 20, 2021 at 5:33 AM Gerd Hoffmann wrote: > > > > Hi, > > > > > > > > > + select TRACE_GPU_MEM > > > > > > > > > +#ifdef CONFIG_TRACE_GPU_MEM > > > > That doesn't make sense btw. > > Do you recommend we just select

Re: [PATCH 1/1] vhost scsi: allocate vhost_scsi with GFP_NOWAIT to avoid delay

2021-01-21 Thread Jason Wang
On 2021/1/21 13:03, Dongli Zhang wrote: The size of 'struct vhost_scsi' is order-10 (~2.3MB). It may take long time delay by kzalloc() to compact memory pages when there is a lack of high-order pages. As a result, there is latency to create a VM (with vhost-scsi) or to hotadd vhost-scsi-based