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
| ||
|
Date: Sun, 25 May 2008 10:03:22 +0200 From: Borislav Petkov <petkovbb@...glemail.com> To: Bartlomiej Zolnierkiewicz <bzolnier@...il.com> Cc: "Rafael J. Wysocki" <rjw@...k.pl>, Stephen Rothwell <sfr@...b.auug.org.au>, linux-next@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>, Jens Axboe <jens.axboe@...cle.com>, FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>, Andrew Morton <akpm@...ux-foundation.org> Subject: Re: linux-next: Tree for May 22 On Sun, May 25, 2008 at 11:55:47PM +0200, Bartlomiej Zolnierkiewicz wrote: > > Hi, > > On Friday 23 May 2008, Rafael J. Wysocki wrote: > > On Thursday, 22 of May 2008, Stephen Rothwell wrote: > > > Hi all, > > > > > > Changes since next-20080522: > > > > > > The three patches reverted from the driver-core tree yesterday have been > > > removed from there. > > > > > > The net tree (actually the wireless tree) had a trivial conflict with the > > > wireless-current tree. > > > > > > One of the sound tree build patches has been merged. > > > > > > The semaphore-removal tree still needed three patches reverted because similar > > > patches were merged into upstream and the scsi-rc-fixes trees. > > > > > > The bkl-removal tree had a conflict with the arm tree because the arm > > > tree removed a file that was changed. > > > > > > I have applied the following temporary patch for known build problems: > > > > > > "Fix various 8390 builds" - the net tree broke builds on various > > > architectures - hopefully this patch will go into the net tree shortly. > > > "build failure with netfilter on __nf_ct_helper_find()" - breakage > > > due to the moving of stuff from list.h to rculist.h > > > > This one and the previous linux-next can't resume from suspend to RAM on my > > HP nx6325. The problem is reliably reproducible 100% of the time. > > > > The patch that causes it to happen, as identified by bisection, is: > > > > commit 3dcefa9419f3b8a49921d2218c93a3ddba8e0855 > > Author: Bartlomiej Zolnierkiewicz <bzolnier@...il.com> > > Date: Thu May 22 10:40:19 2008 +1000 > > > > ide: use __generic_unplug_device() in ide_do_drive_cmd() > > > > Cc: FUJITA Tomonori <fujita.tomonori@....ntt.co.jp> > > Cc: Borislav Petkov <petkovbb@...il.com> > > Cc: Jens Axboe <jens.axboe@...cle.com> > > Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@...il.com> > > Thanks for narrowing it down. > > > Reverting it along with the next IDE patches (up to and including > > commit fd42e6df0fd163411f5c655b4bde4fb6a8910edc > > "ide: remove action argument in ide_do_drive_cmd") makes the problem go away. > > Please check if the patch below (against IDE tree but should also apply > to the current linux-next) fixes the issue (for blk_pm_resume_request() > the queue is stopped so we need to call ->request_fn explicitly). > > --- > block/blk-exec.c | 3 +++ > drivers/ide/ide-io.c | 3 +++ > 2 files changed, 6 insertions(+) > > Index: b/block/blk-exec.c > =================================================================== > --- a/block/blk-exec.c > +++ b/block/blk-exec.c > @@ -57,6 +57,9 @@ void blk_execute_rq_nowait(struct reques > spin_lock_irq(q->queue_lock); > __elv_add_request(q, rq, where, 1); > __generic_unplug_device(q); > + /* the queue is stopped so it won't be plugged+unplugged */ > + if (blk_pm_resume_request(rq)) > + q->request_fn(q); > spin_unlock_irq(q->queue_lock); > } > EXPORT_SYMBOL_GPL(blk_execute_rq_nowait); > Index: b/drivers/ide/ide-io.c > =================================================================== > --- a/drivers/ide/ide-io.c > +++ b/drivers/ide/ide-io.c > @@ -1540,6 +1540,9 @@ void ide_do_drive_cmd(ide_drive_t *drive > hwgroup->rq = NULL; > __elv_add_request(drive->queue, rq, ELEVATOR_INSERT_FRONT, 1); > __generic_unplug_device(drive->queue); > + /* the queue is stopped so it won't be plugged+unplugged */ > + if (blk_pm_resume_request(rq)) > + do_ide_request(drive->queue); > spin_unlock_irqrestore(&ide_lock, flags); > } I could reproduce the bug on my Asus M6B00N and can confirm that the patch fixes it. -- Regards/Gruß, Boris. -- 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/
Powered by blists - more mailing lists