Re: ATAng regression: cdcontrol close not working
From: Lars Eggert (larse_at_ISI.EDU)
Date: 10/30/03
- Previous message: Pav Lucistnik: "Re: ATAng regression: cdcontrol close not working"
- In reply to: Soren Schmidt: "Re: ATAng regression: cdcontrol close not working"
- Next in thread: Bruce Evans: "Re: ATAng regression: cdcontrol close not working"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Date: Thu, 30 Oct 2003 13:32:45 -0800 To: Soren Schmidt <sos@spider.deepcore.dk>
Soren Schmidt wrote:
>
> Anyhow if you loose the test for error in atapi-cd.c::acd_tray in the
> close case, does it work then ?
If you mean as below, no, that didn't help:
Index: atapi-cd.c
===================================================================
RCS file: /home/ncvs/src/sys/dev/ata/atapi-cd.c,v
retrieving revision 1.149
diff -u -r1.149 atapi-cd.c
--- atapi-cd.c 18 Oct 2003 17:24:51 -0000 1.149
+++ atapi-cd.c 30 Oct 2003 21:31:23 -0000
@@ -1884,11 +1884,10 @@
if (cdp->cap.mechanism & MST_EJECT) {
if (close) {
- if (!(error = acd_start_stop(cdp, 3))) {
- acd_read_toc(cdp);
- acd_prevent_allow(cdp, 1);
- cdp->flags |= F_LOCKED;
- }
+ error = acd_start_stop(cdp, 3);
+ acd_read_toc(cdp);
+ acd_prevent_allow(cdp, 1);
+ cdp->flags |= F_LOCKED;
}
else {
acd_start_stop(cdp, 0);
Lars
-- Lars Eggert <larse@isi.edu> USC Information Sciences Institute
- application/x-pkcs7-signature attachment: S/MIME Cryptographic Signature
- Previous message: Pav Lucistnik: "Re: ATAng regression: cdcontrol close not working"
- In reply to: Soren Schmidt: "Re: ATAng regression: cdcontrol close not working"
- Next in thread: Bruce Evans: "Re: ATAng regression: cdcontrol close not working"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Relevant Pages
- Re: Whats the point of __KERNEL_SYSCALLS__?
... retrieving revision 1.4 ... diff -u -p -r1.4 process.c ... -static
int errno; ... * calls - which means inline code for fork too, ... (Linux-Kernel) - New Patch [was: Re: cvs rm sys/posix4 && enable sem]
... posix4/sched.h to sys/psched.h in this patch). ... "steps" as, in place of a
repo-copy, I'll commit the diff to ... retrieving revision 1.2 ... +To compile
this driver into the kernel, ... (freebsd-arch) - CONFIG_64_BIT
... retrieving revision 1.3 ... diff -u -p -r1.3 Kconfig ...
config MMU ... (Linux-Kernel) - Re: vmstats entries type
... while updating the 64-bit per cpu counters. ... retrieving revision 1.80
... diff -u -r1.80 vmstat.c ... (freebsd-current) - cvs rm sys/posix4 && enable sem
... diff -u -r1.265 Makefile ... retrieving revision 1.40 ... .Ft
int ... +To compile this driver into the kernel, ... (freebsd-arch)