public inbox for util-linux@vger.kernel.org
 help / color / mirror / Atom feed
* sfdisk 2.20.1 --no-reread do BLKRRPART
@ 2012-02-16 20:16 Bertrand Jacquin
  2013-05-20 11:15 ` Karel Zak
  0 siblings, 1 reply; 3+ messages in thread
From: Bertrand Jacquin @ 2012-02-16 20:16 UTC (permalink / raw)
  To: util-linux

[-- Attachment #1: Type: text/plain, Size: 2026 bytes --]

Hi,

When creating partitions on a destination that is not a block device,
sfdisk do a BLKRRPART ioctl even when --no-reread option is used, but
looking at documentation --no-reread only apply to the beginning
process.

Would it be fine to extend that to the complete process ?

Bertrand

  dd if=/dev/zero of=/tmp/disk.img bs=1024 count=4032
  strace -fF -s 1024 -e trace=ioctl \
    sfdisk --no-reread -C2 -H32 -S63 -q -f \
      /tmp/disk.img << EOF
unit: sectors

part1 : start= 63, size= 3969, Id=83
part2 : start= 4032, size= 98784, Id=83
part3 : start= 102816, size= 22176, Id=83
part4 : start= 124992, size= 0, Id=83
EOF

ioctl(0, SNDCTL_TMR_TIMEBASE or SNDRV_TIMER_IOCTL_NEXT_DEVICE or TCGETS, 0x7fff7b917ce0) = -1 ENOTTY (Inappropriate ioctl for device)
Warning: /tmp/disk.img is not a block device
ioctl(3, HDIO_GETGEO, 0x7fff7b917c10)   = -1 ENOTTY (Inappropriate ioctl for device)
Disk /tmp/disk.img: cannot get geometry
ioctl(3, BLKGETSIZE64, 0x7fff7b917be0)  = -1 ENOTTY (Inappropriate ioctl for device)
ioctl(3, BLKGETSIZE, 0x7fff7b917b18)    = -1 ENOTTY (Inappropriate ioctl for device)
ioctl(3, FDGETPRM, 0x7fff7b917af0)      = -1 ENOTTY (Inappropriate ioctl for device)

Disk /tmp/disk.img: 2 cylinders, 32 heads, 63 sectors/track

sfdisk: ERROR: sector 0 does not have an msdos signature
 /tmp/disk.img: unrecognized partition table type
Old situation:
No partitions found
New situation:
Units = sectors of 512 bytes, counting from 0

   Device Boot    Start       End   #sectors  Id  System
/tmp/disk.img1            63      4031       3969  83  Linux
/tmp/disk.img2          4032    102815      98784  83  Linux
/tmp/disk.img3        102816    124991      22176  83  Linux
/tmp/disk.img4        124992    124991          0  83  Linux
Successfully wrote the new partition table

Re-reading the partition table ...
ioctl(3, BLKRRPART, 0)                  = -1 ENOTTY (Inappropriate ioctl for device)
BLKRRPART: Inappropriate ioctl for device

-- 
Beber

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: sfdisk 2.20.1 --no-reread do BLKRRPART
  2012-02-16 20:16 sfdisk 2.20.1 --no-reread do BLKRRPART Bertrand Jacquin
@ 2013-05-20 11:15 ` Karel Zak
  2013-05-21 13:35   ` Bertrand Jacquin
  0 siblings, 1 reply; 3+ messages in thread
From: Karel Zak @ 2013-05-20 11:15 UTC (permalink / raw)
  To: Bertrand Jacquin; +Cc: util-linux


 Sorry for the delay :-)

On Thu, Feb 16, 2012 at 09:16:18PM +0100, Bertrand Jacquin wrote:
> When creating partitions on a destination that is not a block device,
> sfdisk do a BLKRRPART ioctl even when --no-reread option is used, but
> looking at documentation --no-reread only apply to the beginning
> process.
> 
> Would it be fine to extend that to the complete process ?

 Not sure, but I think it should be enough to don't call BLKRRPART
 ioctl for non-block devices.

>   dd if=/dev/zero of=/tmp/disk.img bs=1024 count=4032
>   strace -fF -s 1024 -e trace=ioctl \
>     sfdisk --no-reread -C2 -H32 -S63 -q -f \
>       /tmp/disk.img << EOF
> unit: sectors
> 
> part1 : start= 63, size= 3969, Id=83
> part2 : start= 4032, size= 98784, Id=83
> part3 : start= 102816, size= 22176, Id=83
> part4 : start= 124992, size= 0, Id=83
> EOF

...

> Re-reading the partition table ...
> ioctl(3, BLKRRPART, 0)                  = -1 ENOTTY (Inappropriate ioctl for device)
> BLKRRPART: Inappropriate ioctl for device

 This use case (disk images partitioning) should be fixed in the git
 tree.

 Thanks!

    Karel

-- 
 Karel Zak  <kzak@redhat.com>
 http://karelzak.blogspot.com

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: sfdisk 2.20.1 --no-reread do BLKRRPART
  2013-05-20 11:15 ` Karel Zak
@ 2013-05-21 13:35   ` Bertrand Jacquin
  0 siblings, 0 replies; 3+ messages in thread
From: Bertrand Jacquin @ 2013-05-21 13:35 UTC (permalink / raw)
  To: Karel Zak; +Cc: util-linux

[-- Attachment #1: Type: text/plain, Size: 1598 bytes --]

Hi,

Le lundi 20 mai 2013 à 13h15, « Karel Zak » a écrit :
> 
>  Sorry for the delay :-)

No rush at all ! This was only a report about some ugly report in some
scripts :)

> On Thu, Feb 16, 2012 at 09:16:18PM +0100, Bertrand Jacquin wrote:
> > When creating partitions on a destination that is not a block device,
> > sfdisk do a BLKRRPART ioctl even when --no-reread option is used, but
> > looking at documentation --no-reread only apply to the beginning
> > process.
> > 
> > Would it be fine to extend that to the complete process ?
> 
>  Not sure, but I think it should be enough to don't call BLKRRPART
>  ioctl for non-block devices.
> 
> >   dd if=/dev/zero of=/tmp/disk.img bs=1024 count=4032
> >   strace -fF -s 1024 -e trace=ioctl \
> >     sfdisk --no-reread -C2 -H32 -S63 -q -f \
> >       /tmp/disk.img << EOF
> > unit: sectors
> > 
> > part1 : start= 63, size= 3969, Id=83
> > part2 : start= 4032, size= 98784, Id=83
> > part3 : start= 102816, size= 22176, Id=83
> > part4 : start= 124992, size= 0, Id=83
> > EOF
> 
> ...
> 
> > Re-reading the partition table ...
> > ioctl(3, BLKRRPART, 0)                  = -1 ENOTTY (Inappropriate ioctl for device)
> > BLKRRPART: Inappropriate ioctl for device
> 
>  This use case (disk images partitioning) should be fixed in the git
>  tree.

Seen and approved ! Thanks Karel

-- 
Bertrand Jacquin, EXOSEC (http://www.exosec.fr/)
ZAC des Metz - 3 Rue du petit robinson - 78350 JOUY EN JOSAS
Tel: +33 1 30 67 60 65  -  Fax: +33 1 75 43 40 70
GSM: +33 6 71 01 70 30  -  mailto:bjacquin@exosec.fr

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-05-21 13:41 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-02-16 20:16 sfdisk 2.20.1 --no-reread do BLKRRPART Bertrand Jacquin
2013-05-20 11:15 ` Karel Zak
2013-05-21 13:35   ` Bertrand Jacquin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox