linux-kernel - RE: [usb gadget]use dev_kfree_skb_irq instead of dev_kfree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Fri, 22 Dec 2017 08:19:07 +0000
From:   Lipengcheng <lpc.li@...ilicon.com>
To:     Greg KH <gregkh@...uxfoundation.org>
CC:     "balbi@...nel.org" <balbi@...nel.org>,
        "linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Lidongpo <lidongpo@...ilicon.com>
Subject: RE: [usb gadget]use dev_kfree_skb_irq instead of dev_kfree_skb

Hi,
> -----Original Message-----
> From: Greg KH [mailto:gregkh@...uxfoundation.org]
> Sent: Friday, December 22, 2017 3:50 PM
> To: Lipengcheng
> Cc: balbi@...nel.org; linux-usb@...r.kernel.org; linux-kernel@...r.kernel.org
> Subject: Re: [usb gadget]use dev_kfree_skb_irq instead of dev_kfree_skb
> 
> On Fri, Dec 22, 2017 at 03:43:15PM +0800, lpc.li wrote:
> > Hi,
> > Ethernet Gadget (with CDC Ethernet support) and RNDIS support, device
> > is Hisilicon asic and host is PC Windows. Howerver, it has one
> > warning. In the rndis_add_header of the file f_rndis.c,
> > dev_kfree_skb_irq replaces dev_kfree_skb. And the waning is
> > eliminated. I think we can't use dev_kfree_skb in irq disable context,
> > because spin_lock_irqsave make sure we are always in irq disable
> > context, use dev_kfree_skb_irq instead of dev_kfree_skb is better than
> > dev_kfree_skb_any.
> 
> Can you make up a patch for this please?
Ok.
> 
> >
> > #<--CO------------[ cut here ]------------
> > WARNING: CPU: 0 PID: 22 at kernel/softirq.c:146
> > __local_bh_enable_ip+0x8c/0xb4() Modules linked in: g_ether usb_f_ecm
> > usb_f_rndis u_ether libcomposite configfs udc_hisi hi_gpio(O) hifb(O)
> > hi3559_tde(PO) hi3559_sys(PO) hi3559_base(PO) hi_osal(O) hi_ipcm(O)
> > CPU: 0 PID: 22 Comm: kworker/0:1 Tainted: P O 3.18.20 #1
> 
> Note, 3.18 is _VERY_ old, are you sure this isn't resolved in a modern and secure kernel release? :)
The problem still exists in a modern and secure kernel release
> 
> thanks,
> 
> greg k-h