b...@bsd.de (Christoph Badura) writes:
>> rootspec is set by config(1) for three cases:
>> config root on major 4 minor 4
>> - sets rootspec to "sd0e" (or maybe "<4/4>") and rootdev to makedev(4,4)
>> config root on sd0e
>> - sets rootspec to "sd0e" and rootdev to makedev(4,4)
>> config root
Christoph Badura writes:
> Here is some instrumentation I found useful during my recent debugging.
> If there are no objections, I'd like to commit soon.
>
> The change to rf_containsroot() simplifies the second DPRINTF that I added.
>
> Index: rf_netbsdkintf.c
>
On Mon, Jan 21, 2019 at 10:42:10AM -0500, Greg Troxel wrote:
> Christoph Badura writes:
> > Index: rf_netbsdkintf.c
> > ===
> > RCS file: /cvsroot/src/sys/dev/raidframe/rf_netbsdkintf.c,v
> > retrieving revision 1.356
> > diff -u
On Mon, Jan 21, 2019 at 06:36:37PM +0100, Christoph Badura wrote:
> I think the following is better. Compile-tested only for both #ifdef
> conditions, but I think that is OK.
Ugh. I forgot to put a comment on that function. How about this:
/*
* Provide a wrapper around rf_containsboot that
> >>> @@ -472,6 +472,9 @@
> >>> const char *bootname = device_xname(bdv);
> >>> size_t len = strlen(bootname);
> >>>
> >>> + if (bdv == NULL)
> >>> + return 0;
> >>> +
> >>
> >> This looked suspicious, even before I read the code.
> >>
> >> The question is if it is ever legitimate for
On Tue, 22 Jan 2019, matthew green wrote:
@@ -472,6 +472,9 @@
const char *bootname = device_xname(bdv);
size_t len = strlen(bootname);
+ if (bdv == NULL)
+ return 0;
+
This looked suspicious, even before I read the code.
The question is if it is ever
Christoph Badura writes:
>> > + if (bdv == NULL)
>> > + return 0;
>> > +
>>
>> This looked suspicious, even before I read the code.
>> The question is if it is ever legitimate for bdv to be NULL.
>
> That is an excellent point. The short answer is, no it isn't. And it
> never was
On Mon, Jan 21, 2019 at 04:24:49PM -0500, Greg Troxel wrote:
> Separetaly from debug code being careful, if it's a rule that bdv can't
> be NULL, it's just as well to put in a KASSERT. Then we'll find out
> where that isn't true and can fix it.
I must not be getting something. If
> > @@ -472,6 +472,9 @@
> > const char *bootname = device_xname(bdv);
> > size_t len = strlen(bootname);
> >
> > + if (bdv == NULL)
> > + return 0;
> > +
>
> This looked suspicious, even before I read the code.
>
> The question is if it is ever legitimate for bdv to be
> It's better to use write(2) for this purpose.
Often, yes, but not, I think, always.
> [...], mmap'ing audio device does not lead to improve any performance
> or latency.
Then something is wrong.
(a) It's the rare port on which copying to mmap()ped memory is no
faster than a user/kernel
Christoph Badura writes:
> On Mon, Jan 21, 2019 at 04:24:49PM -0500, Greg Troxel wrote:
>> Separetaly from debug code being careful, if it's a rule that bdv can't
>> be NULL, it's just as well to put in a KASSERT. Then we'll find out
>> where that isn't true and can fix it.
>
> I must not be
On Tue, Jan 22, 2019 at 08:32:48AM +1100, matthew green wrote:
> > > @@ -472,6 +472,9 @@
> > > const char *bootname = device_xname(bdv);
> > > size_t len = strlen(bootname);
> > >
> > > + if (bdv == NULL)
> > > + return 0;
> > > +
> >
> > This looked suspicious, even before I read
In article <24279.1548106...@splode.eterna.com.au>,
matthew green wrote:
>> > @@ -472,6 +472,9 @@
>> >const char *bootname = device_xname(bdv);
>> >size_t len = strlen(bootname);
>> >
>> > + if (bdv == NULL)
>> > + return 0;
>> > +
>>
>> This looked suspicious, even before I
13 matches
Mail list logo