On 02/07/2015 16:18, Laurent Vivier wrote:
>> > I'm okay with doing the simple thing, but it needs a comment for 
>> > non-BSDers.
> So, what we have to do, in our case, for MacOS X cdrom, is something like:
> 
> ... GetBSDPath ...
> ...
>     if (flags & BDRV_O_NOCACHE) {
>         strcat(bsdPath, "r");
>     }
> ...
> 
> ?

Well, what to do with Mac OS X CD-ROM is another story...  Raw access
"seems not do work well" according to John, so we may have a comment
there explaining why we're not adding the "r".

A FIXME comment saying "we should probe for alignment here" would be
placed where you check S_ISCHR and set need_alignment to true.

Paolo

Reply via email to