linux-kernel - Re: [patch]block: fix NULL icq

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] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 18 Jan 2012 17:43:56 -0800
From:	Tejun Heo <tj@...nel.org>
To:	Shaohua Li <shaohua.li@...el.com>
Cc:	Jens Axboe <axboe@...nel.dk>, Vivek Goyal <vgoyal@...hat.com>,
	linux kernel mailing list <linux-kernel@...r.kernel.org>
Subject: Re: [patch]block: fix NULL icq_cache reference

Hello,

On Thu, Jan 19, 2012 at 09:41:34AM +0800, Shaohua Li wrote:
> > Hmmm... but I'm having a difficult time following the description.
> > Maybe we can simplify a bit?  e.g. sth like the following?
> > 
> >   Once a queue is quiesced, it's not supposed to have any elvpriv data
> >   or icq's, and elevator switching depends on that.  Request alloc
> >   path followed the rule for elvpriv data but forgot apply it to
> >   icq's leading to the following crash during elevator switch.
> > 
> >   <oops log>
> > 
> >   Fix it by not allocating icq's if ELVPRIV is not set for the
> >   request.
>
> I'm trying to explain why there is a crash, but fine to use your
> description.

Yeah, giving details is great.  It just seemed that high level
description seemed lost among the details.  It would have been great
if the description started with high level description and then went
into details.

> Once a queue is quiesced, it's not supposed to have any elvpriv data or
> icq's, and elevator switching depends on that.  Request alloc path
> followed the rule for elvpriv data but forgot apply it to icq's
> leading to the following crash during elevator switch. Fix it by not
> allocating icq's if ELVPRIV is not set for the request.
> 
> Reported-by: Vivek Goyal <vgoyal@...hat.com>
> Tested-by: Vivek Goyal <vgoyal@...hat.com>
> Signed-off-by: Shaohua Li <shaohua.li@...el.com>

Acked-by: Tejun Heo <tj@...nel.org>

Thanks a lot.

-- 
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/