Re: [PATCH i-g-t 3/5] i915: Exercise preemption timeout controls in sysfs

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

 



> > > +void dyn_sysfs_engines(int i915, int engines, const char *file,
> > > +                    void (*test)(int, int))
> > > +{
> > > +     char buf[512];
> > > +     int len;
> > > +
> > > +     lseek(engines, 0, SEEK_SET);
> > > +     while ((len = syscall(SYS_getdents64, engines, buf, sizeof(buf))) > 0) {
> > > +             void *ptr = buf;
> > > +
> > > +             while (len) {
> > > +                     struct linux_dirent64 {
> > > +                             ino64_t        d_ino;
> > > +                             off64_t        d_off;
> > > +                             unsigned short d_reclen;
> > > +                             unsigned char  d_type;
> > > +                             char           d_name[];
> > > +                     } *de = ptr;
> > 
> > what is the need for having your own linux_dirent64?
> 
> fdopendir() takes ownership of the fd, preventing reuse. And
> fdopendir(dup()) is getting ridiculous.

why not using dirent64?

Andi
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx



[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux