Re: [PATCH] serial: amba-pl011: fix regression, causing an Oops on rmmod

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 5 April 2014 20:01, Guennadi Liakhovetski <g.liakhovetski@xxxxxx> wrote:
> A recent commit ef2889f7ffee67f0aed49e854c72be63f1466759 "serial: pl011:
> Move uart_register_driver call to device probe" introduced a regression,
> causing the pl011 driver to Oops if more than 1 port have been probed. Fix
> the Oops by only calling uart_unregister_driver() once after the last port
> has been removed.
>
> Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@xxxxxx>

Sorry about this. There were some additional suggestion in following
thread [1] about registering it once during first probe and then
unregistering during the last remove.

[1] http://linux-kernel.2935.n7.nabble.com/PATCH-0-2-serial-Move-uart-register-driver-call-to-device-probe-td789746.html#a790221

> ---
>  drivers/tty/serial/amba-pl011.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/tty/serial/amba-pl011.c b/drivers/tty/serial/amba-pl011.c
> index d4eda24..07325f2 100644
> --- a/drivers/tty/serial/amba-pl011.c
> +++ b/drivers/tty/serial/amba-pl011.c
> @@ -2176,6 +2176,7 @@ static int pl011_probe(struct amba_device *dev, const struct amba_id *id)
>  static int pl011_remove(struct amba_device *dev)
>  {
>         struct uart_amba_port *uap = amba_get_drvdata(dev);
> +       bool busy = false;
>         int i;
>
>         uart_remove_one_port(&amba_reg, &uap->port);
> @@ -2183,9 +2184,12 @@ static int pl011_remove(struct amba_device *dev)
>         for (i = 0; i < ARRAY_SIZE(amba_ports); i++)
>                 if (amba_ports[i] == uap)
>                         amba_ports[i] = NULL;
> +               else if (amba_ports[i])
> +                       busy = true;
>
>         pl011_dma_remove(uap);
> -       uart_unregister_driver(&amba_reg);
> +       if (!busy)
> +               uart_unregister_driver(&amba_reg);
>         return 0;
>  }
>
> --
> 1.9.1
>



-- 
Tushar Behera
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux