Re: [LEDE-DEV] [PATCH mountd 2/2] support for disk without partition table
On 23/06/2016 10:17, Olivier Hardouin wrote: > I have a USB flash drive fat32 formatted without any partition table. > I agree this is unusual nowadays (manufacturers usually put a > partition table on their devices), but as it mounts ok on my laptop, I > wanted the same behavior. Hi, that is called superfloppy iirc. not seen those in a while. i've pushed the changes to the mountd repo John > > > On Thu, Jun 23, 2016 at 9:24 AM, John Crispin wrote: >> >> >> On 22/06/2016 09:51, olivier.hardo...@gmail.com wrote: >>> if no partition found, try to mount the block device itself >>> >>> Signed-off-by: Olivier Hardouin >> >> Hi, >> >> looks ok, just wondering what kind of storage you see this on. i have >> only seen superfloppy type formating on mmc cards and that is almost a >> decade ago. >> >> John >> >>> --- >>> mount.c | 14 ++ >>> 1 file changed, 10 insertions(+), 4 deletions(-) >>> >>> diff --git a/mount.c b/mount.c >>> index 0c2862a..7cbb8ff 100644 >>> --- a/mount.c >>> +++ b/mount.c >>> @@ -693,13 +693,19 @@ static void mount_enum_drives(void) >>> char tmp[64]; >>> snprintf(tmp, 64, "/sys/block/%s/", >>> namelist[n]->d_name); >>> m = scandir(tmp, &namelist2, dir_filter2, >>> dir_sort); >>> - while(m--) >>> + if(m > 0) >>> { >>> - strncpy(&block[blk_cnt][0], >>> namelist2[m]->d_name, MAX_BLOCK); >>> + while(m--) >>> + { >>> + strncpy(&block[blk_cnt][0], >>> namelist2[m]->d_name, MAX_BLOCK); >>> + blk_cnt++; >>> + free(namelist2[m]); >>> + } >>> + free(namelist2); >>> + } else { >>> + strncpy(&block[blk_cnt][0], >>> namelist[n]->d_name, MAX_BLOCK); >>> blk_cnt++; >>> - free(namelist2[m]); >>> } >>> - free(namelist2); >>> } >>> free(namelist[n]); >>> } >>> > > ___ > Lede-dev mailing list > Lede-dev@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/lede-dev > ___ Lede-dev mailing list Lede-dev@lists.infradead.org http://lists.infradead.org/mailman/listinfo/lede-dev
Re: [LEDE-DEV] [PATCH mountd 2/2] support for disk without partition table
I have a USB flash drive fat32 formatted without any partition table. I agree this is unusual nowadays (manufacturers usually put a partition table on their devices), but as it mounts ok on my laptop, I wanted the same behavior. On Thu, Jun 23, 2016 at 9:24 AM, John Crispin wrote: > > > On 22/06/2016 09:51, olivier.hardo...@gmail.com wrote: >> if no partition found, try to mount the block device itself >> >> Signed-off-by: Olivier Hardouin > > Hi, > > looks ok, just wondering what kind of storage you see this on. i have > only seen superfloppy type formating on mmc cards and that is almost a > decade ago. > > John > >> --- >> mount.c | 14 ++ >> 1 file changed, 10 insertions(+), 4 deletions(-) >> >> diff --git a/mount.c b/mount.c >> index 0c2862a..7cbb8ff 100644 >> --- a/mount.c >> +++ b/mount.c >> @@ -693,13 +693,19 @@ static void mount_enum_drives(void) >> char tmp[64]; >> snprintf(tmp, 64, "/sys/block/%s/", >> namelist[n]->d_name); >> m = scandir(tmp, &namelist2, dir_filter2, >> dir_sort); >> - while(m--) >> + if(m > 0) >> { >> - strncpy(&block[blk_cnt][0], >> namelist2[m]->d_name, MAX_BLOCK); >> + while(m--) >> + { >> + strncpy(&block[blk_cnt][0], >> namelist2[m]->d_name, MAX_BLOCK); >> + blk_cnt++; >> + free(namelist2[m]); >> + } >> + free(namelist2); >> + } else { >> + strncpy(&block[blk_cnt][0], >> namelist[n]->d_name, MAX_BLOCK); >> blk_cnt++; >> - free(namelist2[m]); >> } >> - free(namelist2); >> } >> free(namelist[n]); >> } >> ___ Lede-dev mailing list Lede-dev@lists.infradead.org http://lists.infradead.org/mailman/listinfo/lede-dev
Re: [LEDE-DEV] [PATCH mountd 2/2] support for disk without partition table
On 22/06/2016 09:51, olivier.hardo...@gmail.com wrote: > if no partition found, try to mount the block device itself > > Signed-off-by: Olivier Hardouin Hi, looks ok, just wondering what kind of storage you see this on. i have only seen superfloppy type formating on mmc cards and that is almost a decade ago. John > --- > mount.c | 14 ++ > 1 file changed, 10 insertions(+), 4 deletions(-) > > diff --git a/mount.c b/mount.c > index 0c2862a..7cbb8ff 100644 > --- a/mount.c > +++ b/mount.c > @@ -693,13 +693,19 @@ static void mount_enum_drives(void) > char tmp[64]; > snprintf(tmp, 64, "/sys/block/%s/", > namelist[n]->d_name); > m = scandir(tmp, &namelist2, dir_filter2, > dir_sort); > - while(m--) > + if(m > 0) > { > - strncpy(&block[blk_cnt][0], > namelist2[m]->d_name, MAX_BLOCK); > + while(m--) > + { > + strncpy(&block[blk_cnt][0], > namelist2[m]->d_name, MAX_BLOCK); > + blk_cnt++; > + free(namelist2[m]); > + } > + free(namelist2); > + } else { > + strncpy(&block[blk_cnt][0], > namelist[n]->d_name, MAX_BLOCK); > blk_cnt++; > - free(namelist2[m]); > } > - free(namelist2); > } > free(namelist[n]); > } > ___ Lede-dev mailing list Lede-dev@lists.infradead.org http://lists.infradead.org/mailman/listinfo/lede-dev