On Mon, 14 Mar 2022, Michael S. Tsirkin wrote:
> On Mon, Mar 14, 2022 at 08:43:02AM +0000, Lee Jones wrote:
> > vhost_vsock_handle_tx_kick() already holds the mutex during its call
> > to vhost_get_vq_desc(). All we have to do here is take the same lock
> > during virt
conditions which may pop up over time.
Cc:
Signed-off-by: Lee Jones
---
drivers/vhost/vhost.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/vhost/vhost.c b/drivers/vhost/vhost.c
index 59edb5a1ffe28..bbaff6a5e21b8 100644
--- a/drivers/vhost/vhost.c
+++ b/drivers/vhost/vhost.c
2022 at 09:15:27AM +, Lee Jones wrote:
> > > > > On Tue, 08 Mar 2022, Greg KH wrote:
> > > > >
> > > > > > On Tue, Mar 08, 2022 at 08:10:06AM +, Lee Jones wrote:
> > > > > > > On Mon, 07 Mar 2022, Greg
On Tue, 08 Mar 2022, Greg KH wrote:
> On Tue, Mar 08, 2022 at 09:15:27AM +0000, Lee Jones wrote:
> > On Tue, 08 Mar 2022, Greg KH wrote:
> >
> > > On Tue, Mar 08, 2022 at 08:10:06AM +, Lee Jones wrote:
> > > > On Mon, 07 Mar 2022, Greg KH wrote:
> >
On Tue, 08 Mar 2022, Greg KH wrote:
> On Tue, Mar 08, 2022 at 08:10:06AM +0000, Lee Jones wrote:
> > On Mon, 07 Mar 2022, Greg KH wrote:
> >
> > > On Mon, Mar 07, 2022 at 07:17:57PM +, Lee Jones wrote:
> > > > vhost_vsock_handle_tx_kick() already holds
On Tue, 08 Mar 2022, Lee Jones wrote:
> On Mon, 07 Mar 2022, Greg KH wrote:
>
> > On Mon, Mar 07, 2022 at 07:17:57PM +0000, Lee Jones wrote:
> > > vhost_vsock_handle_tx_kick() already holds the mutex during its call
> > > to vhost_get_vq_desc(). All we have to
On Mon, 07 Mar 2022, Greg KH wrote:
> On Mon, Mar 07, 2022 at 07:17:57PM +0000, Lee Jones wrote:
> > vhost_vsock_handle_tx_kick() already holds the mutex during its call
> > to vhost_get_vq_desc(). All we have to do here is take the same lock
> > during virtqueue clean-u
On Tue, 08 Mar 2022, Jason Wang wrote:
> On Tue, Mar 8, 2022 at 3:18 AM Lee Jones wrote:
> >
> > vhost_vsock_handle_tx_kick() already holds the mutex during its call
> > to vhost_get_vq_desc(). All we have to do here is take the same lock
> > during virtqueue c
On Mon, 07 Mar 2022, Michael S. Tsirkin wrote:
> On Mon, Mar 07, 2022 at 07:17:57PM +0000, Lee Jones wrote:
> > vhost_vsock_handle_tx_kick() already holds the mutex during its call
> > to vhost_get_vq_desc(). All we have to do here is take the same lock
> > during virt
conditions which may pop up over time.
Link: https://syzkaller.appspot.com/bug?extid=279432d30d825e63ba00
Cc:
Reported-by: syzbot+adc3cb32385586bec...@syzkaller.appspotmail.com
Signed-off-by: Lee Jones
---
drivers/vhost/vhost.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a
On Fri, 04 Mar 2022, Michael S. Tsirkin wrote:
> On Wed, Mar 02, 2022 at 07:54:21AM +0000, Lee Jones wrote:
> > vhost_vsock_handle_tx_kick() already holds the mutex during its call
> > to vhost_get_vq_desc(). All we have to do is take the same lock
> > during virtqueue cle
On Fri, 04 Mar 2022, Michael S. Tsirkin wrote:
> On Wed, Mar 02, 2022 at 07:54:21AM +0000, Lee Jones wrote:
> > vhost_vsock_handle_tx_kick() already holds the mutex during its call
> > to vhost_get_vq_desc(). All we have to do is take the same lock
> > during virtqueue cle
On Fri, 04 Mar 2022, Stefano Garzarella wrote:
> On Thu, Mar 03, 2022 at 04:01:06PM -0500, Michael S. Tsirkin wrote:
> > On Thu, Mar 03, 2022 at 09:14:36PM +0200, Leon Romanovsky wrote:
> > > On Thu, Mar 03, 2022 at 03:19:29PM +, Lee Jones wrote:
> > > > All
On Fri, 04 Mar 2022, Leon Romanovsky wrote:
> On Thu, Mar 03, 2022 at 04:01:06PM -0500, Michael S. Tsirkin wrote:
> > On Thu, Mar 03, 2022 at 09:14:36PM +0200, Leon Romanovsky wrote:
> > > On Thu, Mar 03, 2022 at 03:19:29PM +, Lee Jones wrote:
> > > > All workers
On Thu, 03 Mar 2022, Leon Romanovsky wrote:
> On Thu, Mar 03, 2022 at 03:19:29PM +0000, Lee Jones wrote:
> > All workers/users should be halted before any clean-up should take place.
> >
> > Suggested-by: Michael S. Tsirkin
> > Signed-off-by: Lee Jones
> > -
All workers/users should be halted before any clean-up should take place.
Suggested-by: Michael S. Tsirkin
Signed-off-by: Lee Jones
---
drivers/vhost/vhost.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/vhost/vhost.c b/drivers/vhost/vhost.c
index bbaff6a5e21b8..d935d2506963f
On Wed, 02 Mar 2022, Stefano Garzarella wrote:
> On Wed, Mar 02, 2022 at 04:49:17PM +0000, Lee Jones wrote:
> > On Wed, 02 Mar 2022, Michael S. Tsirkin wrote:
> >
> > > On Wed, Mar 02, 2022 at 05:28:31PM +0100, Stefano Garzarella wrote:
> > > > On Wed, Mar
On Wed, 02 Mar 2022, Michael S. Tsirkin wrote:
> On Wed, Mar 02, 2022 at 05:28:31PM +0100, Stefano Garzarella wrote:
> > On Wed, Mar 2, 2022 at 3:57 PM Lee Jones wrote:
> > >
> > > On Wed, 02 Mar 2022, Michael S. Tsirkin wrote:
> > >
> > > > On
On Wed, 02 Mar 2022, Michael S. Tsirkin wrote:
> On Wed, Mar 02, 2022 at 01:56:35PM +0000, Lee Jones wrote:
> > On Wed, 02 Mar 2022, Michael S. Tsirkin wrote:
> >
> > > On Wed, Mar 02, 2022 at 07:54:21AM +, Lee Jones wrote:
> > > > vhost_vsock_handle_tx_k
On Wed, 02 Mar 2022, Michael S. Tsirkin wrote:
> On Wed, Mar 02, 2022 at 07:54:21AM +0000, Lee Jones wrote:
> > vhost_vsock_handle_tx_kick() already holds the mutex during its call
> > to vhost_get_vq_desc(). All we have to do is take the same lock
> > during virtqueue cle
On Wed, 02 Mar 2022, Stefano Garzarella wrote:
> On Wed, Mar 02, 2022 at 07:54:21AM +0000, Lee Jones wrote:
> > vhost_vsock_handle_tx_kick() already holds the mutex during its call
> > to vhost_get_vq_desc(). All we have to do is take the same lock
> > during virtqueue cle
ers/vhost/vhost.c:372
> > kthread+0x2e9/0x3a0 kernel/kthread.c:377
> > ret_from_fork+0x1f/0x30 arch/x86/entry/entry_64.S:295
>
> I don't see how this can trigger normally so I'm assuming
> another case of use after free.
Yes, exactly.
I patched it. Please see:
+adc3cb32385586bec...@syzkaller.appspotmail.com
Signed-off-by: Lee Jones
---
drivers/vhost/vhost.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/vhost/vhost.c b/drivers/vhost/vhost.c
index 59edb5a1ffe28..bbaff6a5e21b8 100644
--- a/drivers/vhost/vhost.c
+++ b/drivers/vhost/vhost.c
@@ -693,6
Stefan Hajnoczi
Cc: Stefano Garzarella
Cc: "David S. Miller"
Cc: Jakub Kicinski
Cc: virtualization@lists.linux-foundation.org
Cc: k...@vger.kernel.org
Cc: net...@vger.kernel.org
Signed-off-by: Ram Muthiah
Signed-off-by: Lee Jones
---
include/linux/virtio_vsock.h
/master.c | 4 +---
> drivers/input/gameport/gameport.c | 3 +--
> drivers/input/serio/serio.c | 3 +--
> drivers/ipack/ipack.c | 4 +---
> drivers/macintosh/macio_asic.c| 4 +---
> drivers/mcb/mcb-core.c
On Mon, 13 Jul 2020, Michael S. Tsirkin wrote:
> On Mon, Jul 13, 2020 at 08:59:49AM +0100, Lee Jones wrote:
> > This is the only use of kerneldoc in the sourcefile and no
> > descriptions are provided.
> >
> > Fixes the following W=1 kernel build warning(s):
> >
_scsi.c:109: warning: Function parameter or member 'buf'
not described in 'virtscsi_complete_cmd'
Cc: "Michael S. Tsirkin"
Cc: Jason Wang
Cc: Stefan Hajnoczi
Cc: virtualization@lists.linux-foundation.org
Signed-off-by: Lee Jones
Acked-by: Paolo Bonzini
---
dr
_scsi.c:109: warning: Function parameter or member 'buf'
not described in 'virtscsi_complete_cmd'
Cc: "Michael S. Tsirkin"
Cc: Jason Wang
Cc: Paolo Bonzini
Cc: Stefan Hajnoczi
Cc: virtualization@lists.linux-foundation.org
Signed-off-by: Lee Jones
---
dr
for (i = 0; i < ST_SLIM_MAX_CLK && slim_rproc->clks[i]; i++)
> > + clk_put(slim_rproc->clks[i]);
> > +err:
> > + rproc_put(rproc);
> > + return ERR_PTR(err);
> > +}
> > +EXPORT_SYMBOL(st_slim_rproc_alloc);
> > +
> &
b's the boss. We certainly never used to take 'device ID' or
> > 'version' attributes. I guess something must have changed.
>
> I will try to provide patches for code and bindings rework this week.
Wonderful.. Thanks Arnaud.
--
Lee Jones
Linaro STMicroelectronics L
On Tue, 30 Aug 2016, Peter Griffin wrote:
> Thanks for reviewing and your very valuable feedback.
> On Tue, 30 Aug 2016, Lee Jones wrote:
> > On Fri, 26 Aug 2016, Peter Griffin wrote:
> >
> > > This patch adds the DT node for the uniperif reader
> > > IP b
On Tue, 30 Aug 2016, Peter Griffin wrote:
> On Tue, 30 Aug 2016, Lee Jones wrote:
> > On Fri, 26 Aug 2016, Peter Griffin wrote:
> >
> > > slim core is used as a basis for many IPs in the STi
> > > chipsets such as fdma and demux. To avoid duplicating
> >
> +{
> + int clk;
> +
> + if (!slim_rproc)
> + return;
> +
> + for (clk = 0; clk < ST_SLIM_MAX_CLK && slim_rproc->clks[clk]; clk++)
> + clk_put(slim_rproc->clks[clk]);
> +
> + rproc_del(slim_rproc->rproc);
> + rproc_put(slim_
On Fri, 26 Aug 2016, Peter Griffin wrote:
> This patch adds the slim core rproc driver to the STi section
> of the MAINTAINERS file.
>
> Signed-off-by: Peter Griffin
> ---
> MAINTAINERS | 2 ++
> 1 file changed, 2 insertions(+)
Acked-by: Lee Jones
> diff --git a/M
52 insertions(+)
Acked-by: Lee Jones
> diff --git a/arch/arm/boot/dts/stih407-family.dtsi
> b/arch/arm/boot/dts/stih407-family.dtsi
> index d294e82..45cab30 100644
> --- a/arch/arm/boot/dts/stih407-family.dtsi
> +++ b/arch/arm/boot/dts/stih407-family.dtsi
> @@ -821,5 +821,57
liq...@st.com
> Cc: broo...@kernel.org
> ---
> arch/arm/configs/multi_v7_defconfig | 3 +++
> 1 file changed, 3 insertions(+)
Acked-by: Lee Jones
> diff --git a/arch/arm/configs/multi_v7_defconfig
> b/arch/arm/configs/multi_v7_defconfig
> index 998578a..51a38b1 100644
>
+++
> 1 file changed, 23 insertions(+)
Acked-by: Lee Jones
> diff --git a/arch/arm/boot/dts/stih407-pinctrl.dtsi
> b/arch/arm/boot/dts/stih407-pinctrl.dtsi
> index a538ae5..0fb5c8a 100644
> --- a/arch/arm/boot/dts/stih407-pinctrl.dtsi
> +++ b/arch/arm/boot/dts/sti
+++
> 1 file changed, 24 insertions(+)
Acked-by: Lee Jones
> diff --git a/arch/arm/boot/dts/stih407-pinctrl.dtsi
> b/arch/arm/boot/dts/stih407-pinctrl.dtsi
> index 0fb5c8a..537db7e 100644
> --- a/arch/arm/boot/dts/stih407-pinctrl.dtsi
> +++ b/arch/arm/boot/dts/sti
le changed, 8 insertions(+)
Acked-by: Lee Jones
> diff --git a/arch/arm/boot/dts/stih407-pinctrl.dtsi
> b/arch/arm/boot/dts/stih407-pinctrl.dtsi
> index 537db7e..598dbab 100644
> --- a/arch/arm/boot/dts/stih407-pinctrl.dtsi
> +++ b/arch/arm/boot/dts/sti
On Fri, 26 Aug 2016, Peter Griffin wrote:
> This patch adds the dt node for the internal audio
> codec IP.
>
> Signed-off-by: Arnaud Pouliquen
> Signed-off-by: Peter Griffin
> ---
> arch/arm/boot/dts/stih407-family.dtsi | 7 +++
> 1 file changed, 7 insertions
On Tue, 30 Aug 2016, Lee Jones wrote:
> On Fri, 26 Aug 2016, Peter Griffin wrote:
>
> > This patch adds the DT nodes for the uniperif player
> > IP blocks found on STiH407 family silicon.
> >
> > Signed-off-by: Arnaud Pouliquen
> > Signed-off-by: Peter Gri
ells = <0>;
> + st,syscfg = <&syscfg_core>;
> + clocks = <&clk_s_d0_flexgen CLK_SPDIFF>;
> + assigned-clocks = <&clk_s_d0_quadfs 3>,
> <&clk_s_d0_flexgen CLK_SPDIFF>;
> +
reg = <0x8D84000 0x158>;
> + interrupts = ;
> + dmas = <&fdma0 6 0 1>;
> + dma-names = "rx";
> + dai-name = "Uni Reader #1 (HDMI RX)";
> + st,version = <3>;
>
On Tue, 30 Aug 2016, Lee Jones wrote:
> On Fri, 26 Aug 2016, Peter Griffin wrote:
>
> > Use tabs instead of spaces.
> >
> > Signed-off-by: Peter Griffin
> > ---
> > drivers/gpu/drm/virtio/Kconfig | 6 +++---
> > 1 file changed, 3 insertions(+)
by: Peter Griffin
> ---
> arch/arm/boot/dts/stihxxx-b2120.dtsi | 45
>
> 1 file changed, 45 insertions(+)
Acked-by: Lee Jones
> diff --git a/arch/arm/boot/dts/stihxxx-b2120.dtsi
> b/arch/arm/boot/dts/stihxxx-b2120.dtsi
> index 722c
works is
usually helpful here.
> Signed-off-by: Peter Griffin
> ---
> drivers/gpu/drm/virtio/Kconfig | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/virtio/Kconfig b/drivers/gpu/drm/virtio/Kconfig
> index e1afc3d..90357d9
On Fri, 26 Aug 2016, Peter Griffin wrote:
> Use tabs instead of spaces.
>
> Signed-off-by: Peter Griffin
> ---
> drivers/gpu/drm/virtio/Kconfig | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
For my own reference:
Acked-by: Lee Jones
> diff --git a/d
> > Resorted to poaching now have we Pawel?
> I hope you were joking!
Yes, of course. I thought that was clearly indicated by the jovial winking
smiley. :)
I realise it wasn't obvious soley by this exchange, but Pawel and I are
actually ol' friends.
> Doing your work for you isn't poaching.
Th
On Mon, 05 Nov 2012, Pawel Moll wrote:
> On Mon, 2012-11-05 at 13:44 +0000, Lee Jones wrote:
> > On Mon, 05 Nov 2012, Pawel Moll wrote:
> >
> > > On 64-bit machines resource_size_t is a 64-bit value, while
> > > sscanf() format for this argument was defined
and adde extra
> comments to make the parameters parsing easier to follow.
>
> Reported-by: Lee Jones
> Signed-off-by: Pawel Moll
> ---
> drivers/virtio/virtio_mmio.c | 26 ++
> 1 file changed, 18 insertions(+), 8 deletions(-)
Resorted to poaching
On Mon, 05 Nov 2012, Rusty Russell wrote:
> Lee Jones writes:
> > drivers/virtio/virtio_mmio.c: In function ‘vm_cmdline_set’:
> > drivers/virtio/virtio_mmio.c:535:4: warning: format ‘%u’ expects argument
> > of type ‘unsigned int *’, but argument 4 has type ‘resource_siz
: Lee Jones
---
drivers/virtio/virtio_mmio.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/virtio/virtio_mmio.c b/drivers/virtio/virtio_mmio.c
index 6b1b7e1..077e9ca 100644
--- a/drivers/virtio/virtio_mmio.c
+++ b/drivers/virtio/virtio_mmio.c
@@ -531,7 +531,7
52 matches
Mail list logo