On Sunday 10 November 2013 16:52:04 you wrote: > Le 10/11/2013 14:56, Tito a écrit : > > On Sunday 10 November 2013 12:57:41 souf wrote: > >> Hi, > >> Please apply this patch to fstrim applet > >> > >> Thanks. > >> > >> > > > > Hi, > > this patch looks wrong to me: > > > > --- busybox/util-linux/fstrim.c.orig 2013-11-10 11:33:33.966789413 +0100
snip > > > > Ciao, > > Tito > from man fstrim: > > "The mountpoint argument is the pathname of the directory where the > filesystem is mounted." > > from fstrim (util-linux-2.24): > > path = argv[optind++]; > > if (optind != argc) { > warnx(_("unexpected number of arguments")); > usage(stderr); > } > > if (stat(path, &sb) == -1) > err(EXIT_FAILURE, _("stat failed %s"), path); > if (!S_ISDIR(sb.st_mode)) > errx(EXIT_FAILURE, _("%s: not a directory"), path); > > fd = open(path, O_RDONLY); > if (fd < 0) > err(EXIT_FAILURE, _("cannot open %s"), path); > > if (ioctl(fd, FITRIM, &range)) > err(EXIT_FAILURE, _("%s: FITRIM ioctl failed"), path); > > Hi, yes you are right, should have done my research before hitting send. Sorry for the noise. Ciao, Tito _______________________________________________ busybox mailing list busybox@busybox.net http://lists.busybox.net/mailman/listinfo/busybox