Re: [RFC PATCH 2/9] vringh: remove vringh_iov and unite to vringh_kiov

2023-01-10 Thread Shunsuke Mie
On 2023/01/11 14:54, Jason Wang wrote: On Wed, Jan 11, 2023 at 11:27 AM Shunsuke Mie wrote: On 2022/12/28 15:36, Jason Wang wrote: On Tue, Dec 27, 2022 at 3:06 PM Shunsuke Mie wrote: 2022年12月27日(火) 15:04 Jason Wang : On Tue, Dec 27, 2022 at 10:25 AM Shunsuke Mie wrote: struct

Re: [RFC PATCH 2/9] vringh: remove vringh_iov and unite to vringh_kiov

2023-01-10 Thread Jason Wang
On Wed, Jan 11, 2023 at 11:27 AM Shunsuke Mie wrote: > > > On 2022/12/28 15:36, Jason Wang wrote: > > On Tue, Dec 27, 2022 at 3:06 PM Shunsuke Mie wrote: > >> 2022年12月27日(火) 15:04 Jason Wang : > >>> On Tue, Dec 27, 2022 at 10:25 AM Shunsuke Mie wrote: > struct vringh_iov is defined to hold

Re: [RFC PATCH 2/9] vringh: remove vringh_iov and unite to vringh_kiov

2023-01-10 Thread Shunsuke Mie
On 2022/12/28 15:36, Jason Wang wrote: On Tue, Dec 27, 2022 at 3:06 PM Shunsuke Mie wrote: 2022年12月27日(火) 15:04 Jason Wang : On Tue, Dec 27, 2022 at 10:25 AM Shunsuke Mie wrote: struct vringh_iov is defined to hold userland addresses. However, to use common function, __vring_iov, finally

Re: [RFC PATCH 2/9] vringh: remove vringh_iov and unite to vringh_kiov

2022-12-27 Thread Jason Wang
On Tue, Dec 27, 2022 at 3:06 PM Shunsuke Mie wrote: > > 2022年12月27日(火) 15:04 Jason Wang : > > > > On Tue, Dec 27, 2022 at 10:25 AM Shunsuke Mie wrote: > > > > > > struct vringh_iov is defined to hold userland addresses. However, to use > > > common function, __vring_iov, finally the vringh_iov

Re: [RFC PATCH 2/9] vringh: remove vringh_iov and unite to vringh_kiov

2022-12-26 Thread Shunsuke Mie
2022年12月27日(火) 16:56 Michael S. Tsirkin : > > On Tue, Dec 27, 2022 at 04:13:49PM +0900, Shunsuke Mie wrote: > > 2022年12月27日(火) 16:05 Michael S. Tsirkin : > > > > > > On Tue, Dec 27, 2022 at 02:04:03PM +0800, Jason Wang wrote: > > > > On Tue, Dec 27, 2022 at 10:25 AM Shunsuke Mie wrote: > > > > >

Re: [RFC PATCH 2/9] vringh: remove vringh_iov and unite to vringh_kiov

2022-12-26 Thread Michael S. Tsirkin
On Tue, Dec 27, 2022 at 04:13:49PM +0900, Shunsuke Mie wrote: > 2022年12月27日(火) 16:05 Michael S. Tsirkin : > > > > On Tue, Dec 27, 2022 at 02:04:03PM +0800, Jason Wang wrote: > > > On Tue, Dec 27, 2022 at 10:25 AM Shunsuke Mie wrote: > > > > > > > > struct vringh_iov is defined to hold userland

Re: [RFC PATCH 2/9] vringh: remove vringh_iov and unite to vringh_kiov

2022-12-26 Thread Shunsuke Mie
2022年12月27日(火) 16:05 Michael S. Tsirkin : > > On Tue, Dec 27, 2022 at 02:04:03PM +0800, Jason Wang wrote: > > On Tue, Dec 27, 2022 at 10:25 AM Shunsuke Mie wrote: > > > > > > struct vringh_iov is defined to hold userland addresses. However, to use > > > common function, __vring_iov, finally the

Re: [RFC PATCH 2/9] vringh: remove vringh_iov and unite to vringh_kiov

2022-12-26 Thread Shunsuke Mie
2022年12月27日(火) 15:04 Jason Wang : > > On Tue, Dec 27, 2022 at 10:25 AM Shunsuke Mie wrote: > > > > struct vringh_iov is defined to hold userland addresses. However, to use > > common function, __vring_iov, finally the vringh_iov converts to the > > vringh_kiov with simple cast. It includes

Re: [RFC PATCH 2/9] vringh: remove vringh_iov and unite to vringh_kiov

2022-12-26 Thread Michael S. Tsirkin
On Tue, Dec 27, 2022 at 02:04:03PM +0800, Jason Wang wrote: > On Tue, Dec 27, 2022 at 10:25 AM Shunsuke Mie wrote: > > > > struct vringh_iov is defined to hold userland addresses. However, to use > > common function, __vring_iov, finally the vringh_iov converts to the > > vringh_kiov with simple

Re: [RFC PATCH 2/9] vringh: remove vringh_iov and unite to vringh_kiov

2022-12-26 Thread Jason Wang
On Tue, Dec 27, 2022 at 10:25 AM Shunsuke Mie wrote: > > struct vringh_iov is defined to hold userland addresses. However, to use > common function, __vring_iov, finally the vringh_iov converts to the > vringh_kiov with simple cast. It includes compile time check code to make > sure it can be

[RFC PATCH 2/9] vringh: remove vringh_iov and unite to vringh_kiov

2022-12-26 Thread Shunsuke Mie
struct vringh_iov is defined to hold userland addresses. However, to use common function, __vring_iov, finally the vringh_iov converts to the vringh_kiov with simple cast. It includes compile time check code to make sure it can be cast correctly. To simplify the code, this patch removes the