Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/share/man/man9

> >>>> +The functions were originally introduced to handle
> >>>> +.Tn PCI
> >>>> +bus master devices, which assumed little endian byte order in
> >>>> +.Tn DMA
> >>>> +transfers, even on big endian systems.
> >>
> >> There are a few PCI bus master devices that are big endian aware
> >> (epic(4) for example), so I don't think it's appropriate to add
> >> commas around the which clause.
> >
> > I agree that the first comma should go, but the second one (before  
> > "even") should remain.
> No it shouldn't. Anyway, I just rearranged it to hopefully avoid such
> problems entirely.

cvs rdiff -u -r1.6 -r1.7 src/share/man/man9/byteorder.9
>> -bus master devices which assumed little endian byte order in
>> +bus master devices that do little endian
>>  .Tn DMA
>> -transfers, even on big endian systems.
>> +transfers on big endian systems.

What's "little endian transfer"?

IMO, DMA transfers are always stream (no byte reordering) and
hto*()/*toh() functions are used to encode/decode values
larger than a byte into/from the stream before/after transfers
per byte order the target bus master device assumes.

That was what a dumb sentence in rev 1.1 meant.
Izumi Tsutsui

Home | Main Index | Thread Index | Old Index