[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <ab2ea6f6-330f-068a-e9dd-d2803e203b45@163.com>
Date: Thu, 22 Jun 2017 18:52:45 +0800
From: Jia-Ju Bai <baijiaju1990@....com>
To: Dan Carpenter <dan.carpenter@...cle.com>,
Kalle Valo <kvalo@...eaurora.org>
Cc: David Miller <davem@...emloft.net>, manish.chopra@...ium.com,
rahul.verma@...ium.com, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] netxen: Fix a sleep-in-atomic bug in
netxen_nic_pci_mem_access_direct
On 2017/6/22 14:08, Dan Carpenter wrote:
> We should probably add a might_sleep() to ioremap() to prevent these
> bugs in the future.
I think it is right to do this.
And it will be very useful to summarize common kernel interface
functions which may sleep into a list. When writing a new driver, the
developer can refer to this list to reduce or avoid sleep-in-atomic bugs.
>
> This bug is eight years old. You can report it, but it's going to hard
> to get anyone to fix it. I sometimes ignore ancient bugs. On the other
> hand, netxen is fairly well supported so it doesn't hurt to try.
>
> I try to report bugs as soon as they are introduced. I report it to
> the author and CC the relevant list. If people don't respond to my
> email after a month then I complain again.
>
> regards,
> dan carpenter
>
Thanks for your helpful advice.
Thanks,
Jia-Ju Bai
Powered by blists - more mailing lists